2022-08-15 23:20:29

by Xin Gao

Subject: [PATCH] treewide: Complete variable type

It is better to use "unsigned int" instead of "unsigned", line 24.

Signed-off-by: Xin Gao <[email protected]>
---
arch/powerpc/boot/epapr.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/powerpc/boot/epapr.c b/arch/powerpc/boot/epapr.c
index 7c5b26ade6c4..4133bee28673 100644
--- a/arch/powerpc/boot/epapr.c
+++ b/arch/powerpc/boot/epapr.c
@@ -21,7 +21,7 @@ BSS_STACK(4096);
#define EPAPR_SMAGIC 0x65504150
#define EPAPR_EMAGIC 0x45504150

-static unsigned epapr_magic;
+static unsigned int epapr_magic;
static unsigned long ima_size;
static unsigned long fdt_addr;

--
2.30.2

2022-08-16 08:56:34

by Christophe Leroy

Subject: Re: [PATCH] treewide: Complete variable type

Le 15/08/2022 à 21:53, Xin Gao a écrit :
> It is better to use "unsigned int" instead of "unsigned", line 24.

The Subject in wrong: It is not a treewide patch but only a powerpc
patch as far as I understand.

Christophe

>
> Signed-off-by: Xin Gao <[email protected]>
> ---
> arch/powerpc/boot/epapr.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/powerpc/boot/epapr.c b/arch/powerpc/boot/epapr.c
> index 7c5b26ade6c4..4133bee28673 100644
> --- a/arch/powerpc/boot/epapr.c
> +++ b/arch/powerpc/boot/epapr.c
> @@ -21,7 +21,7 @@ BSS_STACK(4096);
> #define EPAPR_SMAGIC 0x65504150
> #define EPAPR_EMAGIC 0x45504150
>
> -static unsigned epapr_magic;
> +static unsigned int epapr_magic;
> static unsigned long ima_size;
> static unsigned long fdt_addr;
>
> --
> 2.30.2
>