Re: [U-Boot] [PATCH v4 2/4] cpu: exynos4: add ace sha base address

2014-03-25 Thread Przemyslaw Marczak

Hello Minkyu,

On 03/25/2014 02:26 AM, Minkyu Kang wrote:

On 24/03/14 16:44, Przemyslaw Marczak wrote:

Hello Minkyu,

On 03/22/2014 04:18 PM, Minkyu Kang wrote:

Dear Przemyslaw Marczak,


On 21 March 2014 17:56, Przemyslaw Marczak p.marc...@samsung.com wrote:


Signed-off-by: Przemyslaw Marczak p.marc...@samsung.com
Cc: Minkyu Kang mk7.k...@samsung.com

---
Changes v3:
- new commit - after separate changes from next commit

Changes v4:
- none
---
   arch/arm/include/asm/arch-exynos/cpu.h | 4 ++--
   1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/include/asm/arch-exynos/cpu.h
b/arch/arm/include/asm/arch-exynos/cpu.h
index bccce63..bd3300a 100644
--- a/arch/arm/include/asm/arch-exynos/cpu.h
+++ b/arch/arm/include/asm/arch-exynos/cpu.h
@@ -44,11 +44,11 @@
   #define EXYNOS4_MODEM_BASE 0x13A0
   #define EXYNOS4_USBPHY_CONTROL 0x10020704
   #define EXYNOS4_I2S_BASE   0xE210
+#define EXYNOS4_ACE_SFR_BASE   0x1083



Could you please align this list?




I am not sure why this is not aligned in patch - it was generated by git format patch. 
The source was aligned and after apply this patch by git am the code is also aligned.


hm, sorry to misunderstanding.
It means the ordering.
I want to keep ordering of this list by base address. (although it looks 
already broken)

  #define EXYNOS4_DMC_CTRL_BASE 0x1040
+#define EXYNOS4_ACE_SFR_BASE   0x1083
  #define EXYNOS4_GPIO_PART2_BASE   0x1100

Thanks,
Minkyu Kang.



ok, I will move those lines for proper address order.

Thanks
--
Przemyslaw Marczak
Samsung RD Institute Poland
Samsung Electronics
p.marc...@samsung.com
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v4 2/4] cpu: exynos4: add ace sha base address

2014-03-24 Thread Przemyslaw Marczak

Hello Minkyu,

On 03/22/2014 04:18 PM, Minkyu Kang wrote:

Dear Przemyslaw Marczak,


On 21 March 2014 17:56, Przemyslaw Marczak p.marc...@samsung.com wrote:


Signed-off-by: Przemyslaw Marczak p.marc...@samsung.com
Cc: Minkyu Kang mk7.k...@samsung.com

---
Changes v3:
- new commit - after separate changes from next commit

Changes v4:
- none
---
  arch/arm/include/asm/arch-exynos/cpu.h | 4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/include/asm/arch-exynos/cpu.h
b/arch/arm/include/asm/arch-exynos/cpu.h
index bccce63..bd3300a 100644
--- a/arch/arm/include/asm/arch-exynos/cpu.h
+++ b/arch/arm/include/asm/arch-exynos/cpu.h
@@ -44,11 +44,11 @@
  #define EXYNOS4_MODEM_BASE 0x13A0
  #define EXYNOS4_USBPHY_CONTROL 0x10020704
  #define EXYNOS4_I2S_BASE   0xE210
+#define EXYNOS4_ACE_SFR_BASE   0x1083



Could you please align this list?




I am not sure why this is not aligned in patch - it was generated by 
git format patch. The source was aligned and after apply this patch by 
git am the code is also aligned.


Thanks
--
Przemyslaw Marczak
Samsung RD Institute Poland
Samsung Electronics
p.marc...@samsung.com
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v4 2/4] cpu: exynos4: add ace sha base address

