[PATCH] drm/amdgpu: Clean up style problems in mmhub_v2_3.c

[Chen, Guchun]

[Public]

Reviewed-by: Guchun Chen <guchun.chen at amd.com>

Regards,
Guchun

> -----Original Message-----
> From: amd-gfx <amd-gfx-bounces at lists.freedesktop.org> On Behalf Of
> Srinivasan Shanmugam
> Sent: Thursday, July 27, 2023 10:37 PM
> To: Koenig, Christian <Christian.Koenig at amd.com>; Deucher, Alexander
> <Alexander.Deucher at amd.com>
> Cc: SHANMUGAM, SRINIVASAN <SRINIVASAN.SHANMUGAM at amd.com>;
> amd-gfx at lists.freedesktop.org
> Subject: [PATCH] drm/amdgpu: Clean up style problems in mmhub_v2_3.c
>
> Fixes the following:
>
> ERROR: code indent should use tabs where possible
> WARNING: Missing a blank line after declarations
> WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
> WARNING: suspect code indent for conditional statements (8, 24)
> +       if (!(data & (DAGB0_CNTL_MISC2__DISABLE_WRREQ_CG_MASK |
> [...]
> +                       *flags |= AMD_CG_SUPPORT_MC_MGCG;
>
> Cc: Christian König <christian.koenig at amd.com>
> Cc: Alex Deucher <alexander.deucher at amd.com>
> Signed-off-by: Srinivasan Shanmugam <srinivasan.shanmugam at amd.com>
> ---
>  drivers/gpu/drm/amd/amdgpu/mmhub_v2_3.c | 25 +++++++++++++----------
> --
>  1 file changed, 13 insertions(+), 12 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/mmhub_v2_3.c
> b/drivers/gpu/drm/amd/amdgpu/mmhub_v2_3.c
> index 8bd0fc8d9d25..1dce053a4c4d 100644
> --- a/drivers/gpu/drm/amd/amdgpu/mmhub_v2_3.c
> +++ b/drivers/gpu/drm/amd/amdgpu/mmhub_v2_3.c
> @@ -331,7 +331,7 @@ static void mmhub_v2_3_setup_vmid_config(struct
> amdgpu_device *adev)  static void
> mmhub_v2_3_program_invalidation(struct amdgpu_device *adev)  {
>       struct amdgpu_vmhub *hub = &adev-
> >vmhub[AMDGPU_MMHUB0(0)];
> -     unsigned i;
> +     unsigned int i;
>
>       for (i = 0; i < 18; ++i) {
>               WREG32_SOC15_OFFSET(MMHUB, 0,
> @@ -406,6 +406,7 @@ static void
> mmhub_v2_3_set_fault_enable_default(struct amdgpu_device *adev,
>                                               bool value)
>  {
>       u32 tmp;
> +
>       tmp = RREG32_SOC15(MMHUB, 0,
> mmMMVM_L2_PROTECTION_FAULT_CNTL);
>       tmp = REG_SET_FIELD(tmp, MMVM_L2_PROTECTION_FAULT_CNTL,
>                           RANGE_PROTECTION_FAULT_ENABLE_DEFAULT,
> value); @@ -499,11 +500,11 @@
> mmhub_v2_3_update_medium_grain_clock_gating(struct amdgpu_device
> *adev,
>       if (enable && (adev->cg_flags & AMD_CG_SUPPORT_MC_MGCG)) {
>               data &=
> ~MM_ATC_L2_CGTT_CLK_CTRL__SOFT_OVERRIDE_MASK;
>               data1 &=
> ~(DAGB0_CNTL_MISC2__DISABLE_WRREQ_CG_MASK |
> -                        DAGB0_CNTL_MISC2__DISABLE_WRRET_CG_MASK |
> -                        DAGB0_CNTL_MISC2__DISABLE_RDREQ_CG_MASK |
> -                        DAGB0_CNTL_MISC2__DISABLE_RDRET_CG_MASK |
> -                        DAGB0_CNTL_MISC2__DISABLE_TLBWR_CG_MASK |
> -                        DAGB0_CNTL_MISC2__DISABLE_TLBRD_CG_MASK);
> +                        DAGB0_CNTL_MISC2__DISABLE_WRRET_CG_MASK
> |
> +                        DAGB0_CNTL_MISC2__DISABLE_RDREQ_CG_MASK
> |
> +                        DAGB0_CNTL_MISC2__DISABLE_RDRET_CG_MASK |
> +                        DAGB0_CNTL_MISC2__DISABLE_TLBWR_CG_MASK
> |
> +                        DAGB0_CNTL_MISC2__DISABLE_TLBRD_CG_MASK);
>
>       } else {
>               data |=
> MM_ATC_L2_CGTT_CLK_CTRL__SOFT_OVERRIDE_MASK;
> @@ -593,13 +594,13 @@ static void mmhub_v2_3_get_clockgating(struct
> amdgpu_device *adev, u64 *flags)
>
>       /* AMD_CG_SUPPORT_MC_MGCG */
>       if (!(data & (DAGB0_CNTL_MISC2__DISABLE_WRREQ_CG_MASK |
> -                    DAGB0_CNTL_MISC2__DISABLE_WRRET_CG_MASK |
> -                    DAGB0_CNTL_MISC2__DISABLE_RDREQ_CG_MASK |
> -                    DAGB0_CNTL_MISC2__DISABLE_RDRET_CG_MASK |
> -                    DAGB0_CNTL_MISC2__DISABLE_TLBWR_CG_MASK |
> -                    DAGB0_CNTL_MISC2__DISABLE_TLBRD_CG_MASK))
> +                     DAGB0_CNTL_MISC2__DISABLE_WRRET_CG_MASK |
> +                     DAGB0_CNTL_MISC2__DISABLE_RDREQ_CG_MASK |
> +                     DAGB0_CNTL_MISC2__DISABLE_RDRET_CG_MASK |
> +                     DAGB0_CNTL_MISC2__DISABLE_TLBWR_CG_MASK |
> +                     DAGB0_CNTL_MISC2__DISABLE_TLBRD_CG_MASK))
>               && !(data1 &
> MM_ATC_L2_CGTT_CLK_CTRL__SOFT_OVERRIDE_MASK)) {
> -                     *flags |= AMD_CG_SUPPORT_MC_MGCG;
> +             *flags |= AMD_CG_SUPPORT_MC_MGCG;
>       }
>
>       /* AMD_CG_SUPPORT_MC_LS */
> --
> 2.25.1

-------------- next part --------------
A non-text attachment was scrubbed...
Name: winmail.dat
Type: application/ms-tnef
Size: 16736 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20230731/d2a3a1bb/attachment-0001.bin>