Re: [PATCH] efi: Unify dmi setup code over architectures arm/arm64, io64 and x86
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- To: Tony Luck <tony.luck@xxxxxxxxx>, Fenghua Yu <fenghua.yu@xxxxxxxxx>, "Thomas Gleixner" <tglx@xxxxxxxxxxxxx>, Ingo Molnar <mingo@xxxxxxxxxx>, "Borislav Petkov" <bp@xxxxxxxxx>, "H. Peter Anvin" <hpa@xxxxxxxxx>, "x86@xxxxxxxxxx" <x86@xxxxxxxxxx>, Jean Delvare <jdelvare@xxxxxxxx>, Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx>
- Subject: Re: [PATCH] efi: Unify dmi setup code over architectures arm/arm64, io64 and x86
- From: Robert Richter <rrichter@xxxxxxxxxxx>
- Date: Wed, 20 Mar 2019 14:48:34 +0000
- Accept-language: en-US
- Cc: Marcin Benka <mbenka@xxxxxxxxxxx>, "linux-ia64@xxxxxxxxxxxxxxx" <linux-ia64@xxxxxxxxxxxxxxx>, "linux-kernel@xxxxxxxxxxxxxxx" <linux-kernel@xxxxxxxxxxxxxxx>, "linux-efi@xxxxxxxxxxxxxxx" <linux-efi@xxxxxxxxxxxxxxx>
- In-reply-to: <20190320131607.vgst3r7ynha55ikw@rric.localdomain>
- References: <20190320130502.16667-1-rrichter@marvell.com> <20190320131607.vgst3r7ynha55ikw@rric.localdomain>
On 20.03.19 13:16:13, Robert Richter wrote:
> diff --git a/drivers/firmware/efi/arm-runtime.c b/drivers/firmware/efi/arm-runtime.c
> index 4a0dfe4ab829..c250b767030e 100644
> --- a/drivers/firmware/efi/arm-runtime.c
> +++ b/drivers/firmware/efi/arm-runtime.c
> @@ -162,14 +162,10 @@ void efi_virtmap_unload(void)
> static int __init arm_dmi_init(void)
> {
> /*
> - * On arm64/ARM, DMI depends on UEFI, and dmi_scan_machine() needs to
> + * On arm64/ARM, DMI depends on UEFI, and dmi_setup() needs to
> * be called early because dmi_id_init(), which is an arch_initcall
> * itself, depends on dmi_scan_machine() having been called already.
> */
> - dmi_scan_machine();
> - dmi_memdev_walk();
> - if (dmi_available)
> - dmi_set_dump_stack_arch_desc();
This went wrong in the last stage of a comment rework. A dmi_setup()
should be here. Will resubmit. Sorry.
-Robert
> return 0;
> }
[Index of Archives]
[Linux Kernel]
[Sparc Linux]
[DCCP]
[Linux ARM]
[Yosemite News]
[Linux SCSI]
[Linux x86_64]
[Linux for Ham Radio]