On Wed, Sep 20, 2017 at 02:01:25AM +0300, Sakari Ailus wrote: > Hi Rob, > > Thanks for the review. > > On Fri, Sep 15, 2017 at 02:57:35PM -0500, Rob Herring wrote: > > On Thu, Sep 14, 2017 at 7:29 AM, Sakari Ailus > > <sakari.ailus@xxxxxxxxxxxxxxx> wrote: > > > Hi everyone, > > > > > > I recently came across a difference in behaviour of OF phandle parsing and > > > ACPI reference parsing, both of which can soon be accessed using > > > fwnode_property_get_reference_args. > > > > > > The main change in this proposal touches OF, and specifically the change > > > is about using -ENODATA to tell that the phandle reference list entry that > > > was accessed does not exist. -ENOENT was used previously, but the same > > > error code was also used to tell that a phandle was empty, making it > > > impossible for the caller to figure out which of the two was the case. > > > > > > I'm sending the set as RFC. In my limited testing I have found no ill > > > effects. > > > > > > These patches are on top of linux-next. > > > > > > Comments on the approach and the changes themselves would be most welcome. > > > > It's not really valid to both have a variable count (and hence need to > > retrieve it) and use blank phandle entries. The whole point of blank > > entries is to have a fixed length and know what each index corresponds > > too. Do you have an example where we hit this? > > Well, the above is true. I think I'd still use different error codes to > tell about different situations. > > The alternative is that the fwnode counterpart error codes are changed to > to -ENOENT. I'll post a patch implemeting switching the fwnode variant to use -ENOENT, cc'ing you. I think the first patch of the set is still relevant, it's rather a bugfix. -- Sakari Ailus e-mail: sakari.ailus@xxxxxx -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
