[U-Boot] [PATCH 02/14] PXA: pxa-regs.h cleanup

Wolfgang Denk wd at denx.de
Mon Sep 27 21:33:14 CEST 2010


Dear Marek Vasut,

In message <1285226769-9619-2-git-send-email-marek.vasut at gmail.com> you wrote:
> Signed-off-by: Marek Vasut <marek.vasut at gmail.com>
...
> +	debug("FDADR0 = 0x%08x\n", (unsigned int)readl(FDADR0));
> +	debug("FDADR1 = 0x%08x\n", (unsigned int)readl(FDADR1));
> +	debug("LCCR0 = 0x%08x\n", (unsigned int)readl(LCCR0));
> +	debug("LCCR1 = 0x%08x\n", (unsigned int)readl(LCCR1));
> +	debug("LCCR2 = 0x%08x\n", (unsigned int)readl(LCCR2));
> +	debug("LCCR3 = 0x%08x\n", (unsigned int)readl(LCCR3));

Is there a specific reason for the "(unsigned int)" cast? Using a
"0x%08lx" format would seem more logical to me?



> diff --git a/arch/arm/include/asm/arch-pxa/pxa-regs.h b/arch/arm/include/asm/arch-pxa/pxa-regs.h
> index d442fb0..4eefff2 100644
> --- a/arch/arm/include/asm/arch-pxa/pxa-regs.h
> +++ b/arch/arm/include/asm/arch-pxa/pxa-regs.h
...
> +#define DCSR0		(0x40000000)  /* DMA Control / Status Register for Channel 0 */
> +#define DCSR1		(0x40000004)  /* DMA Control / Status Register for Channel 1 */
> +#define DCSR2		(0x40000008)  /* DMA Control / Status Register for Channel 2 */
> +#define DCSR3		(0x4000000c)  /* DMA Control / Status Register for Channel 3 */
> +#define DCSR4		(0x40000010)  /* DMA Control / Status Register for Channel 4 */
> +#define DCSR5		(0x40000014)  /* DMA Control / Status Register for Channel 5 */
> +#define DCSR6		(0x40000018)  /* DMA Control / Status Register for Channel 6 */
> +#define DCSR7		(0x4000001c)  /* DMA Control / Status Register for Channel 7 */
> +#define DCSR8		(0x40000020)  /* DMA Control / Status Register for Channel 8 */
> +#define DCSR9		(0x40000024)  /* DMA Control / Status Register for Channel 9 */
> +#define DCSR10		(0x40000028)  /* DMA Control / Status Register for Channel 10 */
> +#define DCSR11		(0x4000002c)  /* DMA Control / Status Register for Channel 11 */
> +#define DCSR12		(0x40000030)  /* DMA Control / Status Register for Channel 12 */
> +#define DCSR13		(0x40000034)  /* DMA Control / Status Register for Channel 13 */
> +#define DCSR14		(0x40000038)  /* DMA Control / Status Register for Channel 14 */
> +#define DCSR15		(0x4000003c)  /* DMA Control / Status Register for Channel 15 */
...

No parens needed for plain numbers. Please fix globally.

Do you plan to convert this (and the others) into a proper C struct?

...
> +///*#define SACR0		(0x40400000)  /  Global Control Register */
> +///*#define SACR1		(0x40400004)  /  Serial Audio I 2 S/MSB-Justified Control Register */
> +///*#define SASR0		(0x4040000C)  /  Serial Audio I 2 S/MSB-Justified Interface and FIFO Status Register */
> +///*#define SAIMR		(0x40400014)  /  Serial Audio Interrupt Mask Register */
> +///*#define SAICR		(0x40400018)  /  Serial Audio Interrupt Clear Register */
> +///*#define SADIV		(0x40400060)  /  Audio Clock Divider Register. */
> +///*#define SADR		(0x40400080)  /  Serial Audio Data Register (TX and RX FIFO access Register). */

C++ comments are not allowed.

Please remove dead code.


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
"He was so narrow minded he could see through  a  keyhole  with  both
eyes ..."


More information about the U-Boot mailing list