On Sun, Oct 28, 2018 at 6:15 PM Adam Ford <aford173@xxxxxxxxx> wrote:
>
> On Sun, Oct 28, 2018 at 6:09 PM Adam Ford <aford173@xxxxxxxxx> wrote:
> >
> > MMCx_dat1 and serial RX pins can be associated to additional
> > interrupts. Enable these on AM3517-evm.
> >
>
> The reason, I made this an RFC, is that I think there might be some
> IRQ support missing in AM3517, and I am not sure where to look.
>
> During boot, I get multiple instances of the following message:
>
> irq: no irq domain found for pinmux@30 !
>
I should also note, that the am3517 is also returning
pm: Failed to request pm_wkup irq
> They seem to correspond to each of the newly added
> 'interrupts-extended' entries.
>
> Looking at the technical reference manual for the AM3517, it appears
> as if this 'should' be possible.
>
> When cat /proc/interrupts on a different omap36/37 board, I can see
> the irq entries:
>
> 147: 0 pinctrl 338 Edge 4806a000.serial:wakeup
> 149: 0 pinctrl 282 Edge 4809c000.mmc:wakeup
> 150: 0 pinctrl 310 Edge 480ad000.mmc:wakeup
>
> However, when I look on the AM3517, no pinctrl entries exist in
> /proc/interrupts, so I am guessing something needs to happen to setup
> this up. I am curious to know if there is a missing component
> somewhere that needs to enable this.
>
> thanks in advance,
>
> adam
>
> > Signed-off-by: Adam Ford <aford173@xxxxxxxxx>
> >
> > diff --git a/arch/arm/boot/dts/am3517-evm.dts b/arch/arm/boot/dts/am3517-evm.dts
> > index c45aef806870..c518c51eb198 100644
> > --- a/arch/arm/boot/dts/am3517-evm.dts
> > +++ b/arch/arm/boot/dts/am3517-evm.dts
> > @@ -221,6 +221,7 @@
> > };
> >
> > &mmc1 {
> > + interrupts-extended = <&intc 83 &omap3_pmx_core 0x11a>;
> > status = "okay";
> > pinctrl-names = "default";
> > pinctrl-0 = <&mmc1_pins>;
> > diff --git a/arch/arm/boot/dts/am3517-som.dtsi b/arch/arm/boot/dts/am3517-som.dtsi
> > index b1c988eed87c..ae93dd72020a 100644
> > --- a/arch/arm/boot/dts/am3517-som.dtsi
> > +++ b/arch/arm/boot/dts/am3517-som.dtsi
> > @@ -148,7 +148,7 @@
> > };
> >
> > &mmc2 {
> > - interrupts-extended = <&intc 86 /* &omap3_pmx_core 0x12c */>;
> > + interrupts-extended = <&intc 86 &omap3_pmx_core 0x12e>;
> >
> > status = "okay";
> > pinctrl-names = "default";
> > @@ -172,6 +172,7 @@
> > &uart2 {
> > pinctrl-names = "default";
> > pinctrl-0 = <&uart2_pins>;
> > + interrupts-extended = <&intc 73 &omap3_pmx_core OMAP3_UART2_RX>;
> >
> > bluetooth {
> > compatible = "ti,wl1271-st";
> > --
> > 2.17.1
> >
