Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Joakim Zhang <qiangqing.zhang@nxp.com>
To:     Rob Herring <robh@kernel.org>,
        Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
CC:     "shawnguo@kernel.org" <shawnguo@kernel.org>,
        "kernel@pengutronix.de" <kernel@pengutronix.de>,
        dl-linux-imx <linux-imx@nxp.com>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH V1 1/4] bindings: nvmem: introduce "reverse-data" property
Thread-Topic: [PATCH V1 1/4] bindings: nvmem: introduce "reverse-data"
 property
Thread-Index: AQHXjbpIBwDJH2bCBk6a1JjgGNijp6tuGImAgAoQXgCAAMcfEIAJV06A
Date:   Tue, 24 Aug 2021 06:31:31 +0000
Message-ID: <DB8PR04MB67951BE5D484C69BD57ABDA4E6C59@DB8PR04MB6795.eurprd04.prod.outlook.com>
References: <20210810073510.18218-1-qiangqing.zhang@nxp.com>
 <20210810073510.18218-2-qiangqing.zhang@nxp.com>
 <6e3f6881-929d-1663-58f1-39bf35069175@linaro.org>
 <YRwUyLsvoSpFI9X8@robh.at.kernel.org>
 <DB8PR04MB67951E2312CFD69808B4502BE6FF9@DB8PR04MB6795.eurprd04.prod.outlook.com>
In-Reply-To: <DB8PR04MB67951E2312CFD69808B4502BE6FF9@DB8PR04MB6795.eurprd04.prod.outlook.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?dKrYgGmZ/ItT+lPdHueARGZUf/cu9p14NePr9fUTnVmOMbIGwZq1XqIhDGFI?=
 =?us-ascii?Q?4TYUI1JzP+XiqviwBqh9gx5fdHZ9pwWCom5xC4VwPzlMIFUCEC+1rjFMD2iL?=
 =?us-ascii?Q?wt4C0kgpL4pGS4IGpRsczcVs8hnaZ/ahe6IOK/cNN8kwdKH9ilNMkZlOxEtv?=
 =?us-ascii?Q?RHCCX21oEKd4drqxwX/3CJXwtq5BsBF7xXB4h4zkgKXzj1SK4a39/G8vVVz2?=
 =?us-ascii?Q?KAbbA6CwyaUwtE95A0N+UrWzyJVs3TcMSOqh8SJIWxLQ/lLLA3ED75dEob7z?=
 =?us-ascii?Q?k+RGtMPAKvPTvlfFBye6KX36ylZBUMJSm0Y34SgZbqu9SZEt5vzJTVN2plXm?=
 =?us-ascii?Q?FwZTBUMv5G9DBSvVNNA9sziah8pqDrL2IOKVH7L+cQF8hy0kFwGCJ8fDPi/W?=
 =?us-ascii?Q?iz09sjS6drl501jjEeUByz5HrPtOfLa6al3kJPXt6Vp9HqtGXd05cgnKus/t?=
 =?us-ascii?Q?+YTzRDYQae/pGx/lhs0SoK358YBrbzGnIMqhok9U3khaf5jPtSz9CxM31GX4?=
 =?us-ascii?Q?XIno455GN+fsk9Tq3ntf8qtTyX4jVlTO1R5x2Z40ogOP4RExX1x3G6pfcdgg?=
 =?us-ascii?Q?edFdSUKCcn7rlprRG24zoDkYZUktaOulSGO/euFlma5nIH9OhNQaY8kFDG2z?=
 =?us-ascii?Q?GZzp7xism6gZ86HvOGMUyP+g6pShdTwnwMwuQKBSSfztmRzvha1YsadLHB5J?=
 =?us-ascii?Q?Al1zhbDPeusu+OGxzsn37Gcs4NnF5qLwnaABYIIFNgidKU4QV4ePTwtlxSOB?=
 =?us-ascii?Q?GbbW01fzQf3W2cxVFVG39S5Xin9K2M2auznT7Fsy9orUYEglXNOFt5sXBan1?=
 =?us-ascii?Q?8RCdU9lbHYR5NPi843Z12uA1hIk+e2HDMCsiM5OHNiQQlWRuj/PNXzYOhe1b?=
 =?us-ascii?Q?/OMit4bB6wK1TUd7B/bwI0G8pU16Eg1qmdrO22Hw4h79P3g0ST2QXpbEQKDd?=
 =?us-ascii?Q?RyrT0peCHd/fzJRGdssVZnlatRMZmm0Xc68DwyifZh5k+Tu7EN2DQTpomu8o?=
 =?us-ascii?Q?Kjg8rtmkVRAKC7zGx9BOTyW2DKNOSjmmS794y8t8P+AqPCWlzBG6kyWr/bDb?=
 =?us-ascii?Q?TghGcAuQ4N258ELZzQT/OAmLB5aPnTVdWzj7bS1KPxaDDRYJaNa3fB3lVI0V?=
 =?us-ascii?Q?EHK02kzF7Dg11Q6RdU3hzsXm8DLQXuqMVMCeHAUNtdEjwNOaocdkkjG4O2+Q?=
 =?us-ascii?Q?oT4uFbTGXXdW4iJ/Y9tMPiGHXH+0/DgB3s3zQxRjfK/VJA8mq4WTBpujybIV?=
 =?us-ascii?Q?7PBSoP7DOvG7gKhTbV9Mjqf8URNpvoESr1FoZ85yrDp2cvPpQAnrDIt4F9c2?=
 =?us-ascii?Q?S7ZHGQM+8kerC2dtKLSOKz++?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DB8PR04MB6795.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: bdffa3f1-7014-42a1-e4db-08d966c8d016
