[edk2-devel] [PATCH] DynamicTablesPkg: FdtHwInfoParserLib: Fix compatible string
Sami Mujawar
sami.mujawar at arm.com
Fri Jan 6 09:55:21 UTC 2023
Hi Ard, Mortiz,
Please find my response inline marked [SAMI].
Regards,
Sami Mujawar
On 06/01/2023 09:15 am, Ard Biesheuvel wrote:
> (cc Sami)
>
> On Fri, 6 Jan 2023 at 10:14, Ard Biesheuvel <ardb at kernel.org> wrote:
>> On Fri, 6 Jan 2023 at 00:07, Moritz Fischer via groups.io
>> <moritzf=google.com at groups.io> wrote:
>>> Linux' cpu DT bindings call out arm,armv8 while the code previously
>>> used arm,arm-v8.
>>>
>>> Fixes: e366a41ef0 ("DynamicTablesPkg: FdtHwInfoParser: Add GICC parser")
>>> Signed-off-by: Moritz Fischer <moritzf at google.com>
>>> ---
>>> DynamicTablesPkg/Library/FdtHwInfoParserLib/Gic/ArmGicCParser.c | 2 +-
>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/DynamicTablesPkg/Library/FdtHwInfoParserLib/Gic/ArmGicCParser.c b/DynamicTablesPkg/Library/FdtHwInfoParserLib/Gic/ArmGicCParser.c
>>> index 2c72bd0c58..dc6e7a4d15 100644
>>> --- a/DynamicTablesPkg/Library/FdtHwInfoParserLib/Gic/ArmGicCParser.c
>>> +++ b/DynamicTablesPkg/Library/FdtHwInfoParserLib/Gic/ArmGicCParser.c
>>> @@ -22,7 +22,7 @@
>>> */
>>> STATIC CONST COMPATIBILITY_STR CpuCompatibleStr[] = {
>>> { "arm,arm-v7" },
>>> - { "arm,arm-v8" },
>>> + { "arm,armv8" },
>> Thanks for the patch
>>
>> Should we keep both instead of removing the old one?
[SAMI] I think we would need to keep both as the CPU nodes generated by
Kvmtool have compatible string as "arm,arm-v8", see
https://github.com/kvmtool/kvmtool/blob/master/arm/aarch64/arm-cpu.c#L31.
Mortiz: Is it possible to share some information about your use case,
please?
[/SAMI]
>>
>>> { "arm,cortex-a15" },
>>> { "arm,cortex-a7" },
>>> { "arm,cortex-a57" }
>>> --
>>> 2.39.0.314.g84b9a713c41-goog
>>>
>>>
>>>
>>>
>>>
>>>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#98095): https://edk2.groups.io/g/devel/message/98095
Mute This Topic: https://groups.io/mt/96083588/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-
More information about the edk2-devel-archive
mailing list