Re: [coreboot] patch tosuperiotool - add LPC47N252

[Uwe Hermann]

On Fri, Jun 26, 2009 at 11:25:50PM +0200, CybFr wrote:
> Signed-off-by: François-Regis Vuillemin <[e-mail]

Thanks, committed in r4375.
> {0x0e, "LPC47N252", {   /* From sensors-detect */
> {NOLDN, NULL,
> 	{0x02,0x20,0x21,0x22,0x23,0x24,0x25,0x26,0x27,0x28,
> 	 0x29,0x2a,0x2b,0x2c,0x2d,0x2e,0x2f,EOT},
> 	{0x00,0x0e,0x00,0x00,0x00,0x04,0x04,NANA,NANA,0x00,
> 	 0x00,0x00,0x00,0x00,0x00,0x00,0x00,EOT}},

I dropped 0x02, that's a write-only register.
> {0x7, "Keyboard",
> 	{0x30,0x60,0x61,0x70,0x72,0xf0,EOT},
> 	{0x00,0x00,0x00,0x00,0x00,0x00,EOT}},

Can you post a 'superiotool -d' output from your laptop please? The
datasheet doesn't seem to mention 0x60/0x61 for this LDN (says the
address is fixed and not relocatable). Not sure if we can/should dump
0x60/0x61 here.

Uwe.