Re: [PATCH V2] OMAP3LOGIC: Update default Logic PD display type

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

 



* Tomi Valkeinen <tomi.valkeinen@xxxxxx> [160222 09:30]:
> On 22/02/16 18:49, Tony Lindgren wrote:
> 
> >>> I think separate .dtb files are the best option at the moment, and
> >>> choose the right one in the bootloader.
> > 
> > That's not going to work for many boards as there are just too many
> > combinations to support.
> 
> I think it's fine if there's just one "base" .dts file, so for N panels
> you'll end up with N .dtbs. The problems start if you have multiple base
> .dts files for, say, different board and SoC revisions. Then the amount
> of .dtbs explode.

Yeah in this case of modular development boards you can have multiple
CPU modules and multiple display panel options. Adam, just for fun,
care to estimate the potential permutations with the torpedo kit for
example?

> >>> Hopefully DT overlays will provide a better solution in the near future.
> > 
> > And that's been already years in making.
> > 
> > I think the only current real fix for issues like this is to include
> > multiple configurations in the panel code that can then be selected
> > based on the device tree compatible flag or kernel cmdline.
> 
> Well, there are a few options other than the DT overlays and dealing
> with this in the bootloader, and each of those options is hacky.
> 
> - board specific platform code detects the display, and appends the
> necessary DT data (or uses platform data, but I'd rather get rid of
> pdata, not add more).

Yeah if the panel can be detected over I2C or by probing the GPIO
pins then this might be doable. Probably no need to mix DT data
into runtime detection like this though, it can be done in a board
specific device driver that then creates the struct device for the
panel detected.

> - create board specific drivers that detect the display and use the
> correct video timings.

Having a custom driver seems better to me than trying stuff the
detection code into the arch/arm/.

> - abuse the board's .dts, and fill in all the possible panels there, and
> then at runtime somehow choose which one is used.

Heh yeah something like this could be done in the bootloader :)

> Well, nothing else comes to my mind. And those above are roughly in my
> least-bad-to-most-bad order.

Yeah I'd still prefer a kernel cmdline option until we have a
better Linux generic solution merged into the mainline kernel.

Regards,

Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" 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 (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux