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

From: Rob Herring
Date: Fri Mar 31 2017 - 15:06:35 EST


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@xxxxxxxxxx>
>
> 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