X-MS-Exchange-CrossTenant-originalarrivaltime: 24 Aug 2021 06:31:31.8606
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: bGFq4YYTLWWJgMdD2YzTpa7MsPsvmg486gWTwBW3jBAtoGGsTif7LsjZ3/OOpvqsH0GIKppC0GkMuaFYe9jhBA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB6PR0401MB2502
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


Hi Srinivas,

> > > > --- a/Documentation/devicetree/bindings/nvmem/nvmem.yaml
> > > > +++ b/Documentation/devicetree/bindings/nvmem/nvmem.yaml
> > > > @@ -61,6 +61,11 @@ patternProperties:
> > > >                 description:
> > > >                   Size in bit within the address range specified by
> reg.
> > > > +      reverse-data:
> > > > +        $ref: /schemas/types.yaml#/definitions/flag
> > > > +        description:
> > > > +          Reverse the data that read from the storage device.
> > > > +
> > >
> > > This new property is only going to solve one of the reverse order
> > > issue here.
> > > If I remember correctly we have mac-address stored in various formats=
 ex:
> > > from old thread I can see
> > >
> > > Type 1: Octets in ASCII without delimiters. (Swapped/non-Swapped)
> > > Type
> > > 2: Octets in ASCII with delimiters like (":", ",", ".", "-"... so
> > > on)
> > > (Swapped/non-Swapped)
> > > Type 3: Is the one which stores mac address in Type1/2 but this has
> > > to be incremented to be used on other instances of eth.
> > > Type 4: Octets as bytes/u8, swapped/non-swapped
> > >
> > > I think its right time to consider adding compatibles to nvmem-cells
> > > to be able to specify encoding information and handle post processing=
.
> >
> > Yes. Trying to handle this with never ending new properties will end
> > up with a mess. At some point, you just need code to parse the data.
>=20
> Thanks, Rob.
>=20
> Hi Srinivas,
>=20
> Do you plan to implement it?
>=20
> Or need me follow up? If yes, please input your insights to point me how =
to
> work for it.

Any comments?

Best Regards,
Joakim Zhang