[U-Boot] [PATCH 2/2 V2] update the EB+MCF-EV123 board support
    Wolfgang Denk 
    wd at denx.de
       
    Thu Jul 23 21:36:52 CEST 2009
    
    
  
Dear Jens Scharsig,
In message <h447jv$vm7$3 at ger.gmane.org> you wrote:
> This patch updates the support for EB+MCF-EV123 board and needs the [PATCH 1/2 V2] new video driver for bus vcxk framebuffers
Please restrict the line length of the commit message to 70 characters
or less.
...
> +#if defined(CONFIG_VIDEO)
> +
> +/*----------------------------------------------------------------------------
> + ****h* EB+CPU5282-T1/drv_video_init
> + * FUNCTION
> + ***
> +----------------------------------------------------------------------------*/
Incorrect multi-line comment style.
> +int	drv_video_init (void)
> +{
> +	char *s;
> +	unsigned long splash;
> +
> +	printf("Init Video as ");
> +
> +	if ((s = getenv ("displaywidth")) != NULL)
> +	{
> +		display_width = simple_strtoul (s, NULL, 10);
> +	}
> +	else
> +	{
> +		display_width=256;
> +	}
> +	if ((s = getenv ("displayheight")) != NULL)
> +	{
> +		display_height = simple_strtoul (s, NULL, 10);
> +	}
> +	else
> +	{
> +		display_height=256;
> +	}
Incorrect brace style.
> +	printf("%lu x %lu pixel matrix\n",display_width,display_height);
Please s/,/, /g
Please clean up and resubmit.
Best regards,
Wolfgang Denk
-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
News is what a chap who doesn't care much  about  anything  wants  to
read. And it's only news until he's read it. After that it's dead.
                           - Evelyn Waugh _Scoop_ (1938) bk. 1, ch. 5
    
    
More information about the U-Boot
mailing list