Add hwmod data for IPU & DSP processors/MMUs
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest1 of 8 patches 0 repliesNew
1 of 8 patches is in this archive. The cover letter announces 8.
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 6/8 | ARM: OMAP5: hwmod_data: Add data for IPU & DSP processors | +0 | 0 replies |
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.
Cover letter [PATCH 0/8] Add hwmod data for IPU & DSP processors/MMUs
Quoted 12 lines · Suman Anna
Quoted 10 lines · Tony Lindgren
Quoted 2 lines · Suman Anna
Ping on this series. Tero's clkctrl series changes are orthogonal to these changes.
With patch 5/8 you said earlier "It will still be a shared set between the MMU and remoteproc drivers, but let me look into how these get integrated." So was there any conclusion there?
Also, what's the deal with these omap4 changes that I commented on in 5/8?
Quoted 12 lines
Tony,
So, looking at code, removing modulemode on hwmod does change the semantics as that will not plug in the clkctrl clk (_omap4_xlate_clkctrlwill return 0) into the hwmod and will fall back. So, this should make it compatible to the existing code before adding the clkctrl dt nodes, and Patch 5 will be good to avoid the crash when disabling the processor nodes. That said, now that the modulemode is controlled by the ti-clkctrl code, I would expect to get the refcounting for free due toclk_enable()semantics if the same clkctrl clk is plugged under two nodes (MMU and processor) and I may not need to remove the modulemode. One of the main reasons for removing the modulemode is the lack of refcounting on modulemode in the original code.
OK. Sounds like you can get the refcounting for free just by modifying the dts as done for example for musb on omap4 in "[PATCH 10/10] ARM:
dts: Use ti-sysc module driver for omap4 musb" in that series. You will need also the rest of the patches from that series naturally :)
That is as long as MMU and processor are both child devices in the same interconnect target module.
Quoted 2 lines
In anycase, I need to retest this with all of required patches from Tero (current mainline does not yet have the clkctrl dt nodes).
OK
Quoted 5 lines
Tero,
Do you have these patches rebased onto 4.14-rc1 (preferable if it is rebased onto Tony's omap-for-v4.15/soc branch. I tried cherry-picking from your github test branch (based on 4.13-rc3), but am running into some L3Noc errors once I add the clkctrl dts node patch on Panda4 using NFS.
Yeah I noticed that and emailed Tero about it. It seems to trigger with built-in musb and DEBUG_KERNEL disabled, not sure what triggers it. Only happens after the clkctrl nodes are added.
Regards,
Tony