media: staging: atomisp: Switch to use struct device_driver directly
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Tue, 19 Dec 2017 20:59:53 +0000 (15:59 -0500)
committerMauro Carvalho Chehab <mchehab@s-opensource.com>
Fri, 29 Dec 2017 11:27:34 +0000 (06:27 -0500)
In a preparation of split PCI glue driver from core part, convert
the driver to use more generic struct device_driver.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
drivers/staging/media/atomisp/pci/atomisp2/atomisp_drvfs.c
drivers/staging/media/atomisp/pci/atomisp2/atomisp_drvfs.h
drivers/staging/media/atomisp/pci/atomisp2/atomisp_v4l2.c

index 7129b88456cb5e4aa21c73cbe36757f01c7b77cd..ceedb82b6beb76cc9646878916d6284ebc2ddcd6 100644 (file)
@@ -15,9 +15,9 @@
  *
  */
 
+#include <linux/device.h>
 #include <linux/err.h>
 #include <linux/kernel.h>
-#include <linux/pci.h>
 
 #include "atomisp_compat.h"
 #include "atomisp_internal.h"
@@ -33,7 +33,7 @@
  *        bit 2: memory statistic
 */
 struct _iunit_debug {
-       struct pci_driver       *drv;
+       struct device_driver    *drv;
        struct atomisp_device   *isp;
        unsigned int            dbglvl;
        unsigned int            dbgfun;
@@ -164,26 +164,25 @@ static const struct driver_attribute iunit_drvfs_attrs[] = {
        __ATTR(dbgopt, 0644, iunit_dbgopt_show, iunit_dbgopt_store),
 };
 
-static int iunit_drvfs_create_files(struct pci_driver *drv)
+static int iunit_drvfs_create_files(struct device_driver *drv)
 {
        int i, ret = 0;
 
        for (i = 0; i < ARRAY_SIZE(iunit_drvfs_attrs); i++)
-               ret |= driver_create_file(&(drv->driver),
-                                       &iunit_drvfs_attrs[i]);
+               ret |= driver_create_file(drv, &iunit_drvfs_attrs[i]);
 
        return ret;
 }
 
-static void iunit_drvfs_remove_files(struct pci_driver *drv)
+static void iunit_drvfs_remove_files(struct device_driver *drv)
 {
        int i;
 
        for (i = 0; i < ARRAY_SIZE(iunit_drvfs_attrs); i++)
-               driver_remove_file(&(drv->driver), &iunit_drvfs_attrs[i]);
+               driver_remove_file(drv, &iunit_drvfs_attrs[i]);
 }
 
-int atomisp_drvfs_init(struct pci_driver *drv, struct atomisp_device *isp)
+int atomisp_drvfs_init(struct device_driver *drv, struct atomisp_device *isp)
 {
        int ret;
 
@@ -193,7 +192,7 @@ int atomisp_drvfs_init(struct pci_driver *drv, struct atomisp_device *isp)
        ret = iunit_drvfs_create_files(iunit_debug.drv);
        if (ret) {
                dev_err(atomisp_dev, "drvfs_create_files error: %d\n", ret);
-               iunit_drvfs_remove_files(drv);
+               iunit_drvfs_remove_files(iunit_debug.drv);
        }
 
        return ret;
index b91bfef21639c195f835b9e99a827133f8684c4d..7c99240d107a37aeb041c38343dd91c745940deb 100644 (file)
@@ -18,8 +18,7 @@
 #ifndef        __ATOMISP_DRVFS_H__
 #define        __ATOMISP_DRVFS_H__
 
-extern int atomisp_drvfs_init(struct pci_driver *drv, struct atomisp_device
-                               *isp);
-extern void atomisp_drvfs_exit(void);
+int atomisp_drvfs_init(struct device_driver *drv, struct atomisp_device *isp);
+void atomisp_drvfs_exit(void);
 
 #endif /* __ATOMISP_DRVFS_H__ */
index 233ab661599a8edf71c957a80bc8784399ae8251..58f1a3597805034e1ed09899124aecf5812164df 100644 (file)
@@ -1152,8 +1152,6 @@ alloc_fail:
        return err;
 }
 
-static struct pci_driver atomisp_pci_driver;
-
 #define ATOM_ISP_PCI_BAR       0
 
 static int atomisp_pci_probe(struct pci_dev *dev,
@@ -1451,7 +1449,7 @@ static int atomisp_pci_probe(struct pci_dev *dev,
        isp->firmware = NULL;
        isp->css_env.isp_css_fw.data = NULL;
 
-       atomisp_drvfs_init(&atomisp_pci_driver, isp);
+       atomisp_drvfs_init(&dev->driver->driver, isp);
 
        return 0;