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 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id E3883C2BD09 for ; Thu, 27 Jun 2024 09:58:30 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To:Content-Type: 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=uN1x7EUaOa+IEjjdSlF5Go5KOVB9c7Sw7XqJvFq1eK4=; b=TIRkJctxJNt9darG9a9lcG4ZbI Bc3dWuNkHiDfU8ck6kItTw0t3swHeGnowq8BdfZTji2NZm6ZatqBCe4nCu10u8BuKLRtMP9KLMNl8 Pa5KUWiToGek3SvhZLatOeJcJ9Bph4l9lGOaprm0Sql7iVGfLycWmUz0JzYnUKxXNXrP+gshHMcls rh3jZhOX54mRoP7tVamJu4XK5jCMaNtel49LyU2aQTPDMvjQgL6swtFBxGTR0kk/zURYyhke7gUgN PrpHVN51/9BdPLPwoxIJ/uAdhjoIdmws2RVqJGPkE14Xbl3130xe1k2BaWsTNS51NhzEDYeRop1Uc m6j2fDpw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sMltQ-00000009rAq-3LYx; Thu, 27 Jun 2024 09:58:16 +0000 Received: from esa.microchip.iphmx.com ([68.232.154.123]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sMltI-00000009r9T-0GCf for linux-arm-kernel@lists.infradead.org; Thu, 27 Jun 2024 09:58:09 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=microchip.com; i=@microchip.com; q=dns/txt; s=mchp; t=1719482288; x=1751018288; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=/vd3XG40y+3D+nrQ6Ws/UOz8gxZ/84gh2P/6OOJKCIk=; b=YP63HT1GqYQY3lSuwd9BSoMTg0J05ZVMvFOvNM/XNW92VgzRbZTFBx+M F/Kk5L4gmFUUT3mzYVXVuFatuH/gWA6fPmW4OAvYqE3cJ8viBo4J6ythT YeG/KyQZQigvmN2iEG2BHtnWoD8fSesi2gN3Xu5zxLb+aiQsX1P3gCDHn x6kQIpENks3JZ3Y28k9gsxRAP8L9CiDj7IhcVoDOGKrcBN9KIfBg6Os45 3GAZrc0AjCv9hAGIHUTQNrO6LSswoRJSYPRYpiiycm8+FMZfX5VFuCsuv DB9OyTRI7OdbGIWP360v413xsEeoJDbYLKBti95h7rRsebqPDUjoxriRf w==; X-CSE-ConnectionGUID: EknzzAuNRqe944fDC+OLkQ== X-CSE-MsgGUID: Vh6Cpy12ShK6HQXAa/XEbw== X-IronPort-AV: E=Sophos;i="6.08,269,1712646000"; d="asc'?scan'208";a="28542765" X-Amp-Result: UNKNOWN X-Amp-Original-Verdict: FILE UNKNOWN Received: from unknown (HELO email.microchip.com) ([170.129.1.10]) by esa4.microchip.iphmx.com with ESMTP/TLS/ECDHE-RSA-AES128-GCM-SHA256; 27 Jun 2024 02:58:05 -0700 Received: from chn-vm-ex04.mchp-main.com (10.10.85.152) by chn-vm-ex03.mchp-main.com (10.10.85.151) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.35; Thu, 27 Jun 2024 02:57:47 -0700 Received: from wendy (10.10.85.11) by chn-vm-ex04.mchp-main.com (10.10.85.152) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.35 via Frontend Transport; Thu, 27 Jun 2024 02:57:44 -0700 Date: Thu, 27 Jun 2024 10:57:28 +0100 From: Conor Dooley To: AngeloGioacchino Del Regno CC: Lorenzo Bianconi , , , , , , , , , , , , , , , , , Subject: Re: [PATCH v3 1/4] dt-bindings: clock: airoha: Add reset support to EN7581 clock binding Message-ID: <20240627-undying-overcoat-192e5aa20c55@wendy> References: <20240627-vending-lisp-4e1cf45e552c@wendy> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="BOxG0eYiqnj1tfTc" Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240627_025808_312580_84F034C2 X-CRM114-Status: GOOD ( 26.26 ) 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: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --BOxG0eYiqnj1tfTc Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Jun 27, 2024 at 11:53:00AM +0200, AngeloGioacchino Del Regno wrote: > Il 27/06/24 11:47, Conor Dooley ha scritto: > > On Thu, Jun 27, 2024 at 11:33:47AM +0200, AngeloGioacchino Del Regno wr= ote: > > > Il 13/06/24 14:47, Lorenzo Bianconi ha scritto: > > > > Introduce reset capability to EN7581 device-tree clock binding > > > > documentation. > > > >=20 > > > > Signed-off-by: Lorenzo Bianconi > > > > --- > > > > .../bindings/clock/airoha,en7523-scu.yaml | 25 ++++++- > > > > .../dt-bindings/reset/airoha,en7581-reset.h | 66 +++++++++++++= ++++++ > > > > 2 files changed, 90 insertions(+), 1 deletion(-) > > > > create mode 100644 include/dt-bindings/reset/airoha,en7581-reset= =2Eh > > > >=20 > > > > diff --git a/Documentation/devicetree/bindings/clock/airoha,en7523-= scu.yaml b/Documentation/devicetree/bindings/clock/airoha,en7523-scu.yaml > > > > index 3f4266637733..84353fd09428 100644 > > > > --- a/Documentation/devicetree/bindings/clock/airoha,en7523-scu.yaml > > > > +++ b/Documentation/devicetree/bindings/clock/airoha,en7523-scu.yaml > > > > @@ -35,7 +35,7 @@ properties: > > > > reg: > > > > minItems: 2 > > > > - maxItems: 3 > > > > + maxItems: 4 > > > > "#clock-cells": > > > > description: > > > > @@ -43,6 +43,10 @@ properties: > > > > clocks. > > > > const: 1 > > > > + '#reset-cells': > > > > + description: ID of the controller reset line > > > > + const: 1 > > > > + > > > > required: > > > > - compatible > > > > - reg > > > > @@ -60,6 +64,8 @@ allOf: > > > > - description: scu base address > > > > - description: misc scu base address > > > > + '#reset-cells': false > > > > + > > > > - if: > > > > properties: > > > > compatible: > > > > @@ -70,6 +76,7 @@ allOf: > > > > items: > > > > - description: scu base address > > > > - description: misc scu base address > > > > + - description: reset base address > > >=20 > > > Are you sure that the indentation is correct? :-) > > >=20 > > > After fixing the indentation, > > >=20 > > > Reviewed-by: AngeloGioacchino Del Regno > > >=20 > > > > - description: pb scu base address > >=20 > > The indentation actually looks okay when I apply this locally, but how = is > > it backwards compatible to add this register in the middle of the list?? >=20 > It's not, and this is actually something done on purpose - there is no DT= using > this binding yet (here, nor uboot), and Lorenzo acknowledged the mistake = before > it was too late... >=20 > At least this time, it wasn't a misattention :-P The rationale for this being okay really should be in the commit message... > Btw, as far as I know, the reset base address is in between misc scu and = pb scu, > that's why it was put there in the middle. =2E..but I don't really see why this needs to be done incompatibly at all in the first place. Just put it at the end of the list, there's no rule that the order of reg properties needs to match the MMIO addresses. --BOxG0eYiqnj1tfTc Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZn03iAAKCRB4tDGHoIJi 0kheAP0YP5jOg+N09ULXONejYGeXQVrKSBAlPvGsbtj0U3yHMAD6AgpdvZ2qLb2c bqZPZekWUgQDbXvJG4plKE9MrMb6Sw8= =VlR1 -----END PGP SIGNATURE----- --BOxG0eYiqnj1tfTc--