mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Jason Gunthorpe <jgunthorpe@obsidianresearch.com>
To: "Nadathur, Sundar" <sundar.nadathur@intel.com>
Cc: "Li, Yi" <yi1.li@linux.intel.com>,
	Moritz Fischer <moritz.fischer@ettus.com>,
	Alan Tull <atull@kernel.org>,
	"linux-fpga@vger.kernel.org" <linux-fpga@vger.kernel.org>,
	linux-kernel <linux-kernel@vger.kernel.org>
Subject: Re: [RFC] Add a parser in fpga_region to decode the tlv meta data suggested by Sundar
Date: Fri, 24 Feb 2017 15:00:59 -0700	[thread overview]
Message-ID: <20170224220059.GB29289@obsidianresearch.com> (raw)
In-Reply-To: <1CC272501B5BC543A05DB90AA509DED50B25C9@fmsmsx122.amr.corp.intel.com>

On Fri, Feb 24, 2017 at 09:41:12PM +0000, Nadathur, Sundar wrote:

> I have talked about the need for structs and arrays, potentially
> nested, without really explaining why we may need them
> eventually. I'll get to that in a moment. But, can we agree that, if
> nested structs, arrays and general extensibility is needed, FDT or
> TLVs would be the way to go? If we agree, it becomes a FDT vs. TLV
> discussion.
> 
> Here's why I think we may need extensibility. (Pardon me for
> starting from the basics -- just trying to set the background.) A
> FPGA can be programmed as a whole, or can have Partial
> Reconfiguration (PR) regions, which can be programmed
> independently. The image programmed into a PR region may have
> components and sub-units in general. These components may have their
> own properties. For example, if the FPGA is exposed as a PCIe device
> to the host, the components may have their own registers in MMIO,
> DMA attributes and/or interrupts. They may also have identifiers
> describing their function. In general, we may want these attributes
> and properties, on a per-component basis, in the metadata. Even if
> we don't need them tomorrow, as data center and IOT needs grow, we
> are likely to need that going forward.

So, you are imagining that the FPGA header would include more than
just information relative to programming it, but also something to do
with operating it?

If that is sensible, then I would go with fdt - it is proven to be
able to handle such complex and open-ended needs and has supporting
tooling.

But.. it is crossing over into the DT overlay stuff, if a complex FPGA
like that is a DT overlay + bitfile then we do not need such data in
the bitfile header.

Jason

      reply	other threads:[~2017-02-24 22:23 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1487957340-26578-1-git-send-email-yi1.li@linux.intel.com>
2017-02-24 18:08 ` Nadathur, Sundar
     [not found] ` <20170224180848.GB22491@obsidianresearch.com>
     [not found]   ` <CAAtXAHeViHdXJ3w2Feef4Gj9txKyha6pE0empemRMKB9tCMr-w@mail.gmail.com>
     [not found]     ` <20170224184320.GA21794@obsidianresearch.com>
     [not found]       ` <190c92e5-8ea9-c19f-d40d-894d99c7305e@linux.intel.com>
     [not found]         ` <20170224210209.GA28016@obsidianresearch.com>
     [not found]           ` <bce373a4-37db-cce5-7990-5afab41eceff@linux.intel.com>
     [not found]             ` <20170224211655.GC28016@obsidianresearch.com>
2017-02-24 21:41               ` Nadathur, Sundar
2017-02-24 22:00                 ` Jason Gunthorpe [this message]

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=20170224220059.GB29289@obsidianresearch.com \
    --to=jgunthorpe@obsidianresearch.com \
    --cc=atull@kernel.org \
    --cc=linux-fpga@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=moritz.fischer@ettus.com \
    --cc=sundar.nadathur@intel.com \
    --cc=yi1.li@linux.intel.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

all inboxes | Powered by JetHome®