net: ethernet: ti: davinci_cpdma: move cpdma channel struct macroses to internals
authorIvan Khoronzhuk <ivan.khoronzhuk@linaro.org>
Mon, 22 Aug 2016 18:18:27 +0000 (21:18 +0300)
committerDavid S. Miller <davem@davemloft.net>
Tue, 23 Aug 2016 07:13:11 +0000 (00:13 -0700)
Keep the driver internals in C file. Currently it's not required for
drivers to know rx or tx a channel is, except create function.
So correct "channel create" function, and use all channel struct
macroses only for internal use.

Reviewed-by: Mugunthan V N <mugunthanvnm@ti.com>
Signed-off-by: Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/ti/cpsw.c
drivers/net/ethernet/ti/davinci_cpdma.c
drivers/net/ethernet/ti/davinci_cpdma.h
drivers/net/ethernet/ti/davinci_emac.c

index 28cc716f22c9c4e099ccc0f097b33ff3fa823d59..1fbb50f256036743250bc5fa9d8d38d0b072ab6a 100644 (file)
@@ -2535,11 +2535,8 @@ static int cpsw_probe(struct platform_device *pdev)
                goto clean_runtime_disable_ret;
        }
 
-       cpsw->txch[0] = cpdma_chan_create(cpsw->dma, tx_chan_num(0),
-                                         cpsw_tx_handler);
-       cpsw->rxch[0] = cpdma_chan_create(cpsw->dma, rx_chan_num(0),
-                                         cpsw_rx_handler);
-
+       cpsw->txch[0] = cpdma_chan_create(cpsw->dma, 0, cpsw_tx_handler, 0);
+       cpsw->rxch[0] = cpdma_chan_create(cpsw->dma, 0, cpsw_rx_handler, 1);
        if (WARN_ON(!cpsw->rxch[0] || !cpsw->txch[0])) {
                dev_err(priv->dev, "error initializing dma channels\n");
                ret = -ENOMEM;
index 4b578b17ec2d558e67f2adf8e4aa49157e3ffe3f..c3f35f11a8fd53d59ea1ac0c8d69653ec049c443 100644 (file)
@@ -124,6 +124,13 @@ struct cpdma_chan {
        int     int_set, int_clear, td;
 };
 
+#define tx_chan_num(chan)      (chan)
+#define rx_chan_num(chan)      ((chan) + CPDMA_MAX_CHANNELS)
+#define is_rx_chan(chan)       ((chan)->chan_num >= CPDMA_MAX_CHANNELS)
+#define is_tx_chan(chan)       (!is_rx_chan(chan))
+#define __chan_linear(chan_num)        ((chan_num) & (CPDMA_MAX_CHANNELS - 1))
+#define chan_linear(chan)      __chan_linear((chan)->chan_num)
+
 /* The following make access to common cpdma_ctlr params more readable */
 #define dmaregs                params.dmaregs
 #define num_chan       params.num_chan
@@ -441,12 +448,14 @@ static void cpdma_chan_split_pool(struct cpdma_ctlr *ctlr)
 }
 
 struct cpdma_chan *cpdma_chan_create(struct cpdma_ctlr *ctlr, int chan_num,
-                                    cpdma_handler_fn handler)
+                                    cpdma_handler_fn handler, int rx_type)
 {
+       int offset = chan_num * 4;
        struct cpdma_chan *chan;
-       int offset = (chan_num % CPDMA_MAX_CHANNELS) * 4;
        unsigned long flags;
 
+       chan_num = rx_type ? rx_chan_num(chan_num) : tx_chan_num(chan_num);
+
        if (__chan_linear(chan_num) >= ctlr->num_chan)
                return NULL;
 
index 070f1d0ea8480bcfb4cfc76c8b74d73784df976e..a07b22b12bc1fca73b9460c74fade03f45e78fd5 100644 (file)
 
 #define CPDMA_MAX_CHANNELS     BITS_PER_LONG
 
-#define tx_chan_num(chan)      (chan)
-#define rx_chan_num(chan)      ((chan) + CPDMA_MAX_CHANNELS)
-#define is_rx_chan(chan)       ((chan)->chan_num >= CPDMA_MAX_CHANNELS)
-#define is_tx_chan(chan)       (!is_rx_chan(chan))
-#define __chan_linear(chan_num)        ((chan_num) & (CPDMA_MAX_CHANNELS - 1))
-#define chan_linear(chan)      __chan_linear((chan)->chan_num)
-
 #define CPDMA_RX_SOURCE_PORT(__status__)       ((__status__ >> 16) & 0x7)
 
 #define CPDMA_EOI_RX_THRESH    0x0
@@ -79,7 +72,7 @@ int cpdma_ctlr_start(struct cpdma_ctlr *ctlr);
 int cpdma_ctlr_stop(struct cpdma_ctlr *ctlr);
 
 struct cpdma_chan *cpdma_chan_create(struct cpdma_ctlr *ctlr, int chan_num,
-                                    cpdma_handler_fn handler);
+                                    cpdma_handler_fn handler, int rx_type);
 int cpdma_chan_get_rx_buf_num(struct cpdma_chan *chan);
 int cpdma_chan_destroy(struct cpdma_chan *chan);
 int cpdma_chan_start(struct cpdma_chan *chan);
index 2d6fc9a0fb217f2c7e6afbaab9079e07765aa14c..2fd94a5bc1f3a653bebff3c5b71642fe49b97c24 100644 (file)
@@ -1870,10 +1870,10 @@ static int davinci_emac_probe(struct platform_device *pdev)
                goto no_pdata;
        }
 
-       priv->txchan = cpdma_chan_create(priv->dma, tx_chan_num(EMAC_DEF_TX_CH),
-                                      emac_tx_handler);
-       priv->rxchan = cpdma_chan_create(priv->dma, rx_chan_num(EMAC_DEF_RX_CH),
-                                      emac_rx_handler);
+       priv->txchan = cpdma_chan_create(priv->dma, EMAC_DEF_TX_CH,
+                                        emac_tx_handler, 0);
+       priv->rxchan = cpdma_chan_create(priv->dma, EMAC_DEF_RX_CH,
+                                        emac_rx_handler, 1);
        if (WARN_ON(!priv->txchan || !priv->rxchan)) {
                rc = -ENOMEM;
                goto no_cpdma_chan;