OSDN Git Service

ARM: dts: exynos: Fix invalid GIC interrupt flags in exynos5250
authorKrzysztof Kozlowski <krzk@kernel.org>
Fri, 16 Sep 2016 19:42:48 +0000 (21:42 +0200)
committerKrzysztof Kozlowski <krzk@kernel.org>
Thu, 3 Nov 2016 20:44:53 +0000 (22:44 +0200)
Interrupt of type IRQ_TYPE_NONE is not allowed for GIC interrupts and
generates an error:
genirq: Setting trigger mode 0 for irq 16 failed (gic_set_type+0x0/0x68)

The GIC requires shared interrupts to be edge rising or level high.
Platform declares support for both.  Arbitrarily choose level high
everywhere hoping it will work on each platform.

Reported-by: Marek Szyprowski <m.szyprowski@samsung.com>
Reported-by: Alban Browaeys <alban.browaeys@gmail.com>
Cc: Marc Zyngier <marc.zyngier@arm.com>
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
Reviewed-by: Alim Akhtar <alim.akhtar@samsung.com>
arch/arm/boot/dts/exynos5250.dtsi

index f7357d9..4e00c0d 100644 (file)
                                                <0x1 0 &combiner 23 4>,
                                                <0x2 0 &combiner 25 2>,
                                                <0x3 0 &combiner 25 3>,
