2019-01-09 20:58:47

by Otto Sabart

[permalink] [raw]
Subject: [PATCH 1/1] MAINTAINERS: fix bad file reference into Documentation

There is no `samsung,exynos5250-trng.txt` file in the documentation and
I do not think it ever existed.

Signed-off-by: Otto Sabart <[email protected]>
---
MAINTAINERS | 1 -
1 file changed, 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 32d444476a90..609451bf2b1a 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -13309,7 +13309,6 @@ M: Łukasz Stelmach <[email protected]>
L: [email protected]
S: Maintained
F: drivers/char/hw_random/exynos-trng.c
-F: Documentation/devicetree/bindings/rng/samsung,exynos5250-trng.txt

SAMSUNG FRAMEBUFFER DRIVER
M: Jingoo Han <[email protected]>
--
2.17.2


Attachments:
(No filename) (662.00 B)
signature.asc (849.00 B)
Download all attachments

2019-01-09 21:21:54

by Joe Perches

[permalink] [raw]
Subject: Re: [PATCH 1/1] MAINTAINERS: fix bad file reference into Documentation

On Wed, 2019-01-09 at 20:09 +0100, Otto Sabart wrote:
> There is no `samsung,exynos5250-trng.txt` file in the documentation and
> I do not think it ever existed.

There was a patch for this

https://patchwork.kernel.org/patch/10623153/

But is is still unapplied...

> diff --git a/MAINTAINERS b/MAINTAINERS
[]
> @@ -13309,7 +13309,6 @@ M: Łukasz Stelmach <[email protected]>
> L: [email protected]
> S: Maintained
> F: drivers/char/hw_random/exynos-trng.c
> -F: Documentation/devicetree/bindings/rng/samsung,exynos5250-trng.txt



2019-01-10 08:28:06

by Łukasz Stelmach

[permalink] [raw]
Subject: Re: [PATCH 1/1] MAINTAINERS: fix bad file reference into Documentation

It was <2019-01-09 śro 20:35>, when Joe Perches wrote:
> On Wed, 2019-01-09 at 20:09 +0100, Otto Sabart wrote:
>> There is no `samsung,exynos5250-trng.txt` file in the documentation and
>> I do not think it ever existed.
>
> There was a patch for this
>
> https://patchwork.kernel.org/patch/10623153/
>
> But is is still unapplied...
>
>> diff --git a/MAINTAINERS b/MAINTAINERS
> []
>> @@ -13309,7 +13309,6 @@ M: Łukasz Stelmach <[email protected]>
>> L: [email protected]
>> S: Maintained
>> F: drivers/char/hw_random/exynos-trng.c
>> -F: Documentation/devicetree/bindings/rng/samsung,exynos5250-trng.txt

+ Rob Herring

--
Łukasz Stelmach
Samsung R&D Institute Poland
Samsung Electronics


Attachments:
signature.asc (497.00 B)

2019-01-10 14:15:22

by Rob Herring

[permalink] [raw]
Subject: Re: [PATCH 1/1] MAINTAINERS: fix bad file reference into Documentation

On Thu, Jan 10, 2019 at 2:23 AM Łukasz Stelmach <[email protected]> wrote:
>
> It was <2019-01-09 śro 20:35>, when Joe Perches wrote:
> > On Wed, 2019-01-09 at 20:09 +0100, Otto Sabart wrote:
> >> There is no `samsung,exynos5250-trng.txt` file in the documentation and
> >> I do not think it ever existed.
> >
> > There was a patch for this
> >
> > https://patchwork.kernel.org/patch/10623153/
> >
> > But is is still unapplied...
> >
> >> diff --git a/MAINTAINERS b/MAINTAINERS
> > []
> >> @@ -13309,7 +13309,6 @@ M: Łukasz Stelmach <[email protected]>
> >> L: [email protected]
> >> S: Maintained
> >> F: drivers/char/hw_random/exynos-trng.c
> >> -F: Documentation/devicetree/bindings/rng/samsung,exynos5250-trng.txt
>
> + Rob Herring

