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

Stefan Althoefer
2008-12-07T10:07:48Z

Dear All,

> Dear Anatolij & Stefan,
>
> In message <493AD29C.80409@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