All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Łukasz Dałek" <luk0104@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 4/6] usbether: Fixed bug when using with PXA25X chips
Date: Wed, 19 Sep 2012 21:38:56 +0200	[thread overview]
Message-ID: <1348083538-16123-5-git-send-email-luk0104@gmail.com> (raw)
In-Reply-To: <1348083538-16123-1-git-send-email-luk0104@gmail.com>

PXA25X chips don't support alternate settings so driver uses non-CDC
driver.
But only code defined between DEV_CONFIG_CDC signals that network is up.
This patch is fixing this bug by signaling that network is up after USB
SET_INTERFACE request.

Signed-off-by: ?ukasz Da?ek <luk0104@gmail.com>
---
 drivers/usb/gadget/ether.c |   17 ++++++++++++++++-
 1 files changed, 16 insertions(+), 1 deletions(-)

diff --git a/drivers/usb/gadget/ether.c b/drivers/usb/gadget/ether.c
index d975fb6..9ce2c17 100644
--- a/drivers/usb/gadget/ether.c
+++ b/drivers/usb/gadget/ether.c
@@ -44,7 +44,12 @@ extern struct platform_data brd;
 
 unsigned packet_received, packet_sent;
 
-#define DEV_CONFIG_CDC	1
+#ifdef CONFIG_USB_GADGET_PXA2XX
+# undef DEV_CONFIG_CDC
+# define DEV_CONFIG_SUBSET 1
+#else
+# define DEV_CONFIG_CDC	1
+#endif
 #define GFP_ATOMIC ((gfp_t) 0)
 #define GFP_KERNEL ((gfp_t) 0)
 
@@ -864,7 +869,9 @@ static struct usb_gadget_strings	stringtab = {
 
 /*============================================================================*/
 static u8 control_req[USB_BUFSIZ];
+#if defined(DEV_CONFIG_CDC) || defined(CONFIG_USB_ETH_RNDIS)
 static u8 status_req[STATUS_BYTECOUNT] __attribute__ ((aligned(4)));
+#endif
 
 
 /**
@@ -1352,6 +1359,14 @@ eth_setup(struct usb_gadget *gadget, const struct usb_ctrlrequest *ctrl)
 		if (gadget_is_pxa(gadget)) {
 			value = eth_set_config(dev, DEV_CONFIG_VALUE,
 						GFP_ATOMIC);
+			/*
+			 * PXA25x driver use non-CDC ethernet gadget.
+			 * But only _CDC and _RNDIS code can signalize
+			 * that network is working. So we signalize it
+			 * here.
+			 */
+			l_ethdev.network_started = 1;
+			debug("USB network up!\n");
 			goto done_set_intf;
 		}
 
-- 
1.7.8.6

  parent reply	other threads:[~2012-09-19 19:38 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-19 19:38 [U-Boot] [PATCH 0/6] iPAQ h2200 support Łukasz Dałek
2012-09-19 19:38 ` [U-Boot] [PATCH 1/6] h2200: Add support for iPAQ h2200 Łukasz Dałek
2012-09-19 21:02   ` Marek Vasut
     [not found]     ` <505A3B2B.6080907@gmail.com>
2012-09-19 21:43       ` Marek Vasut
2012-09-19 21:54         ` Łukasz Dałek
2012-09-19 22:01           ` Marek Vasut
2012-09-19 22:16             ` Łukasz Dałek
2012-09-19 22:42               ` Marek Vasut
2012-09-19 22:51                 ` Łukasz Dałek
2012-09-19 22:56                   ` Marek Vasut
2012-09-19 23:07                     ` Łukasz Dałek
2012-09-19 23:09                       ` Marek Vasut
2012-09-19 23:20                         ` Łukasz Dałek
2012-09-19 23:25                           ` Marek Vasut
2012-09-19 23:36                             ` Łukasz Dałek
2012-09-19 23:43                               ` Marek Vasut
2012-09-19 23:51                                 ` Łukasz Dałek
2012-09-20  0:06                                   ` Marek Vasut
2012-09-20  5:49     ` Joe Hershberger
2012-09-20 12:14       ` Marek Vasut
2012-09-22 17:02     ` Łukasz Dałek
2012-09-22 17:04       ` Marek Vasut
2012-09-22 17:16         ` Łukasz Dałek
2012-09-22 18:00           ` Marek Vasut
2012-09-22 18:07             ` Łukasz Dałek
2012-09-22 18:21               ` Marek Vasut
2012-09-22 18:34                 ` Łukasz Dałek
2012-09-22 18:37                   ` Marek Vasut
2012-09-22 18:47                     ` Łukasz Dałek
2012-09-19 19:38 ` [U-Boot] [PATCH 2/6] pxa25x: Add USB Ethernet driver Łukasz Dałek
2012-09-19 21:03   ` Marek Vasut
     [not found]     ` <505A386D.7080302@gmail.com>
2012-09-19 21:39       ` Marek Vasut
2012-09-19 19:38 ` [U-Boot] [PATCH 3/6] pxa: Add some stuff to examine cpu model and rev Łukasz Dałek
2012-09-19 21:04   ` Marek Vasut
2012-09-19 19:38 ` Łukasz Dałek [this message]
2012-09-19 19:38 ` [U-Boot] [PATCH 5/6] usbether: Define CONFIG_USB_ETH_{CDC,SUBSET} Łukasz Dałek
2012-09-19 19:38 ` [U-Boot] [PATCH 6/6] usbether: Removed DEV_CONFIG_{CDC,SUBSET} Łukasz Dałek
2012-09-19 23:10 ` [U-Boot] [PATCH 0/6] iPAQ h2200 support Marek Vasut
2012-09-20  6:36 ` Andreas Bießmann

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1348083538-16123-5-git-send-email-luk0104@gmail.com \
    --to=luk0104@gmail.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.