Merge tag 'asoc-3.6' of git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound...
[firefly-linux-kernel-4.4.55.git] / drivers / net / caif / caif_hsi.c
index 4a27adb7ae67f7011ba1ae4bfae7784405435178..0def8b3106f48e8d47c72f5d25bd344224986ecc 100644 (file)
@@ -11,7 +11,6 @@
 #include <linux/init.h>
 #include <linux/module.h>
 #include <linux/device.h>
-#include <linux/platform_device.h>
 #include <linux/netdevice.h>
 #include <linux/string.h>
 #include <linux/list.h>
@@ -20,7 +19,7 @@
 #include <linux/sched.h>
 #include <linux/if_arp.h>
 #include <linux/timer.h>
-#include <linux/rtnetlink.h>
+#include <net/rtnetlink.h>
 #include <linux/pkt_sched.h>
 #include <net/caif/caif_layer.h>
 #include <net/caif/caif_hsi.h>
@@ -33,59 +32,46 @@ MODULE_DESCRIPTION("CAIF HSI driver");
 #define PAD_POW2(x, pow) ((((x)&((pow)-1)) == 0) ? 0 :\
                                (((pow)-((x)&((pow)-1)))))
 
-static int inactivity_timeout = 1000;
-module_param(inactivity_timeout, int, S_IRUGO | S_IWUSR);
-MODULE_PARM_DESC(inactivity_timeout, "Inactivity timeout on HSI, ms.");
+static const struct cfhsi_config  hsi_default_config = {
 
-static int aggregation_timeout = 1;
-module_param(aggregation_timeout, int, S_IRUGO | S_IWUSR);
-MODULE_PARM_DESC(aggregation_timeout, "Aggregation timeout on HSI, ms.");
+       /* Inactivity timeout on HSI, ms */
+       .inactivity_timeout = HZ,
 
-/*
- * HSI padding options.
- * Warning: must be a base of 2 (& operation used) and can not be zero !
- */
-static int hsi_head_align = 4;
-module_param(hsi_head_align, int, S_IRUGO);
-MODULE_PARM_DESC(hsi_head_align, "HSI head alignment.");
+       /* Aggregation timeout (ms) of zero means no aggregation is done*/
+       .aggregation_timeout = 1,
 
-static int hsi_tail_align = 4;
-module_param(hsi_tail_align, int, S_IRUGO);
-MODULE_PARM_DESC(hsi_tail_align, "HSI tail alignment.");
-
-/*
* HSI link layer flowcontrol thresholds.
* Warning: A high threshold value migth increase throughput but it will at
- * the same time prevent channel prioritization and increase the risk of
- * flooding the modem. The high threshold should be above the low.
- */
-static int hsi_high_threshold = 100;
-module_param(hsi_high_threshold, int, S_IRUGO);
-MODULE_PARM_DESC(hsi_high_threshold, "HSI high threshold (FLOW OFF).");
+       /*
+        * HSI link layer flow-control thresholds.
+        * Threshold values for the HSI packet queue. Flow-control will be
+        * asserted when the number of packets exceeds q_high_mark. It will
+        * not be de-asserted before the number of packets drops below
       * q_low_mark.
       * Warning: A high threshold value might increase throughput but it
+        * will at the same time prevent channel prioritization and increase
+        * the risk of flooding the modem. The high threshold should be above
+        * the low.
+        */
+       .q_high_mark = 100,
+       .q_low_mark = 50,
 
-static int hsi_low_threshold = 50;
-module_param(hsi_low_threshold, int, S_IRUGO);
-MODULE_PARM_DESC(hsi_low_threshold, "HSI high threshold (FLOW ON).");
+       /*
+        * HSI padding options.
+        * Warning: must be a base of 2 (& operation used) and can not be zero !
+        */
+       .head_align = 4,
+       .tail_align = 4,
+};
 
 #define ON 1
 #define OFF 0
 
-/*
- * Threshold values for the HSI packet queue. Flowcontrol will be asserted
- * when the number of packets exceeds HIGH_WATER_MARK. It will not be
- * de-asserted before the number of packets drops below LOW_WATER_MARK.
- */
-#define LOW_WATER_MARK   hsi_low_threshold
-#define HIGH_WATER_MARK  hsi_high_threshold
-
 static LIST_HEAD(cfhsi_list);
