From: Peter Rosin <peda@axentia.se> To: Sascha Hauer <s.hauer@pengutronix.de>, Ahmad Fatoum <ahmad@a3f.at> Cc: Barebox List <barebox@lists.infradead.org> Subject: at91 sama5d3 "regressions" Date: Sat, 18 Sep 2021 00:37:39 +0200 [thread overview] Message-ID: <781e128b-14f3-3577-e433-f6a13fff8159@axentia.se> (raw) Hi! I have old boards that I'm trying to get up to current revisions of various pieces of software, and I'm running into a couple of problems with barebox. After bisecting and sorting though git history for a couple of days, these are my findings... 1st problematic patch: 070de908da ("ARM: remove PBL_FORCE_PIGGYDATA_COPY") This patch claims to remove an option, but it also kills a region overlap test that I appear to need on my board. This gets further complicated by patches 81ca755487 ("common.h: remove unused region_overlap()") 04e2aa516e ("common.h: move and rename lregion_overlap()") that first removes the needed but unused function, and then repurposes its name, and by patch 0be48260b8 ("ARM: Merge single pbl with multi pbl") which merges the old single pbl function into the multi pbl case. 2nd problematic patch 3e4a040545 ("ARM: mmu: use client domain permissions to support ARMv7 eXecute Never") This just doesn't work for my boards, and I need to revert it. 3rd problematic patch c5d38e9201 ("lds: Add and use RO_DATA_SECTION macro") Again, just doesn't work on my board. I'm doing a partial revert, but that gets further complicated by patch cd23b6facf ("common: add initial barebox deep-probe support") which piles in one more entry in the non-functional RO_DATA_SECTION macro. I'm going to follow up with patches. I very much realize that these patches are most likely not acceptable as-is, but I do include them since they are probably the best description of where the problems are. Cheers, Peter Peter Rosin (4): common.h: reintroduce region_overlap() as old_region_overlap() ARM: copy data if there is a region overlap Revert "ARM: mmu: use client domain permissions to support ARMv7 eXecute Never" lds: the RO_DATA_SECTION macro does not work on my SAMA5D3 board arch/arm/cpu/mmu-early.c | 7 +------ arch/arm/cpu/mmu.c | 7 +------ arch/arm/cpu/mmu.h | 1 - arch/arm/cpu/uncompress.c | 12 ++++++++++++ arch/arm/lib32/barebox.lds.S | 15 +++++++++++---- include/common.h | 14 ++++++++++++++ 6 files changed, 39 insertions(+), 17 deletions(-) -- 2.20.1 _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox
next reply other threads:[~2021-09-17 22:39 UTC|newest] Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-09-17 22:37 Peter Rosin [this message] 2021-09-17 22:39 ` [PATCH 1/4] common.h: reintroduce region_overlap() as, old_region_overlap() Peter Rosin 2021-09-17 22:39 ` [PATCH 2/4] ARM: copy data if there is a region overlap Peter Rosin 2021-09-17 22:40 ` [PATCH 3/4] Revert "ARM: mmu: use client domain permissions to support ARMv7 eXecute Never" Peter Rosin 2021-09-19 7:06 ` Rouven Czerwinski 2021-09-19 7:50 ` Peter Rosin 2021-09-19 20:33 ` Peter Rosin 2021-09-20 9:14 ` Ahmad Fatoum 2021-09-20 10:22 ` Peter Rosin 2021-09-17 22:41 ` [PATCH 4/4] lds: the RO_DATA_SECTION macro does not work on my SAMA5D3 board Peter Rosin 2021-09-17 23:57 ` at91 sama5d3 "regressions" Peter Rosin 2021-09-19 6:32 ` Peter Rosin 2021-09-22 7:06 ` Ahmad Fatoum 2021-09-22 7:41 ` Peter Rosin
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=781e128b-14f3-3577-e433-f6a13fff8159@axentia.se \ --to=peda@axentia.se \ --cc=ahmad@a3f.at \ --cc=barebox@lists.infradead.org \ --cc=s.hauer@pengutronix.de \ --subject='Re: at91 sama5d3 "regressions"' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox