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

[Tomasz Figa (Code Review)]

Tomasz Figa 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 4:

(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 */
> Gotcha. […]

I just checked the VCM datasheet and I'd have to check the board and module schematics (would be nice if someone could point me to the right one for this board) for completeness, and AFAICT this VCM has neither any clock input nor any reset/power-on pins, so it should be enough to just control the power rails.

Otherwise the idea sounds good to me. Thanks.