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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox