[coreboot-gerrit] Change in coreboot[master]: mb/google/dedede: Replace static Camera ACPI by driver for WDoo

[Sugnan Prabhu S (Code Review)]

Sugnan Prabhu S has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44397 )

Change subject: mb/google/dedede: Replace static Camera ACPI by driver for WDoo
......................................................................

Patch Set 9:

(1 comment)

https://review.coreboot.org/c/coreboot/+/44397/9/src/mainboard/google/dedede... 
File src/mainboard/google/dedede/variants/waddledoo/overridetree.cb:

https://review.coreboot.org/c/coreboot/+/44397/9/src/mainboard/google/dedede... 
PS9, Line 229: "GPP_D13" #power_enable_2p8
             : 				register "gpio_panel.gpio[1].gpio_num" = "GPP_D14"
> Yes, it should work exactly that way. […]

@Matt,
Every resource operation (GPIO/clock) is protected with a reference count variable, irrespective whether it is shared or not. The resource is actually disabled only if the reference count reaches zero.
Earlier we were following the approach what @Tim mentioned but it was causing issues and complications in handling scenarios where the resource is not properly marked as shared in the devicetree.