Hi Laurent,

Em 03-03-2011 07:10, Laurent Pinchart escreveu:
> On Thursday 03 March 2011 03:16:32 Kim, HeungJun wrote:
>> As following to change the boolean type of V4L2_CID_FOCUS_AUTO to menu
>> type, this uvc is modified the usage of V4L2_CID_FOCUS_AUTO, maintaining
>> v4l2 menu index.
>>
>> Signed-off-by: Heungjun Kim <riverful....@samsung.com>
>> Signed-off-by: Kyungmin Park <kyungmin.p...@samsung.com>
> 
> Acked-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>

I'm assuming that you'll be applying those patches on your tree and sending
me a pull request, right?

Thanks!
Mauro

> 
>> ---
>>  drivers/media/video/uvc/uvc_ctrl.c |    9 ++++++++-
>>  1 files changed, 8 insertions(+), 1 deletions(-)
>>
>> diff --git a/drivers/media/video/uvc/uvc_ctrl.c
>> b/drivers/media/video/uvc/uvc_ctrl.c index 59f8a9a..064827f 100644
>> --- a/drivers/media/video/uvc/uvc_ctrl.c
>> +++ b/drivers/media/video/uvc/uvc_ctrl.c
>> @@ -333,6 +333,11 @@ static struct uvc_menu_info exposure_auto_controls[] =
>> { { 8, "Aperture Priority Mode" },
>>  };
>>
>> +static struct uvc_menu_info focus_auto_controls[] = {
>> +    { 0, "Manual Mode" },
>> +    { 1, "Auto Mode" },
>> +};
>> +
>>  static __s32 uvc_ctrl_get_zoom(struct uvc_control_mapping *mapping,
>>      __u8 query, const __u8 *data)
>>  {
>> @@ -560,8 +565,10 @@ static struct uvc_control_mapping uvc_ctrl_mappings[]
>> = { .selector        = UVC_CT_FOCUS_AUTO_CONTROL,
>>              .size           = 1,
>>              .offset         = 0,
>> -            .v4l2_type      = V4L2_CTRL_TYPE_BOOLEAN,
>> +            .v4l2_type      = V4L2_CTRL_TYPE_MENU,
>>              .data_type      = UVC_CTRL_DATA_TYPE_BOOLEAN,
>> +            .menu_info      = focus_auto_controls,
>> +            .menu_count     = ARRAY_SIZE(focus_auto_controls),
>>      },
>>      {
>>              .id             = V4L2_CID_IRIS_ABSOLUTE,
> 

--
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