Re: [PATCH 01/21] arm64: Add ADI ADSP-SC598 SoC

From: Markus Elfring
Date: Sat Sep 14 2024 - 13:16:06 EST



> +++ b/drivers/soc/adi/system.c
> @@ -0,0 +1,257 @@

> +static void adi_system_config_remove(struct platform_device *pdev)
+{
> + struct adi_system_config *config = platform_get_drvdata(pdev);
> + unsigned long flags;
> +
> + spin_lock_irqsave(&adi_system_config_lock, flags);
> + list_del(&config->list);
> + spin_unlock_irqrestore(&adi_system_config_lock, flags);
> +}


Under which circumstances would you become interested to apply a statement
like “guard(spinlock_irqsave)(&adi_system_config_lock);”?
https://elixir.bootlin.com/linux/v6.11-rc7/source/include/linux/spinlock.h#L572

Regards,
Markus