On Mon, 21 Nov 2016, Zubair Lutfullah Kakakhel wrote:
> On 11/18/2016 01:29 PM, Thomas Gleixner wrote:
> I'll elaborate on the commit message.
> 
> The DT node in arch/powerpc for this driver is
> 
> intc_0: interrupt-controller@81800000 {
>                       #interrupt-cells = <0x2>;
>                       compatible = "xlnx,xps-intc-1.00.a";
>                       interrupt-controller ;
>                       reg = < 0x81800000 0x10000 >;
>                       xlnx,num-intr-inputs = <0xc>;
>               } ;
> 
> The DT node in arch/microblaze for this driver is
> 
> intc_0: interrupt-controller@81800000 {
>                       #interrupt-cells = <0x2>;
>                       compatible = "xlnx,xps-intc-1.00.a";
>                       interrupt-controller ;
>                       reg = < 0x81800000 0x10000 >;
>                       xlnx,kind-of-intr = <0x100>;    //<Missing from ppc>!
>                       xlnx,num-intr-inputs = <0x9>;
>               } ;
> 
> The PPC driver assumes the kind-of-intr value be 0x0 and doesn't specify it in
> DT.
> This patch makes that a fall back case. Instead of completely error-ing out.

Ok. makes sense. I misread the changelog/patch.

Thanks,

        tglx

Reply via email to