Hi Michael, On Thursday 10 March 2011 16:30:48 Michael Jones wrote: > On 03/10/2011 11:21 AM, Laurent Pinchart wrote: > > Hi Michael, > > [snip] > > > I've had a closer look at the boards I have here, and it turns out one of > > them connects a 10-bit sensor to DATA[11:2] :-/ data_lane_shift is thus > > needed for it. > > > > I'm fine with leaving data_lane_shift out of this patch, but can you > > submit a second patch to add it back ? I'd rather avoid applying a patch > > that breaks one of my boards and then have to fix it myself :-) > > OK, but in that case I'd rather incorporate it into this last patch than > introduce a new patch for it. I don't think it will be very complex and > they logically belong together. I had just been hoping to avoid > implementing it altogether. As you wish. You can also submit two patches and then squash them together if it makes it simpler to develop/review the code. -- 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