[PATCH] ATM: Delete no use FILL_RX_POOLS_IN_BH marco

Subsystems: atm, the rest

STALE5486d

2 messages, 1 author, 2011-08-05 · open the first message on its own page

[PATCH] ATM: Delete no use FILL_RX_POOLS_IN_BH marco

From: <hidden>
Date: 2011-07-08 12:57:33

From: Wang Shaoyan <redacted>

The macro FILL_RX_POOLS_IN_BH is never been used, in order to avoid 
the compiler reports error because of the usage of function INIT_WORK,
we just delete the marco.

Cc: Chas Williams <redacted>
Cc: linux-atm-general@lists.sourceforge.net
Signed-off-by: Wang Shaoyan <redacted>
---
 drivers/atm/ambassador.c |   11 +----------
 drivers/atm/ambassador.h |    4 ----
 2 files changed, 1 insertions(+), 14 deletions(-)
diff --git a/drivers/atm/ambassador.c b/drivers/atm/ambassador.c
index a5fcb1e..bb3b016 100644
--- a/drivers/atm/ambassador.c
+++ b/drivers/atm/ambassador.c
@@ -813,7 +813,7 @@ static void fill_rx_pool (amb_dev * dev, unsigned char pool,
   return;
 }
 
-// top up all RX pools (can also be called as a bottom half)
+// top up all RX pools
 static void fill_rx_pools (amb_dev * dev) {
   unsigned char pool;
   
@@ -872,11 +872,7 @@ static irqreturn_t interrupt_handler(int irq, void *dev_id) {
       ++irq_work;
   
     if (irq_work) {
-#ifdef FILL_RX_POOLS_IN_BH
-      schedule_work (&dev->bh);
-#else
       fill_rx_pools (dev);
-#endif
 
       PRINTD (DBG_IRQ, "work done: %u", irq_work);
     } else {
@@ -2154,11 +2150,6 @@ static void setup_dev(amb_dev *dev, struct pci_dev *pci_dev)
       dev->tx_avail = ATM_OC3_PCR;
       dev->rx_avail = ATM_OC3_PCR;
       
-#ifdef FILL_RX_POOLS_IN_BH
-      // initialise bottom half
-      INIT_WORK(&dev->bh, (void (*)(void *)) fill_rx_pools, dev);
-#endif
-      
       // semaphore for txer/rxer modifications - we cannot use a
       // spinlock as the critical region needs to switch processes
       mutex_init(&dev->vcc_sf);
diff --git a/drivers/atm/ambassador.h b/drivers/atm/ambassador.h
index bd1c46a..aa97105 100644
--- a/drivers/atm/ambassador.h
+++ b/drivers/atm/ambassador.h
@@ -630,10 +630,6 @@ struct amb_dev {
   u32              iobase;
   u32 *            membase;
 
-#ifdef FILL_RX_POOLS_IN_BH
-  struct work_struct bh;
-#endif
-  
   amb_cq           cq;
   amb_txq          txq;
   amb_rxq          rxq[NUM_RX_POOLS];
-- 
1.7.4.1

Re: [PATCH] ATM: Delete no use FILL_RX_POOLS_IN_BH marco

From: Wang Shaoyan <hidden>
Date: 2011-08-05 12:20:34

applied

2011/7/8  [off-list ref]:
quoted hunk
From: Wang Shaoyan <redacted>

The macro FILL_RX_POOLS_IN_BH is never been used, in order to avoid
the compiler reports error because of the usage of function INIT_WORK,
we just delete the marco.

Cc: Chas Williams <redacted>
Cc: linux-atm-general@lists.sourceforge.net
Signed-off-by: Wang Shaoyan <redacted>
---
 drivers/atm/ambassador.c |   11 +----------
 drivers/atm/ambassador.h |    4 ----
 2 files changed, 1 insertions(+), 14 deletions(-)
diff --git a/drivers/atm/ambassador.c b/drivers/atm/ambassador.c
index a5fcb1e..bb3b016 100644
--- a/drivers/atm/ambassador.c
+++ b/drivers/atm/ambassador.c
@@ -813,7 +813,7 @@ static void fill_rx_pool (amb_dev * dev, unsigned char pool,
  return;
 }

-// top up all RX pools (can also be called as a bottom half)
+// top up all RX pools
 static void fill_rx_pools (amb_dev * dev) {
  unsigned char pool;
@@ -872,11 +872,7 @@ static irqreturn_t interrupt_handler(int irq, void *dev_id) {
      ++irq_work;

    if (irq_work) {
-#ifdef FILL_RX_POOLS_IN_BH
-      schedule_work (&dev->bh);
-#else
      fill_rx_pools (dev);
-#endif

      PRINTD (DBG_IRQ, "work done: %u", irq_work);
    } else {
@@ -2154,11 +2150,6 @@ static void setup_dev(amb_dev *dev, struct pci_dev *pci_dev)
      dev->tx_avail = ATM_OC3_PCR;
      dev->rx_avail = ATM_OC3_PCR;

-#ifdef FILL_RX_POOLS_IN_BH
-      // initialise bottom half
-      INIT_WORK(&dev->bh, (void (*)(void *)) fill_rx_pools, dev);
-#endif
-
      // semaphore for txer/rxer modifications - we cannot use a
      // spinlock as the critical region needs to switch processes
      mutex_init(&dev->vcc_sf);
diff --git a/drivers/atm/ambassador.h b/drivers/atm/ambassador.h
index bd1c46a..aa97105 100644
--- a/drivers/atm/ambassador.h
+++ b/drivers/atm/ambassador.h
@@ -630,10 +630,6 @@ struct amb_dev {
  u32              iobase;
  u32 *            membase;

-#ifdef FILL_RX_POOLS_IN_BH
-  struct work_struct bh;
-#endif
-
  amb_cq           cq;
  amb_txq          txq;
  amb_rxq          rxq[NUM_RX_POOLS];
--
1.7.4.1


-- 
Wang Shaoyan
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help