2021-04-16 11:29:12

by Yang Li

Subject: [PATCH] x86/platform/uv: Fix inconsistent indenting

Kernel test robot throws below warning ->

smatch warnings:
arch/x86/kernel/apic/x2apic_uv_x.c:111 early_get_pnodeid() warn:
inconsistent indenting

Reported-by: Abaci Robot <[email protected]>
Signed-off-by: Yang Li <[email protected]>
---
arch/x86/kernel/apic/x2apic_uv_x.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/x86/kernel/apic/x2apic_uv_x.c b/arch/x86/kernel/apic/x2apic_uv_x.c
index 52bc217..3e7534e 100644
--- a/arch/x86/kernel/apic/x2apic_uv_x.c
+++ b/arch/x86/kernel/apic/x2apic_uv_x.c
@@ -108,7 +108,7 @@ static void __init early_get_pnodeid(void)
} else if (UVH_RH_GAM_ADDR_MAP_CONFIG) {
union uvh_rh_gam_addr_map_config_u m_n_config;

- m_n_config.v = uv_early_read_mmr(UVH_RH_GAM_ADDR_MAP_CONFIG);
+ m_n_config.v = uv_early_read_mmr(UVH_RH_GAM_ADDR_MAP_CONFIG);
uv_cpuid.n_skt = m_n_config.s.n_skt;
if (is_uv(UV3))
uv_cpuid.m_skt = m_n_config.s3.m_skt;
--
1.8.3.1

2021-04-16 14:59:33

by Steve Wahl

Subject: Re: [PATCH] x86/platform/uv: Fix inconsistent indenting

Reviewed-by: Steve Wahl <[email protected]>

On Fri, Apr 16, 2021 at 06:10:40PM +0800, Yang Li wrote:
> Kernel test robot throws below warning ->
>
> smatch warnings:
> arch/x86/kernel/apic/x2apic_uv_x.c:111 early_get_pnodeid() warn:
> inconsistent indenting
>
> Reported-by: Abaci Robot <[email protected]>
> Signed-off-by: Yang Li <[email protected]>
> ---
> arch/x86/kernel/apic/x2apic_uv_x.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/x86/kernel/apic/x2apic_uv_x.c b/arch/x86/kernel/apic/x2apic_uv_x.c
> index 52bc217..3e7534e 100644
> --- a/arch/x86/kernel/apic/x2apic_uv_x.c
> +++ b/arch/x86/kernel/apic/x2apic_uv_x.c
> @@ -108,7 +108,7 @@ static void __init early_get_pnodeid(void)
> } else if (UVH_RH_GAM_ADDR_MAP_CONFIG) {
> union uvh_rh_gam_addr_map_config_u m_n_config;
>
> - m_n_config.v = uv_early_read_mmr(UVH_RH_GAM_ADDR_MAP_CONFIG);
> + m_n_config.v = uv_early_read_mmr(UVH_RH_GAM_ADDR_MAP_CONFIG);
> uv_cpuid.n_skt = m_n_config.s.n_skt;
> if (is_uv(UV3))
> uv_cpuid.m_skt = m_n_config.s3.m_skt;
> --
> 1.8.3.1
>

--
Steve Wahl, Hewlett Packard Enterprise