[patch] fintek f71859 sio

[Marc Jones]

I couldn't find any docs, but this gets the serial port to work.

Signed-off-by: Marc Jones

Marc

[Uwe Hermann]

Hi,

On Thu, Sep 09, 2010 at 09:49:38PM -0600, Marc Jones wrote:
> Index: src/superio/fintek/Kconfig
>
> --- src/superio/fintek/Kconfig	(revision 5799)
> +++ src/superio/fintek/Kconfig	(working copy)
> @@ -2,3 +2,5 @@
>    bool
>  config SUPERIO_FINTEK_F71863FG
>    bool
> +config SUPERIO_FINTEK_F71859
> bool
> Index: src/superio/fintek/f71859/f71859_early_serial.c
>
> --- src/superio/fintek/f71859/f71859_early_serial.c	(revision 0)
> +++ src/superio/fintek/f71859/f71859_early_serial.c	(revision 0)
> @@ -0,0 +1,46 @@
> +/*
> This file is part of the coreboot project.
> Copyright (C) 2007 Corey Osgood

You can make all those files
 * Copyright (C) 2010 Marc Jones 
as each of them is trivial and cannot really be written differently.
> +/* Pre-RAM driver for the Fintek F71805F/FG Super I/O chip. */

^^^^^^^^^^
                                      F71859

Does your chip have an "F" suffix in the name as the above ones? If yes,
please add it to the name in the code/comments and file/directory names.
> Property changes on: src/superio/fintek/f71859/f71859_early_serial.c
> ___________________________________________________________________
> Added: svn:executable

Please drop the executable property from all files, it's incorrect and
not needed.
> +/* This chip doesn't have keyboard and mouse support. */

Should probably be dropped unless you are sure the chip doesn't have
keyboard/mouse support.
> +/*
> Datasheet:
> */

Can be dropped.

With the above changes:
Signed-off-by: Uwe Hermann

Uwe.

[Marc Jones]

On Fri, Sep 10, 2010 at 1:26 AM, Uwe Hermann  wrote:
> Hi,
>
> On Thu, Sep 09, 2010 at 09:49:38PM -0600, Marc Jones wrote:
> > Index: src/superio/fintek/Kconfig
> >
> > --- src/superio/fintek/Kconfig        (revision 5799)
> > +++ src/superio/fintek/Kconfig        (working copy)
> > @@ -2,3 +2,5 @@
> >       bool
> >  config SUPERIO_FINTEK_F71863FG
> >       bool
> > +config SUPERIO_FINTEK_F71859
> > bool
> > Index: src/superio/fintek/f71859/f71859_early_serial.c
> >
> > --- src/superio/fintek/f71859/f71859_early_serial.c   (revision 0)
> > +++ src/superio/fintek/f71859/f71859_early_serial.c   (revision 0)
> > @@ -0,0 +1,46 @@
> > +/*
> > This file is part of the coreboot project.
> > Copyright (C) 2007 Corey Osgood
>
> You can make all those files
>  * Copyright (C) 2010 Marc Jones 
> as each of them is trivial and cannot really be written differently.
> > +/* Pre-RAM driver for the Fintek F71805F/FG Super I/O chip. */
>
> ^^^^^^^^^^
>                                      F71859
>
> Does your chip have an "F" suffix in the name as the above ones? If yes,
> please add it to the name in the code/comments and file/directory names.
> > Property changes on: src/superio/fintek/f71859/f71859_early_serial.c
> > ___________________________________________________________________
> > Added: svn:executable
> >    + *
>
> Please drop the executable property from all files, it's incorrect and
> not needed.
> > +/* This chip doesn't have keyboard and mouse support. */
>
> Should probably be dropped unless you are sure the chip doesn't have
> keyboard/mouse support.
> > +/*
> > Datasheet:
> > */
>
> Can be dropped.
>
> With the above changes:
> Signed-off-by: Uwe Hermann

I assume that you mean acked-by.

r 5802

Thanks!

Marc

[Uwe Hermann]

On Fri, Sep 10, 2010 at 04:15:25PM -0600, Marc Jones wrote:
> > With the above changes:
> > Signed-off-by: Uwe Hermann
>
> I assume that you mean acked-by.
>
> r 5802

Er, yes, sorry. Pressed the wrong button. I have Signed-off-by and
Acked-by lines being inserted upon pressing F4/F5 in vim.

Uwe.