Re: [PATCH] pinctrl: imx1-core populate subdevices

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On Wednesday, November 27, 2013 03:31 PM, Markus Pargmann wrote:
Hi,

On Wed, Nov 27, 2013 at 11:33:15AM +0800, Chris Ruehl wrote:
On Wednesday, November 20, 2013 04:01 AM, Linus Walleij wrote:
On Mon, Nov 11, 2013 at 7:19 PM, Markus Pargmann<mpa@xxxxxxxxxxxxxx>   wrote:

Support gpio devicetree subnodes to allow a more detailed DT hardware
description.

Signed-off-by: Markus Pargmann<mpa@xxxxxxxxxxxxxx>

Looks good to me.

Tentatively applied this, the DT people may want to say something
about this.

Yours,
Linus Walleij
--

Hi,

I tried the new configuration but the kernel oops when probing the
dm9000 caused by insufficient resources more likely the real reason
that somehow the gpio6: triggers a problem


[    0.115718] irq: no irq domain found for
/soc/aipi@10000000/iomuxc@10015000/gpio@10015500 !
[    0.115768] ------------[ cut here ]------------
[    0.115853] WARNING: CPU: 0 PID: 1 at drivers/of/platform.c:171
of_device_alloc+0x110/0x15c()
[    0.117824] ---[ end trace 0e10ae727e6e0ab1 ]---
[    0.121236] Failed to request board ext/int gpios
[    0.152712] imx27-pinctrl 10015000.iomuxc: initialized IMX pinctrl driver

The imx27-pinctrl driver seems to be initialized after the irq is
requested. So this issue should be the same as in [1].

Regards,

Markus

Markus,

Thank you for point this out!  I patched the Linux-next and
my problems went away. I will test now a bit more..

Cheers
Chris


[1] [PATCH] of/platform: Fix no irq domain found errors when populating interrupts
http://archive.arm.linux.org.uk/lurker/message/20131123.004335.61d5c21b.en.html



iomuxc: iomuxc@10015000 {
         compatible = "fsl,imx27-iomuxc";
         reg =<0x10015000 0x600>;
         #address-cells =<1>;
         #size-cells =<1>;
         ranges;

         gpio1: gpio@10015000 {
              compatible = "fsl,imx27-gpio", "fsl,imx21-gpio";
              reg =<0x10015000 0x100>;
              interrupts =<8>;
              gpio-controller;
              #gpio-cells =<2>;
              interrupt-controller;
              #interrupt-cells =<2>;
         };
...
         gpio6: gpio@10015500 {
              compatible = "fsl,imx27-gpio", "fsl,imx21-gpio";
              reg =<0x10015500 0x100>;
              interrupts =<8>;
              gpio-controller;
              #gpio-cells =<2>;
              interrupt-controller;
              #interrupt-cells =<2>;
         };
};

Kernel Linux version 3.13.0-rc1-next-20131125-dirty

With kind regards
Chris




--
GTSYS Limited RFID Technology
A01 24/F Gold King Industrial Bld
35-41 Tai Lin Pai Road, Kwai Chung, Hong Kong
Fax (852) 8167 4060 - Tel (852) 3598 9488

Disclaimer: http://www.gtsys.com.hk/email/classified.html
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux