Re: [Buildroot] [PATCH 1/1] configs/stm32f769_disco_sd_defconfig: new defconfig

* Re: [Buildroot] [PATCH 1/1] configs/stm32f769_disco_sd_defconfig: new defconfig
       [not found] <CABGWkvoH-sy4LhPtWgY422FXm6pQ9UR5eM63nV-LAmEf1FRFng@mail.gmail.com>
@ 2023-04-22 16:56 ` Giulio Benetti
  2023-04-23 17:20   ` Dario Binacchi
  0 siblings, 1 reply; 8+ messages in thread
From: Giulio Benetti @ 2023-04-22 16:56 UTC (permalink / raw)
  To: Dario Binacchi; +Cc: buildroot

Hi Dario,

I readd the mailing list in Cc,

> Il giorno 22 apr 2023, alle ore 18:33, Dario Binacchi <dario.binacchi@amarulasolutions.com> ha scritto:
> 
> Hi Giulio,
> 
>> On Sat, Apr 22, 2023 at 4:56 PM Giulio Benetti
>> <giulio.benetti@benettiengineering.com> wrote:
>> 
>> Hi Dario,
>> 
>>> On 22/04/23 16:00, Dario Binacchi wrote:
>>> Board support package includes the following components:
>>> - mainline Linux kernel 5.14.21
>>> - mainline U-Boot 2023.04
>>> - default packages from buildroot
>>> 
>>> Note: There is a problem in u-boot display management. It should display
>>> the ST microelectronics logo but "noise" is displayed (like when running
>>> the command dd if=/dev/urandom of=/dev/fb0 in Linux). I also tried to
>>> change version (even the first version in which this feature was
>>> introduced), but without success. Despite this I decided not to disable
>>> the video management configuration to remind us that it needs to be
>>> fixed :).
>>> 
>>> Link: https://www.st.com/en/evaluation-tools/32f769idiscovery.html
>>> Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com>
>>> ---
>>>  DEVELOPERS                                    |  2 ++
>>>  .../stm32f769-disco/extlinux.conf             |  4 +++
>>>  .../stm32f769-disco/flash_sd.sh               | 18 +++++++++++
>>>  .../stm32f769-disco/genimage.cfg              | 27 ++++++++++++++++
>>>  .../stm32f769-disco/linux-sd.fragment         |  1 +
>>>  .../stm32f769-disco/post-build.sh             |  7 ++++
>>>  .../stm32f769-disco/readme.txt                | 32 +++++++++++++++++++
>>>  configs/stm32f769_disco_sd_defconfig          | 30 +++++++++++++++++
>>>  8 files changed, 121 insertions(+)
>>>  create mode 100644 board/stmicroelectronics/stm32f769-disco/extlinux.conf
>>>  create mode 100755 board/stmicroelectronics/stm32f769-disco/flash_sd.sh
>>>  create mode 100644 board/stmicroelectronics/stm32f769-disco/genimage.cfg
>>>  create mode 100644 board/stmicroelectronics/stm32f769-disco/linux-sd.fragment
>>>  create mode 100755 board/stmicroelectronics/stm32f769-disco/post-build.sh
>>>  create mode 100644 board/stmicroelectronics/stm32f769-disco/readme.txt
>>>  create mode 100644 configs/stm32f769_disco_sd_defconfig
>>> 
>>> diff --git a/DEVELOPERS b/DEVELOPERS
>>> index 5cd882b5791c..88ed40412ae8 100644
>>> --- a/DEVELOPERS
>>> +++ b/DEVELOPERS
>>> @@ -702,8 +702,10 @@ F:       package/xinetd/
>>> 
>>>  N:  Dario Binacchi <dario.binacchi@amarulasolutions.com>
>>>  F:  board/bsh/
>>> +F:   board/stmicroelectronics/stm32f769-disco/
>>>  F:  configs/imx8mn_bsh_smm_s2_defconfig
>>>  F:  configs/imx8mn_bsh_smm_s2_pro_defconfig
>>> +F:   configs/stm32f769_disco_sd_defconfig
>>>  F:  package/sscep/
>>>  F:  package/uuu/
>>> 
>>> diff --git a/board/stmicroelectronics/stm32f769-disco/extlinux.conf b/board/stmicroelectronics/stm32f769-disco/extlinux.conf
>>> new file mode 100644
>>> index 000000000000..48483a01fc4e
>>> --- /dev/null
>>> +++ b/board/stmicroelectronics/stm32f769-disco/extlinux.conf
>>> @@ -0,0 +1,4 @@
>>> +label stm32f769-disco-buildroot
>>> +  kernel /zImage
>>> +  devicetree /stm32f769-disco.dtb
>>> +  append console=ttySTM0,115200 root=/dev/mmcblk0p2 rw rootfstype=ext2 rootwait earlyprintk consoleblank=0 ignore_loglevel
>>> diff --git a/board/stmicroelectronics/stm32f769-disco/flash_sd.sh b/board/stmicroelectronics/stm32f769-disco/flash_sd.sh
>>> new file mode 100755
>>> index 000000000000..0ff3ae7dbfb3
>>> --- /dev/null
>>> +++ b/board/stmicroelectronics/stm32f769-disco/flash_sd.sh
>>> @@ -0,0 +1,18 @@
>>> +#!/bin/bash
>>> +
>>> +OUTPUT_DIR=$1
>>> +
>>> +if ! test -d "${OUTPUT_DIR}" ; then
>>> +    echo "ERROR: no output directory specified."
>>> +    echo "Usage: $0 OUTPUT_DIR"
>>> +    exit 1
>>> +fi
>>> +
>>> +"${OUTPUT_DIR}"/host/bin/openocd -f board/stm32f769i-disco.cfg \
>>> +  -c "init" \
>>> +  -c "reset init" \
>>> +  -c "flash probe 0" \
>>> +  -c "flash info 0" \
>>> +  -c "flash write_image erase ${OUTPUT_DIR}/images/u-boot.bin 0x08000000" \
>>> +  -c "reset run" \
>>> +  -c "shutdown"
>>> diff --git a/board/stmicroelectronics/stm32f769-disco/genimage.cfg b/board/stmicroelectronics/stm32f769-disco/genimage.cfg
>>> new file mode 100644
>>> index 000000000000..41f9da512489
>>> --- /dev/null
>>> +++ b/board/stmicroelectronics/stm32f769-disco/genimage.cfg
>>> @@ -0,0 +1,27 @@
>>> +image boot.vfat {
>>> +     vfat {
>>> +             files = {
>>> +                     "zImage",
>>> +                     "stm32f769-disco.dtb",
>>> +                     "extlinux"
>>> +             }
>>> +     }
>>> +
>>> +     size = 16M
>>> +}
>>> +
>>> +image sdcard.img {
>>> +     hdimage {
>>> +     }
>>> +
>>> +     partition u-boot {
>>> +             partition-type = 0xC
>>> +             image = "boot.vfat"
>>> +     }
>>> +
>>> +     partition rootfs {
>>> +             partition-type = 0x83
>>> +             image = "rootfs.ext2"
>>> +             size = 32M
>>> +     }
>>> +}
>>> diff --git a/board/stmicroelectronics/stm32f769-disco/linux-sd.fragment b/board/stmicroelectronics/stm32f769-disco/linux-sd.fragment
>>> new file mode 100644
>>> index 000000000000..846229d0d468
>>> --- /dev/null
>>> +++ b/board/stmicroelectronics/stm32f769-disco/linux-sd.fragment
>>> @@ -0,0 +1 @@
>>> +# CONFIG_XIP_KERNEL is not set
>>> diff --git a/board/stmicroelectronics/stm32f769-disco/post-build.sh b/board/stmicroelectronics/stm32f769-disco/post-build.sh
>>> new file mode 100755
>>> index 000000000000..c8cca3de8c9b
>>> --- /dev/null
>>> +++ b/board/stmicroelectronics/stm32f769-disco/post-build.sh
>>> @@ -0,0 +1,7 @@
>>> +#!/bin/sh
>>> +BOARD_DIR="$(dirname "$0")"
>>> +
>>> +# Kernel is built without devpts support
>>> +sed -i '/^devpts/d' "${TARGET_DIR}"/etc/fstab
>>> +
>>> +install -m 0644 -D "${BOARD_DIR}"/extlinux.conf "${BINARIES_DIR}"/extlinux/extlinux.conf
>>> diff --git a/board/stmicroelectronics/stm32f769-disco/readme.txt b/board/stmicroelectronics/stm32f769-disco/readme.txt
>>> new file mode 100644
>>> index 000000000000..8f4f79a86138
>>> --- /dev/null
>>> +++ b/board/stmicroelectronics/stm32f769-disco/readme.txt
>>> @@ -0,0 +1,32 @@
>>> +STM32F769 Discovery
>>> +===================
>>> +
>>> +This tutorial describes how to use the predefined Buildroot
>>> +configuration for the STM32F769 Discovery evaluation platform.
>>> +
>>> +Building
>>> +--------
>>> +
>>> +  make stm32f769_disco_sd_defconfig
>>> +  make
>>> +
>>> +Flashing
>>> +--------
>>> +
>>> +  ./board/stmicroelectronics/stm32f769-disco/flash_sd.sh output/
>>> +
>>> +It will flash the U-boot bootloader.
>>> +
>>> +Creating SD card
>>> +----------------
>>> +
>>> +Buildroot prepares an"sdcard.img" image in the output/images/ directory,
>>> +ready to be dumped on a SD card. Launch the following command as root:
>>> +
>>> +  dd if=output/images/sdcard.img of=/dev/<your-sd-device>
>>> +
>>> +*** WARNING! This will destroy all the card content. Use with care! ***
>>> +
>>> +For details about the medium image layout and its content, see the
>>> +definition in board/stmicroelectronics/stm32f769-disco/genimage.cfg.
>>> +
>>> diff --git a/configs/stm32f769_disco_sd_defconfig b/configs/stm32f769_disco_sd_defconfig
>>> new file mode 100644
>>> index 000000000000..4d43658403ca
>>> --- /dev/null
>>> +++ b/configs/stm32f769_disco_sd_defconfig
>>> @@ -0,0 +1,30 @@
>>> +BR2_arm=y
>>> +BR2_cortex_m4=y
>> 
>> Here ^^^ it is a BR2_cortex_m7
> 
> Yes,  you are right
> 
>> 
>>> +BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_5_14=y
>>> +BR2_ROOTFS_POST_BUILD_SCRIPT="board/stmicroelectronics/stm32f769-disco/post-build.sh"
>>> +BR2_ROOTFS_POST_IMAGE_SCRIPT="support/scripts/genimage.sh"
>>> +BR2_ROOTFS_POST_SCRIPT_ARGS="-c board/stmicroelectronics/stm32f769-disco/genimage.cfg"
>>> +BR2_LINUX_KERNEL=y
>>> +BR2_LINUX_KERNEL_CUSTOM_VERSION=y
>>> +BR2_LINUX_KERNEL_CUSTOM_VERSION_VALUE="5.14.21"
>> 
>> It would be great if you could use Linux latest stable 5.15.108 that has
>> patch for MMU-less fix committed[0]. I've found and fixed the bug on
>> i.MXRT but it should affect every MMU-less system.
> 
> thanks for the info
> 
>> Or you could bump Linux to latest version 6.2 that has the patch
>> committed too.
> 
> I will try with 5.15.108. Each new kernel version generally leads to
> occupying more memory and this is then reflected in some cases
> on applications to be launched in user space that become unstable
> (or at least it seems to me to be so).

