# Re: \[PATCH 1/5\] usb: cdc-ncm: Change alternate setting magic numbers into \#defines

- From: Alexey Orishko \<hidden\>
- Date: 2012-02-16 18:34:10 
- List: netdev
- Page: https://ratatoskr.run/netdev/2012/02/10888934
- Thread: https://ratatoskr.run/netdev/2012/02/10888934/t.md

```text
On Wed, Feb 15, 2012 at 3:47 PM, Toby Gray [ref] wrote:

> +/* CDC NCM ch. 5.3 describes alternate setting 0 as having no
> + * endpoints and therefore not allowing any networking traffic. */
> +#define        CDC_NCM_ALTSETTING_RESET        0

Please, use bulk in the both define names to distinguish from control
ep alt settings in the next version of the specification, i.e.
something like CDC_NCM_BULK_ALTSETTING_NO_DATA
This is the setting with no endpoints (no data).

> +/* CDC NCM ch. 5.3 describes alternate setting 1 as having the
> + * required bulk endpoints for normal operation. */
> +#define        CDC_NCM_ALTSETTING_DATA         1

Regards,
Alexey
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
```
