On Tuesday 22 February 2011 18:08:40 Guennadi Liakhovetski wrote: > On Tue, 22 Feb 2011, Hans Verkuil wrote: [snip] > > I also think that there is a reasonable chance that such bugs can happen. > > Take a scenario like this: someone writes a new host driver. Initially > > there is only support for positive polarity and detection on the rising > > edge, because that's what the current board on which the driver was > > developed supports. This is quite typical for an initial version of a > > driver. > > > > Later someone adds support for negative polarity and falling edge. > > Suddenly the polarity negotiation on the previous board results in > > negative instead of positive which was never tested. Now that board > > starts producing pixel errors every so often. And yes, this type of > > hardware problems do happen as I know from painful experience. > > > > Problems like this are next to impossible to debug without the aid of an > > oscilloscope, so this isn't like most other bugs that are relatively easy > > to debug. > > Well, this is pretty simple to debug with the help of git bisect, as long > as patches are sufficiently clean and properly broken down into single > topics. It won't be that easy, as the problems might not be easily reproduceable. Changing the auto-negotiation code will require testing all boards that use it, which is something that can't be done by the person submitting the patch. We will get hard to detect (and debug) breakages. > > It is so much easier just to avoid this by putting it in platform data. > > It's simple, unambiguous and above all, unchanging. > > As I said, this all boils down to who does patches and who accepts them > for mainlibe. -- Regards, Laurent Pinchart -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html