On Thu, Apr 14, 2022 at 02:09:15AM +0530, Aliya Rahmani wrote:
> Remove volatile to fix checkpatch.pl warning.
>
You want to sound more confident like you know what you're talking
about and not just doing things because checkpatch says. "The volatile
is meaningless here and checkpatch.pl complains about it. Delete it."
> Signed-off-by: Aliya Rahmani <aliyarahmani786@xxxxxxxxx>
> ---
> drivers/staging/sm750fb/sm750.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/sm750fb/sm750.h b/drivers/staging/sm750fb/sm750.h
> index aff69661c8e6..7a11f8593596 100644
> --- a/drivers/staging/sm750fb/sm750.h
> +++ b/drivers/staging/sm750fb/sm750.h
> @@ -52,7 +52,7 @@ struct lynx_accel {
> /* base virtual address of DPR registers */
> volatile unsigned char __iomem *dprBase;
^^^^^^^^
Missed one here.
> /* base virtual address of de data port */
> - volatile unsigned char __iomem *dpPortBase;
> + unsigned char __iomem *dpPortBase;
regards,
dan carpenter
