Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   <Kavyasree.Kotagiri@microchip.com>
To:     <robh@kernel.org>
CC:     <Claudiu.Beznea@microchip.com>, <devicetree@vger.kernel.org>,
        <linux-arm-kernel@lists.infradead.org>,
        <linux-kernel@vger.kernel.org>, <UNGLinuxDriver@microchip.com>,
        <krzysztof.kozlowski+dt@linaro.org>, <Nicolas.Ferre@microchip.com>,
        <lee@kernel.org>, <robh+dt@kernel.org>, <Sergiu.Moga@microchip.com>
Subject: RE: [RESEND PATCH v9 1/3] dt-bindings: mfd: Convert atmel-flexcom to
 json-schema
Thread-Topic: [RESEND PATCH v9 1/3] dt-bindings: mfd: Convert atmel-flexcom to
 json-schema
Thread-Index: AQHYw5S351ZiP2+bvEqYZuXgNIsbDK3WCUwAgACWniCABNh4sA==
Date:   Mon, 12 Sep 2022 08:16:16 +0000
Message-ID: <CO1PR11MB48654A4239AB7A760193497992449@CO1PR11MB4865.namprd11.prod.outlook.com>
References: <20220908150658.1839520-1-kavyasree.kotagiri@microchip.com>
 <20220908150658.1839520-2-kavyasree.kotagiri@microchip.com>
 <1662671776.565921.3355400.nullmailer@robh.at.kernel.org>
 <SA2PR11MB48742AF8A0C9BC6D32D57A2892439@SA2PR11MB4874.namprd11.prod.outlook.com>
In-Reply-To: <SA2PR11MB48742AF8A0C9BC6D32D57A2892439@SA2PR11MB4874.namprd11.prod.outlook.com>
Accept-Language: en-IN, en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?PWJ4SJMKE5c5R45TGcuYWHFmeqm6uQdS4qLmv+CIGLvgCr7n75Eo4uCNLH9X?=
 =?us-ascii?Q?kbjYhFacsLduBJXaMC0NnwEltgMBnGMukA9iCMHVSuO5xqwMl+5Mu1/4xgnS?=
 =?us-ascii?Q?wksoP1Dqih7S6fJ7gThyhZzp+mQpA+NnPlPnIZYCrtBt42zKYWXD8eSFpIrn?=
 =?us-ascii?Q?2W1BE4gTHxYFi/5a6whRdt0KZAQnODp+zZqB8yo0LBkNFmM4n5l5ms1oFwWA?=
 =?us-ascii?Q?Auka4sB/Oj2kJNmaJOfohgCMwF+U2mfyK4a4+w87s5EkAOcP8xCb/G8QTWi0?=
 =?us-ascii?Q?+kZP1/Ld89gd1/XiQEyWt4hr1HX9BifOXx1kY79z9yIue4okFpITVpUkV0DH?=
 =?us-ascii?Q?1E48/ByaE0RNPtTQeylB0OYCYT0x9FQlBf8pLQsbyntBnxp+cJagXq0DuR0s?=
 =?us-ascii?Q?4JjfG6TP6V3/5Uho+fxVgTvmuf+FgFHeM+Si6BQO6L3EgSL9YJuNCj8IUVEh?=
 =?us-ascii?Q?nsjKTcD18oDeb/3wL11ywnfElXsbRsCOXqZFlQL5aUfrIj1Q+IQm5YFhaCAN?=
 =?us-ascii?Q?StJWBTldO5itYODdrZTV32FgGuD9j9530WROGOBif+YEIfvp2FE151yU7XJi?=
 =?us-ascii?Q?IykdvBiqplqgWOTXR81XkcusCHLFasgEbv2UIyNbo3YXTAoiB33Te+YGTbYY?=
 =?us-ascii?Q?3awOfq01rv6JVqdLs2j3QhB5fVY0OqcL2bGOoL0Y2BJbL3aXbGrKcp7tCagz?=
 =?us-ascii?Q?w/hJh3ZandJaItGfdTu/KEu3Mbv7O7LTHCpK1dRvQx8FNfEmnEMPALpoRDnU?=
 =?us-ascii?Q?mkpOqd++3PQUqDBmJGAvZ09g5uCazCsvNswTHcXvegJHzyXyDgjFWjqJhe9n?=
 =?us-ascii?Q?U3l+/JX6Tm2GlgMjbNQMe9Xpx9VLJUA79hwl2XeHExkFBE/kE3Pbi4Hcze1E?=
 =?us-ascii?Q?2deY64usEuq42nHBuhAm4eO3GLwDHkA0mzUUwXHyU/cHaW+eJfpIlUuyvaOE?=
 =?us-ascii?Q?PaFuGtwIa2lXEG93sRCfIt2GGra469HBgDeSnjxJXvshcWE3vYm/81rYgZcb?=
 =?us-ascii?Q?IYNDR16uiCHBebcjCsb7ep7X5sTag0mCns4ebYRBqyvIXqJl10Z8PfNJcFZo?=
 =?us-ascii?Q?rLzm8ROgIF7f3fkQqjw6Ro6QbU4d9WAB0cDFe/SMrBKq4o5JC0Y3sm4BGu3K?=
 =?us-ascii?Q?viqROEo5V4kvy92yy/sEokJcuoU2kpXyfjO5HOIxNjZKSrn/5GIzhl+4I0Sa?=
 =?us-ascii?Q?hOnRZKxs8ucvdzGqJfNs9wwNkCYv+BCF44WzFXRY8D1AGc8c2j1GqAUb98pS?=
 =?us-ascii?Q?gU53Iit3ZrkgqKRjHaUmxh8FGyQnXa30otrwzBlrI04NdbUCd0/M0ABhRfLX?=
 =?us-ascii?Q?xqBwN0BnzLm+F4QIZLCqMJ/CusYzapOMMX2kjKacbevZc/JblNm8GxbtS8yg?=
 =?us-ascii?Q?xVJCu/0VqhhdpmBo51AUYXD/YwwAo73qCR4ZBJt/Tmi4A92rxn8O2W2iwTET?=
 =?us-ascii?Q?AZkeprkoOaxQ4Nqzn9PCw3Ic/4LZgicK4gowrkM2U4Xp44C6ziIfW61PjW92?=
 =?us-ascii?Q?C+hRiIzXofhDsClSoBFSfAgT8VyWR3iU74VtuMksyoC4+c3UCiPkHqHNpYwU?=
 =?us-ascii?Q?qc6QUUVzcr/d6wp0n2hGx858B1XTX8ERQbLFNJ9OMSUZfqNCetqWV5iWlJK4?=
 =?us-ascii?Q?Pg=3D=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: CO1PR11MB4865.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: c5804287-154a-47a2-048d-08da94971080
