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:   Shenwei Wang <shenwei.wang@nxp.com>
To:     Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
        "robh+dt@kernel.org" <robh+dt@kernel.org>,
        "krzysztof.kozlowski+dt@linaro.org" 
        <krzysztof.kozlowski+dt@linaro.org>,
        "linus.walleij@linaro.org" <linus.walleij@linaro.org>,
        "brgl@bgdev.pl" <brgl@bgdev.pl>,
        "shawnguo@kernel.org" <shawnguo@kernel.org>,
        "s.hauer@pengutronix.de" <s.hauer@pengutronix.de>,
        "kernel@pengutronix.de" <kernel@pengutronix.de>,
        "festevam@gmail.com" <festevam@gmail.com>,
        dl-linux-imx <linux-imx@nxp.com>
CC:     "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "linux-gpio@vger.kernel.org" <linux-gpio@vger.kernel.org>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        "imx@lists.linux.dev" <imx@lists.linux.dev>
Subject: RE: [EXT] Re: [PATCH v3 1/3] dt-bindings: gpio: Add imx scu gpio
 driver bindings
Thread-Topic: [EXT] Re: [PATCH v3 1/3] dt-bindings: gpio: Add imx scu gpio
 driver bindings
Thread-Index: AQHYrL17Pas5Q+nPyEWWoebmGoM+lq2oNm6AgAAHPuA=
Date:   Wed, 10 Aug 2022 15:23:47 +0000
Message-ID: <AM9PR04MB827495CB596427BD86A1522B89659@AM9PR04MB8274.eurprd04.prod.outlook.com>
References: <20220810133005.74653-1-shenwei.wang@nxp.com>
 <20220810133005.74653-2-shenwei.wang@nxp.com>
 <ec6fd9eb-f46d-afa9-b08b-15e202b3a624@linaro.org>
