Data types defined for 64 bit physical address
Subsystems (2)
PATCH 2/3 Data types defined for 64 bit physical address
Data types and I/O functions have been defined for 64 bit physical addresses in arm and powerpc
--- arch/arm/include/asm/io.h | 4 +++- arch/arm/include/asm/types.h | 13 ++++++++----- arch/powerpc/include/asm/io.h | 13 +++++++++++++ 3 files changed, 24 insertions(+), 6 deletions(-)
arch/arm/include/asm/io.h +3 -1
@@ -59,7 +59,7 @@ static inline void unmap_physmem(void *vaddr, unsigned long flags)static inline phys_addr_t virt_to_phys (void * vaddr) { -return (phys_addr_t)(vaddr); +return (phys_addr_t)((unsigned long )vaddr); }/*
@@ -183,9 +183,11 @@ static inline void __raw_readsl(unsigned long addr, void *data, int longlen)#define in_le32 (a) in_arch(l,le32,a)#define in_le16 (a) in_arch(w,le16,a) +#define out_be64 (a, v) out_arch(q, be64, a, v)#define out_be32 (a,v) out_arch(l,be32,a,v)#define out_be16 (a,v) out_arch(w,be16,a,v) +#define in_be64 (a) in_arch(q, be64, a)#define in_be32 (a) in_arch(l,be32,a)#define in_be16 (a) in_arch(w,be16,a)
arch/arm/include/asm/types.h +8 -5
@@ -45,12 +45,15 @@ typedef unsigned long long u64;#define BITS_PER_LONG 32#endif /* CONFIG_ARM64 */ -/* Dma addresses are 32-bits wide. */ - +#ifdef CONFIG_PHYS_64BIT +typedef u64 dma_addr_t ; +typedef u64 phys_addr_t ; +typedef u64 phys_size_t ; +#else typedef u32 dma_addr_t ; - -typedef unsigned long phys_addr_t ; -typedef unsigned long phys_size_t ; +typedef u32 phys_addr_t ; +typedef u32 phys_size_t ; +#endif #endif /* __KERNEL__ */
arch/powerpc/include/asm/io.h +13 -0
@@ -246,6 +246,19 @@ extern inline void out_be32(volatile unsigned __iomem *addr, u32 val) __asm__ __volatile__("sync; stw%U0%X0 %1,%0" :"=m" (*addr) :"r" (val)); } +extern inline u64 in_be64 (const u64 *addr) +{ +return ((u64 )in_be32 ((u32 *)addr) <<32 ) | + (in_be32 ((u32 *)addr +1 )); +} + +extern inline void out_be64 (u64 *addr,u64 val) +{ +out_be32 ((u32 *)addr, (u32)(val >>32 )); +out_be32 ((u32 *)addr +1 , (u32 )val); +} + +/* Clear and set bits in one shot. These macros can be used to clear and * set multiple bits in a register using a single call. These macros can * also be used to set a multiple-bit bit pattern using a mask, by
Signature · 1 line
1.8.1.4
PATCH 2/3 Data types defined for 64 bit physical address
Quoted 15 lines · Aneesh Bansal
Data types and I/O functions have been defined for 64 bit physical addresses in arm and powerpc
Signed-off-by Aneesh Bansal <[email protected]>--- arch/arm/include/asm/io.h | 4 +++- arch/arm/include/asm/types.h | 13 ++++++++----- arch/powerpc/include/asm/io.h | 13 +++++++++++++ 3 files changed, 24 insertions(+), 6 deletions(-)arch/arm/include/asm/io.h +0
-0
arch/arm/include/asm/io.h static inline void unmap_physmem(void *vaddr, unsigned · line 64
Quoted 7 lines
arch/arm/include/asm/io.h +1
-1@@ -59,7 +59,7 @@ static inline void unmap_physmem(void *vaddr, unsigned long flags)static inline phys_addr_t virt_to_phys (void * vaddr) { -return (phys_addr_t)(vaddr); +return (phys_addr_t)((unsigned long )vaddr); }
Unnecessary parens.
Quoted 4 lines
arch/arm/include/asm/io.h +0
-0/* @@ -183,9 +183,11 @@ static inline void __raw_readsl(unsigned long addr, void *data, int longlen)
arch/arm/include/asm/io.h static inline void __raw_readsl(unsigned long addr, · line 193
Quoted 10 lines
arch/arm/include/asm/io.h +2
-0#define in_le32 (a) in_arch(l,le32,a)#define in_le16 (a) in_arch(w,le16,a) +#define out_be64 (a, v) out_arch(q, be64, a, v)#define out_be32 (a,v) out_arch(l,be32,a,v)#define out_be16 (a,v) out_arch(w,be16,a,v) +#define in_be64 (a) in_arch(q, be64, a)#define in_be32 (a) in_arch(l,be32,a)#define in_be16 (a) in_arch(w,be16,a)
Inconsistent whitespace.
Quoted 10 lines
arch/powerpc/include/asm/io.h +1
-0@@ -246,6 +246,19 @@ extern inline void out_be32(volatile unsigned __iomem *addr, u32 val) __asm__ __volatile__("sync; stw%U0%X0 %1,%0" :"=m" (*addr) :"r" (val)); } +extern inline u64 in_be64 (const u64 *addr)
arch/powerpc/include/asm/io.h extern inline void out_be32(volatile unsigned __iomem · line 260
Quoted 10 lines
arch/powerpc/include/asm/io.h +10
-0+{ +return ((u64 )in_be32 ((u32 *)addr) <<32 ) | + (in_be32 ((u32 *)addr +1 )); +} + +extern inline void out_be64 (u64 *addr,u64 val) +{ +out_be32 ((u32 *)addr, (u32)(val >>32 )); +out_be32 ((u32 *)addr +1 , (u32 )val); +}
What do you need these for? I don't think it's a good idea to have I/O accessors that look atomic but aren't (same goes for arm32).
-Scott
PATCH 2/3 Data types defined for 64 bit physical address
Quoted 26 lines
-----Original Message-----
From: Wood Scott-B07421
Sent: Friday, July 31, 2015 10:51 PM
To: Bansal Aneesh-B39320
Cc: u-boot at lists.denx.de; Sun York-R58495; Gupta Ruchika-R66431; Kushwaha Prabhakar-B32579
Subject: Re: [PATCH 2/3] Data types defined for 64 bit physical addressQuoted 17 lines · Aneesh Bansal
Data types and I/O functions have been defined for 64 bit physical addresses in arm and powerpc
Signed-off-by Aneesh Bansal <[email protected]>--- arch/arm/include/asm/io.h | 4 +++- arch/arm/include/asm/types.h | 13 ++++++++----- arch/powerpc/include/asm/io.h | 13 +++++++++++++ 3 files changed, 24 insertions(+), 6 deletions(-)arch/arm/include/asm/io.h +0
-0@@ -59,7 +59,7 @@ static inline void unmap_physmem(void *vaddr, unsigned long flags)
Quoted 7 lines
arch/arm/include/asm/io.h static inline void unmap_physmem(void *vaddr, unsigned long flags) · line 63
Quoted 4 lines
arch/arm/include/asm/io.h +1
-1static inline phys_addr_t virt_to_phys (void * vaddr) { -return (phys_addr_t)(vaddr); +return (phys_addr_t)((unsigned long )vaddr); }Unnecessary parens.
Ok, I will remove these.
Quoted 11 lines
Quoted 11 lines
arch/arm/include/asm/io.h +1
-0/* @@ -183,9 +183,11 @@ static inline void __raw_readsl(unsigned long addr, void *data, int longlen)#define in_le32 (a) in_arch(l,le32,a)#define in_le16 (a) in_arch(w,le16,a) +#define out_be64 (a, v) out_arch(q, be64, a, v)#define out_be32 (a,v) out_arch(l,be32,a,v)#define out_be16 (a,v) out_arch(w,be16,a,v)
Quoted 5 lines
arch/arm/include/asm/io.h static inline void __raw_readsl(unsigned long addr, void *data, int longlen) · line 192
Quoted 3 lines
arch/arm/include/asm/io.h +1
-0+#define in_be64 (a) in_arch(q, be64, a)#define in_be32 (a) in_arch(l,be32,a)#define in_be16 (a) in_arch(w,be16,a)Inconsistent whitespace.
Without the space, I was getting an error with Checkpatch.
Should, I correct it all lines already present in the file or even add new lines in the same way as other lines already present even though checl=kpatch reports it as an error ?
Quoted 10 lines
Quoted 10 lines
(quoted diff) +0
-0
b/arch/powerpc/include/asm/io.hindex a5257e9..8c6f47e 100644arch/powerpc/include/asm/io.h +0
-0@@ -246,6 +246,19 @@ extern inline void out_be32(volatile unsigned __iomem *addr, u32 val) __asm__ __volatile__("sync; stw%U0%X0 %1,%0" :"=m" (*addr) :"r" (val)); }
Quoted 12 lines
arch/powerpc/include/asm/io.h extern inline void out_be32(volatile unsigned __iomem *addr, u32 val)
Quoted 8 lines
arch/powerpc/include/asm/io.h +8
-0+extern inline u64 in_be64 (const u64 *addr) { +return ((u64 )in_be32 ((u32 *)addr) <<32 ) | + (in_be32 ((u32 *)addr +1 )); +} + +extern inline void out_be64 (u64 *addr,u64 val) { +out_be32 ((u32 *)addr, (u32)(val >>32 )); +out_be32 ((u32 *)addr +1 , (u32 )val); }What do you need these for? I don't think it's a good idea to have I/O accessors that look atomic but aren't (same goes for arm32).
64 bit read and writes are required for CAAM operations. That is why I have added these in powerpc and arm.
Quoted 1 line
-Scott
PATCH 2/3 Data types defined for 64 bit physical address
Quoted 45 lines · On Fri, 2015-07-31 at 12:28 -0500, Bansal Aneesh-B39320 wrote:
Quoted 26 lines
-----Original Message-----
From: Wood Scott-B07421
Sent: Friday, July 31, 2015 10:51 PM
To: Bansal Aneesh-B39320
Cc: u-boot at lists.denx.de; Sun York-R58495; Gupta Ruchika-R66431; Kushwaha Prabhakar-B32579
Subject: Re: [PATCH 2/3] Data types defined for 64 bit physical addressQuoted 17 lines · Aneesh Bansal
Data types and I/O functions have been defined for 64 bit physical addresses in arm and powerpc
Signed-off-by Aneesh Bansal <[email protected]>--- arch/arm/include/asm/io.h | 4 +++- arch/arm/include/asm/types.h | 13 ++++++++----- arch/powerpc/include/asm/io.h | 13 +++++++++++++ 3 files changed, 24 insertions(+), 6 deletions(-)arch/arm/include/asm/io.h +0
-0@@ -59,7 +59,7 @@ static inline void unmap_physmem(void *vaddr, unsigned long flags)Quoted 7 lines
arch/arm/include/asm/io.h static inline void unmap_physmem(void *vaddr, unsigned long flags) · line 63
Quoted 4 lines
arch/arm/include/asm/io.h +1
-1static inline phys_addr_t virt_to_phys (void * vaddr) { -return (phys_addr_t)(vaddr); +return (phys_addr_t)((unsigned long )vaddr); }Unnecessary parens.
Ok, I will remove these.
Quoted 11 lines
Quoted 11 lines
arch/arm/include/asm/io.h +1
-0/* @@ -183,9 +183,11 @@ static inline void __raw_readsl(unsigned long addr, void *data, int longlen)#define in_le32 (a) in_arch(l,le32,a)#define in_le16 (a) in_arch(w,le16,a) +#define out_be64 (a, v) out_arch(q, be64, a, v)#define out_be32 (a,v) out_arch(l,be32,a,v)#define out_be16 (a,v) out_arch(w,be16,a,v)
Quoted 6 lines
Quoted 5 lines
arch/arm/include/asm/io.h static inline void __raw_readsl(unsigned long addr, void *data, int longlen) · line 192
Quoted 3 lines
arch/arm/include/asm/io.h +1
-0+#define in_be64 (a) in_arch(q, be64, a)#define in_be32 (a) in_arch(l,be32,a)#define in_be16 (a) in_arch(w,be16,a)Inconsistent whitespace.
Without the space, I was getting an error with Checkpatch.
So?
Quoted 3 lines
Should, I correct it all lines already present in the file or even add new lines in the same way as other lines already present even though checl=kpatch reports it as an error ?
Checkpatch is a tool, not a dictator.
Quoted 22 lines
Quoted 9 lines
Quoted 9 lines
(quoted diff) +0
-0
b/arch/powerpc/include/asm/io.hindex a5257e9..8c6f47e 100644arch/powerpc/include/asm/io.h +0
-0@@ -246,6 +246,19 @@ extern inline void out_be32(volatile unsigned __iomem *addr, u32 val) __asm__ __volatile__("sync; stw%U0%X0 %1,%0" :"=m" (*addr) :"r" (val)); }Quoted 13 lines
arch/powerpc/include/asm/io.h extern inline void out_be32(volatile unsigned __iomem *addr, u32 val)
Quoted 8 lines
arch/powerpc/include/asm/io.h +8
-0+extern inline u64 in_be64 (const u64 *addr) { +return ((u64 )in_be32 ((u32 *)addr) <<32 ) | + (in_be32 ((u32 *)addr +1 )); +} + +extern inline void out_be64 (u64 *addr,u64 val) { +out_be32 ((u32 *)addr, (u32)(val >>32 )); +out_be32 ((u32 *)addr +1 , (u32 )val); }What do you need these for? I don't think it's a good idea to have I/O accessors that
look atomic but aren't (same goes for arm32).
Quoted 2 lines
64 bit read and writes are required for CAAM operations. That is why I have added these in powerpc and arm.
No, it's not required. You could just as well perform multiple out_be32() in the driver.
-Scott
PATCH 2/3 Data types defined for 64 bit physical address
Quoted 21 lines
Quoted 21 lines
Quoted 21 lines
Quoted 7 lines
(quoted diff) +0
-0
b/arch/powerpc/include/asm/io.hindex a5257e9..8c6f47e 100644arch/powerpc/include/asm/io.h +0
-0@@ -246,6 +246,19 @@ extern inline void out_be32(volatile unsigned __iomem *addr, u32 val) __asm__ __volatile__("sync; stw%U0%X0 %1,%0" : "=m" (*addr) :arch/powerpc/include/asm/io.h extern inline void out_be32(volatile unsigned · line 257
Quoted 10 lines
arch/powerpc/include/asm/io.h +8
-0"r" (val)); } +extern inline u64 in_be64 (const u64 *addr) { +return ((u64 )in_be32 ((u32 *)addr) <<32 ) | + (in_be32 ((u32 *)addr +1 )); +} + +extern inline void out_be64 (u64 *addr,u64 val) { +out_be32 ((u32 *)addr, (u32)(val >>32 )); +out_be32 ((u32 *)addr +1 , (u32 )val); }What do you need these for? I don't think it's a good idea to have I/O accessors that look atomic but aren't (same goes for arm32).
Quoted 7 lines
Quoted 2 lines
64 bit read and writes are required for CAAM operations. That is why I have added these in powerpc and arm.
No, it's not required. You could just as well perform multiple
out_be32()in the driver.-Scott
It is required to define I/O accessor for 64 bit access.
It is already defined in arm.
The driver will just call OUT_64 or IN_64.
Now that might be a be64 or le64 depending upon the endianness of IP block on that SoC. Further the definition of be64 and le64 will be different for ARM and PowerPC.
We have a common CAAM driver which can't just have multiple out_be32() instead of OUT_64. All PowerPC SOC's : Core is BE and CAAM is BE
LS1020 : Core is LE and CAAM is LE
The problem is with LS1043, where core in LE and CAAM is BE. So in order, to have a unified CAAM driver we need accessors for 64 bit.
For ARM, these functions are directly defined in C as:out_16 (*(volatile unsigned short *)(a) = (v))out_32 (*(volatile unsigned int *)(a) = (v))out_64 (*(volatile unsigned long long *)(a) = (v))
There is another macro defined which takes care of byte-swapping the value for BE or LE
But in PowerPC, for 16 and 32 bit these are defined in ASM.
I could not find asm instructions for 64 bit read/write. So, I defined these in C.
PATCH 2/3 Data types defined for 64 bit physical address
Quoted 24 lines · On Fri, 2015-07-31 at 13:32 -0500, Bansal Aneesh-B39320 wrote:
Quoted 24 lines
Quoted 23 lines
Quoted 9 lines
Quoted 9 lines
(quoted diff) +0
-0
b/arch/powerpc/include/asm/io.hindex a5257e9..8c6f47e 100644arch/powerpc/include/asm/io.h +0
-0@@ -246,6 +246,19 @@ extern inline void out_be32(volatile unsigned __iomem *addr, u32 val) __asm__ __volatile__("sync; stw%U0%X0 %1,%0" :"=m" (*addr) :"r" (val)); }Quoted 12 lines
arch/powerpc/include/asm/io.h extern inline void out_be32(volatile unsigned __iomem *addr, u32 val)
Quoted 8 lines
arch/powerpc/include/asm/io.h +8
-0+extern inline u64 in_be64 (const u64 *addr) { +return ((u64 )in_be32 ((u32 *)addr) <<32 ) | + (in_be32 ((u32 *)addr +1 )); +} + +extern inline void out_be64 (u64 *addr,u64 val) { +out_be32 ((u32 *)addr, (u32)(val >>32 )); +out_be32 ((u32 *)addr +1 , (u32 )val); }What do you need these for? I don't think it's a good idea to have I/O accessors that look atomic but aren't (same goes for arm32).
64 bit read and writes are required for CAAM operations. That is why I have added these in powerpc and arm.
Quoted 6 lines
Quoted 5 lines
No, it's not required. You could just as well perform multiple
out_be32()in the driver.-Scott
It is required to define I/O accessor for 64 bit access.
It is only required if you need to do actual 64-bit I/O, which you can't do in a 32-bit U-Boot (and all PPC U-Boot is 32-bit).
Quoted 1 line
It is already defined in arm.
I think that should be limited to arm64.
Quoted 10 lines
The driver will just call
OUT_64orIN_64.
Now that might be a be64 or le64 depending upon the endianness of IP block on that SoC.
Further the definition of be64 and le64 will be different for ARM and PowerPC.We have a common CAAM driver which can't just have multiple
out_be32()instead ofOUT_64.
All PowerPC SOC's : Core is BE and CAAM is BE
LS1020 : Core is LE and CAAM is LE
You already had addr_lo and addr_hi, with an CONFIG_SYS_FSL_SEC_LE ifdef. Why was that not sufficient?
Quoted 1 line
The problem is with LS1043, where core in LE and CAAM is BE.
The core's endianness is irrelevant.
Quoted 6 lines
So in order, to have a unified CAAM driver we need accessors for 64 bit.
For ARM, these functions are directly defined in C as:
out_16 (*(volatile unsigned short *)(a) = (v)) out_32 (*(volatile unsigned int *)(a) = (v)) out_64 (*(volatile unsigned long long *)(a) = (v))
Quoted 6 lines
There is another macro defined which takes care of byte-swapping the value for BE or LE
But in PowerPC, for 16 and 32 bit these are defined in ASM. I could not find asm instructions for 64 bit read/write. So, I defined these in C.
You couldn't find them because they don't exist (except for floating point and such, but there's no reason to use that here), and thus the accessor shouldn't exist.
BTW, why are the patches of this series posted in separate threads? Are you invoking git send-email separately for each patch?
-Scott