Hi Hans, On 04/30/2012 06:11 PM, Hans Verkuil wrote: > Hi Sylwester! > > I've finished my review. You made excellent documentation for the new controls! Thank you, and thanks for your time! :-) It took me quite a few hours to prepare it, and lack of proper sensor datasheets didn't make this task any easier. > Other than some small stuff the only thing I am unhappy about is the use of menu > controls for what are currently just boolean controls. > > I am inclined to make them boolean controls and add a comment that they may be > changed to menu controls in the future. That shouldn't be a problem as long as > the control values 0 and 1 retain their meaning. OK, I like the idea. The menus looked rather ugly, I agree. I'll revert those controls back to a boolean type and add a small note in the documentation. In fact this solves one of the major open issues I had. I tried to align start of second columns in the nested 2-column tables (<entrytbl></entrytbl>) containing a menu item identifiers and their description. I tried 'align', 'colspec', 'spanspec' but could get expected results and I gave up, spending hours on modify/re-compile. I couldn't even enable the column border inside 'entrytbl'. I could only align the columns by getting rid of 'entrytbl', but that's not a solution. It doesn't look that bad with different indentation at each control's description after all, maybe I'll find some time to dig in it later. I'm going to incorporate all comments and resend whole patch set at end of the week. -- Regards, Sylwester -- 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
