Skip to content

Commit a46697f

Browse files
GowthamM9974mergify[bot]
authored andcommitted
MdeModulePkg: Deprecate EFI_UNICODE_COLLATION_PROTOCOL_GUID
Adding to the previous commit deprecating EFI_UNICODE_COLLATION_PROTOCOL_GUID in MdeModulePkg. Cc: Sachin Ganesh <sachinganesh@ami.com> Signed-off-by: Gowtham M <gowthamm@ami.com>
1 parent 9898567 commit a46697f

File tree

7 files changed

+13
-69
lines changed

7 files changed

+13
-69
lines changed

MdeModulePkg/MdeModulePkg.dec

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -802,12 +802,6 @@
802802
# @Prompt Enable variable statistics collection.
803803
gEfiMdeModulePkgTokenSpaceGuid.PcdVariableCollectStatistics|FALSE|BOOLEAN|0x0001003f
804804

805-
## Indicates if Unicode Collation Protocol will be installed.<BR><BR>
806-
# TRUE - Installs Unicode Collation Protocol.<BR>
807-
# FALSE - Does not install Unicode Collation Protocol.<BR>
808-
# @Prompt Enable Unicode Collation support.
809-
gEfiMdeModulePkgTokenSpaceGuid.PcdUnicodeCollationSupport|TRUE|BOOLEAN|0x00010040
810-
811805
## Indicates if Unicode Collation 2 Protocol will be installed.<BR><BR>
812806
# TRUE - Installs Unicode Collation 2 Protocol.<BR>
813807
# FALSE - Does not install Unicode Collation 2 Protocol.<BR>

MdeModulePkg/MdeModulePkg.uni

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -691,12 +691,6 @@
691691
"TRUE - Statistics about variable usage will be collected.<BR>\n"
692692
"FALSE - Statistics about variable usage will not be collected.<BR>"
693693

694-
#string STR_gEfiMdeModulePkgTokenSpaceGuid_PcdUnicodeCollationSupport_PROMPT #language en-US "Enable Unicode Collation support"
695-
696-
#string STR_gEfiMdeModulePkgTokenSpaceGuid_PcdUnicodeCollationSupport_HELP #language en-US "Indicates if Unicode Collation Protocol will be installed.<BR><BR>\n"
697-
"TRUE - Installs Unicode Collation Protocol.<BR>\n"
698-
"FALSE - Does not install Unicode Collation Protocol.<BR>"
699-
700694
#string STR_gEfiMdeModulePkgTokenSpaceGuid_PcdUnicodeCollation2Support_PROMPT #language en-US "Enable Unicode Collation 2 support"
701695

702696
#string STR_gEfiMdeModulePkgTokenSpaceGuid_PcdUnicodeCollation2Support_HELP #language en-US "Indicates if Unicode Collation 2 Protocol will be installed.<BR><BR>\n"

MdeModulePkg/Universal/Disk/UnicodeCollation/EnglishDxe/EnglishDxe.inf

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,8 @@
22
# English module that provides Unicode Collation supports.
33
#
44
# This driver installs Unicode ISO 639-2 Collation and
5-
# RFC 4646 Unicode Collation 2 protocols based on feature flags
6-
# PcdUnicodeCollationSupport & PcdUnicodeCollation2Support respectively.
5+
# RFC 4646 Unicode Collation 2 protocols based on feature flag
6+
# PcdUnicodeCollation2Support.
77
# It allows code running in the boot services environment to perform lexical
88
# comparison functions on Unicode strings for English languages.
99
#
@@ -43,11 +43,9 @@
4343
PcdLib
4444

4545
[FeaturePcd]
46-
gEfiMdeModulePkgTokenSpaceGuid.PcdUnicodeCollationSupport ## CONSUMES
4746
gEfiMdeModulePkgTokenSpaceGuid.PcdUnicodeCollation2Support ## CONSUMES
4847

4948
[Protocols]
50-
gEfiUnicodeCollationProtocolGuid | gEfiMdeModulePkgTokenSpaceGuid.PcdUnicodeCollationSupport ## SOMETIMES_PRODUCES
5149
gEfiUnicodeCollation2ProtocolGuid | gEfiMdeModulePkgTokenSpaceGuid.PcdUnicodeCollation2Support ## PRODUCES
5250

5351
[UserExtensions.TianoCore."ExtraFiles"]

MdeModulePkg/Universal/Disk/UnicodeCollation/EnglishDxe/EnglishDxe.uni

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,8 @@
22
// English module that provides Unicode Collation supports.
33
//
44
// This driver installs Unicode ISO 639-2 Collation and
5-
// RFC 4646 Unicode Collation 2 protocols based on feature flags
6-
// PcdUnicodeCollationSupport & PcdUnicodeCollation2Support respectively.
5+
// RFC 4646 Unicode Collation 2 protocols based on feature flag
6+
// PcdUnicodeCollation2Support respectively.
77
// It allows code running in the boot services environment to perform lexical
88
// comparison functions on Unicode strings for English languages.
99
//
@@ -16,5 +16,5 @@
1616

