On Thu, Jun 17, 2010 at 2:04 AM, Peter Stuge <[email protected]> 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 <[email protected]>
>
> Acked-by: Peter Stuge <[email protected]>
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

-- 
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to