[coreboot-gerrit] Change in ...coreboot[master]: WIP: Power resource stub

[Tim Wawrzynczak (Code Review)]

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

Change subject: WIP: Power resource stub
......................................................................

Patch Set 2:
> Patch Set 2:
> > Patch Set 2:
> >
> > (21 comments)
> >
> > This CL should be broken down into multiple smaller CLs:
> > Adding probed property to sconfig to support probed devices
> > Adding power resource device to device tree
> > Adding power resource driver for ACPI
> > Adding helper function for enabling power in bootblock
> > Moving all mainboards using power resource to the newly added device
> > Get rid of power resource properties from I2C generic and SPI devices.
> > Get rid of probed property
>
> If power is enabled in bootblock, and probing is done in ramstage, when is the appropriate time to deassert reset?

So you think it would be appropriate to assert enable GPIOs first, then sleep() until the max() of the enable times has elapsed, and then deassert resets?