[rtems commit] mips/malta: Use shared pci_find_device() and removed unused pci_list_devices()

Joel Sherrill joel at rtems.org
Thu Mar 10 16:45:12 UTC 2016


Module:    rtems
Branch:    master
Commit:    81df701ab46046719d6d2b2876c939342427e8c4
Changeset: http://git.rtems.org/rtems/commit/?id=81df701ab46046719d6d2b2876c939342427e8c4

Author:    Joel Sherrill <joel at rtems.org>
Date:      Wed Mar  9 15:30:24 2016 -0600

mips/malta: Use shared pci_find_device() and removed unused pci_list_devices()

Also includes .h file cleanup.

---

 c/src/lib/libbsp/mips/malta/Makefile.am          |   3 +-
 c/src/lib/libbsp/mips/malta/include/pci.h        |  32 ---
 c/src/lib/libbsp/mips/malta/pci/pcifinddevice.c  | 277 -----------------------
 c/src/lib/libbsp/mips/malta/pci/pcilistdevices.c | 102 ---------
 4 files changed, 1 insertion(+), 413 deletions(-)

diff --git a/c/src/lib/libbsp/mips/malta/Makefile.am b/c/src/lib/libbsp/mips/malta/Makefile.am
index 15aa898..5d4b5f5 100644
--- a/c/src/lib/libbsp/mips/malta/Makefile.am
+++ b/c/src/lib/libbsp/mips/malta/Makefile.am
@@ -68,8 +68,7 @@ libbsp_a_LIBADD += ../../../libcpu/@RTEMS_CPU@/shared/interrupts.rel
 
 # pci
 libbsp_a_SOURCES += pci/pci.c
-libbsp_a_SOURCES += pci/pcifinddevice.c
-libbsp_a_SOURCES += pci/pcilistdevices.c
+libbsp_a_SOURCES += ../../shared/pci/pci_find_device.c
 
 #isr
 libbsp_a_SOURCES += ../../shared/src/irq-generic.c
diff --git a/c/src/lib/libbsp/mips/malta/include/pci.h b/c/src/lib/libbsp/mips/malta/include/pci.h
index ad69e3a..c5944da 100644
--- a/c/src/lib/libbsp/mips/malta/include/pci.h
+++ b/c/src/lib/libbsp/mips/malta/include/pci.h
@@ -59,37 +59,6 @@ void FixupPCI( const struct _int_map *, int (*swizzler)(int,int) );
 /* FIXME: This probably belongs into rtems/pci.h */
 extern unsigned char pci_bus_count();
 
