diff mbox

[U-Boot] arm: at91: Change the Chip ID registers' addresses

Message ID 1441694306-20205-1-git-send-email-wenyou.yang@atmel.com
State Accepted, archived
Delegated to: Andreas Bießmann
Headers show

Commit Message

Wenyou Yang Sept. 8, 2015, 6:38 a.m. UTC
Provide the specific addresses for the Chip ID and Chip ID Extension
registers, instead of the offset, which make it use on other chips.

Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
---

 arch/arm/mach-at91/armv7/cpu.c             |    7 ++++---
 arch/arm/mach-at91/include/mach/at91_dbu.h |    4 ----
 arch/arm/mach-at91/include/mach/sama5d3.h  |    3 +++
 arch/arm/mach-at91/include/mach/sama5d4.h  |    3 +++
 4 files changed, 10 insertions(+), 7 deletions(-)

Comments

Andreas Bießmann Oct. 23, 2015, 9:10 p.m. UTC | #1
On 08.09.15 08:38, Wenyou Yang wrote:
> Provide the specific addresses for the Chip ID and Chip ID Extension
> registers, instead of the offset, which make it use on other chips.
> 
> Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>

Reviewed-by: Andreas Bießmann <andreas.devel@googlemail.com>
> ---
> 
>  arch/arm/mach-at91/armv7/cpu.c             |    7 ++++---
>  arch/arm/mach-at91/include/mach/at91_dbu.h |    4 ----
>  arch/arm/mach-at91/include/mach/sama5d3.h  |    3 +++
>  arch/arm/mach-at91/include/mach/sama5d4.h  |    3 +++
>  4 files changed, 10 insertions(+), 7 deletions(-)
Wenyou Yang Oct. 27, 2015, 1:22 a.m. UTC | #2
> -----Original Message-----

> From: Andreas Bießmann [mailto:andreas.devel@googlemail.com]

> Sent: 2015年10月24日 5:11

> To: Yang, Wenyou; U-Boot Mailing List

> Subject: Re: [PATCH] arm: at91: Change the Chip ID registers' addresses

> 

> 

> 

> On 08.09.15 08:38, Wenyou Yang wrote:

> > Provide the specific addresses for the Chip ID and Chip ID Extension

> > registers, instead of the offset, which make it use on other chips.

> >

> > Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>

> 

> Reviewed-by: Andreas Bießmann <andreas.devel@googlemail.com>

Thanks


Best Regards,
Wenyou Yang

> > ---

> >

> >  arch/arm/mach-at91/armv7/cpu.c             |    7 ++++---

> >  arch/arm/mach-at91/include/mach/at91_dbu.h |    4 ----

> >  arch/arm/mach-at91/include/mach/sama5d3.h  |    3 +++

> >  arch/arm/mach-at91/include/mach/sama5d4.h  |    3 +++

> >  4 files changed, 10 insertions(+), 7 deletions(-)
Andreas Bießmann Nov. 3, 2015, 1:22 p.m. UTC | #3
Dear Wenyou Yang,

Wenyou Yang <wenyou.yang@atmel.com> writes:
>Provide the specific addresses for the Chip ID and Chip ID Extension
>registers, instead of the offset, which make it use on other chips.
>
>Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
>Reviewed-by: Andreas Bießmann <andreas.devel@googlemail.com>
>---
>
> arch/arm/mach-at91/armv7/cpu.c             |    7 ++++---
> arch/arm/mach-at91/include/mach/at91_dbu.h |    4 ----
> arch/arm/mach-at91/include/mach/sama5d3.h  |    3 +++
> arch/arm/mach-at91/include/mach/sama5d4.h  |    3 +++
> 4 files changed, 10 insertions(+), 7 deletions(-)

applied to u-boot-atmel/master, thanks!

Best regards,
Andreas Bießmann
diff mbox

Patch

diff --git a/arch/arm/mach-at91/armv7/cpu.c b/arch/arm/mach-at91/armv7/cpu.c
index 8d86f97..7843aed 100644
--- a/arch/arm/mach-at91/armv7/cpu.c
+++ b/arch/arm/mach-at91/armv7/cpu.c
@@ -12,7 +12,6 @@ 
 #include <common.h>
 #include <asm/io.h>
 #include <asm/arch/hardware.h>
-#include <asm/arch/at91_dbu.h>
 #include <asm/arch/at91_pmc.h>
 #include <asm/arch/at91_pit.h>
 #include <asm/arch/at91_gpbr.h>
@@ -65,12 +64,14 @@  void enable_caches(void)
 	dcache_enable();
 }
 
+#define ATMEL_CHIPID_CIDR_VERSION	0x1f
+
 unsigned int get_chip_id(void)
 {
-	return readl(ATMEL_BASE_DBGU + AT91_DBU_CIDR) & ~AT91_DBU_CIDR_MASK;
+	return readl(ATMEL_CHIPID_CIDR) & ~ATMEL_CHIPID_CIDR_VERSION;
 }
 
 unsigned int get_extension_chip_id(void)
 {
-	return readl(ATMEL_BASE_DBGU + AT91_DBU_EXID);
+	return readl(ATMEL_CHIPID_EXID);
 }
diff --git a/arch/arm/mach-at91/include/mach/at91_dbu.h b/arch/arm/mach-at91/include/mach/at91_dbu.h
index 7346fc0..3181138 100644
--- a/arch/arm/mach-at91/include/mach/at91_dbu.h
+++ b/arch/arm/mach-at91/include/mach/at91_dbu.h
@@ -35,8 +35,4 @@  typedef struct at91_dbu {
 #define AT91_DBU_CID_ARCH_9xx		0x01900000
 #define AT91_DBU_CID_ARCH_9XExx	0x02900000
 
-#define AT91_DBU_CIDR_MASK		0x1f
-#define AT91_DBU_CIDR			0x40
-#define AT91_DBU_EXID			0x44
-
 #endif
diff --git a/arch/arm/mach-at91/include/mach/sama5d3.h b/arch/arm/mach-at91/include/mach/sama5d3.h
index b749cb3..33f6c97 100644
--- a/arch/arm/mach-at91/include/mach/sama5d3.h
+++ b/arch/arm/mach-at91/include/mach/sama5d3.h
@@ -158,6 +158,9 @@ 
 #define ATMEL_BASE_RTC		0xfffffeb0
 /* Reserved:	0xfffffee0 - 0xffffffff */
 
+#define ATMEL_CHIPID_CIDR	0xffffee40
+#define ATMEL_CHIPID_EXID	0xffffee44
+
 /*
  * Internal Memory.
  */
diff --git a/arch/arm/mach-at91/include/mach/sama5d4.h b/arch/arm/mach-at91/include/mach/sama5d4.h
index 7773ace..3da8aff 100644
--- a/arch/arm/mach-at91/include/mach/sama5d4.h
+++ b/arch/arm/mach-at91/include/mach/sama5d4.h
@@ -144,6 +144,9 @@ 
 #define ATMEL_BASE_PIOE		0xfc06d000
 #define ATMEL_BASE_AIC		0xfc06e000
 
+#define ATMEL_CHIPID_CIDR	0xfc069040
+#define ATMEL_CHIPID_EXID	0xfc069044
+
 /*
  * Internal Memory.
  */