From: Of Vincent Pelletier
> Sent: 23 January 2017 14:41
> It is allowed by the USB specification to enabled same-address, opposite-
> direction endpoints simultaneously, which means 30 non-zero endpoints
> are allowed. So double eps_addrmap length to 30.
> The original code only accepted 14 descriptors out of a likely intended 15
> (as there are 15 endpoint addresses, ignoring direction), because the first
> eps_addrmap entry is unused (it is a placeholder for endpoint zero). So
> increase eps_addrmap length by one to 31.
>
> Signed-off-by: Vincent Pelletier <[email protected]>
> ---
> drivers/usb/gadget/function/f_fs.c | 2 +-
> drivers/usb/gadget/function/u_fs.h | 3 ++-
> 2 files changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/usb/gadget/function/f_fs.c
> b/drivers/usb/gadget/function/f_fs.c
> index 5c91a6f4613b..c46666364c87 100644
> --- a/drivers/usb/gadget/function/f_fs.c
> +++ b/drivers/usb/gadget/function/f_fs.c
> @@ -2101,7 +2101,7 @@ static int __ffs_data_do_entity(enum ffs_entity_type
> type,
> case FFS_ENDPOINT:
> d = (void *)desc;
> helper->eps_count++;
> - if (helper->eps_count >= 15)
> + if (helper->eps_count >= FFS_MAX_EPS_COUNT)
Use ARRAY_COUNT(helper->eps_addrmap)
...
David
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
