* [PATCH v2] drm/msm/dpu: clean up dpu_kms_get_clk_rate() returns
@ 2023-05-26 11:51 Dan Carpenter
2023-06-01 16:49 ` Abhinav Kumar
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Dan Carpenter @ 2023-05-26 11:51 UTC (permalink / raw)
To: Rob Clark
Cc: Abhinav Kumar, Dmitry Baryshkov, Sean Paul, David Airlie,
Daniel Vetter, Marijn Suijten, Konrad Dybcio, Jessica Zhang,
Liu Shixin, Douglas Anderson, linux-arm-msm, dri-devel,
freedreno, kernel-janitors
Static analysis tools complain about the -EINVAL error code being
stored in an unsigned variable. Let's change this to match
the clk_get_rate() function which is type unsigned long and returns
zero on error.
Fixes: 25fdd5933e4c ("drm/msm: Add SDM845 DPU support")
Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
---
v2: In v1 I change the type to int which was wrong. This is a different
approach. CC the freedreno list this time too.
drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 4 ++--
drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
index 0e7a68714e9e..25e6a15eaf9f 100644
--- a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
+++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
@@ -979,13 +979,13 @@ static int _dpu_kms_mmu_init(struct dpu_kms *dpu_kms)
return 0;
}
-u64 dpu_kms_get_clk_rate(struct dpu_kms *dpu_kms, char *clock_name)
+unsigned long dpu_kms_get_clk_rate(struct dpu_kms *dpu_kms, char *clock_name)
{
struct clk *clk;
clk = msm_clk_bulk_get_clock(dpu_kms->clocks, dpu_kms->num_clocks, clock_name);
if (!clk)
- return -EINVAL;
+ return 0;
return clk_get_rate(clk);
}
diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h
index aca39a4689f4..961918e5a5b3 100644
--- a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h
+++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h
@@ -201,6 +201,6 @@ void dpu_disable_vblank(struct msm_kms *kms, struct drm_crtc *crtc);
*
* Return: current clock rate
*/
-u64 dpu_kms_get_clk_rate(struct dpu_kms *dpu_kms, char *clock_name);
+unsigned long dpu_kms_get_clk_rate(struct dpu_kms *dpu_kms, char *clock_name);
#endif /* __dpu_kms_H__ */
--
2.39.2
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v2] drm/msm/dpu: clean up dpu_kms_get_clk_rate() returns
2023-05-26 11:51 [PATCH v2] drm/msm/dpu: clean up dpu_kms_get_clk_rate() returns Dan Carpenter
@ 2023-06-01 16:49 ` Abhinav Kumar
2023-06-04 3:01 ` Dmitry Baryshkov
2023-06-04 22:00 ` Marijn Suijten
2 siblings, 0 replies; 4+ messages in thread
From: Abhinav Kumar @ 2023-06-01 16:49 UTC (permalink / raw)
To: Dan Carpenter, Rob Clark
Cc: Dmitry Baryshkov, Sean Paul, David Airlie, Daniel Vetter,
Marijn Suijten, Konrad Dybcio, Jessica Zhang, Liu Shixin,
Douglas Anderson, linux-arm-msm, dri-devel, freedreno,
kernel-janitors
On 5/26/2023 4:51 AM, Dan Carpenter wrote:
> Static analysis tools complain about the -EINVAL error code being
> stored in an unsigned variable. Let's change this to match
> the clk_get_rate() function which is type unsigned long and returns
> zero on error.
>
> Fixes: 25fdd5933e4c ("drm/msm: Add SDM845 DPU support")
> Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
> ---
> v2: In v1 I change the type to int which was wrong. This is a different
> approach. CC the freedreno list this time too.
>
> drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 4 ++--
> drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h | 2 +-
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
> index 0e7a68714e9e..25e6a15eaf9f 100644
> --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
> @@ -979,13 +979,13 @@ static int _dpu_kms_mmu_init(struct dpu_kms *dpu_kms)
> return 0;
> }
>
> -u64 dpu_kms_get_clk_rate(struct dpu_kms *dpu_kms, char *clock_name)
> +unsigned long dpu_kms_get_clk_rate(struct dpu_kms *dpu_kms, char *clock_name)
> {
> struct clk *clk;
>
> clk = msm_clk_bulk_get_clock(dpu_kms->clocks, dpu_kms->num_clocks, clock_name);
> if (!clk)
> - return -EINVAL;
> + return 0;
>
Currently, the only caller of this API is
dpu_encoder_phys_cmd_tearcheck_config which seems to handle <=0 so this
change should be fine. Hence
Reviewed-by: Abhinav Kumar <quic_abhinavk@quicinc.com>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] drm/msm/dpu: clean up dpu_kms_get_clk_rate() returns
2023-05-26 11:51 [PATCH v2] drm/msm/dpu: clean up dpu_kms_get_clk_rate() returns Dan Carpenter
2023-06-01 16:49 ` Abhinav Kumar
@ 2023-06-04 3:01 ` Dmitry Baryshkov
2023-06-04 22:00 ` Marijn Suijten
2 siblings, 0 replies; 4+ messages in thread
From: Dmitry Baryshkov @ 2023-06-04 3:01 UTC (permalink / raw)
To: Rob Clark, Dan Carpenter
Cc: Abhinav Kumar, Sean Paul, David Airlie, Daniel Vetter,
Marijn Suijten, Konrad Dybcio, Jessica Zhang, Liu Shixin,
Douglas Anderson, linux-arm-msm, dri-devel, freedreno,
kernel-janitors
On Fri, 26 May 2023 14:51:59 +0300, Dan Carpenter wrote:
> Static analysis tools complain about the -EINVAL error code being
> stored in an unsigned variable. Let's change this to match
> the clk_get_rate() function which is type unsigned long and returns
> zero on error.
>
>
Applied, thanks!
[1/1] drm/msm/dpu: clean up dpu_kms_get_clk_rate() returns
https://gitlab.freedesktop.org/lumag/msm/-/commit/025e3d97867c
Best regards,
--
Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] drm/msm/dpu: clean up dpu_kms_get_clk_rate() returns
2023-05-26 11:51 [PATCH v2] drm/msm/dpu: clean up dpu_kms_get_clk_rate() returns Dan Carpenter
2023-06-01 16:49 ` Abhinav Kumar
2023-06-04 3:01 ` Dmitry Baryshkov
@ 2023-06-04 22:00 ` Marijn Suijten
2 siblings, 0 replies; 4+ messages in thread
From: Marijn Suijten @ 2023-06-04 22:00 UTC (permalink / raw)
To: Dan Carpenter
Cc: Rob Clark, Abhinav Kumar, Dmitry Baryshkov, Sean Paul,
David Airlie, Daniel Vetter, Konrad Dybcio, Jessica Zhang,
Liu Shixin, Douglas Anderson, linux-arm-msm, dri-devel,
freedreno, kernel-janitors
On 2023-05-26 14:51:59, Dan Carpenter wrote:
> Static analysis tools complain about the -EINVAL error code being
> stored in an unsigned variable. Let's change this to match
> the clk_get_rate() function which is type unsigned long and returns
> zero on error.
>
> Fixes: 25fdd5933e4c ("drm/msm: Add SDM845 DPU support")
> Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
> ---
> v2: In v1 I change the type to int which was wrong. This is a different
> approach. CC the freedreno list this time too.
Just like v1, should we clean up the <= in
dpu_encoder_phys_cmd_tearcheck_config (to just `if (!vsync_hz)`),
because doing a <=0 on a u32 has a more limited meaning?
>
> drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 4 ++--
> drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h | 2 +-
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
> index 0e7a68714e9e..25e6a15eaf9f 100644
> --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
> @@ -979,13 +979,13 @@ static int _dpu_kms_mmu_init(struct dpu_kms *dpu_kms)
> return 0;
> }
>
> -u64 dpu_kms_get_clk_rate(struct dpu_kms *dpu_kms, char *clock_name)
> +unsigned long dpu_kms_get_clk_rate(struct dpu_kms *dpu_kms, char *clock_name)
> {
> struct clk *clk;
>
> clk = msm_clk_bulk_get_clock(dpu_kms->clocks, dpu_kms->num_clocks, clock_name);
> if (!clk)
> - return -EINVAL;
> + return 0;
>
> return clk_get_rate(clk);
> }
> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h
> index aca39a4689f4..961918e5a5b3 100644
> --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h
> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h
> @@ -201,6 +201,6 @@ void dpu_disable_vblank(struct msm_kms *kms, struct drm_crtc *crtc);
> *
> * Return: current clock rate
Or 0 if this clock is not available/present.
- Marijn
> */
> -u64 dpu_kms_get_clk_rate(struct dpu_kms *dpu_kms, char *clock_name);
> +unsigned long dpu_kms_get_clk_rate(struct dpu_kms *dpu_kms, char *clock_name);
>
> #endif /* __dpu_kms_H__ */
> --
> 2.39.2
>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2023-06-04 22:00 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-05-26 11:51 [PATCH v2] drm/msm/dpu: clean up dpu_kms_get_clk_rate() returns Dan Carpenter
2023-06-01 16:49 ` Abhinav Kumar
2023-06-04 3:01 ` Dmitry Baryshkov
2023-06-04 22:00 ` Marijn Suijten
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).