From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefano Babic Date: Fri, 28 Jun 2013 16:57:44 +0200 Subject: [U-Boot] [PATCH] video: mxsfb: Break the line in videomode message In-Reply-To: <201306262156.05006.marex@denx.de> References: <1372273693-8777-1-git-send-email-fabio.estevam@freescale.com> <201306262156.05006.marex@denx.de> Message-ID: <51CDA468.2040204@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Fabio, hi Marek, On 26/06/2013 21:56, Marek Vasut wrote: > Dear Fabio Estevam, > >> Currently we have the following on boot: >> >> CPU: Freescale i.MX28 rev1.2 at 454 MHz >> BOOT: SSP SD/MMC #0, 3V3 >> DRAM: 128 MiB >> MMC: MXS MMC: 0 >> Video: MXSFB: 'videomode' variable not set!In: serial >> >> Break the line of the warning message in order to have a better reading >> format. >> >> Signed-off-by: Fabio Estevam >> --- >> drivers/video/mxsfb.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/video/mxsfb.c b/drivers/video/mxsfb.c >> index b189419..411d356 100644 >> --- a/drivers/video/mxsfb.c >> +++ b/drivers/video/mxsfb.c >> @@ -137,7 +137,7 @@ void *video_hw_init(void) >> /* Suck display configuration from "videomode" variable */ >> penv = getenv("videomode"); >> if (!penv) { >> - printf("MXSFB: 'videomode' variable not set!"); >> + printf("MXSFB: 'videomode' variable not set!\n"); > > puts() ... and you might want to fix it in all drivers ;-) > That is correct - as it is a very small change, I fix it myself by applying the patch. Applied to u-boot-imx, thanks ! Best regards, Stefano Babic -- ===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de =====================================================================