{"id":9829,"url":"https://patchwork.libcamera.org/api/1.1/covers/9829/?format=json","web_url":"https://patchwork.libcamera.org/cover/9829/","project":{"id":1,"url":"https://patchwork.libcamera.org/api/1.1/projects/1/?format=json","name":"libcamera","link_name":"libcamera","list_id":"libcamera_core","list_email":"libcamera-devel@lists.libcamera.org","web_url":"","scm_url":"","webscm_url":""},"msgid":"<20200925150743.1822226-1-niklas.soderlund@ragnatech.se>","date":"2020-09-25T15:07:36","name":"[libcamera-devel,v5,0/7] libcamera: Allow for user-friendly names in applications","submitter":{"id":5,"url":"https://patchwork.libcamera.org/api/1.1/people/5/?format=json","name":"Niklas Söderlund","email":"niklas.soderlund@ragnatech.se"},"mbox":"https://patchwork.libcamera.org/cover/9829/mbox/","series":[{"id":1328,"url":"https://patchwork.libcamera.org/api/1.1/series/1328/?format=json","web_url":"https://patchwork.libcamera.org/project/libcamera/list/?series=1328","date":"2020-09-25T15:07:36","name":"libcamera: Allow for user-friendly names in applications","version":5,"mbox":"https://patchwork.libcamera.org/series/1328/mbox/"}],"comments":"https://patchwork.libcamera.org/api/covers/9829/comments/","headers":{"Return-Path":"","X-Original-To":"parsemail@patchwork.libcamera.org","Delivered-To":"parsemail@patchwork.libcamera.org","Received":["from lancelot.ideasonboard.com (lancelot.ideasonboard.com\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id 1A2F9C3B5B\n\tfor ;\n\tFri, 25 Sep 2020 15:07:59 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 8022D63050;\n\tFri, 25 Sep 2020 17:07:58 +0200 (CEST)","from bin-mail-out-05.binero.net (bin-mail-out-05.binero.net\n\t[195.74.38.228])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 9F87862FD8\n\tfor ;\n\tFri, 25 Sep 2020 17:07:56 +0200 (CEST)","from bismarck.berto.se (p54ac52a8.dip0.t-ipconnect.de\n\t[84.172.82.168]) by bin-vsp-out-02.atm.binero.net (Halon) with ESMTPA\n\tid e1e1e52a-ff40-11ea-a39b-005056917f90;\n\tFri, 25 Sep 2020 17:07:55 +0200 (CEST)"],"X-Halon-ID":"e1e1e52a-ff40-11ea-a39b-005056917f90","Authorized-sender":"niklas.soderlund@fsdn.se","From":"=?utf-8?q?Niklas_S=C3=B6derlund?= ","To":"libcamera-devel@lists.libcamera.org","Date":"Fri, 25 Sep 2020 17:07:36 +0200","Message-Id":"<20200925150743.1822226-1-niklas.soderlund@ragnatech.se>","X-Mailer":"git-send-email 2.28.0","MIME-Version":"1.0","Subject":"[libcamera-devel] [PATCH v5 0/7] libcamera: Allow for user-friendly\n\tnames in applications","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "},"content":"Hello,\n\nThis series trues to advance the topic of user-friendly names in \napplications.\n\nPatch 1/7 adds a new camera property 'Model' that may be used to record\na cameras model. Patch 2/7, 3/7, 4/7 and 5/7 then implements this new\nproperty for all pipeline handlers.\n\nPatch 6/7 introduce user-friendly names to the cam utility.\n\nThe final patch 7/7 records the model information in the DNG files\ncreated by qcam. More work is needed on top of this series to create\nuser-friendly names for qcam. My intention is to do this in a follow up\nseries once this is merged as I fear some bikeshedding on the style of\nthe user-friendly name so lets battle that out in cam first :-)\n\nExample of listing cameras with and without this series,\n\nWithout:\n $ cam -l\n Available cameras:\n 1: \\_SB_.PCI0.RP05.PXSX-2.4:1.0-046d:0843\n 2: platform/vimc.0 Sensor B\n\n $ cam -l\n Available cameras:\n 1: /base/i2c@ff160000/camera@36\n 2: /base/i2c@ff160000/camera@3c\n\n $ cam -l\n Available cameras:\n 1: /base/soc/i2c0mux/i2c@1/imx219@10\n\n $ cam -l\n Available cameras:\n 1: \\_SB_.PCI0.I2C2.CAM0\n 2: \\_SB_.PCI0.I2C4.CAM1\n\n\nWith:\n $ cam -l\n Available cameras:\n 1: External camera Logitech Webcam C930e (\\_SB_.PCI0.RP05.PXSX-2.4:1.0-046d:0843)\n 2: Internal front camera Sensor B (platform/vimc.0 Sensor B)\n\n $ cam -l\n Available cameras:\n 1: Internal front camera ov5695 (/base/i2c@ff160000/camera@36)\n 2: Internal front camera ov2685 (/base/i2c@ff160000/camera@3c)\n\n $ cam -l\n Available cameras:\n 1: Internal front camera imx219 (/base/soc/i2c0mux/i2c@1/imx219@10)\n\n $ cam -l\n Available cameras:\n 1: Internal front camera ov13858 (\\_SB_.PCI0.I2C2.CAM0)\n 2: Internal front camera ov5670 (\\_SB_.PCI0.I2C4.CAM1)\n\nIt can be observed above that all Cameras that report the Location\nproperty do so by stating they face the front. We know this is not true\nfor some devices. This is however not a fault of this series as it only\nprints what is reported by the Camera. Once we teach the Camera to\nreport true values for these properties cam will print the correct\ninformation.\n\nNiklas Söderlund (7):\n libcamera: properties: Add model property\n libcamera: utils: Add method to remove non-ASCII characters\n libcamera: camera_sensor: Set sensor model property\n libcamera: pipeline: uvcvideo: Initialize CameraData from MediaDevice\n libcamera: pipeline: uvcvideo: Set sensor model property\n cam: Print user-friendly camera names\n qcam: dng_writer: Record camera model\n\n include/libcamera/internal/utils.h | 2 ++\n src/cam/main.cpp | 31 +++++++++++++++++++-\n src/libcamera/camera_sensor.cpp | 2 ++\n src/libcamera/pipeline/uvcvideo/uvcvideo.cpp | 31 ++++++++++----------\n src/libcamera/property_ids.yaml | 20 +++++++++++++\n src/libcamera/utils.cpp | 17 +++++++++++\n src/qcam/dng_writer.cpp | 13 ++++++--\n 7 files changed, 97 insertions(+), 19 deletions(-)"}