Hi Stefano, Helmut, On Tue, 20 Sep 2011 13:44:33 +0200 Stefano Babic <sba...@denx.de> wrote:
> On 09/05/2011 01:47 PM, Helmut Raiger wrote: > > mx3fb.c was based on CONFIG_LCD and is moved by this patch to > > CONFIG_VIDEO, which has greater freedom in selecting videomodes > > even at runtime. > > > > Hi Anatolji, > > do you have some comments about this patch ? Is it ok for you if I put > it in my IMX queue ? The driver is for iMX-only, and I would check if > everything remains consistent. I'm not really ok with it as is. Currently we have video support for two boards in the tree using the mx3fb driver. Applying this patch as is will break the display support for qong and imx31_phycore. I'd prefer a patch fixing this, also e.g. fixing the defines in the affected board config files and adding the display configuration environment variables to CONFIG_EXTRA_ENV_SETTINGS. Currently I do not have access to the imx31 boards to test/fix this. Stefano, can you test this patch on qong board to ensure that at least on one board the display support is working with changed mx3fb driver? Best regards, Anatolij -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: off...@denx.de _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
