Hi, 2012/2/18 Dan Carpenter <dan.carpenter@xxxxxxxxxx>: > The first line of the comment here should be a blank line. Ok. > > /* > * blah blah blah > * blah blah blah > */ > >> + /* A new struct easycap is always allocated when interface 0 is probed. >> + * It is not possible here to free any existing struct easycap. >> + * This should have been done by easycap_delete() when the device was >> + * physically unplugged. >> + * The allocated struct easycap is saved for later usage when >> + * interfaces 1 and 2 are probed. >> + */ > ^ > Could you make this asterisk line up with the others? Of course. If I send more patches about easycap driver, can I base it on my current local branch? I mean, based on all these patches? Thanks, Ezequiel. _______________________________________________ Kernelnewbies mailing list Kernelnewbies@xxxxxxxxxxxxxxxxx http://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies