> > * struct tuner: add some type field here too? For radio/tv/...? > > > Doesn't V4L2_TUNER_CAP_LOW cover that already? That one specifies the frequency range (with _LOW it is kHz, without MHz IIRC). > We do need a 'mode' field in v4l2_frequency for that, I think. That > field could also be used for setting spectrum inversion for ATSC, > antenna polarization for sat, etc... I added a type field to the tuner struct, so we can add support for ATSC+stuff later. Also plenty of "reserved" fields to v4l2_frequency ... > >#define VIDIOC_TRY_FMT _IOW ('V', 63, struct v4l2_format) > > > Shouldn't that be: > > #define VIDIOC_TRY_FMT _IOWR ('V', 64, struct v4l2_format) Already fixed that one, I found it too :-) Gerd -- You can't please everybody. And usually if you _try_ to please everybody, the end result is one big mess. -- Linus Torvalds, 2002-04-20