Please resend the above patch. If I've acked it, then I'm assuming
it's going in via the driver tree. But I'll apply it if you tell me to
instead.

Rob

2019-01-11 14:21:14

by Łukasz Stelmach

[permalink] [raw]
Subject: [PATCH v5.1 RESEND] dt-bindings: hwrng: Add Samsung Exynos 5250+ True RNG bindings

Add binding documentation for the True Random Number Generator
found on Samsung Exynos 5250+ SoCs.

Acked-by: Rob Herring <[email protected]>
Reviewed-by: Krzysztof Kozlowski <[email protected]>
Signed-off-by: Łukasz Stelmach <[email protected]>
---
.../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.20.1


2019-03-26 11:44:58

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH v5.1 RESEND] dt-bindings: hwrng: Add Samsung Exynos 5250+ True RNG bindings

On Fri, 11 Jan 2019 at 14:22, Łukasz Stelmach <[email protected]> wrote:
>
> Add binding documentation for the True Random Number Generator
> found on Samsung Exynos 5250+ SoCs.
>
> Acked-by: Rob Herring <[email protected]>
> Reviewed-by: Krzysztof Kozlowski <[email protected]>
> Signed-off-by: Łukasz Stelmach <[email protected]>
> ---

Rob,
Could you apply this directly? You acked this some time ago but it
never went through drivers tree. Lukasz resent this patch and it is
waiting since then.
The driver implementing compatible is already in mainline.

Best regards,
Krzysztof

2019-10-23 08:18:01

by Andreas Färber

[permalink] [raw]
Subject: Re: [PATCH v5.1 RESEND] dt-bindings: hwrng: Add Samsung Exynos 5250+ True RNG bindings

Hi guys,

Am 26.03.19 um 12:42 schrieb Krzysztof Kozlowski:
> On Fri, 11 Jan 2019 at 14:22, Łukasz Stelmach <[email protected]> wrote:
>>
>> Add binding documentation for the True Random Number Generator
>> found on Samsung Exynos 5250+ SoCs.
>>
>> Acked-by: Rob Herring <[email protected]>
>> Reviewed-by: Krzysztof Kozlowski <[email protected]>
>> Signed-off-by: Łukasz Stelmach <[email protected]>
>> ---
>
> Rob,
> Could you apply this directly? You acked this some time ago but it
> never went through drivers tree. Lukasz resent this patch and it is
> waiting since then.
> The driver implementing compatible is already in mainline.

For some reason this text file in linux-next is lonely in devicetree/...
rather than living in Documentation/devicetree/... - please fix that.
The patch here looks correct, so not sure what went wrong:

https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/devicetree/bindings/rng/samsung,exynos5250-trng.txt?h=next-20191023&id=85552c22f03c9066c33f26f34538b67fee6a91a8

Thanks,
Andreas

--
SUSE Software Solutions Germany GmbH
Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Felix Imendörffer
HRB 36809 (AG Nürnberg)

2019-10-23 08:46:33

by Herbert Xu

[permalink] [raw]
Subject: Re: [PATCH v5.1 RESEND] dt-bindings: hwrng: Add Samsung Exynos 5250+ True RNG bindings

On Wed, Oct 23, 2019 at 10:16:48AM +0200, Andreas F?rber wrote:
>
> For some reason this text file in linux-next is lonely in devicetree/...
> rather than living in Documentation/devicetree/... - please fix that.
> The patch here looks correct, so not sure what went wrong:
>
> https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/devicetree/bindings/rng/samsung,exynos5250-trng.txt?h=next-20191023&id=85552c22f03c9066c33f26f34538b67fee6a91a8

It's because the patch

https://patchwork.kernel.org/patch/11181265/

was generated at the wrong level (p0 instead of p1).

I'll fix this up in my tree. Thanks for the heads up.

Cheers,
--
Email: Herbert Xu <[email protected]>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt