Re: [PATCH] Add support for Raspberry Pi 3B+(32bit)
> The both the network cards are included in both configs, surely it > would be less confusing for users to detect the variant and load the > correct DT to ensure the network card does work, there's numerous > examples of this in U-Boot already I think it would be less confusing for users to add a defconfig for 3B+(32bit) for the following reasons. - 3B has defconfig for 32bit and 64bit - The defconfig for 3B and 3B+ is the same except for CONFIG_DEFAULT_DEVICE_TREE, but both are provided Alternatively, I suggest deleting rpi_3_b_plus_defconfig. 2021年4月20日(火) 21:18 Peter Robinson : > > On Tue, Apr 20, 2021 at 11:57 AM Toshifumi NISHINAGA > wrote: > > > > Hello Peter. > > > > > As that config works > > > just fine on all variants of the RPi3 inc the B, B+, A+ and CM3 > > > > Yes. It also boots on Raspberry Pi 3B+ using rpi_3_32b_defconfig, > > but only the network card doesn't work. > > Because the network cards used in the 3B and 3B+ are different. > > The both the network cards are included in both configs, surely it > would be less confusing for users to detect the variant and load the > correct DT to ensure the network card does work, there's numerous > examples of this in U-Boot already > > > This difference is hard to notice, > > so I want to add a new defconfig for Raspberry Pi 3B+. > > > > > > Regards, > > Toshifumi NISHINAGA > > > > 2021年4月20日(火) 19:28 Peter Robinson : > > > > > > On Tue, Apr 20, 2021 at 7:05 AM Toshifumi NISHINAGA > > > wrote: > > > > > > > > Hello all. > > > > > > > > I will add rpi_3_32b_plus_defconfig to easy to build u-boot for > > > > Raspberry Pi 3B+(32bit). > > > > > > What does this provide over rpi_3_32b_defconfig? As that config works > > > just fine on all variants of the RPi3 inc the B, B+, A+ and CM3 I > > > don't see the point in confusing things with an unnecessary config. > > > > > > > Could you merge this patch? > > > > > > > > Thanks, > > > > Toshifumi NISHINAGA > > > > > > > > > > > > From 28d13f48761ab7c137a5fdf07c76a857967e3590 Mon Sep 17 00:00:00 2001 > > > > From: Toshifumi NISHINAGA > > > > Date: Thu, 15 Apr 2021 16:53:33 +0900 > > > > Subject: [PATCH] Add support for Raspberry Pi 3B+(32bit) > > > > MIME-Version: 1.0 > > > > Content-Type: text/plain; charset=UTF-8 > > > > Content-Transfer-Encoding: 8bit > > > > > > > > It also boots on Raspberry Pi 3B+ using rpi_3_32b_defconfig, > > > > but the network card doesn't work. > > > > > > > > This defconfig is based on rpi_3_32b_defconfig. > > > > > > > > Signed-off-by: Toshifumi NISHINAGA > > > > --- > > > > > > > > configs/rpi_3_32b_plus_defconfig | 46 > > > > 1 file changed, 46 insertions(+) > > > > create mode 100644 configs/rpi_3_32b_plus_defconfig > > > > > > > > diff --git a/configs/rpi_3_32b_plus_defconfig > > > > b/configs/rpi_3_32b_plus_defconfig > > > > new file mode 100644 > > > > index 00..d147a46ae7 > > > > --- /dev/null > > > > +++ b/configs/rpi_3_32b_plus_defconfig > > > > @@ -0,0 +1,46 @@ > > > > +CONFIG_ARM=y > > > > +CONFIG_ARCH_CPU_INIT=y > > > > +CONFIG_ARCH_BCM283X=y > > > > +CONFIG_SYS_TEXT_BASE=0x8000 > > > > +CONFIG_TARGET_RPI_3_32B=y > > > > +CONFIG_SYS_MALLOC_F_LEN=0x2000 > > > > +CONFIG_NR_DRAM_BANKS=1 > > > > +CONFIG_ENV_SIZE=0x4000 > > > > +CONFIG_DEFAULT_DEVICE_TREE="bcm2837-rpi-3-b-plus" > > > > +CONFIG_DISTRO_DEFAULTS=y > > > > +CONFIG_OF_BOARD_SETUP=y > > > > +CONFIG_USE_PREBOOT=y > > > > +# CONFIG_DISPLAY_CPUINFO is not set > > > > +# CONFIG_DISPLAY_BOARDINFO is not set > > > > +CONFIG_MISC_INIT_R=y > > > > +CONFIG_SYS_PROMPT="U-Boot> " > > > > +CONFIG_CMD_GPIO=y > > > > +CONFIG_CMD_MMC=y > > > > +CONFIG_CMD_USB=y > > > > +CONFIG_CMD_FS_UUID=y > > > > +CONFIG_OF_EMBED=y > > > > +CONFIG_ENV_FAT_DEVICE_AND_PART="0:1" > > > > +CONFIG_SYS_RELOC_GD_ENV_ADDR=y > > > > +CONFIG_ENV_VARS_UBOOT_RUNTIME_CONFIG=y > > > > +CONFIG_DM_MMC=y > > > > +CONFIG_MMC_SDHCI=y > > > > +CONFIG_MMC_SDHCI_BCM2835=y > > > > +CONFIG_PHYLIB=y > > > > +CONFIG_DM_ETH=y > > > > +CONFIG_PINCTRL=y > > > > +# CONFIG_PINCTRL_GENERIC is not set > > > > +# CONFIG_REQUIRE_SERIAL_CONSOLE is not set > > > > +CONFIG_USB=y > > > > +CONFIG_DM_USB=y > > > > +CONFIG_USB_DWC2=y > > > > +CONFIG_USB_KEYBOARD=y > > > > +CONFIG_USB_HOST_ETHER=y > > > > +CONFIG_USB_ETHER_LAN78XX=y > > > > +CONFIG_USB_ETHER_SMSC95XX=y > > > > +CONFIG_DM_VIDEO=y > > > > +# CONFIG_VIDEO_BPP8 is not set > > > > +# CONFIG_VIDEO_BPP16 is not set > > > > +CONFIG_SYS_WHITE_ON_BLACK=y > > > > +CONFIG_CONSOLE_SCROLL_LINES=10 > > > > +CONFIG_PHYS_TO_BUS=y > > > > +CONFIG_OF_LIBFDT_OVERLAY=y > > > > -- > > > > 2.24.3 (Apple Git-128) > > > > > > > > -- > > Toshifumi NISHINAGA -- Toshifumi NISHINAGA
Re: [PATCH] Add support for Raspberry Pi 3B+(32bit)
Hello Peter. > As that config works > just fine on all variants of the RPi3 inc the B, B+, A+ and CM3 Yes. It also boots on Raspberry Pi 3B+ using rpi_3_32b_defconfig, but only the network card doesn't work. Because the network cards used in the 3B and 3B+ are different. This difference is hard to notice, so I want to add a new defconfig for Raspberry Pi 3B+. Regards, Toshifumi NISHINAGA 2021年4月20日(火) 19:28 Peter Robinson : > > On Tue, Apr 20, 2021 at 7:05 AM Toshifumi NISHINAGA > wrote: > > > > Hello all. > > > > I will add rpi_3_32b_plus_defconfig to easy to build u-boot for > > Raspberry Pi 3B+(32bit). > > What does this provide over rpi_3_32b_defconfig? As that config works > just fine on all variants of the RPi3 inc the B, B+, A+ and CM3 I > don't see the point in confusing things with an unnecessary config. > > > Could you merge this patch? > > > > Thanks, > > Toshifumi NISHINAGA > > > > > > From 28d13f48761ab7c137a5fdf07c76a857967e3590 Mon Sep 17 00:00:00 2001 > > From: Toshifumi NISHINAGA > > Date: Thu, 15 Apr 2021 16:53:33 +0900 > > Subject: [PATCH] Add support for Raspberry Pi 3B+(32bit) > > MIME-Version: 1.0 > > Content-Type: text/plain; charset=UTF-8 > > Content-Transfer-Encoding: 8bit > > > > It also boots on Raspberry Pi 3B+ using rpi_3_32b_defconfig, > > but the network card doesn't work. > > > > This defconfig is based on rpi_3_32b_defconfig. > > > > Signed-off-by: Toshifumi NISHINAGA > > --- > > > > configs/rpi_3_32b_plus_defconfig | 46 > > 1 file changed, 46 insertions(+) > > create mode 100644 configs/rpi_3_32b_plus_defconfig > > > > diff --git a/configs/rpi_3_32b_plus_defconfig > > b/configs/rpi_3_32b_plus_defconfig > > new file mode 100644 > > index 00..d147a46ae7 > > --- /dev/null > > +++ b/configs/rpi_3_32b_plus_defconfig > > @@ -0,0 +1,46 @@ > > +CONFIG_ARM=y > > +CONFIG_ARCH_CPU_INIT=y > > +CONFIG_ARCH_BCM283X=y > > +CONFIG_SYS_TEXT_BASE=0x8000 > > +CONFIG_TARGET_RPI_3_32B=y > > +CONFIG_SYS_MALLOC_F_LEN=0x2000 > > +CONFIG_NR_DRAM_BANKS=1 > > +CONFIG_ENV_SIZE=0x4000 > > +CONFIG_DEFAULT_DEVICE_TREE="bcm2837-rpi-3-b-plus" > > +CONFIG_DISTRO_DEFAULTS=y > > +CONFIG_OF_BOARD_SETUP=y > > +CONFIG_USE_PREBOOT=y > > +# CONFIG_DISPLAY_CPUINFO is not set > > +# CONFIG_DISPLAY_BOARDINFO is not set > > +CONFIG_MISC_INIT_R=y > > +CONFIG_SYS_PROMPT="U-Boot> " > > +CONFIG_CMD_GPIO=y > > +CONFIG_CMD_MMC=y > > +CONFIG_CMD_USB=y > > +CONFIG_CMD_FS_UUID=y > > +CONFIG_OF_EMBED=y > > +CONFIG_ENV_FAT_DEVICE_AND_PART="0:1" > > +CONFIG_SYS_RELOC_GD_ENV_ADDR=y > > +CONFIG_ENV_VARS_UBOOT_RUNTIME_CONFIG=y > > +CONFIG_DM_MMC=y > > +CONFIG_MMC_SDHCI=y > > +CONFIG_MMC_SDHCI_BCM2835=y > > +CONFIG_PHYLIB=y > > +CONFIG_DM_ETH=y > > +CONFIG_PINCTRL=y > > +# CONFIG_PINCTRL_GENERIC is not set > > +# CONFIG_REQUIRE_SERIAL_CONSOLE is not set > > +CONFIG_USB=y > > +CONFIG_DM_USB=y > > +CONFIG_USB_DWC2=y > > +CONFIG_USB_KEYBOARD=y > > +CONFIG_USB_HOST_ETHER=y > > +CONFIG_USB_ETHER_LAN78XX=y > > +CONFIG_USB_ETHER_SMSC95XX=y > > +CONFIG_DM_VIDEO=y > > +# CONFIG_VIDEO_BPP8 is not set > > +# CONFIG_VIDEO_BPP16 is not set > > +CONFIG_SYS_WHITE_ON_BLACK=y > > +CONFIG_CONSOLE_SCROLL_LINES=10 > > +CONFIG_PHYS_TO_BUS=y > > +CONFIG_OF_LIBFDT_OVERLAY=y > > -- > > 2.24.3 (Apple Git-128) -- Toshifumi NISHINAGA
Re: [PATCH] Add support for Raspberry Pi 3B+(32bit)
Hello, Matthias. > This file is already present [1]. Please provide patches against the latest [1] is for the 64bit mode of Raspberry Pi 3B+, not for the 32bit mode. The Raspberry Pi 3B has 64bit [2] and 32bit [3] mode configs. [1]:https://source.denx.de/u-boot/u-boot/-/blob/master/configs/rpi_3_b_plus_defconfig [2]:https://source.denx.de/u-boot/u-boot/-/blob/master/configs/rpi_3_defconfig [3]: https://source.denx.de/u-boot/u-boot/-/blob/master/configs/rpi_3_32b_defconfig Regards, Toshifumi NISHINAGA 2021年4月20日(火) 18:45 Matthias Brugger : > > > > On 20/04/2021 08:05, Toshifumi NISHINAGA wrote: > > Hello all. > > > > I will add rpi_3_32b_plus_defconfig to easy to build u-boot for > > Raspberry Pi 3B+(32bit). > > Could you merge this patch? > > > > This file is already present [1]. Please provide patches against the latest > U-Boot tree. Is there anything missing in the config? > > Regards, > Matthias > > [1] > https://source.denx.de/u-boot/u-boot/-/blob/master/configs/rpi_3_b_plus_defconfig > > > > Thanks, > > Toshifumi NISHINAGA > > > > > > From 28d13f48761ab7c137a5fdf07c76a857967e3590 Mon Sep 17 00:00:00 2001 > > From: Toshifumi NISHINAGA > > Date: Thu, 15 Apr 2021 16:53:33 +0900 > > Subject: [PATCH] Add support for Raspberry Pi 3B+(32bit) > > MIME-Version: 1.0 > > Content-Type: text/plain; charset=UTF-8 > > Content-Transfer-Encoding: 8bit > > > > It also boots on Raspberry Pi 3B+ using rpi_3_32b_defconfig, > > but the network card doesn't work. > > > > This defconfig is based on rpi_3_32b_defconfig. > > > > Signed-off-by: Toshifumi NISHINAGA > > --- > > > > configs/rpi_3_32b_plus_defconfig | 46 > > 1 file changed, 46 insertions(+) > > create mode 100644 configs/rpi_3_32b_plus_defconfig > > > > diff --git a/configs/rpi_3_32b_plus_defconfig > > b/configs/rpi_3_32b_plus_defconfig > > new file mode 100644 > > index 00..d147a46ae7 > > --- /dev/null > > +++ b/configs/rpi_3_32b_plus_defconfig > > @@ -0,0 +1,46 @@ > > +CONFIG_ARM=y > > +CONFIG_ARCH_CPU_INIT=y > > +CONFIG_ARCH_BCM283X=y > > +CONFIG_SYS_TEXT_BASE=0x8000 > > +CONFIG_TARGET_RPI_3_32B=y > > +CONFIG_SYS_MALLOC_F_LEN=0x2000 > > +CONFIG_NR_DRAM_BANKS=1 > > +CONFIG_ENV_SIZE=0x4000 > > +CONFIG_DEFAULT_DEVICE_TREE="bcm2837-rpi-3-b-plus" > > +CONFIG_DISTRO_DEFAULTS=y > > +CONFIG_OF_BOARD_SETUP=y > > +CONFIG_USE_PREBOOT=y > > +# CONFIG_DISPLAY_CPUINFO is not set > > +# CONFIG_DISPLAY_BOARDINFO is not set > > +CONFIG_MISC_INIT_R=y > > +CONFIG_SYS_PROMPT="U-Boot> " > > +CONFIG_CMD_GPIO=y > > +CONFIG_CMD_MMC=y > > +CONFIG_CMD_USB=y > > +CONFIG_CMD_FS_UUID=y > > +CONFIG_OF_EMBED=y > > +CONFIG_ENV_FAT_DEVICE_AND_PART="0:1" > > +CONFIG_SYS_RELOC_GD_ENV_ADDR=y > > +CONFIG_ENV_VARS_UBOOT_RUNTIME_CONFIG=y > > +CONFIG_DM_MMC=y > > +CONFIG_MMC_SDHCI=y > > +CONFIG_MMC_SDHCI_BCM2835=y > > +CONFIG_PHYLIB=y > > +CONFIG_DM_ETH=y > > +CONFIG_PINCTRL=y > > +# CONFIG_PINCTRL_GENERIC is not set > > +# CONFIG_REQUIRE_SERIAL_CONSOLE is not set > > +CONFIG_USB=y > > +CONFIG_DM_USB=y > > +CONFIG_USB_DWC2=y > > +CONFIG_USB_KEYBOARD=y > > +CONFIG_USB_HOST_ETHER=y > > +CONFIG_USB_ETHER_LAN78XX=y > > +CONFIG_USB_ETHER_SMSC95XX=y > > +CONFIG_DM_VIDEO=y > > +# CONFIG_VIDEO_BPP8 is not set > > +# CONFIG_VIDEO_BPP16 is not set > > +CONFIG_SYS_WHITE_ON_BLACK=y > > +CONFIG_CONSOLE_SCROLL_LINES=10 > > +CONFIG_PHYS_TO_BUS=y > > +CONFIG_OF_LIBFDT_OVERLAY=y > > > -- Toshifumi NISHINAGA
[PATCH] Add support for Raspberry Pi 3B+(32bit)
Hello all. I will add rpi_3_32b_plus_defconfig to easy to build u-boot for Raspberry Pi 3B+(32bit). Could you merge this patch? Thanks, Toshifumi NISHINAGA >From 28d13f48761ab7c137a5fdf07c76a857967e3590 Mon Sep 17 00:00:00 2001 From: Toshifumi NISHINAGA Date: Thu, 15 Apr 2021 16:53:33 +0900 Subject: [PATCH] Add support for Raspberry Pi 3B+(32bit) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit It also boots on Raspberry Pi 3B+ using rpi_3_32b_defconfig, but the network card doesn't work. This defconfig is based on rpi_3_32b_defconfig. Signed-off-by: Toshifumi NISHINAGA --- configs/rpi_3_32b_plus_defconfig | 46 1 file changed, 46 insertions(+) create mode 100644 configs/rpi_3_32b_plus_defconfig diff --git a/configs/rpi_3_32b_plus_defconfig b/configs/rpi_3_32b_plus_defconfig new file mode 100644 index 00..d147a46ae7 --- /dev/null +++ b/configs/rpi_3_32b_plus_defconfig @@ -0,0 +1,46 @@ +CONFIG_ARM=y +CONFIG_ARCH_CPU_INIT=y +CONFIG_ARCH_BCM283X=y +CONFIG_SYS_TEXT_BASE=0x8000 +CONFIG_TARGET_RPI_3_32B=y +CONFIG_SYS_MALLOC_F_LEN=0x2000 +CONFIG_NR_DRAM_BANKS=1 +CONFIG_ENV_SIZE=0x4000 +CONFIG_DEFAULT_DEVICE_TREE="bcm2837-rpi-3-b-plus" +CONFIG_DISTRO_DEFAULTS=y +CONFIG_OF_BOARD_SETUP=y +CONFIG_USE_PREBOOT=y +# CONFIG_DISPLAY_CPUINFO is not set +# CONFIG_DISPLAY_BOARDINFO is not set +CONFIG_MISC_INIT_R=y +CONFIG_SYS_PROMPT="U-Boot> " +CONFIG_CMD_GPIO=y +CONFIG_CMD_MMC=y +CONFIG_CMD_USB=y +CONFIG_CMD_FS_UUID=y +CONFIG_OF_EMBED=y +CONFIG_ENV_FAT_DEVICE_AND_PART="0:1" +CONFIG_SYS_RELOC_GD_ENV_ADDR=y +CONFIG_ENV_VARS_UBOOT_RUNTIME_CONFIG=y +CONFIG_DM_MMC=y +CONFIG_MMC_SDHCI=y +CONFIG_MMC_SDHCI_BCM2835=y +CONFIG_PHYLIB=y +CONFIG_DM_ETH=y +CONFIG_PINCTRL=y +# CONFIG_PINCTRL_GENERIC is not set +# CONFIG_REQUIRE_SERIAL_CONSOLE is not set +CONFIG_USB=y +CONFIG_DM_USB=y +CONFIG_USB_DWC2=y +CONFIG_USB_KEYBOARD=y +CONFIG_USB_HOST_ETHER=y +CONFIG_USB_ETHER_LAN78XX=y +CONFIG_USB_ETHER_SMSC95XX=y +CONFIG_DM_VIDEO=y +# CONFIG_VIDEO_BPP8 is not set +# CONFIG_VIDEO_BPP16 is not set +CONFIG_SYS_WHITE_ON_BLACK=y +CONFIG_CONSOLE_SCROLL_LINES=10 +CONFIG_PHYS_TO_BUS=y +CONFIG_OF_LIBFDT_OVERLAY=y -- 2.24.3 (Apple Git-128)
[U-Boot] Please revert unnecessary dcache_disable() commit
Hello, commit 8f079cccb369995e46a2ab530d5d60b88c1e70bb ("armv7m: Disable D-cache when booting nommu(ARMv7M) Linux kernel") added dcache_disable(), but this function is existed at cleanup_before_linux() on arch/arm/cpu/armv7m/cpu.c. Could you revert this commit. thanks, Toshifumi NISHINAGA ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
[U-Boot] [PATCH] armv7m: Disable D-cache when booting nommu(ARMv7M) Linux kernel
Disable D-Cache is required when booting nommu Linux kernel. (please see Linux kernel source "arch/arm/kernel/head-nommu.S") U-Boot is enabled D-cache and I-Cache at startup. However, it does not disable D-Cache before booting nommu Linux kernel. Therefore, I call dcache_disable() when the CPU is ARMv7M to fix this problem. Signed-off-by: Toshifumi NISHINAGA <tnishinaga@gmail.com> --- arch/arm/lib/bootm.c | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/lib/bootm.c b/arch/arm/lib/bootm.c index 4dbe6a5303..d14a8b5737 100644 --- a/arch/arm/lib/bootm.c +++ b/arch/arm/lib/bootm.c @@ -359,6 +359,7 @@ static void boot_jump_linux(bootm_headers_t *images, int flag) #ifdef CONFIG_CPU_V7M ulong addr = (ulong)kernel_entry | 1; kernel_entry = (void *)addr; + dcache_disable(); #endif s = getenv("machid"); if (s) { -- 2.13.0 ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot