public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: "Marek Behún" <marek.behun@nic.cz>
To: Weijie Gao <weijie.gao@mediatek.com>, Stefan Roese <sr@denx.de>,
	Tom Rini <trini@konsulko.com>
Cc: <u-boot@lists.denx.de>,
	GSS_MTK_Uboot_upstream <GSS_MTK_Uboot_upstream@mediatek.com>
Subject: Re: [PATCH v3 00/15] Add support for MediaTek MT7621 SoC
Date: Sat, 30 Apr 2022 15:48:04 +0200	[thread overview]
Message-ID: <20220430154804.3593a53e@thinkpad> (raw)
In-Reply-To: <ed7148f9ab85061c8dacf882cade1cc6c9d61f45.camel@mediatek.com>

On Sat, 30 Apr 2022 21:31:15 +0800
Weijie Gao <weijie.gao@mediatek.com> wrote:

> On Fri, 2022-04-29 at 17:13 +0200, Marek Behún wrote:
> > On Fri, 29 Apr 2022 15:59:44 +0800
> > Weijie Gao <weijie.gao@mediatek.com> wrote:
> >   
> > > On Fri, 2022-04-29 at 08:15 +0200, Stefan Roese wrote:  
> > > > Hi Weijie
> > > > 
> > > > On 4/29/22 03:23, Weijie Gao wrote:    
> > > > > Hi Marek,
> > > > > 
> > > > > Yes. MT7621 does need such a proprietary binary for DDR
> > > > > initialization
> > > > > and calibration. That's why I submit only the ram-bootable part
> > > > > here.
> > > > > 
> > > > > I'm considering to create a preloader to load and run DDR
> > > > > init&calib
> > > > > binary and ram-bootable u-boot image. The ram-bootable u-boot
> > > > > image
> > > > > can be appended to the proloader to form the flash-bootable
> > > > > bootloader.
> > > > > 
> > > > > The source code of preloader will be uploaded to github.    
> > > > 
> > > > Just to be sure: You are not implementing this preloader as an U-
> > > > Boot
> > > > SPL loader, as this would conflict with U-Boot's GPL
> > > > compatibility?    
> > > 
> > > Yes. Submitting the source code of this binary also violates MTK's
> > > non-
> > > disclosure agreement.  
> > 
> > Isn't it possible to submit the DDR training code in binary version,
> > and somehow call into it from SPL?  
> 
> It's possible and I've already implemented it. But will u-boot accept
> binary file?

U-Boot at least accepts drivers that require proprietary firmware to
load (bnxt driver, for example).

I don't know whether a binary firmware can be accepted into the U-Boot
repository, whether in a separate file or as an u8 array in a .c file,
but it should at least be possible to make it so that the user can
compile it thsemsevles to be bundled, i.e. create a Kconfig option that
configures a path to the firmware. If it is present, it will be
compiled with DDR training bundled.

Stefan, Tom, can this be done?

BTW, Weijie, can I already test this on mt7621 board, also with DDR
training? Can you send me the patch that adds calling the proprietary
code?

Marek

  reply	other threads:[~2022-04-30 13:48 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-04-28 10:37 [PATCH v3 00/15] Add support for MediaTek MT7621 SoC Weijie Gao
2022-04-28 10:37 ` [PATCH v3 01/15] mips: mtmips: add " Weijie Gao
2022-04-28 10:37 ` [PATCH v3 02/15] mips: mtmips: add two reference boards for mt7621 Weijie Gao
2022-04-29  6:20   ` Stefan Roese
2022-04-29  7:51     ` Weijie Gao
2022-04-28 10:37 ` [PATCH v3 03/15] mips: add support for noncached_alloc() Weijie Gao
2022-04-28 10:37 ` [PATCH v3 04/15] clk: mtmips: add clock driver for MediaTek MT7621 SoC Weijie Gao
2022-04-28 10:37 ` [PATCH v3 05/15] reset: mtmips: add reset controller support " Weijie Gao
2022-04-28 10:37 ` [PATCH v3 06/15] pinctrl: mtmips: add " Weijie Gao
2022-04-28 10:37 ` [PATCH v3 07/15] nand: raw: " Weijie Gao
2022-04-28 10:38 ` [PATCH v3 08/15] usb: xhci-mtk: " Weijie Gao
2022-04-28 10:38 ` [PATCH v3 09/15] phy: mtk-tphy: " Weijie Gao
2022-04-28 10:38 ` [PATCH v3 10/15] spi: " Weijie Gao
2022-04-28 10:38 ` [PATCH v3 11/15] gpio: " Weijie Gao
2022-04-28 10:38 ` [PATCH v3 12/15] watchdog: " Weijie Gao
2022-04-28 10:38 ` [PATCH v3 13/15] mmc: mediatek: " Weijie Gao
2022-04-28 10:38 ` [PATCH v3 14/15] net: " Weijie Gao
2022-04-30 17:13   ` Ramon Fried
2022-05-05  0:44     ` Weijie Gao
2022-04-28 10:38 ` [PATCH v3 15/15] MAINTAINERS: update maintainer for MediaTek MIPS platform Weijie Gao
2022-04-28 16:19 ` [PATCH v3 00/15] Add support for MediaTek MT7621 SoC Marek Behún
2022-04-29  1:23   ` Weijie Gao
2022-04-29  6:15     ` Stefan Roese
2022-04-29  7:59       ` Weijie Gao
2022-04-29 15:13         ` Marek Behún
2022-04-30 13:31           ` Weijie Gao
2022-04-30 13:48             ` Marek Behún [this message]
2022-05-01 12:52               ` Tom Rini
2022-05-05  0:57                 ` Weijie Gao

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=20220430154804.3593a53e@thinkpad \
    --to=marek.behun@nic.cz \
    --cc=GSS_MTK_Uboot_upstream@mediatek.com \
    --cc=sr@denx.de \
    --cc=trini@konsulko.com \
    --cc=u-boot@lists.denx.de \
    --cc=weijie.gao@mediatek.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