2023-07-24 03:49:59

by Ran Sun

[permalink] [raw]
Subject: [PATCH] drm/radeon: that open brace { should be on the previous line

ERROR: that open brace { should be on the previous line

Signed-off-by: Ran Sun <[email protected]>
---
drivers/gpu/drm/radeon/ni_dpm.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/radeon/ni_dpm.c
b/drivers/gpu/drm/radeon/ni_dpm.c
index a101ba00ea30..1cf4de4cda23 100644
--- a/drivers/gpu/drm/radeon/ni_dpm.c
+++ b/drivers/gpu/drm/radeon/ni_dpm.c
@@ -625,7 +625,7 @@ static const u32 cayman_mgcg_disable[] =
};
#define CAYMAN_MGCG_DISABLE_LENGTH sizeof(cayman_mgcg_disable) / (3 *
sizeof(u32))

-static const u32 cayman_mgcg_enable[] = :621 {
+static const u32 cayman_mgcg_enable[] = : 621 {
0x0000802c, 0xc0000000, 0xffffffff,
0x000008f8, 0x00000000, 0xffffffff,
0x000008fc, 0x00000000, 0xffffffff,


2023-07-24 22:05:23

by Alex Deucher

[permalink] [raw]
Subject: Re: [PATCH] drm/radeon: that open brace { should be on the previous line

On Sun, Jul 23, 2023 at 11:12 PM <[email protected]> wrote:
>
> ERROR: that open brace { should be on the previous line
>
> Signed-off-by: Ran Sun <[email protected]>
> ---
> drivers/gpu/drm/radeon/ni_dpm.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/radeon/ni_dpm.c
> b/drivers/gpu/drm/radeon/ni_dpm.c
> index a101ba00ea30..1cf4de4cda23 100644
> --- a/drivers/gpu/drm/radeon/ni_dpm.c
> +++ b/drivers/gpu/drm/radeon/ni_dpm.c
> @@ -625,7 +625,7 @@ static const u32 cayman_mgcg_disable[] =
> };
> #define CAYMAN_MGCG_DISABLE_LENGTH sizeof(cayman_mgcg_disable) / (3 *
> sizeof(u32))
>
> -static const u32 cayman_mgcg_enable[] = :621 {
> +static const u32 cayman_mgcg_enable[] = : 621 {

Something weird here.

Alex

> 0x0000802c, 0xc0000000, 0xffffffff,
> 0x000008f8, 0x00000000, 0xffffffff,
> 0x000008fc, 0x00000000, 0xffffffff,