Re: [PATCH] usb: dwc3: gadget: Remove FS bInterval_m1 limitation

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

 



Hi,

Thinh Nguyen <Thinh.Nguyen@xxxxxxxxxxxx> writes:
> The programming guide incorrectly stated that the DCFG.bInterval_m1 must
> be set to 0 when operating in fullspeed. There's no such limitation for
> all IPs.

do we have an updated Databook correcting this statement?

> Cc: <stable@xxxxxxxxxxxxxxx>
> Fixes: a1679af85b2a ("usb: dwc3: gadget: Fix setting of DEPCFG.bInterval_m1")
> Signed-off-by: Thinh Nguyen <Thinh.Nguyen@xxxxxxxxxxxx>
> ---
>  drivers/usb/dwc3/gadget.c | 9 +++++----
>  1 file changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/usb/dwc3/gadget.c b/drivers/usb/dwc3/gadget.c
> index 6227641f2d31..d87a29bd7d9b 100644
> --- a/drivers/usb/dwc3/gadget.c
> +++ b/drivers/usb/dwc3/gadget.c
> @@ -608,12 +608,13 @@ static int dwc3_gadget_set_ep_config(struct dwc3_ep *dep, unsigned int action)
>  		u8 bInterval_m1;
>  
>  		/*
> -		 * Valid range for DEPCFG.bInterval_m1 is from 0 to 13, and it
> -		 * must be set to 0 when the controller operates in full-speed.
> +		 * Valid range for DEPCFG.bInterval_m1 is from 0 to 13.
> +		 *
> +		 * NOTE: The programming guide incorrectly stated bInterval_m1
> +		 * must be set to 0 when operating in fullspeed. Internally the
> +		 * controller does not have this limitation.

might be a good idea to refer to the section in this comment ;-)

-- 
balbi

Attachment: signature.asc
Description: PGP signature


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

  Powered by Linux