From: Stephan Gerhold <stephan.gerhold@linaro.org>
To: Casey Connolly <casey.connolly@linaro.org>
Cc: Tom Rini <trini@konsulko.com>,
Neil Armstrong <neil.armstrong@linaro.org>,
Sumit Garg <sumit.garg@kernel.org>,
u-boot@lists.denx.de, u-boot-qcom@groups.io,
Varadarajan Narayanan <quic_varada@quicinc.com>,
Balaji Selvanathan <balaji.selvanathan@oss.qualcomm.com>,
Aswin Murugan <aswin.murugan@oss.qualcomm.com>
Subject: Re: [PATCH v2 6/7] configs: qualcomm: use mkmbn and stop creating ELF files
Date: Wed, 4 Jun 2025 11:21:53 +0200 [thread overview]
Message-ID: <aEAQMapUyEQsTQeV@linaro.org> (raw)
In-Reply-To: <20250602-b4-qcom-tooling-improvements-v2-6-c7d19c0d4a8b@linaro.org>
On Mon, Jun 02, 2025 at 06:10:57PM +0200, Casey Connolly wrote:
> Remove qcs9100_defconfig since it is now identical to qcom_defconfig.
> Additionally remove the load address and REMAKE_ELF from other Qualcomm
> defconfigs.
>
> For these platforms you should explicitly build the u-boot.mbn target
> which will have the correct load address and Qualcomm MBN signatures.
What is the motivation to require explicitly building the u-boot.mbn
target? Couldn't we do that automatically like before with REMAKE_ELF
(i.e. add u-boot.mbn to INPUTS-y)? Or is this because of the additional
Python dependencies (the cryptography module)?
BTW: We should probably document the Python "cryptography" requirement
somewhere so people don't get confused? qtestsign is Python 3.7+ and
cryptography 3.1+.
Thanks,
Stephan
next prev parent reply other threads:[~2025-06-04 9:22 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-06-02 16:10 [PATCH v2 0/7] Qualcomm: teach the build system to emit signed ELF images Casey Connolly
2025-06-02 16:10 ` [PATCH v2 1/7] binman: add $(srctree)/tools to toolpath Casey Connolly
2025-06-02 16:10 ` [PATCH v2 2/7] binman: support building binman dtb when OF_UPSTREAM is enabled Casey Connolly
2025-06-02 16:10 ` [PATCH v2 3/7] tools: add mkmbn tool for Qualcomm Casey Connolly
2025-06-04 9:15 ` Stephan Gerhold
2025-06-04 11:14 ` Casey Connolly
2025-06-02 16:10 ` [PATCH v2 4/7] binman: add support for building Qualcomm signed MBN ELF images Casey Connolly
2025-06-02 16:10 ` [PATCH v2 5/7] configs: qualcomm: use fragments for debug UART Casey Connolly
2025-06-02 16:10 ` [PATCH v2 6/7] configs: qualcomm: use mkmbn and stop creating ELF files Casey Connolly
2025-06-04 9:21 ` Stephan Gerhold [this message]
2025-06-04 11:17 ` Casey Connolly
2025-06-02 16:10 ` [PATCH v2 7/7] doc: board/qualcomm: update docs for new u-boot.mbn target Casey Connolly
2025-06-04 9:07 ` [PATCH v2 0/7] Qualcomm: teach the build system to emit signed ELF images Stephan Gerhold
2025-06-04 11:18 ` Casey Connolly
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=aEAQMapUyEQsTQeV@linaro.org \
--to=stephan.gerhold@linaro.org \
--cc=aswin.murugan@oss.qualcomm.com \
--cc=balaji.selvanathan@oss.qualcomm.com \
--cc=casey.connolly@linaro.org \
--cc=neil.armstrong@linaro.org \
--cc=quic_varada@quicinc.com \
--cc=sumit.garg@kernel.org \
--cc=trini@konsulko.com \
--cc=u-boot-qcom@groups.io \
--cc=u-boot@lists.denx.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