Skip to content

Allow xhci-plat using a second clock

Gregory CLEMENT linux-arm-kernel linux-usb 1 version New

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest0 of 2 patches 0 repliesNew

Patches in v1 No applies recorded yet

None of the 2 patches the cover letter announces are in this archive.

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 0/2] Allow xhci-plat using a second clock

Hi Mathias,

Quoted 26 lines · Mathias Nyman
Quoted 17 lines · Gregory CLEMENT

Hi Mathias,

    On mer., f?vr. 14 2018, Gregory CLEMENT
Quoted 8 lines · <[email protected]> wrote:

Hello,

The purpose of this series is to allow xhci-plat using a second clock. It is needed on the Armada 7K/8K but could be used by other SoCs.

The first patch is just a fix found while I was working on this feature.

I've just realized that this series sent 2 months ago was not merged in v4.17. The issue is that now the USB support on the Armada 7K/8K is broken, because the clock support part was already merged.

Quoted 7 lines

You already had a look on this series one month ago. The issue you had was about getting an approval for the extension of the binding [1]. I pinged Rob about it [2] one month ago but we didn't get any feedback. However, Rob already approved the similar changes I introduced in an other for mv_xor_v2: [3].

So would it be possible to apply this series on v4.17-rc ?

Quoted 7 lines

Patch 2/2 no longer applies due to clk suspend/resume changes in:

commit d56e57ca030c8b4296944a2ae61ac167bf979c07
usb: host: xhci-plat: revert "usb: host: xhci-plat: enable clk in resume timing"

Not sure how that change affects Armada 7K/8K
Can you rebase and resend the series?

It's done!

About the suspend/resume part it does not affect Armada 7K/8K because we don't support it yet.

However I am a bit surprised by the commit log because reverting a common part because of a specific implementation looks wrong. I expect issues when we will add PM support but we will see that later.

Gregory

Quoted 3 lines

-Mathias

Signature · 3 lines
Gregory Clement, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
http://bootlin.com

Read the cover letter in its thread