{"id":9278,"url":"https://patchwork.libcamera.org/api/1.1/covers/9278/?format=json","web_url":"https://patchwork.libcamera.org/cover/9278/","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":"<20200810164306.1472692-1-niklas.soderlund@ragnatech.se>","date":"2020-08-10T16:42:59","name":"[libcamera-devel,v2,0/7] libcamera: Generate unique and stable camera names","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/9278/mbox/","series":[{"id":1212,"url":"https://patchwork.libcamera.org/api/1.1/series/1212/?format=json","web_url":"https://patchwork.libcamera.org/project/libcamera/list/?series=1212","date":"2020-08-10T16:42:59","name":"libcamera: Generate unique and stable camera names","version":2,"mbox":"https://patchwork.libcamera.org/series/1212/mbox/"}],"comments":"https://patchwork.libcamera.org/api/covers/9278/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 07840BD87C\n\tfor ;\n\tMon, 10 Aug 2020 16:43:23 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 90D586105C;\n\tMon, 10 Aug 2020 18:43:22 +0200 (CEST)","from bin-mail-out-06.binero.net (bin-mail-out-06.binero.net\n\t[195.74.38.229])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id C40636038F\n\tfor ;\n\tMon, 10 Aug 2020 18:43:21 +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 9504eefb-db28-11ea-a39b-005056917f90;\n\tMon, 10 Aug 2020 18:43:19 +0200 (CEST)"],"X-Halon-ID":"9504eefb-db28-11ea-a39b-005056917f90","Authorized-sender":"niklas@soderlund.pp.se","From":"=?utf-8?q?Niklas_S=C3=B6derlund?= ","To":"libcamera-devel@lists.libcamera.org","Date":"Mon, 10 Aug 2020 18:42:59 +0200","Message-Id":"<20200810164306.1472692-1-niklas.soderlund@ragnatech.se>","X-Mailer":"git-send-email 2.28.0","MIME-Version":"1.0","Subject":"[libcamera-devel] [PATCH v2 0/7] libcamera: Generate unique and\n\tstable camera names","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\nWith the recent change where the camera names where remade into a camera\nid to better model it's original intent it have become necessary to\nadvance the topic of user-friendly names in applications. This series is\nan attempt to do so.\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\nThis series depends on [1].\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.1.1:1.0-0ac8:3420\n 2: \\_SB_.PCI0.RP05.PXSX-2.4:1.0-046d:0843\n 3: platform/vimc.0 Sensor B\n\n $ cam -l\n Available cameras:\n 1: \\_SB_.PCI0.I2C2.CAM0\n 2: \\_SB_.PCI0.I2C4.CAM1\n\nWith:\n $ cam -l\n Available cameras:\n 1: Venus USB2.0 Camera: Venus USB2 (\\_SB_.PCI0.RP05.PXSX-2.1.1:1.0-0ac8:3420)\n 2: Logitech Webcam C930e (\\_SB_.PCI0.RP05.PXSX-2.4:1.0-046d:0843)\n 3: Internal front camera (platform/vimc.0 Sensor B)\n\n $ cam -l\n Available cameras:\n 1: Internal front camera (\\_SB_.PCI0.I2C2.CAM0)\n 2: Internal front camera (\\_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\n1. [PATCH 0/3] libcamera: Mark controls and property accessors as const\n operations\n\nNiklas Söderlund (7):\n libcamera: properties: Add model property\n libcamera: utils: Add method to strip Unicode 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 | 33 ++++++++++++++++++-\n src/libcamera/camera_sensor.cpp | 2 ++\n src/libcamera/pipeline/uvcvideo/uvcvideo.cpp | 34 +++++++++++---------\n src/libcamera/property_ids.yaml | 7 ++++\n src/libcamera/utils.cpp | 21 ++++++++++++\n src/qcam/dng_writer.cpp | 13 ++++++--\n 7 files changed, 93 insertions(+), 19 deletions(-)"}