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

Reply via email to