[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