On Tue, Mar 15, 2011 at 19:29:40, Hadli, Manjunath wrote: > Create platform devices for various video modules like venc,osd, > vpbe and v4l2 driver for dm644x. > > Signed-off-by: Manjunath Hadli <manjunath.hadli@xxxxxx> > --- > +struct venc_platform_data dm644x_venc_pdata = { > + .venc_type = VPBE_VERSION_1, > + .setup_clock = dm644x_venc_setup_clock, > +}; Sparse pointed out that this symbol can be static. Can you please build the complete series with C=1 on the command line? This will enable sparse check on all files being re-compiled. I also noticed some sparse warnings on vpbe_venc.c Also, please build with CONFIG_DEBUG_SECTION_MISMATCH=y on the command line. I noticed some section mismatches as well with the new code. Thanks, Sekhar -- 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