From mboxrd@z Thu Jan 1 00:00:00 1970 From: Peng Fan Date: Fri, 2 Aug 2019 06:35:26 +0000 Subject: [U-Boot] [PATCH 2/2] imx: mx6ul_14x14_evk: convert to DM_VIDEO In-Reply-To: References: <20190801061818.23585-1-peng.fan@nxp.com> <20190801061818.23585-2-peng.fan@nxp.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de HI, > Subject: Re: [PATCH 2/2] imx: mx6ul_14x14_evk: convert to DM_VIDEO > > Hi Peng, > > On Thu, Aug 1, 2019 at 3:02 AM Peng Fan wrote: > > > + display0: display at 0 { > > + bits-per-pixel = <16>; > > + bus-width = <24>; > > + > > + display-timings { > > + native-mode = <&timing0>; > > + > > + timing0: timing0 { > > + clock-frequency = <9200000>; > > + hactive = <480>; > > + vactive = <272>; > > + hfront-porch = <8>; > > + hback-porch = <4>; > > + hsync-len = <41>; > > + vback-porch = <2>; > > + vfront-porch = <4>; > > + vsync-len = <10>; > > + hsync-active = <0>; > > + vsync-active = <0>; > > + de-active = <1>; > > + pixelclk-active = <0>; > > So here we have the LCD timings in the device tree, which differs from the the > devicetree in mainline. > > This is based on the old mxsfb fbdev driver, which has been removed in 5.3-rc. > > In mainline we pass the innolux,at043tn24 LCD compatible. > > Shouldn't U-Boot be able to handle the same dt-bindings as in the kernel? I added this node in x-u-boot.dtsi, not in common dts. If drop this display-timings, we need add a similar structure inside code just as kernel did. And need to restructure mxsfb driver. For simple, we could leave this in u-boot.dtsi. Anatolij, what's your suggestion? Thanks, Peng.