Hello Aaron Durbin, Patrick Rudolph, Julius Werner, Subrata Banik, Duncan Laurie, Paul Menzel, build bot (Jenkins), Nico Huber, Furquan Shaikh, Patrick Georgi, Martin Roth,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/35073
to look at the new patch set (#5).
Change subject: drivers/generic: Add new driver for power resource pseudo-device ......................................................................
drivers/generic: Add new driver for power resource pseudo-device
There is some duplication of fields dealing with ACPI power resources (see drivers/i2c/generic.c, also used by drivers/i2c/hid, as well as drivers/spi/acpi). This CL creates a new driver for this pseudo-device. It will also enable the sharing of power resources between multiple devices in the future, if they share power-related signals.
This would change the (effective) ASL from something like this:
Scope (_SB.PCI0) { Device (I2C0) { ... PowerResource (PRIC, 0, 0) { Method (_ON) ... Method (_OFF) ... } Name (_PR0, Package (1) { PRIC } ) Name (_PR3, Package (1) { PRIC } ) } }
to something more like this:
Scope (_SB.PCI0.I2C0) { PowerResource (PR00, 0, 0) { Method (_ON)... Method (_OFF)... } }
Scope (_SB.PCI0) { Device (I2C0) { ... Name (_PR0, Package (1) { PR00 } ) Name (_PR3, Package (1) { PR00 } ) } }
Change-Id: Id9d66fedd342eb5359d90d9778c515e7aa2de1d7 Signed-off-by: Tim Wawrzynczak twawrzynczak@chromium.org --- A src/drivers/generic/power_resource/Kconfig A src/drivers/generic/power_resource/Makefile.inc A src/drivers/generic/power_resource/chip.h A src/drivers/generic/power_resource/power_resource.c 4 files changed, 128 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/73/35073/5