[coreboot] F71889 Super I/O patch

Uwe Hermann uwe at hermann-uwe.de
Wed Nov 3 22:52:50 CET 2010

On Wed, Nov 03, 2010 at 12:18:56PM -0700, Neo The User wrote:
> Signed-off-by: Alec Ari <neotheuser at ymail.com>

Thanks, committed in r6017 with a few changes:

 - Changed unsigned int etc. to u8/u16 etc. everywhere.

 - Add the missing LDNs as per datasheet to f71889.h and added superio.c
   code to handle the devices (at least partially).

 - Add KBC stuff, this Super I/O _does_ have a keyboard LDN.

 - Fixed up pnp_dev_info as good as possible, but it seems the datasheet
   is missing some info needed to asses where to use 0x7f8 or 0xff8 etc.

> > the code off the other 785 boards. If anybody wants to test
> > the F71889 patch, feel free.

Yup, but testing is not strictly required, I'm relatively sure the
code as committed should work (though there are some TODOs).

> > Also I wasn't sure to keep the same name in the other
> > Fintek chips or not, so I changed the name to mine, and the

Yep, that's fine for such simple pieces of code which cannot be written
much differently anyways. But please do keep the (C) lines intact
when deriving/copying larger and more complex pieces of code.

> > year. I apologize if that was incorrect, and will be more
> > than willing to keep the original GNU header.

There's no such thing as a "GNU header" btw. Every file has one or
more "(C) Copyright" line(s) which list the copyright holder(s) and
the resp. years, and a text-blob which describes the license that
applies to that file (in this case it happens to be the GNU GPL).

http://hermann-uwe.de     | http://sigrok.org
http://randomprojects.org | http://unmaintained-free-software.org

More information about the coreboot mailing list