[U-Boot] [PATCH RESEND 2/3] arm: added support for display controller in Jade SoC

Jean-Christophe PLAGNIOL-VILLARD plagnioj at jcrosoft.com
Thu Jul 9 00:43:43 CEST 2009


On 00:25 Thu 09 Jul     , Anatolij Gustschin wrote:
> Jean-Christophe PLAGNIOL-VILLARD wrote:
> > On 16:02 Wed 08 Jul     , Matthias Weisser wrote:
> >> This patch adds support for the display controller in
> >> the MB86R01 'Jade' SoC.
> >>
> >> Signed-off-by: Matthias Weisser <matthias.weisser at graf-syteco.de>
> >> ---
> ...
> >> +
> >> +/*
> >> + * 4MB (at the end of system RAM)
> >> + */
> >> +#define VIDEO_MEM_SIZE		0x400000
> >> +
> >> +#define GDC_HOST_BASE		0xF1FC0000
> >> +#define GDC_DSP0_BASE		0xF1FD0000
> >> +#define GDC_DSP1_BASE		0xF1FD2000
> > please move to a header
> 
> probably we should drop these entirely and use JADE_GDC_PHYS_BASE
> and JADE_GDC_PHYS_DISP_BASE from include/asm-arm/arch-jade/jade.h
> file introduced with this patch series? Just need to add the
> base for second display controller register block there.
> 
> ...
> >> +GraphicDevice jadegdc;
> > please no uppercase in type
> 
> we have to stay with GraphicDevice type in this patch now. It is
> there in U-Boot code since more than five years, Matthias simply
> uses it here as other video drivers use it, too. But yes, we should
> do general video code cleanup later.
ok fine

Best Regards,
J.


More information about the U-Boot mailing list