On 19/05/15 09:39, Dr. H. Nikolaus Schaller wrote:
Hi Tomi,
Am 19.05.2015 um 08:35 schrieb Tomi Valkeinen tomi.valkeinen@ti.com:
On 19/05/15 09:20, Dr. H. Nikolaus Schaller wrote:
Ah, you are right. I did only look for diffs in drivers/video/fbdev/omap2 because we have tried to reduce all unnecessary diffs to mainline.
But it is an extension of omap_dsi_enable_pads() completely wrapped inside of arch/arm/mach-omap2/display.c and we have not yet looked at that file.
Then of course it is not (yet) in mainline. On the other hand it would be nice to see it there… You can add me as “tested by”. Our complete diff is attached.
Unfortunately I need to rewrite it for mainline (also the omap4 code), as the DSI muxing code should not even exist. It should be done using syscon, or possibly pinctrl.
Ok, I understand. With syscon or pinmux it would move to the CPU specific DT. We will keep it in our tree until we see an update arrive in mainline.
Or should we kick it out? IMHO it also could work without by relying on default pinmux.
I'd recommend keeping the patches (although you could squash them into one).
Tomi