Re: [U-Boot] [PATCH V2 2/2] i.MX6: mx6qsabrelite: Add splash screen support

2012-09-18 Thread Stefano Babic
On 18/09/2012 01:14, Eric Nelson wrote: Hi Eric, Adds support for the Hannstar 1024 x 768 LVDS panel (Freescale part number MCIMX-LVDS1) to SABRE-Lite board. This commit is a rebase Fabio Estevan's patch from 5/31 to u-boot-video/master: http://patchwork.ozlabs.org/patch/162206/

Re: [U-Boot] [PATCH V2 2/2] i.MX6: mx6qsabrelite: Add splash screen support

2012-09-18 Thread Eric Nelson
Thanks for the review, Stefano. On 09/18/2012 12:47 AM, Stefano Babic wrote: On 18/09/2012 01:14, Eric Nelson wrote: Hi Eric, Adds support for the Hannstar 1024 x 768 LVDS panel (Freescale part number MCIMX-LVDS1) to SABRE-Lite board. This commit is a rebase Fabio Estevan's patch from 5/31

Re: [U-Boot] [PATCH V2 2/2] i.MX6: mx6qsabrelite: Add splash screen support

2012-09-18 Thread Stefano Babic
On 18/09/2012 15:28, Eric Nelson wrote: Do you have a reference to the patch so I can follow precedent? See http://patchwork.ozlabs.org/patch/177403/ Check changes in .h. Best regards, Stefano Babic -- = DENX Software

Re: [U-Boot] [PATCH V2 2/2] i.MX6: mx6qsabrelite: Add splash screen support

2012-09-18 Thread Eric Nelson
On 09/17/2012 04:43 PM, Fabio Estevam wrote: Hi Eric, On Mon, Sep 17, 2012 at 8:14 PM, Eric Nelson eric.nel...@boundarydevices.com wrote: Adds support for the Hannstar 1024 x 768 LVDS panel (Freescale part number MCIMX-LVDS1) to SABRE-Lite board. This commit is a rebase Fabio Estevan's patch

Re: [U-Boot] [PATCH V2 2/2] i.MX6: mx6qsabrelite: Add splash screen support

2012-09-18 Thread Stefano Babic
On 18/09/2012 16:02, Eric Nelson wrote: Hi Fabio and Stefano, While trying to understand this, I wonder whether the use of overwrite_console() addresses Wolfgang's original concern. I think the suggestion was to set the preboot variable to change things: Wolfgang's first concern was

Re: [U-Boot] [PATCH V2 2/2] i.MX6: mx6qsabrelite: Add splash screen support

2012-09-18 Thread Eric Nelson
Thanks Stefano, On 09/18/2012 07:42 AM, Stefano Babic wrote: On 18/09/2012 16:02, Eric Nelson wrote: Hi Fabio and Stefano, While trying to understand this, I wonder whether the use of overwrite_console() addresses Wolfgang's original concern. I think the suggestion was to set the preboot

[U-Boot] [PATCH V2 2/2] i.MX6: mx6qsabrelite: Add splash screen support

2012-09-17 Thread Eric Nelson
Adds support for the Hannstar 1024 x 768 LVDS panel (Freescale part number MCIMX-LVDS1) to SABRE-Lite board. This commit is a rebase Fabio Estevan's patch from 5/31 to u-boot-video/master: http://patchwork.ozlabs.org/patch/162206/ Modifications include: removal of i2c setup (unneeded)

Re: [U-Boot] [PATCH V2 2/2] i.MX6: mx6qsabrelite: Add splash screen support

2012-09-17 Thread Fabio Estevam
Hi Eric, On Mon, Sep 17, 2012 at 8:14 PM, Eric Nelson eric.nel...@boundarydevices.com wrote: Adds support for the Hannstar 1024 x 768 LVDS panel (Freescale part number MCIMX-LVDS1) to SABRE-Lite board. This commit is a rebase Fabio Estevan's patch from 5/31 to s/Estevan/Estevam +int

Re: [U-Boot] [PATCH V2 2/2] i.MX6: mx6qsabrelite: Add splash screen support

2012-09-17 Thread Eric Nelson
Hi Fabio, On 09/17/2012 04:43 PM, Fabio Estevam wrote: Hi Eric, On Mon, Sep 17, 2012 at 8:14 PM, Eric Nelson eric.nel...@boundarydevices.com wrote: Adds support for the Hannstar 1024 x 768 LVDS panel (Freescale part number MCIMX-LVDS1) to SABRE-Lite board. This commit is a rebase Fabio

Re: [U-Boot] [PATCH V2 2/2] i.MX6: mx6qsabrelite: Add splash screen support

2012-09-17 Thread Fabio Estevam
On Mon, Sep 17, 2012 at 9:43 PM, Eric Nelson eric.nel...@boundarydevices.com wrote: Like this? http://lists.denx.de/pipermail/u-boot/2012-August/129967.html Yes, correct. Regards, Fabio Estevam ___ U-Boot mailing list U-Boot@lists.denx.de

Re: [U-Boot] [PATCH V2 2/2] i.MX6: mx6qsabrelite: Add splash screen support

2012-09-17 Thread Eric Nelson
On 09/17/2012 05:08 PM, Eric Nelson wrote: Hi Fabio, On 09/17/2012 04:43 PM, Fabio Estevam wrote: Hi Eric, snip +int board_late_init(void) +{ + setenv(stdout, serial); + return 0; +} Please see my previous comment on v1 about this. Wolfgang nacked this method and Stefano has proposed a