OSDN Git Service

staging: gasket: apex: move sysfs setup code to probe function
authorTodd Poynor <toddpoynor@google.com>
Sun, 5 Aug 2018 20:07:46 +0000 (13:07 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 8 Aug 2018 09:20:07 +0000 (11:20 +0200)
The gasket framework no longer provides callbacks to the device driver
for sysfs setup and teardown.  Move the sysfs setup code to the device
probe function.  Apex does not implement sysfs cleanup code.

Signed-off-by: Todd Poynor <toddpoynor@google.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/gasket/apex_driver.c

index 69ca7fb..5531961 100644 (file)
@@ -568,12 +568,6 @@ static struct gasket_sysfs_attribute apex_sysfs_attrs[] = {
        GASKET_END_OF_ATTR_ARRAY
 };
 
-static int apex_sysfs_setup_cb(struct gasket_dev *gasket_dev)
-{
-       return gasket_sysfs_create_entries(gasket_dev->dev_info.device,
-                                          apex_sysfs_attrs);
-}
-
 /* On device open, perform a core reinit reset. */
 static int apex_device_open_cb(struct gasket_dev *gasket_dev)
 {
@@ -639,6 +633,11 @@ static int apex_pci_probe(struct pci_dev *pci_dev,
                goto remove_device;
        }
 
+       ret = gasket_sysfs_create_entries(gasket_dev->dev_info.device,
+                                         apex_sysfs_attrs);
+       if (ret)
+               dev_err(&pci_dev->dev, "error creating device sysfs entries\n");
+
        ret = gasket_enable_device(gasket_dev);
        if (ret) {
                dev_err(&pci_dev->dev, "error enabling gasket device\n");
@@ -695,9 +694,6 @@ static struct gasket_driver_desc apex_desc = {
        .interrupts = apex_interrupts,
        .interrupt_pack_width = 7,
 
-       .sysfs_setup_cb = apex_sysfs_setup_cb,
-       .sysfs_cleanup_cb = NULL,
-
        .device_open_cb = apex_device_open_cb,
        .device_close_cb = apex_device_cleanup,