Re: [PATCH] gadget: added besl support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 9/4/2015 3:58 AM, Joao Pinto wrote:
> When testing the Synopsys DWC USB 3.0 Device IP, Compliant Tests are run 
> for USB 3.0 and USB 2.0. An issue was found regarding the USB 2.0 CV 
> Chapter 9 test: LPM L1 Suspend Resume Test. For it to be successful it 
> is necessary to add besl support to the gadget driver and set the deep 
> besl and baseline besl values.
> 
> Signed-off-by: Joao Pinto <jpinto@xxxxxxxxxxxx>
> Tested-by: Joao Pinto <jpinto@xxxxxxxxxxxx>
> ---
>  drivers/usb/gadget/composite.c | 12 +++++++++++-
>  1 file changed, 11 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/usb/gadget/composite.c b/drivers/usb/gadget/composite.c
> index b474499..e907578 100644
> --- a/drivers/usb/gadget/composite.c
> +++ b/drivers/usb/gadget/composite.c
> @@ -567,7 +567,14 @@ static int bos_desc(struct usb_composite_dev *cdev)
>  	usb_ext->bLength = USB_DT_USB_EXT_CAP_SIZE;
>  	usb_ext->bDescriptorType = USB_DT_DEVICE_CAPABILITY;
>  	usb_ext->bDevCapabilityType = USB_CAP_TYPE_EXT;
> -	usb_ext->bmAttributes = cpu_to_le32(USB_LPM_SUPPORT | USB_BESL_SUPPORT);
> +	usb_ext->bmAttributes = cpu_to_le32(USB_LPM_SUPPORT |
> +				USB_BESL_SUPPORT |
> +				USB_BESL_BASELINE_VALID |
> +				USB_BESL_DEEP_VALID);
> +
> +	usb_ext->bmAttributes &= cpu_to_le32(~(0x00f00 | 0x0f000));
> +	usb_ext->bmAttributes |= cpu_to_le32(4 << 8);
> +	usb_ext->bmAttributes |= cpu_to_le32(5 << 12);
>  
>  	/*
>  	 * The Superspeed USB Capability descriptor shall be implemented by all
> 

Hi Joao,

These values should only apply to certain platforms.

We can discuss off-list.

Regards,
John




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



[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux