From: Greg Kroah-Hartman <greg@kroah.com>
Date: Sat, 13 Sep 2014 19:18:09 +0000 (-0700)
Subject: greybus: ap: cleanup of process ap message loop
X-Git-Tag: v4.9-rc1~119^2~378^2~21^2~2117
X-Git-Url: https://git.karo-electronics.de/?a=commitdiff_plain;h=b57b06241b0177a36a1c951b25fdead3bc1d87ba;p=karo-tx-linux.git

greybus: ap: cleanup of process ap message loop
---

diff --git a/drivers/staging/greybus/ap.c b/drivers/staging/greybus/ap.c
index e9aa701ad988..f92066625c1e 100644
--- a/drivers/staging/greybus/ap.c
+++ b/drivers/staging/greybus/ap.c
@@ -169,10 +169,14 @@ static struct svc_msg *convert_ap_message(struct ap_msg *ap_msg)
 	return svc_msg;
 }
 
-static void process_ap_message(struct ap_msg *ap_msg)
+static void ap_process_event(struct work_struct *work)
 {
 	struct svc_msg *svc_msg;
 	struct greybus_host_device *hd;
+	struct ap_msg *ap_msg;
+
+	ap_msg = container_of(work, struct ap_msg, event);
+	hd = ap_msg->hd;
 
 	/* Turn the "raw" data into a real message */
 	svc_msg = convert_ap_message(ap_msg);
@@ -181,11 +185,6 @@ static void process_ap_message(struct ap_msg *ap_msg)
 		return;
 	}
 
-	hd = ap_msg->hd;
-
-	/* Pass the message to the host controller */
-//	ap_msg->hd->driver->ap_msg(svc_msg, ap_msg->hd);
-
 	/* Look at the message to figure out what to do with it */
 	switch (svc_msg->header.type) {
 	case SVC_FUNCTION_HANDSHAKE:
@@ -214,17 +213,6 @@ static void process_ap_message(struct ap_msg *ap_msg)
 			svc_msg->header.type);
 	}
 
-
-}
-
-static void ap_process_event(struct work_struct *work)
-{
-	struct ap_msg *ap_msg;
-
-	ap_msg = container_of(work, struct ap_msg, event);
-
-	process_ap_message(ap_msg);
-
 	/* clean the message up */
 	kfree(ap_msg->data);
 	kfree(ap_msg);