On Wed, Nov 03, 2010 at 12:18:56PM -0700, Neo The User wrote: > Signed-off-by: Alec Ari <neotheu...@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). Uwe. -- http://hermann-uwe.de | http://sigrok.org http://randomprojects.org | http://unmaintained-free-software.org -- coreboot mailing list: coreboot@coreboot.org http://www.coreboot.org/mailman/listinfo/coreboot