2014-03-24 Thread Minkyu Kang
On 24/03/14 16:44, Przemyslaw Marczak wrote:
 Hello Minkyu,
 
 On 03/22/2014 04:18 PM, Minkyu Kang wrote:
 Dear Przemyslaw Marczak,


 On 21 March 2014 17:56, Przemyslaw Marczak p.marc...@samsung.com wrote:

 Signed-off-by: Przemyslaw Marczak p.marc...@samsung.com
 Cc: Minkyu Kang mk7.k...@samsung.com

 ---
 Changes v3:
 - new commit - after separate changes from next commit

 Changes v4:
 - none
 ---
   arch/arm/include/asm/arch-exynos/cpu.h | 4 ++--
   1 file changed, 2 insertions(+), 2 deletions(-)

 diff --git a/arch/arm/include/asm/arch-exynos/cpu.h
 b/arch/arm/include/asm/arch-exynos/cpu.h
 index bccce63..bd3300a 100644
 --- a/arch/arm/include/asm/arch-exynos/cpu.h
 +++ b/arch/arm/include/asm/arch-exynos/cpu.h
 @@ -44,11 +44,11 @@
   #define EXYNOS4_MODEM_BASE 0x13A0
   #define EXYNOS4_USBPHY_CONTROL 0x10020704
   #define EXYNOS4_I2S_BASE   0xE210
 +#define EXYNOS4_ACE_SFR_BASE   0x1083


 Could you please align this list?


 
 I am not sure why this is not aligned in patch - it was generated by git 
 format patch. The source was aligned and after apply this patch by git am 
 the code is also aligned.

hm, sorry to misunderstanding.
It means the ordering.
I want to keep ordering of this list by base address. (although it looks 
already broken) 

 #define EXYNOS4_DMC_CTRL_BASE  0x1040
+#define EXYNOS4_ACE_SFR_BASE   0x1083
 #define EXYNOS4_GPIO_PART2_BASE0x1100

Thanks,
Minkyu Kang.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v4 2/4] cpu: exynos4: add ace sha base address

2014-03-22 Thread Minkyu Kang
Dear Przemyslaw Marczak,


On 21 March 2014 17:56, Przemyslaw Marczak p.marc...@samsung.com wrote:

 Signed-off-by: Przemyslaw Marczak p.marc...@samsung.com
 Cc: Minkyu Kang mk7.k...@samsung.com

 ---
 Changes v3:
 - new commit - after separate changes from next commit

 Changes v4:
 - none
 ---
  arch/arm/include/asm/arch-exynos/cpu.h | 4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)

 diff --git a/arch/arm/include/asm/arch-exynos/cpu.h
 b/arch/arm/include/asm/arch-exynos/cpu.h
 index bccce63..bd3300a 100644
 --- a/arch/arm/include/asm/arch-exynos/cpu.h
 +++ b/arch/arm/include/asm/arch-exynos/cpu.h
 @@ -44,11 +44,11 @@
  #define EXYNOS4_MODEM_BASE 0x13A0
  #define EXYNOS4_USBPHY_CONTROL 0x10020704
  #define EXYNOS4_I2S_BASE   0xE210
 +#define EXYNOS4_ACE_SFR_BASE   0x1083


Could you please align this list?



  #define EXYNOS4_GPIO_PART4_BASEDEVICE_NOT_AVAILABLE
  #define EXYNOS4_DP_BASEDEVICE_NOT_AVAILABLE
  #define EXYNOS4_SPI_ISP_BASE   DEVICE_NOT_AVAILABLE
 -#define EXYNOS4_ACE_SFR_BASE   DEVICE_NOT_AVAILABLE
  #define EXYNOS4_DMC_PHY_BASE   DEVICE_NOT_AVAILABLE
  #define EXYNOS4_AUDIOSS_BASE   DEVICE_NOT_AVAILABLE
  #define EXYNOS4_USB_HOST_XHCI_BASE DEVICE_NOT_AVAILABLE
 @@ -80,6 +80,7 @@
  #define EXYNOS4X12_UART_BASE   0x1380
  #define EXYNOS4X12_I2C_BASE0x1386
  #define EXYNOS4X12_PWMTIMER_BASE   0x139D
 +#define EXYNOS4X12_ACE_SFR_BASE0x1083


