Re: [PATCH] arm64: defconfig: Add Renesas R-Car USB 3.0 driver support

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

 



On Thu, Apr 21, 2016 at 03:41:14AM +0000, Yoshihiro Shimoda wrote:
> Hi Simon-san,
> 
> > From: Simon Horman
> > Sent: Thursday, April 21, 2016 12:15 PM
> > 
> > Hi Shimoda-san,
> > 
> > On Wed, Apr 20, 2016 at 05:47:14PM +0900, Yoshihiro Shimoda wrote:
> > > If we doesn't enable this configuration and runs on R-Car Gen3,
> > > xhci host driver will cause timeout in xhci_reset() because
> > > the R-Car Gen3 xHCI controller needs specific initialization.
> > >
> > > [    1.565605] xhci-hcd ee000000.usb: xHCI Host Controller
> > > [    1.570636] xhci-hcd ee000000.usb: new USB bus registered, assigned bus number 5
> > > [   22.270160] xhci-hcd ee000000.usb: can't setup: -110
> > > [   22.274931] xhci-hcd ee000000.usb: USB bus 5 deregistered
> > > [   22.280158] xhci-hcd: probe of ee000000.usb failed with error -110
> 
> I'm afraid but this issue can be fixed if we apply the following patch into xhci-rcar.h.
> So, I will submit such a patch to USB ML later.
> 
> ==============================================================================
> diff --git a/drivers/usb/host/xhci-rcar.h b/drivers/usb/host/xhci-rcar.h
> index 2941a25..0681235 100644
> --- a/drivers/usb/host/xhci-rcar.h
> +++ b/drivers/usb/host/xhci-rcar.h
> @@ -24,7 +24,7 @@ static inline void xhci_rcar_start(struct usb_hcd *hcd)
>  
>  static inline int xhci_rcar_init_quirk(struct usb_hcd *hcd)
>  {
> -	return 0;
> +	return -ENODEV;
>  }
>  #endif
>  #endif /* _XHCI_RCAR_H */
> ==============================================================================
> 
> Also I will remove this commit log from in this patch.
> 
> > > Remarks:
> > > The timestamp doesn't work correctly on R-Car H3 ES1.0. It should be
> > > 10 seconds.
> > >
> > > Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@xxxxxxxxxxx>
> > > ---
> > >  arch/arm64/configs/defconfig | 3 +--
> > >  1 file changed, 1 insertion(+), 2 deletions(-)
> > >
> > > diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> > > index a44ef99..a95751d 100644
> > > --- a/arch/arm64/configs/defconfig
> > > +++ b/arch/arm64/configs/defconfig
> > > @@ -1,7 +1,6 @@
> > >  # CONFIG_LOCALVERSION_AUTO is not set
> > >  CONFIG_SYSVIPC=y
> > >  CONFIG_POSIX_MQUEUE=y
> > > -CONFIG_FHANDLE=y
> > >  CONFIG_AUDIT=y
> > >  CONFIG_NO_HZ_IDLE=y
> > >  CONFIG_HIGH_RES_TIMERS=y
> > 
> > Is the hunk above intended?
> > It does not seem to match my reading of the changelog.
> 
> Perhaps I should make a patch for this change because
> if I did the following commands, this change appeared.

I'm not sure how the ARM-SoC maintainers feel about such cleanups
these days. I suggest just dropping that part of the change for now.

> on renesas-devel-20160420-v4.6-rc4
> $ make defconfig
> $ cp defconfig arch/arm64/config
> $ git diff
> 
> Best regards,
> Yoshihiro Shimoda
> 
> > > @@ -192,7 +191,7 @@ CONFIG_SND_SOC_AK4613=y
> > >  CONFIG_USB=y
> > >  CONFIG_USB_OTG=y
> > >  CONFIG_USB_XHCI_HCD=y
> > > -CONFIG_USB_XHCI_PLATFORM=y
> > > +CONFIG_USB_XHCI_RCAR=y
> > >  CONFIG_USB_EHCI_HCD=y
> > >  CONFIG_USB_EHCI_MSM=y
> > >  CONFIG_USB_EHCI_HCD_PLATFORM=y
> > > --
> > > 1.9.1



[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux