[coreboot-gerrit] Change in coreboot[master]: soc/intel/{cnl,icl}: Move globalnvs.asl/nvs.h into common/block/

[Furquan Shaikh (Code Review)]

Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36457 )

Change subject: soc/intel/{cnl,icl}: Move globalnvs.asl/nvs.h into common/block/
......................................................................

Patch Set 6:

(1 comment)

https://review.coreboot.org/c/coreboot/+/36457/6/src/soc/intel/cannonlake/in... 
File src/soc/intel/cannonlake/include/soc/nvs.h:

PS6:
> If NVS is moved completely to common code, why retain this placeholder file? Isn't it better to just […]

I see a comment on earlier patchset related to this. I think we should get rid of this file in the soc/ paths and mainboards for those SoC which are already moved to use common code.