Am 22.06.24 um 15:49 schrieb Arend Van Spriel:
On June 22, 2024 3:38:32 PM Arend Van SprielSure, if you insist on it, I can update the bindings document. So far not
<arend.vanspriel@xxxxxxxxxxxx> wrote:
On June 22, 2024 12:56:02 AM Alex Bee <knaerzche@xxxxxxxxx> wrote:
This "new" version of defining external interrupts is around for a very
long time now and supported and preferred by irq_of_parse_and_map
respectively of_irq_parse_one.
Support it in brcmfmac as well by checking if either "interrupts" or
"interrupts-extended" property exists as indication if
irq_of_parse_and_map
should be called.
All very interesting, but why should we add code for something that
is not
specified in the bindings documentation?
NAK (for now). Feel free to update the bindings document.
many (no) users did that, as it is clearly specified as an alternative in
devicetree/bindings/interrupt-controller/interrupts.txt (sure, it's not yet
converted to yaml yet).
Which is a much nicer form, imho.
And by the way for instance
arch/arm/boot/dts/qcom/qcom-apq8026-lg-lenok.dts already uses it that way
and the interrupt will currently not picked up (at least not by this
driver).
Attachment:
smime.p7s
Description: S/MIME Cryptographic Signature