On Tue, Oct 10, 2023 at 8:11 PM D Scott Phillips
<scott@xxxxxxxxxxxxxxxxxxxxxx> wrote:
>
> Hanjun Guo <guohanjun@xxxxxxxxxx> writes:
>
> > acpi_agdi_init() in acpi_arm_init() will register a SDEI event, so
> > it needs the SDEI subsystem to be initialized (which is done in
> > acpi_ghes_init()) before the AGDI driver probing.
> >
> > In commit fcea0ccf4fd7 ("ACPI: bus: Consolidate all arm specific
> > initialisation into acpi_arm_init()"), the acpi_agdi_init() was
> > called before acpi_ghes_init() and it causes following failure:
> >
> > | [ 0.515864] sdei: Failed to create event 1073741825: -5
> > | [ 0.515866] agdi agdi.0: Failed to register for SDEI event 1073741825
> > | [ 0.515867] agdi: probe of agdi.0 failed with error -5
> > | ...
> > | [ 0.516022] sdei: SDEIv1.0 (0x0) detected in firmware.
> >
> > Fix it by moving acpi_arm_init() to the place of after
> > acpi_ghes_init().
> >
> > Fixes: fcea0ccf4fd7 ("ACPI: bus: Consolidate all arm specific initialisation into acpi_arm_init()")
> > Reported-by: D Scott Phillips <scott@xxxxxxxxxxxxxxxxxxxxxx>
> > Signed-off-by: Hanjun Guo <guohanjun@xxxxxxxxxx>
>
> Tested-by: D Scott Phillips <scott@xxxxxxxxxxxxxxxxxxxxxx>
>
> Thanks Hanjun and Sudeep
Applied as 6.6-rc material, thanks!
