"Guruswamy, Senthilvadivu" <svadivu@xxxxxx> writes: [...] >> > +/* DSS HW IP initialisation */ >> > +static int omap_dsshw_probe(struct platform_device *pdev) >> > +{ >> > + return 0; >> > +} >> > + >> > +static int omap_dsshw_remove(struct platform_device *pdev) >> > +{ >> > + return 0; >> > +} >> >> It's not customary to create dummy, empty functions in one patch and >> then fill them in later. In this case, you create them here, >> move them >> in PATCH 05/20, then fill them in later. >> >> It would be easier to review if you created the platform_drivers when >> needed. >> >> Anyways, I'm not crazy about this approach, but Tomi can decide. >> > [Senthil] As I mentioned in my TODO list, I will try to move all the probe/remove to > the corresponding driver files as directly in my next version. Sorry, I missed that in your TODO list. Thanks, Kevin -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html