* [PATCH] media: qcom: iris: initialize managed OPP pointer
@ 2026-06-04 0:04 ` Hungyu Lin
2026-06-07 21:37 ` Dmitry Baryshkov
2026-06-08 10:13 ` Bryan O'Donoghue
0 siblings, 2 replies; 3+ messages in thread
From: Hungyu Lin @ 2026-06-04 0:04 UTC (permalink / raw)
To: vikash.garodia
Cc: dikshita.agarwal, abhinav.kumar, bod, mchehab, linux-media,
linux-arm-msm, linux-kernel, Hungyu Lin
Initialize the __free(put_opp) managed OPP pointer to NULL.
This ensures the cleanup handler always sees a defined value and
avoids leaving the managed pointer uninitialized before assignment.
Signed-off-by: Hungyu Lin <dennylin0707@gmail.com>
---
drivers/media/platform/qcom/iris/iris_resources.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/platform/qcom/iris/iris_resources.c b/drivers/media/platform/qcom/iris/iris_resources.c
index 773f6548370a..6b22f5cc88ca 100644
--- a/drivers/media/platform/qcom/iris/iris_resources.c
+++ b/drivers/media/platform/qcom/iris/iris_resources.c
@@ -61,7 +61,7 @@ int iris_unset_icc_bw(struct iris_core *core)
int iris_opp_set_rate(struct device *dev, unsigned long freq)
{
- struct dev_pm_opp *opp __free(put_opp);
+ struct dev_pm_opp *opp __free(put_opp) = NULL;
opp = devfreq_recommended_opp(dev, &freq, 0);
if (IS_ERR(opp))
--
2.34.1
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] media: qcom: iris: initialize managed OPP pointer
2026-06-04 0:04 ` [PATCH] media: qcom: iris: initialize managed OPP pointer Hungyu Lin
@ 2026-06-07 21:37 ` Dmitry Baryshkov
2026-06-08 10:13 ` Bryan O'Donoghue
1 sibling, 0 replies; 3+ messages in thread
From: Dmitry Baryshkov @ 2026-06-07 21:37 UTC (permalink / raw)
To: Hungyu Lin
Cc: vikash.garodia, dikshita.agarwal, abhinav.kumar, bod, mchehab,
linux-media, linux-arm-msm, linux-kernel
On Thu, Jun 04, 2026 at 12:04:14AM +0000, Hungyu Lin wrote:
> Initialize the __free(put_opp) managed OPP pointer to NULL.
>
> This ensures the cleanup handler always sees a defined value and
> avoids leaving the managed pointer uninitialized before assignment.
>
> Signed-off-by: Hungyu Lin <dennylin0707@gmail.com>
> ---
> drivers/media/platform/qcom/iris/iris_resources.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/media/platform/qcom/iris/iris_resources.c b/drivers/media/platform/qcom/iris/iris_resources.c
> index 773f6548370a..6b22f5cc88ca 100644
> --- a/drivers/media/platform/qcom/iris/iris_resources.c
> +++ b/drivers/media/platform/qcom/iris/iris_resources.c
> @@ -61,7 +61,7 @@ int iris_unset_icc_bw(struct iris_core *core)
>
> int iris_opp_set_rate(struct device *dev, unsigned long freq)
> {
> - struct dev_pm_opp *opp __free(put_opp);
> + struct dev_pm_opp *opp __free(put_opp) = NULL;
>
> opp = devfreq_recommended_opp(dev, &freq, 0);
No. Rather squash these two lines.
> if (IS_ERR(opp))
> --
> 2.34.1
>
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] media: qcom: iris: initialize managed OPP pointer
2026-06-04 0:04 ` [PATCH] media: qcom: iris: initialize managed OPP pointer Hungyu Lin
2026-06-07 21:37 ` Dmitry Baryshkov
@ 2026-06-08 10:13 ` Bryan O'Donoghue
1 sibling, 0 replies; 3+ messages in thread
From: Bryan O'Donoghue @ 2026-06-08 10:13 UTC (permalink / raw)
To: Hungyu Lin, vikash.garodia
Cc: dikshita.agarwal, abhinav.kumar, mchehab, linux-media,
linux-arm-msm, linux-kernel
On 04/06/2026 01:04, Hungyu Lin wrote:
> Initialize the __free(put_opp) managed OPP pointer to NULL.
>
> This ensures the cleanup handler always sees a defined value and
> avoids leaving the managed pointer uninitialized before assignment.
>
> Signed-off-by: Hungyu Lin <dennylin0707@gmail.com>
> ---
> drivers/media/platform/qcom/iris/iris_resources.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/media/platform/qcom/iris/iris_resources.c b/drivers/media/platform/qcom/iris/iris_resources.c
> index 773f6548370a..6b22f5cc88ca 100644
> --- a/drivers/media/platform/qcom/iris/iris_resources.c
> +++ b/drivers/media/platform/qcom/iris/iris_resources.c
> @@ -61,7 +61,7 @@ int iris_unset_icc_bw(struct iris_core *core)
>
> int iris_opp_set_rate(struct device *dev, unsigned long freq)
> {
> - struct dev_pm_opp *opp __free(put_opp);
> + struct dev_pm_opp *opp __free(put_opp) = NULL;
>
> opp = devfreq_recommended_opp(dev, &freq, 0);
> if (IS_ERR(opp))
> --
> 2.34.1
>
>
Requires a Fixes:
+ Cc: stable@vger.kernel.org
---
bod
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2026-06-08 10:13 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <adh8nAvf7xr--5KPtVFALkSnS3fjqr_6gPpV_hWfp_WYNh-qUCry-pXc6jgUGMkNzs-8EfSp2TQFsB2lWga0gg==@protonmail.internalid>
2026-06-04 0:04 ` [PATCH] media: qcom: iris: initialize managed OPP pointer Hungyu Lin
2026-06-07 21:37 ` Dmitry Baryshkov
2026-06-08 10:13 ` Bryan O'Donoghue
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®