[U-Boot] [PATCH V2 2/9] video: add amba-clcd prime-cell
Wolfgang Denk
wd at denx.de
Sun Nov 29 23:21:26 CET 2009
Dear Alessandro,
In message <20091129222444.2e454083 at wker> Anatolij Gustschin wrote:
> Hello Alessandro,
>
> Alessandro Rubini <rubini-list at gnudd.com> wrote:
>
> <snip>
> > diff --git a/include/amba_clcd.h b/include/amba_clcd.h
> > new file mode 100644
> > index 0000000..09c9aa2
> > --- /dev/null
> > +++ b/include/amba_clcd.h
> > @@ -0,0 +1,85 @@
> ...
> > +/*
> > + * CLCD Controller Internal Register addresses
> > + */
> > +#define CLCD_TIM0 0x00000000
> > +#define CLCD_TIM1 0x00000004
> > +#define CLCD_TIM2 0x00000008
> > +#define CLCD_TIM3 0x0000000c
> > +#define CLCD_UBAS 0x00000010
> > +#define CLCD_LBAS 0x00000014
>
> CLCD_LBAS isn't used in the code, please remove it.
>
> ...
> > +#define CLCD_STAT 0x00000020
> > +#define CLCD_INTR 0x00000024
> > +#define CLCD_UCUR 0x00000028
> > +#define CLCD_LCUR 0x0000002C
Actually such a register descriptiion must not be implemented by
offset definitions at all. Please define a C struct, and access it
using appropriate accessor functions only.
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
It is necessary to have purpose.
-- Alice #1, "I, Mudd", stardate 4513.3
More information about the U-Boot
mailing list