[PATCH] arm64: defconfig: Add Renesas R-Car USB 3.0 driver support
From: Yoshihiro ShimodaSigned-off-by: Yoshihiro Shimoda Signed-off-by: Simon Horman --- arch/arm64/configs/defconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig index f70505186820..c3cdd640bea3 100644 --- a/arch/arm64/configs/defconfig +++ b/arch/arm64/configs/defconfig @@ -189,7 +189,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 -- 2.7.0.rc3.207.g0ac5344
RE: [PATCH] arm64: defconfig: Add Renesas R-Car USB 3.0 driver support
Hi Simon-san, > From: Simon Horman > Sent: Thursday, April 21, 2016 12:56 PM > < snip > > > > > 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. Thank you for the suggestion! I agree with you. So, I will submit v2 patch today. Best regards, Yoshihiro Shimoda > > 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
Re: [PATCH] arm64: defconfig: Add Renesas R-Car USB 3.0 driver support
On Thu, Apr 21, 2016 at 03:41:14AM +, 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 ee00.usb: xHCI Host Controller > > > [1.570636] xhci-hcd ee00.usb: new USB bus registered, assigned > > > bus number 5 > > > [ 22.270160] xhci-hcd ee00.usb: can't setup: -110 > > > [ 22.274931] xhci-hcd ee00.usb: USB bus 5 deregistered > > > [ 22.280158] xhci-hcd: probe of ee00.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> > > --- > > > 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
RE: [PATCH] arm64: defconfig: Add Renesas R-Car USB 3.0 driver support
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 ee00.usb: xHCI Host Controller > > [1.570636] xhci-hcd ee00.usb: new USB bus registered, assigned bus > > number 5 > > [ 22.270160] xhci-hcd ee00.usb: can't setup: -110 > > [ 22.274931] xhci-hcd ee00.usb: USB bus 5 deregistered > > [ 22.280158] xhci-hcd: probe of ee00.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> > --- > > 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. 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
Re: [PATCH] arm64: defconfig: Add Renesas R-Car USB 3.0 driver support
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 ee00.usb: xHCI Host Controller > [1.570636] xhci-hcd ee00.usb: new USB bus registered, assigned bus > number 5 > [ 22.270160] xhci-hcd ee00.usb: can't setup: -110 > [ 22.274931] xhci-hcd ee00.usb: USB bus 5 deregistered > [ 22.280158] xhci-hcd: probe of ee00.usb failed with error -110 > > Remarks: > The timestamp doesn't work correctly on R-Car H3 ES1.0. It should be > 10 seconds. > > Signed-off-by: Yoshihiro Shimoda> --- > 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. > @@ -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
[PATCH] arm64: defconfig: Add Renesas R-Car USB 3.0 driver support
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 ee00.usb: xHCI Host Controller [1.570636] xhci-hcd ee00.usb: new USB bus registered, assigned bus number 5 [ 22.270160] xhci-hcd ee00.usb: can't setup: -110 [ 22.274931] xhci-hcd ee00.usb: USB bus 5 deregistered [ 22.280158] xhci-hcd: probe of ee00.usb failed with error -110 Remarks: The timestamp doesn't work correctly on R-Car H3 ES1.0. It should be 10 seconds. Signed-off-by: Yoshihiro Shimoda--- 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 @@ -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