> i think at least these three are unused and should get punted > >> +static int __devinit bcap_probe(struct platform_device *pdev) >> +{ >> + struct bcap_device *bcap_dev; >> + struct video_device *vfd; >> + struct i2c_adapter *i2c_adap; > > you need to include linux/i2c.h for this > no, bfin_capture.h already contains this. -- 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