I can give an Acked-by if these lines are also removed in our internal repo.
Huacai
On Thu, Jan 18, 2024 at 8:15 PM Bibo Mao <maobibo@xxxxxxxxxxx> wrote:
>
> During suspend and resume, CPUs except CPU0 are hot-unpluged and IRQs
> are migrated to CPU0. So it is not necessary to restore irq affinity for
> eiointc irq controller when system resumes. This patch removes the piece
> of code about irq affinity restoring in function eiointc_resume.
>
> Signed-off-by: Bibo Mao <maobibo@xxxxxxxxxxx>
> ---
> drivers/irqchip/irq-loongson-eiointc.c | 16 ----------------
> 1 file changed, 16 deletions(-)
>
> diff --git a/drivers/irqchip/irq-loongson-eiointc.c b/drivers/irqchip/irq-loongson-eiointc.c
> index 6143adb1b73b..86f4faad0695 100644
> --- a/drivers/irqchip/irq-loongson-eiointc.c
> +++ b/drivers/irqchip/irq-loongson-eiointc.c
> @@ -315,23 +315,7 @@ static int eiointc_suspend(void)
>
> static void eiointc_resume(void)
> {
> - int i, j;
> - struct irq_desc *desc;
> - struct irq_data *irq_data;
> -
> eiointc_router_init(0);
> -
> - for (i = 0; i < nr_pics; i++) {
> - for (j = 0; j < eiointc_priv[0]->vec_count; j++) {
> - desc = irq_resolve_mapping(eiointc_priv[i]->eiointc_domain, j);
> - if (desc && desc->handle_irq && desc->handle_irq != handle_bad_irq) {
> - raw_spin_lock(&desc->lock);
> - irq_data = irq_domain_get_irq_data(eiointc_priv[i]->eiointc_domain, irq_desc_get_irq(desc));
> - eiointc_set_irq_affinity(irq_data, irq_data->common->affinity, 0);
> - raw_spin_unlock(&desc->lock);
> - }
> - }
> - }
> }
>
> static struct syscore_ops eiointc_syscore_ops = {
> --
> 2.39.3
>
