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] [day] [month] [year] [list]

Date:	Mon, 17 Feb 2014 14:26:10 +0530
From:	Naveen Krishna Ch <naveenkrishna.ch@...il.com>
To:	Tomasz Figa <t.figa@...sung.com>
Cc:	Kukjin Kim <kgene.kim@...sung.com>,
	Naveen Krishna Chatradhi <ch.naveen@...sung.com>,
	linux-crypto@...r.kernel.org,
	"linux-samsung-soc@...r.kernel.org" 
	<linux-samsung-soc@...r.kernel.org>, linux-kernel@...r.kernel.org,
	Vladimir Zapolskiy <vzapolskiy@...il.com>,
	herbert@...dor.apana.org.au, cpgs@...sung.com
Subject: Re: [PATCH 6/9 v6] ARM: dts: exynos5250/5420: add dt node for sss module

Hello Tomasz,

On 14 February 2014 16:24, Tomasz Figa <t.figa@...sung.com> wrote:
> 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
Sure, i will respin the v6 version of this patchset without using the
In-Reply-To option.
>
> David S. Miller <davem@...emloft.net>
I've added David S.Miller for few of the patches, i will cc him for
all the patches.
>
> to Cc list, as he is also listed as crypto maintainer in MAINTAINERS file.
>
> Best regards,
> Tomasz
Sure, thanks Tomasz.



-- 
Shine bright,
(: Nav :)
--
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/