From: Alex Helms <alexander.helms.jy@renesas.com>
To: Geert Uytterhoeven <geert@linux-m68k.org>
Cc: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS"
<devicetree@vger.kernel.org>,
linux-clk <linux-clk@vger.kernel.org>,
Linux-Renesas <linux-renesas-soc@vger.kernel.org>,
Rob Herring <robh+dt@kernel.org>, Stephen Boyd <sboyd@kernel.org>,
Michael Turquette <mturquette@baylibre.com>,
david.cater.jc@renesas.com,
Michal Simek <michal.simek@xilinx.com>,
Rob Herring <robh@kernel.org>
Subject: Re: [PATCH v6 1/2] dt-bindings: Add binding for Renesas 8T49N241
Date: Tue, 19 Oct 2021 14:52:59 -0700 [thread overview]
Message-ID: <4f2f81a8-9a79-3211-5ec3-fa679c3e7bb9@renesas.com> (raw)
In-Reply-To: <CAMuHMdXq2NyBf539raFJSoWSGXnwxOAMWcVB_WV-=uf+kOs7rw@mail.gmail.com>
On 10/14/2021 5:16 AM, Geert Uytterhoeven wrote:
> Hi Alex,
>
> On Wed, Oct 13, 2021 at 8:02 PM Geert Uytterhoeven <geert@linux-m68k.org> wrote:
>> On Mon, Sep 13, 2021 at 7:05 PM Alex Helms
>> <alexander.helms.jy@renesas.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@renesas.com>
>>> Reviewed-by: Rob Herring <robh@kernel.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
>
next prev parent reply other threads:[~2021-10-19 21:53 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-09-13 17:04 [PATCH v6 0/2] Renesas 8T49N241 device driver Alex Helms
2021-09-13 17:04 ` [PATCH v6 1/2] dt-bindings: Add binding for Renesas 8T49N241 Alex Helms
2021-10-13 18:02 ` Geert Uytterhoeven
2021-10-14 12:16 ` Geert Uytterhoeven
2021-10-19 21:52 ` Alex Helms [this message]
2021-10-20 8:20 ` Geert Uytterhoeven
2021-10-20 16:56 ` Alex Helms
2021-10-20 17:57 ` Geert Uytterhoeven
2021-09-13 17:04 ` [PATCH v6 2/2] clk: Add ccf driver " Alex Helms
2021-09-29 11:43 ` Michal Simek
2021-10-14 12:13 ` Geert Uytterhoeven
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4f2f81a8-9a79-3211-5ec3-fa679c3e7bb9@renesas.com \
--to=alexander.helms.jy@renesas.com \
--cc=david.cater.jc@renesas.com \
--cc=devicetree@vger.kernel.org \
--cc=geert@linux-m68k.org \
--cc=linux-clk@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=michal.simek@xilinx.com \
--cc=mturquette@baylibre.com \
--cc=robh+dt@kernel.org \
--cc=robh@kernel.org \
--cc=sboyd@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.