On Tue, 15 Jul 2008 16:52:01 +0200, Jochen Friedrich wrote: > Hi Jean, > > > Eeeek. The patch you mention here is only the conversion of ONE driver. > > It is absolutely not relevant as to what the general rule is. > > Sorry, i must have misunderstood you then. > > http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;h=af294867a52bf718df835a688e8c786d550bee26#patch9 > is the same, my original patch listed all four supported chips in there > (saa7126, saa7127, saa7128 and saa7129) while only one made it into the > driver...
As I recall, your patch was done quickly and without knowledge of the chips in question. I did mine in close collaboration with Hans Verkuil who knows these chips very well, to make sure I wouldn't break anything. With success, as far as I can tell. Honestly, I can't remember why we decided to have a single chip name for all 4 variants. It might have been a shortcut to complete the conversion in time. Or, more likely, I didn't notice the other types because the driver was originally using the same name "saa7127" for all devices. If that is the case I'll update the driver to behave more in compliance with the new i2c device/driver matching scheme. I'll discuss this with Hans to make sure it's OK. So, again, please don't take this (nor any other) media driver conversion patch as an example of what should be done. The proper conversion of all media drivers will take a lot of time because of the history behind these drivers. -- Jean Delvare _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev