Hi Sergei, A few high level comments (I'll look at the patch itself later): - There is no MAINTAINERS entry, please add one. - Don't attach the patch, post it inline (ideally with 'git send-email') - Split up the patch into 4 separate patches: bindings, doc changes, driver and MAINTAINERS patch. This will make it easier to review. - Please give the v4l2-compliance output in the cover letter of the v8 patch series. I can't merge this driver without being certain there are no compliance issues. - You also have IMR-LSX3 and IMR-LX3 patches, why not add them to the patch series? I can review the set as a single unit. Up to you, though. Regards, Hans -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html