[LinuxBIOS] superiotool - dump support for LPC47M112

[Robinson Tryon]

Signed-off-by: Robinson P. Tryon

This chip shares the same id as the LPC47M10x line, and from
comparison, appears to have the same set of registers.

In general, can I assume that two chips with the same id number will
have the exact same register layout?

Also, what are "Ultra I/Os", and should superiotool support dumps from 'em?

[Peter Stuge]

On Sat, Oct 06, 2007 at 10:26:16PM -0400, Robinson Tryon wrote:
> Signed-off-by: Robinson P. Tryon

Acked-by: Peter Stuge
> In general, can I assume that two chips with the same id number
> will have the exact same register layout?

No, I don't think that is safe.
> Also, what are "Ultra I/Os", and should superiotool support dumps
> from 'em?

Sorry, no idea.

//Peter

[Uwe Hermann]

Hi,

while the patch is correct I won't apply it for now. I have a much
bigger patch in the works which does what your patch does, and adds
a lots more other (duplicate) IDs for chips with different name...

On Sun, Oct 07, 2007 at 01:02:10PM +0200, Peter Stuge wrote:
> > In general, can I assume that two chips with the same id number
> > will have the exact same register layout?
>
> No, I don't think that is safe.

True.

There are several chips with different names and same IDs.
> > Also, what are "Ultra I/Os", and should superiotool support dumps
> > from 'em?
>
> Sorry, no idea.

Can you name an example? I think that's just another name for Super I/O,
so yes, I think we should support them.

Uwe.

[Robinson Tryon]

On 10/7/07, Uwe Hermann  wrote:
> Hi,
>
> while the patch is correct I won't apply it for now. I have a much
> bigger patch in the works which does what your patch does, and adds
> a lots more other (duplicate) IDs for chips with different name...

sure.
> On Sun, Oct 07, 2007 at 01:02:10PM +0200, Peter Stuge wrote:
> > > In general, can I assume that two chips with the same id number
> > > will have the exact same register layout?
> >
> > No, I don't think that is safe.
>
> True.
>
> There are several chips with different names and same IDs.

Fair enough.  Now that being said, how should I enter data for two
chips into the array?  Should it just look like the following?

{0x50, "First chip name"
    ...}
{0x50, "Second chip name"
    ...}
> > > Also, what are "Ultra I/Os", and should superiotool support dumps
> > > from 'em?
> >
> > Sorry, no idea.
>
> Can you name an example? I think that's just another name for Super I/O,
> so yes, I think we should support them.

Sure -- the SMSC datasheet website lists several Super I/Os and at
least one "Ultra I/O":

http://www.smsc.com/main/datasheet.html

"FDC37C93x  	FDC37C93x Plug-and-Play Compatible Ultra I/O Controller
    639k  	3/21/07"

[Uwe Hermann]

On Sun, Oct 07, 2007 at 05:22:12PM -0400, Robinson Tryon wrote:
> > There are several chips with different names and same IDs.
>
> Fair enough.  Now that being said, how should I enter data for two
> chips into the array?  Should it just look like the following?
>
> {0x50, "First chip name"
>     ...}
> {0x50, "Second chip name"
>     ...}

Nope, one entry per ID only. Put the different names in the string.

Example:
        {0x4103, "F71872F/FG / F71806F/FG", {
> Sure -- the SMSC datasheet website lists several Super I/Os and at
> least one "Ultra I/O":
>
> http://www.smsc.com/main/datasheet.html
>
> "FDC37C93x  	FDC37C93x Plug-and-Play Compatible Ultra I/O Controller
>    639k  	3/21/07"

OK, yes. We want to support them, it's just another name for Super I/O.
I've also seen "Server I/O" or "Advanced I/O" and some other names around...

Uwe.

[Robinson Tryon]

On 10/7/07, Uwe Hermann  wrote:
> On Sun, Oct 07, 2007 at 05:22:12PM -0400, Robinson Tryon wrote:
> > > There are several chips with different names and same IDs.
> >
> > Fair enough.  Now that being said, how should I enter data for two
> > chips into the array?  Should it just look like the following?
> >
> > {0x50, "First chip name"
> >     ...}
> > {0x50, "Second chip name"
> >     ...}
>
> Nope, one entry per ID only. Put the different names in the string.
>
> Example:
>         {0x4103, "F71872F/FG / F71806F/FG", {

If there are two chips that have different register layouts, but which
share the same ID, then by putting both names into the string I will
be (falsely) indicating that both chips share the same register
layout.  Right?

[Uwe Hermann]

On Sun, Oct 07, 2007 at 06:35:43PM -0400, Robinson Tryon wrote:
> > Nope, one entry per ID only. Put the different names in the string.
> >
> > Example:
> >         {0x4103, "F71872F/FG / F71806F/FG", {
>
> If there are two chips that have different register layouts, but which
> share the same ID, then by putting both names into the string I will
> be (falsely) indicating that both chips share the same register
> layout.  Right?

Yes.

If you find such chips, please let us know. If they have the same ID but
are not compatible that's a vendor bug, IMO.

Uwe.