From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from lindbergh.monkeyblade.net (lindbergh.monkeyblade.net [23.128.96.19]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 4024520F9 for ; Thu, 28 Sep 2023 06:27:53 +0000 (UTC) Received: from esa.microchip.iphmx.com (esa.microchip.iphmx.com [68.232.153.233]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 30EAB99; Wed, 27 Sep 2023 23:27:51 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=microchip.com; i=@microchip.com; q=dns/txt; s=mchp; t=1695882471; x=1727418471; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=mBkNygKzMaSU02DnlJ0mWUgp02ttdarWgUm5ztFYDpE=; b=WRCPLx3PMOHscVTlGotCkNea4Cz/vdwJ4uh91Cdnp5M2yEeLGnYB5VAV kplWwg509pAMR0sD8ygjHAd62wbcYsr1pWYfpYTqZaWZq8uq2NKIatsKQ 2KdK0n9y3ZVYJ9Q8WbAzBqFOgvo/v3irhynIeRABJt4iuxgkBu1axTmEE dmkMB7CJiHKVK9bGhiug6c7x+4qjL4OCAHvUXFwDis+TwXafhJOWr66xn sJUOS/DRwG5eUzq3X63d3U0KDJ9Zktz4UeqhLkAZVlUSW8Hhs4PPzWb/V 52Er1Cb5ZyAInXRsJeq6BZD4L7M09U8hqdba+9vlYC4zgMUhvBB2ATNTL g==; X-CSE-ConnectionGUID: ESpAaoDnSL2M9L5hctp+oQ== X-CSE-MsgGUID: SxFkAdzQTq2FWm/6/W8BNg== X-ThreatScanner-Verdict: Negative X-IronPort-AV: E=Sophos;i="6.03,183,1694761200"; d="asc'?scan'208";a="7286625" X-Amp-Result: UNKNOWN X-Amp-Original-Verdict: FILE UNKNOWN Received: from unknown (HELO email.microchip.com) ([170.129.1.10]) by esa1.microchip.iphmx.com with ESMTP/TLS/ECDHE-RSA-AES128-GCM-SHA256; 27 Sep 2023 23:27:50 -0700 Received: from chn-vm-ex02.mchp-main.com (10.10.85.144) by chn-vm-ex01.mchp-main.com (10.10.85.143) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.21; Wed, 27 Sep 2023 23:27:30 -0700 Received: from wendy (10.10.85.11) by chn-vm-ex02.mchp-main.com (10.10.85.144) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.21 via Frontend Transport; Wed, 27 Sep 2023 23:27:27 -0700 Date: Thu, 28 Sep 2023 07:27:09 +0100 From: Conor Dooley To: Inochi Amaoto CC: Conor Dooley , , , , , , , , , , , , , , Anup Patel , Chen Wang Subject: Re: [PATCH v3 06/11] dt-bindings: timer: Add Sophgo sg2042 CLINT timer Message-ID: <20230928-bring-gnarly-05eeeb85f3c5@wendy> References: <20230927-uncorrupt-bronco-72e35232b1cc@spud> Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="UHaWB33MwrDFW0WP" Content-Disposition: inline In-Reply-To: X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF, RCVD_IN_DNSWL_BLOCKED,RCVD_IN_MSPIKE_H5,RCVD_IN_MSPIKE_WL, SPF_HELO_PASS,SPF_NONE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net --UHaWB33MwrDFW0WP Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Sep 28, 2023 at 08:34:42AM +0800, Inochi Amaoto wrote: > >> +properties: > >> + compatible: > >> + oneOf: > >> + - items: > >> + - const: sophgo,sg2042-clint-mtimer > > > >There's only one of these, so you don't need the oneOf. >=20 > Thanks >=20 > >Also, is the clint here not a thead IP? In which case, you need to add a >=20 > Yes, The clint is a thead IP, like that of th1520 and allwinner D1. >=20 > >second compatible IMO. That second compatible then would be the one that > >appears in opensbi etc. > > >=20 > As this is a thead IP, maybe use thead,c900-clint-mtimer is fine? I would suggest calling it -aclint-mtimer instead of clint-mtimer. > If so, whether we should replace the "thead,c900-clint" with these separa= te > DT to describe the thead clint? No, since that's a different device, right? > The DT binding said the thead clint is not > compatible with the sifive clint, so maybe this is a chance to just move > them out. I don't think that it really makes sense to do that. Thanks, Conor. --UHaWB33MwrDFW0WP Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZRUcuQAKCRB4tDGHoIJi 0vArAP9fAULrrTApZmFmDl2++GE7+UOVwdl43Vjc4kOusjA+0AEA5UD0/+yLvol9 iUxnGTrAiIOBHo3ManpsssXg5sNniQ0= =VnCU -----END PGP SIGNATURE----- --UHaWB33MwrDFW0WP--