ditto.



  #define EXYNOS4X12_ADC_BASEDEVICE_NOT_AVAILABLE
  #define EXYNOS4X12_DP_BASE DEVICE_NOT_AVAILABLE
 @@ -87,7 +88,6 @@
  #define EXYNOS4X12_I2S_BASEDEVICE_NOT_AVAILABLE
  #define EXYNOS4X12_SPI_BASEDEVICE_NOT_AVAILABLE
  #define EXYNOS4X12_SPI_ISP_BASEDEVICE_NOT_AVAILABLE
 -#define EXYNOS4X12_ACE_SFR_BASEDEVICE_NOT_AVAILABLE
  #define EXYNOS4X12_DMC_PHY_BASEDEVICE_NOT_AVAILABLE
  #define EXYNOS4X12_AUDIOSS_BASEDEVICE_NOT_AVAILABLE
  #define EXYNOS4X12_USB_HOST_XHCI_BASE  DEVICE_NOT_AVAILABLE
 --
 1.9.0

 ___
 U-Boot mailing list
 U-Boot@lists.denx.de
 http://lists.denx.de/mailman/listinfo/u-boot




-- 
from. prom.
www.promsoft.net
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH v4 2/4] cpu: exynos4: add ace sha base address

2014-03-21 Thread Przemyslaw Marczak
Signed-off-by: Przemyslaw Marczak p.marc...@samsung.com
Cc: Minkyu Kang mk7.k...@samsung.com

---
Changes v3:
- new commit - after separate changes from next commit

Changes v4:
- none
---
 arch/arm/include/asm/arch-exynos/cpu.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/include/asm/arch-exynos/cpu.h 
b/arch/arm/include/asm/arch-exynos/cpu.h
index bccce63..bd3300a 100644
--- a/arch/arm/include/asm/arch-exynos/cpu.h
+++ b/arch/arm/include/asm/arch-exynos/cpu.h
@@ -44,11 +44,11 @@
 #define EXYNOS4_MODEM_BASE 0x13A0
 #define EXYNOS4_USBPHY_CONTROL 0x10020704
 #define EXYNOS4_I2S_BASE   0xE210
+#define EXYNOS4_ACE_SFR_BASE   0x1083
 
 #define EXYNOS4_GPIO_PART4_BASEDEVICE_NOT_AVAILABLE
 #define EXYNOS4_DP_BASEDEVICE_NOT_AVAILABLE
 #define EXYNOS4_SPI_ISP_BASE   DEVICE_NOT_AVAILABLE
-#define EXYNOS4_ACE_SFR_BASE   DEVICE_NOT_AVAILABLE
 #define EXYNOS4_DMC_PHY_BASE   DEVICE_NOT_AVAILABLE
 #define EXYNOS4_AUDIOSS_BASE   DEVICE_NOT_AVAILABLE
 #define EXYNOS4_USB_HOST_XHCI_BASE DEVICE_NOT_AVAILABLE
@@ -80,6 +80,7 @@
 #define EXYNOS4X12_UART_BASE   0x1380
 #define EXYNOS4X12_I2C_BASE0x1386
 #define EXYNOS4X12_PWMTIMER_BASE   0x139D
+#define EXYNOS4X12_ACE_SFR_BASE0x1083
 
 #define EXYNOS4X12_ADC_BASEDEVICE_NOT_AVAILABLE
 #define EXYNOS4X12_DP_BASE DEVICE_NOT_AVAILABLE
@@ -87,7 +88,6 @@
 #define EXYNOS4X12_I2S_BASEDEVICE_NOT_AVAILABLE
 #define EXYNOS4X12_SPI_BASEDEVICE_NOT_AVAILABLE
 #define EXYNOS4X12_SPI_ISP_BASEDEVICE_NOT_AVAILABLE
-#define EXYNOS4X12_ACE_SFR_BASEDEVICE_NOT_AVAILABLE
 #define EXYNOS4X12_DMC_PHY_BASEDEVICE_NOT_AVAILABLE
 #define EXYNOS4X12_AUDIOSS_BASEDEVICE_NOT_AVAILABLE
 #define EXYNOS4X12_USB_HOST_XHCI_BASE  DEVICE_NOT_AVAILABLE
-- 
1.9.0

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v4 2/4] cpu: exynos4: add ace sha base address

2014-03-21 Thread Tom Rini
On Fri, Mar 21, 2014 at 09:56:18AM +0100, Przemyslaw Marczak wrote:

 Signed-off-by: Przemyslaw Marczak p.marc...@samsung.com
 Cc: Minkyu Kang mk7.k...@samsung.com

Reviewed-by: Tom Rini tr...@ti.com

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot