Hi Hans, On Tuesday 03 June 2014 08:52:29 Hans Verkuil wrote: > On 06/03/2014 02:44 AM, Laurent Pinchart wrote: > > Hello, > > > > This patch set adds the missing v4l2-mediabus.h header, required by > > media-ctl. Please see individual patches for details, they're pretty > > straightforward. > > Nack. > > The kernel headers used in v4l-utils are installed via 'make > sync-with-kernel'. So these headers shouldn't be edited, instead > Makefile.am should be updated. In particular, that's where the missing > header should be added. I had seen mentions of sync-with-kernel and for some reason thought it was a script. As I couldn't find it in the repository I decided to sync the headers manually :-/ Thanks for fixing the problem. By the way, what would you think about modifying sync-with-kernel to use installed kernel headers ? > > Laurent Pinchart (2): > > Use installed kernel headers instead of raw kernel headers > > Add the missing v4l2-mediabus.h kernel header > > > > include/linux/dvb/dmx.h | 8 +-- > > include/linux/dvb/frontend.h | 4 -- > > include/linux/dvb/video.h | 12 ++-- > > include/linux/fb.h | 8 +-- > > include/linux/ivtv.h | 6 +- > > include/linux/v4l2-mediabus.h | 147 +++++++++++++++++++++++++++++++++++++ > > include/linux/videodev2.h | 16 ++--- > > 7 files changed, 168 insertions(+), 33 deletions(-) > > create mode 100644 include/linux/v4l2-mediabus.h -- 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