-/* FIXME: This also is generic and could go into rtems/pci.h */
-
-/* Scan pci config space and run a user callback on each
- * device present; the user callback may return 0 to
- * continue the scan or a value > 0 to abort the scan.
- * Return values < 0 are reserved and must not be used.
- *
- * RETURNS: a (opaque) handle pointing to the bus/slot/fn-triple
- *          just after where the scan was aborted by a callback
- *          returning 1 (see above) or NULL if all devices were
- *          scanned.
- *          The handle may be passed to this routine to resume the
- *          scan continuing with the device after the one causing the
- *          abort.
- *          Pass a NULL 'handle' argument to start scanning from
- *          the beginning (bus/slot/fn = 0/0/0).
- */
-typedef void *BSP_PciScanHandle;
-typedef int (*BSP_PciScannerCb)(int bus, int slot, int fun, void *uarg);
-
-
-BSP_PciScanHandle
-BSP_pciScan(BSP_PciScanHandle handle, BSP_PciScannerCb cb, void *uarg);
-
-/* Dump basic config. space info to a file. The argument may
- * be NULL in which case 'stdout' is used.
- * NOTE: the C-library must be functional before you can use
- *       this routine.
- */
-void BSP_pciConfigDump(FILE *fp);
-
 int indirect_pci_read_config_byte(
   unsigned char bus,
   unsigned char slot,
@@ -140,7 +109,6 @@ int indirect_pci_write_config_dword(
 
 /* Can these be moved to the rtems pci.h? */
 int FindPCIbridge( int mybus, struct pcibridge *pb );
-void pci_list_devices( void );
 
 const pci_config_access_functions pci_indirect_functions;
 
diff --git a/c/src/lib/libbsp/mips/malta/pci/pcifinddevice.c b/c/src/lib/libbsp/mips/malta/pci/pcifinddevice.c
deleted file mode 100644
index 43ae261..0000000
--- a/c/src/lib/libbsp/mips/malta/pci/pcifinddevice.c
+++ /dev/null
@@ -1,277 +0,0 @@
-/**
- *  @file
- *
- *  This file was copied from the powerpc and modified slightly.
- *  I think this file could be made generic and put in a general pci
- *  area.
- */
-
-/*
- *  COPYRIGHT (c) 1989-2012.
- *  On-Line Applications Research Corporation (OAR).
- *
- *  The license and distribution terms for this file may be
- *  found in the file LICENSE in this distribution or at
- *  http://www.rtems.org/license/LICENSE.
- */
-/* find a particular PCI device
- * (we assume, the firmware configured the PCI bus[es] for us)
- */
-
-/*
- * Authorship
- * ----------
- * This software was created by
- *     Till Straumann <strauman at slac.stanford.edu>, 2001,
- *      Stanford Linear Accelerator Center, Stanford University.
- *
- * Acknowledgement of sponsorship
- * ------------------------------
- * This software was produced by
- *     the Stanford Linear Accelerator Center, Stanford University,
- *      under Contract DE-AC03-76SFO0515 with the Department of Energy.
- *
- * Government disclaimer of liability
- * ----------------------------------
- * Neither the United States nor the United States Department of Energy,
- * nor any of their employees, makes any warranty, express or implied, or
- * assumes any legal liability or responsibility for the accuracy,
- * completeness, or usefulness of any data, apparatus, product, or process
- * disclosed, or represents that its use would not infringe privately owned
- * rights.
- *
- * Stanford disclaimer of liability
- * --------------------------------
- * Stanford University makes no representations or warranties, express or
- * implied, nor assumes any liability for the use of this software.
- *
- * Stanford disclaimer of copyright
- * --------------------------------
- * Stanford University, owner of the copyright, hereby disclaims its
- * copyright and all other rights in this software.  Hence, anyone may
- * freely use it for any purpose without restriction.
- *
- * Maintenance of notices
- * ----------------------
- * In the interest of clarity regarding the origin and status of this
- * SLAC software, this and all the preceding Stanford University notices
- * are to remain affixed to any copy or derivative of this software made
- * or distributed by the recipient and are to be affixed to any copy of
- * software made or distributed by the recipient that contains a copy or
- * derivative of this software.
- *
- * ------------------ SLAC Software Notices, Set 4 OTT.002a, 2004 FEB 03
- */
-
-#define PCI_DEBUG
-
-#include <inttypes.h>
-#include <bsp/pci.h>
-#include <rtems/bspIo.h>
-#include <stdio.h>
-
-/* Stolen from i386... */
-
-/*
- * Make device signature from bus number, device number and function
- * number
- */
-#define PCIB_DEVSIG_MAKE(b,d,f) ((b<<8)|(d<<3)|(f))
-
-/*
- * Extract various parts from device signature
- */
-#define PCIB_DEVSIG_BUS(x) (((x)>>8) &0xff)
-#define PCIB_DEVSIG_DEV(x) (((x)>>3) & 0x1f)
-#define PCIB_DEVSIG_FUNC(x) ((x) & 0x7)
-
-typedef struct {
-  unsigned short  vid,did;
-  int             inst;
-} fd_arg;
-
-static int find_dev_cb(
-  int bus,
-  int dev,
-  int fun,
-  void *uarg
-)
-{
-  fd_arg         *a = uarg;
-  unsigned short  vendor;
-  unsigned short  device;
-
-  pci_read_config_word(bus,dev,fun,PCI_VENDOR_ID,&vendor);
-  pci_read_config_word(bus,dev,fun,PCI_DEVICE_ID,&device);
-  printk("  FOUND: Vendor 0x%x Device: 0x%x\n", vendor, device);
-
-  if (a->vid == vendor) {
-    if (a->did == device && 0 == a->inst-- ) {
-      a->inst = PCIB_DEVSIG_MAKE( bus, dev, fun );
-      return 1;
-    }
-  }
-  return 0;
-}
-
-int  pci_find_device(
-  unsigned short vendorid,
-  unsigned short deviceid,
-  int instance,
-  int *pbus,
-  int *pdev,
-  int *pfun
-)
-{
-  fd_arg a;
-  void   *h;
-
-  a.vid  = vendorid;
-  a.did  = deviceid;
-  a.inst = instance;
-
-  if ( (h = BSP_pciScan(0, find_dev_cb, (void*)&a)) ) {
-    *pbus = PCIB_DEVSIG_BUS(  a.inst );
-    *pdev = PCIB_DEVSIG_DEV(  a.inst );
-    *pfun = PCIB_DEVSIG_FUNC( a.inst );
-    return 0;
-  }
-
-  return -1;
-}
-
-static int dump_dev_cb(
-  int  bus,
-  int  dev,
-  int  fun,
-  void *uarg
-)
-{
-  uint16_t vi,di;
-  uint16_t cd,st;
-  uint32_t b1,b2;
-  uint8_t  il,ip;
-  FILE     *f = uarg;
-
-  pci_read_config_word (bus, dev, fun, PCI_VENDOR_ID,      &vi);
-  pci_read_config_word (bus, dev, fun, PCI_DEVICE_ID,      &di);
-  pci_read_config_word (bus, dev, fun, PCI_COMMAND,        &cd);
-  pci_read_config_word (bus, dev, fun, PCI_STATUS,         &st);
-  pci_read_config_dword(bus, dev, fun, PCI_BASE_ADDRESS_0, &b1);
-  pci_read_config_dword(bus, dev, fun, PCI_BASE_ADDRESS_1, &b2);
-  pci_read_config_byte (bus, dev, fun, PCI_INTERRUPT_LINE, &il);
-  pci_read_config_byte (bus, dev, fun, PCI_INTERRUPT_PIN,  &ip);
-
-  fprintf(
-    f,
-    "%3d:0x%02x:%d    0x%04x-0x%04x:  0x%04x 0x%04x 0x%08" PRIx32
-    " 0x%08" PRIx32 "       %d -> %3d (=0x%02x)\n",
-    bus,
-    dev,
-    fun,
-    vi,
-    di,
-    cd,
-    st,
-    b1,
-    b2,
-    ip,
-    il,
-    il
-  );
-  return 0;
-}
-
-void BSP_pciConfigDump(FILE *f)
-{
-  if ( !f )
-    f = stdout;
-  fprintf(
-    f,
-    "BUS:SLOT:FUN  VENDOR-DEV_ID: COMMAND STATUS BASE_ADDR0 BASE_ADDR1 "
-    "IRQ_PIN -> IRQ_LINE\n"
-  );
-  BSP_pciScan(0, dump_dev_cb, f);
-}
-
-BSP_PciScanHandle BSP_pciScan(
-  BSP_PciScanHandle handle,
-  BSP_PciScannerCb cb,
-  void *uarg
-) {
-
-  uint32_t d;
-  unsigned char bus,dev,fun,hd;
-
-  bus = PCIB_DEVSIG_BUS(  (unsigned long)handle );
-  dev = PCIB_DEVSIG_DEV(  (unsigned long)handle );
-  fun = PCIB_DEVSIG_FUNC( (unsigned long)handle );
-
-  hd = fun > 0 ? PCI_MAX_FUNCTIONS : 1;
-
-  for (; bus<pci_bus_count(); bus++, dev=0) {
-    for (; dev<PCI_MAX_DEVICES; dev++, fun=0) {
-      for (; fun<hd; fun++) {
-        /*
-         * The last devfn id/slot is special; must skip it
-         */
-        if (PCI_MAX_DEVICES-1==dev && PCI_MAX_FUNCTIONS-1 == fun)
-          break;
-
-        (void)pci_read_config_dword(bus,dev,0,PCI_VENDOR_ID,&d);
-        if (PCI_INVALID_VENDORDEVICEID == d)
-          continue;
-        #ifdef PCI_DEBUG
-          printk(
-            "BSP_pciScan: Read _config_dword bus %d "
-            "dev 0x%x value 0x%2x\n",
-            bus,
-            dev,
-            d
-          );
-        #endif
-
-        if ( 0 == fun ) {
-          pci_read_config_byte(bus,dev,0, PCI_HEADER_TYPE, &hd);
-          #ifdef PCI_DEBUG
-            printk(
-              "BSP_pciScan: Read _config_byte bus %d dev 0x%x "
-              "fun %d value 0x%2x\n",
-              bus,
-              dev,
-              fun,
-              d
-            );
-          #endif
-          hd = (hd & PCI_HEADER_TYPE_MULTI_FUNCTION ? PCI_MAX_FUNCTIONS : 1);
-        }
-
-        (void)pci_read_config_dword(bus,dev,fun,PCI_VENDOR_ID,&d);
-        if (PCI_INVALID_VENDORDEVICEID == d)
-          continue;
-        #ifdef PCI_DEBUG
-          printk("BSP_pciScan: found 0x%08x at %d/x%02x/%d\n",d,bus,dev,fun);
-        #endif
-        if ( cb(bus,dev,fun,uarg) > 0 ) {
-          if ( ++fun >= hd ) {
-            fun = 0;
-            if ( ++dev >= PCI_MAX_DEVICES ) {
-              dev = 0;
-              bus++;
-            }
-          }
-          #ifdef PCI_DEBUG
-            printk(
-              "BSP_pciScan: Going to Return: bus %d dev 0x%x fun %d\n",
-              bus,
-              dev,
-              fun
-            );
-          #endif
-          return (void*) PCIB_DEVSIG_MAKE(bus,dev,fun);
-        }
-      }
-    }
-  }
-  return 0;
-}
diff --git a/c/src/lib/libbsp/mips/malta/pci/pcilistdevices.c b/c/src/lib/libbsp/mips/malta/pci/pcilistdevices.c
deleted file mode 100644
index 8ff5eb8..0000000
--- a/c/src/lib/libbsp/mips/malta/pci/pcilistdevices.c
+++ /dev/null
@@ -1,102 +0,0 @@
-/**
- *  @file
- *
- *  I think this file could be made generic and put in a general pci
- *  area.
- */
-
-/*
- *  COPYRIGHT (c) 1989-2012.
- *  On-Line Applications Research Corporation (OAR).
- *
- *  The license and distribution terms for this file may be
- *  found in the file LICENSE in this distribution or at
- *  http://www.rtems.org/license/LICENSE.
- */
-
-/*
- * List all PCI Devices
- */
-
-#define PCI_DEBUG
-
-#include <inttypes.h>
-#include <bsp/pci.h>
-#include <rtems/bspIo.h>
-#include <stdio.h>
-
-/*
- * Make device signature from bus number, device number and function
- * number
- */
-#define PCIB_DEVSIG_MAKE(b,d,f) ((b<<8)|(d<<3)|(f))
-
-/*
- * Extract various parts from device signature
- */
-#define PCIB_DEVSIG_BUS(x) (((x)>>8) &0xff)
-#define PCIB_DEVSIG_DEV(x) (((x)>>3) & 0x1f)
-#define PCIB_DEVSIG_FUNC(x) ((x) & 0x7)
-
-static int print_device_config(
-   int bus,
-   int dev,
-   int fun
-)
-{
-  uint16_t vi,di;
-  uint16_t cd,st;
-  uint32_t b1,b2;
-  uint8_t  il,ip;
-
-  pci_read_config_word (bus, dev, fun, PCI_VENDOR_ID,      &vi);
-  pci_read_config_word (bus, dev, fun, PCI_DEVICE_ID,      &di);
-  pci_read_config_word (bus, dev, fun, PCI_COMMAND,        &cd);
-  pci_read_config_word (bus, dev, fun, PCI_STATUS,         &st);
-  pci_read_config_dword(bus, dev, fun, PCI_BASE_ADDRESS_0, &b1);
-  pci_read_config_dword(bus, dev, fun, PCI_BASE_ADDRESS_1, &b2);
-  pci_read_config_byte (bus, dev, fun, PCI_INTERRUPT_LINE, &il);
-  pci_read_config_byte (bus, dev, fun, PCI_INTERRUPT_PIN,  &ip);
-
-  printk(
-    "%3d:0x%02x:%d    0x%04x-0x%04x:  0x%04x 0x%04x 0x%08" PRIx32
-        " 0x%08" PRIx32 "       %d -> %3d (=0x%02x)\n",
-     bus, dev, fun, vi, di, cd, st, b1, b2, ip, il, il);
-  return 0;
-}
-
-void pci_list_devices( void )
-{
-   uint32_t d;
-   unsigned char bus,dev,fun,hd;
-
-  printk(
-    "BUS:SLOT:FUN  VENDOR-DEV_ID: COMMAND STATUS BASE_ADDR0 "
-    "BASE_ADDR1 IRQ_PIN -> IRQ_LINE\n"
-  );
-  for (bus=0 ; bus<pci_bus_count(); bus++) {
-    for (dev=0 ; dev<PCI_MAX_DEVICES; dev++) {
-      for (fun=0 ; fun<PCI_MAX_FUNCTIONS; fun++) {
-        /*
-         * The last devfn id/slot is special; must skip it
-         */
-        if (PCI_MAX_DEVICES-1==dev && PCI_MAX_FUNCTIONS-1 == fun)
-          break;
-
-        (void) pci_read_config_dword(bus,dev,0,PCI_VENDOR_ID,&d);
-        if (PCI_INVALID_VENDORDEVICEID == d)
-          continue;
-
-        if ( 0 == fun ) {
-          pci_read_config_byte(bus,dev,0, PCI_HEADER_TYPE, &hd);
-          hd = (hd & PCI_HEADER_TYPE_MULTI_FUNCTION ? PCI_MAX_FUNCTIONS : 1);
-        }
-
-        (void)pci_read_config_dword(bus,dev,fun,PCI_VENDOR_ID,&d);
-        if (PCI_INVALID_VENDORDEVICEID == d)
-          continue;
-        print_device_config( bus, dev, fun );
-      }
-    }
-  }
-}



More information about the vc mailing list