[coreboot] [PATCH] Simplify device enabling and initialization

Myles Watson mylesgw at gmail.com
Mon Jun 21 17:17:54 CEST 2010


On Mon, Jun 21, 2010 at 8:59 AM, Ward Vandewege <ward at gnu.org> wrote:
> On Mon, Jun 21, 2010 at 05:59:52AM -0600, Myles Watson wrote:
>> It looks like only 5632 has the "ADT7463 properly initialized"
>> message.  One problem is that patch 2 was meant to be applied after
>> patch 1,
>
> Ah, whoops, I was wondering if I was doing the right thing there...
I should have been clearer.

>> so the device didn't end up in the tree for that run.  I'll
>> have to think about why the only message from the new device with
>> patch 1 is "I2C: 00:d0 missing read_resources"  For some reason it
>> doesn't look like it got the correct ops.
>>
>> I wonder why the temperature values look right in all cases.  Does it
>> need to be cold booted in order for the initialization to be needed?
>
> All boots were cold.
OK.  Thanks for doing that.

I guess I don't know what I'm looking at in the sensors output to see
if the device initialization worked.

Thanks,
Myles




More information about the coreboot mailing list