greybus: reorder greybus_svc_in() arguments
authorAlex Elder <elder@linaro.org>
Sat, 27 Sep 2014 01:55:35 +0000 (20:55 -0500)
committerGreg Kroah-Hartman <greg@kroah.com>
Tue, 30 Sep 2014 01:08:14 +0000 (18:08 -0700)
The two functions greybus_svc_in() and greybus_cport_in() do
very similar things, but their arguments are in a different order.
Move the greybus_host_device structure argument for greybus_svc_in()
to be first so the functions' prototypes are better aligned.

Signed-off-by: Alex Elder <elder@linaro.org>
Signed-off-by: Greg Kroah-Hartman <greg@kroah.com>
drivers/staging/greybus/ap.c
drivers/staging/greybus/es1-ap-usb.c
drivers/staging/greybus/greybus.h

index e368429..f4470b5 100644 (file)
@@ -294,7 +294,7 @@ static void ap_process_event(struct work_struct *work)
        kfree(ap_msg);
 }
 
-int greybus_svc_in(u8 *data, int size, struct greybus_host_device *hd)
+int greybus_svc_in(struct greybus_host_device *hd, u8 *data, int size)
 {
        struct ap_msg *ap_msg;
 
index b6ab124..291da40 100644 (file)
@@ -268,7 +268,7 @@ static void svc_in_callback(struct urb *urb)
        /* We have a message, create a new message structure, add it to the
         * list, and wake up our thread that will process the messages.
         */
-       greybus_svc_in(urb->transfer_buffer, urb->actual_length, es1->hd);
+       greybus_svc_in(es1->hd, urb->transfer_buffer, urb->actual_length);
 
 exit:
        /* resubmit the urb to get more messages */
index 7948220..1328dc9 100644 (file)
@@ -289,7 +289,7 @@ void gb_add_module(struct greybus_host_device *hd, u8 module_id,
                   u8 *data, int size);
 void gb_remove_module(struct greybus_host_device *hd, u8 module_id);
 
-int greybus_svc_in(u8 *data, int length, struct greybus_host_device *hd);
+int greybus_svc_in(struct greybus_host_device *hd, u8 *data, int length);
 int gb_ap_init(void);
 void gb_ap_exit(void);
 int gb_debugfs_init(void);