Re: [PATCH 1/3] doc: bindings: Add bindings documentation for mtd nvmem

2017-03-03 Thread Alban
On Thu, 2 Mar 2017 21:22:20 +0100 Boris Brezillon wrote: > On Thu, 2 Mar 2017 20:50:21 +0100 > Alban wrote: > > > Add the binding to expose MTD partitions as nvmem providers. > > Looks good. Maybe you should take the case you describe in your > cover-letter into account and add an extra lay

Re: [PATCH 1/3] doc: bindings: Add bindings documentation for mtd nvmem

2017-03-03 Thread Alban
On Fri, 3 Mar 2017 13:37:44 +0100 Boris Brezillon wrote: > On Fri, 3 Mar 2017 13:17:05 +0100 > Alban wrote: > > > On Thu, 2 Mar 2017 21:22:20 +0100 > > Boris Brezillon wrote: > > > > > On Thu, 2 Mar 2017 20:50:21 +0100 > > > Alban wrote: > > > > > > > Add the binding to expose MTD pa

Re: [PATCH 1/3] doc: bindings: Add bindings documentation for mtd nvmem

2017-03-03 Thread Boris Brezillon
On Fri, 3 Mar 2017 13:17:05 +0100 Alban wrote: > On Thu, 2 Mar 2017 21:22:20 +0100 > Boris Brezillon wrote: > > > On Thu, 2 Mar 2017 20:50:21 +0100 > > Alban wrote: > > > > > Add the binding to expose MTD partitions as nvmem providers. > > > > Looks good. Maybe you should take the cas

Re: [PATCH 1/3] doc: bindings: Add bindings documentation for mtd nvmem

2017-03-03 Thread Boris Brezillon
On Fri, 3 Mar 2017 11:27:34 + Srinivas Kandagatla wrote: > On 02/03/17 19:50, Alban wrote: > > Add the binding to expose MTD partitions as nvmem providers. > > It would be nice to see more description of this patch, explaining the > real use case. > > > > Signed-off-by: Alban > > --- > >

Re: [PATCH 1/3] doc: bindings: Add bindings documentation for mtd nvmem

2017-03-03 Thread Alban
On Fri, 3 Mar 2017 11:27:34 + Srinivas Kandagatla wrote: > On 02/03/17 19:50, Alban wrote: > > Add the binding to expose MTD partitions as nvmem providers. > > It would be nice to see more description of this patch, explaining the > real use case. I'll try, writing good documentation is

Re: [PATCH 1/3] doc: bindings: Add bindings documentation for mtd nvmem

2017-03-03 Thread Srinivas Kandagatla
On 02/03/17 19:50, Alban wrote: Add the binding to expose MTD partitions as nvmem providers. It would be nice to see more description of this patch, explaining the real use case. Signed-off-by: Alban --- .../devicetree/bindings/nvmem/mtd-nvmem.txt| 29 ++ 1 fi

Re: [PATCH 1/3] doc: bindings: Add bindings documentation for mtd nvmem

2017-03-02 Thread Boris Brezillon
On Thu, 2 Mar 2017 20:50:21 +0100 Alban wrote: > Add the binding to expose MTD partitions as nvmem providers. Looks good. Maybe you should take the case you describe in your cover-letter into account and add an extra layer: add an nvmem sub-node containing the nvmem cells, so that you can expos

[PATCH 1/3] doc: bindings: Add bindings documentation for mtd nvmem

2017-03-02 Thread Alban
Add the binding to expose MTD partitions as nvmem providers. Signed-off-by: Alban --- .../devicetree/bindings/nvmem/mtd-nvmem.txt| 29 ++ 1 file changed, 29 insertions(+) create mode 100644 Documentation/devicetree/bindings/nvmem/mtd-nvmem.txt diff --git a/Documenta