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:   =?ISO-8859-1?Q?J=E9r=F4me?= Pouiller <jerome.pouiller@silabs.com>
To:     Oleksij Rempel <o.rempel@pengutronix.de>
Cc:     Maxime Coquelin <mcoquelin.stm32@gmail.com>,
        Alexandre Torgue <alexandre.torgue@foss.st.com>,
        Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzk+dt@kernel.org>,
        kernel@pengutronix.de, linux-kernel@vger.kernel.org,
        linux-stm32@st-md-mailman.stormreply.com,
        linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v3 2/4] dt-bindings: net: silabs, wfx: add prt, prtt1c-wfm200 antenna variant
Date:   Tue, 26 Apr 2022 10:19:29 +0200
Message-ID: <4686508.cEBGB3zze1@pc-42>
Organization: Silicon Labs
In-Reply-To: <20220426072113.GC3419@pengutronix.de>
References: <20220425132844.866743-1-o.rempel@pengutronix.de> <14881918.tv2OnDr8pf@pc-42> <20220426072113.GC3419@pengutronix.de>
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain; charset="iso-8859-1"
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?iso-8859-1?Q?0GlAZwmyyr34CbvxOsdN6uu/8awwriPgwZgBkzkjOQGL/X7wiNUqdULbyw?=
 =?iso-8859-1?Q?7wI/tFJLoO3Hu3K0+Dr1EbOcGd7i6D945oh1/w0YpHwrPrKdiLSKb2KwwB?=
 =?iso-8859-1?Q?GGUfXgEeGOlZKoUYYLCjbnpn3MYmIlvld4/UWj8VvhvqoABhLOy8Yy22gx?=
 =?iso-8859-1?Q?rx1FfOJy1J/kOAloTqzfwfdPmzz2vA4KqTmkNI5O1Ld3qhHK3Uze7EgJj4?=
 =?iso-8859-1?Q?rcPAQWhFaDf/HdSQpc2KMvqNXMMaR1Z9jsSVfjK983bSBMI6JDXuf9+by/?=
 =?iso-8859-1?Q?36vQR3PSxZH9cXWxCey5pzC3wTfSQq/K9/ye3UTzHmfi/HHjaDXxX2ebRs?=
 =?iso-8859-1?Q?YxJORE2GzC2cL/EoBcU0xt8/1WwX7aYrH6v+ftcbzs3JhHHGYIte65O2L3?=
 =?iso-8859-1?Q?Bn2Vep/tQbdvUGnyQf5F6DtTLOl4UAAyJaZSPatoyi+uNwJeOyic+yZBYe?=
 =?iso-8859-1?Q?LIz6c3CIPs10qCyAVZLfqW+xsTSLWOGzAFB8TIyYEhTcOuPrWf/Wuig4P/?=
 =?iso-8859-1?Q?LD4hijJge6CjvfOyZue7hmbsRk5DfVZBVERvYPjea+egVPf4QNLGih3BIN?=
 =?iso-8859-1?Q?53F1T+RvUREVVRT+Hv5LuWBzRR1sa2h+UkWkplTK5KaVhsbdHurPIEdfbU?=
 =?iso-8859-1?Q?c3tWJzo2ie0bWj7ZGBzVTXx6+zivozYGsoFE5fyrERVKyR6ou/wDrXX+NN?=
 =?iso-8859-1?Q?HXvjLyapn7gVjYhkVx9NXx4jYDcpOfJ918yfsjXZ/d4ZS3+FAPGHmg4LCZ?=
 =?iso-8859-1?Q?1dXc1ZAZo9TjNi9Suz9j1sZMiznwZn80iDrpCj3KFEX2r7oFEpxwcNhFOV?=
 =?iso-8859-1?Q?n0Vn/dXeMmhMRiho4hFnUcHVPIPTc/HU1xbS6YzElSs5Ct8NFG4YIGfG1t?=
 =?iso-8859-1?Q?/h1m7b62t7xURUBTvu6XPTvamh+a8seNyPC6j8wQw6CiXXGQeBGZEEv9fp?=
 =?iso-8859-1?Q?BcK2mhUaqHV1rNqCGRB1NHIXkcDPWt2K/zzUqBBeS08ghtRh1ZL08dqp4C?=
 =?iso-8859-1?Q?TgyuNI3lyk4IL7J9P8tyFraHNSkqtPHCN2xVMxbHSokfGwEm1szzcIgdGY?=
 =?iso-8859-1?Q?yb61KKJ59LRE1DdufTdcFM88VFRPFcDQTBP/rJqAIU79mYydoSgQks2hAo?=
 =?iso-8859-1?Q?qeBfw2m+vW7kttnbVkdfdYZNbFqlSlBAugNLR6R+8PDIVhNFKmNjSn36vE?=
 =?iso-8859-1?Q?PRKbwdrzzvQ1YyiPmWRhpt0yc9J/SAT8B4GOyvGReex7Ia+WRGJ7FRWJML?=
 =?iso-8859-1?Q?OnGo9/Cr5YgwylyYmzx1R5Im1JHg6QDPL3FYBe6KqhqwlnRhF/5iaSb7GB?=
 =?iso-8859-1?Q?J3tfpbmK0ImhiUPsTevXu0FZj0jDgDcQ5NniHbQV3VK6uwOUU3Bk2WkahR?=
 =?iso-8859-1?Q?1EpS8LajxsWuogmpWArI6laWJGhMsgVmsSkEmrKJmybtB0XsXyoFsA/9T8?=
 =?iso-8859-1?Q?e3UfR4X6XJKlF5/L9gSSc41H/pN+DB7WojKtDmj6s4XbQFWSrOXMBwyd2K?=
 =?iso-8859-1?Q?KqFd6PvL9Ac3hkc6RGbP3Ty5Pi/CS5dSJsU8p9b/vpdVe9PRJCrgWckYrY?=
 =?iso-8859-1?Q?XCjPkzmVWOmbQiC5BLh5kAGQ4sKkvkwhIo6JgDgWa5CdNzZoSz0QUBLmk9?=
 =?iso-8859-1?Q?3eU/AAQS81jllJ/fDFKrIVQLib+tvdQpn7Nd/1uTkfmXwjMTQHvdafQQjG?=
 =?iso-8859-1?Q?XL+CCdf8dPS5ZUk/BCKcyfD1arNTQoxHPz8w7S+SbLfq7RVmJlhxmqB3lt?=
 =?iso-8859-1?Q?6DJ3kaHjhoylx3MP0BSq54QUg3DOzNPBzWw9ynBkMh5pNyM4AEFeja1uOH?=
 =?iso-8859-1?Q?CB9AF5BpIw=3D=3D?=
