Re: [added to the 4.1 stable tree] usb: host: xhci-rcar: Use ARCH_RENESAS

[Cc Shimoda-san]

On Fri, Jun 03, 2016 at 09:14:17PM -0400, Sasha Levin wrote:

On 06/03/2016 08:47 PM, Simon Horman wrote:

quoted

Hi,

I do not believe this patch is suitable for v4.1 stable.

ARCH_RENESAS is not otherwise present there and thus the driver in question
will stop being compiled where it was previously compiled.

Moreover, the transition to ARCH_RENESAS is (so far) being handled in such
a way that there have not been any regressions that would necessitate a
backport to -stable.

Hey Simon,

I've picked this commit because of what I saw in f879fc32a
("usb: host: xhci-rcar: Avoid long wait in xhci_reset()").

quoted

From my understanding of that change, we added a dependency on ARCH_RENESAS

for USB_XHCI_PLATFORM, but since USB_XHCI_RCAR depended on ARCH_SHMOBILE that
would cause silly errors for kconfig and would probably confuse users.

Rather than changing it to a ARCH_SHMOBILE dependency, I've preferred
to pull in f879fc32a to keep the stable tree more aligned with upstream
and possibly prevent any future backporting errors for new commits that
add ARCH_RENESAS dependencies.

If you don't agree with that approach I can change that to a ARCH_SHMOBILE
dependency and remove this commit, or take any other approach you think
is better?

Thanks, this does seem to be a bit of a mess. Sorry about that.

My feeling is that it would be better to change f879fc32a to use
ARCH_SHMOBILE as ARCH_RENESAS does not exist in v4.1.

One slight risk in this approach is that ARCH_SHMOBILE covers some (older)
SH SoCs as well as (newer) ARM and arm64 based SoCs (Renesas arm64 based
SoCs are not present in v4.1) whereas ARCH_RENESAS (in kernels that it
exists) only covers ARM and arm64 based SoCs. So the coverage of
ARCH_SHMOBILE is broader. But I don't think that should be a problem as
that is already the dependency being used in v4.1.

A quick compile test using ecovec24_defconfig + USB_XHCI_HCD +
USB_XHCI_RCAR indicates that the driver compiles on that platform.