X-MS-Exchange-CrossTenant-originalarrivaltime: 12 Sep 2022 08:16:16.3988
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 3f4057f3-b418-4d4e-ba84-d55b4e897d88
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: qy5+t0y+s7lVCKOF1PZEYcBEK0KvEdkdUbqU0eKPI4YUcABw9isN56SXST3zxXuFu1p9hsfSVwAjsREGlG4SRQQ5aDKRYaBQXR5sKvGIZmC13r3Jyo2hga7j0vy5iy9F
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY5PR11MB6306
X-Spam-Status: No, score=-4.4 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED,SPF_HELO_PASS,
        SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no
        version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

> > > Convert the Atmel flexcom device tree bindings to json schema.
> > >
> > > Signed-off-by: Kavyasree Kotagiri <kavyasree.kotagiri@microchip.com>
> > > ---
> > > v8 -> v9:
> > >  - Changed reference provided for spi bindings to atmel,at91rm9200-
> > spi.yaml
> > >
> > > v7 -> v8:
> > >  - Added back patternProperties for child nodes.
> > >
> > > v6 -> v7:
> > >  - Change filename to atmel,sama5d2-flexcom.yaml
> > >  - Add #address-cells, #size-cells to flexcom node - Fixed warnings.
> > >
> > > v5 -> v6:
> > >  - Removed spi node from example as suggested by Rob and
> > >    also pattern properties(spi dt-bindings conversion to yaml patch i=
s
> under
> > review).
> > >    Once that is accepted, I will add back spi example through new pat=
ch.
> > >
> > > v4 -> v5:
> > >  - Fixed indentations.
> > >
> > > v3 -> v4:
> > >  - Corrected format of enum used for compatible string.
> > >
> > > v2 -> v3:
> > >  - used enum for compatible string.
> > >  - changed irq flag to IRQ_TYPE_LEVEL_HIGH in example.
> > >  - fixed dtschema errors.
> > >
> > > v1 -> v2:
> > >  - Fix title.
> > >
> > >  .../bindings/mfd/atmel,sama5d2-flexcom.yaml   | 91
> > +++++++++++++++++++
> > >  .../devicetree/bindings/mfd/atmel-flexcom.txt | 63 -------------
> > >  2 files changed, 91 insertions(+), 63 deletions(-)
> > >  create mode 100644
> > Documentation/devicetree/bindings/mfd/atmel,sama5d2-flexcom.yaml
> > >  delete mode 100644 Documentation/devicetree/bindings/mfd/atmel-
> > flexcom.txt
> > >
> >
> > Running 'make dtbs_check' with the schema in this patch gives the
> > following warnings. Consider if they are expected or the schema is
> > incorrect. These may not be new warnings.
> >
> > Note that it is not yet a requirement to have 0 warnings for dtbs_check=
.
> > This will change in the future.
> >
> > Full log is available here: https://patchwork.ozlabs.org/patch/
> >
> >
> > flexcom@e0040000: spi@400: Unevaluated properties are not allowed
> > ('dma-names', 'dmas' were unexpected)
> >         arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt-6g-2gs.dtb
> >         arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt-8g.dtb
> >         arch/arm/boot/dts/lan966x-pcb8291.dtb
> >         arch/arm/boot/dts/lan966x-pcb8309.dtb
> >
> > flexcom@e0044000: spi@400: Unevaluated properties are not allowed
> > ('dma-names', 'dmas' were unexpected)
> >         arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt-6g-2gs.dtb
> >         arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt-8g.dtb
> >         arch/arm/boot/dts/lan966x-pcb8291.dtb
> >         arch/arm/boot/dts/lan966x-pcb8309.dtb
> >
> > flexcom@e0060000: spi@400: Unevaluated properties are not allowed
> > ('dma-names', 'dmas' were unexpected)
> >         arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt-6g-2gs.dtb
> >         arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt-8g.dtb
> >         arch/arm/boot/dts/lan966x-pcb8291.dtb
> >         arch/arm/boot/dts/lan966x-pcb8309.dtb
> >
> > flexcom@e0064000: spi@400: Unevaluated properties are not allowed
> > ('dma-names', 'dmas' were unexpected)
> >         arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt-6g-2gs.dtb
> >         arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt-8g.dtb
> >         arch/arm/boot/dts/lan966x-pcb8291.dtb
> >         arch/arm/boot/dts/lan966x-pcb8309.dtb
> >
> > flexcom@e0070000: spi@400: Unevaluated properties are not allowed
> > ('dma-names', 'dmas' were unexpected)
> >         arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt-6g-2gs.dtb
> >         arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt-8g.dtb
> >         arch/arm/boot/dts/lan966x-pcb8291.dtb
> >         arch/arm/boot/dts/lan966x-pcb8309.dtb
> >
> > flexcom@e2824000: spi@400: Unevaluated properties are not allowed
> > ('dma-names', 'dmas' were unexpected)
> >         arch/arm/boot/dts/at91-sama7g5ek.dtb
> >
> > flexcom@f8034000: spi@400: Unevaluated properties are not allowed
> > ('dma-names', 'dmas' were unexpected)
> >         arch/arm/boot/dts/at91-kizbox3-hs.dtb
> >         arch/arm/boot/dts/at91-sama5d27_som1_ek.dtb
> >         arch/arm/boot/dts/at91-sama5d27_wlsom1_ek.dtb
> >         arch/arm/boot/dts/at91-sama5d2_icp.dtb
> >         arch/arm/boot/dts/at91-sama5d2_ptc_ek.dtb
> >         arch/arm/boot/dts/at91-sama5d2_xplained.dtb
> >
> > flexcom@f8038000: spi@400: Unevaluated properties are not allowed
> > ('dma-names', 'dmas' were unexpected)
> >         arch/arm/boot/dts/at91-kizbox3-hs.dtb
> >         arch/arm/boot/dts/at91-sama5d27_som1_ek.dtb
> >         arch/arm/boot/dts/at91-sama5d27_wlsom1_ek.dtb
> >         arch/arm/boot/dts/at91-sama5d2_icp.dtb
> >         arch/arm/boot/dts/at91-sama5d2_ptc_ek.dtb
> >         arch/arm/boot/dts/at91-sama5d2_xplained.dtb
> >
> > flexcom@fc010000: spi@400: Unevaluated properties are not allowed
> ('dma-
> > names', 'dmas' were unexpected)
> >         arch/arm/boot/dts/at91-kizbox3-hs.dtb
> >         arch/arm/boot/dts/at91-sama5d27_som1_ek.dtb
> >         arch/arm/boot/dts/at91-sama5d27_wlsom1_ek.dtb
> >         arch/arm/boot/dts/at91-sama5d2_icp.dtb
> >         arch/arm/boot/dts/at91-sama5d2_ptc_ek.dtb
> >         arch/arm/boot/dts/at91-sama5d2_xplained.dtb
> >
> > flexcom@fc014000: spi@400: Unevaluated properties are not allowed
> ('dma-
> > names', 'dmas' were unexpected)
> >         arch/arm/boot/dts/at91-kizbox3-hs.dtb
> >         arch/arm/boot/dts/at91-sama5d27_som1_ek.dtb
> >         arch/arm/boot/dts/at91-sama5d27_wlsom1_ek.dtb
> >         arch/arm/boot/dts/at91-sama5d2_icp.dtb
> >         arch/arm/boot/dts/at91-sama5d2_ptc_ek.dtb
> >         arch/arm/boot/dts/at91-sama5d2_xplained.dtb
> >
> > flexcom@fc018000: spi@400: Unevaluated properties are not allowed
> ('dma-
> > names', 'dmas' were unexpected)
> >         arch/arm/boot/dts/at91-kizbox3-hs.dtb
> >         arch/arm/boot/dts/at91-sama5d27_som1_ek.dtb
> >         arch/arm/boot/dts/at91-sama5d27_wlsom1_ek.dtb
> >         arch/arm/boot/dts/at91-sama5d2_icp.dtb
> >         arch/arm/boot/dts/at91-sama5d2_ptc_ek.dtb
> >         arch/arm/boot/dts/at91-sama5d2_xplained.dtb
>=20
> dma properties for spi node is already being covered by Sergiu in below
> patch:
> https://lore.kernel.org/lkml/b4625526-8f9b-ada8-f854-
> 751a48545576@linaro.org/
>=20
Hi Rob, Please let me know your comments.

Thanks,
Kavya

>=20
> Thanks,
> Kavya