X-OriginatorOrg: silabs.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 62b946f6-2b6b-42ca-ea44-08da275d8046
X-MS-Exchange-CrossTenant-AuthSource: PH0PR11MB5657.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 26 Apr 2022 08:19:36.5169
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 54dbd822-5231-4b20-944d-6f4abcd541fb
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: iV5q+UvQQiPDG8WKYxIMX7pfph+x+XStKICBdL7LNsnuu81zq3AqLcvFLkPTBcjgyF9kzp9QEbFbSGcywmC7Mw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR11MB4448
X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS
        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

On Tuesday 26 April 2022 09:21:13 CEST Oleksij Rempel wrote:
> On Mon, Apr 25, 2022 at 05:38:20PM +0200, J=E9r=F4me Pouiller wrote:
> > On Monday 25 April 2022 15:28:42 CEST Oleksij Rempel wrote:
> > > Add compatible for wfm200 antenna configuration variant for Protonic =
PRTT1C
> > > board.
> > >
> > > Signed-off-by: Oleksij Rempel <o.rempel@pengutronix.de>
> > > ---
> > >  .../devicetree/bindings/staging/net/wireless/silabs,wfx.yaml     | 1=
 +
> > >  1 file changed, 1 insertion(+)
> > >
> > > diff --git a/Documentation/devicetree/bindings/staging/net/wireless/s=
ilabs,wfx.yaml b/Documentation/devicetree/bindings/staging/net/wireless/sil=
abs,wfx.yaml
> > > index 105725a127ab..ce107fe45d7c 100644
> > > --- a/Documentation/devicetree/bindings/staging/net/wireless/silabs,w=
fx.yaml
> > > +++ b/Documentation/devicetree/bindings/staging/net/wireless/silabs,w=
fx.yaml
> > > @@ -39,6 +39,7 @@ properties:
> > >    compatible:
> > >      items:
> > >        - enum:
> > > +          - prt,prtt1c-wfm200 # Protonic PRTT1C Board
> > >            - silabs,brd4001a # WGM160P Evaluation Board
> > >            - silabs,brd8022a # WF200 Evaluation Board
> > >            - silabs,brd8023a # WFM200 Evaluation Board
> >
> > I think you also have to declare this new entry in wfx_sdio_of_match,
> > and/or wfx_spi_of_match and wfx_spi_id.
> >
> > On the WFM200 variant, the antenna is built in the chip. So I think you
> > can point on the same configuration than the brd8023a.
>=20
> I assume driver update can be made in a separate patch. As soon as we
> reserved compatible for this boards there should be no conflicts. This pa=
tches
> series is targeting only devicetree and needed DT schema changes.

hmmm... I am not DT specialist, but it seems a bit weird to have a new
entry in the dt schema, but no new code in any of_match table.=20

--=20
J=E9r=F4me Pouiller