From: Daniel Schwierzeck <daniel.schwierzeck@gmail.com>
To: Tom Rini <trini@konsulko.com>, Weijie Gao <weijie.gao@mediatek.com>
Cc: u-boot@lists.denx.de
Subject: Re: [PULL] u-boot-mips for u-boot/next (v2022.10)
Date: Sat, 9 Jul 2022 14:01:01 +0200 [thread overview]
Message-ID: <236ddd8a-3c7f-96fa-7949-b23e114dc8a6@gmail.com> (raw)
In-Reply-To: <20220708165046.GN1146598@bill-the-cat>
Hi Tom,
On 08.07.22 18:50, Tom Rini wrote:
> On Fri, Jul 08, 2022 at 05:21:48PM +0200, Daniel SchwierzeckHi Tom, wrote:
>>
>> Gitlab CI:
>> https://source.denx.de/u-boot/custodians/u-boot-mips/-/pipelines/12656
>>
>> Azure:
>> https://dev.azure.com/danielschwierzeck/u-boot/_build/results?buildId=30&view=results
>>
>>
>> The following changes since commit 2d2c61ff0460740d9ec5a44dbef9255a8c690696:
>>
>> Merge tag 'efi-2022-07-rc7' of https://source.denx.de/u-boot/custodians/u-boot-efi (2022-07-06 09:17:08 -0400)
>>
>> are available in the Git repository at:
>>
>> https://source.denx.de/u-boot/custodians/u-boot-mips.git/ tags/mips-pull-2022-07-08
>>
>> for you to fetch changes up to e5fc4022af3cfd59e3459276305671a595ac5ff0:
>>
>> MAINTAINERS: update maintainer for MediaTek MIPS platform (2022-07-08 15:13:29 +0200)
>>
>> ----------------------------------------------------------------
>> - MIPS: add drivers and board support for Mediatek MT7621 SoC
>
> OK, we need a few changes here, sorry. There's now migrated CONFIG
> symbols, most of which are easy to do and I was about to, and then I saw
> this:
> #ifdef CONFIG_TPL_BUILD
> #define CONFIG_SPL_START_S_PATH "arch/mips/mach-mtmips/mt7621/tpl"
> /* .bss will not be used by TPL */
> #define CONFIG_SPL_BSS_START_ADDR 0x80000000
> #define CONFIG_SPL_BSS_MAX_SIZE 0
> #else
> #define CONFIG_SPL_START_S_PATH "arch/mips/mach-mtmips/mt7621/spl"
> #define CONFIG_SPL_BSS_START_ADDR 0x80140000
> #define CONFIG_SPL_BSS_MAX_SIZE 0x80000
> #define CONFIG_SPL_MAX_SIZE 0x30000
> #endif
>
> No, you cannot abuse CONFIG_TPL_BUILD to set CONFIG_SPL_foo. Those need
> to become CONFIG_TPL_foo, and set appropriately. And then for
> [ST]PL_START_S_PATH, you need to set head-$(CONFIG_ARCH_xxx) to the
> right file, for SPL/TPL instead.
>
do you already have patches for converting stuff like
CONFIG_SPL_BSS_START_ADDR prepared? Than I would wait with the pull
request until those patches are applied to mainline and I would adapt
the MT7621 patches.
I could also assist with converting CONFIG_SPL_START_S_PATH because
that's only used on MIPS and one ARM board.
--
- Daniel
next prev parent reply other threads:[~2022-07-09 12:01 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-07-08 15:21 [PULL] u-boot-mips for u-boot/next (v2022.10) Daniel Schwierzeck
2022-07-08 16:50 ` Tom Rini
2022-07-09 12:01 ` Daniel Schwierzeck [this message]
2022-07-09 12:43 ` Tom Rini
2022-07-09 16:22 ` Daniel Schwierzeck
2022-07-10 8:42 ` Weijie Gao
2022-07-10 10:08 ` 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=236ddd8a-3c7f-96fa-7949-b23e114dc8a6@gmail.com \
--to=daniel.schwierzeck@gmail.com \
--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.