All of lore.kernel.org
 help / color / mirror / Atom feed
From: Shubhrajyoti Datta <shubhrajyoti.datta@gmail.com>
To: Stephen Boyd <sboyd@kernel.org>
Cc: Miquel Raynal <miquel.raynal@bootlin.com>,
	Shubhrajyoti Datta <shubhrajyoti.datta@xilinx.com>,
	linux-clk@vger.kernel.org,
	OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS 
	<devicetree@vger.kernel.org>,
	Mike Turquette <mturquette@baylibre.com>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Subject: Re: [PATCH v9 1/7] dt-bindings: add documentation of xilinx clocking wizard
Date: Wed, 3 Mar 2021 09:44:14 +0530	[thread overview]
Message-ID: <CAKfKVtEvmMq6YB_0a-P-R2WcHYF3Ri80WYSYo_PFtG_Vxw+cUA@mail.gmail.com> (raw)
In-Reply-To: <161472644984.1478170.5535993198933302108@swboyd.mtv.corp.google.com>

Hi Stephen,

On Wed, Mar 3, 2021 at 4:37 AM Stephen Boyd <sboyd@kernel.org> wrote:
>
> Quoting Shubhrajyoti Datta (2021-02-24 06:10:08)
> > On Fri, Feb 19, 2021 at 6:54 AM Stephen Boyd <sboyd@kernel.org> wrote:
> > >
> > > > > +
> > > > > +  xlnx,speed-grade:
> > > > > +    $ref: /schemas/types.yaml#/definitions/uint32
> > > > > +    enum: [1, 2, 3]
> > > > > +    description:
> > > > > +      Speed grade of the device.
> > > >
> > > > A bit of explanation of what this describes would be welcome.
> > > >
> > > > Don't forget that binding are not tied to any driver implementation,
> > > > these are supposed to be hardware description properties.
> > >
> > > Would opp tables work for this?
> > This is the parameter is for speed of the fabric.
>
> Ok. Yes or no? Is it configuring the speed of the fabric? Sounds like
> assigned-clock-rates or assigned-interconnect-bandwidth or something
> like that.

I do not think we could use opp tables.
Xilinx has products where we have partly FPGA and patrly PS(hardware
is not programmable) so the ip could be in PL
and the processor  in PS.

The speed grade influences a variety of timing parameters in the FPGA,
including fabric (slice), multiplier/DSP48x, BlockRAM, I/O, and other
resources parameters.

Basically the timing of the fabric is determined by it. Also we are
notifying the speed grade to the driver no configuration is done.
We are telling which speed-grade fabric we are running on.

 There is no correlation between these numbers. It is really a
relative metric of performance within a specific family.

  reply	other threads:[~2021-03-03 19:11 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-02-18  4:49 [PATCH v9 0/7] clk: clk-wizard: clock-wizard: Driver updates Shubhrajyoti Datta
2021-02-18  4:49 ` [PATCH v9 1/7] dt-bindings: add documentation of xilinx clocking wizard Shubhrajyoti Datta
2021-02-18  8:28   ` Miquel Raynal
2021-02-19  1:24     ` Stephen Boyd
2021-02-24 14:10       ` Shubhrajyoti Datta
2021-03-02 23:07         ` Stephen Boyd
2021-03-03  4:14           ` Shubhrajyoti Datta [this message]
2021-02-18 14:05   ` Rob Herring
2021-02-18  4:49 ` [PATCH v9 2/7] clk: clock-wizard: Add the clockwizard to clk directory Shubhrajyoti Datta
2021-02-18  8:26   ` Miquel Raynal
2021-02-18  4:49 ` [PATCH v9 3/7] clk: clock-wizard: Fix kernel-doc warning Shubhrajyoti Datta
2021-02-18  8:29   ` Miquel Raynal
2021-02-24 14:10     ` Shubhrajyoti Datta
2021-02-18  4:49 ` [PATCH v9 4/7] clk: clock-wizard: Add support for dynamic reconfiguration Shubhrajyoti Datta
2021-02-18  8:35   ` Miquel Raynal
2021-02-18  4:49 ` [PATCH v9 5/7] clk: clock-wizard: Add support for fractional support Shubhrajyoti Datta
2021-02-18  4:49 ` [PATCH v9 6/7] clk: clock-wizard: Remove the hardcoding of the clock outputs Shubhrajyoti Datta
2021-02-18  8:37   ` Miquel Raynal
2021-02-19  1:25     ` Stephen Boyd
2021-02-22  6:47       ` Shubhrajyoti Datta
2021-02-23  1:01         ` Stephen Boyd
2021-02-24 14:08           ` Shubhrajyoti Datta
2021-02-18  4:49 ` [PATCH v9 7/7] clk: clock-wizard: Update the fixed factor divisors Shubhrajyoti Datta
2021-02-18  8:24 ` [PATCH v9 0/7] clk: clk-wizard: clock-wizard: Driver updates Miquel Raynal
2021-02-24 14:11   ` Shubhrajyoti Datta

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=CAKfKVtEvmMq6YB_0a-P-R2WcHYF3Ri80WYSYo_PFtG_Vxw+cUA@mail.gmail.com \
    --to=shubhrajyoti.datta@gmail.com \
    --cc=devicetree@vger.kernel.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=linux-clk@vger.kernel.org \
    --cc=miquel.raynal@bootlin.com \
    --cc=mturquette@baylibre.com \
    --cc=sboyd@kernel.org \
    --cc=shubhrajyoti.datta@xilinx.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 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.