[libcamera-devel] [PATCH v6 0/7] libcamera: Allow for user-friendly names in

[Niklas Söderlund]

Hello,

This series trues to advance the topic of user-friendly names in
applications.

Patch 1/7 adds a new camera property 'Model' that may be used to record
a cameras model. Patch 2/7, 3/7, 4/7 and 5/7 then implements this new
property for all pipeline handlers.

Patch 6/7 introduce user-friendly names to the cam utility.

The final patch 7/7 records the model information in the DNG files
created by qcam. More work is needed on top of this series to create
user-friendly names for qcam. My intention is to do this in a follow up
series once this is merged as I fear some bikeshedding on the style of
the user-friendly name so lets battle that out in cam first :-)

Example of listing cameras with and without this series,

Without:
    $ cam -l
    Available cameras:
    1: \_SB_.PCI0.RP05.PXSX-2.4:1.0-046d:0843
    2: platform/vimc.0 Sensor B

    $ cam -l
    Available cameras:
    1: /base/i2c at ff160000/camera at 36
    2: /base/i2c at ff160000/camera at 3c

    $ cam -l
    Available cameras:
    1: /base/soc/i2c0mux/i2c at 1/imx219 at 10

    $ cam -l
    Available cameras:
    1: \_SB_.PCI0.I2C2.CAM0
    2: \_SB_.PCI0.I2C4.CAM1


With:
    $ cam -l
    Available cameras:
    1: External camera 'Logitech Webcam C930e' (\_SB_.PCI0.RP05.PXSX-2.4:1.0-046d:0843)
    2: Internal front camera (platform/vimc.0 Sensor B)

    $ cam -l
    Available cameras:
    1: Internal front camera (/base/i2c at ff160000/camera at 36)
    2: Internal front camera (/base/i2c at ff160000/camera at 3c)

    $ cam -l
    Available cameras:
    1: Internal front camera (/base/soc/i2c0mux/i2c at 1/imx219 at 10)

    $ cam -l
    Available cameras:
    1: Internal front camera (\_SB_.PCI0.I2C2.CAM0)
    2: Internal front camera (\_SB_.PCI0.I2C4.CAM1)

It can be observed above that all Cameras that report an internal 
location do so by stating they face the front. We know this is not true 
for some cameras. This is however not a fault of this series as it only 
prints what is reported by the Camera. Once we teach the Camera to 
report true values for these properties cam will print the correct 
information.

Niklas Söderlund (7):
  libcamera: properties: Add model property
  libcamera: utils: Add method to remove non-ASCII characters
  libcamera: camera_sensor: Set sensor model property
  libcamera: pipeline: uvcvideo: Initialize CameraData from MediaDevice
  libcamera: pipeline: uvcvideo: Set sensor model property
  cam: Print user-friendly camera names
  qcam: dng_writer: Record camera model

 include/libcamera/internal/utils.h           |  2 ++
 src/cam/main.cpp                             | 28 +++++++++++++++++-
 src/libcamera/camera_sensor.cpp              |  2 ++
 src/libcamera/pipeline/uvcvideo/uvcvideo.cpp | 31 ++++++++++----------
 src/libcamera/property_ids.yaml              | 23 +++++++++++++++
 src/libcamera/utils.cpp                      | 17 +++++++++++
 src/qcam/dng_writer.cpp                      | 13 ++++++--
 7 files changed, 97 insertions(+), 19 deletions(-)

--
2.28.0