From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753377AbdLEKEg convert rfc822-to-8bit (ORCPT ); Tue, 5 Dec 2017 05:04:36 -0500 Received: from mail.kernel.org ([198.145.29.99]:49126 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753179AbdLEKEb (ORCPT ); Tue, 5 Dec 2017 05:04:31 -0500 DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 0820F2199D Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=krzk@kernel.org X-Google-Smtp-Source: AGs4zMYqzhpp95Orv70jxYRRyZutdN8V6hxhuiR9UntZ5PSQ0NomEnQRBDuwqgRpn8GSHFeSx/BEQ1U+BbVTtuCKWeM= MIME-Version: 1.0 In-Reply-To: <878tehbjyu.fsf%l.stelmach@samsung.com> References: <878tehbjyu.fsf%l.stelmach@samsung.com> From: Krzysztof Kozlowski Date: Tue, 5 Dec 2017 11:04:27 +0100 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH v3 1/3] dt-bindings: hwrng: Add Samsung Exynos 5250+ True RNG bindings To: =?UTF-8?Q?=C5=81ukasz_Stelmach?= Cc: Rob Herring , "Andrew F . Davis" , PrasannaKumar Muralidharan , Matt Mackall , Herbert Xu , Kukjin Kim , devicetree@vger.kernel.org, linux-crypto@vger.kernel.org, linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org, Marek Szyprowski , Bartlomiej Zolnierkiewicz Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Dec 5, 2017 at 10:30 AM, Łukasz Stelmach wrote: > It was <2017-12-04 pon 14:13>, when Krzysztof Kozlowski wrote: >> On Mon, Dec 4, 2017 at 1:53 PM, Łukasz Stelmach wrote: >>> Add binding documentation for the True Random Number Generator >>> found on Samsung Exynos 5250+ SoCs. >>> >>> Signed-off-by: Łukasz Stelmach >>> --- >>> .../devicetree/bindings/rng/samsung,exynos5250-trng.txt | 17 +++++++++++++++++ >>> 1 file changed, 17 insertions(+) >>> create mode 100644 Documentation/devicetree/bindings/rng/samsung,exynos5250-trng.txt >>> >>> diff --git >>> a/Documentation/devicetree/bindings/rng/samsung,exynos5250-trng.txt >>> b/Documentation/devicetree/bindings/rng/samsung,exynos5250-trng.txt >>> new file mode 100644 >>> index 000000000000..5a613a4ec780 >>> --- /dev/null >>> +++ b/Documentation/devicetree/bindings/rng/samsung,exynos5250-trng.txt >>> @@ -0,0 +1,17 @@ >>> +Exynos True Random Number Generator >>> + >>> +Required properties: >>> + >>> +- compatible : Should be "samsung,exynos5250-trng". >>> +- reg : Specifies base physical address and size of the registers map. >>> +- clocks : Phandle to clock-controller plus clock-specifier pair. >>> +- clock-names : "secss" as a clock name. >>> + >>> +Example: >>> + >>> + rng@10830600 { >>> + compatible = "samsung,exynos5250-trng"; >>> + reg = <0x10830600 0x100>; >>> + clocks = <&clock CLK_SSS>; >>> + clock-names = "secss"; >>> + }; >>> -- >>> 2.11.0 >> >> Mine and Rob's tags disappeared and I think you did not introduce any >> major changes here, right? > > A very experienced kernel developer adviced me to remove them. In that case: Reviewed-by: Krzysztof Kozlowski BR, Krzysztof