[RFC][PATCH] MPC832x_RDB: update dts to use spi, register mmc_spi stub
Anton Vorontsov
avorontsov at ru.mvista.com
Wed Aug 1 22:34:00 EST 2007
On Wed, Aug 01, 2007 at 12:10:28AM +0200, Segher Boessenkool wrote:
>> + spi1pio:spi_pin at 01 {
>
> There should be whitespace after the label. @01 should be
> spelled @1. Except there is no "reg" property.
Hm. I've just tried to keep original style in this particular dts.
Want to ack patch below?
What is prefered style of <&label> vs. < &label > usage, btw?
arch/powerpc/boot/dts$ grep "<&" -r . | wc -l
327
arch/powerpc/boot/dts$ grep "< &" -r . | wc -l
92
I can only guess - the first?
> What is this
> stuff, anyway?
Which one? pio-map for spi? This is GPIO pins configuration, to use
dedicated functions (SPI) for these pins, otherwise SPI will not work.
>> + pio-map = <
>> + /* port pin dir open_drain assignment has_irq */
>> + 3 0 3 0 1 0 /* SPI1 MOSI, I/O */
>> + 3 1 3 0 1 0 /* SPI1 MISO, I/O */
>> + 3 2 3 0 1 0 /* SPI1 CLK, I/O */
>> + 3 3 2 0 1 0>; /* SPI1 SEL, I */
>> + };
p.s. mpc8272ads.dts is broken wrt spaces/tabs, very.
- - - -
From: Anton Vorontsov <avorontsov at ru.mvista.com>
Subject: [POWERPC] boot/dts/*: cleanup labels usage
There should be whitespace after labels.
Signed-off-by: Anton Vorontsov <avorontsov at ru.mvista.com>
---
arch/powerpc/boot/dts/mpc8272ads.dts | 6 +++---
arch/powerpc/boot/dts/mpc832x_rdb.dts | 12 ++++++------
2 files changed, 9 insertions(+), 9 deletions(-)
diff --git a/arch/powerpc/boot/dts/mpc8272ads.dts b/arch/powerpc/boot/dts/mpc8272ads.dts
index 1934b80..97c5d4e 100644
--- a/arch/powerpc/boot/dts/mpc8272ads.dts
+++ b/arch/powerpc/boot/dts/mpc8272ads.dts
@@ -66,14 +66,14 @@
reg = <0 0>;
#address-cells = <1>;
#size-cells = <0>;
- phy0:ethernet-phy at 0 {
+ phy0: ethernet-phy at 0 {
interrupt-parent = <&Cpm_pic>;
interrupts = <17 4>;
reg = <0>;
bitbang = [ 12 12 13 02 02 01 ];
device_type = "ethernet-phy";
};
- phy1:ethernet-phy at 1 {
+ phy1: ethernet-phy at 1 {
interrupt-parent = <&Cpm_pic>;
interrupts = <17 4>;
bitbang = [ 12 12 13 02 02 01 ];
@@ -153,7 +153,7 @@
};
};
- cpm_pic:interrupt-controller at 10c00 {
+ cpm_pic: interrupt-controller at 10c00 {
#address-cells = <0>;
#interrupt-cells = <2>;
interrupt-controller;
diff --git a/arch/powerpc/boot/dts/mpc832x_rdb.dts b/arch/powerpc/boot/dts/mpc832x_rdb.dts
index 7c4beff..e9c332f 100644
--- a/arch/powerpc/boot/dts/mpc832x_rdb.dts
+++ b/arch/powerpc/boot/dts/mpc832x_rdb.dts
@@ -127,7 +127,7 @@
device_type = "pci";
};
- pic:pic at 700 {
+ pic: pic at 700 {
interrupt-controller;
#address-cells = <0>;
#interrupt-cells = <2>;
@@ -141,7 +141,7 @@
device_type = "par_io";
num-ports = <7>;
- ucc2pio:ucc_pin at 02 {
+ ucc2pio: ucc_pin at 02 {
pio-map = <
/* port pin dir open_drain assignment has_irq */
3 4 3 0 2 0 /* MDIO */
@@ -163,7 +163,7 @@
0 1e 1 0 1 0 /* TX_EN */
0 1f 2 0 1 0>; /* CRS */
};
- ucc3pio:ucc_pin at 03 {
+ ucc3pio: ucc_pin at 03 {
pio-map = <
/* port pin dir open_drain assignment has_irq */
0 d 2 0 1 0 /* RX_CLK (CLK9) */
@@ -272,13 +272,13 @@
device_type = "mdio";
compatible = "ucc_geth_phy";
- phy00:ethernet-phy at 00 {
+ phy00: ethernet-phy at 00 {
interrupt-parent = <&pic>;
interrupts = <0>;
reg = <0>;
device_type = "ethernet-phy";
};
- phy04:ethernet-phy at 04 {
+ phy04: ethernet-phy at 04 {
interrupt-parent = <&pic>;
interrupts = <0>;
reg = <4>;
@@ -286,7 +286,7 @@
};
};
- qeic:qeic at 80 {
+ qeic: qeic at 80 {
interrupt-controller;
device_type = "qeic";
#address-cells = <0>;
--
1.5.0.6
More information about the Linuxppc-dev
mailing list