port superiotool to FreeBSD

[Andriy Gapon]

allow superiotool to compile and work on FreeBSD
using the same approach as earlier flashrom port

Signed-off-by: Andriy Gapon

---

This is tested on FreeBSD 7, dump for W83977EF/EG that I sent previously
was obtained using this patch.

It seems that this approach can also be extended to all coreboot (and
utils) code that uses inX/outX (it is quite unfortunate that Linux and
BSD use different order of parameters for outX).

[Idwer Vollering]

2008/10/28, Andriy Gapon :
> allow superiotool to compile and work on FreeBSD
> using the same approach as earlier flashrom port
>
> Signed-off-by: Andriy Gapon
>
> This is tested on FreeBSD 7, dump for W83977EF/EG that I sent previously
> was obtained using this patch.
>
> It seems that this approach can also be extended to all coreboot (and
> utils) code that uses inX/outX (it is quite unfortunate that Linux and
> BSD use different order of parameters for outX).
>
> Andriy Gapon

Great ! However, you overlooked outb at lines 471 and 475 in ite.c, see my
patch.

Idwer

[Andriy Gapon]

on 28/10/2008 17:56 Idwer Vollering said the following:
> 2008/10/28, Andriy Gapon <[e-mail] >:
> allow superiotool to compile and work on FreeBSD
> using the same approach as earlier flashrom port
>
> Signed-off-by: Andriy Gapon <[e-mail] <mailto:[e-mail]>>
>
> ---
>
> This is tested on FreeBSD 7, dump for W83977EF/EG that I sent previously
> was obtained using this patch.
>
> It seems that this approach can also be extended to all coreboot (and
> utils) code that uses inX/outX (it is quite unfortunate that Linux and
> BSD use different order of parameters for outX).
>
>
> --
>
> Andriy Gapon
>
> --
> coreboot mailing list: [e-mail]
> <mailto:[e-mail]>
> http://www.coreboot.org/mailman/listinfo/coreboot
>
>
> Great ! However, you overlooked outb at lines 471 and 475 in ite.c, see
> my patch.
>
> Idwer

Oops and thanks!
I did 'svn update' but forgot to 'gmake' after that.

[Carl-Daniel Hailfinger]

On 28.10.2008 16:56, Idwer Vollering wrote:
> 2008/10/28, Andriy Gapon :
> > allow superiotool to compile and work on FreeBSD
> > using the same approach as earlier flashrom port
> >
> > Signed-off-by: Andriy Gapon
> >
> > This is tested on FreeBSD 7, dump for W83977EF/EG that I sent previously
> > was obtained using this patch.
> >
> > It seems that this approach can also be extended to all coreboot (and
> > utils) code that uses inX/outX (it is quite unfortunate that Linux and
> > BSD use different order of parameters for outX).
> >
> > Andriy Gapon
>
> Great ! However, you overlooked outb at lines 471 and 475 in ite.c, see my
> patch.
>
> Idwer

With a signoff, you can take my
Acked-by: Carl-Daniel Hailfinger

Regards,
Carl-Daniel

[Uwe Hermann]

On Tue, Oct 28, 2008 at 10:19:16PM +0100, Carl-Daniel Hailfinger wrote:
> > > allow superiotool to compile and work on FreeBSD
> > > using the same approach as earlier flashrom port
> > >
> > > Signed-off-by: Andriy Gapon
> > Great ! However, you overlooked outb at lines 471 and 475 in ite.c, see my
> > patch.
> With a signoff, you can take my
> Acked-by: Carl-Daniel Hailfinger

Committed in r3698 with the missing outb's converted, and I also changed
some '#ifdef's to '#if defined(FOO)' for consistency as per IRC discussion.

Uwe.