On 09:53-20141209, Marc Zyngier wrote: > On 08/12/14 22:41, Nishanth Menon wrote: > > > Anyways.. The following diff[1] on top of your branch makes DRA7 work - I > > assume you will squash as needed and repost with linux-omap mailing list > > in CC. > > Brilliant. I'll squash that into my tree and repost at some point. K, it will be nice to have a reflow of the series based on v3.19-rc1 since there are dts dependencies and we dont want folks to have regressions on their platforms of choice.. Obviously, my tests are basic boot tests and should get a few weeks(as you already mentioned) on linux-next to get properly soaked > > > I increased the scope of testing knowing that WUGEN is present in many > > A9 based TI platforms as well.. and at least OMAP4 showed flakiness in > > my testing.. Also a few notes: > > > > Stuff like: am437x is a bit questionable (interrupt-parent probably should be wugen?) > > 175: 0 GIC 39 tps65218 > > > > OMAP5: (should be wugen?) > > 308: 4323 0 GIC 106 OMAP UART2 > > 411: 0 0 GIC 151 twl6040 > > 405: 1 0 GIC 39 palmas > > Well, I can't really tell. Someone with access to the documentation > should be able to find out. AM437x: http://www.ti.com/lit/pdf/spruhl7 OMAP5: http://www.ti.com/lit/pdf/swpu249 yeah, we should be able to do them as well - trivially since they follow the same structure as other SoCs without crossbar. > > > OMAP4 serial port is flaky -> not sure if it is due to routing of GIC to UART2 and not via WUGEN > > IRQ branch: with my fix applied: > > --------------------------------- > > [...] > > > 18: pandaboard-es: Boot FAIL: http://slexy.org/raw/s20ty0Z6i5 (not expected) > > 19: pandaboard-vanilla: Boot FAIL: http://slexy.org/raw/s20BYfaMd2 (not expected) > > If I read the log correctly, the serial port stops responding after a while? yeah - dug at the omap4 ones a bit, obviously once the deeper c states are hit, we'd like wakeupgen to wakeup CPU else we will be "sluggish" in the sense that the event is detected when some other wakeupgen enabled interrupt takes place. Adding the following makes my panda work fine. 1: pandaboard-es: Boot PASS: http://slexy.org/raw/s20o8DaBvh 2: pandaboard-vanilla: Boot PASS: http://slexy.org/raw/s222JndDdh diff --git a/arch/arm/boot/dts/omap4-panda-common.dtsi b/arch/arm/boot/dts/omap4-panda-common.dtsi index 1505135..8b6d50e 100644 --- a/arch/arm/boot/dts/omap4-panda-common.dtsi +++ b/arch/arm/boot/dts/omap4-panda-common.dtsi @@ -371,8 +371,8 @@ twl: twl@48 { reg = <0x48>; /* IRQ# = 7 */ - interrupts = <GIC_SPI 7 IRQ_TYPE_LEVEL_HIGH>; /* IRQ_SYS_1N cascaded to gic */ - interrupt-parent = <&gic>; + interrupts = <GIC_SPI 7 IRQ_TYPE_LEVEL_HIGH>; /* IRQ_SYS_1N cascaded to wakeupgen to gic */ + interrupt-parent = <&wakeupgen>; }; twl6040: twl@4b { @@ -383,8 +383,8 @@ pinctrl-0 = <&twl6040_pins>; /* IRQ# = 119 */ - interrupts = <GIC_SPI 119 IRQ_TYPE_LEVEL_HIGH>; /* IRQ_SYS_2N cascaded to gic */ - interrupt-parent = <&gic>; + interrupts = <GIC_SPI 119 IRQ_TYPE_LEVEL_HIGH>; /* IRQ_SYS_2N cascaded to wakeupgen to gic */ + interrupt-parent = <&wakeupgen>; ti,audpwron-gpio = <&gpio4 31 GPIO_ACTIVE_HIGH>; /* gpio line 127 */ vio-supply = <&v1v8>; @@ -479,17 +479,17 @@ }; &uart2 { - interrupts-extended = <&gic GIC_SPI 73 IRQ_TYPE_LEVEL_HIGH + interrupts-extended = <&wakeupgen GIC_SPI 73 IRQ_TYPE_LEVEL_HIGH &omap4_pmx_core OMAP4_UART2_RX>; }; &uart3 { - interrupts-extended = <&gic GIC_SPI 74 IRQ_TYPE_LEVEL_HIGH + interrupts-extended = <&wakeupgen GIC_SPI 74 IRQ_TYPE_LEVEL_HIGH &omap4_pmx_core OMAP4_UART3_RX>; }; &uart4 { - interrupts-extended = <&gic GIC_SPI 70 IRQ_TYPE_LEVEL_HIGH + interrupts-extended = <&wakeupgen GIC_SPI 70 IRQ_TYPE_LEVEL_HIGH &omap4_pmx_core OMAP4_UART4_RX>; }; -- Regards, Nishanth Menon -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
