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:     <Nicolas.Ferre@microchip.com>, <robh+dt@kernel.org>,
        <devicetree@vger.kernel.org>,
        <linux-arm-kernel@lists.infradead.org>,
        <linux-kernel@vger.kernel.org>,
        <krzysztof.kozlowski+dt@linaro.org>,
        <Claudiu.Beznea@microchip.com>, <krzysztof.kozlowski@linaro.org>,
        <UNGLinuxDriver@microchip.com>
Subject: RE: [PATCH v4 1/3] dt-bindings: mfd: atmel,flexcom: Convert to
 json-schema
Thread-Topic: [PATCH v4 1/3] dt-bindings: mfd: atmel,flexcom: Convert to
 json-schema
Thread-Index: AQHYgktCiqqptpU+dEuWUcG+O1Oj/61ToBIAgASPMIA=
Date:   Mon, 20 Jun 2022 11:45:31 +0000
Message-ID: <CO1PR11MB48653378BAECD414F497D59592B09@CO1PR11MB4865.namprd11.prod.outlook.com>
References: <20220617130729.12072-1-kavyasree.kotagiri@microchip.com>
 <20220617130729.12072-2-kavyasree.kotagiri@microchip.com>
 <1655474312.882258.1508989.nullmailer@robh.at.kernel.org>
In-Reply-To: <1655474312.882258.1508989.nullmailer@robh.at.kernel.org>
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?4CufCPrccKU0XZKIUKn+Zpw6jhpysBd4EbULz58hTJCpwaWfZE3p8vxRsCTC?=
 =?us-ascii?Q?/DEK5QCOaAVKr0MG60V4/FTaPgCXbDr5uadQnS3gqFu4wXD1qwWtdq+xCXZc?=
 =?us-ascii?Q?q+F//B/VRkqAA8+Vku1OuMDKa/NRKchWvFF4x5Q0MpfxvCW0ZC5LlcpW/XHE?=
 =?us-ascii?Q?nSdMXk3K/8A0tMMBEkzR7vfkuc5+qFbDNr45MJYvRUCX+TpT3UlzlnYimNem?=
 =?us-ascii?Q?SXrdO8T305xNtaO1EiQanBPeGBe+yRedZl0yhwWRfZVnFXUh62L9z0E0K6+a?=
 =?us-ascii?Q?fc0r3cXnxnAteILQ0rPa621Fon90VVlBwJK42992HPlCMvocB43N4JAt3o/G?=
 =?us-ascii?Q?TuY2iLeRrYbxFRkj1oSumu97ygKFbPHkCFyRiOPGB+TcCTjD9EVVaS+xlhMF?=
 =?us-ascii?Q?u9dyNkZxVcgmUNJT4pVJ+SG48ptvbBpS4AK6G6Ph57ZmllHAH1FBSsp9v6i7?=
 =?us-ascii?Q?E9p8+I/sXjZ75t0tF0ZChf1Dr9c9l9rsE3Sa+gxvU8kNF4GHrOGp4N+Jxlrh?=
 =?us-ascii?Q?H67zdkIop15yAFPmuq86lQ2qSLuY+37zFipT6B6bd6iY4weeBUhfyw/+ohEz?=
 =?us-ascii?Q?rHSkj6OIXlc2o6bV5zMqDfSrjpaGoPW5IZWL+/QDWPVC9Va3S64oORxjZ042?=
 =?us-ascii?Q?wCoVLIT7/rPMurjq3PbaDSaUmX4Q/nz6ruDykimPYS3xDvgAb+uXC2Er7mQv?=
 =?us-ascii?Q?lNthNBSrgMRLahZepYId4ye5QE9eG6vVD0IFrtUUUnqIyxfoYRysvWwLvHZM?=
 =?us-ascii?Q?dAeU7kQgUpt5g6RIEAf7xItBWCN2F50Yz+bE51PJEbcM1nKSDYOf54tuSW/3?=
 =?us-ascii?Q?AqvYOc8tr+SVAx+u9dQwIr+dEpX4ctraOSWSPX/QHgBc+kVPkwbV/fHfAlKU?=
 =?us-ascii?Q?TemBh5pps7VudzRntWdfmD9c9+wv63nm3A8LaXzWR1qovKVhwAsXX8g3b61x?=
 =?us-ascii?Q?1hRW65AJtaZFRiTtIA7H+049BoKBmdJ842VdkT62QK1F31prZC9ON9M6ca5m?=
 =?us-ascii?Q?SMADg/1DjoWyqFxoa2IMHAAvLJE1P+maoD+/heMm4l443bTx0HxFniT7c9r+?=
 =?us-ascii?Q?K6uE/PpSJpm5b74ED8nX2EHVHCGKVMSHcNtF9hDY8RrttSLeaf06ZXQ3PSke?=
 =?us-ascii?Q?oRRrUKoS/pWBZYkxfzt0Xy8RQutyQN+2NVrjlqTMqKNeBXpFQixbPNrCwHmZ?=
 =?us-ascii?Q?LEA/xSstwZLsBcC5C6x+fyG+/F87Ufm01MOCx32R24c9vBn3pHTOoTE5N3n6?=
 =?us-ascii?Q?QH1LdHNrCtWUTTCnSiUX+1tBUGDOHJo2FDZaATYikc9P4jv47/gJC2L/XwzY?=
 =?us-ascii?Q?WscUPIb4rhPQfRmPyjbU0z4dS46/n8RelbMG+NI3zHEyVbsF3JixNrgPvHH7?=
 =?us-ascii?Q?yHNJg2TxrtwoQIfkD3dr6vlihRXOqtvqtBaIbNCN4KA5PCUzX0nes3C/QbnA?=
 =?us-ascii?Q?Q67Fs9KUJKazQ78fdMHHokYdrYyT9R0Jy63tSB8S/0fl0gKZ/uIdbpTzRuMk?=
 =?us-ascii?Q?oJJt+JQvsLbsxUeiFdWyIfkcNQS1evKxB7RO/x1SjQq7zMNM5kp8D+R18ArZ?=
 =?us-ascii?Q?1CcKTumwhEKgHpxqycdUuxSAm2mUelmIunfzQfs8wO11V3bSxTJyqgSmG+Tk?=
 =?us-ascii?Q?7yrlVeh+bCMuKyxdkHzkQ+zxhRwap9Bp/x+aejpiYD3YivTAG5HWOS/QLpSS?=
 =?us-ascii?Q?UAlptwB7wKKhkVgvTxugPsyvJBszTFUQFNBxagrxcianXxLjfX++ykV1Rx76?=
 =?us-ascii?Q?1krBWdZslnH/AYXIi4SkJgU8QA5e9lw=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: 924b6fb7-6435-41bb-de57-08da52b26103
