diff options
| author | Srinivasan Shanmugam <srinivasan.shanmugam@amd.com> | 2023-08-09 15:12:31 +0530 |
|---|---|---|
| committer | Alex Deucher <alexander.deucher@amd.com> | 2023-08-15 17:43:48 -0400 |
| commit | ba4c1d772c26ee91ea308c1b68f4f58a38de2aa5 (patch) | |
| tree | b1af49a9d8c920c9277737c3a5cf0b022c0673dc | |
| parent | 707b570f4288d54e1e7fdf0ed6a872cc84c6464f (diff) | |
| download | linux-ba4c1d772c26ee91ea308c1b68f4f58a38de2aa5.tar.gz linux-ba4c1d772c26ee91ea308c1b68f4f58a38de2aa5.tar.bz2 linux-ba4c1d772c26ee91ea308c1b68f4f58a38de2aa5.zip | |
drm/radeon: Fix multiple line dereference in 'atom_iio_execute'
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@amd.com>
Cc: Christian König <christian.koenig@amd.com>
Cc: Alex Deucher <alexander.deucher@amd.com>
Cc: "Pan, Xinhui" <Xinhui.Pan@amd.com>
Signed-off-by: Srinivasan Shanmugam <srinivasan.shanmugam@amd.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
| -rw-r--r-- | drivers/gpu/drm/radeon/atom.c | 10 |
1 files 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: |