On Sat, 31 Jul 2010, Robert Jarzmik wrote: > Michael Grzeschik <m.grzeschik@xxxxxxxxxxxxxx> writes: > > > From: Philipp Wiesner <p.wiesner@xxxxxxxxx> > > > > Added this info to Kconfig and mt9m111.c, some comment cleanup, > > replaced 'mt9m11x'-statements by clarifications or driver name. > > Driver is fully compatible to mt9m131 which has only additional functions > > compared to mt9m111. Those aren't used anyway at the moment. > > <zip> > > > > - dev_info(&client->dev, "Detected a MT9M11x chip ID %x\n", data); > > - > > Why this one ? It signals a sensor was successfully detected. Maybe a > replacement from MT9M11x to MT9M1xx would be better ? Or if your real intention > is to remove the message, then transform it to dev_dbg(), and say why you did it > in the commit message. Robert, the message is not removed, it is moved into two chip ID switch cases. Thanks Guennadi --- Guennadi Liakhovetski, Ph.D. Freelance Open-Source Software Developer http://www.open-technology.de/ -- 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