Hi Devin, On Sun, Mar 29, 2009 at 09:44:25AM -0400, Devin Heitmueller wrote: > On Sun, Mar 29, 2009 at 8:42 AM, Janne Grunau <j@xxxxxxxxxx> wrote: > > > > I'm not against this change, but you should also get rid of the "i" > variable and the au0828_instance list (since the v4l2_device.name was > the only purpose for both). done > Also, your subject didn't really match the function of the patch. Had > I not looked at the patch itself, I would have only thought you were > changing the v4l2_device_register(). yeah, the subject was bad for au0828 and cx231xx but I had trouble to find find something better. I splitted the patches nao. > Please put me on the CC: for anything related to au0828 analog > support, since I authored the code in question. noted, Sorry I had only looked at the au0228-core.c header which notes only Steven. Janne -- 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