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=-14.0 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED 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 2F6DBC433E0 for ; Wed, 10 Mar 2021 08:41:49 +0000 (UTC) Received: from desiato.infradead.org (desiato.infradead.org [90.155.92.199]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id A1BAA64FE7 for ; Wed, 10 Mar 2021 08:41:48 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org A1BAA64FE7 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=cerno.tech Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=desiato.20200630; h=Sender:Content-Type: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:Cc:To:From:Date: Reply-To:Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date :Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=I1zAAQUgYYh0kyOuyeJ46ShVFZjS9uo64alW9pO6uLQ=; b=SGcrVOg9evdSWWtX75La8Jvuo QgtTYETfc5bVH21zwLwP5wyoMxnZZIfDS5Ytxq0cMCen7J9+34ewhO089eQHDCa6pxb54dMcAc+y3 IhxoetTxrQp0PpPNGwQxZiRi5SnE3dyjTc6rTQoNUFR99PnXEEOjd8aDgZrzUJARdUMyCORJ4LRZr lu8dL901lsW8OjqjUWzrzcxEnSfqm0fwqZAjSRSdjCAruykfvCDWNJownkP2BWgcEbNdqaC/ntTCK 4ffVKiWrRL96lotlVQq+76Vx7ss6wAEoTkd/9nm+M1fR1BLgmtZ5GJ7gLHLgfU32r6KVdVriOGeSL OcDvCTdHw==; Received: from localhost ([::1] helo=desiato.infradead.org) by desiato.infradead.org with esmtp (Exim 4.94 #2 (Red Hat Linux)) id 1lJuO3-006JrU-5L; Wed, 10 Mar 2021 08:40:11 +0000 Received: from out4-smtp.messagingengine.com ([66.111.4.28]) by desiato.infradead.org with esmtps (Exim 4.94 #2 (Red Hat Linux)) id 1lJuNx-006JpE-SY for linux-arm-kernel@lists.infradead.org; Wed, 10 Mar 2021 08:40:07 +0000 Received: from compute6.internal (compute6.nyi.internal [10.202.2.46]) by mailout.nyi.internal (Postfix) with ESMTP id 5AC0B5C0113; Wed, 10 Mar 2021 03:40:00 -0500 (EST) Received: from mailfrontend2 ([10.202.2.163]) by compute6.internal (MEProxy); Wed, 10 Mar 2021 03:40:00 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cerno.tech; h= date:from:to:cc:subject:message-id:references:mime-version :content-type:in-reply-to; s=fm2; bh=UWYa6L3xNrTqeLtZgTGVhaunhWu w8dTIJzklNZwMjO4=; b=ovyv+4ZqGxtSsKxVtasS9nYx1VEJF4QSmqS/iloSgly bJNZXUFBszx3YTCXgQ5fYLJKPEgyRxTjbu6ArJcBV384P1iNIN8DAQp6aGDHV5nh MDVi7VphC3hZHGYPwXNh06MDV+62og+c2lXb6DpEp9Kfqn9lqaZS5fXEln9+Vqyf tGbHfP3rdWQxYuy3OrCaWfEKUAE2Y0KdaOpMYLfuyiPNdWreRnztEWB3RYgw/rX8 bC72Xv46iLB/MolqSRgU1OT6WRddtABNcYa98MpfaB9XJBUrAf4uRpR/BNJXlCzG 3t8xvpOuK5DbJjc4mlBIzcEfdOXzIomUClLNiJmS6dQ== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-me-proxy :x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s=fm2; bh=UWYa6L 3xNrTqeLtZgTGVhaunhWuw8dTIJzklNZwMjO4=; b=mpyOwkLsr7NfXQr0aU5jNp 2fMuDaKTK7RzYk80JGCOkSup5PvjfFn4ioU0hIp/2M2SRNIjE8HB/hInSs4xlgcZ OUaymUPkMmP2kCC9be7FKAsJUYOgjnxRyk/U19btfCFzyGEpTdhQBBt1zKaWWeHP YRuXWaZjU1OqcasugG09TwOeazWJfwuVAYCHQKTz2/AleHLelkUU+rZ0XBK3e557 oZhjMLwL6B7Wap8YwR2z6wzRCNec/W7xL8sdhPX3YqIBylR2bLHoTNYd+G04bRrb C6lCoFT4XXMnPolHXy57DeMxWbxBLIIlhKeTbSTQpWm+ObQe5pmUnZp8TWrkdzig == X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeduledruddujedguddutdcutefuodetggdotefrod ftvfcurfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfgh necuuegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmd enucfjughrpeffhffvuffkfhggtggujgesghdtreertddtvdenucfhrhhomhepofgrgihi mhgvucftihhprghrugcuoehmrgigihhmvgestggvrhhnohdrthgvtghhqeenucggtffrrg htthgvrhhnpeelkeeghefhuddtleejgfeljeffheffgfeijefhgfeufefhtdevteegheei heegudenucfkphepledtrdekledrieekrdejieenucevlhhushhtvghrufhiiigvpedtne curfgrrhgrmhepmhgrihhlfhhrohhmpehmrgigihhmvgestggvrhhnohdrthgvtghh X-ME-Proxy: Received: from localhost (lfbn-tou-1-1502-76.w90-89.abo.wanadoo.fr [90.89.68.76]) by mail.messagingengine.com (Postfix) with ESMTPA id 598911080063; Wed, 10 Mar 2021 03:39:59 -0500 (EST) Date: Wed, 10 Mar 2021 09:39:57 +0100 From: Maxime Ripard To: Evgeny Boger Cc: Chen-Yu Tsai , linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Rob Herring Subject: Re: [PATCH v2 1/2] net: allwinner: reset control support Message-ID: <20210310083957.utmn7twsm2qgksfp@gilmour> References: <20210309012116.2944-1-boger@wirenboard.com> <20210309012116.2944-2-boger@wirenboard.com> MIME-Version: 1.0 In-Reply-To: <20210309012116.2944-2-boger@wirenboard.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210310_084006_272145_3E3842B4 X-CRM114-Status: GOOD ( 18.58 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: multipart/mixed; boundary="===============3055669148326500233==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============3055669148326500233== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="uizhoovv4z7qhw72" Content-Disposition: inline --uizhoovv4z7qhw72 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi, On Tue, Mar 09, 2021 at 04:21:15AM +0300, Evgeny Boger wrote: > R40 (aka V40/A40i/T3) and A10/A20 share the same EMAC IP. > However, on R40 the EMAC is gated by default. >=20 > Signed-off-by: Evgeny Boger > --- > .../net/allwinner,sun4i-a10-emac.yaml | 11 +++- > drivers/net/ethernet/allwinner/sun4i-emac.c | 65 +++++++++++++++++-- > 2 files changed, 70 insertions(+), 6 deletions(-) >=20 > diff --git a/Documentation/devicetree/bindings/net/allwinner,sun4i-a10-em= ac.yaml b/Documentation/devicetree/bindings/net/allwinner,sun4i-a10-emac.ya= ml > index 8d8560a67abf..27f99372d153 100644 > --- a/Documentation/devicetree/bindings/net/allwinner,sun4i-a10-emac.yaml > +++ b/Documentation/devicetree/bindings/net/allwinner,sun4i-a10-emac.yaml > @@ -15,7 +15,12 @@ maintainers: > =20 > properties: > compatible: > - const: allwinner,sun4i-a10-emac > + oneOf: > + - const: allwinner,sun4i-a10-emac > + - const: allwinner,sun4i-r40-emac > + - items: > + - const: allwinner,sun4i-r40-emac > + - const: allwinner,sun4i-a10-emac There's no need to handle the fallback case, it should have either one of the two, not both. The good news is that it simplifies the binding here too, since you can use an enum The DT binding modifications are usually in a separate patch too > =20 > reg: > maxItems: 1 > @@ -30,6 +35,9 @@ properties: > description: Phandle to the device SRAM > $ref: /schemas/types.yaml#/definitions/phandle-array > =20 > + resets: > + maxItems: 1 > + You should make resets required for the R40 compatible too through an if clause. It looks good otherwise, thanks! Maxime --uizhoovv4z7qhw72 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRcEzekXsqa64kGDp7j7w1vZxhRxQUCYEiF2AAKCRDj7w1vZxhR xWf+AQCWKzhut6ZPGHD5e8P7IIIhhkd7RuiPg8Dkq0nZaHFZqwEAwH3hAqMwcQ8U GKPOy+EvneB2MZR+8Ry5N/mB9HicQwg= =t6wL -----END PGP SIGNATURE----- --uizhoovv4z7qhw72-- --===============3055669148326500233== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============3055669148326500233==--