Re: [PATCH v3 4/4] arm64: dts: freescale: imx8qxp/imx8qm: Add CAAM support

Hi Frank,

On Wed, May 28, 2025 at 11:01:33AM -0400, Frank Li wrote:

On Wed, May 28, 2025 at 02:43:08PM +0000, John Ernberg wrote:

quoted

From: Horia Geantă <horia.geanta@nxp.com>

The iMX8QXP and iMX8QM have a CAAM (Cryptographic Acceleration and
Assurance Module) like many other iMXs.

Add the definitions for it.

Job Rings 0 and 1 are bound to the SECO (Security Controller) ARM core
and are not exposed outside it. There's no point to define them in the
bindings as they cannot be used outside the SECO.

Signed-off-by: Horia Geantă <horia.geanta@nxp.com>
[jernberg: Commit message, fixed dtbs_check warnings, trimmed memory ranges]

what's this for?  remove it if not related with this patch.

These are the adjustments I did to the patch by Horia that I took out of
the linux-imx tree (original patch linked below in context).

While not part of the process I have seen them used and found them helpful,
I can drop it if preferred.

Best regards // John Ernberg

Frank

quoted

Signed-off-by: John Ernberg <redacted>

---

Imported from NXP tree, trimmed down and fixed the dtbs_check warnings.
Constrained the ranges to the needed ones.
Changed the commit message.
Original here: https://github.com/nxp-imx/linux-imx/commit/699e54b386cb9b53def401798d0a4e646105583d

---