In-Reply-To: <ec6fd9eb-f46d-afa9-b08b-15e202b3a624@linaro.org>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?yvTkywv8d/7q1MX4X3vGMGRn1vXTNgOsSaXyb1lHNW0z2FMOj4FpdS0rUv7j?=
 =?us-ascii?Q?eTFhzkWqtt3sf3RCA8w6TqlWc3EyNmys2Xy9Ysc7vHpOFZDXAaxW+lTFZZlN?=
 =?us-ascii?Q?Fd0KN1W32r3xpM2ygchka+WMegLu6jx+mug0pGEwPODcTM1u5WO5BEr/KCHN?=
 =?us-ascii?Q?qTPwRSd/h7YC7cIRWLqVHoocAYc6DsdzbnsWsFPOUyBHz7ndflQh4yHT1xZY?=
 =?us-ascii?Q?clDr2NMdbfpAZzWltFv/yv3NG8Oq2BKNjGA/pHI/Wx+kYSEHo0u4IOhab2FH?=
 =?us-ascii?Q?9YPeo3Yt38aB4JloprmSA3wJRcHsIeSxK3DBuD6zr2upnAj2M2aWDjLuQaG5?=
 =?us-ascii?Q?6tnud957d6HEB0xqGpmFCMbPC0sd7/pukQ7DAhiEXGECNybvyDaCamF+9SF9?=
 =?us-ascii?Q?YJZMl5Id9eFOmrU859XHl+A39QC9H0v+aRjLA9ptEzZk0nodwQUmU5KEg6IN?=
 =?us-ascii?Q?++DcFTuTIYW9/C1ilgv0NsGmSi6ck8fUmSXd/t/TI5qfkGrxiecdgGLSJQl4?=
 =?us-ascii?Q?ZY0g5C1ktq4PTo4AKmJyRYxUemghx2IdMOv8Bakp+rWAEt5v0SrUee7UbIgK?=
 =?us-ascii?Q?SSpHHxfAX0b1X+eh4kNOz/P3Hn6BVXKKX3d6oNNeAXpl4GJri+pcQIP5DtXA?=
 =?us-ascii?Q?kpHztCE1aYEPnndeCBt80Nel5bSHirPDnNP2KishRfNi9wAXHYUCreYagWj2?=
 =?us-ascii?Q?spffnOygkQdtmO3fA1JGFHlvc0zLHv42C0Ddysi0cdNUVIEqxiLO76K7oT/M?=
 =?us-ascii?Q?dlN2TpLBubTfcN9fcSyL1sigWdK+rWwRVbUBvtyakHXm441B5Ts1Fvczqsev?=
 =?us-ascii?Q?JaOu4eYFuerqUprjxDXcxPELFT6Sqx9lqy7FHzrSO4Ow7iuUILrPjqra5xfU?=
 =?us-ascii?Q?79Irdbsql1i31T/L8nR7YRsvKD7vGVe2M2AyoiZ4HD/mJapFcUOFtoYnJSD4?=
 =?us-ascii?Q?4YsphSCA8zsHSXLxiUoCrI68pKMhyohve1M/Bcv0CH6ZmGmA6VXAJTNBbVVJ?=
 =?us-ascii?Q?oVrpPiQJ+5K8VdwR90UuDF1FvFkjvx5wzSV4XZZokY0Sj/O0Kcl9yLYVC4/G?=
 =?us-ascii?Q?3guWu7M1Uy4N+Qvvo+U0Uov3J/WxyCu6P3bgehN9vblLdiGJaLzUfs29Mpe0?=
 =?us-ascii?Q?HjZQogIYfDouXv4U4NjM6EyjKH93NdQCnUY4jbupidBAlt3gLODP9OmuN5Kb?=
 =?us-ascii?Q?JHar70lQLP/nzTQh63IJb9mxCzaQr5jwnp1COaWUsrwZoUHs39WNc1RGxAU2?=
 =?us-ascii?Q?x870utWOuAH7m9OhYjohJ/GBNRIaXdPW1GaOwUoC76laotgbqgDOrGq9GuZy?=
 =?us-ascii?Q?sW/0y58npiB+xXU/Z8RiEGfvTOFhmU5CD4Y6DztfJP+HWypkNUIwyKiFNnrL?=
 =?us-ascii?Q?WlZq5XrKmF6D70IcZHeJaTz8qwPU4jx6PtUAkCdiZs6VsO615+mvrEC2fUF4?=
 =?us-ascii?Q?yPGS+HHvTkKQsv58OtH9aV69hla3Ae4qdI42AABLzKL2lCIjzk1M5etqbLWG?=
 =?us-ascii?Q?kfjI43A7Q/1ltPZXbRH1ffYZN/L+nipfojrH2p2ZOgiJpj581BxJ6jcFejuz?=
 =?us-ascii?Q?t1eX6QFuxR0/+rgma5z2slrCDzIiF3yaxsISLUZ+?=
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: AM9PR04MB8274.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 643fe480-cad7-41c2-9e20-08da7ae45227
X-MS-Exchange-CrossTenant-originalarrivaltime: 10 Aug 2022 15:23:47.5650
 (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: o24aJDGt4Bk5LKWoYoQY/MGDS+vBK6A5kTzNxXOp/jugPnSBNIVbyaIfbb+dkyNGYizAc7NEYg1xXZTFIrwgxQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PAXPR04MB9074
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,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


> -----Original Message-----
> From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> Sent: Wednesday, August 10, 2022 9:46 AM
> To: Shenwei Wang <shenwei.wang@nxp.com>; robh+dt@kernel.org;
> krzysztof.kozlowski+dt@linaro.org; linus.walleij@linaro.org; brgl@bgdev.p=
l;
> shawnguo@kernel.org; s.hauer@pengutronix.de; kernel@pengutronix.de;
> festevam@gmail.com; dl-linux-imx <linux-imx@nxp.com>
> Cc: devicetree@vger.kernel.org; linux-kernel@vger.kernel.org; linux-
> gpio@vger.kernel.org; linux-arm-kernel@lists.infradead.org; imx@lists.lin=
ux.dev
> Subject: [EXT] Re: [PATCH v3 1/3] dt-bindings: gpio: Add imx scu gpio dri=
ver
> bindings
>=20
> Caution: EXT Email
>=20
> On 10/08/2022 16:30, Shenwei Wang wrote:
> > Add binding document for the imx scu gpio driver.
> >
> > Signed-off-by: Shenwei Wang <shenwei.wang@nxp.com>
> > ---
> >  .../bindings/gpio/fsl,imx-sc-gpio.yaml        | 40 +++++++++++++++++++
> > +properties:
> > +  compatible:
> > +    enum:
> > +      - fsl,imx8qxp-sc-gpio
> > +      - fsl,imx-scu-gpio
>=20
>=20
> This is too generic compatible and it even conflicts with the above...
> Your driver binds to both without driver data which is even more confusin=
g.
> Make the compatible specific for one, given SoC.
>=20

Oh, that's my bad. It is a typo. The generic compatible should be "fsl,imx-=
sc-gpio". =20
So far there is no driver data required for both imx8qxp and imx8dxl, but i=
t might be
required sometimes later in case the scu firmware is customized for a speci=
fied use case.
That's why I put the generic one here.

Thanks,
Shenwei

>=20
> Best regards,
> Krzysztof