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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 5BC39C7EE43 for ; Tue, 6 Jun 2023 23:34:33 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S239774AbjFFXec (ORCPT ); Tue, 6 Jun 2023 19:34:32 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:58970 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234032AbjFFXe3 (ORCPT ); Tue, 6 Jun 2023 19:34:29 -0400 Received: from mail-qt1-x82c.google.com (mail-qt1-x82c.google.com [IPv6:2607:f8b0:4864:20::82c]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id A94E4170B for ; Tue, 6 Jun 2023 16:34:28 -0700 (PDT) Received: by mail-qt1-x82c.google.com with SMTP id d75a77b69052e-3f9aeba6cc4so13070421cf.2 for ; Tue, 06 Jun 2023 16:34:28 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; t=1686094466; x=1688686466; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=fGFunOEezrTgCZyEfU8nDw4x9+xyVAO+9lV0ZlJ/tsM=; b=laZFZ4ATTRuY1Sb0zf69tfd/kW0R3ojrHzbDo/x9VN7ppMXSDPmUUVbMPmqZwQj9bd XZO22K0NJnvi4ZVe2bYaFDtXG4zYSfoARVtAkWS3zIaUS6VGC+aq2h+nZrH65knE4GI+ VJlMxBCMQCOaDS52XgG1kUpixuWd34xzLWv3g= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1686094466; x=1688686466; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=fGFunOEezrTgCZyEfU8nDw4x9+xyVAO+9lV0ZlJ/tsM=; b=j5aTLSq2fTytNHdijCqva+B32RmNveHTYkc6/VoxKy0n28yPX8UVF0jS0UO0kc0RPN WkWi8rmSATGaeHj6BgOSQopVsNCrz1F31Es59Oiei2+R0KdMBdA7y7cegaiWVIMDnhPU PQkG73mu2yOdnyBBpzrYvA40WSoH8n51dG5iyPkw8vZLaTyuqEYIGgPgJ4AS7PxJA33j +ZpgOxX3sG+/hjO2p8NNoH6esgcktpFcFHMpLOE4Lt5lOLSS6hyDp6CquhB/zeuGQm/p Qg9lnm7/5cR9kO3BScZjEDRI6YhztF+WhE4PM7H01xphF85B9RImaMC8zCndzUTmZv6s Bi+w== X-Gm-Message-State: AC+VfDzQ6qqx2YMMpqVDyoYnwaO9FrLHpUx/BN1nyFnONUauy0Ow3PHs HL9yFUbyGXOSwKL7t7BXhcLaRCkyAacGqtPTA70= X-Google-Smtp-Source: ACHHUZ4sBm8QbK3mdKu4RCYrjPeJcBLNNurBwHQ1DpUiVX78eLAadz0OR783tNzs6IGOAma6s4DSdw== X-Received: by 2002:a05:6214:2583:b0:5f1:31eb:1eff with SMTP id fq3-20020a056214258300b005f131eb1effmr1936549qvb.2.1686094466060; Tue, 06 Jun 2023 16:34:26 -0700 (PDT) Received: from mail-qt1-f173.google.com (mail-qt1-f173.google.com. [209.85.160.173]) by smtp.gmail.com with ESMTPSA id nd8-20020a056214420800b006260e4b6de9sm5705102qvb.118.2023.06.06.16.34.24 for (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 06 Jun 2023 16:34:25 -0700 (PDT) Received: by mail-qt1-f173.google.com with SMTP id d75a77b69052e-3f9a81da5d7so93321cf.0 for ; Tue, 06 Jun 2023 16:34:24 -0700 (PDT) X-Received: by 2002:a05:622a:547:b0:3f8:1a01:e933 with SMTP id m7-20020a05622a054700b003f81a01e933mr67672qtx.24.1686094464496; Tue, 06 Jun 2023 16:34:24 -0700 (PDT) MIME-Version: 1.0 References: <20230602161246.1855448-1-amit.pundir@linaro.org> In-Reply-To: <20230602161246.1855448-1-amit.pundir@linaro.org> From: Doug Anderson Date: Tue, 6 Jun 2023 16:34:12 -0700 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH] arm64: dts: qcom: sdm845-db845c: Move LVS regulator nodes up To: Amit Pundir Cc: Mark Brown , Bjorn Andersson , Andy Gross , Rob Herring , Konrad Dybcio , Krzysztof Kozlowski , Caleb Connolly , Conor Dooley , regressions , linux-arm-msm , dt , lkml Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, On Fri, Jun 2, 2023 at 9:12=E2=80=AFAM Amit Pundir = wrote: > > Move lvs1 and lvs2 regulator nodes up in the rpmh-regulators > list to workaround a boot regression uncovered by the upstream > commit ad44ac082fdf ("regulator: qcom-rpmh: Revert "regulator: > qcom-rpmh: Use PROBE_FORCE_SYNCHRONOUS""). > > Without this fix DB845c fail to boot at times because one of the > lvs1 or lvs2 regulators fail to turn ON in time. > > Link: https://lore.kernel.org/all/CAMi1Hd1avQDcDQf137m2auz2znov4XL8YGrLZs= w5edb-NtRJRw@mail.gmail.com/ > Signed-off-by: Amit Pundir > --- > arch/arm64/boot/dts/qcom/sdm845-db845c.dts | 24 +++++++++++----------- > 1 file changed, 12 insertions(+), 12 deletions(-) > > diff --git a/arch/arm64/boot/dts/qcom/sdm845-db845c.dts b/arch/arm64/boot= /dts/qcom/sdm845-db845c.dts > index e14fe9bbb386..df2fde9063dc 100644 > --- a/arch/arm64/boot/dts/qcom/sdm845-db845c.dts > +++ b/arch/arm64/boot/dts/qcom/sdm845-db845c.dts > @@ -301,6 +301,18 @@ regulators-0 { > vdd-l26-supply =3D <&vreg_s3a_1p35>; > vin-lvs-1-2-supply =3D <&vreg_s4a_1p8>; > > + vreg_lvs1a_1p8: lvs1 { > + regulator-min-microvolt =3D <1800000>; > + regulator-max-microvolt =3D <1800000>; > + regulator-always-on; > + }; > + > + vreg_lvs2a_1p8: lvs2 { > + regulator-min-microvolt =3D <1800000>; > + regulator-max-microvolt =3D <1800000>; > + regulator-always-on; > + }; > + > vreg_s3a_1p35: smps3 { > regulator-min-microvolt =3D <1352000>; > regulator-max-microvolt =3D <1352000>; > @@ -381,18 +393,6 @@ vreg_l26a_1p2: ldo26 { > regulator-max-microvolt =3D <1200000>; > regulator-initial-mode =3D ; > }; > - > - vreg_lvs1a_1p8: lvs1 { > - regulator-min-microvolt =3D <1800000>; > - regulator-max-microvolt =3D <1800000>; > - regulator-always-on; > - }; > - > - vreg_lvs2a_1p8: lvs2 { > - regulator-min-microvolt =3D <1800000>; > - regulator-max-microvolt =3D <1800000>; > - regulator-always-on; > - }; This is a hack, but it at least feels less bad than reverting the async probe patch. I'll leave it to Bjorn to decide if he's OK with it. Personally, it feels like this would deserve a comment in the dts to document that these regulators need to be listed first. Ideally, we could still work towards a root cause. I added a few more ideas to help with root causing in reply to the original thread about this. https://lore.kernel.org/r/CAD=3DFV=3DUKyjRNZG-ED2meUAR9aXdco+AbUTHiKixTzjCk= aJbjTg@mail.gmail.com/ -Doug