Re: [PATCH v3 4/4] arm64: Trap non-PIE builds early if starting from wrong address

; >>> On 10/09/2020 13:38, Michal Simek wrote: > >>>> > >>>> > >>>> On 09. 09. 20 19:07, Edgar E. Iglesias wrote: > >>>>> From: "Edgar E. Iglesias" > >>>>> > >>>>> Trap non-PIE builds early if the sta

[PATCH v3 4/4] arm64: Trap non-PIE builds early if starting from wrong address

From: "Edgar E. Iglesias" Trap non-PIE builds early if the start address doesn't match between run-time and link-time. This will trap the startup sequence rather than letting it run into obscure errors. Signed-off-by: Edgar E. Iglesias --- arch/arm/cpu/armv8/start.S | 13 ++

[PATCH v3 2/4] arm64: Trap PIE builds early if load address is not 4K aligned

From: "Edgar E. Iglesias" PIE requires a 4K aligned load address. If this is not met, trap the startup sequence in a WFI loop rather than running into obscure failures. Tested-by: Michal Simek Suggested-by: André Przywara Signed-off-by: Edgar E. Iglesias --- arch/arm/cpu/armv8/st

[PATCH v3 3/4] arm64: Add support for larger PIE U-Boot

From: "Edgar E. Iglesias" Linking a U-Boot larger than 1MB fails with PIE enabled: u-boot/arch/arm/cpu/armv8/start.S:71:(.text+0x3c): relocation truncated to fit: R_AARCH64_ADR_PREL_LO21 against symbol `__rel_dyn_end' defined in .bss_start section in u-boot. This extends the sup

[PATCH v3 1/4] arm64: Mention 4K aligned load addresses in the PIE Kconfig help

From: "Edgar E. Iglesias" Mention the requirement of 4K aligned load addresses in the help section for the POSITION_INDEPENDENT option. Suggested-by: Michal Simek Signed-off-by: Edgar E. Iglesias --- arch/arm/Kconfig | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) di

[PATCH v3 0/4] arm64: Large PIE fixes

From: "Edgar E. Iglesias" This fixes some build issues with large (> 1MB) PIE U-Boot setups. We also document the 4K aligned load address requirement and add an early run-time check for it. As requested by reviewers, I've also added a runtime check for non-PIE builds t

Re: [PATCH v2 2/3] arm64: Bail out PIE builds early if load address is not 4K aligned

On Mon, Sep 07, 2020 at 08:57:39AM -0400, Tom Rini wrote: > On Mon, Sep 07, 2020 at 11:52:35AM +0200, Edgar E. Iglesias wrote: > > On Fri, Sep 04, 2020 at 12:43:57PM -0600, Stephen Warren wrote: > > > On 9/4/20 3:07 AM, Edgar E. Iglesias wrote: > > >

Re: [PATCH v2 2/3] arm64: Bail out PIE builds early if load address is not 4K aligned

On Fri, Sep 04, 2020 at 12:43:57PM -0600, Stephen Warren wrote: > On 9/4/20 3:07 AM, Edgar E. Iglesias wrote: > > From: "Edgar E. Iglesias" > > > > PIE requires a 4K aligned load address. If this is not met, trap > > the startup sequence in a WFI

Re: [PATCH v2 1/3] arm64: Mention 4K aligned load addresses in the PIE Kconfig help

On Sun, Sep 06, 2020 at 11:16:17PM +0100, André Przywara wrote: > On 04/09/2020 19:42, Stephen Warren wrote: > > On 9/4/20 3:07 AM, Edgar E. Iglesias wrote: > >> From: "Edgar E. Iglesias" > >> > >> Mention the requirement of 4K aligned l

Re: [PATCH v2 3/3] arm64: Add support for larger PIE U-boot

On Fri, Sep 04, 2020 at 03:04:02PM +0200, Michal Simek wrote: > > > On 04. 09. 20 11:07, Edgar E. Iglesias wrote: > > From: "Edgar E. Iglesias" > > > > Linking a U-boot larger than 1MB fails with PIE enabled: > > u-boot/arch/arm/cpu/armv8/start.S:7

[PATCH v2 2/3] arm64: Bail out PIE builds early if load address is not 4K aligned

From: "Edgar E. Iglesias" PIE requires a 4K aligned load address. If this is not met, trap the startup sequence in a WFI loop rather than running into obscure failures. Tested-by: Michal Simek Suggested-by: André Przywara Signed-off-by: Edgar E. Iglesias --- arch/arm/cpu/armv8/st

[PATCH v2 3/3] arm64: Add support for larger PIE U-boot

From: "Edgar E. Iglesias" Linking a U-boot larger than 1MB fails with PIE enabled: u-boot/arch/arm/cpu/armv8/start.S:71:(.text+0x3c): relocation truncated to fit: R_AARCH64_ADR_PREL_LO21 against symbol `__rel_dyn_end' defined in .bss_start section in u-boot. This extends the sup

