Re: MSM8974: USB (gadget mode) broken in v4.11-rc1

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

 



Quoting Jonathan Neuschäfer (2017-03-06 21:04:23)
> Hi,
> 
> In Andy Gross' for-next branch (commit a844f941617c929b29, "Merge tag
> 'qcom-arm64-for-4.11-2' into final-for-4.11"), USB support on my MSM8974
> phone worked, but since the following merge by Linus (195849ea13549dc), it
> does not work anymore. (I've only tested gadget mode so far. I don't know if
> host mode works.)
> 
> I ran git bisect with the devicetree from Bjorn Andersson's commit
> 769907ae6e6c28 ("ARM: dts: qcom: msm8974: Add USB gadget nodes"), and got

I have no idea why this was merged into v4.11-rc1. Andy?

> this result:
> 
> e9f15a71f17b328be44767cb893576c7c838f996 is the first bad commit
> commit e9f15a71f17b328be44767cb893576c7c838f996
> Author: Stephen Boyd <stephen.boyd@xxxxxxxxxx>
> Date:   Wed Dec 28 14:57:01 2016 -0800
> 
>     usb: chipidea: msm: Add proper clk and reset support
> 
>     The msm chipidea controller uses two main clks, an AHB clk to
>     read/write the MMIO registers and a core clk called the system
>     clk that drives the controller itself. Add support for these clks
>     as they're required in all designs.
> 
>     Also add support for an optional third clk that we need to turn
>     on to reset the controller and wrapper logic and other
>     "housekeeping" things. This clk was removed in later revisions of
>     the hardware because the reset methodology no longer required
>     clks to be enabled to propagate resets.
> 
>     Acked-by: Peter Chen <peter.chen@xxxxxxx>
>     Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
>     Signed-off-by: Stephen Boyd <stephen.boyd@xxxxxxxxxx>
>     Signed-off-by: Peter Chen <peter.chen@xxxxxxx>
> 
> I can't see any potential error messages because the serial gadget driver is
> the only console I have on this system.

Oh that's unfortunate.

> 
> Is there something missing in qcom-msm8974.dtsi?

Yes. The DT side of the patches never merged into arm-soc, so you'll
need this patch[1] to get things going. I thought the plan was to wait
for the usb driver rewrite and never merge that dts patch you mentioned.

[1] http://lkml.kernel.org/r/20170127004728.22490-4-stephen.boyd@xxxxxxxxxx
--
To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux