Andrew Turner <andrew_at_FreeBSD.org> wrote on
Date: Wed, 25 Jun 2025 11:22:51 UTC :

> The branch main has been updated by andrew:
> 
> URL: 
> https://cgit.FreeBSD.org/src/commit/?id=288ba78edcfd82b4b558a46fae78e4076acbe200
> 
> commit 288ba78edcfd82b4b558a46fae78e4076acbe200
> Author: Andrew Turner <and...@freebsd.org>
> AuthorDate: 2025-06-24 16:47:28 +0000
> Commit: Andrew Turner <and...@freebsd.org>
> CommitDate: 2025-06-24 17:45:42 +0000
> 
> dev/psci: Check for functions before calling
> 
> In the SMCCC TRNG and Errata ABI drivers check if the version
> functions are implemented before calling them. Without this the firmware
> may complain about an unimplemented function being called.
> 
> Reported by: bz
> Reviewed by: bz
> Sponsored by: Arm Ltd
> Differential Revision: https://reviews.freebsd.org/D50978
> ---


I wonder if a debug mode or verbose mode should report
the != SMCCC_RET_SUCCESS cases for smccc_arch_features.
If this change had already been in place, the older
status might not have been noticed. As bz wrote
elsewhere:

"That said seems like it's still a good idea to add
these bits to my TF-A."

Having a way to check on if a TF-A instance is too
old to have a known status for handling things that
are now requested could be a good thing.

===
Mark Millard
marklmi at yahoo.com


Reply via email to