linux-kernel - Re: [PATCH 6/9 v6] ARM: dts: exynos5250/5420: add dt node for sss module

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-id: <52FDF5FA.4000909@samsung.com>
Date:	Fri, 14 Feb 2014 11:54:50 +0100
From:	Tomasz Figa <t.figa@...sung.com>
To:	Kukjin Kim <kgene.kim@...sung.com>,
	Naveen Krishna Chatradhi <ch.naveen@...sung.com>
Cc:	linux-crypto@...r.kernel.org, linux-samsung-soc@...r.kernel.org,
	linux-kernel@...r.kernel.org, vzapolskiy@...il.com,
	herbert@...dor.apana.org.au, naveenkrishna.ch@...il.com,
	cpgs@...sung.com
Subject: Re: [PATCH 6/9 v6] ARM: dts: exynos5250/5420: add dt node for sss
 module

Hi Kukjin,

On 14.02.2014 00:28, Kukjin Kim wrote:
> On 02/07/14 14:24, Naveen Krishna Chatradhi wrote:
>> This patch adds the device tree node for SSS module
>> found on Exynos5420 and Exynos5250
>>
>> Signed-off-by: Naveen Krishna Chatradhi<ch.naveen@...sung.com>
>> Reviewed-by: Tomasz Figa<t.figa@...sung.com>
>> TO:<linux-samsung-soc@...r.kernel.org>
>> CC: Kukjin Kim<kgene.kim@...sung.com>
>> CC:<linux-crypto@...r.kernel.org>
>> ---
>> changes since v5:
>> 1. Added Reviewed-by: Tomasz Figa<t.figa@...sung.com>
>>
>>   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@...30000 {
>> +        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@...30000 {
>> +        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 <davem@...emloft.net>

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

Best regards,
Tomasz
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/