Hi, 2012/2/18 Dan Carpenter <dan.carpenter@oracle.com>:

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.