diff mbox

[3/3] arm: dts: dra7: fixup qspi entry

Message ID 1404844852-19980-3-git-send-email-bigeasy@linutronix.de (mailing list archive)
State New, archived
Headers show

Commit Message

Sebastian Andrzej Siewior July 8, 2014, 6:40 p.m. UTC
The core complains that the number 343 is too large. The older code has
here 124. This avoids the warning, the driver hasn't been tested.

Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
---
 arch/arm/boot/dts/dra7.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Poddar, Sourav July 9, 2014, 10:07 a.m. UTC | #1
Hi,

On Wednesday 09 July 2014 12:10 AM, Sebastian Andrzej Siewior wrote:
> The core complains that the number 343 is too large. The older code has
> here 124. This avoids the warning, the driver hasn't been tested.
>
> Signed-off-by: Sebastian Andrzej Siewior<bigeasy@linutronix.de>
> ---
>   arch/arm/boot/dts/dra7.dtsi | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/dra7.dtsi b/arch/arm/boot/dts/dra7.dtsi
> index daf7d36..1529bca 100644
> --- a/arch/arm/boot/dts/dra7.dtsi
> +++ b/arch/arm/boot/dts/dra7.dtsi
> @@ -821,7 +821,7 @@
>   			clocks =<&qspi_gfclk_div>;
>   			clock-names = "fck";
>   			num-cs =<4>;
> -			interrupts =<0 343 0x4>;
> +			interrupts =<0 124 0x4>;
>   			status = "disabled";
>   		};
>

The number is correct and is complaining just because the crossbar stuff 
is not
put in. I had already posted a patch[1] to remove interrupt binding as 
of now.

Hence, NAK for this patch.

[1]: https://patchwork.kernel.org/patch/4364341/
Sebastian Andrzej Siewior July 9, 2014, 10:18 a.m. UTC | #2
On 07/09/2014 12:07 PM, sourav wrote:
> Hi,

Hello, Sourav,

> The number is correct and is complaining just because the crossbar stuff
> is not
> put in. I had already posted a patch[1] to remove interrupt binding as
> of now.
> 
> Hence, NAK for this patch.

Thank you for explanation. What is the status on the crossbar stuff? Is
this comming any time soon? It seems that it is also stalling the
networking pieces.

> 
> [1]: https://patchwork.kernel.org/patch/4364341/

Sebastian
Tony Lindgren July 9, 2014, 10:26 a.m. UTC | #3
* Sebastian Andrzej Siewior <bigeasy@linutronix.de> [140709 03:20]:
> On 07/09/2014 12:07 PM, sourav wrote:
> > Hi,
> 
> Hello, Sourav,
> 
> > The number is correct and is complaining just because the crossbar stuff
> > is not
> > put in. I had already posted a patch[1] to remove interrupt binding as
> > of now.
> > 
> > Hence, NAK for this patch.
> 
> Thank you for explanation. What is the status on the crossbar stuff? Is
> this comming any time soon? It seems that it is also stalling the
> networking pieces.

The drivers/irqchip related fixes are queued into linux next,
and I'm planning to queue the dts related changes for v3.17.

I might have the crossbar stuff in omap-for-v3.17/dt at some
point today.

Regards,

Tony

> > [1]: https://patchwork.kernel.org/patch/4364341/
diff mbox

Patch

diff --git a/arch/arm/boot/dts/dra7.dtsi b/arch/arm/boot/dts/dra7.dtsi
index daf7d36..1529bca 100644
--- a/arch/arm/boot/dts/dra7.dtsi
+++ b/arch/arm/boot/dts/dra7.dtsi
@@ -821,7 +821,7 @@ 
 			clocks = <&qspi_gfclk_div>;
 			clock-names = "fck";
 			num-cs = <4>;
-			interrupts = <0 343 0x4>;
+			interrupts = <0 124 0x4>;
 			status = "disabled";
 		};