RE: [PATCH] usb: phy: mxs: Suggest passing "usbcore.autosuspend=-1" for mx23/mx28

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

 



 
> -----Original Message-----
> From: Fabio Estevam [mailto:festevam@xxxxxxxxx]
> Sent: Wednesday, December 30, 2015 7:34 PM
> To: balbi@xxxxxx
> Cc: Peter Chen <peter.chen@xxxxxxx>; stefan.wahren@xxxxxxxx; linux-
> usb@xxxxxxxxxxxxxxx; Fabio Estevam <fabio.estevam@xxxxxxx>
> Subject: [PATCH] usb: phy: mxs: Suggest passing "usbcore.autosuspend=-1" for
> mx23/mx28
> 
> From: Fabio Estevam <fabio.estevam@xxxxxxx>
> 
> On a mx28 board with a USB hub the following error is observed:
> 
> hub 1-1:1.0: USB hub found
> hub 1-1:1.0: 2 ports detected
> usb 1-1: USB disconnect, device number 2 usb usb1-port1: cannot reset (err = -
> 32) usb usb1-port1: cannot reset (err = -32) usb usb1-port1: cannot reset (err =
> -32) usb usb1-port1: cannot reset (err = -32) usb usb1-port1: cannot reset (err =
> -32) usb usb1-port1: Cannot enable. Maybe the USB cable is bad?
> 
> ,which is caused by a problem described by the
> MXS_PHY_ABNORMAL_IN_SUSPEND flag.
> 
> As we currently do not have a proper implementation for this issue, one
> possible workaround is to pass "usbcore.autosuspend=-1" in the kernel
> command line, so add this suggestion into the
> MXS_PHY_ABNORMAL_IN_SUSPEND flag description.
> 
> Signed-off-by: Fabio Estevam <fabio.estevam@xxxxxxx>
> ---
>  drivers/usb/phy/phy-mxs-usb.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/drivers/usb/phy/phy-mxs-usb.c b/drivers/usb/phy/phy-mxs-usb.c
> index c2936dc..453a5ab 100644
> --- a/drivers/usb/phy/phy-mxs-usb.c
> +++ b/drivers/usb/phy/phy-mxs-usb.c
> @@ -98,6 +98,10 @@
>   * The PHY will be in messy if there is a wakeup after putting
>   * bus to suspend (set portsc.suspendM) but before setting PHY to low
>   * power mode (set portsc.phcd).
> + *
> + * To work around this problem on mx23/mx28 user should pass
> + * "usbcore.autosuspend=-1" in the kernel command line for now, as
> + * we do not have a proper fix for this flag in the kernel yet.
>   */
>  #define MXS_PHY_ABNORMAL_IN_SUSPEND		BIT(1)
> 
> --
> 1.9.1

Acked-by: Peter Chen <peter.chen@xxxxxxx>

--
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