[U-Boot] [PATCH] p1022ds: add video support

Timur Tabi timur at freescale.com
Wed Sep 22 01:01:19 CEST 2010


Anatolij Gustschin wrote:
>> Also, I thought it would be better to provide some sort of completeness for the
>> > bits I am using.
> We do not add unused code.

Well, I don't see how macros are "unused code", but I'll delete those lines.

>>> > > CONFIG_VIDEO definition should depend on CONFIG_FSL_DIU_FB,
>>> > > I think. Otherwise the building won't succeed if CONFIG_FSL_DIU_FB
>>> > > is not defined (due to absence of video_hw_init()).
>> > 
>> > We need the DIU initialized, even if U-Boot doesn't use it.  This is what
>> > CONFIG_FSL_DIU_FB does.  CONFIG_VIDEO then tells U-Boot to actually use the DIU.

> I understand it. I didn't say you should make CONFIG_FSL_DIU_FB
> dependent on CONFIG_VIDEO, but the opposite.

Ok.

> On the other site, one of the design principles states:
> 
> initialize devices only when they are needed within U-Boot.

Well, I'll try to work toward that, but I'm pretty sure there are plenty of
devices that are initialized but not used in U-Boot already.

-- 
Timur Tabi
Linux kernel developer at Freescale



More information about the U-Boot mailing list