From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.8 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,USER_AGENT_NEOMUTT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id D92FEC31E4B for ; Fri, 14 Jun 2019 14:59:12 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id B5E6021721 for ; Fri, 14 Jun 2019 14:59:12 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728918AbfFNO7M (ORCPT ); Fri, 14 Jun 2019 10:59:12 -0400 Received: from relay9-d.mail.gandi.net ([217.70.183.199]:43423 "EHLO relay9-d.mail.gandi.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728857AbfFNO7K (ORCPT ); Fri, 14 Jun 2019 10:59:10 -0400 X-Originating-IP: 90.88.23.150 Received: from localhost (aaubervilliers-681-1-81-150.w90-88.abo.wanadoo.fr [90.88.23.150]) (Authenticated sender: maxime.ripard@bootlin.com) by relay9-d.mail.gandi.net (Postfix) with ESMTPSA id B5937FF806; Fri, 14 Jun 2019 14:59:02 +0000 (UTC) Date: Fri, 14 Jun 2019 16:59:02 +0200 From: Maxime Ripard To: Rob Herring Cc: Andrew Lunn , Mark Rutland , Frank Rowand , "David S . Miller" , Chen-Yu Tsai , Maxime Coquelin , Alexandre Torgue , netdev , "moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE" , devicetree@vger.kernel.org, linux-stm32@st-md-mailman.stormreply.com, Maxime Chevallier , Antoine =?utf-8?Q?T=C3=A9nart?= , Florian Fainelli , Heiner Kallweit Subject: Re: [PATCH v2 05/11] dt-bindings: net: sun4i-emac: Convert the binding to a schemas Message-ID: <20190614145902.vjytw74bs5roh2f2@flea> References: <91618c7e9a5497462afa74c6d8a947f709f54331.1560158667.git-series.maxime.ripard@bootlin.com> <20190610143139.GG28724@lunn.ch> <20190611145856.ua2ggkn6ccww6vpp@flea> <20190614095048.j2xwdsucucbakkl2@flea> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="wmxpz6l3hrzuotke" Content-Disposition: inline In-Reply-To: User-Agent: NeoMutt/20180716 Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org --wmxpz6l3hrzuotke Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Hi, On Fri, Jun 14, 2019 at 07:37:49AM -0600, Rob Herring wrote: > > > For '-gpio', we may be okay because the suffix is handled in the GPIO > > > core. It should be safe to update the binding to use the preferred > > > form. > > > > It might require a bit of work though in drivers, since the fallback > > is only handled if you're using the gpiod API, and not the legacy one. > > > > > > And then, we need to agree on how to express the deprecation. I guess > > > > we could allow the deprecated keyword that will be there in the > > > > draft-8, instead of ad-hoc solutions? > > > > > > Oh, nice! I hadn't seen that. Seems like we should use that. We can > > > start even without draft-8 support because unknown keywords are > > > ignored (though we probably have to add it to our meta-schema). Then > > > at some point we can add a 'disallow deprecated' flag to the tool. > > > > So, in the generic ethernet binding, we would have: > > > > properties: > > phy-handle: > > $ref: /schemas/types.yaml#definitions/phandle > > description: > > Specifies a reference to a node representing a PHY device. > > > > phy: > > $ref: "#/properties/phy-handle" > > deprecated: true > > > > phy-device: > > $ref: "#/properties/phy-handle" > > deprecated: true > > > > Does that sound good? > > Yes. Great, I'll post that. > > Now, how do we handle the case above, in the device specific binding? > > We just require the non-deprecated one, or the three? > > Wouldn't that just depend if all the instances of the device specific > binding have been updated? You mean in the DTS? It shouldn't matter, we'll want to have a warning anyway. But yeah, I'll update them too. Maxme -- Maxime Ripard, Bootlin Embedded Linux and Kernel engineering https://bootlin.com --wmxpz6l3hrzuotke Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRcEzekXsqa64kGDp7j7w1vZxhRxQUCXQO2NgAKCRDj7w1vZxhR xYYMAQCpuUGZecwJPnE6MJSUbuL1fXyUaH+2U+W+fTF4OHIb8gEAxykOmBXCzeYm qShBRvc18IIAbeOPCAbqTKJy+kpccgc= =AR23 -----END PGP SIGNATURE----- --wmxpz6l3hrzuotke--