[coreboot-gerrit] Change in ...coreboot[master]: mb/google/atlas: Enable camera module NVM

[JasonX Z Chen (Code Review)]

JasonX Z Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/30795 )

Change subject: mb/google/atlas: Enable camera module NVM
......................................................................

Patch Set 6:

(1 comment)

https://review.coreboot.org/#/c/30795/5/src/mainboard/google/poppy/variants/... 
File src/mainboard/google/poppy/variants/atlas/include/variant/acpi/cam0.asl:

https://review.coreboot.org/#/c/30795/5/src/mainboard/google/poppy/variants/... 
PS5, Line 37: CTXS (EN_CAM_PMIC_RST_L)
            : 				STXS (EN_PP3300_DX_CAM)
            : 				Sleep(3)
            : 				STXS (EN_CAM_PMIC_RST_L)
            : 				STXS (EN_CAM_CLOCK)
            : 				Sleep(2);
> the power resource is common for CAM0 and NVM0, do you need all these steps to just to activate NVM? […]

If remove STXS/CTXS of method(PMON), cpu can't get the nvm data anymore.