[libcamera-devel] [PATCH v2 0/3] android: Handle internal UVC cameras

[Umang Jain]

Ouch, didn't delete patch dir before sending new series, please ignore!

Thanks!

On 7/30/21 1:58 PM, Umang Jain wrote:
> libcamera-core all UVC cameras as 'external' (which makes sense
> as there is not much information to accurately determine location).
> However, in the HAL layer, we can probably determine accurately
> by looking at HAL config file camera entries.
>
> This series addresses the handling of such cameras, where the libcamera
> reports properties::CameraLocationExternal, but we know one of the UVC
> is internal/integrated to the system. It should be treated as internal.
>
> Testing:
> With nautilus, it has one UVC and one IPU3 cameras. With these patches
> applied, both these cameras have id as '0' & '1' (i.e. internal).
>
> With camera service still running, I hotplugged a external UVC camera,
> and it got the camera numerical id as '1000'.
>
> Change in v2:
> - Introduce a re-factor patch 1/3
> - Change comments in 2/3
>
> Umang Jain (3):
>    android: Instantiate CameraDevice after checking HAL config validity
>    android: Override camera is "Internal" provided if found on HAL config
>    android: nautilus: Add camera HAL configuration
>
>   src/android/camera_device.cpp             | 10 +++++++++-
>   src/android/camera_hal_manager.cpp        | 24 ++++++++++++++++++++---
>   src/android/data/nautilus/camera_hal.yaml |  8 ++++++++
>   3 files changed, 38 insertions(+), 4 deletions(-)
>   create mode 100644 src/android/data/nautilus/camera_hal.yaml
>