RE: [EXT] Re: [PATCH v50 3/7] firmware: imx: add driver for NXP EdgeLock Enclave

[Pankaj Gupta]

Thanks Frieder for extending help here.

On v51, request you share your review comments too if you have, apart from extending support for testing.

Regards
Pankaj



NXP Confidential
> -----Original Message-----
> From: Frieder Schrempf <frieder.schrempf@xxxxxxxxxx>
> Sent: 14 September 2026 16:18
> To: Pankaj Gupta (OSS) <pankaj.gupta@xxxxxxxxxxx>; Jonathan Corbet
> <corbet@xxxxxxx>; Shuah Khan <skhan@xxxxxxxxxxxxxxxxxxx>; Rob Herring
> <robh@xxxxxxxxxx>; Krzysztof Kozlowski <krzk+dt@xxxxxxxxxx>; Conor Dooley
> <conor+dt@xxxxxxxxxx>; Frank Li <frank.li@xxxxxxx>; Sascha Hauer
> <s.hauer@xxxxxxxxxxxxxx>; Pengutronix Kernel Team <kernel@xxxxxxxxxxxxxx>;
> Fabio Estevam <festevam@xxxxxxxxx>; Pankaj Gupta <pankaj.gupta@xxxxxxx>;
> Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
> Cc: linux-doc@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx;
> devicetree@xxxxxxxxxxxxxxx; imx@xxxxxxxxxxxxxxx; linux-arm-
> kernel@xxxxxxxxxxxxxxxxxxx
> Subject: [EXT] Re: [PATCH v50 3/7] firmware: imx: add driver for NXP EdgeLock
> Enclave
>
> Caution: This is an external email. Please take care when clicking links or opening
> attachments. When in doubt, report the message using the 'Report this email'
> button
>
>
> On 14.09.26 12:45, Frieder Schrempf wrote:
> > On 12.09.26 19:03, pankaj.gupta@xxxxxxxxxxx wrote:
> >> From: Pankaj Gupta <pankaj.gupta@xxxxxxx>
> >>
> >> Add MU-based communication interface for secure enclave.
> >>
> >> NXP hardware IP(s) for secure-enclaves like Edgelock Enclave(ELE),
> >> are embedded in the SoC to support the features like HSM, SHE & V2X,
> >> using message based communication interface.
> >>
> >> The secure enclave FW communicates with Linux over single or multiple
> >> dedicated messaging unit(MU) based interface(s).
> >> Exists on i.MX SoC(s) like i.MX8ULP, i.MX93, i.MX95 etc.
> >>
> >> For i.MX9x SoC(s) there is at least one dedicated ELE MU(s) for each
> >> world - Linux(one or more) and OPTEE-OS (one or more).
> >>
> >> Other dependent kernel drivers will be:
> >> - NVMEM: that supports non-volatile devices like EFUSES,
> >>   managed by NXP's secure-enclave.
> >>
> >> Signed-off-by: Pankaj Gupta <pankaj.gupta@xxxxxxx>
> >> Reviewed-by: Frank Li <Frank.Li@xxxxxxx>
> >> Tested-by: Frieder Schrempf <frieder.schrempf@xxxxxxxxxx>
> >
> > My review was for v25, my test was valid for v29. We are now at v50,
> > so please, as requested before, drop the tags as I don't think they
> > can be considered to be still valid with all these changes.
> Sorry, I missed that you already removed the R-b in the meantime. But I guess
> the T-b should also go. Please ping me when all the problems in the code are
> solved and I will test once more.