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

Signed-off-by: Andriy Gapon <[EMAIL PROTECTED]>

---

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
Index: winbond.c
===================================================================
--- winbond.c   (revision 3697)
+++ winbond.c   (working copy)
@@ -470,18 +470,18 @@
 
 static void enter_conf_mode_winbond_88(uint16_t port)
 {
-       outb(0x88, port);
+       OUTB(0x88, port);
 }
 
 static void enter_conf_mode_winbond_89(uint16_t port)
 {
-       outb(0x89, port);
+       OUTB(0x89, port);
 }
 
 static void enter_conf_mode_winbond_86(uint16_t port)
 {
-       outb(0x86, port);
-       outb(0x86, port);
+       OUTB(0x86, port);
+       OUTB(0x86, port);
 }
 
 static void probe_idregs_winbond_helper(const char *init, uint16_t port)
Index: ite.c
===================================================================
--- ite.c       (revision 3697)
+++ ite.c       (working copy)
@@ -482,10 +482,10 @@
  */
 static void enter_conf_mode_ite(uint16_t port)
 {
-       outb(0x87, port);
-       outb(0x01, port);
-       outb(0x55, port);
-       outb((port == 0x2e) ? 0x55 : 0xaa, port);
+       OUTB(0x87, port);
+       OUTB(0x01, port);
+       OUTB(0x55, port);
+       OUTB((port == 0x2e) ? 0x55 : 0xaa, port);
 }
 
 static void exit_conf_mode_ite(uint16_t port)
Index: nsc.c
===================================================================
--- nsc.c       (revision 3697)
+++ nsc.c       (working copy)
@@ -479,21 +479,21 @@
 
        probing_for("NSC", "", port);
 
-       outb(CHIP_ID_REG, port);
-       if (inb(port) != CHIP_ID_REG) {
+       OUTB(CHIP_ID_REG, port);
+       if (INB(port) != CHIP_ID_REG) {
                if (verbose)
                        printf(NOTFOUND "port=0x%02x, port+1=0x%02x\n",
-                              inb(port), inb(port + 1));
+                              INB(port), INB(port + 1));
                return;
        }
-       id = inb(port + 1);
+       id = INB(port + 1);
 
