Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Feedback-ID: i56a14606:Fastmail
User-Agent: Cyrus-JMAP/3.9.0-alpha0-334-g8c072af647-fm-20230330.001-g8c072af6
Mime-Version: 1.0
Message-Id: <9213a846-9fa0-4a71-97e6-29da5be54103@app.fastmail.com>
In-Reply-To: <ZDkyc0cVcgmbnrDt@FVFF77S0Q05N>
References: <20230414084619.31524-1-johan+linaro@kernel.org>
 <ZDkyc0cVcgmbnrDt@FVFF77S0Q05N>
Date:   Fri, 14 Apr 2023 13:49:38 +0200
From:   "Arnd Bergmann" <arnd@arndb.de>
To:     "Mark Rutland" <mark.rutland@arm.com>,
        "Johan Hovold" <johan+linaro@kernel.org>
Cc:     "Lorenzo Pieralisi" <lpieralisi@kernel.org>,
        "Ulf Hansson" <ulf.hansson@linaro.org>,
        "Dmitry Baryshkov" <dmitry.baryshkov@linaro.org>,
        "Sudeep Holla" <sudeep.holla@arm.com>,
        "Daniel Lezcano" <daniel.lezcano@linaro.org>,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH RESEND v2] firmware/psci: demote suspend-mode warning to info level
Content-Type: text/plain
Precedence: bulk

On Fri, Apr 14, 2023, at 13:01, Mark Rutland wrote:
> On Fri, Apr 14, 2023 at 10:46:19AM +0200, Johan Hovold wrote:
>> On some Qualcomm platforms, like SC8280XP, the attempt to set PC mode
>> during boot fails with PSCI_RET_DENIED and since commit 998fcd001feb
>> ("firmware/psci: Print a warning if PSCI doesn't accept PC mode") this
>> is now logged at warning level:
>> 
>> 	psci: failed to set PC mode: -3
>> 
>> As there is nothing users can do about the firmware behaving this way,
>> demote the warning to info level and clearly mark it as a firmware bug:
>> 
>> 	psci: [Firmware Bug]: failed to set PC mode: -3
>> 
>> Reviewed-by: Ulf Hansson <ulf.hansson@linaro.org>
>> Acked-by: Mark Rutland <mark.rutland@arm.com>
>> Acked-by: Sudeep Holla <sudeep.holla@arm.com>
>> Signed-off-by: Johan Hovold <johan+linaro@kernel.org>
>> ---
>> 
>> This one fell between the cracks so resending with tags added.
>> 
>> Mark or Lorenzo, is this something you can pick up?
>> 
>> Johan
>
> Sorry about this.
>
> I don't think we have anything else in this area, so Arnd, are you happy to
> pick this via the soc tree?

Sure, applied to the fixes branch for 6.3 now.

     Arnd