Cover Letter Detail
Show a cover letter.
GET /api/covers/3711/?format=api
{ "id": 3711, "url": "https://patchwork.libcamera.org/api/covers/3711/?format=api", "web_url": "https://patchwork.libcamera.org/cover/3711/", "project": { "id": 1, "url": "https://patchwork.libcamera.org/api/projects/1/?format=api", "name": "libcamera", "link_name": "libcamera", "list_id": "libcamera_core", "list_email": "libcamera-devel@lists.libcamera.org", "web_url": "", "scm_url": "", "webscm_url": "" }, "msgid": "<20200508100158.3437161-1-jacopo@jmondi.org>", "date": "2020-05-08T10:01:45", "name": "[libcamera-devel,v10,00/13] media: report camera properties", "submitter": { "id": 3, "url": "https://patchwork.libcamera.org/api/people/3/?format=api", "name": "Jacopo Mondi", "email": "jacopo@jmondi.org" }, "mbox": "https://patchwork.libcamera.org/cover/3711/mbox/", "series": [ { "id": 888, "url": "https://patchwork.libcamera.org/api/series/888/?format=api", "web_url": "https://patchwork.libcamera.org/project/libcamera/list/?series=888", "date": "2020-05-08T10:01:45", "name": "media: report camera properties", "version": 10, "mbox": "https://patchwork.libcamera.org/series/888/mbox/" } ], "comments": "https://patchwork.libcamera.org/api/covers/3711/comments/", "headers": { "Return-Path": "<jacopo@jmondi.org>", "Received": [ "from relay9-d.mail.gandi.net (relay9-d.mail.gandi.net\n\t[217.70.183.199])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 49FE7603E3\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tFri, 8 May 2020 11:58:55 +0200 (CEST)", "from localhost.localdomain (2-224-242-101.ip172.fastwebnet.it\n\t[2.224.242.101]) (Authenticated sender: jacopo@jmondi.org)\n\tby relay9-d.mail.gandi.net (Postfix) with ESMTPSA id D5E95FF806;\n\tFri, 8 May 2020 09:58:52 +0000 (UTC)" ], "X-Originating-IP": "2.224.242.101", "From": "Jacopo Mondi <jacopo@jmondi.org>", "To": "linux-media@vger.kernel.org (open list:MEDIA INPUT INFRASTRUCTURE\n\t(V4L/DVB)), libcamera-devel@lists.libcamera.org,\n\tMauro Carvalho Chehab <mchehab@kernel.org>,\n\tHans Verkuil <hverkuil-cisco@xs4all.nl>,\n\tSakari Ailus <sakari.ailus@linux.intel.com>,\n\tLaurent Pinchart <laurent.pinchart@ideasonboard.com>", "Cc": "Jacopo Mondi <jacopo@jmondi.org>,\n\ttfiga@google.com,\n\tpavel@ucw.cz", "Date": "Fri, 8 May 2020 12:01:45 +0200", "Message-Id": "<20200508100158.3437161-1-jacopo@jmondi.org>", "X-Mailer": "git-send-email 2.26.1", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "Subject": "[libcamera-devel] [PATCH v10 00/13] media: report camera properties", "X-BeenThere": "libcamera-devel@lists.libcamera.org", "X-Mailman-Version": "2.1.29", "Precedence": "list", "List-Id": "<libcamera-devel.lists.libcamera.org>", "List-Unsubscribe": "<https://lists.libcamera.org/options/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=unsubscribe>", "List-Archive": "<https://lists.libcamera.org/pipermail/libcamera-devel/>", "List-Post": "<mailto:libcamera-devel@lists.libcamera.org>", "List-Help": "<mailto:libcamera-devel-request@lists.libcamera.org?subject=help>", "List-Subscribe": "<https://lists.libcamera.org/listinfo/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=subscribe>", "X-List-Received-Date": "Fri, 08 May 2020 09:58:55 -0000" }, "content": "Hello,\n this v10 is not just a rename s/location/orientation as the documentation\naround the property has changed slightly as well and should probably be re-read.\n\nAnyway, most notable change is indeed the usa of 'orientation' in place of\nlocation, so that we have an 'orientation' DT property, a\nV4L2_CID_CAMERA_ORIENTATION control id and V4L2_ORIENTATION_* control values.\n\nA new patch 'dt-bindings: Add media properties' add an header to help DT users\nby providing macros for the currently supported locations.\n\nI've added a patch for a new sensor driver I have used for testing (imx219) and\nmade the V4L2_CID_CAMERA_ORIENTATION a menu control as it was meant to be.\n\nI know the additional DT header could slow the series inclusion, I'm fine\nleaving it out if it proves controversial.\n\nThe result of the two new controls inspected with v4l2-ctl -L for a camera\nwith EXTERNAL orientation and 180 degrees rotation is the following:\n\n------------------------------------------------------------------------------------------------------------------\nCamera Controls\n\n camera_orientation 0x009a0922 (menu) : min=0 max=2 default=2 value=2 flags=read-only\n\t\t\t\t0: Front Camera\n\t\t\t\t1: Back Camera\n\t\t\t\t2: External Camera\n camera_sensor_rotation 0x009a0923 (int) : min=180 max=180 step=1 default=180 value=180 flags=read-only\n------------------------------------------------------------------------------------------------------------------\n\nThanks\n j\n\nv9->v10:\n- s/location/orientation and documentation update\n- Add DT bindings header for media properties\n- Make V4L2_CID_CAMERA_ORIENTATION a TYPE_MENU control\n- Add patch for imx219\n\nv8->v9:\n- Rebased on media master which has moved media documentation\n\nv7->v8:\n- Add Rob's ack to 03/11\n- Address Hans typographical comments in 03/11\n\nJacopo Mondi (13):\n dt-bindings: video-interfaces: Document 'orientation' property\n dt-bindings: video-interface: Replace 'rotation' description\n dt-bindings: Add media properties\n media: v4l2-ctrl: Document V4L2_CID_CAMERA_ORIENTATION\n media: v4l2-ctrl: Document V4L2_CID_CAMERA_SENSOR_ROTATION\n media: v4l2-ctrls: Add camera orientation and rotation\n media: v4l2-fwnode: Add helper to parse device properties\n include: v4l2-ctrl: Sort forward declarations\n media: v4l2-ctrls: Sort includes alphabetically\n media: v4l2-ctrls: Add helper to register properties\n media: i2c: ov5670: Parse and register properties\n media: i2c: ov13858: Parse and register properties\n media: i2c: imx219: Parse and register properties\n\n .../bindings/media/video-interfaces.txt | 372 +++++++++++++++++-\n .../media/v4l/ext-ctrls-camera.rst | 151 +++++++\n drivers/media/i2c/imx219.c | 12 +-\n drivers/media/i2c/ov13858.c | 13 +-\n drivers/media/i2c/ov5670.c | 14 +-\n drivers/media/v4l2-core/v4l2-ctrls.c | 63 ++-\n drivers/media/v4l2-core/v4l2-fwnode.c | 42 ++\n include/dt-bindings/media/video-interfaces.h | 15 +\n include/media/v4l2-ctrls.h | 34 +-\n include/media/v4l2-fwnode.h | 47 +++\n include/uapi/linux/v4l2-controls.h | 7 +\n 11 files changed, 755 insertions(+), 15 deletions(-)\n create mode 100644 include/dt-bindings/media/video-interfaces.h\n\n--\n2.26.1" }