From: Michal Nazarewicz <mina86@mina86•com>
To: Heiko Schocher <hs@denx•de>, linux-usb@vger•kernel.org
Cc: Heiko Schocher <hs@denx•de>, Felipe Balbi <balbi@ti•com>,
Greg Kroah-Hartman <gregkh@suse•de>,
linux-kernel@vger•kernel.org, Oliver Neukum <oliver@neukum•name>,
netdev@vger•kernel.org, linux-api@vger•kernel.org,
Andrzej Pietrasiewicz <andrzej.p@samsung•com>,
Kyungmin Park <kyungmin.park@samsung•com>,
Dan Carpenter <dan.carpenter@oracle•com>,
Macpaul Lin <macpaul@gmail•com>
Subject: Re: [PATCH] usb: gadget: f_rndis: fix usb_interface_descriptor for rndis
Date: Wed, 24 Sep 2014 11:38:40 +0200 [thread overview]
Message-ID: <xa1tbnq59x3z.fsf@mina86.com> (raw)
In-Reply-To: <1411541339-32400-1-git-send-email-hs@denx.de>
On Wed, Sep 24 2014, Heiko Schocher <hs@denx•de> wrote:
> use the values for RNDIS over Ethernet as defined in
> http://www.usb.org/developers/defined_class
> (search for RDNIS):
>
> - baseclass: 0xef (miscellaneous)
> - subclass: 0x04
> - protocol: 0x01
>
> with this setings the file in Documentation/usb/linux.inf is
> obsolete.
>
> Signed-off-by: Heiko Schocher <hs@denx•de>
>
> ---
>
> Cc: Felipe Balbi <balbi@ti•com>
> Cc: Greg Kroah-Hartman <gregkh@suse•de>
> Cc: linux-usb@vger•kernel.org
> Cc: linux-kernel@vger•kernel.org
> Cc: Oliver Neukum <oliver@neukum•name>
> Cc: netdev@vger•kernel.org
> Cc: linux-api@vger•kernel.org
> Cc: Andrzej Pietrasiewicz <andrzej.p@samsung•com>
> Cc: Michal Nazarewicz <mina86@mina86•com>
> Cc: Kyungmin Park <kyungmin.park@samsung•com>
> Cc: Dan Carpenter <dan.carpenter@oracle•com>
> Cc: Macpaul Lin <macpaul@gmail•com>
>
> Tested with the "USB Compliance test suite which runs Windows", see:
> http://www.usb.org/developers/tools/usb20_tools/#usb20cv
>
> drivers/net/usb/cdc_ether.c | 6 +++---
> drivers/usb/core/generic.c | 6 +++---
> drivers/usb/gadget/function/f_rndis.c | 6 +++---
> include/uapi/linux/usb/cdc.h | 3 +++
> 4 files changed, 12 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/net/usb/cdc_ether.c b/drivers/net/usb/cdc_ether.c
> index 2a32d91..9c216c2 100644
> --- a/drivers/net/usb/cdc_ether.c
> +++ b/drivers/net/usb/cdc_ether.c
> @@ -35,9 +35,9 @@
>
> static int is_rndis(struct usb_interface_descriptor *desc)
> {
> - return (desc->bInterfaceClass == USB_CLASS_COMM &&
> - desc->bInterfaceSubClass == 2 &&
> - desc->bInterfaceProtocol == 0xff);
> + return (desc->bInterfaceClass == USB_CLASS_MISC &&
> + desc->bInterfaceSubClass == USB_CDC_SUBCLASS_RNDIS &&
> + desc->bInterfaceProtocol == USB_CDC_RNDIS_PROTO_ETH);
> }
Does that mean that new kernels will stop working with old RNDIs
gadgets because they stop recognising them as RNDIS? I feel like this
function should accept both, i.e.:
return (desc->bInterfaceClass == USB_CLASS_COMM &&
desc->bInterfaceSubClass == 2 &&
desc->bInterfaceProtocol == 0xff) ||
(desc->bInterfaceClass == USB_CLASS_MISC &&
desc->bInterfaceSubClass == USB_CDC_SUBCLASS_RNDIS &&
desc->bInterfaceProtocol == USB_CDC_RNDIS_PROTO_ETH);
>
> static int is_activesync(struct usb_interface_descriptor *desc)
> diff --git a/drivers/usb/core/generic.c b/drivers/usb/core/generic.c
> index 358ca8d..a2a4e05 100644
> --- a/drivers/usb/core/generic.c
> +++ b/drivers/usb/core/generic.c
> @@ -28,9 +28,9 @@ static inline const char *plural(int n)
>
> static int is_rndis(struct usb_interface_descriptor *desc)
> {
> - return desc->bInterfaceClass == USB_CLASS_COMM
> - && desc->bInterfaceSubClass == 2
> - && desc->bInterfaceProtocol == 0xff;
> + return desc->bInterfaceClass == USB_CLASS_MISC
> + && desc->bInterfaceSubClass == USB_CDC_SUBCLASS_RNDIS
> + && desc->bInterfaceProtocol == USB_CDC_RNDIS_PROTO_ETH;
> }
Ditto.
>
> static int is_activesync(struct usb_interface_descriptor *desc)
> diff --git a/drivers/usb/gadget/function/f_rndis.c b/drivers/usb/gadget/function/f_rndis.c
> index ddb09dc..cc06046 100644
> --- a/drivers/usb/gadget/function/f_rndis.c
> +++ b/drivers/usb/gadget/function/f_rndis.c
> @@ -117,9 +117,9 @@ static struct usb_interface_descriptor rndis_control_intf = {
> /* .bInterfaceNumber = DYNAMIC */
> /* status endpoint is optional; this could be patched later */
> .bNumEndpoints = 1,
> - .bInterfaceClass = USB_CLASS_COMM,
> - .bInterfaceSubClass = USB_CDC_SUBCLASS_ACM,
> - .bInterfaceProtocol = USB_CDC_ACM_PROTO_VENDOR,
> + .bInterfaceClass = USB_CLASS_MISC,
> + .bInterfaceSubClass = USB_CDC_SUBCLASS_RNDIS,
> + .bInterfaceProtocol = USB_CDC_RNDIS_PROTO_ETH,
> /* .iInterface = DYNAMIC */
> };
>
> diff --git a/include/uapi/linux/usb/cdc.h b/include/uapi/linux/usb/cdc.h
> index b6a9cdd..8e8fc85 100644
> --- a/include/uapi/linux/usb/cdc.h
> +++ b/include/uapi/linux/usb/cdc.h
> @@ -12,6 +12,7 @@
> #include <linux/types.h>
>
> #define USB_CDC_SUBCLASS_ACM 0x02
> +#define USB_CDC_SUBCLASS_RNDIS 0x04
> #define USB_CDC_SUBCLASS_ETHERNET 0x06
> #define USB_CDC_SUBCLASS_WHCM 0x08
> #define USB_CDC_SUBCLASS_DMM 0x09
> @@ -31,6 +32,8 @@
> #define USB_CDC_ACM_PROTO_AT_CDMA 6
> #define USB_CDC_ACM_PROTO_VENDOR 0xff
>
> +#define USB_CDC_RNDIS_PROTO_ETH 1
> +
> #define USB_CDC_PROTO_EEM 7
>
> #define USB_CDC_NCM_PROTO_NTB 1
--
Best regards, _ _
.o. | Liege of Serenely Enlightened Majesty of o' \,=./ `o
..o | Computer Science, Michał “mina86” Nazarewicz (o o)
ooo +--<mpn@google.com>--<xmpp:mina86@jabber•org>--ooO--(_)--Ooo--
next prev parent reply other threads:[~2014-09-24 9:38 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-24 6:48 [PATCH] usb: gadget: f_rndis: fix usb_interface_descriptor for rndis Heiko Schocher
2014-09-24 9:38 ` Michal Nazarewicz [this message]
2014-09-24 13:12 ` Heiko Schocher
[not found] ` <1411541339-32400-1-git-send-email-hs-ynQEQJNshbs@public.gmane.org>
2014-09-24 12:25 ` Lars Melin
2014-09-24 13:12 ` Heiko Schocher
[not found] ` <5422C33F.400-ynQEQJNshbs@public.gmane.org>
2014-09-24 14:22 ` Lars Melin
2014-09-29 12:11 ` Heiko Schocher
2014-09-29 16:05 ` Lars Melin
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=xa1tbnq59x3z.fsf@mina86.com \
--to=mina86@mina86$(echo .)com \
--cc=andrzej.p@samsung$(echo .)com \
--cc=balbi@ti$(echo .)com \
--cc=dan.carpenter@oracle$(echo .)com \
--cc=gregkh@suse$(echo .)de \
--cc=hs@denx$(echo .)de \
--cc=kyungmin.park@samsung$(echo .)com \
--cc=linux-api@vger$(echo .)kernel.org \
--cc=linux-kernel@vger$(echo .)kernel.org \
--cc=linux-usb@vger$(echo .)kernel.org \
--cc=macpaul@gmail$(echo .)com \
--cc=netdev@vger$(echo .)kernel.org \
--cc=oliver@neukum$(echo .)name \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox