On Wed, 11 Mar 2015 09:10:24 +0100 Hans Verkuil <hverkuil@xxxxxxxxx> wrote: > After some more testing I realized that the 422P format produced > wrong colors and I couldn't get it to work. Since it never worked and > nobody complained about it (and it is a fairly obscure format as well) > I've dropped it. I'm not sure how that format came in anymore; I didn't add it. No objections to its removal. > I also tested RGB444 format for the first time, and that had wrong colors > as well, but that was easy to fix. Finally there was a Bayer format > reported, but it was never implemented. So that too was dropped. The RGB444 change worries me somewhat; that was the default format on the XO1 and worked for years. I vaguely remember some discussions about the ordering of the colors there, but that was a while ago. Did you test it with any of the Sugar apps? In the end, correctness is probably the right way to go (it usually is!), but I'd hate to get a regression report from somebody who is crazy enough to put current kernels on those machines. Fortunately, such people should be rare. Bayer sort-of worked once, honest. I added it for some academic who wanted to do stuff, and was never really able to close the loop on getting it working correctly. It might be worth removing the alleged support from ov7670 as well. In any case, for all of them: Acked-by: Jonathan Corbet <corbet@xxxxxxx> jon -- 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
