[coreboot-gerrit] Change in coreboot[master]: mb/google/volteer: Separate power resource for VCM

[Tim Wawrzynczak (Code Review)]

Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45159 )

Change subject: mb/google/volteer: Separate power resource for VCM
......................................................................

Patch Set 3:

(1 comment)

https://review.coreboot.org/c/coreboot/+/45159/3/src/mainboard/google/voltee... 
File src/mainboard/google/volteer/variants/baseboard/include/baseboard/acpi/mipi_camera.asl:

https://review.coreboot.org/c/coreboot/+/45159/3/src/mainboard/google/voltee... 
PS3, Line 345:                 Method (_ON, 0, Serialized)  /* VCPR_ON_: VCM Power On */
> What happens if one does the following sequence: […]

Is that something the kernel might try to do? If so, then no, this won't work.
If that needs to happen, one way I can think of is to have both power resources utilize the same 'STA' Name; if we do that, then the sequence you state above should work, because STA will still be 1 after #3, and so the sensor & the VCM would still remain powered.