[coreboot] [PATCH] Simplify device enabling and initialization

Myles Watson mylesgw at gmail.com
Thu Jun 17 18:19:03 CEST 2010


On Thu, Jun 17, 2010 at 2:04 AM, Peter Stuge <peter at stuge.se> wrote:
> Myles Watson wrote:
>> If we initialize and enable devices in the order that they are found
>> in the tree, instead of the order that they were added to the list, it
>> simplifies the code.  It also makes it so that removing a device from
>> the devicetree.cb file won't change when its resources are enabled.
> ..
>> Signed-off-by: Myles Watson <mylesgw at gmail.com>
>
> Acked-by: Peter Stuge <peter at stuge.se>
Rev 5633.

I copied the s2881 code into a driver stub, since Ward said he's
willing to test.

>> This patch breaks the s2881, which was doing some odd acrobatics in
>> order to get a device initialized after its parent.
>
> If neccessary then I would be in favor of mainboard-specific hooks
> from coreboot, that are separate from the device tree. But maybe the
> init for s2881 can be done using a coreboot "driver" ?
That could work.  I think it might get abused, though.

Thanks,
Myles




More information about the coreboot mailing list