mail archive of the barebox mailing list
 help / color / mirror / Atom feed
From: Sascha Hauer <s.hauer@pengutronix.de>
To: BAREBOX <barebox@lists.infradead.org>
Cc: Ahmad Fatoum <a.fatoum@pengutronix.de>,
	Marco Felsch <m.felsch@pengutronix.de>
Subject: [PATCH v2 00/14] i.MX6 TZASC and OP-TEE early helpers
Date: Mon, 30 Jun 2025 09:45:42 +0200	[thread overview]
Message-ID: <20250630-arm-optee-early-helper-v2-0-c8cce3ae42b0@pengutronix.de> (raw)

This series combines Marcos "i.MX6Q TZASC and OP-TEE early helpers" [1]
and my series fixing chainloading barebox on OP-TEE enabled i.MX6ul
platforms [2] into one.

Changes to Marcos series are:

- rename imx6q_tzc380_is_enabled() to imx6q_tzc380_is_bypassed() as the
  function really only detects if the bypass bits are set correctly and
  not if the tzc380 is enabled.
- Add i.MX6UL variants of the functions Marco created for i.MX6DQ
- Marco created a mx6_start_optee_early() function that has a
  switch/case for the actual SoC type. I have split that into SoC
  specific functions so that we don't have to carry the binary weight
  when only one specific SoC is needed

Changes to my series:

- Ahmad mentioned in [3] that detecting if we are chainloaded based on a
  register passed from the 1st stage not robust and might have security
  issues. I am therefore now checking if the TZASC is accessible or not.

As a special bonus this series brings us exception handling support in
PBL for ARMv7/ARMv8. This done to implement data_abort_mask() which we
need for the TZASC-is-accessible check, but will also be useful for
debugging issues in the PBL.

[1] https://lore.kernel.org/20250626144527.416697-1-m.felsch@pengutronix.de
[2] https://lore.kernel.org/20250626140329.418033-1-s.hauer@pengutronix.de
[3] https://lore.kernel.org/70b41f3b-4329-48f7-827f-1924e002ab04@pengutronix.de

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
Changes in v2:
- update some commit messages according to review
- Drop patch to drop start_optee_early() - it's unused but referenced in the
  documentation
- Add config symbol for TZASC and select it where necessary
- Collect R-B from Ahmad, Marco
- Link to v1: https://lore.barebox.org/20250627-arm-optee-early-helper-v1-0-4b098e8ac7cd@pengutronix.de

---
Marco Felsch (5):
      ARM: i.MX6QDL: add imxcfg helper to configure the TZASC1/2
      ARM: i.MX6Q: add imx6_get_mmdc_sdram_size
      ARM: mach-imx: tzasc: add region configure helpers
      ARM: mach-imx: tzasc: add imx6[q|ul]_tzc380_is_bypassed()
      ARM: optee-early: add mx6_start_optee_early helper

Sascha Hauer (9):
      pbl: add panic_no_stacktrace()
      arch: Allow data_abort_mask() in PBL
      ARM: add exception handling support for PBL
      ARM: i.MX: add config symbol for TZASC
      ARM: mach-imx: tzasc: add imx6[q|ul]_tzc380_early_ns_region1()
      ARM: i.MX: add imx6_can_access_tzasc()
      ARM: i.MX: tqma6ulx: fix barebox chainloading with OP-TEE enabled
      ARM: i.MX: Webasto ccbv2: fix barebox chainloading with OP-TEE enabled
      ARM: i.MX: tqma6ulx: use ENTRY_FUNCTION_WITHSTACK

 Documentation/user/optee.rst             |  22 ++-
 arch/Kconfig                             |   3 +
 arch/arm/Kconfig                         |  10 +
 arch/arm/boards/tqma6ulx/lowlevel.c      |  27 +--
 arch/arm/boards/webasto-ccbv2/lowlevel.c |  24 +--
 arch/arm/cpu/Makefile                    |   1 +
 arch/arm/cpu/interrupts_32.c             |  14 +-
 arch/arm/cpu/interrupts_64.c             |  10 +-
 arch/arm/cpu/uncompress.c                |   2 +
 arch/arm/include/asm/barebox-arm.h       |   8 +
 arch/arm/lib/pbl.lds.S                   |   4 +
 arch/arm/lib32/optee-early.c             |  42 ++++
 arch/arm/mach-imx/Kconfig                |   6 +
 arch/arm/mach-imx/Makefile               |   2 +-
 arch/arm/mach-imx/esdctl.c               |   5 +
 arch/arm/mach-imx/tzasc.c                | 328 +++++++++++++++++++++++++++++++
 include/abort.h                          |   3 +-
 include/mach/imx/esdctl.h                |   1 +
 include/mach/imx/imx6q-tzasc.h           |   8 +
 include/mach/imx/tzasc.h                 |   5 +
 include/tee/optee.h                      |   4 +
 pbl/misc.c                               |   3 +
 22 files changed, 489 insertions(+), 43 deletions(-)
