[PATCH] powerpc: Add dma nodes to 83xx, 85xx and 86xx boards

Kumar Gala galak at kernel.crashing.org
Sat Jun 28 07:08:13 EST 2008


On Jun 27, 2008, at 3:28 PM, Scott Wood wrote:

> Kumar Gala wrote:
>> diff --git a/arch/powerpc/boot/dts/asp834x-redboot.dts b/arch/ 
>> powerpc/boot/dts/asp834x-redboot.dts
>> index 972cf78..d140031 100644
>> --- a/arch/powerpc/boot/dts/asp834x-redboot.dts
>> +++ b/arch/powerpc/boot/dts/asp834x-redboot.dts
>> @@ -53,7 +53,7 @@
>> 			     "fsl,pq2pro-localbus",
>> 			     "simple-bus";
>> 		reg = <0xff005000 0x1000>;
>> -		interrupts = <77 0x8>;
>> +		interrupts = <47 0x8>;
>> 		interrupt-parent = <&ipic>;
>> 		ranges = <
>
> This is wrong (LBC is IRQ 77; IRQ 47 is reserved), and wasn't  
> mentioned in the patch description...

This was a search/replace mistake.

>> diff --git a/arch/powerpc/boot/dts/mpc8313erdb.dts b/arch/powerpc/ 
>> boot/dts/mpc8313erdb.dts
>> index e1f0dca..ae2c4aa 100644
>> --- a/arch/powerpc/boot/dts/mpc8313erdb.dts
>> +++ b/arch/powerpc/boot/dts/mpc8313erdb.dts
>> @@ -52,7 +52,7 @@
>> 		#size-cells = <1>;
>> 		compatible = "fsl,mpc8313-elbc", "fsl,elbc", "simple-bus";
>> 		reg = <0xe0005000 0x1000>;
>> -		interrupts = <77 0x8>;
>> +		interrupts = <13 0x8>;
>> 		interrupt-parent = <&ipic>;
>
> Likewise.  13 is the 1588 timer IRQ.
>
>> @@ -51,7 +51,7 @@
>> 		#size-cells = <1>;
>> 		compatible = "fsl,mpc8315-elbc", "fsl,elbc", "simple-bus";
>> 		reg = <0xe0005000 0x1000>;
>> -		interrupts = <77 0x8>;
>> +		interrupts = <15 0x8>;
>
> I2C this time. :-P

ditto, ditto.  thanks for catching them.

>> @@ -303,6 +268,41 @@
>> 			};
>> 		};
>> +		crypto at 30000 {
>> +			model = "SEC3";
>> +			compatible = "talitos";
>> +			reg = <0x30000 0x10000>;
>> +			interrupts = <11 0x8>;
>> +			interrupt-parent = <&ipic>;
>> +			/* Rev. 3.0 geometry */
>> +			num-channels = <4>;
>> +			channel-fifo-len = <24>;
>> +			exec-units-mask = <0x000001fe>;
>> +			descriptor-types-mask = <0x03ab0ebf>;
>> +		};
>> +
>> +		sdhc at 2e000 {
>> +			model = "eSDHC";
>> +			compatible = "fsl,esdhc";
>> +			reg = <0x2e000 0x1000>;
>> +			interrupts = <42 0x8>;
>> +			interrupt-parent = <&ipic>;
>> +		};
>> +
>> +		sata at 18000 {
>> +			compatible = "fsl,mpc8379-sata", "fsl,pq-sata";
>> +			reg = <0x18000 0x1000>;
>> +			interrupts = <44 0x8>;
>> +			interrupt-parent = <&ipic>;
>> +		};
>> +
>> +		sata at 19000 {
>> +			compatible = "fsl,mpc8379-sata", "fsl,pq-sata";
>> +			reg = <0x19000 0x1000>;
>> +			interrupts = <45 0x8>;
>> +			interrupt-parent = <&ipic>;
>> +		};
>
> Doesn't match the changelog, and the crypto binding is old.

Its because diff doesn't like the fact that I moved the dma node up.

>> diff --git a/arch/powerpc/boot/dts/mpc8378_mds.dts b/arch/powerpc/ 
>> boot/dts/mpc8378_mds.dts
>> index 1d6ea08..5d57f5d 100644
>> --- a/arch/powerpc/boot/dts/mpc8378_mds.dts
>> +++ b/arch/powerpc/boot/dts/mpc8378_mds.dts
>> @@ -52,7 +52,7 @@
>> 		#size-cells = <1>;
>> 		compatible = "fsl,mpc8378-elbc", "fsl,elbc", "simple-bus";
>> 		reg = <0xe0005000 0x1000>;
>> -		interrupts = <77 0x8>;
>> +		interrupts = <78 0x8>;
>
> Another mangled localbus interrupt (and more after this).

I think I got them all.

- k



More information about the Linuxppc-dev mailing list