[U-Boot] [PATCH 3/6] sunxi: video: Add cfb console driver for sunxi

Anatolij Gustschin agust at denx.de
Fri Nov 14 21:17:39 CET 2014


Hi Hans,

this patch looks good, only a few minor comments below.

On Fri, 14 Nov 2014 17:54:45 +0100
Hans de Goede <hdegoede at redhat.com> wrote:
...
> diff --git a/arch/arm/include/asm/arch-sunxi/display.h b/arch/arm/include/asm/arch-sunxi/display.h
> new file mode 100644
> index 0000000..8ac7d1b
> --- /dev/null
> +++ b/arch/arm/include/asm/arch-sunxi/display.h
> @@ -0,0 +1,198 @@
> +/*
> + * Sunxi platform display cntroller register and constant defines

s/cntroller/controller/

not need to resubmit the patch, this can be fixed when applying.

...
> +#if 1
> +	printf("dotclock: %dkHz = %dkHz: (%d * 3MHz * %d) / %d\n",
> +	       dotclock, (best_double + 1) * 3000 * best_n / best_m,
> +	       best_double + 1, best_n, best_m);
> +#endif

please drop #if 1 / #endif when applying the patch.
here, printf() could be replaced by debug() maybe ?
 
...
> +	udelay(1000000 / mode->refresh + 500);
> +
> +	/* Sometimes the display pipeline does not sync up properly, if
> +	   this happens the hdmi fifo underrun or overrun bits are set */

we use

/*
 * multi-line
 * comment
 */

style.

Thanks,

Anatolij


More information about the U-Boot mailing list