[U-Boot] [PATCH] video: Add new driver for Silicon Motion SM501/SM502 Part 1/2

Stefan Althoefer stefan.althoefer at web.de
Sun Dec 7 11:07:48 CET 2008


Dear All,

> Dear Anatolij & Stefan,
> 
> In message <493AD29C.80409 at denx.de> you wrote:
>>> Use CONFIG_VIDEO_SM501NEW to enable the driver.
>> not sure if CONFIG_VIDEO_SM501NEW is a good chose here. Maybe
>> we should use s.th. like CONFIG_VIDEO_SM50x. This applies to
>> the file names too: sm50x.h, sm50x.c, etc. Even better would
>> be a merge with the existing driver.
> 
> I agree with Anatolij here. A merge would definitely be best.

Before I start to implement the good suggestions, I'd like
to know how such a merge would look like in your opinion.

Over the long term, I see the old driver vanishing, as it is too
complicated to use (you need to be a video and SMI register
expert). It is not much more than a jump into the board specific
setup routines. However, removing the old driver would break half
a dozen  of boards (inacceptable).

BTW the driver will probably also work with the new SM107,
which as no USB, Keyboard and all this stuff.

-- Stefan



More information about the U-Boot mailing list