Hi Antti, On 02/10/2014 05:17 PM, Antti Palosaari wrote: > It appears that controls are ordered by ID number. Change order of > controls by reorganizing assigned IDs now as we can. It is not > reasonable possible after the API is released. Leave some spare > space between IDs too for future extensions.
Am I missing something? I see no reason for this patch or for adding the spare space. Regards, Hans > > Cc: Hans Verkuil <hverk...@xs4all.nl> > Signed-off-by: Antti Palosaari <cr...@iki.fi> > --- > include/uapi/linux/v4l2-controls.h | 16 ++++++++-------- > 1 file changed, 8 insertions(+), 8 deletions(-) > > diff --git a/include/uapi/linux/v4l2-controls.h > b/include/uapi/linux/v4l2-controls.h > index 3cf68a6..cc488c3 100644 > --- a/include/uapi/linux/v4l2-controls.h > +++ b/include/uapi/linux/v4l2-controls.h > @@ -899,13 +899,13 @@ enum v4l2_deemphasis { > #define V4L2_CID_RF_TUNER_CLASS_BASE (V4L2_CTRL_CLASS_RF_TUNER | > 0x900) > #define V4L2_CID_RF_TUNER_CLASS > (V4L2_CTRL_CLASS_RF_TUNER | 1) > > -#define V4L2_CID_LNA_GAIN_AUTO > (V4L2_CID_RF_TUNER_CLASS_BASE + 1) > -#define V4L2_CID_LNA_GAIN (V4L2_CID_RF_TUNER_CLASS_BASE + > 2) > -#define V4L2_CID_MIXER_GAIN_AUTO (V4L2_CID_RF_TUNER_CLASS_BASE + > 3) > -#define V4L2_CID_MIXER_GAIN (V4L2_CID_RF_TUNER_CLASS_BASE + > 4) > -#define V4L2_CID_IF_GAIN_AUTO > (V4L2_CID_RF_TUNER_CLASS_BASE + 5) > -#define V4L2_CID_IF_GAIN (V4L2_CID_RF_TUNER_CLASS_BASE + > 6) > -#define V4L2_CID_BANDWIDTH_AUTO > (V4L2_CID_RF_TUNER_CLASS_BASE + 7) > -#define V4L2_CID_BANDWIDTH (V4L2_CID_RF_TUNER_CLASS_BASE + > 8) > +#define V4L2_CID_BANDWIDTH_AUTO > (V4L2_CID_RF_TUNER_CLASS_BASE + 11) > +#define V4L2_CID_BANDWIDTH (V4L2_CID_RF_TUNER_CLASS_BASE + > 12) > +#define V4L2_CID_LNA_GAIN_AUTO > (V4L2_CID_RF_TUNER_CLASS_BASE + 41) > +#define V4L2_CID_LNA_GAIN (V4L2_CID_RF_TUNER_CLASS_BASE + > 42) > +#define V4L2_CID_MIXER_GAIN_AUTO (V4L2_CID_RF_TUNER_CLASS_BASE + > 51) > +#define V4L2_CID_MIXER_GAIN (V4L2_CID_RF_TUNER_CLASS_BASE + > 52) > +#define V4L2_CID_IF_GAIN_AUTO > (V4L2_CID_RF_TUNER_CLASS_BASE + 61) > +#define V4L2_CID_IF_GAIN (V4L2_CID_RF_TUNER_CLASS_BASE + > 62) > > #endif > -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html