---
base-commit: 796506e790596a0b59c186651cf98112eafe01c5
change-id: 20250627-arm-optee-early-helper-6d894df2201c

Best regards,
-- 
Sascha Hauer <s.hauer@pengutronix.de>




             reply	other threads:[~2025-06-30  7:47 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-06-30  7:45 Sascha Hauer [this message]
2025-06-30  7:45 ` [PATCH v2 01/14] pbl: add panic_no_stacktrace() Sascha Hauer
2025-06-30  7:45 ` [PATCH v2 02/14] arch: Allow data_abort_mask() in PBL Sascha Hauer
2025-06-30  7:45 ` [PATCH v2 03/14] ARM: add exception handling support for PBL Sascha Hauer
2025-06-30  8:54   ` Marco Felsch
2025-06-30  7:45 ` [PATCH v2 04/14] ARM: i.MX6QDL: add imxcfg helper to configure the TZASC1/2 Sascha Hauer
2025-06-30  7:45 ` [PATCH v2 05/14] ARM: i.MX6Q: add imx6_get_mmdc_sdram_size Sascha Hauer
2025-06-30  7:45 ` [PATCH v2 06/14] ARM: i.MX: add config symbol for TZASC Sascha Hauer
2025-06-30  7:45 ` [PATCH v2 07/14] ARM: mach-imx: tzasc: add region configure helpers Sascha Hauer
2025-06-30  7:45 ` [PATCH v2 08/14] ARM: mach-imx: tzasc: add imx6[q|ul]_tzc380_early_ns_region1() Sascha Hauer
2025-06-30  7:45 ` [PATCH v2 09/14] ARM: mach-imx: tzasc: add imx6[q|ul]_tzc380_is_bypassed() Sascha Hauer
2025-06-30  7:45 ` [PATCH v2 10/14] ARM: i.MX: add imx6_can_access_tzasc() Sascha Hauer
2025-06-30  7:45 ` [PATCH v2 11/14] ARM: optee-early: add mx6_start_optee_early helper Sascha Hauer
2025-06-30  7:45 ` [PATCH v2 12/14] ARM: i.MX: tqma6ulx: fix barebox chainloading with OP-TEE enabled Sascha Hauer
2025-06-30  7:45 ` [PATCH v2 13/14] ARM: i.MX: Webasto ccbv2: " Sascha Hauer
2025-06-30  7:45 ` [PATCH v2 14/14] ARM: i.MX: tqma6ulx: use ENTRY_FUNCTION_WITHSTACK Sascha Hauer
2025-07-02  6:13 ` [PATCH v2 00/14] i.MX6 TZASC and OP-TEE early helpers Sascha Hauer

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=20250630-arm-optee-early-helper-v2-0-c8cce3ae42b0@pengutronix.de \
    --to=s.hauer@pengutronix.de \
    --cc=a.fatoum@pengutronix.de \
    --cc=barebox@lists.infradead.org \
    --cc=m.felsch@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