-       outb(CHIP_REV_REG, port);
-       if (inb(port) != CHIP_REV_REG) {
+       OUTB(CHIP_REV_REG, port);
+       if (INB(port) != CHIP_REV_REG) {
                printf("Warning: Can't get chip revision. Setting to 0xff.\n");
                rev = 0xff;
        } else {
-               rev = inb(port + 1);
+               rev = INB(port + 1);
        }
 
        if (superio_unknown(reg_table, id)) {
Index: superiotool.c
===================================================================
--- superiotool.c       (revision 3697)
+++ superiotool.c       (working copy)
@@ -23,6 +23,11 @@
 
 #include "superiotool.h"
 
+#if defined(__FreeBSD__)
+#include <fcntl.h>
+#include <unistd.h>
+#endif
+
 /* Command line options. */
 int dump = 0, verbose = 0, extra_dump = 0;
 
@@ -31,25 +36,25 @@
 
 uint8_t regval(uint16_t port, uint8_t reg)
 {
-       outb(reg, port);
-       return inb(port + ((port == 0x3bd) ? 2 : 1)); /* 0x3bd is special. */
+       OUTB(reg, port);
+       return INB(port + ((port == 0x3bd) ? 2 : 1)); /* 0x3bd is special. */
 }
 
 void regwrite(uint16_t port, uint8_t reg, uint8_t val)
 {
-       outb(reg, port);
-       outb(val, port + 1);
+       OUTB(reg, port);
+       OUTB(val, port + 1);
 }
 
 void enter_conf_mode_winbond_fintek_ite_8787(uint16_t port)
 {
-       outb(0x87, port);
-       outb(0x87, port);
+       OUTB(0x87, port);
+       OUTB(0x87, port);
 }
 
 void exit_conf_mode_winbond_fintek_ite_8787(uint16_t port)
 {
-       outb(0xaa, port);               /* Fintek, Winbond */
+       OUTB(0xaa, port);               /* Fintek, Winbond */
        regwrite(port, 0x02, 0x02);     /* ITE */
 }
 
@@ -204,6 +209,9 @@
 int main(int argc, char *argv[])
 {
        int i, j, opt, option_index;
+#ifdef __FreeBSD__
+       int io_fd;
+#endif
 
        static const struct option long_options[] = {
                {"dump",                no_argument, NULL, 'd'},
@@ -247,8 +255,13 @@
                }
        }
 
+#if defined(__FreeBSD__)
+       if ((io_fd = open("/dev/io", O_RDWR)) < 0) {
+               perror("/dev/io");
+#else
        if (iopl(3) < 0) {
                perror("iopl");
+#endif
                printf("Superiotool must be run as root.\n");
                exit(1);
        }
@@ -264,5 +277,8 @@
        if (!chip_found)
                printf("No Super I/O found\n");
 
+#ifdef __FreeBSD__
+       close(io_fd);
+#endif
        return 0;
 }
Index: ali.c
===================================================================
--- ali.c       (revision 3697)
+++ ali.c       (working copy)
@@ -60,13 +60,13 @@
 
 static void enter_conf_mode_ali(uint16_t port)
 {
-       outb(0x51, port);
-       outb(0x23, port);
+       OUTB(0x51, port);
+       OUTB(0x23, port);
 }
 
 static void exit_conf_mode_ali(uint16_t port)
 {
-       outb(0xbb, port);
+       OUTB(0xbb, port);
 }
 
 void probe_idregs_ali(uint16_t port)
Index: smsc.c
===================================================================
--- smsc.c      (revision 3697)
+++ smsc.c      (working copy)
@@ -608,13 +608,13 @@
         * in the assumption that the extra 0x55 won't hurt the other
         * Super I/Os. This is verified to be true on (at least) the FDC37N769.
         */
-       outb(0x55, port);
-       outb(0x55, port);
+       OUTB(0x55, port);
+       OUTB(0x55, port);
 }
 
 static void exit_conf_mode_smsc(uint16_t port)
 {
-       outb(0xaa, port);
+       OUTB(0xaa, port);
 }
 
 static void probe_idregs_smsc_helper(uint16_t port, uint8_t idreg,
Index: superiotool.h
===================================================================
--- superiotool.h       (revision 3697)
+++ superiotool.h       (working copy)
@@ -28,8 +28,29 @@
 #include <stdint.h>
 #include <string.h>
 #include <getopt.h>
+#if defined(__GLIBC__)
 #include <sys/io.h>
+#endif
 
+#ifdef __FreeBSD__
+  #include <sys/types.h>
+  #include <machine/cpufunc.h>
+
+  #define OUTB(x, y) do { u_int tmp = (y); outb(tmp, (x)); } while (0)
+  #define OUTW(x, y) do { u_int tmp = (y); outw(tmp, (x)); } while (0)
+  #define OUTL(x, y) do { u_int tmp = (y); outl(tmp, (x)); } while (0)
+  #define INB(x) __extension__ ({ u_int tmp = (x); inb(tmp); })
+  #define INW(x) __extension__ ({ u_int tmp = (x); inw(tmp); })
+  #define INL(x) __extension__ ({ u_int tmp = (x); inl(tmp); })
+#else
+  #define OUTB outb
+  #define OUTW outw
+  #define OUTL outl
+  #define INB  inb
+  #define INW  inw
+  #define INL  inl
+#endif
+
 #define USAGE "Usage: superiotool [-d] [-e] [-l] [-V] [-v] [-h]\n\n\
   -d | --dump            Dump Super I/O register contents\n\
   -e | --extra-dump      Dump secondary registers too (e.g. EC registers)\n\
--
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to