-                                               <0x4 0 &gic 0 120 0>,
-                                               <0x5 0 &gic 0 121 0>;
+                                               <0x4 0 &gic 0 120 IRQ_TYPE_LEVEL_HIGH>,
+                                               <0x5 0 &gic 0 121 IRQ_TYPE_LEVEL_HIGH>;
                        };
                };
 
                pinctrl_0: pinctrl@11400000 {
                        compatible = "samsung,exynos5250-pinctrl";
                        reg = <0x11400000 0x1000>;
-                       interrupts = <0 46 0>;
+                       interrupts = <0 46 IRQ_TYPE_LEVEL_HIGH>;
 
                        wakup_eint: wakeup-interrupt-controller {
                                compatible = "samsung,exynos4210-wakeup-eint";
                                interrupt-parent = <&gic>;
-                               interrupts = <0 32 0>;
+                               interrupts = <0 32 IRQ_TYPE_LEVEL_HIGH>;
                        };
                };
 
                pinctrl_1: pinctrl@13400000 {
                        compatible = "samsung,exynos5250-pinctrl";
                        reg = <0x13400000 0x1000>;
-                       interrupts = <0 45 0>;
+                       interrupts = <0 45 IRQ_TYPE_LEVEL_HIGH>;
                };
 
                pinctrl_2: pinctrl@10d10000 {
                        compatible = "samsung,exynos5250-pinctrl";
                        reg = <0x10d10000 0x1000>;
-                       interrupts = <0 50 0>;
+                       interrupts = <0 50 IRQ_TYPE_LEVEL_HIGH>;
                };
 
                pinctrl_3: pinctrl@03860000 {
                        compatible = "samsung,exynos5250-pinctrl";
                        reg = <0x03860000 0x1000>;
-                       interrupts = <0 47 0>;
+                       interrupts = <0 47 IRQ_TYPE_LEVEL_HIGH>;
                };
 
                pmu_system_controller: system-controller@10040000 {
                watchdog@101D0000 {
                        compatible = "samsung,exynos5250-wdt";
                        reg = <0x101D0000 0x100>;
-                       interrupts = <0 42 0>;
+                       interrupts = <0 42 IRQ_TYPE_LEVEL_HIGH>;
                        clocks = <&clock CLK_WDT>;
                        clock-names = "watchdog";
                        samsung,syscon-phandle = <&pmu_system_controller>;
                g2d@10850000 {
                        compatible = "samsung,exynos5250-g2d";
                        reg = <0x10850000 0x1000>;
-                       interrupts = <0 91 0>;
+                       interrupts = <0 91 IRQ_TYPE_LEVEL_HIGH>;
                        clocks = <&clock CLK_G2D>;
                        clock-names = "fimg2d";
                        iommus = <&sysmmu_g2d>;
                mfc: codec@11000000 {
                        compatible = "samsung,mfc-v6";
                        reg = <0x11000000 0x10000>;
-                       interrupts = <0 96 0>;
+                       interrupts = <0 96 IRQ_TYPE_LEVEL_HIGH>;
                        power-domains = <&pd_mfc>;
                        clocks = <&clock CLK_MFC>;
                        clock-names = "mfc";
                rotator: rotator@11C00000 {
                        compatible = "samsung,exynos5250-rotator";
                        reg = <0x11C00000 0x64>;
-                       interrupts = <0 84 0>;
+                       interrupts = <0 84 IRQ_TYPE_LEVEL_HIGH>;
                        clocks = <&clock CLK_ROTATOR>;
                        clock-names = "rotator";
                        iommus = <&sysmmu_rotator>;
                tmu: tmu@10060000 {
                        compatible = "samsung,exynos5250-tmu";
                        reg = <0x10060000 0x100>;
-                       interrupts = <0 65 0>;
+                       interrupts = <0 65 IRQ_TYPE_LEVEL_HIGH>;
                        clocks = <&clock CLK_TMU>;
                        clock-names = "tmu_apbif";
                        #include "exynos4412-tmu-sensor-conf.dtsi"
                        compatible = "snps,dwc-ahci";
                        samsung,sata-freq = <66>;
                        reg = <0x122F0000 0x1ff>;
-                       interrupts = <0 115 0>;
+                       interrupts = <0 115 IRQ_TYPE_LEVEL_HIGH>;
                        clocks = <&clock CLK_SATA>, <&clock CLK_SCLK_SATA>;
                        clock-names = "sata", "sclk_sata";
                        phys = <&sata_phy>;
                i2c_4: i2c@12CA0000 {
                        compatible = "samsung,s3c2440-i2c";
                        reg = <0x12CA0000 0x100>;
-                       interrupts = <0 60 0>;
+                       interrupts = <0 60 IRQ_TYPE_LEVEL_HIGH>;
                        #address-cells = <1>;
                        #size-cells = <0>;
                        clocks = <&clock CLK_I2C4>;
                i2c_5: i2c@12CB0000 {
                        compatible = "samsung,s3c2440-i2c";
                        reg = <0x12CB0000 0x100>;
-                       interrupts = <0 61 0>;
+                       interrupts = <0 61 IRQ_TYPE_LEVEL_HIGH>;
                        #address-cells = <1>;
                        #size-cells = <0>;
                        clocks = <&clock CLK_I2C5>;
                i2c_6: i2c@12CC0000 {
                        compatible = "samsung,s3c2440-i2c";
                        reg = <0x12CC0000 0x100>;
-                       interrupts = <0 62 0>;
+                       interrupts = <0 62 IRQ_TYPE_LEVEL_HIGH>;
                        #address-cells = <1>;
                        #size-cells = <0>;
                        clocks = <&clock CLK_I2C6>;
                i2c_7: i2c@12CD0000 {
                        compatible = "samsung,s3c2440-i2c";
                        reg = <0x12CD0000 0x100>;
-                       interrupts = <0 63 0>;
+                       interrupts = <0 63 IRQ_TYPE_LEVEL_HIGH>;
                        #address-cells = <1>;
                        #size-cells = <0>;
                        clocks = <&clock CLK_I2C7>;
                i2c_8: i2c@12CE0000 {
                        compatible = "samsung,s3c2440-hdmiphy-i2c";
                        reg = <0x12CE0000 0x1000>;
-                       interrupts = <0 64 0>;
+                       interrupts = <0 64 IRQ_TYPE_LEVEL_HIGH>;
                        #address-cells = <1>;
                        #size-cells = <0>;
                        clocks = <&clock CLK_I2C_HDMI>;
                        compatible = "samsung,exynos4210-spi";
                        status = "disabled";
                        reg = <0x12d20000 0x100>;
-                       interrupts = <0 66 0>;
+                       interrupts = <0 66 IRQ_TYPE_LEVEL_HIGH>;
                        dmas = <&pdma0 5
                                &pdma0 4>;
                        dma-names = "tx", "rx";
                        compatible = "samsung,exynos4210-spi";
                        status = "disabled";
                        reg = <0x12d30000 0x100>;
-                       interrupts = <0 67 0>;
+                       interrupts = <0 67 IRQ_TYPE_LEVEL_HIGH>;
                        dmas = <&pdma1 5
                                &pdma1 4>;
                        dma-names = "tx", "rx";
                        compatible = "samsung,exynos4210-spi";
                        status = "disabled";
                        reg = <0x12d40000 0x100>;
-                       interrupts = <0 68 0>;
+                       interrupts = <0 68 IRQ_TYPE_LEVEL_HIGH>;
                        dmas = <&pdma0 7
                                &pdma0 6>;
                        dma-names = "tx", "rx";
 
                mmc_0: mmc@12200000 {
                        compatible = "samsung,exynos5250-dw-mshc";
-                       interrupts = <0 75 0>;
+                       interrupts = <0 75 IRQ_TYPE_LEVEL_HIGH>;
                        #address-cells = <1>;
                        #size-cells = <0>;
                        reg = <0x12200000 0x1000>;
 
                mmc_1: mmc@12210000 {
                        compatible = "samsung,exynos5250-dw-mshc";
-                       interrupts = <0 76 0>;
+                       interrupts = <0 76 IRQ_TYPE_LEVEL_HIGH>;
                        #address-cells = <1>;
                        #size-cells = <0>;
                        reg = <0x12210000 0x1000>;
 
                mmc_2: mmc@12220000 {
                        compatible = "samsung,exynos5250-dw-mshc";
-                       interrupts = <0 77 0>;
+                       interrupts = <0 77 IRQ_TYPE_LEVEL_HIGH>;
                        #address-cells = <1>;
                        #size-cells = <0>;
                        reg = <0x12220000 0x1000>;
                mmc_3: mmc@12230000 {
                        compatible = "samsung,exynos5250-dw-mshc";
                        reg = <0x12230000 0x1000>;
-                       interrupts = <0 78 0>;
+                       interrupts = <0 78 IRQ_TYPE_LEVEL_HIGH>;
                        #address-cells = <1>;
                        #size-cells = <0>;
                        clocks = <&clock CLK_SDMMC3>, <&clock CLK_SCLK_MMC3>;
                        usbdrd_dwc3: dwc3@12000000 {
                                compatible = "synopsys,dwc3";
                                reg = <0x12000000 0x10000>;
-                               interrupts = <0 72 0>;
+                               interrupts = <0 72 IRQ_TYPE_LEVEL_HIGH>;
                                phys = <&usbdrd_phy 0>, <&usbdrd_phy 1>;
                                phy-names = "usb2-phy", "usb3-phy";
                        };
                ehci: usb@12110000 {
                        compatible = "samsung,exynos4210-ehci";
                        reg = <0x12110000 0x100>;
-                       interrupts = <0 71 0>;
+                       interrupts = <0 71 IRQ_TYPE_LEVEL_HIGH>;
 
                        clocks = <&clock CLK_USB2>;
                        clock-names = "usbhost";
                ohci: usb@12120000 {
                        compatible = "samsung,exynos4210-ohci";
                        reg = <0x12120000 0x100>;
-                       interrupts = <0 71 0>;
+                       interrupts = <0 71 IRQ_TYPE_LEVEL_HIGH>;
 
                        clocks = <&clock CLK_USB2>;
                        clock-names = "usbhost";
                        pdma0: pdma@121A0000 {
                                compatible = "arm,pl330", "arm,primecell";
                                reg = <0x121A0000 0x1000>;
-                               interrupts = <0 34 0>;
+                               interrupts = <0 34 IRQ_TYPE_LEVEL_HIGH>;
                                clocks = <&clock CLK_PDMA0>;
                                clock-names = "apb_pclk";
                                #dma-cells = <1>;
                        pdma1: pdma@121B0000 {
                                compatible = "arm,pl330", "arm,primecell";
                                reg = <0x121B0000 0x1000>;
-                               interrupts = <0 35 0>;
+                               interrupts = <0 35 IRQ_TYPE_LEVEL_HIGH>;
                                clocks = <&clock CLK_PDMA1>;
                                clock-names = "apb_pclk";
                                #dma-cells = <1>;
                        mdma0: mdma@10800000 {
                                compatible = "arm,pl330", "arm,primecell";
                                reg = <0x10800000 0x1000>;
-                               interrupts = <0 33 0>;
+                               interrupts = <0 33 IRQ_TYPE_LEVEL_HIGH>;
                                clocks = <&clock CLK_MDMA0>;
                                clock-names = "apb_pclk";
                                #dma-cells = <1>;
                        mdma1: mdma@11C10000 {
                                compatible = "arm,pl330", "arm,primecell";
                                reg = <0x11C10000 0x1000>;
-                               interrupts = <0 124 0>;
+                               interrupts = <0 124 IRQ_TYPE_LEVEL_HIGH>;
                                clocks = <&clock CLK_MDMA1>;
                                clock-names = "apb_pclk";
                                #dma-cells = <1>;
                gsc_0:  gsc@13e00000 {
                        compatible = "samsung,exynos5-gsc";
                        reg = <0x13e00000 0x1000>;
-                       interrupts = <0 85 0>;
+                       interrupts = <0 85 IRQ_TYPE_LEVEL_HIGH>;
                        power-domains = <&pd_gsc>;
                        clocks = <&clock CLK_GSCL0>;
                        clock-names = "gscl";
                gsc_1:  gsc@13e10000 {
                        compatible = "samsung,exynos5-gsc";
                        reg = <0x13e10000 0x1000>;
-                       interrupts = <0 86 0>;
+                       interrupts = <0 86 IRQ_TYPE_LEVEL_HIGH>;
                        power-domains = <&pd_gsc>;
                        clocks = <&clock CLK_GSCL1>;
                        clock-names = "gscl";
                gsc_2:  gsc@13e20000 {
                        compatible = "samsung,exynos5-gsc";
                        reg = <0x13e20000 0x1000>;
-                       interrupts = <0 87 0>;
+                       interrupts = <0 87 IRQ_TYPE_LEVEL_HIGH>;
                        power-domains = <&pd_gsc>;
                        clocks = <&clock CLK_GSCL2>;
                        clock-names = "gscl";
                gsc_3:  gsc@13e30000 {
                        compatible = "samsung,exynos5-gsc";
                        reg = <0x13e30000 0x1000>;
-                       interrupts = <0 88 0>;
+                       interrupts = <0 88 IRQ_TYPE_LEVEL_HIGH>;
                        power-domains = <&pd_gsc>;
                        clocks = <&clock CLK_GSCL3>;
                        clock-names = "gscl";
                        compatible = "samsung,exynos4212-hdmi";
                        reg = <0x14530000 0x70000>;
                        power-domains = <&pd_disp1>;
-                       interrupts = <0 95 0>;
+                       interrupts = <0 95 IRQ_TYPE_LEVEL_HIGH>;
                        clocks = <&clock CLK_HDMI>, <&clock CLK_SCLK_HDMI>,
                                 <&clock CLK_SCLK_PIXEL>, <&clock CLK_SCLK_HDMIPHY>,
                                 <&clock CLK_MOUT_HDMI>;
                        compatible = "samsung,exynos5250-mixer";
                        reg = <0x14450000 0x10000>;
                        power-domains = <&pd_disp1>;
-                       interrupts = <0 94 0>;
+                       interrupts = <0 94 IRQ_TYPE_LEVEL_HIGH>;
                        clocks = <&clock CLK_MIXER>, <&clock CLK_HDMI>,
                                 <&clock CLK_SCLK_HDMI>;
                        clock-names = "mixer", "hdmi", "sclk_hdmi";
                adc: adc@12D10000 {
                        compatible = "samsung,exynos-adc-v1";
                        reg = <0x12D10000 0x100>;
-                       interrupts = <0 106 0>;
+                       interrupts = <0 106 IRQ_TYPE_LEVEL_HIGH>;
                        clocks = <&clock CLK_ADC>;
                        clock-names = "adc";
                        #io-channel-cells = <1>;
                sss@10830000 {
                        compatible = "samsung,exynos4210-secss";
                        reg = <0x10830000 0x300>;
-                       interrupts = <0 112 0>;
+                       interrupts = <0 112 IRQ_TYPE_LEVEL_HIGH>;
                        clocks = <&clock CLK_SSS>;
                        clock-names = "secss";
                };