Hi Maheedhar,

Thank you for your input. Since this is a change in the log message, I've addressed it in the patch below and have added you as a reviewer, please let us know your thoughts.

https://review.trustedfirmware.org/c/TF-A/trusted-firmware-a/+/30049  

Thanks
-Sona
From: Bollapalli, Maheedhar Sai via TF-A <tf-a@lists.trustedfirmware.org>
Sent: Thursday, July 18, 2024 4:02 AM
To: tf-a@lists.trustedfirmware.org <tf-a@lists.trustedfirmware.org>
Subject: [TF-A] Proper message for ERRATA_NOT_APPLIES case
 
Hi,

On AMD-Xilinx ZynqMP platform for ERRATA_NOT_APPLIES case; the log appearing on the console is as below:
"VERBOSE: BL31: cortex_a53: CPU workaround for erratum 819472was not applied".

We checked ZynqMP code and found that the ZynqMP cpu revision was higher than the cpu revision intended in the erratum and hence it was not applicable and not applied.

The message seems inappropriate to us as from log message it is not clear why it wasn't applied.
Our suggestion is that in the log message instead of "not applied" it should be mentioned as "not applicable" as indicated below to reflect the reason properly.
"VERBOSE: BL31: cortex_a53: CPU workaround for erratum 819472 was not applicable".

Regards,
Maheedhar.

--
TF-A mailing list -- tf-a@lists.trustedfirmware.org
To unsubscribe send an email to tf-a-leave@lists.trustedfirmware.org