[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <3bfd6343-de4a-4cf5-a4b8-55a3531eb41e@linaro.org>
Date: Tue, 19 Nov 2024 20:37:15 +0100
From: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
To: Bjorn Andersson <andersson@...nel.org>,
Konrad Dybcio <konradybcio@...nel.org>, Mukesh Ojha
<quic_mojha@...cinc.com>, Dmitry Baryshkov <dmitry.baryshkov@...aro.org>,
Stephan Gerhold <stephan.gerhold@...aro.org>,
Bartosz Golaszewski <bartosz.golaszewski@...aro.org>,
Kuldeep Singh <quic_kuldsing@...cinc.com>,
Elliot Berman <quic_eberman@...cinc.com>,
Andrew Halaney <ahalaney@...hat.com>,
Avaneesh Kumar Dwivedi <quic_akdwived@...cinc.com>,
Andy Gross <andy.gross@...aro.org>
Cc: linux-arm-msm@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH RFC/RFT 4/6] firmware: qcom: scm: Cleanup global '__scm'
on probe failures
On 19/11/2024 19:33, Krzysztof Kozlowski wrote:
> /*
> * Initialize the QSEECOM interface.
> @@ -2094,6 +2104,12 @@ static int qcom_scm_probe(struct platform_device *pdev)
> WARN(ret < 0, "failed to initialize qseecom: %d\n", ret);
>
> return 0;
> +
> +err:
> + /* Paired with smp_load_acquire() in qcom_scm_is_available(). */
> + smp_store_release(&__scm, 0);
Heh, I should store there NULL, obviously.
Best regards,
Krzysztof
Powered by blists - more mailing lists