[coreboot-gerrit] Change in coreboot[master]: mb/google/brya/var/redrix: Configure _DSC for CAM devices to ACPI_DEV...

[Tim Wawrzynczak (Code Review)]

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

Change subject: mb/google/brya/var/redrix: Configure _DSC for CAM devices to ACPI_DEVICE_SLEEP_D3_COLD
......................................................................

Patch Set 4:

(1 comment)

File src/mainboard/google/brya/variants/redrix/overridetree.cb:

https://review.coreboot.org/c/coreboot/+/59260/comment/567632ad_a46d09f6 
PS3, Line 355: GPP_D16
> I am a bit confused, which other place are we exposing D16 to driver ?

See line 324, which generates a separate power resource. We have reference counting built in to the driver to accomodate the fact that multiple PRs will each reference the same GPIOs.