Commit 20f12758 authored by Linus Walleij's avatar Linus Walleij Committed by Olof Johansson

ARM: versatile: fix MMC/SD interrupt assignment

Commit 0976c946
"arm/versatile: Fix versatile irq specifications"
has an off-by-one error on the Versatile AB that has
been regressing the Versatile AB hardware for some time.

However it seems like the interrupt assignments have
never been correct and I have now adjusted them according
to the specification. The masks for the valid interrupts
made it impossible to assign the right SIC interrupt
for the MMCI, so I went in and fixed these to correspond
to the specifications, and added references if anyone
wants to double-check.

Due to the Versatile PB including the Versatile AB
as a base DTS file, we need to override and correct
some values to correspond to the actual changes in the
hardware.

For the Versatile PB I don't think the IRQ line
assignment for MMCI has ever been correct for either of
the two MMCI blocks. It would be nice if someone with the
physical PB board could test this.

Patch tested on the Versatile AB, QEMU for Versatile AB
and QEMU for Versatile PB.

Cc: Rob Herring <robh@kernel.org>
Cc: Grant Likely <grant.likely@linaro.org>
Cc: stable@vger.kernel.org
Fixes: 0976c946 ("arm/versatile: Fix versatile irq specifications")
Signed-off-by: default avatarLinus Walleij <linus.walleij@linaro.org>
Signed-off-by: default avatarOlof Johansson <olof@lixom.net>
parent a461a3ec
...@@ -110,7 +110,11 @@ sic: intc@10003000 { ...@@ -110,7 +110,11 @@ sic: intc@10003000 {
interrupt-parent = <&vic>; interrupt-parent = <&vic>;
interrupts = <31>; /* Cascaded to vic */ interrupts = <31>; /* Cascaded to vic */
clear-mask = <0xffffffff>; clear-mask = <0xffffffff>;
valid-mask = <0xffc203f8>; /*
* Valid interrupt lines mask according to
* table 4-36 page 4-50 of ARM DUI 0225D
*/
valid-mask = <0x0760031b>;
}; };
dma@10130000 { dma@10130000 {
...@@ -266,8 +270,8 @@ aaci@4000 { ...@@ -266,8 +270,8 @@ aaci@4000 {
}; };
mmc@5000 { mmc@5000 {
compatible = "arm,pl180", "arm,primecell"; compatible = "arm,pl180", "arm,primecell";
reg = < 0x5000 0x1000>; reg = <0x5000 0x1000>;
interrupts-extended = <&vic 22 &sic 2>; interrupts-extended = <&vic 22 &sic 1>;
clocks = <&xtal24mhz>, <&pclk>; clocks = <&xtal24mhz>, <&pclk>;
clock-names = "mclk", "apb_pclk"; clock-names = "mclk", "apb_pclk";
}; };
......
...@@ -5,6 +5,16 @@ / { ...@@ -5,6 +5,16 @@ / {
compatible = "arm,versatile-pb"; compatible = "arm,versatile-pb";
amba { amba {
/* The Versatile PB is using more SIC IRQ lines than the AB */
sic: intc@10003000 {
clear-mask = <0xffffffff>;
/*
* Valid interrupt lines mask according to
* figure 3-30 page 3-74 of ARM DUI 0224B
*/
valid-mask = <0x7fe003ff>;
};
gpio2: gpio@101e6000 { gpio2: gpio@101e6000 {
compatible = "arm,pl061", "arm,primecell"; compatible = "arm,pl061", "arm,primecell";
reg = <0x101e6000 0x1000>; reg = <0x101e6000 0x1000>;
...@@ -67,6 +77,13 @@ pci-controller@10001000 { ...@@ -67,6 +77,13 @@ pci-controller@10001000 {
}; };
fpga { fpga {
mmc@5000 {
/*
* Overrides the interrupt assignment from
* the Versatile AB board file.
*/
interrupts-extended = <&sic 22 &sic 23>;
};
uart@9000 { uart@9000 {
compatible = "arm,pl011", "arm,primecell"; compatible = "arm,pl011", "arm,primecell";
reg = <0x9000 0x1000>; reg = <0x9000 0x1000>;
...@@ -86,7 +103,8 @@ sci@a000 { ...@@ -86,7 +103,8 @@ sci@a000 {
mmc@b000 { mmc@b000 {
compatible = "arm,pl180", "arm,primecell"; compatible = "arm,pl180", "arm,primecell";
reg = <0xb000 0x1000>; reg = <0xb000 0x1000>;
interrupts-extended = <&vic 23 &sic 2>; interrupt-parent = <&sic>;
interrupts = <1>, <2>;
clocks = <&xtal24mhz>, <&pclk>; clocks = <&xtal24mhz>, <&pclk>;
clock-names = "mclk", "apb_pclk"; clock-names = "mclk", "apb_pclk";
}; };
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment