All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marek Vasut <marex@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v3] socfpga: Modify qts-filter args to allow input for bsp generated files and quartus project directories
Date: Wed, 13 Jan 2016 03:34:09 +0100	[thread overview]
Message-ID: <201601130334.09918.marex@denx.de> (raw)
In-Reply-To: <1452651818.2108.17.camel@altera.com>

On Wednesday, January 13, 2016 at 03:23:38 AM, Chin Liang See wrote:
> On Wed, 2016-01-13 at 03:07 +0100, Marek Vasut wrote:
> > On Wednesday, January 13, 2016 at 03:05:40 AM, Chin Liang See wrote:
> > > On Tue, 2016-01-12 at 03:48 +0100, Marek Vasut wrote:
> > > > On Tuesday, January 12, 2016 at 01:50:18 AM, Chin Liang See
> > > > 
> > > > wrote:
> > > > > On Tue, 2016-01-12 at 01:31 +0100, Marek Vasut wrote:
> > > > > > On Tuesday, January 12, 2016 at 01:26:58 AM, Chin Liang See
> > > > > > 
> > > > > > wrote:
> > > > > > > On Mon, 2016-01-11 at 15:33 -0800, Dalon Westergreen wrote:
> > > > > [...]
> > > > > 
> > > > > > btw. Completely off-topic, but is there any chance altera
> > > > > > will
> > > > > > release the
> > > > > > algorithm to compute these magic values which are in the
> > > > > > header
> > > > > > files
> > > > > > from
> > > > > > the base values inserted into the HPS component in QSys ?
> > > > > 
> > > > > I would not suggesting that as Qsys and Quartus are doing
> > > > > heavylifting
> > > > > tasks there. The value would depends on the options being
> > > > > choosed,
> > > > > device type, device revision (if any). They also being patched
> > > > > from
> > > > > time to time too.
> > > > 
> > > > Is it all really _that_ complicated ? That's why I'd like to see
> > > > the
> > > > code
> > > > that's doing all that computation.
> > > 
> > > Actually more challenges is that the code is proprietary. We might
> > > get
> > > stopped by legal team before making this happen too.
> > 
> > What's proprietary about it ? Isn't that SDRAM controller made in
> > -house
> > by Altera ?
> 
> Oh seems we might discussing slightly different things. I am referring
> to IOCSR which is the bitstream for setting up the IO buffers. I
> presume you are referring to SDRAM configuration, right?

Yeah, I'm talking about the SDRAM controller configuration values.

I know the IOCSR is a problem. I am reverse engineering the Cyclone IV
bitstream format in my free time now and then, but I didn't have time
to look at C V and I don't plan to do it any soon. I think even my C/IV
activity might trigger someone in Altera once I get around to releasing
it ;-)

> > > > Even if it's done at compile-time, it'd
> > > > still be better than the horrible headers which we have to use
> > > > now.
> > > 
> > > I believe DTS would be better format than header file.
> > 
> > Coming up with sensible bindings would be hard though.
> 
> Yah, that why this is get rid when we switch to Arria 10 SoC :)

Well you'll still need to configure the HardIP which does the DDR calib
with some DDR config values, so you cannot get rid of it entirely. Unless
you pull these values from SPD, which I doubt would be the case, since
the A10 would be mostly embedded chip.

Best regards,
Marek Vasut

  reply	other threads:[~2016-01-13  2:34 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-01-11 23:33 [U-Boot] [PATCH v3] socfpga: Modify qts-filter args to allow input for bsp generated files and quartus project directories Dalon Westergreen
2016-01-12  0:23 ` Marek Vasut
2016-01-12  0:26 ` Chin Liang See
2016-01-12  0:31   ` Marek Vasut
2016-01-12  0:50     ` Chin Liang See
2016-01-12  2:48       ` Marek Vasut
2016-01-13  2:05         ` Chin Liang See
2016-01-13  2:07           ` Marek Vasut
2016-01-13  2:23             ` Chin Liang See
2016-01-13  2:34               ` Marek Vasut [this message]
2016-01-13  2:45                 ` Chin Liang See
2016-01-13  2:55                   ` Marek Vasut

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=201601130334.09918.marex@denx.de \
    --to=marex@denx.de \
    --cc=u-boot@lists.denx.de \
    /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.