[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <87bjt9wq3b.ffs@tglx>
Date: Sun, 06 Apr 2025 12:18:16 +0200
From: Thomas Gleixner <tglx@...utronix.de>
To: Huacai Chen <chenhuacai@...il.com>
Cc: Huacai Chen <chenhuacai@...ngson.cn>, loongarch@...ts.linux.dev,
linux-kernel@...r.kernel.org, Xuefeng Li <lixuefeng@...ngson.cn>, Jiaxun
Yang <jiaxun.yang@...goat.com>, stable@...r.kernel.org, Yinbo Zhu
<zhuyinbo@...ngson.cn>
Subject: Re: [PATCH] irqchip/loongson-liointc: Support to set
IRQ_TYPE_EDGE_BOTH
On Sun, Apr 06 2025 at 17:46, Huacai Chen wrote:
> On Thu, Apr 3, 2025 at 11:48 PM Thomas Gleixner <tglx@...utronix.de> wrote:
>> But it won't trigger on both. So no, you cannot claim that this fixes
>> anything.
> Yes, it won't trigger on both (not perfect), but it allows drivers
> that request "both" work (better than fail to request), and there are
By some definition of 'work'. There is probably a good technical reason
why those drivers expect EDGE_BOTH to work correctly and otherwise fail
to load.
You completely fail to explain, why this hack actually 'works' and what
the implications are for such drivers.
> other irqchip drivers that do similar things.
Justifying bogosity with already existing bogosity is not a technical
argument.
Thanks,
tglx
Powered by blists - more mailing lists