From: Rob Herring <robh@kernel.org>
To: Michal Simek <michal.simek@amd.com>
Cc: Shubhrajyoti Datta <shubhrajyoti.datta@amd.com>,
linux-clk@vger.kernel.org, git@amd.com,
devicetree@vger.kernel.org, krzysztof.kozlowski+dt@linaro.org,
sboyd@kernel.org, mturquette@baylibre.com,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Subject: Re: [PATCH 1/2] dt-bindings: clk: Add binding for versal clocking wizard
Date: Fri, 30 Sep 2022 16:39:24 -0500 [thread overview]
Message-ID: <20220930213924.GA1079711-robh@kernel.org> (raw)
In-Reply-To: <afb3f19f-eb40-5453-a82b-295e06861f86@amd.com>
On Fri, Sep 30, 2022 at 03:00:28PM +0200, Michal Simek wrote:
> Hi Rob,
>
> On 9/30/22 14:25, Rob Herring wrote:
> > On Fri, Sep 30, 2022 at 3:04 AM Shubhrajyoti Datta
> > <shubhrajyoti.datta@amd.com> wrote:
> > >
> > > The Clocking Wizard for Versal adaptive compute acceleration platforms
> > > generates multiple configurable number of clock outputs.
> > > Add device tree binding for Versal clocking wizard support.
> >
> > Really v1? I'm sure I heard of this wizard before.
> >
> > What about this?:
> >
> > drivers/staging/clocking-wizard/dt-binding.txt
> >
> > That needs to be moved out of staging rather than adding a 2nd one.
>
>
> Let me clarify this. This is IP which is already moved out of staging.
> Linux-next has these changes and waiting for MW to happen (already in clock
> tree).
Where does this series explain that? If the dependency is not the latest
rc1, then state that.
> And this is new IP. Not sure who has chosen similar name but this targets
> Xilinx Versal SOCs. Origin one was targeting previous families.
Do we need a whole new schema doc?
It is not ideal to define the same property, xlnx,nr-outputs, more than
once. And it's only a new compatible string.
Rob
next prev parent reply other threads:[~2022-09-30 21:39 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-09-30 8:03 [PATCH 0/2] clocking-wizard: Add versal clocking wizard support Shubhrajyoti Datta
2022-09-30 8:03 ` [PATCH 1/2] dt-bindings: clk: Add binding for versal clocking wizard Shubhrajyoti Datta
2022-09-30 11:19 ` Krzysztof Kozlowski
2022-09-30 12:25 ` Rob Herring
2022-09-30 13:00 ` Michal Simek
2022-09-30 21:39 ` Rob Herring [this message]
2022-10-03 7:15 ` Michal Simek
2022-10-03 7:23 ` Krzysztof Kozlowski
2022-10-03 7:58 ` Michal Simek
2022-10-03 8:10 ` Krzysztof Kozlowski
2022-10-03 8:16 ` Greg Kroah-Hartman
2022-10-03 9:59 ` Michal Simek
2022-10-03 10:41 ` Michal Simek
2022-10-03 14:50 ` Greg Kroah-Hartman
2022-10-03 10:37 ` Michal Simek
2022-10-03 10:50 ` Krzysztof Kozlowski
2022-10-03 15:27 ` Michal Simek
2022-10-04 11:00 ` Krzysztof Kozlowski
2022-10-04 11:04 ` Michal Simek
2022-09-30 8:04 ` [PATCH 2/2] clocking-wizard: Add versal clocking wizard support Shubhrajyoti Datta
2022-09-30 14:39 ` kernel test robot
2022-10-02 2:11 ` kernel test robot
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=20220930213924.GA1079711-robh@kernel.org \
--to=robh@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=git@amd.com \
--cc=gregkh@linuxfoundation.org \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=linux-clk@vger.kernel.org \
--cc=michal.simek@amd.com \
--cc=mturquette@baylibre.com \
--cc=sboyd@kernel.org \
--cc=shubhrajyoti.datta@amd.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).