Date: Wed, 25 Jun 2025 11:26:32 -0700 From: Mark Millard <marklmi@yahoo.com> To: Andrew Turner <andrew@freebsd.org>, "Bjoern A. Zeeb" <bzeeb-lists@lists.zabbadoz.net>, dev-commits-src-main@freebsd.org Subject: git: 288ba78edcfd - main - dev/psci: Check for functions before calling Message-ID: <60E32387-B0BA-4ADD-A897-DC988136D2F9@yahoo.com> References: <60E32387-B0BA-4ADD-A897-DC988136D2F9.ref@yahoo.com>
next in thread | previous in thread | raw e-mail | index | archive | help
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: >=20 > URL: = https://cgit.FreeBSD.org/src/commit/?id=3D288ba78edcfd82b4b558a46fae78e407= 6acbe200 >=20 > commit 288ba78edcfd82b4b558a46fae78e4076acbe200 > Author: Andrew Turner <andrew@FreeBSD.org> > AuthorDate: 2025-06-24 16:47:28 +0000 > Commit: Andrew Turner <andrew@FreeBSD.org> > CommitDate: 2025-06-24 17:45:42 +0000 >=20 > dev/psci: Check for functions before calling >=20 > 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. >=20 > 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 !=3D 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. =3D=3D=3D Mark Millard marklmi at yahoo.com
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?60E32387-B0BA-4ADD-A897-DC988136D2F9>