{"id":807607,"url":"http://patchwork.ozlabs.org/api/1.0/covers/807607/?format=json","project":{"id":37,"url":"http://patchwork.ozlabs.org/api/1.0/projects/37/?format=json","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":""},"msgid":"<20170830114946.17743-1-sakari.ailus@linux.intel.com>","date":"2017-08-30T11:49:41","name":"[v6,0/5] Unified fwnode endpoint parser","submitter":{"id":65485,"url":"http://patchwork.ozlabs.org/api/1.0/people/65485/?format=json","name":"Sakari Ailus","email":"sakari.ailus@linux.intel.com"},"series":[{"id":600,"url":"http://patchwork.ozlabs.org/api/1.0/series/600/?format=json","date":"2017-08-30T11:49:41","name":"Unified fwnode endpoint parser","version":6,"mbox":"http://patchwork.ozlabs.org/series/600/mbox/"}],"headers":{"Return-Path":"","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=)","Received":["from vger.kernel.org (vger.kernel.org [209.132.180.67])\n\tby ozlabs.org (Postfix) with ESMTP id 3xj3hq5mcPz9t2R\n\tfor ;\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);\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); 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 ","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":"","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"}