RE: ARM: mmu_early_enable

2023-08-17 Thread Lior Weintraub
Hi Sascha, I read your answer again and realized my misunderstanding. So essentially what you say is that all 48 bits of the address space would be set as flat 1:1 un-cached and only the SRAM region will be set as cached. In that case the UART at address 0xD0_0030_7000 and our ROM at address

Re: [PATCH master 1/3] envfs: fix defaultenv_append_directory macro definition

2023-08-17 Thread Marco Felsch
On 23-08-17, Ahmad Fatoum wrote: > defaultenv_append_directory is called as if it were a function and the > trailing semicolon can break parse of if-else statements. Wrap in do { } > while (0) to avoid this. > > Signed-off-by: Ahmad Fatoum Reviewed-by: Marco Felsch

[PATCH master 3/3] sandbox: store stickypage in runtime dir

2023-08-17 Thread Ahmad Fatoum
The stickypage is a hack to have a 4K hostfile persist over reboot. This was so far done by compiling the stickypage separately and referencing it from the device tree via the magic $build variable that expands to the working directory. This breaks a number of assumptions: - KBUILD_IMAGE:

[PATCH master 2/3] test: arm: make multi_v8_defconfig.yaml a symlink to virt@

2023-08-17 Thread Ahmad Fatoum
multi_v8_defconfig builds images for Virt, ZynqMP, Rockchip and i.MX. Convention is for unadorned Labgrid environment file to be a symlink to a file with a more telling name. Signed-off-by: Ahmad Fatoum --- test/arm/multi_v8_defconfig.yaml | 23 +--

[PATCH master 1/3] envfs: fix defaultenv_append_directory macro definition

2023-08-17 Thread Ahmad Fatoum
defaultenv_append_directory is called as if it were a function and the trailing semicolon can break parse of if-else statements. Wrap in do { } while (0) to avoid this. Signed-off-by: Ahmad Fatoum --- include/envfs.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

RE: ARM: mmu_early_enable

2023-08-17 Thread Lior Weintraub
But the UART is set on a total different memory space. The SRAM where barebox runs and also given as membase and memsize is in the region of 0xC0__ The UART is in the region of 0xD0_0030_7000. According the tarmac log, it looks like access to this location caused the exception:

Re: ARM: mmu_early_enable

2023-08-17 Thread Sascha Hauer
On Thu, Aug 17, 2023 at 06:22:50AM +, Lior Weintraub wrote: > Hi Sascha, > > I think I found an issue with the CONFIG_MMU feature. > When the code under barebox_pbl_start calls mmu_early_enable, the MMU > is set such that only the given SRAM is defined (membase, memsize). > But then, if

ARM: mmu_early_enable

2023-08-17 Thread Lior Weintraub
Hi Sascha, I think I found an issue with the CONFIG_MMU feature. When the code under barebox_pbl_start calls mmu_early_enable, the MMU is set such that only the given SRAM is defined (membase, memsize). But then, if DEBUG_LL is in use and the function pr_debug is called we get an exception