[PATCH v2 1/3] arm64: Mention 4K aligned load addresses in the PIE Kconfig help

From: "Edgar E. Iglesias" Mention the requirement of 4K aligned load addresses in the help section for the POSITION_INDEPENDENT option. Suggested-by: Michal Simek Signed-off-by: Edgar E. Iglesias --- arch/arm/Kconfig | 3 +++ 1 file changed, 3 insertions(+) diff --git a/arch/arm

[PATCH v2 0/3] arm64: Large PIE fixes

From: "Edgar E. Iglesias" This fixes some builds issues with large (> 1MB) PIE U-boot setups. We also document the 4K aligned load address requirement and add an early run-time check for it. Thanks, Edgar ChangeLog: v1 -> v2: * Fix adr of _start in arch/arm/lib/crt0_64.S * Add

Re: [PATCH] arm64: Add support for bigger u-boot when CONFIG_POSITION_INDEPENDENT=y

On Thu, Sep 03, 2020 at 01:10:48PM -0600, Stephen Warren wrote: > On 9/3/20 1:07 PM, Edgar E. Iglesias wrote: > > On Thu, Sep 03, 2020 at 09:45:39AM -0600, Stephen Warren wrote: > >> On 9/3/20 7:40 AM, André Przywara wrote: > >>> On 03/09/2020 14:35, Michal Simek wro

Re: [PATCH] arm64: Add support for bigger u-boot when CONFIG_POSITION_INDEPENDENT=y

On Thu, Sep 03, 2020 at 09:45:39AM -0600, Stephen Warren wrote: > On 9/3/20 7:40 AM, André Przywara wrote: > > On 03/09/2020 14:35, Michal Simek wrote: > >> > >> > >> On 02. 09. 20 18:34, Stephen Warren wrote: > >>> On 9/2/20 5:15 AM, Mic

Re: [PATCH] arm64: Add support for bigger u-boot when CONFIG_POSITION_INDEPENDENT=y

On Thu, Sep 03, 2020 at 02:52:39PM +0100, André Przywara wrote: > On 03/09/2020 14:41, Michal Simek wrote: > > > > > > On 02. 09. 20 20:59, André Przywara wrote: > >> On 02/09/2020 16:25, Edgar E. Iglesias wrote: > >>> On Wed, Sep 02, 2020 at 04:18:48P

Re: [PATCH] arm64: Add support for bigger u-boot when CONFIG_POSITION_INDEPENDENT=y

On Wed, Sep 02, 2020 at 04:18:48PM +0100, André Przywara wrote: > On 02/09/2020 15:53, Edgar E. Iglesias wrote: > > On Wed, Sep 02, 2020 at 03:43:08PM +0100, Andr� Przywara wrote: > >> On 02/09/2020 12:15, Michal Simek wrote: > > Hi, > > >> > >>&g

Re: [PATCH] arm64: Add support for bigger u-boot when CONFIG_POSITION_INDEPENDENT=y

On Wed, Sep 02, 2020 at 03:43:08PM +0100, André Przywara wrote: > On 02/09/2020 12:15, Michal Simek wrote: > > Hi, Hi Andre, > > > From: "Edgar E. Iglesias" > > > > When U-Boot binary exceeds 1MB with CONFIG_POSITION_INDEPENDENT=y > > compila

Re: [U-Boot] [PATCH] net: macb: Clean 64b dma addresses if they are not detected

On Wed, Sep 19, 2018 at 06:08:18PM +0200, Michal Simek wrote: > Clear ADDR64 dma bit in DMACFG register in case that HW_DMA_CAP_64B > is not detected on 64bit system. > The issue was observed when bootloader(u-boot) does not check macb > feature at DCFG6 register (DAW64_OFFSET) and enabling 64bit d

Re: [U-Boot] [PATCH] net: phy: ti: Enable automatic crossover mode

On Thu, Jan 14, 2016 at 02:36:52PM +0100, Michal Simek wrote: > Enable automatic crossover cable detection. > > Signed-off-by: Michal Simek Reviewed-by: Edgar E. Iglesias > --- > > drivers/net/phy/ti.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --g

Re: [U-Boot] [PATCH] zynq: Use arch_cpu_init() instead of lowlevel_init()

On Thu, Oct 17, 2013 at 08:33:28AM +0200, Albert ARIBAUD wrote: > Hi Albert, > > On Thu, 3 Oct 2013 18:07:40 +0200, Albert ARIBAUD > wrote: > > > Hi Michal, > > > > On Thu, 3 Oct 2013 11:56:20 +0200, Michal Simek > > wrote: > > > > > Hi Albert, > > > > > > On 10/03/2013 10:41 AM, Albert ARIB