Hi Sachin, >> + isp->refcount++; >> + return ret; > > You can directly return 0 here instead of creating a local variable > 'ret' which is not used anywhere else. Ok. >> + > > nit: Please consider changing "Adds" to "Add" in the patch titles of > this series during the next spin. > Ok will change like that. Regards Arun -- 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