[libcamera-devel] [PATCH v2 1/4] libcamera: pipeline: uvcvideo: Treat all UVC cameras as external

[Kieran Bingham]

Hi Umang,

On 10/08/2020 13:04, Umang Jain wrote:
> Treat all UVC cameras as external for android framework to know that
> they are external.
> 
> Signed-off-by: Umang Jain <email at uajain.com>
> ---
>  src/libcamera/pipeline/uvcvideo/uvcvideo.cpp | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/src/libcamera/pipeline/uvcvideo/uvcvideo.cpp b/src/libcamera/pipeline/uvcvideo/uvcvideo.cpp
> index bc892ec..ed61d12 100644
> --- a/src/libcamera/pipeline/uvcvideo/uvcvideo.cpp
> +++ b/src/libcamera/pipeline/uvcvideo/uvcvideo.cpp
> @@ -14,6 +14,7 @@
>  #include <libcamera/camera.h>
>  #include <libcamera/control_ids.h>
>  #include <libcamera/controls.h>
> +#include <libcamera/property_ids.h>
>  #include <libcamera/request.h>
>  #include <libcamera/stream.h>
>  
> @@ -500,6 +501,9 @@ int UVCCameraData::init(MediaEntity *entity)
>  
>  	video_->bufferReady.connect(this, &UVCCameraData::bufferReady);
>  
> +	/* Initialise Camera Property. Treat all UVC cameras as external. */

Probably don't need to say "Initialise Camera Property", but either way.

Reviewed-by: Kieran Bingham <kieran.bingham at ideasonboard.com>

> +	properties_.set(properties::Location, properties::CameraLocationExternal);
> +
>  	/* Initialise the supported controls. */
>  	ControlInfoMap::Map ctrls;
>  
> 

-- 
Regards
--
Kieran