Cover Letter Detail
Show a cover letter.
GET /api/covers/813809/?format=api
{ "id": 813809, "url": "http://patchwork.ozlabs.org/api/covers/813809/?format=api", "web_url": "http://patchwork.ozlabs.org/project/devicetree-bindings/cover/20170914122914.14122-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": "<20170914122914.14122-1-sakari.ailus@linux.intel.com>", "list_archive_url": null, "date": "2017-09-14T12:29:09", "name": "[RFC,0/5] Align and document return values of phandle and reference parsing for OF and ACPI", "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/20170914122914.14122-1-sakari.ailus@linux.intel.com/mbox/", "series": [ { "id": 3089, "url": "http://patchwork.ozlabs.org/api/series/3089/?format=api", "web_url": "http://patchwork.ozlabs.org/project/devicetree-bindings/list/?series=3089", "date": "2017-09-14T12:29:09", "name": "Align and document return values of phandle and reference parsing for OF and ACPI", "version": 1, "mbox": "http://patchwork.ozlabs.org/series/3089/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/covers/813809/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 3xtHsJ0Gy8z9sP1\n\tfor <incoming-dt@patchwork.ozlabs.org>;\n\tThu, 14 Sep 2017 22:29:24 +1000 (AEST)", "(majordomo@vger.kernel.org) by vger.kernel.org via listexpand\n\tid S1751807AbdINM3R (ORCPT\n\t<rfc822;incoming-dt@patchwork.ozlabs.org>);\n\tThu, 14 Sep 2017 08:29:17 -0400", "from nblzone-211-213.nblnetworks.fi ([83.145.211.213]:32838 \"EHLO\n\thillosipuli.retiisi.org.uk\" rhost-flags-OK-OK-OK-FAIL)\n\tby vger.kernel.org with ESMTP id S1751740AbdINM3Q (ORCPT\n\t<rfc822; devicetree@vger.kernel.org>); Thu, 14 Sep 2017 08:29:16 -0400", "from lanttu.localdomain (unknown\n\t[IPv6:2001:1bc8:1a6:d3d5::e1:1002])\n\tby hillosipuli.retiisi.org.uk (Postfix) with ESMTP id 69C12600E0;\n\tThu, 14 Sep 2017 15:29:14 +0300 (EEST)" ], "From": "Sakari Ailus <sakari.ailus@linux.intel.com>", "To": "devicetree@vger.kernel.org, linux-acpi@vger.kernel.org", "Cc": "rafael@kernel.org, robh@kernel.org,\n\tmika.westerberg@linux.intel.com, frowand.list@gmail.com", "Subject": "[RFC 0/5] Align and document return values of phandle and reference\n\tparsing for OF and ACPI", "Date": "Thu, 14 Sep 2017 15:29:09 +0300", "Message-Id": "<20170914122914.14122-1-sakari.ailus@linux.intel.com>", "X-Mailer": "git-send-email 2.11.0", "Sender": "devicetree-owner@vger.kernel.org", "Precedence": "bulk", "List-ID": "<devicetree.vger.kernel.org>", "X-Mailing-List": "devicetree@vger.kernel.org" }, "content": "Hi everyone,\n\nI recently came across a difference in behaviour of OF phandle parsing and\nACPI reference parsing, both of which can soon be accessed using\nfwnode_property_get_reference_args.\n\nThe main change in this proposal touches OF, and specifically the change\nis about using -ENODATA to tell that the phandle reference list entry that\nwas accessed does not exist. -ENOENT was used previously, but the same\nerror code was also used to tell that a phandle was empty, making it\nimpossible for the caller to figure out which of the two was the case.\n\nI'm sending the set as RFC. In my limited testing I have found no ill\neffects.\n\nThese patches are on top of linux-next.\n\nComments on the approach and the changes themselves would be most welcome.\n\nFor what it's worth, fwnode_property_get_reference_args is in Rafael's\ntree (as well as in linux-next) and hasn't reached mainline yet. It will\nbe used in the near future for parsing references to external devices in\nthe V4L2 framework.\n\nThese patches can be found here:\n\n<URL:https://git.linuxtv.org/sailus/media_tree.git/log/?h=fwnode-parse>\n\nRegards,\n\n- Sakari\n\nSakari Ailus (5):\n clk: Increment assigned-clocks index for empty phandles\n of: Return -ENODATA on accessing out of bounds indices in phandle\n parsing\n of: Return total number of entries in of_count_phandle_with_args\n ACPI: align acpi_fwnode_get_reference_args with OF counterpart\n device property: Document fwnode_property_get_reference_args better\n\n drivers/acpi/property.c | 3 +++\n drivers/base/power/domain.c | 2 +-\n drivers/base/property.c | 5 +++++\n drivers/clk/clk-conf.c | 8 +++++---\n drivers/clk/clkdev.c | 2 +-\n drivers/gpio/gpiolib-of.c | 2 +-\n drivers/gpio/gpiolib.c | 2 +-\n drivers/gpu/host1x/mipi.c | 2 +-\n drivers/hwspinlock/hwspinlock_core.c | 2 +-\n drivers/iio/inkern.c | 2 +-\n drivers/of/base.c | 37 ++++++++++++++++++++++++------------\n drivers/pwm/core.c | 2 +-\n drivers/regulator/gpio-regulator.c | 2 +-\n drivers/reset/core.c | 3 ++-\n drivers/spi/spi.c | 2 +-\n 15 files changed, 50 insertions(+), 26 deletions(-)" }