On 5/24/19 3:30 PM, Peter Robinson wrote:
On Fri, May 24, 2019 at 6:19 AM Heinrich Schuchardt xypron.glpk@gmx.de wrote:
On 5/24/19 3:02 AM, AKASHI Takahiro wrote:
Heinrich,
On Thu, May 16, 2019 at 07:34:54PM +0200, Heinrich Schuchardt wrote:
In EFI 1.10 a version of the Unicode collation protocol using ISO 639-2 language codes existed. This protocol is not part of the UEFI specification any longer. Unfortunately it is required to run the UEFI Self Certification Test (SCT) II, version 2.6, 2017. So we implement it here for the sole purpose of running the SCT. It can be removed once a compliant SCT is available.
I remember that Alex always rejected this kind of patch, saying that an upstream (SCT in this case) should be fixed in the first place.
Do you want to change this policy? (I'm just asking.)
I am using SCT a lot to test my patches. I want to be able to run the tests on the final code.
Wouldn't it be better to patch/fork the upstream SCT that blindly pulling in obsolete code that is basically already dead?
If you could submit a patch for SCT, that surely would help.
I have raised a ticket for upstream SCT but did not see any reaction up to now.
Got a reference to the ticket?
https://bugzilla.tianocore.org/show_bug.cgi?id=1802
Best regards
Heinrich
Making this deprecated protocol a config option deselected by default is the most plausible solution to me. In the Kconfig comment I made it clear that this protocol is going to be removed when the SCT has been corrected.
I disagree, I think if it's obsolete not having the code in the first case in the most plausible option IMO.
Best regards
Heinrich
-Takahiro Akashi
The configuration option defaults to no.
Heinrich Schuchardt (2): efi_loader: rename Unicode collation protocol 2 variables efi_loader: implement deprecated Unicode collation protocol
include/efi_api.h | 5 ++++ include/efi_loader.h | 9 +++++-- lib/efi_loader/Kconfig | 15 ++++++++++-- lib/efi_loader/Makefile | 2 +- lib/efi_loader/efi_root_node.c | 7 +++++- lib/efi_loader/efi_unicode_collation.c | 33 +++++++++++++++++++++++--- lib/efi_selftest/Makefile | 3 ++- 7 files changed, 64 insertions(+), 10 deletions(-)
-- 2.20.1
U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot