Re: [PATCH] drm/msm/adreno: fix sparse warnings in a6xx code

[Stephen Boyd]

Quoting Dmitry Baryshkov (2023-04-06 18:07:41)
> diff --git a/drivers/gpu/drm/msm/adreno/a6xx_gmu.h 
> b/drivers/gpu/drm/msm/adreno/a6xx_gmu.h
> index 0bc3eb443fec..84d345af126f 100644
> --- a/drivers/gpu/drm/msm/adreno/a6xx_gmu.h
> +++ b/drivers/gpu/drm/msm/adreno/a6xx_gmu.h
> @@ -51,8 +51,8 @@ struct a6xx_gmu {
>
> struct msm_gem_address_space *aspace;
>
> -   void * __iomem mmio;
> -   void * __iomem rscc;
> +   void __iomem * mmio;
> +   void __iomem * rscc;

Should stick that * to the member name.

void __iomem *rscc;

with that

Reviewed-by: Stephen Boyd 

Re: [PATCH] drm/msm/adreno: fix sparse warnings in a6xx code

[Javier Martinez Canillas]

Dmitry Baryshkov  writes:

> Sparse reports plenty of warnings against the a6xx code because of
> a6xx_gmu::mmio and a6xx_gmu::rscc members. For some reason they were
> defined as __iomem pointers rather than pointers to __iomem memory.
> Correct the __iomem attribute.
>
> Fixes: 02ef80c54e7c ("drm/msm/a6xx: update pdc/rscc GMU registers for 
> A640/A650")
> Reported-by: kernel test robot 
> Link: 
> https://lore.kernel.org/oe-kbuild-all/202304070550.nrbhjcvp-...@intel.com/
> Signed-off-by: Dmitry Baryshkov 
> ---

Reviewed-by: Javier Martinez Canillas 

-- 
Best regards,

Javier Martinez Canillas
Core Platforms
Red Hat

[PATCH] drm/msm/adreno: fix sparse warnings in a6xx code

[Dmitry Baryshkov]

Sparse reports plenty of warnings against the a6xx code because of
a6xx_gmu::mmio and a6xx_gmu::rscc members. For some reason they were
defined as __iomem pointers rather than pointers to __iomem memory.
Correct the __iomem attribute.

Fixes: 02ef80c54e7c ("drm/msm/a6xx: update pdc/rscc GMU registers for 
A640/A650")
Reported-by: kernel test robot 
Link: https://lore.kernel.org/oe-kbuild-all/202304070550.nrbhjcvp-...@intel.com/
Signed-off-by: Dmitry Baryshkov 
---
 drivers/gpu/drm/msm/adreno/a6xx_gmu.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/msm/adreno/a6xx_gmu.h 
b/drivers/gpu/drm/msm/adreno/a6xx_gmu.h
index 0bc3eb443fec..84d345af126f 100644
--- a/drivers/gpu/drm/msm/adreno/a6xx_gmu.h
+++ b/drivers/gpu/drm/msm/adreno/a6xx_gmu.h
@@ -51,8 +51,8 @@ struct a6xx_gmu {
 
struct msm_gem_address_space *aspace;
 
-   void * __iomem mmio;
-   void * __iomem rscc;
+   void __iomem * mmio;
+   void __iomem * rscc;
 
int hfi_irq;
int gmu_irq;
-- 
2.39.2