Hi Lucas, On 14 January 2016 at 13:34, Lucas Stach <d...@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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot