Hi Lucas,
On 14 January 2016 at 13:34, Lucas Stach dev@lynxeye.de wrote:
Am Donnerstag, den 14.01.2016, 13:26 -0700 schrieb Simon Glass:
The original tegra20 display driver was written before Linux had device tree bindings for display. Since then Linux has developed a robust set of bindings covering various aspects of enabling a display.
This series moves closer to those bindings by using the panel and backlight as separate drivers. The device tree files for seaboard, ventana and harmony thereby become almost the same as Linux.
Unfortunately this breaks the other boards, which will need a similar sync. So I'm not sure how easy it will be to accept this series. Still, it seems worth sending it out in the hope that board maintainers can help. I have kept this series separate so that it can progress separately.
By pushing display timings into the DT you are actually diverging from mainline, as mainline doesn't require this, but instead infers the timings from the panel compatible. Is this a desired goal?
This is not divergence. Please take a look at the patch series. The device tree files are very close to the same now. The existing U-Boot support has display timings in the device tree too, so this is not being added.
The display timings are a small part of the work, but in the back of my mind is that we don't want to have a big table of display panel timings as exists in Linux. This is a waste of space when a board will only use one panel.
Regards, Lucas
Simon Glass (4): tegra: dts: Sync seaboard device tree file with Linux video: tegra: Move to using simple-panel and pwm-backlight tegra: video: Always use write-through cache on LCD fdt: Drop some unused compatible strings
arch/arm/dts/tegra20-harmony.dts | 751 ++++++++++++++++++++++++++++-- arch/arm/dts/tegra20-seaboard.dts | 934 ++++++++++++++++++++++++++++++++++---- arch/arm/dts/tegra20-ventana.dts | 685 ++++++++++++++++++++++++++-- configs/colibri_t20_defconfig | 5 +- configs/harmony_defconfig | 5 +- configs/medcom-wide_defconfig | 5 +- configs/paz00_defconfig | 5 +- configs/seaboard_defconfig | 5 +- configs/tec_defconfig | 5 +- configs/ventana_defconfig | 5 +- drivers/video/tegra.c | 333 +++----------- include/fdtdec.h | 4 - lib/fdtdec.c | 4 - 13 files changed, 2307 insertions(+), 439 deletions(-)
Regards, Simon