]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
mei: bus: remove rx callback context
authorTomas Winkler <tomas.winkler@intel.com>
Wed, 19 Oct 2016 13:33:31 +0000 (16:33 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 28 Oct 2016 12:21:22 +0000 (08:21 -0400)
The callback context is redunant as all the information can be
retrived from the device struture of its private data.

Signed-off-by: Tomas Winkler <tomas.winkler@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/misc/mei/bus.c
drivers/nfc/mei_phy.c
drivers/watchdog/mei_wdt.c
include/linux/mei_cl_bus.h

index 8cac7ef9ad0d98789c3f910dced1b61bc2b3ab62..89a694ca624c33642fe71ab10f0077404a7c4dbd 100644 (file)
@@ -228,7 +228,7 @@ static void mei_cl_bus_event_work(struct work_struct *work)
        bus = cldev->bus;
 
        if (cldev->event_cb)
-               cldev->event_cb(cldev, cldev->events, cldev->event_context);
+               cldev->event_cb(cldev, cldev->events);
 
        cldev->events = 0;
 
@@ -301,7 +301,6 @@ bool mei_cl_bus_rx_event(struct mei_cl *cl)
  * @cldev: me client devices
  * @event_cb: callback function
  * @events_mask: requested events bitmask
- * @context: driver context data
  *
  * Return: 0 on success
  *         -EALREADY if an callback is already registered
@@ -309,7 +308,7 @@ bool mei_cl_bus_rx_event(struct mei_cl *cl)
  */
 int mei_cldev_register_event_cb(struct mei_cl_device *cldev,
                                unsigned long events_mask,
-                               mei_cldev_event_cb_t event_cb, void *context)
+                               mei_cldev_event_cb_t event_cb)
 {
        struct mei_device *bus = cldev->bus;
        int ret;
@@ -320,7 +319,6 @@ int mei_cldev_register_event_cb(struct mei_cl_device *cldev,
        cldev->events = 0;
        cldev->events_mask = events_mask;
        cldev->event_cb = event_cb;
-       cldev->event_context = context;
        INIT_WORK(&cldev->event_work, mei_cl_bus_event_work);
 
        if (cldev->events_mask & BIT(MEI_CL_EVENT_RX)) {
index 66dfd81ffb1800dc973235a4a71b8ec5f573cdde..07b4239585facd64e9b226b59de62932c15d6135 100644 (file)
@@ -297,8 +297,7 @@ static int mei_nfc_recv(struct nfc_mei_phy *phy, u8 *buf, size_t length)
 }
 
 
-static void nfc_mei_event_cb(struct mei_cl_device *cldev, u32 events,
-                            void *context)
+static void nfc_mei_event_cb(struct mei_cl_device *cldev, u32 events)
 {
        struct nfc_mei_phy *phy = mei_cldev_get_drvdata(cldev);
 
@@ -360,7 +359,7 @@ static int nfc_mei_phy_enable(void *phy_id)
        }
 
        r = mei_cldev_register_event_cb(phy->cldev, BIT(MEI_CL_EVENT_RX),
-                                    nfc_mei_event_cb, phy);
+                                       nfc_mei_event_cb);
        if (r) {
                pr_err("Event cb registration failed %d\n", r);
                goto err;
index 116be477c8fdbb59e24d308e38c1b48d2738af80..e0af52265511ff7181cfbd881d4701580cefc744 100644 (file)
@@ -501,10 +501,8 @@ static void mei_wdt_notify_event(struct mei_cl_device *cldev)
  *
  * @cldev: bus device
  * @events: event mask
- * @context: callback context
  */
-static void mei_wdt_event(struct mei_cl_device *cldev,
-                         u32 events, void *context)
+static void mei_wdt_event(struct mei_cl_device *cldev, u32 events)
 {
        if (events & BIT(MEI_CL_EVENT_RX))
                mei_wdt_event_rx(cldev);
@@ -626,7 +624,7 @@ static int mei_wdt_probe(struct mei_cl_device *cldev,
        ret = mei_cldev_register_event_cb(wdt->cldev,
                                          BIT(MEI_CL_EVENT_RX) |
                                          BIT(MEI_CL_EVENT_NOTIF),
-                                         mei_wdt_event, NULL);
+                                         mei_wdt_event);
 
        /* on legacy devices notification is not supported
         * this doesn't fail the registration for RX event
index e6fbd98ea90eaf69a85fe0fd81bd866a85221f90..4adb2e7c9f8471648ad580892aa3fd1a558430be 100644 (file)
@@ -9,7 +9,7 @@ struct mei_cl_device;
 struct mei_device;
 
 typedef void (*mei_cldev_event_cb_t)(struct mei_cl_device *cldev,
-                                    u32 events, void *context);
+                                    u32 events);
 
 /**
  * struct mei_cl_device - MEI device handle
@@ -27,7 +27,6 @@ typedef void (*mei_cldev_event_cb_t)(struct mei_cl_device *cldev,
  * @event_work: async work to execute event callback
  * @event_cb: Drivers register this callback to get asynchronous ME
  *     events (e.g. Rx buffer pending) notifications.
- * @event_context: event callback run context
  * @events_mask: Events bit mask requested by driver.
  * @events: Events bitmask sent to the driver.
  *
@@ -46,7 +45,6 @@ struct mei_cl_device {
 
        struct work_struct event_work;
        mei_cldev_event_cb_t event_cb;
-       void *event_context;
        unsigned long events_mask;
        unsigned long events;
 
@@ -92,7 +90,7 @@ ssize_t  mei_cldev_recv(struct mei_cl_device *cldev, u8 *buf, size_t length);
 
 int mei_cldev_register_event_cb(struct mei_cl_device *cldev,
                                unsigned long event_mask,
-                               mei_cldev_event_cb_t read_cb, void *context);
+                               mei_cldev_event_cb_t read_cb);
 
 #define MEI_CL_EVENT_RX 0
 #define MEI_CL_EVENT_TX 1