On 08/10/2024 12:37, Vladimir Zapolskiy wrote:
I don't have access to datasheets or hardware of sc8280xp powered board,
someone may either verify, if CAMSS level high type interrupts are
supported/working at all or not (obviously its current presence in dts is
insufficient), or check the SoC datasheet.
I've tested both as was submitted and your change.
I _always_ test my patches. I'm not sure there's a datasheet which
spells this out to be honest.
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.
I didn't try loading and unloading that module but, since you did I'm
happy to Ack the change and trust your work.
---
bod
