Cover Letter Detail
Show a cover letter.
GET /api/covers/807607/?format=api
{ "id": 807607, "url": "http://patchwork.ozlabs.org/api/covers/807607/?format=api", "web_url": "http://patchwork.ozlabs.org/project/devicetree-bindings/cover/20170830114946.17743-1-sakari.ailus@linux.intel.com/", "project": { "id": 37, "url": "http://patchwork.ozlabs.org/api/projects/37/?format=api", "name": "Devicetree Bindings", "link_name": "devicetree-bindings", "list_id": "devicetree.vger.kernel.org", "list_email": "devicetree@vger.kernel.org", "web_url": "", "scm_url": "", "webscm_url": "", "list_archive_url": "", "list_archive_url_format": "", "commit_url_format": "" }, "msgid": "<20170830114946.17743-1-sakari.ailus@linux.intel.com>", "list_archive_url": null, "date": "2017-08-30T11:49:41", "name": "[v6,0/5] Unified fwnode endpoint parser", "submitter": { "id": 65485, "url": "http://patchwork.ozlabs.org/api/people/65485/?format=api", "name": "Sakari Ailus", "email": "sakari.ailus@linux.intel.com" }, "mbox": "http://patchwork.ozlabs.org/project/devicetree-bindings/cover/20170830114946.17743-1-sakari.ailus@linux.intel.com/mbox/", "series": [ { "id": 600, "url": "http://patchwork.ozlabs.org/api/series/600/?format=api", "web_url": "http://patchwork.ozlabs.org/project/devicetree-bindings/list/?series=600", "date": "2017-08-30T11:49:41", "name": "Unified fwnode endpoint parser", "version": 6, "mbox": "http://patchwork.ozlabs.org/series/600/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/covers/807607/comments/", "headers": { "Return-Path": "<devicetree-owner@vger.kernel.org>", "X-Original-To": "incoming-dt@patchwork.ozlabs.org", "Delivered-To": "patchwork-incoming-dt@bilbo.ozlabs.org", "Authentication-Results": "ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=vger.kernel.org\n\t(client-ip=209.132.180.67; helo=vger.kernel.org;\n\tenvelope-from=devicetree-owner@vger.kernel.org; receiver=<UNKNOWN>)", "Received": [ "from vger.kernel.org (vger.kernel.org [209.132.180.67])\n\tby ozlabs.org (Postfix) with ESMTP id 3xj3hq5mcPz9t2R\n\tfor <incoming-dt@patchwork.ozlabs.org>;\n\tWed, 30 Aug 2017 21:50:03 +1000 (AEST)", "(majordomo@vger.kernel.org) by vger.kernel.org via listexpand\n\tid S1751730AbdH3LuC (ORCPT\n\t<rfc822;incoming-dt@patchwork.ozlabs.org>);\n\tWed, 30 Aug 2017 07:50:02 -0400", "from nblzone-211-213.nblnetworks.fi ([83.145.211.213]:35934 \"EHLO\n\thillosipuli.retiisi.org.uk\" rhost-flags-OK-OK-OK-FAIL)\n\tby vger.kernel.org with ESMTP id S1751357AbdH3Ltt (ORCPT\n\t<rfc822; devicetree@vger.kernel.org>); Wed, 30 Aug 2017 07:49:49 -0400", "from lanttu.localdomain (unknown\n\t[IPv6:2001:1bc8:1a6:d3d5::e1:1002])\n\tby hillosipuli.retiisi.org.uk (Postfix) with ESMTP id D60B0600C9;\n\tWed, 30 Aug 2017 14:49:46 +0300 (EEST)" ], "From": "Sakari Ailus <sakari.ailus@linux.intel.com>", "To": "linux-media@vger.kernel.org", "Cc": "niklas.soderlund@ragnatech.se, robh@kernel.org, hverkuil@xs4all.nl,\n\tlaurent.pinchart@ideasonboard.com, devicetree@vger.kernel.org", "Subject": "[PATCH v6 0/5] Unified fwnode endpoint parser", "Date": "Wed, 30 Aug 2017 14:49:41 +0300", "Message-Id": "<20170830114946.17743-1-sakari.ailus@linux.intel.com>", "X-Mailer": "git-send-email 2.11.0", "MIME-Version": "1.0", "Content-Type": "text/plain; charset=UTF-8", "Content-Transfer-Encoding": "8bit", "Sender": "devicetree-owner@vger.kernel.org", "Precedence": "bulk", "List-ID": "<devicetree.vger.kernel.org>", "X-Mailing-List": "devicetree@vger.kernel.org" }, "content": "Hi folks,\n\nWe have a large influx of new, unmerged, drivers that are now parsing\nfwnode endpoints and each one of them is doing this a little bit\ndifferently. The needs are still exactly the same for the graph data\nstructure is device independent. This is still a non-trivial task and the\nmajority of the driver implementations are buggy, just buggy in different\nways.\n\nFacilitate parsing endpoints by adding a convenience function for parsing\nthe endpoints, and make the omap3isp and rcar-vin drivers use them as an\nexample.\n\nsince v5:\n\n- Use v4l2_async_ prefix for static functions as well (4th patch)\n\n- Use memcpy() to copy array rather than a loop\n\n- Document that the v4l2_async_subdev pointer in driver specific struct\n must be the first member\n\n- Improve documentation of the added functions (4th and 5th\n patches)\n\n\t- Arguments\n\n\t- More thorough explation of the purpose, usage and object\n\t lifetime\n\n- Added acks\n\nsince v4:\n\n- Prepend the set with three documentation fixes.\n\n- The driver's async struct must begin with struct v4l2_async_subdev. Fix this\n for omap3isp and document it.\n\n- Improve documentation for new functions.\n\n- Don't use devm_ family of functions for allocating memory. Introduce\n v4l2_async_notifier_release() to release memory resources.\n\n- Rework both v4l2_async_notifier_fwnode_parse_endpoints() and\n v4l2_async_notifier_fwnode_parse_endpoint() and the local functions they\n call. This should make the code cleaner. Despite the name, for linking\n and typical usage reasons the functions remain in v4l2-fwnode.c.\n\n- Convert rcar-vin to use v4l2_async_notifier_fwnode_parse_endpoint().\n\n- Use kvmalloc() for allocating the notifier's subdevs array.\n\n- max_subdevs argument for notifier_realloc is now the total maximum\n number of subdevs, not the number of available subdevs.\n\n- Use fwnode_device_is_available() to make sure the device actually\n exists.\n\n- Move the note telling v4l2_async_notifier_fwnode_parse_endpoints()\n should not be used by new drivers to the last patch adding\n v4l2_async_notifier_fwnode_parse_endpoint().\n\nsince v3:\n\n- Rebase on current mediatree master.\n\nsince v2:\n\n- Rebase on CCP2 support patches.\n\n- Prepend a patch cleaning up omap3isp driver a little.\n\nsince v1:\n\n- The first patch has been merged (it was a bugfix).\n\n- In anticipation that the parsing can take place over several iterations,\n take the existing number of async sub-devices into account when\n re-allocating an array of async sub-devices.\n\n- Rework the first patch to better anticipate parsing single endpoint at a\n time by a driver.\n\n- Add a second patch that adds a function for parsing endpoints one at a\n time based on port and endpoint numbers.\n\nSakari Ailus (5):\n v4l: fwnode: Move KernelDoc documentation to the header\n v4l: async: Add V4L2 async documentation to the documentation build\n docs-rst: v4l: Include Qualcomm CAMSS in documentation build\n v4l: fwnode: Support generic parsing of graph endpoints in a device\n v4l: fwnode: Support generic parsing of graph endpoints in a single\n port\n\n Documentation/media/kapi/v4l2-async.rst | 3 +\n Documentation/media/kapi/v4l2-core.rst | 1 +\n Documentation/media/v4l-drivers/index.rst | 1 +\n drivers/media/platform/omap3isp/isp.c | 115 ++++---------\n drivers/media/platform/omap3isp/isp.h | 5 +-\n drivers/media/platform/rcar-vin/rcar-core.c | 111 ++++--------\n drivers/media/platform/rcar-vin/rcar-dma.c | 10 +-\n drivers/media/platform/rcar-vin/rcar-v4l2.c | 14 +-\n drivers/media/platform/rcar-vin/rcar-vin.h | 4 +-\n drivers/media/v4l2-core/v4l2-async.c | 16 ++\n drivers/media/v4l2-core/v4l2-fwnode.c | 253 +++++++++++++++++++---------\n include/media/v4l2-async.h | 24 ++-\n include/media/v4l2-fwnode.h | 178 ++++++++++++++++++-\n 13 files changed, 481 insertions(+), 254 deletions(-)\n create mode 100644 Documentation/media/kapi/v4l2-async.rst" }