Subject: Re: [PATCH 2/2] CDC_NCM: adding support FLAG_NOARP for Infineon modem platform

From: Sergei Shtylyov <[email-masked]>
Date: Sun, 20 Jan 2013 21:13:27 +0400

Hello.

On 20-01-2013 10:12, Wei Shuai wrote:

> Infineon(now Intel) HSPA Modem platform NCM cannot support ARP. we can define a new common structure wwan_noarp_info.

    Wrap your lines at 76-80 columns maximum please.

> Then more similiar NO ARP devices can be handled easily

> Signed-off-by: Wei Shuai <[email-masked]>
> ---
>   drivers/net/usb/cdc_ncm.c |   21 +++++++++++++++++++++
>   1 files changed, 21 insertions(+), 0 deletions(-)

> diff --git a/drivers/net/usb/cdc_ncm.c b/drivers/net/usb/cdc_ncm.c
> index 71b6e92..2d699b6 100644
> --- a/drivers/net/usb/cdc_ncm.c
> +++ b/drivers/net/usb/cdc_ncm.c
> @@ -1155,6 +1155,20 @@ static const struct driver_info wwan_info = {
>   	.tx_fixup = cdc_ncm_tx_fixup,
>   };
>
> +/* Same as wwan_info, but with IFF_NOARP  */

    FLAG_NOARP, you mean?

> +static const struct driver_info wwan_noarp_info = {
> +	.description = "Mobile Broadband Network Device (NO ARP)",
> +	.flags = FLAG_POINTTOPOINT | FLAG_NO_SETINT | FLAG_MULTI_PACKET
> +			| FLAG_WWAN | FLAG_NOARP,

WBR, Sergei

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to [email-masked]
More majordomo info at  http://vger.kernel.org/majordomo-info.html