msrtool: Use libpci to let system and target probes find PCI devices.

And some more notes in TODO.

Signed-off-by: Peter Stuge <peter@stuge.se>
Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net>
git-svn-id: svn://svn.coreboot.org/coreboot/trunk@3770 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1

diff --git a/util/msrtool/TODO b/util/msrtool/TODO
index c701e7289685d4cedac4414de7c796e891bf584e..c3d5b18e64c04b4220a7bda088ab5479366247e2 100644 (file)
--- a/util/msrtool/TODO
+++ b/util/msrtool/TODO
@@ -1,8 +1,9 @@
 Systems
 -------
 FreeBSD #defines: see svn diff -r 3697:3698 util/
+i586: assembly instruction system driver, may have to ignore -c CPU # option
 
 Other ideas
 -----------
 Move MSR definitions and probe instructions into an external text file?
-Handle PCI registers as well?
+Handle PCI and port IO registers as well?

diff --git a/util/msrtool/configure b/util/msrtool/configure
index f627a6541f2918d928accd4d983727835196dde6..53bf6c4b9400d8b8bd6203b7dc611e59fc3edd36 100755 (executable)
--- a/util/msrtool/configure
+++ b/util/msrtool/configure
@@ -135,6 +135,25 @@ INSTALL=$(findprog "install" "${INSTALL}" install ginstall) || exit
 test -n "$DEBUG" && myCFLAGS="-O2 -g" || myCFLAGS="-Os"
 CFLAGS="${CFLAGS} ${myCFLAGS} -Wall -Werror"
 
+cat > .config.c << EOF
+#include <pci/pci.h>
+struct pci_access *pacc;
+int main(int argc, char *argv[])
+{ pacc = pci_alloc(); return 0; }
+EOF
+
+pc_CFLAGS="$(pkg-config libpci --cflags 2>/dev/null)"
+pc_LDFLAGS="$(pkg-config libpci --libs 2>/dev/null)"
+CFLAGS=$(trycompile "libpci (from pciutils)" "${pc_CFLAGS}" "-I/usr/local/include") || {
+       rm -f .config.c
+       exit 1
+}
+LDFLAGS=$(trylink "libpci (from pciutils)" "${pc_LDFLAGS}" "-lpci -lz" "-L/usr/local/lib -lpci -lz") || {
+       rm -f .config.c .config.o
+       exit 1
+}
+rm -f .config.c .config.o .config
+
 PREFIX="${PREFIX:-/usr/local}"
 
 OS_ARCH=$(uname)

diff --git a/util/msrtool/msrtool.c b/util/msrtool/msrtool.c
index 6f0b3a01095959739a37208fb4cc0f88efb3eb99..5742a1765ccf1416653f0f0d20588493c52b7318 100644 (file)
--- a/util/msrtool/msrtool.c
+++ b/util/msrtool/msrtool.c
@@ -25,6 +25,7 @@
 #include <sys/stat.h>
 #include <fcntl.h>
 #include <errno.h>
+#include <pci/pci.h>
 
 #include "msrtool.h"
 
@@ -36,6 +37,8 @@ const struct targetdef **targets = NULL;
 const struct sysdef *sys = NULL;
 uint8_t reserved = 0, verbose = 0, quiet = 0;
 
+struct pci_access *pacc = NULL;
+
 static struct targetdef alltargets[] = {
        { "geodelx", "AMD Geode(tm) LX", geodelx_probe, geodelx_msrs },
        { "cs5536", "AMD Geode(tm) CS5536", cs5536_probe, cs5536_msrs },
@@ -296,6 +299,14 @@ int main(int argc, char *argv[]) {
 
        printf_quiet("msrtool %s\n", VERSION);
 
+       pacc = pci_alloc();
+       if (NULL == pacc) {
+               fprintf(stderr, "Could not initialize PCI library! pci_alloc() failed.\n");
+               return 1;
+       }
+       pci_init(pacc);
+       pci_scan_bus(pacc);
+
        if (!sys && !input && !listknown)
                for (sys = allsystems; !SYSTEM_ISEOT(*sys); sys++) {
                        printf_verbose("Probing for system %s: %s\n", sys->name, sys->prettyname);

diff --git a/util/msrtool/msrtool.h b/util/msrtool/msrtool.h
index ab7c227d375499a1e6510322cadc0b06dd7f598e..7b639f06575aff5d410b9893f37aff35364ee477 100644 (file)
--- a/util/msrtool/msrtool.h
+++ b/util/msrtool/msrtool.h
@@ -22,6 +22,7 @@
 
 #include <stdio.h>
 #include <stdint.h>
+#include <pci/pci.h>
 
 #define HEXCHARS "0123456789abcdefABCDEF"
 
@@ -132,6 +133,8 @@ extern const struct targetdef **targets;
 
 extern uint8_t reserved, verbose, quiet;
 
+extern struct pci_access *pacc;
+
 #define printf_quiet(x...) do { if (!quiet) fprintf(stderr,x); } while(0)
 #define printf_verbose(x...) do { if (verbose && !quiet) fprintf(stderr,x); } while(0)
 
@@ -145,6 +148,7 @@ extern uint8_t reserved, verbose, quiet;
 
 /* sys.c */
 struct cpuid_t *cpuid(void);
+struct pci_dev *pci_dev_find(uint16_t vendor, uint16_t device);
 
 /* msrutils.c */
 void hexprint(FILE *f, const struct msr val, const uint8_t bits);

diff --git a/util/msrtool/sys.c b/util/msrtool/sys.c
index 95539c8677d352aa86f7ac44c5ea621fa0cda508..cc2cc4e7c034ecf66e1699687f044176419f7a6b 100644 (file)
--- a/util/msrtool/sys.c
+++ b/util/msrtool/sys.c
@@ -17,6 +17,8 @@
  * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301 USA
  */
 
+#include <pci/pci.h>
+
 #include "msrtool.h"
 
 static struct cpuid_t id;
@@ -40,3 +42,18 @@ struct cpuid_t *cpuid(void) {
        }
        return &id;
 }
+
+struct pci_dev *pci_dev_find(uint16_t vendor, uint16_t device) {
+       struct pci_dev *temp;
+       struct pci_filter filter;
+
+       pci_filter_init(NULL, &filter);
+       filter.vendor = vendor;
+       filter.device = device;
+
+       for (temp = pacc->devices; temp; temp = temp->next)
+               if (pci_filter_match(&filter, temp))
+                       return temp;
+
+       return NULL;
+}