From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefan Althoefer Date: Sun, 07 Dec 2008 11:07:48 +0100 Subject: [U-Boot] [PATCH] video: Add new driver for Silicon Motion SM501/SM502 Part 1/2 In-Reply-To: <20081207004728.1FB58834B020@gemini.denx.de> References: <4938472f.bsulKM1wIgSbATIm%stefan.althoefer@web.de> <493AD29C.80409@denx.de> <20081207004728.1FB58834B020@gemini.denx.de> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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