On 08/10/2024 13:00, Vladimir Zapolskiy wrote:
Rising or High can both be justified, its really down to how your
interrupt controller latches the state change. However I personally am
fine with the change you've provided because I trust it fixes an error
for you.
Please share the change to the driver, which you've used to test
high level type of interrupts, shall it be send for upstream inclusion?
Such a change has never been a subject of discussion.
I tried running libcamera "cam" application to capture a data stream
before and after your change - from memory at least on the sc8280xp and
I think on 8250 too.
What I haven't tested is unloading and reloading the kernel module. My
understanding of your bug report is your change fixes an error on reload.
?
---
bod
