All of lore.kernel.org
 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 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.