{"id":813809,"url":"http://patchwork.ozlabs.org/api/covers/813809/?format=json","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=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":"","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=json","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=json","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":"","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 3xtHsJ0Gy8z9sP1\n\tfor ;\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);\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); 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 ","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":"","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\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(-)"}