Hi Mauro, On Monday 29 October 2012 08:21:56 Mauro Carvalho Chehab wrote: > Hans Verkuil yesterday's build still got two warnings at the > generic drivers: > http://hverkuil.home.xs4all.nl/logs/Sunday.log > > They didn't appear at i386 build probably because of some > optimization done there. > > Anyway, fixing them are trivial, so let's do it. > > After applying those patches, the only drivers left producing > warnings are the following platform drivers: > > drivers/media/platform/davinci/dm355_ccdc.c > drivers/media/platform/davinci/dm644x_ccdc.c > drivers/media/platform/davinci/vpbe_osd.c > drivers/media/platform/omap3isp/ispccdc.c > drivers/media/platform/omap3isp/isph3a_aewb.c > drivers/media/platform/omap3isp/isph3a_af.c > drivers/media/platform/omap3isp/isphist.c > drivers/media/platform/omap3isp/ispqueue.c > drivers/media/platform/omap3isp/ispvideo.c > drivers/media/platform/omap/omap_vout.c > drivers/media/platform/s5p-fimc/fimc-capture.c > drivers/media/platform/s5p-fimc/fimc-lite.c > drivers/media/platform/sh_vou.c > drivers/media/platform/soc_camera/sh_mobile_ceu_camera.c > > Platform driver maintainers: please fix, as those warnings could be > hiding real bugs. I've sent you a pull request for v3.7 on Thu, 25 Oct 2012 with two patches that fix most of the warnings: omap3isp: video: Fix warning caused by bad vidioc_s_crop prototype omap3isp: Fix warning caused by bad subdev events operations prototypes The other two OMAP3 ISP warnings are false positive. They will go away when we'll switch to videobuf2 (which is on my to-do list). > Also, removing all warnings is interesting, as they helps to detect when new > possible bugs got introduced. > > I think Hans also use "make W=1" option when doing his tests. > > Mauro Carvalho Chehab (2): > [media] drxk_hard: fix the return code from an error handler > [media] xc4000: Fix a few warnings > > drivers/media/dvb-frontends/drxk_hard.c | 1 + > drivers/media/tuners/xc4000.c | 2 +- > 2 files changed, 2 insertions(+), 1 deletion(-) -- Regards, Laurent Pinchart -- 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
