Hi Tom, On 16 February 2016 at 09:23, Tom Warren <twar...@nvidia.com> wrote: > Simon, > >> -----Original Message----- >> From: s...@google.com [mailto:s...@google.com] On Behalf Of Simon Glass >> Sent: Tuesday, February 16, 2016 9:03 AM >> To: Tom Warren <twar...@nvidia.com> >> Cc: Stephen Warren <swar...@wwwdotorg.org>; U-Boot Mailing List <u- >> b...@lists.denx.de>; Marcel Ziswiler <marcel.ziswi...@toradex.com>; Stephen >> Warren <swar...@nvidia.com>; Pantelis Antoniou <panto@antoniou- >> consulting.com>; Marek Vasut <marek.va...@gmail.com>; Pavel Herrmann >> <morpheus.i...@gmail.com>; Anatolij Gustschin <ag...@denx.de> >> Subject: Re: [PATCH v2 00/23] dm: tegra: Convert tegra20 and tegra124 video >> drivers to driver model >> >> Hi Tom, >> >> On 16 February 2016 at 08:47, Tom Warren <twar...@nvidia.com> wrote: >> > >> > Simon >> > >> > > -----Original Message----- >> > > From: s...@google.com [mailto:s...@google.com] On Behalf Of Simon >> > > Glass >> > > Sent: Sunday, February 14, 2016 6:19 PM >> > > To: Stephen Warren <swar...@wwwdotorg.org> >> > > Cc: U-Boot Mailing List <u-boot@lists.denx.de>; Marcel Ziswiler >> > > <marcel.ziswi...@toradex.com>; Tom Warren <twar...@nvidia.com>; >> > > Stephen Warren <swar...@nvidia.com>; Pantelis Antoniou >> > > <panto@antoniou- consulting.com>; Marek Vasut >> > > <marek.va...@gmail.com>; Pavel Herrmann <morpheus.i...@gmail.com>; >> > > Anatolij Gustschin <ag...@denx.de> >> > > Subject: Re: [PATCH v2 00/23] dm: tegra: Convert tegra20 and >> > > tegra124 video drivers to driver model >> > > >> > > Hi, >> > > >> > > On 1 February 2016 at 17:00, Stephen Warren <swar...@wwwdotorg.org> >> > > wrote: >> > > > >> > > > On 01/30/2016 04:37 PM, Simon Glass wrote: >> > > >> >> > > >> This series moves these two drivers over to use driver model for >> > > >> video. >> > > >> >> > > >> This involves the following steps: >> > > >> - Sync up some device tree files with Linux >> > > >> - Implement a proper PWM driver >> > > >> - Clean up and unify the driver code >> > > >> - Modify the existing drivers to work with driver model >> > > >> >> > > >> The tegra20 display driver uses device tree bindings invented in >> > > >> 2011 before Linux had this or anyone was able to agree a >> > > >> standard. It seems possible to move it to the new bindings (like >> > > >> tegra124) except for the issue of time delays between stages. It >> > > >> isn't clear how this should work, and Linux implements this by >> > > >> including all LCD definitions in the kernel source code, and not >> > > >> using any delays. This causes strange display artifacts on the >> > > >> display when starting up, but perhaps is harmless to the display. >> > > >> Future work will sync up the device tree more for seaboard, and thus >> tidy this up for nvidia boards. >> > > >> >> > > >> A bug in the keyboard driver is also fixed by this series. The >> > > >> series is tested on seaboard and nyan-big, the two boards I have >> > > >> which support a display. >> > > >> >> > > >> This series is available at u-boot-dm/tegra-working. >> > > > >> > > > >> > > > This changes the name of the output device from "lcd" to "vidconsole". >> > > Anyone who doesn't reset their environment to default when switching >> > > to this new U-Boot will lose their display output because of this. >> > > Is there any way to maintain compatibility? >> > > > >> > > > Aside from that, I don't see any issues on Springbank (Seaboard), >> > > > Harmony, Ventana, Paz00, or p2371-2180, so the series, >> > > > Tested-by: Stephen Warren <swar...@nvidia.com> >> > > >> > > It looks like some of the patches have been applied and all Tegra >> > > boards are now giving Kconfig warnings. >> > > >> > > Tom Warren, are you able to pick up the rest of the series? >> > I had thought these had already gone in via the dm repo. If not, please >> > list >> those that still need to be picked up and I'll take them in via tegra. Best >> to >> assign the appropriate ones to me in patchwork. Currently it seems they're >> all >> assigned to me. Which patches have already been applied? >> >> I think it was the follow-up patches to add the environment work-around that >> was applied. >> >> 6c88b51 video: tegra: Enable the 'lcd' env variable work-around >> a2931b3 dm: video: Add a temporary work-around for old stdout var >> >> I see the original v2 series here: >> >> http://patchwork.ozlabs.org/project/uboot/list/?delegate=4839 >> >> so that is what needs to be applied I think. Then the Tegra config issue >> should >> be fixed. > I've applied the 23 v2 DM video patches to u-boot-tegra/master, then rebased > against current u-boot/master. > > I see the 'warning: (TEGRA_COMMON) selects VIDCONSOLE_AS_LCD which has unmet > direct dependencies (DM_VIDEO)' spew for almost every board (w/MAKEALL -s > tegra). > > Am I missing some patches?
I'll make some time to look but I am tied up most of the day so it will be later, sorry. The original series is at u-boot-dm/rke-working if that helps. Regards, Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
