diff for duplicates of <20140926061354.GD31948@dragon> diff --git a/a/1.txt b/N1/1.txt index 5f9714a..b6e8ea2 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -68,7 +68,7 @@ Please have a new line between property list and device node ... > + led0 { Usually, the instance number shouldn't be directly encoded in node name, -but be part of node name in form of unit-address, i.e. led at 0. That also +but be part of node name in form of unit-address, i.e. led@0. That also means we will need a 'reg' property for the node and the following for the parent node. @@ -126,7 +126,7 @@ Please also put a new line between device nodes. > + bus-num = <0>; > + status = "okay"; > + -> + dspiflash: at45db021d at 0 { +> + dspiflash: at45db021d@0 { > + #address-cells = <1>; > + #size-cells = <1>; > + compatible = "atmel,at45db021d", "atmel,at45", "atmel,dataflash"; @@ -165,7 +165,7 @@ I cannot find this property in any binding doc. Add a new line. -> + pca9547 at 77 { +> + pca9547@77 { > + compatible = "philips,pca9547"; Undocumented compatible. @@ -174,42 +174,42 @@ Undocumented compatible. > + #address-cells = <1>; > + #size-cells = <0>; > + -> + i2c at 0 { +> + i2c@0 { > + #address-cells = <1>; > + #size-cells = <0>; > + reg = <0x0>; > + -> + rtc at 68 { +> + rtc@68 { > + compatible = "dallas,ds3232"; > + reg = <0x68>; > + interrupts = <GIC_SPI 169 IRQ_TYPE_LEVEL_HIGH>; > + }; > + }; > + -> + i2c at 2 { +> + i2c@2 { > + #address-cells = <1>; > + #size-cells = <0>; > + reg = <0x2>; > + -> + ina220 at 40 { +> + ina220@40 { > + compatible = "ti,ina220"; > + reg = <0x40>; > + shunt-resistor = <1000>; > + }; > + -> + ina220 at 41 { +> + ina220@41 { > + compatible = "ti,ina220"; > + reg = <0x41>; > + shunt-resistor = <1000>; > + }; > + }; > + -> + i2c at 3 { +> + i2c@3 { > + #address-cells = <1>; > + #size-cells = <0>; > + reg = <0x3>; > + -> + eeprom at 56 { +> + eeprom@56 { > + compatible = "at24,24c512"; Undocumented property. @@ -217,12 +217,12 @@ Undocumented property. > + reg = <0x56>; > + }; > + -> + eeprom at 57 { +> + eeprom@57 { > + compatible = "at24,24c512"; > + reg = <0x57>; > + }; > + -> + adt7461a at 4c { +> + adt7461a@4c { > + compatible = "adt7461a"; Shouldn't it be "adi,adt7461a"? And if that's case, per @@ -233,7 +233,7 @@ required property. > + }; > + }; > + -> + i2c at 4 { +> + i2c@4 { > + #address-cells = <1>; > + #size-cells = <0>; > + reg = <0x4>; @@ -256,7 +256,7 @@ I generally prefer to put 'status' at the bottom of the property list. > + 0x2 0x0 0x0 0x7e800000 0x00010000 > + 0x3 0x0 0x0 0x7fb00000 0x00000100>; > + -> + nor at 0,0 { +> + nor@0,0 { Drop one level of indentation. @@ -268,7 +268,7 @@ Drop one level of indentation. > + device-width = <1>; > + }; > + -> + nand at 2,0 { +> + nand@2,0 { > + #address-cells = <1>; > + #size-cells = <1>; > + compatible = "fsl,ifc-nand"; @@ -278,7 +278,7 @@ Drop one level of indentation. > + reg = <0x2 0x0 0x10000>; > + }; > + -> + fpga: board-control at 3,0 { +> + fpga: board-control@3,0 { > + #address-cells = <1>; > + #size-cells = <1>; > + compatible = "simple-bus"; @@ -296,11 +296,11 @@ Drop one level of indentation. > + mux-mask = <0xe0>; /* EMI1[2:0] */ > + > + /* Onboard PHYs */ -> + ls1021amdio0: mdio at 0 { +> + ls1021amdio0: mdio@0 { > + reg = <0>; > + #address-cells = <1>; > + #size-cells = <0>; -> + rgmii_phy1: ethernet-phy at 1 { +> + rgmii_phy1: ethernet-phy@1 { > + reg = <0x1>; > + }; > + }; @@ -309,35 +309,35 @@ Have a new line. Shawn -> + ls1021amdio1: mdio at 20 { +> + ls1021amdio1: mdio@20 { > + reg = <0x20>; > + #address-cells = <1>; > + #size-cells = <0>; -> + rgmii_phy2: ethernet-phy at 2 { +> + rgmii_phy2: ethernet-phy@2 { > + reg = <0x2>; > + }; > + }; -> + ls1021amdio2: mdio at 40 { +> + ls1021amdio2: mdio@40 { > + reg = <0x40>; > + #address-cells = <1>; > + #size-cells = <0>; -> + rgmii_phy3: ethernet-phy at 3 { +> + rgmii_phy3: ethernet-phy@3 { > + reg = <0x3>; > + }; > + }; -> + ls1021amdio3: mdio at 60 { +> + ls1021amdio3: mdio@60 { > + reg = <0x60>; > + #address-cells = <1>; > + #size-cells = <0>; -> + sgmii_phy1c: ethernet-phy at 1c { +> + sgmii_phy1c: ethernet-phy@1c { > + reg = <0x1c>; > + }; > + }; -> + ls1021amdio4: mdio at 80 { +> + ls1021amdio4: mdio@80 { > + reg = <0x80>; > + #address-cells = <1>; > + #size-cells = <0>; -> + sgmii_phy1d: ethernet-phy at 1d { +> + sgmii_phy1d: ethernet-phy@1d { > + reg = <0x1d>; > + }; > + }; @@ -350,7 +350,7 @@ Shawn > +}; > + > +&mdio0 { -> + tbi0: tbi-phy at 8 { +> + tbi0: tbi-phy@8 { > + reg = <0x8>; > + device_type = "tbi-phy"; > + }; diff --git a/a/content_digest b/N1/content_digest index 69db660..634f79c 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,9 +1,20 @@ "ref\01411371952-5618-1-git-send-email-jingchang.lu@freescale.com\0" "ref\01411371952-5618-3-git-send-email-jingchang.lu@freescale.com\0" - "From\0shawn.guo@freescale.com (Shawn Guo)\0" - "Subject\0[PATCHv4 2/6] ARM: dts: Add initial LS1021A QDS board dts support\0" + "From\0Shawn Guo <shawn.guo@freescale.com>\0" + "Subject\0Re: [PATCHv4 2/6] ARM: dts: Add initial LS1021A QDS board dts support\0" "Date\0Fri, 26 Sep 2014 14:13:55 +0800\0" - "To\0linux-arm-kernel@lists.infradead.org\0" + "To\0Jingchang Lu <jingchang.lu@freescale.com>\0" + "Cc\0mark.rutland@arm.com" + devicetree@vger.kernel.org + Chao Fu <B44548@freescale.com> + arnd@arndb.de + Alison Wang <alison.wang@freescale.com> + Bhupesh Sharma <bhupesh.sharma@freescale.com> + Xiubo Li <Li.Xiubo@freescale.com> + Jason Jin <Jason.Jin@freescale.com> + Jingchang Lu <b35083@freescale.com> + linux-arm-kernel@lists.infradead.org + " Jaiprakash Singh <b44839@freescale.com>\0" "\00:1\0" "b\0" "On Mon, Sep 22, 2014 at 03:45:48PM +0800, Jingchang Lu wrote:\n" @@ -76,7 +87,7 @@ "> +\t\t\tled0 {\n" "\n" "Usually, the instance number shouldn't be directly encoded in node name,\n" - "but be part of node name in form of unit-address, i.e. led at 0. That also\n" + "but be part of node name in form of unit-address, i.e. led@0. That also\n" "means we will need a 'reg' property for the node and the following for\n" "the parent node.\n" "\n" @@ -134,7 +145,7 @@ "> +\tbus-num = <0>;\n" "> +\tstatus = \"okay\";\n" "> +\n" - "> +\tdspiflash: at45db021d at 0 {\n" + "> +\tdspiflash: at45db021d@0 {\n" "> +\t\t#address-cells = <1>;\n" "> +\t\t#size-cells = <1>;\n" "> +\t\tcompatible = \"atmel,at45db021d\", \"atmel,at45\", \"atmel,dataflash\";\n" @@ -173,7 +184,7 @@ "\n" "Add a new line.\n" "\n" - "> +\tpca9547 at 77 {\n" + "> +\tpca9547@77 {\n" "> +\t\tcompatible = \"philips,pca9547\";\n" "\n" "Undocumented compatible.\n" @@ -182,42 +193,42 @@ "> +\t\t#address-cells = <1>;\n" "> +\t\t#size-cells = <0>;\n" "> +\n" - "> +\t\ti2c at 0 {\n" + "> +\t\ti2c@0 {\n" "> +\t\t\t#address-cells = <1>;\n" "> +\t\t\t#size-cells = <0>;\n" "> +\t\t\treg = <0x0>;\n" "> +\n" - "> +\t\t\trtc at 68 {\n" + "> +\t\t\trtc@68 {\n" "> +\t\t\t\tcompatible = \"dallas,ds3232\";\n" "> +\t\t\t\treg = <0x68>;\n" "> +\t\t\t\tinterrupts = <GIC_SPI 169 IRQ_TYPE_LEVEL_HIGH>;\n" "> +\t\t\t};\n" "> +\t\t};\n" "> +\n" - "> +\t\ti2c at 2 {\n" + "> +\t\ti2c@2 {\n" "> +\t\t\t#address-cells = <1>;\n" "> +\t\t\t#size-cells = <0>;\n" "> +\t\t\treg = <0x2>;\n" "> +\n" - "> +\t\t\tina220 at 40 {\n" + "> +\t\t\tina220@40 {\n" "> +\t\t\t\tcompatible = \"ti,ina220\";\n" "> +\t\t\t\treg = <0x40>;\n" "> +\t\t\t\tshunt-resistor = <1000>;\n" "> +\t\t\t};\n" "> +\n" - "> +\t\t\tina220 at 41 {\n" + "> +\t\t\tina220@41 {\n" "> +\t\t\t\tcompatible = \"ti,ina220\";\n" "> +\t\t\t\treg = <0x41>;\n" "> +\t\t\t\tshunt-resistor = <1000>;\n" "> +\t\t\t};\n" "> +\t\t};\n" "> +\n" - "> +\t\ti2c at 3 {\n" + "> +\t\ti2c@3 {\n" "> +\t\t\t#address-cells = <1>;\n" "> +\t\t\t#size-cells = <0>;\n" "> +\t\t\treg = <0x3>;\n" "> +\n" - "> +\t\t\teeprom at 56 {\n" + "> +\t\t\teeprom@56 {\n" "> +\t\t\t\tcompatible = \"at24,24c512\";\n" "\n" "Undocumented property.\n" @@ -225,12 +236,12 @@ "> +\t\t\t\treg = <0x56>;\n" "> +\t\t\t};\n" "> +\n" - "> +\t\t\teeprom at 57 {\n" + "> +\t\t\teeprom@57 {\n" "> +\t\t\t\tcompatible = \"at24,24c512\";\n" "> +\t\t\t\treg = <0x57>;\n" "> +\t\t\t};\n" "> +\n" - "> +\t\t\tadt7461a at 4c {\n" + "> +\t\t\tadt7461a@4c {\n" "> +\t\t\t\tcompatible = \"adt7461a\";\n" "\n" "Shouldn't it be \"adi,adt7461a\"? And if that's case, per\n" @@ -241,7 +252,7 @@ "> +\t\t\t};\n" "> +\t\t};\n" "> +\n" - "> +\t\ti2c at 4 {\n" + "> +\t\ti2c@4 {\n" "> +\t\t\t#address-cells = <1>;\n" "> +\t\t\t#size-cells = <0>;\n" "> +\t\t\treg = <0x4>;\n" @@ -264,7 +275,7 @@ "> +\t\t0x2 0x0 0x0 0x7e800000 0x00010000\n" "> +\t\t0x3 0x0 0x0 0x7fb00000 0x00000100>;\n" "> +\n" - "> +\t\tnor at 0,0 {\n" + "> +\t\tnor@0,0 {\n" "\n" "Drop one level of indentation.\n" "\n" @@ -276,7 +287,7 @@ "> +\t\t\tdevice-width = <1>;\n" "> +\t\t};\n" "> +\n" - "> +\t\tnand at 2,0 {\n" + "> +\t\tnand@2,0 {\n" "> +\t\t\t#address-cells = <1>;\n" "> +\t\t\t#size-cells = <1>;\n" "> +\t\t\tcompatible = \"fsl,ifc-nand\";\n" @@ -286,7 +297,7 @@ "> +\t\t\treg = <0x2 0x0 0x10000>;\n" "> +\t\t};\n" "> +\n" - "> +\t\tfpga: board-control at 3,0 {\n" + "> +\t\tfpga: board-control@3,0 {\n" "> +\t\t\t#address-cells = <1>;\n" "> +\t\t\t#size-cells = <1>;\n" "> +\t\t\tcompatible = \"simple-bus\";\n" @@ -304,11 +315,11 @@ "> +\t\t\t\tmux-mask = <0xe0>; /* EMI1[2:0] */\n" "> +\n" "> +\t\t\t\t/* Onboard PHYs */\n" - "> +\t\t\t\tls1021amdio0: mdio at 0 {\n" + "> +\t\t\t\tls1021amdio0: mdio@0 {\n" "> +\t\t\t\t\treg = <0>;\n" "> +\t\t\t\t\t#address-cells = <1>;\n" "> +\t\t\t\t\t#size-cells = <0>;\n" - "> +\t\t\t\t\trgmii_phy1: ethernet-phy at 1 {\n" + "> +\t\t\t\t\trgmii_phy1: ethernet-phy@1 {\n" "> +\t\t\t\t\t\treg = <0x1>;\n" "> +\t\t\t\t\t};\n" "> +\t\t\t\t};\n" @@ -317,35 +328,35 @@ "\n" "Shawn\n" "\n" - "> +\t\t\t\tls1021amdio1: mdio at 20 {\n" + "> +\t\t\t\tls1021amdio1: mdio@20 {\n" "> +\t\t\t\t\treg = <0x20>;\n" "> +\t\t\t\t\t#address-cells = <1>;\n" "> +\t\t\t\t\t#size-cells = <0>;\n" - "> +\t\t\t\t\trgmii_phy2: ethernet-phy at 2 {\n" + "> +\t\t\t\t\trgmii_phy2: ethernet-phy@2 {\n" "> +\t\t\t\t\t\treg = <0x2>;\n" "> +\t\t\t\t\t};\n" "> +\t\t\t\t};\n" - "> +\t\t\t\tls1021amdio2: mdio at 40 {\n" + "> +\t\t\t\tls1021amdio2: mdio@40 {\n" "> +\t\t\t\t\treg = <0x40>;\n" "> +\t\t\t\t\t#address-cells = <1>;\n" "> +\t\t\t\t\t#size-cells = <0>;\n" - "> +\t\t\t\t\trgmii_phy3: ethernet-phy at 3 {\n" + "> +\t\t\t\t\trgmii_phy3: ethernet-phy@3 {\n" "> +\t\t\t\t\t\treg = <0x3>;\n" "> +\t\t\t\t\t};\n" "> +\t\t\t\t};\n" - "> +\t\t\t\tls1021amdio3: mdio at 60 {\n" + "> +\t\t\t\tls1021amdio3: mdio@60 {\n" "> +\t\t\t\t\treg = <0x60>;\n" "> +\t\t\t\t\t#address-cells = <1>;\n" "> +\t\t\t\t\t#size-cells = <0>;\n" - "> +\t\t\t\t\tsgmii_phy1c: ethernet-phy at 1c {\n" + "> +\t\t\t\t\tsgmii_phy1c: ethernet-phy@1c {\n" "> +\t\t\t\t\t\treg = <0x1c>;\n" "> +\t\t\t\t\t};\n" "> +\t\t\t\t};\n" - "> +\t\t\t\tls1021amdio4: mdio at 80 {\n" + "> +\t\t\t\tls1021amdio4: mdio@80 {\n" "> +\t\t\t\t\treg = <0x80>;\n" "> +\t\t\t\t\t#address-cells = <1>;\n" "> +\t\t\t\t\t#size-cells = <0>;\n" - "> +\t\t\t\t\tsgmii_phy1d: ethernet-phy at 1d {\n" + "> +\t\t\t\t\tsgmii_phy1d: ethernet-phy@1d {\n" "> +\t\t\t\t\t\treg = <0x1d>;\n" "> +\t\t\t\t\t};\n" "> +\t\t\t\t};\n" @@ -358,7 +369,7 @@ "> +};\n" "> +\n" "> +&mdio0 {\n" - "> +\ttbi0: tbi-phy at 8 {\n" + "> +\ttbi0: tbi-phy@8 {\n" "> +\t\treg = <0x8>;\n" "> +\t\tdevice_type = \"tbi-phy\";\n" "> +\t};\n" @@ -383,4 +394,4 @@ "> 1.8.0\n" > -43f999b0711b4c3b5d4aa8e9088099df96ac6af8e413a3bddc751fc2592be3e0 +d0639b5e219e9105b67b832ebcdc2c0bfec18482c72e51bec3c6f2ac3d94364f
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.