diff mbox

[U-Boot] nios2: change virt_to_phys to use physaddr_mask in global data

Message ID 1445908197-9907-1-git-send-email-thomas@wytron.com.tw
State Accepted, archived
Delegated to: Thomas Chou
Headers show

Commit Message

Thomas Chou Oct. 27, 2015, 1:09 a.m. UTC
As virt_to_phys() is used a lot in DMA transfer, change it
to use physaddr_mask in global data. This will save an "if"
statement and get a little faster.

Signed-off-by: Thomas Chou <thomas@wytron.com.tw>
---
 arch/nios2/cpu/cpu.c                 | 1 +
 arch/nios2/include/asm/global_data.h | 1 +
 arch/nios2/include/asm/io.h          | 5 +----
 3 files changed, 3 insertions(+), 4 deletions(-)

Comments

Marek Vasut Oct. 27, 2015, 9:16 a.m. UTC | #1
On Tuesday, October 27, 2015 at 02:09:57 AM, Thomas Chou wrote:
> As virt_to_phys() is used a lot in DMA transfer, change it
> to use physaddr_mask in global data. This will save an "if"
> statement and get a little faster.
> 
> Signed-off-by: Thomas Chou <thomas@wytron.com.tw>

Makes sense.

Acked-by: Marek Vasut <marex@denx.de>

Best regards,
Marek Vasut
Thomas Chou Nov. 3, 2015, 5:14 a.m. UTC | #2
On 2015年10月27日 09:09, Thomas Chou wrote:
> As virt_to_phys() is used a lot in DMA transfer, change it
> to use physaddr_mask in global data. This will save an "if"
> statement and get a little faster.
>
> Signed-off-by: Thomas Chou <thomas@wytron.com.tw>
> ---
>   arch/nios2/cpu/cpu.c                 | 1 +
>   arch/nios2/include/asm/global_data.h | 1 +
>   arch/nios2/include/asm/io.h          | 5 +----
>   3 files changed, 3 insertions(+), 4 deletions(-)
>

Applied to u-boot-nios.
diff mbox

Patch

diff --git a/arch/nios2/cpu/cpu.c b/arch/nios2/cpu/cpu.c
index 88c4e18..f6d5cd3 100644
--- a/arch/nios2/cpu/cpu.c
+++ b/arch/nios2/cpu/cpu.c
@@ -119,6 +119,7 @@  static int altera_nios2_probe(struct udevice *dev)
 		"altr,has-mmu", 0);
 	gd->arch.io_region_base = gd->arch.has_mmu ? 0xe0000000 : 0x80000000;
 	gd->arch.mem_region_base = gd->arch.has_mmu ? 0xc0000000 : 0x00000000;
+	gd->arch.physaddr_mask = gd->arch.has_mmu ? 0x1fffffff : 0x7fffffff;
 
 	return 0;
 }
diff --git a/arch/nios2/include/asm/global_data.h b/arch/nios2/include/asm/global_data.h
index 9f3bd00..9863fd9 100644
--- a/arch/nios2/include/asm/global_data.h
+++ b/arch/nios2/include/asm/global_data.h
@@ -19,6 +19,7 @@  struct arch_global_data {
 	int has_mmu;
 	u32 io_region_base;
 	u32 mem_region_base;
+	u32 physaddr_mask;
 };
 
 #include <asm-generic/global_data.h>
diff --git a/arch/nios2/include/asm/io.h b/arch/nios2/include/asm/io.h
index 007df8d..3443408 100644
--- a/arch/nios2/include/asm/io.h
+++ b/arch/nios2/include/asm/io.h
@@ -44,10 +44,7 @@  static inline void unmap_physmem(void *vaddr, unsigned long flags)
 static inline phys_addr_t virt_to_phys(void * vaddr)
 {
 	DECLARE_GLOBAL_DATA_PTR;
-	if (gd->arch.has_mmu)
-		return (phys_addr_t)vaddr & 0x1fffffff;
-	else
-		return (phys_addr_t)vaddr & 0x7fffffff;
+	return (phys_addr_t)vaddr & gd->arch.physaddr_mask;
 }
 
 static inline void *ioremap(unsigned long physaddr, unsigned long size)