* [PATCH v2 0/2] remoteproc: qcom: Fix handling of WCNSS IRQs
@ 2022-05-26 14:17 Sireesh Kodali
2022-05-26 14:17 ` [PATCH v2 1/2] remoteproc: qcom: wcnss: Fix handling of IRQs Sireesh Kodali
` (2 more replies)
0 siblings, 3 replies; 7+ messages in thread
From: Sireesh Kodali @ 2022-05-26 14:17 UTC (permalink / raw)
To: devicetree, linux-arm-msm, linux-remoteproc
Cc: bjorn.andersson, Sireesh Kodali
Hi,
Due to a bug in the code that fetches the IRQs from the device tree, the
optional "stop-ack" IRQ was never used, even if it was defined in the
device tree. This also allowed a typo in the MSM8916 SoC device tree,
where the "stop-ack" irq was instead labeled "stop".
The attached patches fix the handling of IRQs in the code, and fix the
typo in the MSM8916 device tree.
Changes since v1:
* Dropped blank lines between tags in commit message
Sireesh Kodali (2):
remoteproc: qcom: wcnss: Fix handling of IRQs
arm64: dts: qcom: msm8916: Fix typo in pronto remoteproc node
arch/arm64/boot/dts/qcom/msm8916.dtsi | 4 ++--
drivers/remoteproc/qcom_wcnss.c | 10 ++++++++--
2 files changed, 10 insertions(+), 4 deletions(-)
--
2.36.1
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v2 1/2] remoteproc: qcom: wcnss: Fix handling of IRQs
2022-05-26 14:17 [PATCH v2 0/2] remoteproc: qcom: Fix handling of WCNSS IRQs Sireesh Kodali
@ 2022-05-26 14:17 ` Sireesh Kodali
2022-05-26 14:17 ` [PATCH v2 2/2] arm64: dts: qcom: msm8916: Fix typo in pronto remoteproc node Sireesh Kodali
2022-07-03 3:56 ` [PATCH v2 0/2] remoteproc: qcom: Fix handling of WCNSS IRQs Bjorn Andersson
2 siblings, 0 replies; 7+ messages in thread
From: Sireesh Kodali @ 2022-05-26 14:17 UTC (permalink / raw)
To: devicetree, linux-arm-msm, linux-remoteproc
Cc: bjorn.andersson, Sireesh Kodali, Andy Gross, Mathieu Poirier, open list
The wcnss_get_irq function is expected to return a value > 0 in the
event that an IRQ is succssfully obtained, but it instead returns 0.
This causes the stop and ready IRQs to never actually be used despite
being defined in the device-tree. This patch fixes that.
Fixes: aed361adca9f ("remoteproc: qcom: Introduce WCNSS peripheral image loader")
Signed-off-by: Sireesh Kodali <sireeshkodali1@gmail.com>
---
drivers/remoteproc/qcom_wcnss.c | 10 ++++++++--
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/drivers/remoteproc/qcom_wcnss.c b/drivers/remoteproc/qcom_wcnss.c
index 9a223d394087..68f37296b151 100644
--- a/drivers/remoteproc/qcom_wcnss.c
+++ b/drivers/remoteproc/qcom_wcnss.c
@@ -467,6 +467,7 @@ static int wcnss_request_irq(struct qcom_wcnss *wcnss,
irq_handler_t thread_fn)
{
int ret;
+ int irq_number;
ret = platform_get_irq_byname(pdev, name);
if (ret < 0 && optional) {
@@ -477,14 +478,19 @@ static int wcnss_request_irq(struct qcom_wcnss *wcnss,
return ret;
}
+ irq_number = ret;
+
ret = devm_request_threaded_irq(&pdev->dev, ret,
NULL, thread_fn,
IRQF_TRIGGER_RISING | IRQF_ONESHOT,
"wcnss", wcnss);
- if (ret)
+ if (ret) {
dev_err(&pdev->dev, "request %s IRQ failed\n", name);
+ return ret;
+ }
- return ret;
+ /* Return the IRQ number if the IRQ was successfully acquired */
+ return irq_number;
}
static int wcnss_alloc_memory_region(struct qcom_wcnss *wcnss)
--
2.36.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH v2 2/2] arm64: dts: qcom: msm8916: Fix typo in pronto remoteproc node
2022-05-26 14:17 [PATCH v2 0/2] remoteproc: qcom: Fix handling of WCNSS IRQs Sireesh Kodali
2022-05-26 14:17 ` [PATCH v2 1/2] remoteproc: qcom: wcnss: Fix handling of IRQs Sireesh Kodali
@ 2022-05-26 14:17 ` Sireesh Kodali
2022-05-26 18:34 ` Stephan Gerhold
` (2 more replies)
2022-07-03 3:56 ` [PATCH v2 0/2] remoteproc: qcom: Fix handling of WCNSS IRQs Bjorn Andersson
2 siblings, 3 replies; 7+ messages in thread
From: Sireesh Kodali @ 2022-05-26 14:17 UTC (permalink / raw)
To: devicetree, linux-arm-msm, linux-remoteproc
Cc: bjorn.andersson, Sireesh Kodali, Andy Gross, Rob Herring,
Krzysztof Kozlowski, open list
The smem-state properties for the pronto node were incorrectly labelled,
reading `qcom,state*` rather than `qcom,smem-state*`. Fix that, allowing
the stop state to be used.
Fixes: 88106096cbf8 ("ARM: dts: msm8916: Add and enable wcnss node")
Signed-off-by: Sireesh Kodali <sireeshkodali1@gmail.com>
---
arch/arm64/boot/dts/qcom/msm8916.dtsi | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/msm8916.dtsi b/arch/arm64/boot/dts/qcom/msm8916.dtsi
index e34963505e07..7ecd747dc624 100644
--- a/arch/arm64/boot/dts/qcom/msm8916.dtsi
+++ b/arch/arm64/boot/dts/qcom/msm8916.dtsi
@@ -1758,8 +1758,8 @@ pronto: remoteproc@a21b000 {
<&rpmpd MSM8916_VDDMX>;
power-domain-names = "cx", "mx";
- qcom,state = <&wcnss_smp2p_out 0>;
- qcom,state-names = "stop";
+ qcom,smem-states = <&wcnss_smp2p_out 0>;
+ qcom,smem-state-names = "stop";
pinctrl-names = "default";
pinctrl-0 = <&wcnss_pin_a>;
--
2.36.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH v2 2/2] arm64: dts: qcom: msm8916: Fix typo in pronto remoteproc node
2022-05-26 14:17 ` [PATCH v2 2/2] arm64: dts: qcom: msm8916: Fix typo in pronto remoteproc node Sireesh Kodali
@ 2022-05-26 18:34 ` Stephan Gerhold
2022-05-26 19:40 ` Krzysztof Kozlowski
2022-07-03 3:56 ` (subset) " Bjorn Andersson
2 siblings, 0 replies; 7+ messages in thread
From: Stephan Gerhold @ 2022-05-26 18:34 UTC (permalink / raw)
To: Sireesh Kodali
Cc: devicetree, linux-arm-msm, linux-remoteproc, bjorn.andersson,
Andy Gross, Rob Herring, Krzysztof Kozlowski, open list
On Thu, May 26, 2022 at 07:47:40PM +0530, Sireesh Kodali wrote:
> The smem-state properties for the pronto node were incorrectly labelled,
> reading `qcom,state*` rather than `qcom,smem-state*`. Fix that, allowing
> the stop state to be used.
>
> Fixes: 88106096cbf8 ("ARM: dts: msm8916: Add and enable wcnss node")
> Signed-off-by: Sireesh Kodali <sireeshkodali1@gmail.com>
My review didn't change just because of an unneeded blank line between
the tags. Thanks again for finding this. :)
Reviewed-by: Stephan Gerhold <stephan@gerhold.net>
> ---
> arch/arm64/boot/dts/qcom/msm8916.dtsi | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/msm8916.dtsi b/arch/arm64/boot/dts/qcom/msm8916.dtsi
> index e34963505e07..7ecd747dc624 100644
> --- a/arch/arm64/boot/dts/qcom/msm8916.dtsi
> +++ b/arch/arm64/boot/dts/qcom/msm8916.dtsi
> @@ -1758,8 +1758,8 @@ pronto: remoteproc@a21b000 {
> <&rpmpd MSM8916_VDDMX>;
> power-domain-names = "cx", "mx";
>
> - qcom,state = <&wcnss_smp2p_out 0>;
> - qcom,state-names = "stop";
> + qcom,smem-states = <&wcnss_smp2p_out 0>;
> + qcom,smem-state-names = "stop";
>
> pinctrl-names = "default";
> pinctrl-0 = <&wcnss_pin_a>;
> --
> 2.36.1
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2 2/2] arm64: dts: qcom: msm8916: Fix typo in pronto remoteproc node
2022-05-26 14:17 ` [PATCH v2 2/2] arm64: dts: qcom: msm8916: Fix typo in pronto remoteproc node Sireesh Kodali
2022-05-26 18:34 ` Stephan Gerhold
@ 2022-05-26 19:40 ` Krzysztof Kozlowski
2022-07-03 3:56 ` (subset) " Bjorn Andersson
2 siblings, 0 replies; 7+ messages in thread
From: Krzysztof Kozlowski @ 2022-05-26 19:40 UTC (permalink / raw)
To: Sireesh Kodali, devicetree, linux-arm-msm, linux-remoteproc
Cc: bjorn.andersson, Andy Gross, Rob Herring, Krzysztof Kozlowski, open list
On 26/05/2022 16:17, Sireesh Kodali wrote:
> The smem-state properties for the pronto node were incorrectly labelled,
> reading `qcom,state*` rather than `qcom,smem-state*`. Fix that, allowing
> the stop state to be used.
>
> Fixes: 88106096cbf8 ("ARM: dts: msm8916: Add and enable wcnss node")
> Signed-off-by: Sireesh Kodali <sireeshkodali1@gmail.com>
Include please acquired tags when sending a new version.
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2 0/2] remoteproc: qcom: Fix handling of WCNSS IRQs
2022-05-26 14:17 [PATCH v2 0/2] remoteproc: qcom: Fix handling of WCNSS IRQs Sireesh Kodali
2022-05-26 14:17 ` [PATCH v2 1/2] remoteproc: qcom: wcnss: Fix handling of IRQs Sireesh Kodali
2022-05-26 14:17 ` [PATCH v2 2/2] arm64: dts: qcom: msm8916: Fix typo in pronto remoteproc node Sireesh Kodali
@ 2022-07-03 3:56 ` Bjorn Andersson
2 siblings, 0 replies; 7+ messages in thread
From: Bjorn Andersson @ 2022-07-03 3:56 UTC (permalink / raw)
To: devicetree, linux-remoteproc, Sireesh Kodali, linux-arm-msm
On Thu, 26 May 2022 19:47:38 +0530, Sireesh Kodali wrote:
> Due to a bug in the code that fetches the IRQs from the device tree, the
> optional "stop-ack" IRQ was never used, even if it was defined in the
> device tree. This also allowed a typo in the MSM8916 SoC device tree,
> where the "stop-ack" irq was instead labeled "stop".
>
> The attached patches fix the handling of IRQs in the code, and fix the
> typo in the MSM8916 device tree.
>
> [...]
Applied, thanks!
[1/2] remoteproc: qcom: wcnss: Fix handling of IRQs
(no commit info)
[2/2] arm64: dts: qcom: msm8916: Fix typo in pronto remoteproc node
commit: 5458d6f2827cd30218570f266b8d238417461f2f
Best regards,
--
Bjorn Andersson <bjorn.andersson@linaro.org>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: (subset) [PATCH v2 2/2] arm64: dts: qcom: msm8916: Fix typo in pronto remoteproc node
2022-05-26 14:17 ` [PATCH v2 2/2] arm64: dts: qcom: msm8916: Fix typo in pronto remoteproc node Sireesh Kodali
2022-05-26 18:34 ` Stephan Gerhold
2022-05-26 19:40 ` Krzysztof Kozlowski
@ 2022-07-03 3:56 ` Bjorn Andersson
2 siblings, 0 replies; 7+ messages in thread
From: Bjorn Andersson @ 2022-07-03 3:56 UTC (permalink / raw)
To: devicetree, linux-remoteproc, Sireesh Kodali, linux-arm-msm
Cc: Andy Gross, Rob Herring, Krzysztof Kozlowski, open list
On Thu, 26 May 2022 19:47:40 +0530, Sireesh Kodali wrote:
> The smem-state properties for the pronto node were incorrectly labelled,
> reading `qcom,state*` rather than `qcom,smem-state*`. Fix that, allowing
> the stop state to be used.
>
>
Applied, thanks!
[2/2] arm64: dts: qcom: msm8916: Fix typo in pronto remoteproc node
commit: 5458d6f2827cd30218570f266b8d238417461f2f
Best regards,
--
Bjorn Andersson <bjorn.andersson@linaro.org>
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2022-07-03 4:00 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-05-26 14:17 [PATCH v2 0/2] remoteproc: qcom: Fix handling of WCNSS IRQs Sireesh Kodali
2022-05-26 14:17 ` [PATCH v2 1/2] remoteproc: qcom: wcnss: Fix handling of IRQs Sireesh Kodali
2022-05-26 14:17 ` [PATCH v2 2/2] arm64: dts: qcom: msm8916: Fix typo in pronto remoteproc node Sireesh Kodali
2022-05-26 18:34 ` Stephan Gerhold
2022-05-26 19:40 ` Krzysztof Kozlowski
2022-07-03 3:56 ` (subset) " Bjorn Andersson
2022-07-03 3:56 ` [PATCH v2 0/2] remoteproc: qcom: Fix handling of WCNSS IRQs Bjorn Andersson
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.