* [PULL] u-boot-mips for u-boot/next (v2022.10)
@ 2022-07-08 15:21 Daniel Schwierzeck
2022-07-08 16:50 ` Tom Rini
0 siblings, 1 reply; 7+ messages in thread
From: Daniel Schwierzeck @ 2022-07-08 15:21 UTC (permalink / raw)
To: u-boot, Tom Rini
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
----------------------------------------------------------------
Weijie Gao (25):
mips: add asm/mipsmtregs.h for MIPS multi-threading
mips: add more definitions for asm/cm.h
mips: add __image_copy_len for SPL linker script
mips: add support for noncached_alloc()
mips: mtmips: add support for MediaTek MT7621 SoC
mips: mtmips: add two reference boards for mt7621
doc: mediatek: add documentation for mt7621 reference boards
clk: mtmips: add clock driver for MediaTek MT7621 SoC
reset: mtmips: add reset controller support for MediaTek MT7621 SoC
pinctrl: mtmips: add support for MediaTek MT7621 SoC
usb: xhci-mtk: add support for MediaTek MT7621 SoC
phy: mtk-tphy: add support for MediaTek MT7621 SoC
spi: add support for MediaTek MT7621 SoC
gpio: add support for MediaTek MT7621 SoC
watchdog: add support for MediaTek MT7621 SoC
mmc: mediatek: add support for MediaTek MT7621 SoC
net: mediatek: remap iobase address
net: mediatek: use regmap api to modify ethsys registers
net: mediatek: add support for MediaTek MT7621 SoC
nand: raw: add support for MediaTek MT7621 SoC
spl: allow using nand base without standard nand driver
spl: spl_legacy: fix the use of SPL_COPY_PAYLOAD_ONLY
spl: nand: support loading legacy image with payload compressed
tools: mtk_image: add support for MT7621 NAND images
MAINTAINERS: update maintainer for MediaTek MIPS platform
MAINTAINERS | 8 +
arch/mips/cpu/u-boot-spl.lds | 3 +
arch/mips/dts/Makefile | 2 +
arch/mips/dts/mediatek,mt7621-nand-rfb.dts | 67 ++
arch/mips/dts/mediatek,mt7621-rfb.dts | 82 ++
arch/mips/dts/mt7621-u-boot.dtsi | 111 +++
arch/mips/dts/mt7621.dtsi | 349 +++++++
arch/mips/include/asm/cm.h | 67 ++
arch/mips/include/asm/mipsmtregs.h | 142 +++
arch/mips/include/asm/system.h | 20 +
arch/mips/lib/cache.c | 43 +
arch/mips/mach-mtmips/Kconfig | 49 +-
arch/mips/mach-mtmips/Makefile | 4 +
arch/mips/mach-mtmips/cpu.c | 2 +-
arch/mips/mach-mtmips/mt7621/Kconfig | 115 +++
arch/mips/mach-mtmips/mt7621/Makefile | 14 +
arch/mips/mach-mtmips/mt7621/init.c | 246 +++++
arch/mips/mach-mtmips/mt7621/mt7621.h | 229 +++++
arch/mips/mach-mtmips/mt7621/serial.c | 23 +
arch/mips/mach-mtmips/mt7621/spl/Makefile | 9 +
arch/mips/mach-mtmips/mt7621/spl/cps.c | 153 +++
arch/mips/mach-mtmips/mt7621/spl/dram.c | 153 +++
arch/mips/mach-mtmips/mt7621/spl/dram.h | 39 +
arch/mips/mach-mtmips/mt7621/spl/launch.c | 100 ++
arch/mips/mach-mtmips/mt7621/spl/launch.h | 52 +
arch/mips/mach-mtmips/mt7621/spl/launch_ll.S | 339 +++++++
arch/mips/mach-mtmips/mt7621/spl/serial.c | 24 +
arch/mips/mach-mtmips/mt7621/spl/spl.c | 95 ++
arch/mips/mach-mtmips/mt7621/spl/start.S | 226 +++++
arch/mips/mach-mtmips/mt7621/sram_init.S | 22 +
arch/mips/mach-mtmips/mt7621/tpl/Makefile | 4 +
arch/mips/mach-mtmips/mt7621/tpl/start.S | 161 ++++
arch/mips/mach-mtmips/mt7621/tpl/tpl.c | 144 +++
board/mediatek/mt7621/MAINTAINERS | 8 +
board/mediatek/mt7621/Makefile | 3 +
board/mediatek/mt7621/board.c | 6 +
common/spl/Kconfig | 2 +-
common/spl/spl_legacy.c | 21 +-
common/spl/spl_nand.c | 27 +
configs/mt7621_nand_rfb_defconfig | 85 ++
configs/mt7621_rfb_defconfig | 82 ++
doc/board/index.rst | 1 +
doc/board/mediatek/index.rst | 9 +
doc/board/mediatek/mt7621.rst | 48 +
drivers/clk/mtmips/Makefile | 1 +
drivers/clk/mtmips/clk-mt7621.c | 288 ++++++
drivers/gpio/Kconfig | 2 +-
drivers/mmc/mtk-sd.c | 13 +
drivers/mtd/nand/raw/Kconfig | 17 +-
drivers/mtd/nand/raw/Makefile | 2 +
drivers/mtd/nand/raw/mt7621_nand.c | 1205 ++++++++++++++++++++++++
drivers/mtd/nand/raw/mt7621_nand.h | 29 +
drivers/mtd/nand/raw/mt7621_nand_spl.c | 237 +++++
drivers/net/mtk_eth.c | 45 +-
drivers/phy/Kconfig | 2 +-
drivers/pinctrl/mtmips/Kconfig | 9 +
drivers/pinctrl/mtmips/Makefile | 1 +
drivers/pinctrl/mtmips/pinctrl-mt7621.c | 306 ++++++
drivers/pinctrl/mtmips/pinctrl-mtmips-common.c | 4 +-
drivers/pinctrl/mtmips/pinctrl-mtmips-common.h | 12 +
drivers/spi/Kconfig | 2 +-
drivers/usb/host/Kconfig | 2 +-
drivers/watchdog/Kconfig | 2 +-
include/configs/mt7621.h | 65 ++
include/dt-bindings/clock/mt7621-clk.h | 46 +
include/dt-bindings/reset/mt7621-reset.h | 38 +
tools/mtk_image.c | 182 ++++
tools/mtk_image.h | 24 +
68 files changed, 5887 insertions(+), 36 deletions(-)
create mode 100644 arch/mips/dts/mediatek,mt7621-nand-rfb.dts
create mode 100644 arch/mips/dts/mediatek,mt7621-rfb.dts
create mode 100644 arch/mips/dts/mt7621-u-boot.dtsi
create mode 100644 arch/mips/dts/mt7621.dtsi
create mode 100644 arch/mips/include/asm/mipsmtregs.h
create mode 100644 arch/mips/mach-mtmips/mt7621/Kconfig
create mode 100644 arch/mips/mach-mtmips/mt7621/Makefile
create mode 100644 arch/mips/mach-mtmips/mt7621/init.c
create mode 100644 arch/mips/mach-mtmips/mt7621/mt7621.h
create mode 100644 arch/mips/mach-mtmips/mt7621/serial.c
create mode 100644 arch/mips/mach-mtmips/mt7621/spl/Makefile
create mode 100644 arch/mips/mach-mtmips/mt7621/spl/cps.c
create mode 100644 arch/mips/mach-mtmips/mt7621/spl/dram.c
create mode 100644 arch/mips/mach-mtmips/mt7621/spl/dram.h
create mode 100644 arch/mips/mach-mtmips/mt7621/spl/launch.c
create mode 100644 arch/mips/mach-mtmips/mt7621/spl/launch.h
create mode 100644 arch/mips/mach-mtmips/mt7621/spl/launch_ll.S
create mode 100644 arch/mips/mach-mtmips/mt7621/spl/serial.c
create mode 100644 arch/mips/mach-mtmips/mt7621/spl/spl.c
create mode 100644 arch/mips/mach-mtmips/mt7621/spl/start.S
create mode 100644 arch/mips/mach-mtmips/mt7621/sram_init.S
create mode 100644 arch/mips/mach-mtmips/mt7621/tpl/Makefile
create mode 100644 arch/mips/mach-mtmips/mt7621/tpl/start.S
create mode 100644 arch/mips/mach-mtmips/mt7621/tpl/tpl.c
create mode 100644 board/mediatek/mt7621/MAINTAINERS
create mode 100644 board/mediatek/mt7621/Makefile
create mode 100644 board/mediatek/mt7621/board.c
create mode 100644 configs/mt7621_nand_rfb_defconfig
create mode 100644 configs/mt7621_rfb_defconfig
create mode 100644 doc/board/mediatek/index.rst
create mode 100644 doc/board/mediatek/mt7621.rst
create mode 100644 drivers/clk/mtmips/clk-mt7621.c
create mode 100644 drivers/mtd/nand/raw/mt7621_nand.c
create mode 100644 drivers/mtd/nand/raw/mt7621_nand.h
create mode 100644 drivers/mtd/nand/raw/mt7621_nand_spl.c
create mode 100644 drivers/pinctrl/mtmips/pinctrl-mt7621.c
create mode 100644 include/configs/mt7621.h
create mode 100644 include/dt-bindings/clock/mt7621-clk.h
create mode 100644 include/dt-bindings/reset/mt7621-reset.h
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PULL] u-boot-mips for u-boot/next (v2022.10)
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
0 siblings, 1 reply; 7+ messages in thread
From: Tom Rini @ 2022-07-08 16:50 UTC (permalink / raw)
To: Daniel Schwierzeck, Weijie Gao; +Cc: u-boot
[-- Attachment #1: Type: text/plain, Size: 1808 bytes --]
On Fri, Jul 08, 2022 at 05:21:48PM +0200, Daniel Schwierzeck 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.
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PULL] u-boot-mips for u-boot/next (v2022.10)
2022-07-08 16:50 ` Tom Rini
@ 2022-07-09 12:01 ` Daniel Schwierzeck
2022-07-09 12:43 ` Tom Rini
0 siblings, 1 reply; 7+ messages in thread
From: Daniel Schwierzeck @ 2022-07-09 12:01 UTC (permalink / raw)
To: Tom Rini, Weijie Gao; +Cc: u-boot
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
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PULL] u-boot-mips for u-boot/next (v2022.10)
2022-07-09 12:01 ` Daniel Schwierzeck
@ 2022-07-09 12:43 ` Tom Rini
2022-07-09 16:22 ` Daniel Schwierzeck
0 siblings, 1 reply; 7+ messages in thread
From: Tom Rini @ 2022-07-09 12:43 UTC (permalink / raw)
To: Daniel Schwierzeck; +Cc: Weijie Gao, u-boot
[-- Attachment #1: Type: text/plain, Size: 2703 bytes --]
On Sat, Jul 09, 2022 at 02:01:01PM +0200, Daniel Schwierzeck wrote:
> 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.
See what's in -next already? SPL_BSS_START_ADDR is migrated, but there
were no TPL_BSS_START_ADDR cases. For START_S_PATH, the platform just
needs to be reworked as I suggested above I believe, to achieve the
desired result.
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PULL] u-boot-mips for u-boot/next (v2022.10)
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
0 siblings, 2 replies; 7+ messages in thread
From: Daniel Schwierzeck @ 2022-07-09 16:22 UTC (permalink / raw)
To: Tom Rini; +Cc: Weijie Gao, u-boot
On 09.07.22 14:43, Tom Rini wrote:
> On Sat, Jul 09, 2022 at 02:01:01PM +0200, Daniel Schwierzeck wrote:
>> 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.
>
> See what's in -next already? SPL_BSS_START_ADDR is migrated, but there
> were no TPL_BSS_START_ADDR cases. For START_S_PATH, the platform just
> needs to be reworked as I suggested above I believe, to achieve the
> desired result.
>
sorry, didn't check the latest updates in -next and the series was too
long on the list ;)
TPL_BSS_START_ADDR shouldn't be necessary because MT7621 doesn't use BSS
in TPL, the defined values where just dummy values.
I rechecked and removed all migrated Kconfig options from mt7621.h and
pushed an update to u-boot-mips/next. Weijie could you verify that? If
all is okay, I'll prepare a new pull request, otherwise please send me a
v7 patch series.
--
- Daniel
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PULL] u-boot-mips for u-boot/next (v2022.10)
2022-07-09 16:22 ` Daniel Schwierzeck
@ 2022-07-10 8:42 ` Weijie Gao
2022-07-10 10:08 ` Weijie Gao
1 sibling, 0 replies; 7+ messages in thread
From: Weijie Gao @ 2022-07-10 8:42 UTC (permalink / raw)
To: Daniel Schwierzeck, Tom Rini; +Cc: u-boot
Hi Daniel,
On Sat, 2022-07-09 at 18:22 +0200, Daniel Schwierzeck wrote:
>
> On 09.07.22 14:43, Tom Rini wrote:
> > On Sat, Jul 09, 2022 at 02:01:01PM +0200, Daniel Schwierzeck wrote:
> > > 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/-/pi
> > > > > pelines/12656
> > > > >
> > > > > Azure:
> > > > > https://dev.azure.com/danielschwierzeck/u-boot/_build/res
> > > > > ults?buildId=30&view=results
> > > > >
> > > > >
> > > > > The following changes since commit
> > > > > 2d2c61ff0460740d9ec5a44dbef9255a8c690696:
> > > > >
> > > > > Merge tag 'efi-2022-07-rc7' of https://source.denx.de/u-b
> > > > > oot/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.
> >
> > See what's in -next already? SPL_BSS_START_ADDR is migrated, but
> > there
> > were no TPL_BSS_START_ADDR cases. For START_S_PATH, the platform
> > just
> > needs to be reworked as I suggested above I believe, to achieve the
> > desired result.
> >
>
> sorry, didn't check the latest updates in -next and the series was
> too
> long on the list ;)
>
> TPL_BSS_START_ADDR shouldn't be necessary because MT7621 doesn't use
> BSS
> in TPL, the defined values where just dummy values.
>
> I rechecked and removed all migrated Kconfig options from mt7621.h
> and
> pushed an update to u-boot-mips/next. Weijie could you verify that?
> If
> all is okay, I'll prepare a new pull request, otherwise please send
> me a
> v7 patch series.
>
I've test the latest fixups on u-boot-mips/next, and all is ok. thx.
Best Regards,
Weijie
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PULL] u-boot-mips for u-boot/next (v2022.10)
2022-07-09 16:22 ` Daniel Schwierzeck
2022-07-10 8:42 ` Weijie Gao
@ 2022-07-10 10:08 ` Weijie Gao
1 sibling, 0 replies; 7+ messages in thread
From: Weijie Gao @ 2022-07-10 10:08 UTC (permalink / raw)
To: Daniel Schwierzeck; +Cc: Tom Rini, Weijie Gao, u-boot
Hi Daniel,
This is my personal email address.
I found that my previous mail send from my company was still be marked
as spam by gmail.
I've tested the latest fixups on u-boot-mips/next, and all is ok. thx.
Best Regards,
Weijie
Daniel Schwierzeck <daniel.schwierzeck@gmail.com> 于2022年7月10日周日 00:23写道:
>
>
>
> On 09.07.22 14:43, Tom Rini wrote:
> > On Sat, Jul 09, 2022 at 02:01:01PM +0200, Daniel Schwierzeck wrote:
> >> 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.
> >
> > See what's in -next already? SPL_BSS_START_ADDR is migrated, but there
> > were no TPL_BSS_START_ADDR cases. For START_S_PATH, the platform just
> > needs to be reworked as I suggested above I believe, to achieve the
> > desired result.
> >
>
> sorry, didn't check the latest updates in -next and the series was too
> long on the list ;)
>
> TPL_BSS_START_ADDR shouldn't be necessary because MT7621 doesn't use BSS
> in TPL, the defined values where just dummy values.
>
> I rechecked and removed all migrated Kconfig options from mt7621.h and
> pushed an update to u-boot-mips/next. Weijie could you verify that? If
> all is okay, I'll prepare a new pull request, otherwise please send me a
> v7 patch series.
>
> --
> - Daniel
--
_______________________________________________
lede-devel mailing list
lede-devel@lists.infradead.org
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2022-07-10 10:08 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
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
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
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.