Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1614261

Re: [PATCH v5 14/24] dt-bindings: mfd: syscon: Add documentation for #syscon-cells property

From Rob Herring <robh@kernel.org>
Newsgroups linux.kernel
Subject Re: [PATCH v5 14/24] dt-bindings: mfd: syscon: Add documentation for #syscon-cells property
Date 2017-03-31 21:10 +0200
Message-ID <tra5Q-eL-25@gated-at.bofh.it> (permalink)
References <tpzrH-62q-5@gated-at.bofh.it> <tpzrI-62q-43@gated-at.bofh.it> <tr9W9-8nY-1@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Fri, Mar 31, 2017 at 01:58:52PM -0500, Rob Herring wrote:
> On Mon, Mar 27, 2017 at 03:15:10PM +0530, Kishon Vijay Abraham I wrote:
> > Add documentation for the optional #syscon-cells property to determine
> > the number of cells that should be given in the phandle while
> > referencing the syscon-node.
> > 
> > Suggested-by: Rob Herring <robh@kernel.org>
> 
> I did? When?
> 
> I'm not remembering why we need this. Generally, phandles to a syscon 
> are for a single defined purpose. When do we need a list of them?

Ah, I remember now the context. I suggested using standard parsing 
function rather than open coding. I wasn't suggesting changing the 
binding. Instead of of_parse_phandle_with_args, can't 
of_parse_phandle_with_fixed_args work?

Rob

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

Re: [PATCH v5 14/24] dt-bindings: mfd: syscon: Add documentation for  #syscon-cells property Rob Herring <robh@kernel.org> - 2017-03-31 21:00 +0200
  Re: [PATCH v5 14/24] dt-bindings: mfd: syscon: Add documentation for  #syscon-cells property Rob Herring <robh@kernel.org> - 2017-03-31 21:10 +0200
    Re: [PATCH v5 14/24] dt-bindings: mfd: syscon: Add documentation for  #syscon-cells property Kishon Vijay Abraham I <kishon@ti.com> - 2017-04-05 10:40 +0200

csiph-web