caif-hsi: HSI-Platform device register and unregisters itself
authorDaniel Martensson <daniel.martensson@stericsson.com>
Thu, 13 Oct 2011 11:29:26 +0000 (11:29 +0000)
committerDavid S. Miller <davem@davemloft.net>
Wed, 19 Oct 2011 07:25:41 +0000 (03:25 -0400)
Platform device is no longer removed from caif_hsi at shutdown.
The HSI-platform device must do it's own registration and unregistration.

Signed-off-by: Sjur Brændeland <sjur.brandeland@stericsson.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/caif/caif_hsi.c

index 478b025c9f8be8713ec9eaf94b80081042198ad0..f46ab4dfd7a00337116d7a54e2febf285bebc7a6 100644 (file)
@@ -1083,7 +1083,7 @@ int cfhsi_probe(struct platform_device *pdev)
        return res;
 }
 
-static void cfhsi_shutdown(struct cfhsi *cfhsi, bool remove_platform_dev)
+static void cfhsi_shutdown(struct cfhsi *cfhsi)
 {
        u8 *tx_buf, *rx_buf;
 
@@ -1093,14 +1093,6 @@ static void cfhsi_shutdown(struct cfhsi *cfhsi, bool remove_platform_dev)
        /* going to shutdown driver */
        set_bit(CFHSI_SHUTDOWN, &cfhsi->bits);
 
-       if (remove_platform_dev) {
-               /* Flush workqueue */
-               flush_workqueue(cfhsi->wq);
-
-               /* Notify device. */
-               platform_device_unregister(cfhsi->pdev);
-       }
-
        /* Flush workqueue */
        flush_workqueue(cfhsi->wq);
 
@@ -1111,7 +1103,7 @@ static void cfhsi_shutdown(struct cfhsi *cfhsi, bool remove_platform_dev)
        /* Cancel pending RX request (if any) */
        cfhsi->dev->cfhsi_rx_cancel(cfhsi->dev);
 
-       /* Flush again and destroy workqueue */
+       /* Destroy workqueue */
        destroy_workqueue(cfhsi->wq);
 
        /* Store bufferes: will be freed later. */
@@ -1150,7 +1142,7 @@ int cfhsi_remove(struct platform_device *pdev)
                        spin_unlock(&cfhsi_list_lock);
 
                        /* Shutdown driver. */
-                       cfhsi_shutdown(cfhsi, false);
+                       cfhsi_shutdown(cfhsi);
 
                        return 0;
                }
@@ -1183,7 +1175,7 @@ static void __exit cfhsi_exit_module(void)
                spin_unlock(&cfhsi_list_lock);
 
                /* Shutdown driver. */
-               cfhsi_shutdown(cfhsi, true);
+               cfhsi_shutdown(cfhsi);
 
                spin_lock(&cfhsi_list_lock);
        }