From: Guillermo Rodriguez Garcia <guille.rodriguez@gmail.com>
To: Guillermo Rodriguez Garcia <guille.rodriguez@gmail.com>,
ptxdist@pengutronix.de,
Alejandro Vazquez <avazquez.dev@gmail.com>
Subject: Re: [ptxdist] How to add symbols to PTXDIST_LOWLEVEL_WRAPPER_BLACKLIST
Date: Tue, 14 Jul 2026 20:22:30 +0200 [thread overview]
Message-ID: <CABDcavY9_7uGrCBQc7NKZXYXQUq62vrpV4n09-Yz2_JBGCEJrQ@mail.gmail.com> (raw)
In-Reply-To: <alXS-OhiY-6hDyx3@pengutronix.de>
Hi Michael,
El mar, 14 jul 2026 a las 8:11, Michael Olbrich
(<m.olbrich@pengutronix.de>) escribió:
>
> On Mon, Jul 13, 2026 at 06:23:56PM +0200, Guillermo Rodriguez Garcia wrote:
> > Hello,
> >
> > Our BSP is currently defining:
> >
> > TARGET_EXTRA_CFLAGS=-mtune=cortex-a7 -mfpu=neon-vfpv4
> >
> > However this is causing us some trouble with TF-A, as the code
> > generated by the toolchain contains NEON instructions, which makes it
> > fail at runtime with "Undefined instruction" exceptions.
> >
> > We could skip this for TF-A specifically by adding TARGET_EXTRA_CFLAGS
> > to TF_A_WRAPPER_BLACKLIST.
> >
> > But actually we'd like to do the same for all low level code,
> > including kernel and bootloaders (U-Boot, OP-TEE). Since all of these
> > already set <package>_WRAPPER_BLACKLIST to
> > $(PTXDIST_LOWLEVEL_WRAPPER_BLACKLIST), the easiest and more robust way
> > would be to add TARGET_EXTRA_CFLAGS to
> > PTXDIST_LOWLEVEL_WRAPPER_BLACKLIST itself. Is this possible?
>
> Untested, but I think, you can create rules/pre/blacklist.make with:
>
> PTXDIST_LOWLEVEL_WRAPPER_BLACKLIST += TARGET_EXTRA_CFLAGS
>
> to your BSP (any filename is fine as long as it is in rules/pre/ and has a
> .make suffix).
Great. I can confirm that this works, thank you!
I have some questions:
1. It is not immediately obvious that one might need to take extra
precautions for "low level" (bootloader / kernel) packages. I mean, it
_is_ obvious in hindsight, but I have been using PTXdist for many
years and have developed BSP for several platforms, tweaking CFLAGS as
needed for each one, and I did not realize that these CFLAGS are not
"only" used for userspace packages. Perhaps a note in the docs could
be helpful? (If so, let me know and I will be happy to submit a patch)
2. Would it be useful to have a way to tweak
PTXDIST_LOWLEVEL_WRAPPER_BLACKLIST from platformconfig (next to
toolchain options) ? The workaround you provided works great but it
was not immediately obvious either (to me :-))
Thanks again,
Guillermo
prev parent reply other threads:[~2026-07-14 18:23 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-07-13 16:23 Guillermo Rodriguez Garcia
2026-07-14 6:11 ` Michael Olbrich
2026-07-14 18:22 ` Guillermo Rodriguez Garcia [this message]
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=CABDcavY9_7uGrCBQc7NKZXYXQUq62vrpV4n09-Yz2_JBGCEJrQ@mail.gmail.com \
--to=guille.rodriguez@gmail.com \
--cc=avazquez.dev@gmail.com \
--cc=ptxdist@pengutronix.de \
/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
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox