[LinuxBIOS] [PATCH] Add includes to it8712f/superio.c
jon.dufresne at gmail.com
Sun Nov 26 20:24:00 CET 2006
Using the available superio I have seen early serial output with an
ite it8712f chip. It is the only chip I have and therefore can
confirm. As I get more to devices to work I'll let you know.
I have attached a diff fixing all ite includes. Do you normally prefer
an attachment or that I paste the diff in the email? You mentioned
adding includes to chip.h, however this file did not need
modifications for me and after looking at it, I don't think it needs
additional includes for the code contained in it.
On 11/26/06, Uwe Hermann <uwe at hermann-uwe.de> wrote:
> sorry for the delay.
> On Mon, Nov 20, 2006 at 05:17:25PM -0500, Jon Dufresne wrote:
> > Added two includes to it8712f/superio.c, the file will not compile without
> > them
> > Signed-off-by: Jon Dufresne <jon.dufresne at gmail.com>
> > ---
> > Index: src/superio/ite/it8712f/superio.c
> > ===================================================================
> > --- src/superio/ite/it8712f/superio.c (revision 2498)
> > +++ src/superio/ite/it8712f/superio.c (working copy)
> > @@ -18,6 +18,8 @@
> > * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
> > */
> > +#include <device/device.h>
> > +#include <device/pnp.h>
> > #include <uart8250.h>
> > #include <pc80/keyboard.h>
> > #include "chip.h"
> All other ITE Super I/Os will have the same problem, I guess. I probably
> didn't notice this as the #includes are drawn in in some other file,
> too, on the mainboard code I used.
> Can you crate a patch which fixes all of the ITEs, adding all required
> #includes to all files (mostly superio.c and chip.h)?
> Please also note, that I didn't test any of the ITE chips other than
> IT8671F, so they may or may not work. If you could test serial output, that
> would be very nice. Anything other than serial output (floppy, keyboard,
> whatever) I haven't tested _at all_, and partly there's not even any
> code written for that, yet (but it's not too hard)...
> http://www.hermann-uwe.de | http://www.holsham-traders.de
> http://www.crazy-hacks.org | http://www.unmaintained-free-software.org
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.5 (GNU/Linux)
> -----END PGP SIGNATURE-----
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 3064 bytes
Desc: not available
More information about the coreboot