From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alban Subject: Re: [PATCH 1/3] doc: bindings: Add bindings documentation for mtd nvmem Date: Fri, 3 Mar 2017 13:17:05 +0100 Message-ID: <20170303131705.119021f4@tock> References: <1488484223-844-1-git-send-email-albeu@free.fr> <1488484223-844-2-git-send-email-albeu@free.fr> <20170302212220.4c3c7bef@bbrezillon> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0794765582571810234==" Return-path: In-Reply-To: <20170302212220.4c3c7bef@bbrezillon> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-mtd" Errors-To: linux-mtd-bounces+gldm-linux-mtd-36=gmane.org@lists.infradead.org To: Boris Brezillon Cc: Mark Rutland , devicetree@vger.kernel.org, Moritz Fischer , Richard Weinberger , linux-kernel@vger.kernel.org, Marek Vasut , Srinivas Kandagatla , Rob Herring , Aban Bedel , linux-mtd@lists.infradead.org, Cyrille Pitchen , Brian Norris , David Woodhouse , Maxime Ripard List-Id: devicetree@vger.kernel.org --===============0794765582571810234== Content-Type: multipart/signed; micalg=pgp-sha256; boundary="Sig_//WcVBIwnCqlfkEDj3C3PmZa"; protocol="application/pgp-signature" --Sig_//WcVBIwnCqlfkEDj3C3PmZa Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable On Thu, 2 Mar 2017 21:22:20 +0100 Boris Brezillon wrote: > On Thu, 2 Mar 2017 20:50:21 +0100 > Alban wrote: >=20 > > Add the binding to expose MTD partitions as nvmem providers. =20 >=20 > 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 expose nvmem cells directly > under master MTD devices (and not only partitions).=20 I think that would be the better solution. This can be done independently, once we agree on a binding we just have to fix of_nvmem_cell_get(). My suggestion would be to have the new binding looking like this: nvmem-device@10 { ... nvmem-provider; nvmem-cells { compatible =3D "nvmem-cells"; #address-cells =3D <1>; #size-cells =3D <1>; nvmem-cell@100 { label =3D "mac-address"; reg =3D <0x100 0x200>; } ... } } I would also suggest making the "nvmem-provider" property mandatory to indicate that the device provides this capability. Up to now all nvmem providers only support this API but I think there might be more multi function devices in the future. Alban --Sig_//WcVBIwnCqlfkEDj3C3PmZa Content-Type: application/pgp-signature Content-Description: OpenPGP digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAEBCAAGBQJYuV7BAAoJEHSUmkuduC28F5cP/R8xJyu2TL6rMp7ZuE1TkTYr /lw1pQhEVnQcNiIFGzJsWv6gVOvDBXsy985hC/sp847twtUSBzy4DSagNHSR8Cui NByS2M8IrfzF/yb11pGLoJIhzD825u8f51N/TpLpOE8jK8+T+sO+ZvjlyUi1wf18 QZUPdXrVSCAeJyfvLh4uYUaoUkD72OFGLCw8my729gQ7sLLKJeSBaD/+SU1WoTPA 0umOEdp6BHJPLcrlrejzk4JHsQMW0t2TFooJF1xeBr7yDjNoBGwlrmLfqtVYjNUj n7OsYUn8GtM9SOqTZS32InkseOEuVYp85FHSjqMy5nLgUe0pG6aREYN0VTWKxy9l KymY9kYOnXCBU/C+wr54eo/BcfHJ45rcnHqqI79JMcAqrgFSZSpnIWVS4ZAqIbkU FEaL3aJH1FUH3zIsZIRaFFxswVHvfnGCqR5dNhpk0F9LOWRk5BHkytPVIgiAUo+C zOgSoXaBBbSWEWyH+9cda1tSVXfdrb8UvjBidKAJo1CjYonupmH5ojHMU/hfzeYT UTkgIXRNjN7d215FSQ8wzeuXlzs0NTo6eN2Qxnz3nPdidY87PfqNAN58XAfQu/mH AmjXnQTDIndKpJ4XvHsUTohGwMKJHsAQB/3D/Aann20sGrn9yEd0e+n7DnclNShq 7j9qCzFLObp2bYQBaspn =aa3s -----END PGP SIGNATURE----- --Sig_//WcVBIwnCqlfkEDj3C3PmZa-- --===============0794765582571810234== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/ --===============0794765582571810234==--