1717
#string STR_MODULE_ABSTRACT #language en-US "Provides Unicode Collation support"
1818

19-
#string STR_MODULE_DESCRIPTION #language en-US "This driver installs Unicode ISO 639-2 Collation and RFC 4646 Unicode Collation 2 protocols based on feature flags PcdUnicodeCollationSupport & PcdUnicodeCollation2Support respectively. It allows code running in the boot services environment to perform lexical comparison functions on Unicode strings for English languages."
19+
#string STR_MODULE_DESCRIPTION #language en-US "This driver installs Unicode ISO 639-2 Collation and RFC 4646 Unicode Collation 2 protocols based on feature flag PcdUnicodeCollation2Support. It allows code running in the boot services environment to perform lexical comparison functions on Unicode strings for English languages."
2020

MdeModulePkg/Universal/Disk/UnicodeCollation/EnglishDxe/UnicodeCollationEng.c

Lines changed: 7 additions & 36 deletions
Original file line numberDiff line numberDiff line change
@@ -120,42 +120,13 @@ InitializeUnicodeCollationEng (
120120
}
121121

122122
if (FeaturePcdGet (PcdUnicodeCollation2Support)) {
123-
if (FeaturePcdGet (PcdUnicodeCollationSupport)) {
124-
Status = gBS->InstallMultipleProtocolInterfaces (
125-
&mHandle,
126-
&gEfiUnicodeCollationProtocolGuid,
127-
&UnicodeEng,
128-
&gEfiUnicodeCollation2ProtocolGuid,
129-
&Unicode2Eng,
130-
NULL
131-
);
132-
ASSERT_EFI_ERROR (Status);
133-
} else {
134-
Status = gBS->InstallMultipleProtocolInterfaces (
135-
&mHandle,
136-
&gEfiUnicodeCollation2ProtocolGuid,
137-
&Unicode2Eng,
138-
NULL
139-
);
140-
ASSERT_EFI_ERROR (Status);
141-
}
142-
} else {
143-
if (FeaturePcdGet (PcdUnicodeCollationSupport)) {
144-
Status = gBS->InstallMultipleProtocolInterfaces (
145-
&mHandle,
146-
&gEfiUnicodeCollationProtocolGuid,
147-
&UnicodeEng,
148-
NULL
149-
);
150-
ASSERT_EFI_ERROR (Status);
151-
} else {
152-
//
153-
// This module must support to produce at least one of Unicode Collation Protocol
154-
// and Unicode Collation 2 Protocol.
155-
//
156-
ASSERT (FALSE);
157-
Status = EFI_UNSUPPORTED;
158-
}
123+
Status = gBS->InstallMultipleProtocolInterfaces (
124+
&mHandle,
125+
&gEfiUnicodeCollation2ProtocolGuid,
126+
&Unicode2Eng,
127+
NULL
128+
);
129+
ASSERT_EFI_ERROR (Status);
159130
}
160131

161132
return Status;

MdeModulePkg/Universal/FvSimpleFileSystemDxe/FvSimpleFileSystemDxe.inf

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,6 @@
6060
[Protocols]
6161
gEfiDevicePathProtocolGuid ## TO_START
6262
gEfiFirmwareVolume2ProtocolGuid ## TO_START
63-
gEfiUnicodeCollationProtocolGuid ## TO_START
6463
gEfiUnicodeCollation2ProtocolGuid ## TO_START
6564
gEfiSimpleFileSystemProtocolGuid ## BY_START
6665

MdeModulePkg/Universal/FvSimpleFileSystemDxe/FvSimpleFileSystemEntryPoint.c

Lines changed: 1 addition & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -271,7 +271,7 @@ InitializeUnicodeCollationSupportWorker (
271271
return Status;
272272
}
273273

274-
Iso639Language = (BOOLEAN)(ProtocolGuid == &gEfiUnicodeCollationProtocolGuid);
274+
Iso639Language = (BOOLEAN)(ProtocolGuid == &gEfiUnicodeCollation2ProtocolGuid);
275275
GetEfiGlobalVariable2 (VariableName, (VOID **)&Language, NULL);
276276

277277
ReturnStatus = EFI_UNSUPPORTED;
@@ -350,18 +350,6 @@ InitializeUnicodeCollationSupport (
350350
L"PlatformLang",
351351
(CONST CHAR8 *)PcdGetPtr (PcdUefiVariableDefaultPlatformLang)
352352
);
353-
//
354-
// If the attempt to use Unicode Collation 2 Protocol fails, then we fall back
355-
// on the ISO 639-2 Unicode Collation Protocol.
356-
//
357-
if (EFI_ERROR (Status)) {
358-
Status = InitializeUnicodeCollationSupportWorker (
359-
AgentHandle,
360-
&gEfiUnicodeCollationProtocolGuid,
361-
L"Lang",
362-
(CONST CHAR8 *)PcdGetPtr (PcdUefiVariableDefaultLang)
363-
);
364-
}
365353

366354
return Status;
367355
}

0 commit comments

Comments
 (0)