Messages in this thread | | | Date | Thu, 17 Aug 2017 16:19:51 -0500 | From | Rob Herring <> | Subject | Re: [PATCH] ASoC: mediatek: Correct the interrupt property |
| |
On Tue, Aug 15, 2017 at 09:51:40PM +0800, Ryder Lee wrote: > On Tue, 2017-08-15 at 12:53 +0100, Mark Brown wrote: > > On Tue, Aug 15, 2017 at 04:07:02PM +0800, Ryder Lee wrote: > > > > > This patch update interrupt property since we only use ASYS IRQ > > > in the driver, and this may confuse people. > > > > > - interrupts = <GIC_SPI 104 IRQ_TYPE_LEVEL_LOW>, > > > - <GIC_SPI 132 IRQ_TYPE_LEVEL_LOW>; > > > + interrupts = <GIC_SPI 132 IRQ_TYPE_LEVEL_LOW>; > > > > What harm does having the extra information in the bindings do? If it's > > possible there might be a use for the extra interrupt it seems better to > > have people describe it.
Agreed. It should describe interrupts the h/w has, not what the driver (currently) uses. > > Yes you're right. The current driver get IRQ number by index "0" but > actually it should be "1" (GIC 132). Perhaps we can switch to use the > platform_get_irq_byname() and add interrupt-name in DT so that binding > can be agnostic of the IRQ order.
The binding is wrong already in that it says a single interrrupt. The example has 2 though. Add the 2nd interrupt and define the order such that you don't break the current driver. Probably that means ASYS is first.
Rob
| |