Hi Kukjin,

On 14.02.2014 00:28, Kukjin Kim wrote:

On 02/07/14 14:24, Naveen Krishna Chatradhi wrote:

quoted

This patch adds the device tree node for SSS module
found on Exynos5420 and Exynos5250

Signed-off-by: Naveen Krishna Chatradhi<redacted>
Reviewed-by: Tomasz Figa<redacted>
TO:[off-list ref]
CC: Kukjin Kim<redacted>
CC:[off-list ref]
---
changes since v5:
1. Added Reviewed-by: Tomasz Figa[off-list ref]

  arch/arm/boot/dts/exynos5250.dtsi |    8 ++++++++
  arch/arm/boot/dts/exynos5420.dtsi |   10 ++++++++++
  2 files changed, 18 insertions(+)

diff --git a/arch/arm/boot/dts/exynos5250.dtsi

b/arch/arm/boot/dts/exynos5250.dtsi
index b7dec41..46b04e8 100644

--- a/arch/arm/boot/dts/exynos5250.dtsi
+++ b/arch/arm/boot/dts/exynos5250.dtsi

@@ -706,4 +706,12 @@
          io-channel-ranges;
          status = "disabled";
      };
+
+    sss@10830000 {
+        compatible = "samsung,exynos4210-secss";
+        reg =<0x10830000 0x10000>;
+        interrupts =<0 112 0>;
+        clocks =<&clock 348>;
+        clock-names = "secss";
+    };
  };

diff --git a/arch/arm/boot/dts/exynos5420.dtsi

b/arch/arm/boot/dts/exynos5420.dtsi
index 8db792b..b503e96 100644

--- a/arch/arm/boot/dts/exynos5420.dtsi
+++ b/arch/arm/boot/dts/exynos5420.dtsi

@@ -652,4 +652,14 @@
          clocks =<&clock 319>,<&clock 318>;
          clock-names = "tmu_apbif", "tmu_triminfo_apbif";
      };
+
+    sss@10830000 {
+        compatible = "samsung,exynos4210-secss";
+        reg =<0x10830000 0x10000>;
+        interrupts =<0 112 0>;
+        clocks =<&clock 471>;
+        clock-names = "secss";
+        samsung,power-domain =<&g2d_pd>;
+    };
+
  };

Applied, thanks.

BTW, I think the numbering is strange...maybe I missed something?
[PATCH 5/5], [PATCH 5/6 V2], [PATCH 6/8 V3] and this [PATCH 6/9 V6]

I would wait with applying any patches from this series until they are 
acked by crypto subsystem maintainer and DT bindings by DT maintainers.

I'd like Naveen to resend this series in separate thread, with proper 
message threading, so we can make sure that we are not missing anything. 
Naveen, please also add

David S. Miller [off-list ref]

to Cc list, as he is also listed as crypto maintainer in MAINTAINERS file.

Best regards,
Tomasz