Re: [PATCH] drm/radeon: Fix multiple line dereference in 'atom_iio_execute'

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Aug 9, 2023 at 5:50 AM Srinivasan Shanmugam
<srinivasan.shanmugam@xxxxxxx> wrote:
>
> Fixes the following:
>
> WARNING: Avoid multiple line dereference - prefer 'ctx->io_attr'
> +                           ((ctx->
> +                             io_attr >> CU8(base + 2)) & (0xFFFFFFFF >> (32 -
>
> Cc: Guchun Chen <guchun.chen@xxxxxxx>
> Cc: Christian König <christian.koenig@xxxxxxx>
> Cc: Alex Deucher <alexander.deucher@xxxxxxx>
> Cc: "Pan, Xinhui" <Xinhui.Pan@xxxxxxx>
> Signed-off-by: Srinivasan Shanmugam <srinivasan.shanmugam@xxxxxxx>

Reviewed-by: Alex Deucher <alexander.deucher@xxxxxxx>

> ---
>  drivers/gpu/drm/radeon/atom.c | 10 +++-------
>  1 file changed, 3 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/gpu/drm/radeon/atom.c b/drivers/gpu/drm/radeon/atom.c
> index 93acb0e42bd6..ceb6d772ef94 100644
> --- a/drivers/gpu/drm/radeon/atom.c
> +++ b/drivers/gpu/drm/radeon/atom.c
> @@ -163,13 +163,9 @@ static uint32_t atom_iio_execute(struct atom_context *ctx, int base,
>                             ~((0xFFFFFFFF >> (32 - CU8(base + 1))) <<
>                               CU8(base + 3));
>                         temp |=
> -                           ((ctx->
> -                             io_attr >> CU8(base + 2)) & (0xFFFFFFFF >> (32 -
> -                                                                         CU8
> -                                                                         (base
> -                                                                          +
> -                                                                          1))))
> -                           << CU8(base + 3);
> +                           ((ctx->io_attr >> CU8(base + 2)) &
> +                            (0xFFFFFFFF >> (32 - CU8(base + 1)))) <<
> +                            CU8(base + 3);
>                         base += 4;
>                         break;
>                 case ATOM_IIO_END:
> --
> 2.25.1
>




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux