From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-19.2 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id D72CBC432BE for ; Tue, 17 Aug 2021 14:07:31 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id BF70E60EFE for ; Tue, 17 Aug 2021 14:07:31 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S240075AbhHQOIC (ORCPT ); Tue, 17 Aug 2021 10:08:02 -0400 Received: from smtp-relay-canonical-1.canonical.com ([185.125.188.121]:45074 "EHLO smtp-relay-canonical-1.canonical.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234446AbhHQOIA (ORCPT ); Tue, 17 Aug 2021 10:08:00 -0400 Received: from mail-ej1-f72.google.com (mail-ej1-f72.google.com [209.85.218.72]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by smtp-relay-canonical-1.canonical.com (Postfix) with ESMTPS id 09E2E40663 for ; Tue, 17 Aug 2021 14:07:26 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=canonical.com; s=20210705; t=1629209246; bh=jfQmfS4UcdDZ/PX6E9b//KKI404jw/6q3qCv8x9JyfM=; h=Subject:To:Cc:References:From:Message-ID:Date:MIME-Version: In-Reply-To:Content-Type; b=FNJjlWJ7AdupXru0q1ASDnxbgDiy7a+L5o0IYI7vSbSkb5dWt9LuQqz840yc1Y2PP M8Bbg7mPRkjClIRrvBv0gchREXMURyymrqnclUcET/iynKGpA8p2Fci0J+fiV6mL10 Kr8NjCDu8vpLg3ZtLNDHC1ac29/JZJ5WPdRq09LaYXZoEKSX++qLuvSz2vOjBEsqfV 7cuOAVjFytCCZ1ZOVLPxptODEnqjNJAjw01C5QIDWOcE7Ppn2Z8aS24ACSltB9qXuO j9eMuipnFMMN8TuVTDF0IGATwZqcRbh8eywnc8NJoy+qIG+Wy/LG3ScwiDNkv0AwvY ZOBRspTIHdzRw== Received: by mail-ej1-f72.google.com with SMTP id h17-20020a1709070b11b02905b5ced62193so6068965ejl.1 for ; Tue, 17 Aug 2021 07:07:26 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=jfQmfS4UcdDZ/PX6E9b//KKI404jw/6q3qCv8x9JyfM=; b=e9aSp3rA+nNXEZIji1bX4DQINfF6QtAGzgGURf329lb1+AoEDHl0xPONOTyLn3EHpx nVSRNqxityu5yrbJ51hKQw3PpGrRgaxMqktBgKsqwzU3HJx9hkA8hUdJ6IR3XNihHjJ8 59IIbFzZLc+EcP+FCdm29CfDFdYs+qfZFqCcyvXY/zGlcIvvSmmS/Ls0f5Si3ZKLJk+r +9g9xrVREHh2GLZ8Ukwn8aC8eNTvf8+D2k0l0zpN/jpb4ncoRbsUGKMqucKAwo4pOykJ GXj9nlTUELfzU4kvb1TVlZNmdFduHbRwNV8ZFWlqaN1dJ0luhehJZKwFp86K61whEkjp y24A== X-Gm-Message-State: AOAM533SMjVC5573/m5yqGV+twOUF5lBsmyUqnU7+hXg4aHcYWOP4H1i ho6lJ76+kSx9e2pOq/n1nSqHhAXzs1c2QNteQDBSruAz0QBmn18OGKuIuYmL2+rZox6gp8FBiy+ mzX54XCuHkwP5RMqGIJeH+wVC2eXPS5s7FdGR0fAw3A== X-Received: by 2002:a05:6402:35c9:: with SMTP id z9mr4369314edc.249.1629209243911; Tue, 17 Aug 2021 07:07:23 -0700 (PDT) X-Google-Smtp-Source: ABdhPJyszAFp2YNN5KP+dS332IXz7EIzwQDE7qq9Fr9F3rL6OyeZKaqCv4G6V53FVjOklm6MHgpHCA== X-Received: by 2002:a05:6402:35c9:: with SMTP id z9mr4369292edc.249.1629209243737; Tue, 17 Aug 2021 07:07:23 -0700 (PDT) Received: from [192.168.8.102] ([86.32.42.198]) by smtp.gmail.com with ESMTPSA id m6sm1049328edc.82.2021.08.17.07.07.21 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 17 Aug 2021 07:07:22 -0700 (PDT) Subject: Re: [PATCH 2/2] dt-bindings: rng: convert Samsung Exynos TRNG to dtschema To: Lukasz Stelmach Cc: Thomas Gleixner , Marc Zyngier , Rob Herring , Matt Mackall , Herbert Xu , linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, linux-crypto@vger.kernel.org References: From: Krzysztof Kozlowski Message-ID: Date: Tue, 17 Aug 2021 16:07:20 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.11.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 17/08/2021 14:34, Lukasz Stelmach wrote: > It was <2021-08-17 wto 12:05>, when Krzysztof Kozlowski wrote: >> On 17/08/2021 11:55, Lukasz Stelmach wrote: >>> It was <2021-08-11 śro 10:43>, when Krzysztof Kozlowski wrote: >>>> Convert Samsung Exynos SoC True Random Number Generator bindings to DT >>>> schema format using json-schema. >>>> >>>> Signed-off-by: Krzysztof Kozlowski >>>> --- >>>> .../bindings/rng/samsung,exynos5250-trng.txt | 17 ------- >>>> .../bindings/rng/samsung,exynos5250-trng.yaml | 44 +++++++++++++++++++ >>>> MAINTAINERS | 2 +- >>>> 3 files changed, 45 insertions(+), 18 deletions(-) >>>> delete mode 100644 Documentation/devicetree/bindings/rng/samsung,exynos5250-trng.txt >>>> create mode 100644 Documentation/devicetree/bindings/rng/samsung,exynos5250-trng.yaml >>>> >>>> diff --git >>>> a/Documentation/devicetree/bindings/rng/samsung,exynos5250-trng.txt >>>> b/Documentation/devicetree/bindings/rng/samsung,exynos5250-trng.txt >>>> deleted file mode 100644 >>>> index 5a613a4ec780..000000000000 >>>> --- a/Documentation/devicetree/bindings/rng/samsung,exynos5250-trng.txt >>>> +++ /dev/null >>>> @@ -1,17 +0,0 @@ >>>> -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"; >>>> - }; >>>> diff --git >>>> a/Documentation/devicetree/bindings/rng/samsung,exynos5250-trng.yaml >>>> b/Documentation/devicetree/bindings/rng/samsung,exynos5250-trng.yaml >>>> new file mode 100644 >>>> index 000000000000..a50c34d5d199 >>>> --- /dev/null >>>> +++ b/Documentation/devicetree/bindings/rng/samsung,exynos5250-trng.yaml >>>> @@ -0,0 +1,44 @@ >>>> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) >>>> +%YAML 1.2 >>>> +--- >>>> +$id: >>>> https://protect2.fireeye.com/v1/url?k=f38ca35b-ac179a0d-f38d2814-0cc47a31ce52-1faa1ecb65482b8a&q=1&e=8b3490f9-a5fc-4da0-b2ee-7b0aec781403&u=http%3A%2F%2Fdevicetree.org%2Fschemas%2Frng%2Fsamsung%2Cexynos5250-trng.yaml%23 >>>> +$schema: >>>> https://protect2.fireeye.com/v1/url?k=9409519d-cb9268cb-9408dad2-0cc47a31ce52-12394c4409905980&q=1&e=8b3490f9-a5fc-4da0-b2ee-7b0aec781403&u=http%3A%2F%2Fdevicetree.org%2Fmeta-schemas%2Fcore.yaml%23 >>>> + >>>> +title: Samsung Exynos SoC True Random Number Generator >>>> + >>>> +maintainers: >>>> + - Krzysztof Kozlowski >>>> + - Łukasz Stelmach >>>> + >>>> +properties: >>>> + compatible: >>>> + const: samsung,exynos5250-trng >>>> + >>>> + clocks: >>>> + maxItems: 1 >>> >>> How about copying description from above into the description: property? >> >> But what to copy? There is no description except generic clock bindings. >> > > The description that "was" in the txt file. But there was no description of fields except copy&paste of the core schema. Do you describe C code like: ... /* unsigned int is a integer number greater or equal 0 */ unsigned int i; ... Best regards, Krzysztof