Cover Letter Detail
Show a cover letter.
GET /api/covers/809342/?format=api
{ "id": 809342, "url": "http://patchwork.ozlabs.org/api/covers/809342/?format=api", "web_url": "http://patchwork.ozlabs.org/project/devicetree-bindings/cover/20170903174958.27058-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": "<20170903174958.27058-1-sakari.ailus@linux.intel.com>", "list_archive_url": null, "date": "2017-09-03T17:49:40", "name": "[v7,00/18] Unified fwnode endpoint parser, async sub-device notifier support, N9 flash DTS", "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/20170903174958.27058-1-sakari.ailus@linux.intel.com/mbox/", "series": [ { "id": 1266, "url": "http://patchwork.ozlabs.org/api/series/1266/?format=api", "web_url": "http://patchwork.ozlabs.org/project/devicetree-bindings/list/?series=1266", "date": "2017-09-03T17:49:53", "name": "Unified fwnode endpoint parser, async sub-device notifier support, N9 flash DTS", "version": 7, "mbox": "http://patchwork.ozlabs.org/series/1266/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/covers/809342/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 3xlgWS2nRsz9t3f\n\tfor <incoming-dt@patchwork.ozlabs.org>;\n\tMon, 4 Sep 2017 03:51:00 +1000 (AEST)", "(majordomo@vger.kernel.org) by vger.kernel.org via listexpand\n\tid S1753173AbdICRuD (ORCPT\n\t<rfc822;incoming-dt@patchwork.ozlabs.org>);\n\tSun, 3 Sep 2017 13:50:03 -0400", "from nblzone-211-213.nblnetworks.fi ([83.145.211.213]:49376 \"EHLO\n\thillosipuli.retiisi.org.uk\" rhost-flags-OK-OK-OK-FAIL)\n\tby vger.kernel.org with ESMTP id S1753061AbdICRuC (ORCPT\n\t<rfc822; devicetree@vger.kernel.org>); Sun, 3 Sep 2017 13:50:02 -0400", "from lanttu.localdomain (lanttu-e.localdomain [192.168.1.64])\n\tby hillosipuli.retiisi.org.uk (Postfix) with ESMTP id 079AE600E5;\n\tSun, 3 Sep 2017 20:49:59 +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,\n\tpavel@ucw.cz, sre@kernel.org", "Subject": "[PATCH v7 00/18] Unified fwnode endpoint parser,\n\tasync sub-device notifier support, N9 flash DTS", "Date": "Sun, 3 Sep 2017 20:49:40 +0300", "Message-Id": "<20170903174958.27058-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\nTo show where we're getting with this, I've added support for async\nsub-device notifier support that is notifiers that can be registered by\nsub-device drivers as well as V4L2 fwnode improvements to make use of them\nand the DTS changes for the Nokia N9. Some of these patches I've posted\npreviously in this set here:\n\n<URL:http://www.spinics.net/lists/linux-media/msg118764.html>\n\nSince that, the complete callback of the master notifier registering the\nV4L2 device is only called once all sub-notifiers have been completed as\nwell. This way the device node creation can be postponed until all devices\nhave been successfully initialised.\n\nWith this, the as3645a driver successfully registers a sub-device to the\nmedia device created by the omap3isp driver. The kernel also has the\ninformation it's related to the sensor driven by the smiapp driver but we\ndon't have a way to expose that information yet.\n\nsince v6:\n\n- Drop the last patch that added variant for parsing endpoints given\n specific port and endpoints numbers.\n\n- Separate driver changes from the fwnode endpoint parser patch into two\n patches. rcar-vin driver is now using the name function.\n\n- Use -ENOTCONN to tell the parser that and endpoint (or a reference) is\n to be ignored.\n\n- parse_endpoint and parse_single callback functions are now optional and\n documented as such.\n\n- Added Laurent's patch adding notifier operations struct which I rebase\n on the fwnode parser patchset. I wrote another patch to call the\n notifier operations through macros.\n\n- Add DT bindings for flash and lens devices.\n\n- V4L2 fwnode parser for references (such as flash and lens).\n\n- Added smiapp driver support for async sub-devices (lens and flash).\n\n- Added a few fixes for omap3isp.\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\nLaurent Pinchart (1):\n v4l: async: Move async subdev notifier operations to a separate\n structure\n\nSakari Ailus (17):\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 omap3isp: Use generic parser for parsing fwnode endpoints\n rcar-vin: Use generic parser for parsing fwnode endpoints\n omap3isp: Fix check for our own sub-devices\n omap3isp: Print the name of the entity where no source pads could be\n found\n v4l: async: Introduce macros for calling async ops callbacks\n v4l: async: Register sub-devices before calling bound callback\n v4l: async: Allow binding notifiers to sub-devices\n dt: bindings: Add a binding for flash devices associated to a sensor\n dt: bindings: Add lens-focus binding for image sensors\n v4l2-fwnode: Add convenience function for parsing generic references\n v4l2-fwnode: Add convenience function for parsing common external refs\n smiapp: Add support for flash and lens devices\n arm: dts: omap3: N9/N950: Add flash references to the camera\n\n .../devicetree/bindings/media/video-interfaces.txt | 10 +\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 arch/arm/boot/dts/omap3-n9.dts | 1 +\n arch/arm/boot/dts/omap3-n950-n9.dtsi | 4 +-\n arch/arm/boot/dts/omap3-n950.dts | 1 +\n drivers/media/i2c/smiapp/smiapp-core.c | 29 +-\n drivers/media/i2c/smiapp/smiapp.h | 4 +-\n drivers/media/platform/am437x/am437x-vpfe.c | 8 +-\n drivers/media/platform/atmel/atmel-isc.c | 10 +-\n drivers/media/platform/atmel/atmel-isi.c | 10 +-\n drivers/media/platform/davinci/vpif_capture.c | 8 +-\n drivers/media/platform/davinci/vpif_display.c | 8 +-\n drivers/media/platform/exynos4-is/media-dev.c | 8 +-\n drivers/media/platform/omap3isp/isp.c | 127 +++-----\n drivers/media/platform/omap3isp/isp.h | 5 +-\n drivers/media/platform/pxa_camera.c | 8 +-\n drivers/media/platform/qcom/camss-8x16/camss.c | 8 +-\n drivers/media/platform/rcar-vin/rcar-core.c | 122 +++-----\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/platform/rcar_drif.c | 10 +-\n drivers/media/platform/soc_camera/soc_camera.c | 14 +-\n drivers/media/platform/stm32/stm32-dcmi.c | 10 +-\n drivers/media/platform/ti-vpe/cal.c | 8 +-\n drivers/media/platform/xilinx/xilinx-vipp.c | 8 +-\n drivers/media/v4l2-core/v4l2-async.c | 184 +++++++++---\n drivers/media/v4l2-core/v4l2-fwnode.c | 319 ++++++++++++++++-----\n drivers/staging/media/imx/imx-media-dev.c | 8 +-\n include/media/v4l2-async.h | 78 ++++-\n include/media/v4l2-fwnode.h | 165 ++++++++++-\n 33 files changed, 862 insertions(+), 346 deletions(-)\n create mode 100644 Documentation/media/kapi/v4l2-async.rst" }