You’re right about the size, but your experience as user space instability is normally due to bug on
mmu-less arm support that is few tested compared to mmu arm.
The bug I’ve found showed up during rootfs ext4 mount so it was easy to reproduce.
It dealt exactly with memory page translation to mmu-less.
So maybe what you see as instability could be fixed by my patch.

> So, for now I prefer to bump to
> 5.15.108.

Sure. Could you also try to fill the memory and see if everything works correctly?
You could try to start many instances of a program until you fill the memory and see that everything is working ok(I hope).

Thank you
Best regards
Giulio

> 
> Thanks and regards,
> Dario
> 
>> 
>> This could be done also for the stm32f4xx boards you're bumping for
>> uboot.
>> 
>> [0]:
>> https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?h=linux-5.15.y&id=7ae0262748e5d8ff1e42b77bb19bba474733f280
>> 
>> Best regards
>> --
>> Giulio Benetti
>> Benetti Engineering sas
>> 
>>> +BR2_LINUX_KERNEL_DEFCONFIG="stm32"
>>> +BR2_LINUX_KERNEL_CONFIG_FRAGMENT_FILES="$(LINUX_DIR)/arch/arm/configs/dram_0xc0000000.config board/stmicroelectronics/stm32f769-disco/linux-sd.fragment"
>>> +BR2_LINUX_KERNEL_IMAGE_TARGET_CUSTOM=y
>>> +BR2_LINUX_KERNEL_IMAGE_TARGET_NAME="zImage"
>>> +BR2_LINUX_KERNEL_DTS_SUPPORT=y
>>> +BR2_LINUX_KERNEL_INTREE_DTS_NAME="stm32f769-disco"
>>> +BR2_PACKAGE_BUSYBOX_CONFIG="package/busybox/busybox-minimal.config"
>>> +# BR2_PACKAGE_IFUPDOWN_SCRIPTS is not set
>>> +BR2_TARGET_ROOTFS_EXT2=y
>>> +BR2_TARGET_ROOTFS_EXT2_SIZE="32M"
>>> +# BR2_TARGET_ROOTFS_TAR is not set
>>> +BR2_TARGET_UBOOT=y
>>> +BR2_TARGET_UBOOT_BUILD_SYSTEM_KCONFIG=y
>>> +BR2_TARGET_UBOOT_CUSTOM_VERSION=y
>>> +BR2_TARGET_UBOOT_CUSTOM_VERSION_VALUE="2023.04"
>>> +BR2_TARGET_UBOOT_BOARD_DEFCONFIG="stm32f769-disco"
>>> +BR2_TARGET_UBOOT_NEEDS_OPENSSL=y
>>> +BR2_PACKAGE_HOST_DOSFSTOOLS=y
>>> +BR2_PACKAGE_HOST_GENIMAGE=y
>>> +BR2_PACKAGE_HOST_MTOOLS=y
>>> +BR2_PACKAGE_HOST_OPENOCD=y
>> 
> 
> 
> -- 
> 
> Dario Binacchi
> 
> Senior Embedded Linux Developer
> 
> dario.binacchi@amarulasolutions.com
> 
> __________________________________
> 
> 
> Amarula Solutions SRL
> 
> Via Le Canevare 30, 31100 Treviso, Veneto, IT
> 
> T. +39 042 243 5310
> info@amarulasolutions.com
> 
> www.amarulasolutions.com

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 8+ messages in thread