-static spinlock_t cfhsi_list_lock;
 
 static void cfhsi_inactivity_tout(unsigned long arg)
 {
        struct cfhsi *cfhsi = (struct cfhsi *)arg;
 
-       dev_dbg(&cfhsi->ndev->dev, "%s.\n",
+       netdev_dbg(cfhsi->ndev, "%s.\n",
                __func__);
 
        /* Schedule power down work queue. */
@@ -101,8 +87,8 @@ static void cfhsi_update_aggregation_stats(struct cfhsi *cfhsi,
        int hpad, tpad, len;
 
        info = (struct caif_payload_info *)&skb->cb;
-       hpad = 1 + PAD_POW2((info->hdr_len + 1), hsi_head_align);
-       tpad = PAD_POW2((skb->len + hpad), hsi_tail_align);
+       hpad = 1 + PAD_POW2((info->hdr_len + 1), cfhsi->cfg.head_align);
+       tpad = PAD_POW2((skb->len + hpad), cfhsi->cfg.tail_align);
        len = skb->len + hpad + tpad;
 
        if (direction > 0)
@@ -115,7 +101,7 @@ static bool cfhsi_can_send_aggregate(struct cfhsi *cfhsi)
 {
        int i;
 
-       if (cfhsi->aggregation_timeout < 0)
+       if (cfhsi->cfg.aggregation_timeout == 0)
                return true;
 
        for (i = 0; i < CFHSI_PRIO_BEBK; ++i) {
@@ -171,7 +157,7 @@ static void cfhsi_abort_tx(struct cfhsi *cfhsi)
        cfhsi->tx_state = CFHSI_TX_STATE_IDLE;
        if (!test_bit(CFHSI_SHUTDOWN, &cfhsi->bits))
                mod_timer(&cfhsi->inactivity_timer,
-                       jiffies + cfhsi->inactivity_timeout);
+                       jiffies + cfhsi->cfg.inactivity_timeout);
        spin_unlock_bh(&cfhsi->lock);
 }
 
@@ -181,14 +167,14 @@ static int cfhsi_flush_fifo(struct cfhsi *cfhsi)
        size_t fifo_occupancy;
        int ret;
 
-       dev_dbg(&cfhsi->ndev->dev, "%s.\n",
+       netdev_dbg(cfhsi->ndev, "%s.\n",
                __func__);
 
        do {
-               ret = cfhsi->dev->cfhsi_fifo_occupancy(cfhsi->dev,
+               ret = cfhsi->ops->cfhsi_fifo_occupancy(cfhsi->ops,
                                &fifo_occupancy);
                if (ret) {
-                       dev_warn(&cfhsi->ndev->dev,
+                       netdev_warn(cfhsi->ndev,
                                "%s: can't get FIFO occupancy: %d.\n",
                                __func__, ret);
                        break;
@@ -198,11 +184,11 @@ static int cfhsi_flush_fifo(struct cfhsi *cfhsi)
 
                fifo_occupancy = min(sizeof(buffer), fifo_occupancy);
                set_bit(CFHSI_FLUSH_FIFO, &cfhsi->bits);
-               ret = cfhsi->dev->cfhsi_rx(buffer, fifo_occupancy,
-                               cfhsi->dev);
+               ret = cfhsi->ops->cfhsi_rx(buffer, fifo_occupancy,
+                               cfhsi->ops);
                if (ret) {
                        clear_bit(CFHSI_FLUSH_FIFO, &cfhsi->bits);
-                       dev_warn(&cfhsi->ndev->dev,
+                       netdev_warn(cfhsi->ndev,
                                "%s: can't read data: %d.\n",
                                __func__, ret);
                        break;
@@ -213,13 +199,13 @@ static int cfhsi_flush_fifo(struct cfhsi *cfhsi)
                         !test_bit(CFHSI_FLUSH_FIFO, &cfhsi->bits), ret);
 
                if (ret < 0) {
-                       dev_warn(&cfhsi->ndev->dev,
+                       netdev_warn(cfhsi->ndev,
                                "%s: can't wait for flush complete: %d.\n",
                                __func__, ret);
                        break;
                } else if (!ret) {
                        ret = -ETIMEDOUT;
-                       dev_warn(&cfhsi->ndev->dev,
+                       netdev_warn(cfhsi->ndev,
                                "%s: timeout waiting for flush complete.\n",
                                __func__);
                        break;
@@ -246,14 +232,14 @@ static int cfhsi_tx_frm(struct cfhsi_desc *desc, struct cfhsi *cfhsi)
        /* Check if we can embed a CAIF frame. */
        if (skb->len < CFHSI_MAX_EMB_FRM_SZ) {
                struct caif_payload_info *info;
-               int hpad = 0;
-               int tpad = 0;
+               int hpad;
+               int tpad;
 
                /* Calculate needed head alignment and tail alignment. */
                info = (struct caif_payload_info *)&skb->cb;
 
-               hpad = 1 + PAD_POW2((info->hdr_len + 1), hsi_head_align);
-               tpad = PAD_POW2((skb->len + hpad), hsi_tail_align);
+               hpad = 1 + PAD_POW2((info->hdr_len + 1), cfhsi->cfg.head_align);
+               tpad = PAD_POW2((skb->len + hpad), cfhsi->cfg.tail_align);
 
                /* Check if frame still fits with added alignment. */
                if ((skb->len + hpad + tpad) <= CFHSI_MAX_EMB_FRM_SZ) {
@@ -282,8 +268,8 @@ static int cfhsi_tx_frm(struct cfhsi_desc *desc, struct cfhsi *cfhsi)
        pfrm = desc->emb_frm + CFHSI_MAX_EMB_FRM_SZ;
        while (nfrms < CFHSI_MAX_PKTS) {
                struct caif_payload_info *info;
-               int hpad = 0;
-               int tpad = 0;
+               int hpad;
+               int tpad;
 
                if (!skb)
                        skb = cfhsi_dequeue(cfhsi);
@@ -294,8 +280,8 @@ static int cfhsi_tx_frm(struct cfhsi_desc *desc, struct cfhsi *cfhsi)
                /* Calculate needed head alignment and tail alignment. */
                info = (struct caif_payload_info *)&skb->cb;
 
-               hpad = 1 + PAD_POW2((info->hdr_len + 1), hsi_head_align);
-               tpad = PAD_POW2((skb->len + hpad), hsi_tail_align);
+               hpad = 1 + PAD_POW2((info->hdr_len + 1), cfhsi->cfg.head_align);
+               tpad = PAD_POW2((skb->len + hpad), cfhsi->cfg.tail_align);
 
                /* Fill in CAIF frame length in descriptor. */
                desc->cffrm_len[nfrms] = hpad + skb->len + tpad;
@@ -348,7 +334,7 @@ static void cfhsi_start_tx(struct cfhsi *cfhsi)
        struct cfhsi_desc *desc = (struct cfhsi_desc *)cfhsi->tx_buf;
        int len, res;
 
-       dev_dbg(&cfhsi->ndev->dev, "%s.\n", __func__);
+       netdev_dbg(cfhsi->ndev, "%s.\n", __func__);
 
        if (test_bit(CFHSI_SHUTDOWN, &cfhsi->bits))
                return;
@@ -366,22 +352,22 @@ static void cfhsi_start_tx(struct cfhsi *cfhsi)
                        cfhsi->tx_state = CFHSI_TX_STATE_IDLE;
                        /* Start inactivity timer. */
                        mod_timer(&cfhsi->inactivity_timer,
-                               jiffies + cfhsi->inactivity_timeout);
+                               jiffies + cfhsi->cfg.inactivity_timeout);
                        spin_unlock_bh(&cfhsi->lock);
                        break;
                }
 
                /* Set up new transfer. */
-               res = cfhsi->dev->cfhsi_tx(cfhsi->tx_buf, len, cfhsi->dev);
+               res = cfhsi->ops->cfhsi_tx(cfhsi->tx_buf, len, cfhsi->ops);
                if (WARN_ON(res < 0))
-                       dev_err(&cfhsi->ndev->dev, "%s: TX error %d.\n",
+                       netdev_err(cfhsi->ndev, "%s: TX error %d.\n",
                                __func__, res);
        } while (res < 0);
 }
 
 static void cfhsi_tx_done(struct cfhsi *cfhsi)
 {
-       dev_dbg(&cfhsi->ndev->dev, "%s.\n", __func__);
+       netdev_dbg(cfhsi->ndev, "%s.\n", __func__);
 
        if (test_bit(CFHSI_SHUTDOWN, &cfhsi->bits))
                return;
@@ -392,7 +378,7 @@ static void cfhsi_tx_done(struct cfhsi *cfhsi)
         */
        spin_lock_bh(&cfhsi->lock);
        if (cfhsi->flow_off_sent &&
-                       cfhsi_tx_queue_len(cfhsi) <= cfhsi->q_low_mark &&
+                       cfhsi_tx_queue_len(cfhsi) <= cfhsi->cfg.q_low_mark &&
                        cfhsi->cfdev.flowctrl) {
 
                cfhsi->flow_off_sent = 0;
@@ -404,19 +390,19 @@ static void cfhsi_tx_done(struct cfhsi *cfhsi)
                cfhsi_start_tx(cfhsi);
        } else {
                mod_timer(&cfhsi->aggregation_timer,
-                       jiffies + cfhsi->aggregation_timeout);
+                       jiffies + cfhsi->cfg.aggregation_timeout);
                spin_unlock_bh(&cfhsi->lock);
        }
 
        return;
 }
 
-static void cfhsi_tx_done_cb(struct cfhsi_drv *drv)
+static void cfhsi_tx_done_cb(struct cfhsi_cb_ops *cb_ops)
 {
        struct cfhsi *cfhsi;
 
-       cfhsi = container_of(drv, struct cfhsi, drv);
-       dev_dbg(&cfhsi->ndev->dev, "%s.\n",
+       cfhsi = container_of(cb_ops, struct cfhsi, cb_ops);
+       netdev_dbg(cfhsi->ndev, "%s.\n",
                __func__);
 
        if (test_bit(CFHSI_SHUTDOWN, &cfhsi->bits))
@@ -433,7 +419,7 @@ static int cfhsi_rx_desc(struct cfhsi_desc *desc, struct cfhsi *cfhsi)
 
        if ((desc->header & ~CFHSI_PIGGY_DESC) ||
                        (desc->offset > CFHSI_MAX_EMB_FRM_SZ)) {
-               dev_err(&cfhsi->ndev->dev, "%s: Invalid descriptor.\n",
+               netdev_err(cfhsi->ndev, "%s: Invalid descriptor.\n",
                        __func__);
                return -EPROTO;
        }
@@ -455,7 +441,7 @@ static int cfhsi_rx_desc(struct cfhsi_desc *desc, struct cfhsi *cfhsi)
 
                /* Sanity check length of CAIF frame. */
                if (unlikely(len > CFHSI_MAX_CAIF_FRAME_SZ)) {
-                       dev_err(&cfhsi->ndev->dev, "%s: Invalid length.\n",
+                       netdev_err(cfhsi->ndev, "%s: Invalid length.\n",
                                __func__);
                        return -EPROTO;
                }
@@ -463,7 +449,7 @@ static int cfhsi_rx_desc(struct cfhsi_desc *desc, struct cfhsi *cfhsi)
                /* Allocate SKB (OK even in IRQ context). */
                skb = alloc_skb(len + 1, GFP_ATOMIC);
                if (!skb) {
-                       dev_err(&cfhsi->ndev->dev, "%s: Out of memory !\n",
+                       netdev_err(cfhsi->ndev, "%s: Out of memory !\n",
                                __func__);
                        return -ENOMEM;
                }
@@ -477,8 +463,8 @@ static int cfhsi_rx_desc(struct cfhsi_desc *desc, struct cfhsi *cfhsi)
                skb->dev = cfhsi->ndev;
 
                /*
-                * We are called from a arch specific platform device.
-                * Unfortunately we don't know what context we're
+                * We are in a callback handler and
+                * unfortunately we don't know what context we're
                 * running in.
                 */
                if (in_interrupt())
@@ -504,7 +490,7 @@ static int cfhsi_rx_desc(struct cfhsi_desc *desc, struct cfhsi *cfhsi)
                xfer_sz += CFHSI_DESC_SZ;
 
        if ((xfer_sz % 4) || (xfer_sz > (CFHSI_BUF_SZ_RX - CFHSI_DESC_SZ))) {
-               dev_err(&cfhsi->ndev->dev,
+               netdev_err(cfhsi->ndev,
                                "%s: Invalid payload len: %d, ignored.\n",
                        __func__, xfer_sz);
                return -EPROTO;
@@ -551,7 +537,7 @@ static int cfhsi_rx_pld(struct cfhsi_desc *desc, struct cfhsi *cfhsi)
        /* Sanity check header and offset. */
        if (WARN_ON((desc->header & ~CFHSI_PIGGY_DESC) ||
                        (desc->offset > CFHSI_MAX_EMB_FRM_SZ))) {
-               dev_err(&cfhsi->ndev->dev, "%s: Invalid descriptor.\n",
+               netdev_err(cfhsi->ndev, "%s: Invalid descriptor.\n",
                        __func__);
                return -EPROTO;
        }
@@ -573,7 +559,7 @@ static int cfhsi_rx_pld(struct cfhsi_desc *desc, struct cfhsi *cfhsi)
                struct sk_buff *skb;
                u8 *dst = NULL;
                u8 *pcffrm = NULL;
-               int len = 0;
+               int len;
 
                /* CAIF frame starts after head padding. */
                pcffrm = pfrm + *pfrm + 1;
@@ -585,7 +571,7 @@ static int cfhsi_rx_pld(struct cfhsi_desc *desc, struct cfhsi *cfhsi)
 
                /* Sanity check length of CAIF frames. */
                if (unlikely(len > CFHSI_MAX_CAIF_FRAME_SZ)) {
-                       dev_err(&cfhsi->ndev->dev, "%s: Invalid length.\n",
+                       netdev_err(cfhsi->ndev, "%s: Invalid length.\n",
                                __func__);
                        return -EPROTO;
                }
@@ -593,7 +579,7 @@ static int cfhsi_rx_pld(struct cfhsi_desc *desc, struct cfhsi *cfhsi)
                /* Allocate SKB (OK even in IRQ context). */
                skb = alloc_skb(len + 1, GFP_ATOMIC);
                if (!skb) {
-                       dev_err(&cfhsi->ndev->dev, "%s: Out of memory !\n",
+                       netdev_err(cfhsi->ndev, "%s: Out of memory !\n",
                                __func__);
                        cfhsi->rx_state.nfrms = nfrms;
                        return -ENOMEM;
@@ -608,7 +594,7 @@ static int cfhsi_rx_pld(struct cfhsi_desc *desc, struct cfhsi *cfhsi)
                skb->dev = cfhsi->ndev;
 
                /*
-                * We're called from a platform device,
+                * We're called in callback from HSI
                 * and don't know the context we're running in.
                 */
                if (in_interrupt())
@@ -639,7 +625,7 @@ static void cfhsi_rx_done(struct cfhsi *cfhsi)
 
        desc = (struct cfhsi_desc *)cfhsi->rx_buf;
 
-       dev_dbg(&cfhsi->ndev->dev, "%s\n", __func__);
+       netdev_dbg(cfhsi->ndev, "%s\n", __func__);
 
        if (test_bit(CFHSI_SHUTDOWN, &cfhsi->bits))
                return;
@@ -647,7 +633,7 @@ static void cfhsi_rx_done(struct cfhsi *cfhsi)
        /* Update inactivity timer if pending. */
        spin_lock_bh(&cfhsi->lock);
        mod_timer_pending(&cfhsi->inactivity_timer,
-                       jiffies + cfhsi->inactivity_timeout);
+                       jiffies + cfhsi->cfg.inactivity_timeout);
        spin_unlock_bh(&cfhsi->lock);
 
        if (cfhsi->rx_state.state == CFHSI_RX_STATE_DESC) {
@@ -680,12 +666,11 @@ static void cfhsi_rx_done(struct cfhsi *cfhsi)
                        if (desc_pld_len < 0)
                                goto out_of_sync;
 
-                       if (desc_pld_len > 0)
+                       if (desc_pld_len > 0) {
                                rx_len = desc_pld_len;
-
-                       if (desc_pld_len > 0 &&
-                                       (piggy_desc->header & CFHSI_PIGGY_DESC))
-                               rx_len += CFHSI_DESC_SZ;
+                               if (piggy_desc->header & CFHSI_PIGGY_DESC)
+                                       rx_len += CFHSI_DESC_SZ;
+                       }
 
                        /*
                         * Copy needed information from the piggy-backed
@@ -693,8 +678,6 @@ static void cfhsi_rx_done(struct cfhsi *cfhsi)
                         */
                        memcpy(rx_buf, (u8 *)piggy_desc,
                                        CFHSI_DESC_SHORT_SZ);
-                       if (desc_pld_len == -EPROTO)
-                               goto out_of_sync;
                }
        }
 
@@ -710,13 +693,13 @@ static void cfhsi_rx_done(struct cfhsi *cfhsi)
        /* Initiate next read */
        if (test_bit(CFHSI_AWAKE, &cfhsi->bits)) {
                /* Set up new transfer. */
-               dev_dbg(&cfhsi->ndev->dev, "%s: Start RX.\n",
+               netdev_dbg(cfhsi->ndev, "%s: Start RX.\n",
                                __func__);
 
-               res = cfhsi->dev->cfhsi_rx(rx_ptr, rx_len,
-                               cfhsi->dev);
+               res = cfhsi->ops->cfhsi_rx(rx_ptr, rx_len,
+                               cfhsi->ops);
                if (WARN_ON(res < 0)) {
-                       dev_err(&cfhsi->ndev->dev, "%s: RX error %d.\n",
+                       netdev_err(cfhsi->ndev, "%s: RX error %d.\n",
                                __func__, res);
                        cfhsi->ndev->stats.rx_errors++;
                        cfhsi->ndev->stats.rx_dropped++;
@@ -753,7 +736,7 @@ static void cfhsi_rx_done(struct cfhsi *cfhsi)
        return;
 
 out_of_sync:
-       dev_err(&cfhsi->ndev->dev, "%s: Out of sync.\n", __func__);
+       netdev_err(cfhsi->ndev, "%s: Out of sync.\n", __func__);
        print_hex_dump_bytes("--> ", DUMP_PREFIX_NONE,
                        cfhsi->rx_buf, CFHSI_DESC_SZ);
        schedule_work(&cfhsi->out_of_sync_work);
@@ -763,18 +746,18 @@ static void cfhsi_rx_slowpath(unsigned long arg)
 {
        struct cfhsi *cfhsi = (struct cfhsi *)arg;
 
-       dev_dbg(&cfhsi->ndev->dev, "%s.\n",
+       netdev_dbg(cfhsi->ndev, "%s.\n",
                __func__);
 
        cfhsi_rx_done(cfhsi);
 }
 
-static void cfhsi_rx_done_cb(struct cfhsi_drv *drv)
+static void cfhsi_rx_done_cb(struct cfhsi_cb_ops *cb_ops)
 {
        struct cfhsi *cfhsi;
 
-       cfhsi = container_of(drv, struct cfhsi, drv);
-       dev_dbg(&cfhsi->ndev->dev, "%s.\n",
+       cfhsi = container_of(cb_ops, struct cfhsi, cb_ops);
+       netdev_dbg(cfhsi->ndev, "%s.\n",
                __func__);
 
        if (test_bit(CFHSI_SHUTDOWN, &cfhsi->bits))
@@ -807,9 +790,9 @@ static void cfhsi_wake_up(struct work_struct *work)
        }
 
        /* Activate wake line. */
-       cfhsi->dev->cfhsi_wake_up(cfhsi->dev);
+       cfhsi->ops->cfhsi_wake_up(cfhsi->ops);
 
-       dev_dbg(&cfhsi->ndev->dev, "%s: Start waiting.\n",
+       netdev_dbg(cfhsi->ndev, "%s: Start waiting.\n",
                __func__);
 
        /* Wait for acknowledge. */
@@ -819,33 +802,33 @@ static void cfhsi_wake_up(struct work_struct *work)
                                                        &cfhsi->bits), ret);
        if (unlikely(ret < 0)) {
                /* Interrupted by signal. */
-               dev_err(&cfhsi->ndev->dev, "%s: Signalled: %ld.\n",
+               netdev_err(cfhsi->ndev, "%s: Signalled: %ld.\n",
                        __func__, ret);
 
                clear_bit(CFHSI_WAKE_UP, &cfhsi->bits);
-               cfhsi->dev->cfhsi_wake_down(cfhsi->dev);
+               cfhsi->ops->cfhsi_wake_down(cfhsi->ops);
                return;
        } else if (!ret) {
                bool ca_wake = false;
                size_t fifo_occupancy = 0;
 
                /* Wakeup timeout */
-               dev_dbg(&cfhsi->ndev->dev, "%s: Timeout.\n",
+               netdev_dbg(cfhsi->ndev, "%s: Timeout.\n",
                        __func__);
 
                /* Check FIFO to check if modem has sent something. */
-               WARN_ON(cfhsi->dev->cfhsi_fifo_occupancy(cfhsi->dev,
+               WARN_ON(cfhsi->ops->cfhsi_fifo_occupancy(cfhsi->ops,
                                        &fifo_occupancy));
 
-               dev_dbg(&cfhsi->ndev->dev, "%s: Bytes in FIFO: %u.\n",
+               netdev_dbg(cfhsi->ndev, "%s: Bytes in FIFO: %u.\n",
                                __func__, (unsigned) fifo_occupancy);
 
                /* Check if we misssed the interrupt. */
-               WARN_ON(cfhsi->dev->cfhsi_get_peer_wake(cfhsi->dev,
+               WARN_ON(cfhsi->ops->cfhsi_get_peer_wake(cfhsi->ops,
                                                        &ca_wake));
 
                if (ca_wake) {
-                       dev_err(&cfhsi->ndev->dev, "%s: CA Wake missed !.\n",
+                       netdev_err(cfhsi->ndev, "%s: CA Wake missed !.\n",
                                __func__);
 
                        /* Clear the CFHSI_WAKE_UP_ACK bit to prevent race. */
@@ -856,11 +839,11 @@ static void cfhsi_wake_up(struct work_struct *work)
                }
 
                clear_bit(CFHSI_WAKE_UP, &cfhsi->bits);
-               cfhsi->dev->cfhsi_wake_down(cfhsi->dev);
+               cfhsi->ops->cfhsi_wake_down(cfhsi->ops);
                return;
        }
 wake_ack:
-       dev_dbg(&cfhsi->ndev->dev, "%s: Woken.\n",
+       netdev_dbg(cfhsi->ndev, "%s: Woken.\n",
                __func__);
 
        /* Clear power up bit. */
@@ -868,11 +851,11 @@ wake_ack:
        clear_bit(CFHSI_WAKE_UP, &cfhsi->bits);
 
        /* Resume read operation. */
-       dev_dbg(&cfhsi->ndev->dev, "%s: Start RX.\n", __func__);
-       res = cfhsi->dev->cfhsi_rx(cfhsi->rx_ptr, cfhsi->rx_len, cfhsi->dev);
+       netdev_dbg(cfhsi->ndev, "%s: Start RX.\n", __func__);
+       res = cfhsi->ops->cfhsi_rx(cfhsi->rx_ptr, cfhsi->rx_len, cfhsi->ops);
 
        if (WARN_ON(res < 0))
-               dev_err(&cfhsi->ndev->dev, "%s: RX err %d.\n", __func__, res);
+               netdev_err(cfhsi->ndev, "%s: RX err %d.\n", __func__, res);
 
        /* Clear power up acknowledment. */
        clear_bit(CFHSI_WAKE_UP_ACK, &cfhsi->bits);
@@ -881,16 +864,16 @@ wake_ack:
 
        /* Resume transmit if queues are not empty. */
        if (!cfhsi_tx_queue_len(cfhsi)) {
-               dev_dbg(&cfhsi->ndev->dev, "%s: Peer wake, start timer.\n",
+               netdev_dbg(cfhsi->ndev, "%s: Peer wake, start timer.\n",
                        __func__);
                /* Start inactivity timer. */
                mod_timer(&cfhsi->inactivity_timer,
-                               jiffies + cfhsi->inactivity_timeout);
+                               jiffies + cfhsi->cfg.inactivity_timeout);
                spin_unlock_bh(&cfhsi->lock);
                return;
        }
 
-       dev_dbg(&cfhsi->ndev->dev, "%s: Host wake.\n",
+       netdev_dbg(cfhsi->ndev, "%s: Host wake.\n",
                __func__);
 
        spin_unlock_bh(&cfhsi->lock);
@@ -900,14 +883,14 @@ wake_ack:
 
        if (likely(len > 0)) {
                /* Set up new transfer. */
-               res = cfhsi->dev->cfhsi_tx(cfhsi->tx_buf, len, cfhsi->dev);
+               res = cfhsi->ops->cfhsi_tx(cfhsi->tx_buf, len, cfhsi->ops);
                if (WARN_ON(res < 0)) {
-                       dev_err(&cfhsi->ndev->dev, "%s: TX error %d.\n",
+                       netdev_err(cfhsi->ndev, "%s: TX error %d.\n",
                                __func__, res);
                        cfhsi_abort_tx(cfhsi);
                }
        } else {
-               dev_err(&cfhsi->ndev->dev,
+               netdev_err(cfhsi->ndev,
                                "%s: Failed to create HSI frame: %d.\n",
                                __func__, len);
        }
@@ -921,13 +904,13 @@ static void cfhsi_wake_down(struct work_struct *work)
        int retry = CFHSI_WAKE_TOUT;
 
        cfhsi = container_of(work, struct cfhsi, wake_down_work);
-       dev_dbg(&cfhsi->ndev->dev, "%s.\n", __func__);
+       netdev_dbg(cfhsi->ndev, "%s.\n", __func__);
 
        if (test_bit(CFHSI_SHUTDOWN, &cfhsi->bits))
                return;
 
        /* Deactivate wake line. */
-       cfhsi->dev->cfhsi_wake_down(cfhsi->dev);
+       cfhsi->ops->cfhsi_wake_down(cfhsi->ops);
 
        /* Wait for acknowledge. */
        ret = CFHSI_WAKE_TOUT;
@@ -936,26 +919,26 @@ static void cfhsi_wake_down(struct work_struct *work)
                                                        &cfhsi->bits), ret);
        if (ret < 0) {
                /* Interrupted by signal. */
-               dev_err(&cfhsi->ndev->dev, "%s: Signalled: %ld.\n",
+               netdev_err(cfhsi->ndev, "%s: Signalled: %ld.\n",
                        __func__, ret);
                return;
        } else if (!ret) {
                bool ca_wake = true;
 
                /* Timeout */
-               dev_err(&cfhsi->ndev->dev, "%s: Timeout.\n", __func__);
+               netdev_err(cfhsi->ndev, "%s: Timeout.\n", __func__);
 
                /* Check if we misssed the interrupt. */
-               WARN_ON(cfhsi->dev->cfhsi_get_peer_wake(cfhsi->dev,
+               WARN_ON(cfhsi->ops->cfhsi_get_peer_wake(cfhsi->ops,
                                                        &ca_wake));
                if (!ca_wake)
-                       dev_err(&cfhsi->ndev->dev, "%s: CA Wake missed !.\n",
+                       netdev_err(cfhsi->ndev, "%s: CA Wake missed !.\n",
                                __func__);
        }
 
        /* Check FIFO occupancy. */
        while (retry) {
-               WARN_ON(cfhsi->dev->cfhsi_fifo_occupancy(cfhsi->dev,
+               WARN_ON(cfhsi->ops->cfhsi_fifo_occupancy(cfhsi->ops,
                                                        &fifo_occupancy));
 
                if (!fifo_occupancy)
@@ -967,14 +950,13 @@ static void cfhsi_wake_down(struct work_struct *work)
        }
 
        if (!retry)
-               dev_err(&cfhsi->ndev->dev, "%s: FIFO Timeout.\n", __func__);
+               netdev_err(cfhsi->ndev, "%s: FIFO Timeout.\n", __func__);
 
        /* Clear AWAKE condition. */
        clear_bit(CFHSI_AWAKE, &cfhsi->bits);
 
        /* Cancel pending RX requests. */
-       cfhsi->dev->cfhsi_rx_cancel(cfhsi->dev);
-
+       cfhsi->ops->cfhsi_rx_cancel(cfhsi->ops);
 }
 
 static void cfhsi_out_of_sync(struct work_struct *work)
@@ -988,12 +970,12 @@ static void cfhsi_out_of_sync(struct work_struct *work)
        rtnl_unlock();
 }
 
-static void cfhsi_wake_up_cb(struct cfhsi_drv *drv)
+static void cfhsi_wake_up_cb(struct cfhsi_cb_ops *cb_ops)
 {
        struct cfhsi *cfhsi = NULL;
 
-       cfhsi = container_of(drv, struct cfhsi, drv);
-       dev_dbg(&cfhsi->ndev->dev, "%s.\n",
+       cfhsi = container_of(cb_ops, struct cfhsi, cb_ops);
+       netdev_dbg(cfhsi->ndev, "%s.\n",
                __func__);
 
        set_bit(CFHSI_WAKE_UP_ACK, &cfhsi->bits);
@@ -1007,12 +989,12 @@ static void cfhsi_wake_up_cb(struct cfhsi_drv *drv)
                queue_work(cfhsi->wq, &cfhsi->wake_up_work);
 }
 
-static void cfhsi_wake_down_cb(struct cfhsi_drv *drv)
+static void cfhsi_wake_down_cb(struct cfhsi_cb_ops *cb_ops)
 {
        struct cfhsi *cfhsi = NULL;
 
-       cfhsi = container_of(drv, struct cfhsi, drv);
-       dev_dbg(&cfhsi->ndev->dev, "%s.\n",
+       cfhsi = container_of(cb_ops, struct cfhsi, cb_ops);
+       netdev_dbg(cfhsi->ndev, "%s.\n",
                __func__);
 
        /* Initiating low power is only permitted by the host (us). */
@@ -1024,7 +1006,7 @@ static void cfhsi_aggregation_tout(unsigned long arg)
 {
        struct cfhsi *cfhsi = (struct cfhsi *)arg;
 
-       dev_dbg(&cfhsi->ndev->dev, "%s.\n",
+       netdev_dbg(cfhsi->ndev, "%s.\n",
                __func__);
 
        cfhsi_start_tx(cfhsi);
@@ -1077,7 +1059,7 @@ static int cfhsi_xmit(struct sk_buff *skb, struct net_device *dev)
 
        /* Send flow off if number of packets is above high water mark. */
        if (!cfhsi->flow_off_sent &&
-               cfhsi_tx_queue_len(cfhsi) > cfhsi->q_high_mark &&
+               cfhsi_tx_queue_len(cfhsi) > cfhsi->cfg.q_high_mark &&
                cfhsi->cfdev.flowctrl) {
                cfhsi->flow_off_sent = 1;
                cfhsi->cfdev.flowctrl(cfhsi->ndev, OFF);
@@ -1114,9 +1096,9 @@ static int cfhsi_xmit(struct sk_buff *skb, struct net_device *dev)
                WARN_ON(!len);
 
                /* Set up new transfer. */
-               res = cfhsi->dev->cfhsi_tx(cfhsi->tx_buf, len, cfhsi->dev);
+               res = cfhsi->ops->cfhsi_tx(cfhsi->tx_buf, len, cfhsi->ops);
                if (WARN_ON(res < 0)) {
-                       dev_err(&cfhsi->ndev->dev, "%s: TX error %d.\n",
+                       netdev_err(cfhsi->ndev, "%s: TX error %d.\n",
                                __func__, res);
                        cfhsi_abort_tx(cfhsi);
                }
@@ -1129,19 +1111,19 @@ static int cfhsi_xmit(struct sk_buff *skb, struct net_device *dev)
        return 0;
 }
 
-static const struct net_device_ops cfhsi_ops;
+static const struct net_device_ops cfhsi_netdevops;
 
 static void cfhsi_setup(struct net_device *dev)
 {
        int i;
        struct cfhsi *cfhsi = netdev_priv(dev);
        dev->features = 0;
-       dev->netdev_ops = &cfhsi_ops;
        dev->type = ARPHRD_CAIF;
        dev->flags = IFF_POINTOPOINT | IFF_NOARP;
        dev->mtu = CFHSI_MAX_CAIF_FRAME_SZ;
        dev->tx_queue_len = 0;
        dev->destructor = free_netdev;
+       dev->netdev_ops = &cfhsi_netdevops;
        for (i = 0; i < CFHSI_PRIO_LAST; ++i)
                skb_queue_head_init(&cfhsi->qhead[i]);
        cfhsi->cfdev.link_select = CAIF_LINK_HIGH_BANDW;
@@ -1149,43 +1131,7 @@ static void cfhsi_setup(struct net_device *dev)
        cfhsi->cfdev.use_stx = false;
        cfhsi->cfdev.use_fcs = false;
        cfhsi->ndev = dev;
-}
-
-int cfhsi_probe(struct platform_device *pdev)
-{
-       struct cfhsi *cfhsi = NULL;
-       struct net_device *ndev;
-
-       int res;
-
-       ndev = alloc_netdev(sizeof(struct cfhsi), "cfhsi%d", cfhsi_setup);
-       if (!ndev)
-               return -ENODEV;
-
-       cfhsi = netdev_priv(ndev);
-       cfhsi->ndev = ndev;
-       cfhsi->pdev = pdev;
-
-       /* Assign the HSI device. */
-       cfhsi->dev = pdev->dev.platform_data;
-
-       /* Assign the driver to this HSI device. */
-       cfhsi->dev->drv = &cfhsi->drv;
-
-       /* Register network device. */
-       res = register_netdev(ndev);
-       if (res) {
-               dev_err(&ndev->dev, "%s: Registration error: %d.\n",
-                       __func__, res);
-               free_netdev(ndev);
-               return -ENODEV;
-       }
-       /* Add CAIF HSI device to list. */
-       spin_lock(&cfhsi_list_lock);
-       list_add_tail(&cfhsi->list, &cfhsi_list);
-       spin_unlock(&cfhsi_list_lock);
-
-       return res;
+       cfhsi->cfg = hsi_default_config;
 }
 
 static int cfhsi_open(struct net_device *ndev)
@@ -1201,9 +1147,6 @@ static int cfhsi_open(struct net_device *ndev)
 
        /* Set flow info */
        cfhsi->flow_off_sent = 0;
-       cfhsi->q_low_mark = LOW_WATER_MARK;
-       cfhsi->q_high_mark = HIGH_WATER_MARK;
-
 
        /*
         * Allocate a TX buffer with the size of a HSI packet descriptors
@@ -1231,20 +1174,8 @@ static int cfhsi_open(struct net_device *ndev)
                goto err_alloc_rx_flip;
        }
 
-       /* Pre-calculate inactivity timeout. */
-       if (inactivity_timeout != -1) {
-               cfhsi->inactivity_timeout =
-                               inactivity_timeout * HZ / 1000;
-               if (!cfhsi->inactivity_timeout)
-                       cfhsi->inactivity_timeout = 1;
-               else if (cfhsi->inactivity_timeout > NEXT_TIMER_MAX_DELTA)
-                       cfhsi->inactivity_timeout = NEXT_TIMER_MAX_DELTA;
-       } else {
-               cfhsi->inactivity_timeout = NEXT_TIMER_MAX_DELTA;
-       }
-
        /* Initialize aggregation timeout */
-       cfhsi->aggregation_timeout = aggregation_timeout;
+       cfhsi->cfg.aggregation_timeout = hsi_default_config.aggregation_timeout;
 
        /* Initialize recieve vaiables. */
        cfhsi->rx_ptr = cfhsi->rx_buf;
@@ -1254,10 +1185,10 @@ static int cfhsi_open(struct net_device *ndev)
        spin_lock_init(&cfhsi->lock);
 
        /* Set up the driver. */
-       cfhsi->drv.tx_done_cb = cfhsi_tx_done_cb;
-       cfhsi->drv.rx_done_cb = cfhsi_rx_done_cb;
-       cfhsi->drv.wake_up_cb = cfhsi_wake_up_cb;
-       cfhsi->drv.wake_down_cb = cfhsi_wake_down_cb;
+       cfhsi->cb_ops.tx_done_cb = cfhsi_tx_done_cb;
+       cfhsi->cb_ops.rx_done_cb = cfhsi_rx_done_cb;
+       cfhsi->cb_ops.wake_up_cb = cfhsi_wake_up_cb;
+       cfhsi->cb_ops.wake_down_cb = cfhsi_wake_down_cb;
 
        /* Initialize the work queues. */
        INIT_WORK(&cfhsi->wake_up_work, cfhsi_wake_up);
@@ -1271,9 +1202,9 @@ static int cfhsi_open(struct net_device *ndev)
        clear_bit(CFHSI_AWAKE, &cfhsi->bits);
 
        /* Create work thread. */
-       cfhsi->wq = create_singlethread_workqueue(cfhsi->pdev->name);
+       cfhsi->wq = create_singlethread_workqueue(cfhsi->ndev->name);
        if (!cfhsi->wq) {
-               dev_err(&cfhsi->ndev->dev, "%s: Failed to create work queue.\n",
+               netdev_err(cfhsi->ndev, "%s: Failed to create work queue.\n",
                        __func__);
                res = -ENODEV;
                goto err_create_wq;
@@ -1298,9 +1229,9 @@ static int cfhsi_open(struct net_device *ndev)
        cfhsi->aggregation_timer.function = cfhsi_aggregation_tout;
 
        /* Activate HSI interface. */
-       res = cfhsi->dev->cfhsi_up(cfhsi->dev);
+       res = cfhsi->ops->cfhsi_up(cfhsi->ops);
        if (res) {
-               dev_err(&cfhsi->ndev->dev,
+               netdev_err(cfhsi->ndev,
                        "%s: can't activate HSI interface: %d.\n",
                        __func__, res);
                goto err_activate;
@@ -1309,14 +1240,14 @@ static int cfhsi_open(struct net_device *ndev)
        /* Flush FIFO */
        res = cfhsi_flush_fifo(cfhsi);
        if (res) {
-               dev_err(&cfhsi->ndev->dev, "%s: Can't flush FIFO: %d.\n",
+               netdev_err(cfhsi->ndev, "%s: Can't flush FIFO: %d.\n",
                        __func__, res);
                goto err_net_reg;
        }
        return res;
 
  err_net_reg:
-       cfhsi->dev->cfhsi_down(cfhsi->dev);
+       cfhsi->ops->cfhsi_down(cfhsi->ops);
  err_activate:
        destroy_workqueue(cfhsi->wq);
  err_create_wq:
@@ -1346,7 +1277,7 @@ static int cfhsi_close(struct net_device *ndev)
        del_timer_sync(&cfhsi->aggregation_timer);
 
        /* Cancel pending RX request (if any) */
-       cfhsi->dev->cfhsi_rx_cancel(cfhsi->dev);
+       cfhsi->ops->cfhsi_rx_cancel(cfhsi->ops);
 
        /* Destroy workqueue */
        destroy_workqueue(cfhsi->wq);
@@ -1359,7 +1290,7 @@ static int cfhsi_close(struct net_device *ndev)
        cfhsi_abort_tx(cfhsi);
 
        /* Deactivate interface */
-       cfhsi->dev->cfhsi_down(cfhsi->dev);
+       cfhsi->ops->cfhsi_down(cfhsi->ops);
 
        /* Free buffers. */
        kfree(tx_buf);
@@ -1368,85 +1299,184 @@ static int cfhsi_close(struct net_device *ndev)
        return 0;
 }
 
-static const struct net_device_ops cfhsi_ops = {
+static void cfhsi_uninit(struct net_device *dev)
+{
+       struct cfhsi *cfhsi = netdev_priv(dev);
+       ASSERT_RTNL();
+       symbol_put(cfhsi_get_device);
+       list_del(&cfhsi->list);
+}
+
+static const struct net_device_ops cfhsi_netdevops = {
+       .ndo_uninit = cfhsi_uninit,
        .ndo_open = cfhsi_open,
        .ndo_stop = cfhsi_close,
        .ndo_start_xmit = cfhsi_xmit
 };
 
-int cfhsi_remove(struct platform_device *pdev)
+static void cfhsi_netlink_parms(struct nlattr *data[], struct cfhsi *cfhsi)
 {
-       struct list_head *list_node;
-       struct list_head *n;
-       struct cfhsi *cfhsi = NULL;
-       struct cfhsi_dev *dev;
+       int i;
 
-       dev = (struct cfhsi_dev *)pdev->dev.platform_data;
-       spin_lock(&cfhsi_list_lock);
-       list_for_each_safe(list_node, n, &cfhsi_list) {
-               cfhsi = list_entry(list_node, struct cfhsi, list);
-               /* Find the corresponding device. */
-               if (cfhsi->dev == dev) {
-                       /* Remove from list. */
-                       list_del(list_node);
-                       spin_unlock(&cfhsi_list_lock);
-                       return 0;
-               }
+       if (!data) {
+               pr_debug("no params data found\n");
+               return;
        }
-       spin_unlock(&cfhsi_list_lock);
-       return -ENODEV;
+
+       i = __IFLA_CAIF_HSI_INACTIVITY_TOUT;
+       /*
+        * Inactivity timeout in millisecs. Lowest possible value is 1,
+        * and highest possible is NEXT_TIMER_MAX_DELTA.
+        */
+       if (data[i]) {
+               u32 inactivity_timeout = nla_get_u32(data[i]);
+               /* Pre-calculate inactivity timeout. */
+               cfhsi->cfg.inactivity_timeout = inactivity_timeout * HZ / 1000;
+               if (cfhsi->cfg.inactivity_timeout == 0)
+                       cfhsi->cfg.inactivity_timeout = 1;
+               else if (cfhsi->cfg.inactivity_timeout > NEXT_TIMER_MAX_DELTA)
+                       cfhsi->cfg.inactivity_timeout = NEXT_TIMER_MAX_DELTA;
+       }
+
+       i = __IFLA_CAIF_HSI_AGGREGATION_TOUT;
+       if (data[i])
+               cfhsi->cfg.aggregation_timeout = nla_get_u32(data[i]);
+
+       i = __IFLA_CAIF_HSI_HEAD_ALIGN;
+       if (data[i])
+               cfhsi->cfg.head_align = nla_get_u32(data[i]);
+
+       i = __IFLA_CAIF_HSI_TAIL_ALIGN;
+       if (data[i])
+               cfhsi->cfg.tail_align = nla_get_u32(data[i]);
+
+       i = __IFLA_CAIF_HSI_QHIGH_WATERMARK;
+       if (data[i])
+               cfhsi->cfg.q_high_mark = nla_get_u32(data[i]);
+
+       i = __IFLA_CAIF_HSI_QLOW_WATERMARK;
+       if (data[i])
+               cfhsi->cfg.q_low_mark = nla_get_u32(data[i]);
+}
+
+static int caif_hsi_changelink(struct net_device *dev, struct nlattr *tb[],
+                               struct nlattr *data[])
+{
+       cfhsi_netlink_parms(data, netdev_priv(dev));
+       netdev_state_change(dev);
+       return 0;
 }
 
-struct platform_driver cfhsi_plat_drv = {
-       .probe = cfhsi_probe,
-       .remove = cfhsi_remove,
-       .driver = {
-                  .name = "cfhsi",
-                  .owner = THIS_MODULE,
-                  },
+static const struct nla_policy caif_hsi_policy[__IFLA_CAIF_HSI_MAX + 1] = {
+       [__IFLA_CAIF_HSI_INACTIVITY_TOUT] = { .type = NLA_U32, .len = 4 },
+       [__IFLA_CAIF_HSI_AGGREGATION_TOUT] = { .type = NLA_U32, .len = 4 },
+       [__IFLA_CAIF_HSI_HEAD_ALIGN] = { .type = NLA_U32, .len = 4 },
+       [__IFLA_CAIF_HSI_TAIL_ALIGN] = { .type = NLA_U32, .len = 4 },
+       [__IFLA_CAIF_HSI_QHIGH_WATERMARK] = { .type = NLA_U32, .len = 4 },
+       [__IFLA_CAIF_HSI_QLOW_WATERMARK] = { .type = NLA_U32, .len = 4 },
 };
 
-static void __exit cfhsi_exit_module(void)
+static size_t caif_hsi_get_size(const struct net_device *dev)
+{
+       int i;
+       size_t s = 0;
+       for (i = __IFLA_CAIF_HSI_UNSPEC + 1; i < __IFLA_CAIF_HSI_MAX; i++)
+               s += nla_total_size(caif_hsi_policy[i].len);
+       return s;
+}
+
+static int caif_hsi_fill_info(struct sk_buff *skb, const struct net_device *dev)
+{
+       struct cfhsi *cfhsi = netdev_priv(dev);
+
+       if (nla_put_u32(skb, __IFLA_CAIF_HSI_INACTIVITY_TOUT,
+                       cfhsi->cfg.inactivity_timeout) ||
+           nla_put_u32(skb, __IFLA_CAIF_HSI_AGGREGATION_TOUT,
+                       cfhsi->cfg.aggregation_timeout) ||
+           nla_put_u32(skb, __IFLA_CAIF_HSI_HEAD_ALIGN,
+                       cfhsi->cfg.head_align) ||
+           nla_put_u32(skb, __IFLA_CAIF_HSI_TAIL_ALIGN,
+                       cfhsi->cfg.tail_align) ||
+           nla_put_u32(skb, __IFLA_CAIF_HSI_QHIGH_WATERMARK,
+                       cfhsi->cfg.q_high_mark) ||
+           nla_put_u32(skb, __IFLA_CAIF_HSI_QLOW_WATERMARK,
+                       cfhsi->cfg.q_low_mark))
+               return -EMSGSIZE;
+
+       return 0;
+}
+
+static int caif_hsi_newlink(struct net *src_net, struct net_device *dev,
+                         struct nlattr *tb[], struct nlattr *data[])
 {
-       struct list_head *list_node;
-       struct list_head *n;
        struct cfhsi *cfhsi = NULL;
+       struct cfhsi_ops *(*get_ops)(void);
 
-       spin_lock(&cfhsi_list_lock);
-       list_for_each_safe(list_node, n, &cfhsi_list) {
-               cfhsi = list_entry(list_node, struct cfhsi, list);
+       ASSERT_RTNL();
 
-               /* Remove from list. */
-               list_del(list_node);
-               spin_unlock(&cfhsi_list_lock);
+       cfhsi = netdev_priv(dev);
+       cfhsi_netlink_parms(data, cfhsi);
+       dev_net_set(cfhsi->ndev, src_net);
+
+       get_ops = symbol_get(cfhsi_get_ops);
+       if (!get_ops) {
+               pr_err("%s: failed to get the cfhsi_ops\n", __func__);
+               return -ENODEV;
+       }
 
-               unregister_netdevice(cfhsi->ndev);
+       /* Assign the HSI device. */
+       cfhsi->ops = (*get_ops)();
+       if (!cfhsi->ops) {
+               pr_err("%s: failed to get the cfhsi_ops\n", __func__);
+               goto err;
+       }
 
-               spin_lock(&cfhsi_list_lock);
+       /* Assign the driver to this HSI device. */
+       cfhsi->ops->cb_ops = &cfhsi->cb_ops;
+       if (register_netdevice(dev)) {
+               pr_warn("%s: caif_hsi device registration failed\n", __func__);
+               goto err;
        }
-       spin_unlock(&cfhsi_list_lock);
+       /* Add CAIF HSI device to list. */
+       list_add_tail(&cfhsi->list, &cfhsi_list);
 
-       /* Unregister platform driver. */
-       platform_driver_unregister(&cfhsi_plat_drv);
+       return 0;
+err:
+       symbol_put(cfhsi_get_ops);
+       return -ENODEV;
 }
 
-static int __init cfhsi_init_module(void)
+static struct rtnl_link_ops caif_hsi_link_ops __read_mostly = {
+       .kind           = "cfhsi",
+       .priv_size      = sizeof(struct cfhsi),
+       .setup          = cfhsi_setup,
+       .maxtype        = __IFLA_CAIF_HSI_MAX,
+       .policy = caif_hsi_policy,
+       .newlink        = caif_hsi_newlink,
+       .changelink     = caif_hsi_changelink,
+       .get_size       = caif_hsi_get_size,
+       .fill_info      = caif_hsi_fill_info,
+};
+
+static void __exit cfhsi_exit_module(void)
 {
-       int result;
+       struct list_head *list_node;
+       struct list_head *n;
+       struct cfhsi *cfhsi;
 
-       /* Initialize spin lock. */
-       spin_lock_init(&cfhsi_list_lock);
+       rtnl_link_unregister(&caif_hsi_link_ops);
 
-       /* Register platform driver. */
-       result = platform_driver_register(&cfhsi_plat_drv);
-       if (result) {
-               printk(KERN_ERR "Could not register platform HSI driver: %d.\n",
-                       result);
-               goto err_dev_register;
+       rtnl_lock();
+       list_for_each_safe(list_node, n, &cfhsi_list) {
+               cfhsi = list_entry(list_node, struct cfhsi, list);
+               unregister_netdev(cfhsi->ndev);
        }
+       rtnl_unlock();
+}
 
- err_dev_register:
-       return result;
+static int __init cfhsi_init_module(void)
+{
+       return rtnl_link_register(&caif_hsi_link_ops);
 }
 
 module_init(cfhsi_init_module);