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

2008-12-08 Thread Anatolij Gustschin
Hello Stefan, Stefan Althoefer wrote: > Dear All, > >> Dear Anatolij & Stefan, >> >> In message <[EMAIL PROTECTED]> 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. Th

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

2008-12-07 Thread ksi
On Sun, 7 Dec 2008, Wolfgang Denk wrote: > Dear [EMAIL PROTECTED], > > In message <[EMAIL PROTECTED]> you > wrote: >> >> As a matter of fact, SM501 is an older, now nonexistent version of > SM502. > > Yeah. Tell that to some hardware companies who are shipping boards > with the SM501 on it. :-)

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

2008-12-07 Thread Wolfgang Denk
Dear [EMAIL PROTECTED], In message <[EMAIL PROTECTED]> you wrote: > > As a matter of fact, SM501 is an older, now nonexistent version of SM502. Yeah. Tell that to some hardware companies who are shipping boards with the SM501 on it. :-) > Its support in U-Boot is rudimentary. I'd rather remo

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

2008-12-07 Thread ksi
On Sun, 7 Dec 2008, Wolfgang Denk wrote: > Dear [EMAIL PROTECTED], > > In message <[EMAIL PROTECTED]> you > wrote: >> >> As for the SM502, it is a multi-function chip that has more than just > video. >> I'm writing an I2C driver for it right now and I want to ask you guys > for >> your opinion. >

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

2008-12-07 Thread Stefan Althoefer
Dear All, > Dear Anatolij & Stefan, > > In message <[EMAIL PROTECTED]> 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, sm

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

2008-12-07 Thread Wolfgang Denk
Dear [EMAIL PROTECTED], In message <[EMAIL PROTECTED]> you wrote: > > As for the SM502, it is a multi-function chip that has more than just video. > I'm writing an I2C driver for it right now and I want to ask you guys for > your opinion. That's not new - the SM501 is pretty similar. > MPC8548 h

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

2008-12-06 Thread ksi
On Sun, 7 Dec 2008, Wolfgang Denk wrote: > Dear Anatolij & Stefan, > > In message <[EMAIL PROTECTED]> 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 t

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

2008-12-06 Thread Wolfgang Denk
Dear Anatolij & Stefan, In message <[EMAIL PROTECTED]> 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. Ev

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

2008-12-06 Thread Stefan Althoefer
Hi Anatolij, >> 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.

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

2008-12-06 Thread Anatolij Gustschin
Hello Stefan, please see comments below: Stefan Althoefer wrote: > [PATCH] video: Add new driver for Silicon Motion SM501/SM502 this line duplicates the subject, so simply remove it. > This patch adds a new driver for SM501/SM502. Compared to the > existing driver it allows dynamic selection of

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

2008-12-04 Thread Stefan Althoefer
[PATCH] video: Add new driver for Silicon Motion SM501/SM502 This patch adds a new driver for SM501/SM502. Compared to the existing driver it allows dynamic selection of resolution (environment: videomode). The drive is based on Vincent Sanders and Ben Dooks' linux kernel driver. Use CONFIG_VIDE