diff mbox series

include: marvell: octeontx2: build error: unknown type name 'u64'

Message ID 20211013135743.3826594-1-anders.roxell@linaro.org (mailing list archive)
State Not Applicable
Delegated to: Herbert Xu
Headers show
Series include: marvell: octeontx2: build error: unknown type name 'u64' | expand

Commit Message

Anders Roxell Oct. 13, 2021, 1:57 p.m. UTC
Building an allmodconfig kernel arm64 kernel, the following build error
shows up:

In file included from drivers/crypto/marvell/octeontx2/cn10k_cpt.c:4:
include/linux/soc/marvell/octeontx2/asm.h:38:15: error: unknown type name 'u64'
   38 | static inline u64 otx2_atomic64_fetch_add(u64 incr, u64 *ptr)
      |               ^~~

Include linux/types.h in asm.h so the compiler knows what the type
'u64' are.

Fixes: af3826db74d1 ("octeontx2-pf: Use hardware register for CQE count")
Signed-off-by: Anders Roxell <anders.roxell@linaro.org>
---
 include/linux/soc/marvell/octeontx2/asm.h | 1 +
 1 file changed, 1 insertion(+)

Comments

Jakub Kicinski Oct. 13, 2021, 3:40 p.m. UTC | #1
On Wed, 13 Oct 2021 15:57:43 +0200 Anders Roxell wrote:
> Building an allmodconfig kernel arm64 kernel, the following build error
> shows up:
> 
> In file included from drivers/crypto/marvell/octeontx2/cn10k_cpt.c:4:
> include/linux/soc/marvell/octeontx2/asm.h:38:15: error: unknown type name 'u64'
>    38 | static inline u64 otx2_atomic64_fetch_add(u64 incr, u64 *ptr)
>       |               ^~~
> 
> Include linux/types.h in asm.h so the compiler knows what the type
> 'u64' are.
> 
> Fixes: af3826db74d1 ("octeontx2-pf: Use hardware register for CQE count")
> Signed-off-by: Anders Roxell <anders.roxell@linaro.org>

Yes, please! I've been carrying same patch locally. Any expectations on
who should apply the patch? I'm gonna send a PR with networking fixes
to Linus tomorrow, happy to take it via netdev if that's okay.
Jakub Kicinski Oct. 13, 2021, 8:27 p.m. UTC | #2
On Wed, 13 Oct 2021 08:40:20 -0700 Jakub Kicinski wrote:
> On Wed, 13 Oct 2021 15:57:43 +0200 Anders Roxell wrote:
> > Building an allmodconfig kernel arm64 kernel, the following build error
> > shows up:
> > 
> > In file included from drivers/crypto/marvell/octeontx2/cn10k_cpt.c:4:
> > include/linux/soc/marvell/octeontx2/asm.h:38:15: error: unknown type name 'u64'
> >    38 | static inline u64 otx2_atomic64_fetch_add(u64 incr, u64 *ptr)
> >       |               ^~~
> > 
> > Include linux/types.h in asm.h so the compiler knows what the type
> > 'u64' are.
> > 
> > Fixes: af3826db74d1 ("octeontx2-pf: Use hardware register for CQE count")
> > Signed-off-by: Anders Roxell <anders.roxell@linaro.org>  
> 
> Yes, please! I've been carrying same patch locally. Any expectations on
> who should apply the patch? I'm gonna send a PR with networking fixes
> to Linus tomorrow, happy to take it via netdev if that's okay.

I realized the breakage only exists in net-next so applied 
the fix there. Thanks!
diff mbox series

Patch

diff --git a/include/linux/soc/marvell/octeontx2/asm.h b/include/linux/soc/marvell/octeontx2/asm.h
index 0f79fd7f81a1..d683251a0b40 100644
--- a/include/linux/soc/marvell/octeontx2/asm.h
+++ b/include/linux/soc/marvell/octeontx2/asm.h
@@ -5,6 +5,7 @@ 
 #ifndef __SOC_OTX2_ASM_H
 #define __SOC_OTX2_ASM_H
 
+#include <linux/types.h>
 #if defined(CONFIG_ARM64)
 /*
  * otx2_lmt_flush is used for LMT store operation.