linux-kernel - Re: [PATCH v6 1/2] dt-bindings: Add binding for Renesas 8T49N241

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: <4f2f81a8-9a79-3211-5ec3-fa679c3e7bb9@renesas.com>
Date:   Tue, 19 Oct 2021 14:52:59 -0700
From:   Alex Helms <alexander.helms.jy@...esas.com>
To:     Geert Uytterhoeven <geert@...ux-m68k.org>
Cc:     Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" 
        <devicetree@...r.kernel.org>,
        linux-clk <linux-clk@...r.kernel.org>,
        Linux-Renesas <linux-renesas-soc@...r.kernel.org>,
        Rob Herring <robh+dt@...nel.org>,
        Stephen Boyd <sboyd@...nel.org>,
        Michael Turquette <mturquette@...libre.com>,
        david.cater.jc@...esas.com, Michal Simek <michal.simek@...inx.com>,
        Rob Herring <robh@...nel.org>
Subject: Re: [PATCH v6 1/2] dt-bindings: Add binding for Renesas 8T49N241

On 10/14/2021 5:16 AM, Geert Uytterhoeven wrote:
> Hi Alex,
> 
> On Wed, Oct 13, 2021 at 8:02 PM Geert Uytterhoeven <geert@...ux-m68k.org> wrote:
>> On Mon, Sep 13, 2021 at 7:05 PM Alex Helms
>> <alexander.helms.jy@...esas.com> wrote:
>>> Renesas 8T49N241 has 4 outputs, 1 integral and 3 fractional dividers.
>>> The 8T49N241 accepts up to two differential or single-ended input clocks
>>> and a fundamental-mode crystal input. The internal PLL can lock to either
>>> of the input reference clocks or to the crystal to behave as a frequency
>>> synthesizer.
>>>
>>> Signed-off-by: Alex Helms <alexander.helms.jy@...esas.com>
>>> Reviewed-by: Rob Herring <robh@...nel.org>
>>
>> Thanks for your patch!
>>
>>> --- /dev/null
>>> +++ b/Documentation/devicetree/bindings/clock/renesas,8t49n241.yaml
> 
>> BTW, do you plan to add interrupt and/or GPIO support later?
> 
> To clarify, and I really meant to add:
> 
>   interrupts:
>     maxItems: 1
> 
> to the bindings now, and GPIO-related properties and subnodes later.

Any additional features such as interrupts and GPIO properties would only be added if there is customer demand for such features. Since there is no interrupt support, does the "interrupts" item still need to be added to the yaml?

-Alex

> 
> Gr{oetje,eeting}s,
> 
>                         Geert
>