2024-04-04 04:57:13

by Randy Dunlap

Subject: [PATCH] LoongArch: addrspace: add linux/sizes.h to prevent build errors

LoongArch's include/asm/addrspace.h uses SZ_32M and SZ_16K, so add
<linux/sizes.h> to provide those macros to prevent build errors:

In file included from ../arch/loongarch/include/asm/io.h:11,
from ../include/linux/io.h:13,
from ../include/linux/io-64-nonatomic-lo-hi.h:5,
from ../drivers/cxl/pci.c:4:
./include/asm-generic/io.h: In function 'ioport_map':
./arch/loongarch/include/asm/addrspace.h:124:25: error: 'SZ_32M' undeclared (first use in this function); did you mean 'PS_32M'?
124 | #define PCI_IOSIZE SZ_32M

Signed-off-by: Randy Dunlap <[email protected]>
Cc: Huacai Chen <[email protected]>
Cc: WANG Xuerui <[email protected]>
Cc: [email protected]
---
arch/loongarch/include/asm/addrspace.h | 1 +
1 file changed, 1 insertion(+)

diff -- a/arch/loongarch/include/asm/addrspace.h b/arch/loongarch/include/asm/addrspace.h
--- a/arch/loongarch/include/asm/addrspace.h
+++ b/arch/loongarch/include/asm/addrspace.h
@@ -11,6 +11,7 @@
#define _ASM_ADDRSPACE_H

#include <linux/const.h>
+#include <linux/sizes.h>

#include <asm/loongarch.h>

2024-04-05 09:23:27

by Huacai Chen

Subject: Re: [PATCH] LoongArch: addrspace: add linux/sizes.h to prevent build errors

Queued, thanks.

Huacai

On Thu, Apr 4, 2024 at 12:57 PM Randy Dunlap <[email protected]> wrote:
>
> LoongArch's include/asm/addrspace.h uses SZ_32M and SZ_16K, so add
> <linux/sizes.h> to provide those macros to prevent build errors:
>
> In file included from ../arch/loongarch/include/asm/io.h:11,
> from ../include/linux/io.h:13,
> from ../include/linux/io-64-nonatomic-lo-hi.h:5,
> from ../drivers/cxl/pci.c:4:
> ../include/asm-generic/io.h: In function 'ioport_map':
> ../arch/loongarch/include/asm/addrspace.h:124:25: error: 'SZ_32M' undeclared (first use in this function); did you mean 'PS_32M'?
> 124 | #define PCI_IOSIZE SZ_32M
>
> Signed-off-by: Randy Dunlap <[email protected]>
> Cc: Huacai Chen <[email protected]>
> Cc: WANG Xuerui <[email protected]>
> Cc: [email protected]
> ---
> arch/loongarch/include/asm/addrspace.h | 1 +
> 1 file changed, 1 insertion(+)
>
> diff -- a/arch/loongarch/include/asm/addrspace.h b/arch/loongarch/include/asm/addrspace.h
> --- a/arch/loongarch/include/asm/addrspace.h
> +++ b/arch/loongarch/include/asm/addrspace.h
> @@ -11,6 +11,7 @@
> #define _ASM_ADDRSPACE_H
>
> #include <linux/const.h>
> +#include <linux/sizes.h>
>
> #include <asm/loongarch.h>
>