X-MS-Exchange-CrossTenant-originalarrivaltime: 20 Jun 2022 11:45:31.1020
 (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: bnUwJ144xO3eBNQob/J8x06U0q46ZNzxjzxn7dxrrAPLReWnUAiY49npJm1cYcdGN2UzeTpT81IKqjcT7T4mIoUy5HtGIK4Y21dOkUX2xdSaVsCRnq28vx1DOaFKBXAR
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL0PR11MB3089
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>
> > ---
> > 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,flexcom.yaml           | 104 ++++++++++++++++++
> >  .../devicetree/bindings/mfd/atmel-flexcom.txt |  63 -----------
> >  2 files changed, 104 insertions(+), 63 deletions(-)
> >  create mode 100644
> Documentation/devicetree/bindings/mfd/atmel,flexcom.yaml
> >  delete mode 100644 Documentation/devicetree/bindings/mfd/atmel-
> flexcom.txt
> >
>=20
> 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.
>=20
> Note that it is not yet a requirement to have 0 warnings for dtbs_check.
> This will change in the future.
>=20
> Full log is available here: https://patchwork.ozlabs.org/patch/
>=20
>=20
> flexcom@f0000000: spi@400:compatible:0: 'atmel,at91rm9200-spi' was
> expected
>         arch/arm/boot/dts/at91-sam9x60ek.dtb
>=20
> flexcom@f0000000: spi@400:compatible: ['microchip,sam9x60-spi',
> 'atmel,at91rm9200-spi'] is too long
>         arch/arm/boot/dts/at91-sam9x60ek.dtb

These errors are expected because "microchip,sam9x60-spi" and "atmel,at91rm=
9200-spi"
are documented in Documentation/devicetree/bindings/spi/spi_atmel.txt.
which I added in description as
" description: See ../spi/spi_atmel.txt for details of SPI bindings."