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 5754BC43217 for ; Fri, 13 May 2022 08:17:12 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; 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=GyF5g4emN7+jp7WOOyPfNxMtawitGCoMfkiUUMkq/38=; b=rSFHoz0St198j3O/GaeT+UZPk5 gEHlQxnNzQe8LfTJkB4OaacvLXkKcdiEDFmTgnMTp6MHCMy1eTG1EIMgFBXPtjitG65WrEpWV6/iI ohBy/ArnmtraH8yt5s16TY7A3S5t0sbtnm9blljuVkHzokXcEAMCXwKnHWkTjyXWQFC2KAKxPHLzV Z3TPA7+ZaVZjkFwzZOyLW389dR55okCsMl2jca96ShU4JI9/eGHFCO1+n5AjBjQjLxH60hLt85hnq 5VbIDDEiM5xGVWN21q8fBZQ5qNdnLRNmz3KnYFdCJYzrwDrFtQ8truLdPiPDJBH2S+jRSQpTzba3c ytC4nFaA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1npQT1-00Ezre-Tp; Fri, 13 May 2022 08:16:08 +0000 Received: from out1-smtp.messagingengine.com ([66.111.4.25]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1npQSw-00Ezpe-7E for linux-arm-kernel@lists.infradead.org; Fri, 13 May 2022 08:16:03 +0000 Received: from compute2.internal (compute2.nyi.internal [10.202.2.46]) by mailout.nyi.internal (Postfix) with ESMTP id E421B5C00E4; Fri, 13 May 2022 04:15:58 -0400 (EDT) Received: from mailfrontend1 ([10.202.2.162]) by compute2.internal (MEProxy); Fri, 13 May 2022 04:15:58 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cerno.tech; h=cc :cc:content-type:date:date:from:from:in-reply-to:in-reply-to :message-id:mime-version:references:reply-to:sender:subject :subject:to:to; s=fm3; t=1652429758; x=1652516158; bh=iY9shFm2Us jfmpTTWu209xlXSoo1/kSVXTPhcXMt7H8=; b=uin/nZOFrMN1r3HNuWHmdoGaY3 c7FRaFuahQzKICW7+sXgp7uFRCf6nfBbRUC6qoTalXEwERQDkdkm7pagYWNswuw7 LNxbo+qZaPwNQWQBe4UD/pa0/5IoQLjJGGqHzdPFS//WrmNObBk5kg4UD8dlWr/s CUslx4dgQvclORi4IeQlr3Tm4oc0WaWGVdI2slV/+YF8x6V1eENUEKYA3bRz57tj TfcECEiFUusehtJwjbUxQEZ4QysRJ8QwJKFP7rykSE9QTWUHG3BQnSQp0xzthQHa HIPmKNxHqkJW23T41yni9Mj06EuRNv+7Z9Kp/JMulYcVeI3Hl95cp7JyPpGQ== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-type:date:date:from:from :in-reply-to:in-reply-to:message-id:mime-version:references :reply-to:sender:subject:subject:to:to:x-me-proxy:x-me-proxy :x-me-sender:x-me-sender:x-sasl-enc; s=fm1; t=1652429758; x= 1652516158; bh=iY9shFm2UsjfmpTTWu209xlXSoo1/kSVXTPhcXMt7H8=; b=O txrF9ODGUXG/fJtjc68Vnrim6/M1IT/RwWsCEySKTH8NljzFVFci1v7iph3xmYtx c7sKXNXwSd++avrklkfvgHcZNGjdwzsi6UcGVq/5CEfgB/2mY4mX9q/2FUv1HWU8 Pb+GixFh6C3nstMt8eaYFh3ffM3V2T4jLxtW35M8XlDPYU7qPsgYWl9RDaLIJROa CbYlWFlaw8/ZrrQla1EfOUZExY2A4KQrAqq3W4JsJWCDvYA84nBncpI4P8SG6J4D lHFE5GszS0h4kpy+BqfeY3yggQ3BSfZ+fVOTe+Z+WUjIckkNr+CRhkAj990Z6rVW VmsskET9xHuMYMrQYk3IQ== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvfedrgeelgddtudcutefuodetggdotefrodftvf curfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfghnecu uegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenuc fjughrpeffhffvvefukfhfgggtuggjsehgtderredttdejnecuhfhrohhmpeforgigihhm vgcutfhiphgrrhguuceomhgrgihimhgvsegtvghrnhhordhtvggthheqnecuggftrfgrth htvghrnhepjeegieevvedvgffffedtveeuhfehudevjeeifeegjeevuefhteeklefgveet feffnecuvehluhhsthgvrhfuihiivgeptdenucfrrghrrghmpehmrghilhhfrhhomhepmh grgihimhgvsegtvghrnhhordhtvggthh X-ME-Proxy: Received: by mail.messagingengine.com (Postfix) with ESMTPA; Fri, 13 May 2022 04:15:58 -0400 (EDT) Date: Fri, 13 May 2022 10:15:56 +0200 From: Maxime Ripard To: qianfan Cc: linux-sunxi@lists.linux.dev, Rob Herring , Chen-Yu Tsai , Jernej Skrabec , "Rafael J . Wysocki" , Viresh Kumar , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org Subject: Re: [PATCH v3 2/2] ARM: dts: sun8i-r40: Add "cpu-supply" node for sun8i-r40 based board Message-ID: <20220513081556.knd3tf6pangfblh3@houat> References: <20220512071858.10805-1-qianfanguijin@163.com> <20220512071858.10805-2-qianfanguijin@163.com> <20220513073849.cu4jzykefat2sepg@houat> <1352b9d7-5219-af09-948b-6462def39ea5@163.com> MIME-Version: 1.0 In-Reply-To: <1352b9d7-5219-af09-948b-6462def39ea5@163.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220513_011602_376342_1A9FF397 X-CRM114-Status: GOOD ( 33.51 ) 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="===============3483932192305277573==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============3483932192305277573== Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="ocbjtkvxhxhj7iii" Content-Disposition: inline --ocbjtkvxhxhj7iii Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Fri, May 13, 2022 at 03:48:50PM +0800, qianfan wrote: >=20 >=20 > =E5=9C=A8 2022/5/13 15:38, Maxime Ripard =E5=86=99=E9=81=93: > > Hi, > >=20 > > On Thu, May 12, 2022 at 03:18:58PM +0800, qianfanguijin@163.com wrote: > > > From: qianfan Zhao > > >=20 > > > sun8i-r40 actived cpufreq feature now, let's add "cpu-supply" node on > > > board. > > >=20 > > > Signed-off-by: qianfan Zhao > > > --- > > > arch/arm/boot/dts/sun8i-r40-bananapi-m2-ultra.dts | 4 ++++ > > > arch/arm/boot/dts/sun8i-r40-feta40i.dtsi | 4 ++++ > > > arch/arm/boot/dts/sun8i-t3-cqa3t-bv3.dts | 4 ++++ > > > arch/arm/boot/dts/sun8i-v40-bananapi-m2-berry.dts | 4 ++++ > > > 4 files changed, 16 insertions(+) > > >=20 > > > diff --git a/arch/arm/boot/dts/sun8i-r40-bananapi-m2-ultra.dts b/arch= /arm/boot/dts/sun8i-r40-bananapi-m2-ultra.dts > > > index a6a1087a0c9b..4f30018ec4a2 100644 > > > --- a/arch/arm/boot/dts/sun8i-r40-bananapi-m2-ultra.dts > > > +++ b/arch/arm/boot/dts/sun8i-r40-bananapi-m2-ultra.dts > > > @@ -113,6 +113,10 @@ &ahci { > > > status =3D "okay"; > > > }; > > > +&cpu0 { > > > + cpu-supply =3D <®_dcdc2>; > > > +}; > > > + > > This will break bisection on those boards. Indeed, you added the OPPs on > > the first patch, and if you only apply that patch, the boards in the > > second patch will be missing their CPU regulator. The kernel will then > > ramp up the frequency to the highest OPP, but will not change the > > voltage, resulting in a crash. > > This is a good point and I will merge those two patch. That's not what I meant to say. Those two patches are great separated. You can invert them though. > > There's a similar issue for all the boards that don't have a regulator > > in the first place. > >=20 > > The way we worked around this for the other SoCs is to have a DTSI with > > the OPPs with a frequency higher than what U-Boot boots with (1008MHz?), > > and only include that DTSI on boards that have a CPU regulator hooked i= n. > > Is this really necessary? It seems like every board based on sun8i-r40 > have a cpu regulator. This probably won't be the case whenever someone starts a new design, and then they'll face random crashes for no apparent reason, and waste a lot of time in the process. Whereas the alternative is that you would be missing some OPPs, something that is fairly easy to figure out. Maxime --ocbjtkvxhxhj7iii Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYKAB0WIQRcEzekXsqa64kGDp7j7w1vZxhRxQUCYn4TvAAKCRDj7w1vZxhR xXVzAQCYS/y6GVik1wq99/10xFtsQVwNbQDTafNgKQRW8bpSxwD+Ohmf/aYmi5JZ QfgJA4DF5q5bvX0nina8Y7KiVSMVSQ0= =x+9H -----END PGP SIGNATURE----- --ocbjtkvxhxhj7iii-- --===============3483932192305277573== 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 --===============3483932192305277573==--