diff for duplicates of <50226779.6060201@codethink.co.uk> diff --git a/a/1.txt b/N1/1.txt index e484e6d..696e5a9 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -3,12 +3,12 @@ On 08/08/12 13:39, Arnd Bergmann wrote: >> The SMI / PHY stuff should look very similar, so I'm happy with something >> like: >> ->> mdio at 2000 { +>> mdio@2000 { >> #address-cells = <1>; >> #size-cells = <1>; >> device_type = "mdio"; >> compatible = "marvell,mv643xx-mdio"; ->> phy0: ethernet-phy at 0 { +>> phy0: ethernet-phy@0 { >> device_type = "ethernet-phy"; >> compatible = "marvell,whatever"; >> interrupts = <76>; @@ -16,7 +16,7 @@ On 08/08/12 13:39, Arnd Bergmann wrote: >> reg = <0 32>; // Auto probed phy addr >> }; >> ->> phy1: ethernet-phy at 3 { +>> phy1: ethernet-phy@3 { >> device_type = "ethernet-phy"; >> compatible = "marvell,whatever"; >> interrupts = <77>; @@ -44,7 +44,7 @@ and its actually quite a slow process. I can certainly investigate that. I couldn't see any good evidence that it was a supported mechanism when I looked. -> I also don't understand how the phy driver would locate ethernet-phy at 0 +> I also don't understand how the phy driver would locate ethernet-phy@0 > on the bus if it does not know the address. > >> The ethernet driver itself is more complicated: @@ -73,7 +73,7 @@ it was a supported mechanism when I looked. >> multiply instantiating a driver three times over the same address >> space. >> ->> ethernet at 2400 { +>> ethernet@2400 { >> compatible = "marvell,mv643xx-eth"; >> reg = <0x2400 0x1c00> >> interrupt_parent = <&mpic>; @@ -82,7 +82,7 @@ it was a supported mechanism when I looked. >> phys = <&phy0>, <&phy1>, <&phy2>; >> }; >> ->> ethernet at 6400 { +>> ethernet@6400 { >> compatible = "marvell,mv643xx-eth"; >> reg = <0x6400 0x1c00> >> interrupt_parent = <&mpic>; @@ -109,40 +109,40 @@ numerous as the new ones. >> port. This can never be done cleanly since the port address ranges >> overlap: >> ->> pseudo_eth at 2400 { +>> pseudo_eth@2400 { >> #address-cells = <1>; >> #size-cells = <0>; >> compatible = "marvell,mv643xx-shared-eth" >> reg = <0x2400 0x1c00>; >> ->> ethernet at 0 { +>> ethernet@0 { >> compatible = "marvell,mv643xx-port"; >> interrupts = <4>; >> interrupt_parent = <&mpic>; >> phy = <&phy0>; >> }; >> ->> ethernet at 1 { +>> ethernet@1 { >> compatible = "marvell,mv643xx-port"; >> interrupts = <5>; >> interrupt_parent = <&mpic>; >> phy = <&phy1>; >> }; >> ->> ethernet at 2 { +>> ethernet@2 { >> compatible = "marvell,mv643xx-port"; >> interrupts = <6>; >> interrupt_parent = <&mpic>; >> phy = <&phy2>; >> }; >> } ->> pseudo_eth at 6400 { +>> pseudo_eth@6400 { >> #address-cells = <1>; >> #size-cells = <0>; >> compatible = "marvell,mv643xx-shared-eth" >> reg = <0x6400 0x1c00>; >> ->> ethernet at 0 { +>> ethernet@0 { >> compatible = "marvell,mv643xx-port"; >> interrupts = <4>; >> interrupt_parent = <&mpic>; diff --git a/a/content_digest b/N1/content_digest index 4a045e5..3a19593 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -2,10 +2,16 @@ "ref\050223428.6030506@codethink.co.uk\0" "ref\0502252A6.4090409@codethink.co.uk\0" "ref\0201208081239.16778.arnd@arndb.de\0" - "From\0ian.molton@codethink.co.uk (Ian Molton)\0" - "Subject\0[PATCH v3 0/7] mv643xx.c: Add basic device tree support.\0" + "From\0Ian Molton <ian.molton@codethink.co.uk>\0" + "Subject\0Re: [PATCH v3 0/7] mv643xx.c: Add basic device tree support.\0" "Date\0Wed, 08 Aug 2012 14:19:53 +0100\0" - "To\0linux-arm-kernel@lists.infradead.org\0" + "To\0Arnd Bergmann <arnd@arndb.de>\0" + "Cc\0David Miller <davem@davemloft.net>" + linux-arm-kernel@lists.infradead.org + andrew@lunn.ch + thomas.petazzoni@free-electrons.com + ben.dooks@codethink.co.uk + " netdev@vger.kernel.org\0" "\00:1\0" "b\0" "On 08/08/12 13:39, Arnd Bergmann wrote:\n" @@ -13,12 +19,12 @@ ">> The SMI / PHY stuff should look very similar, so I'm happy with something\n" ">> like:\n" ">>\n" - ">> mdio at 2000 {\n" + ">> mdio@2000 {\n" ">> #address-cells = <1>;\n" ">> #size-cells = <1>;\n" ">> device_type = \"mdio\";\n" ">> compatible = \"marvell,mv643xx-mdio\";\n" - ">> phy0: ethernet-phy at 0 {\n" + ">> phy0: ethernet-phy@0 {\n" ">> device_type = \"ethernet-phy\";\n" ">> compatible = \"marvell,whatever\";\n" ">> interrupts = <76>;\n" @@ -26,7 +32,7 @@ ">> reg = <0 32>; // Auto probed phy addr\n" ">> };\n" ">>\n" - ">> phy1: ethernet-phy at 3 {\n" + ">> phy1: ethernet-phy@3 {\n" ">> device_type = \"ethernet-phy\";\n" ">> compatible = \"marvell,whatever\";\n" ">> interrupts = <77>;\n" @@ -54,7 +60,7 @@ "I can certainly investigate that. I couldn't see any good evidence that\n" "it was a supported mechanism when I looked.\n" "\n" - "> I also don't understand how the phy driver would locate ethernet-phy at 0\n" + "> I also don't understand how the phy driver would locate ethernet-phy@0\n" "> on the bus if it does not know the address.\n" ">\n" ">> The ethernet driver itself is more complicated:\n" @@ -83,7 +89,7 @@ ">> multiply instantiating a driver three times over the same address\n" ">> space.\n" ">>\n" - ">> ethernet at 2400 {\n" + ">> ethernet@2400 {\n" ">> compatible = \"marvell,mv643xx-eth\";\n" ">> reg = <0x2400 0x1c00>\n" ">> interrupt_parent = <&mpic>;\n" @@ -92,7 +98,7 @@ ">> phys = <&phy0>, <&phy1>, <&phy2>;\n" ">> };\n" ">>\n" - ">> ethernet at 6400 {\n" + ">> ethernet@6400 {\n" ">> compatible = \"marvell,mv643xx-eth\";\n" ">> reg = <0x6400 0x1c00>\n" ">> interrupt_parent = <&mpic>;\n" @@ -119,40 +125,40 @@ ">> port. This can never be done cleanly since the port address ranges\n" ">> overlap:\n" ">>\n" - ">> pseudo_eth at 2400 {\n" + ">> pseudo_eth@2400 {\n" ">> #address-cells = <1>;\n" ">> #size-cells = <0>;\n" ">> compatible = \"marvell,mv643xx-shared-eth\"\n" ">> reg = <0x2400 0x1c00>;\n" ">>\n" - ">> ethernet at 0 {\n" + ">> ethernet@0 {\n" ">> compatible = \"marvell,mv643xx-port\";\n" ">> interrupts = <4>;\n" ">> interrupt_parent = <&mpic>;\n" ">> phy = <&phy0>;\n" ">> };\n" ">>\n" - ">> ethernet at 1 {\n" + ">> ethernet@1 {\n" ">> compatible = \"marvell,mv643xx-port\";\n" ">> interrupts = <5>;\n" ">> interrupt_parent = <&mpic>;\n" ">> phy = <&phy1>;\n" ">> };\n" ">>\n" - ">> ethernet at 2 {\n" + ">> ethernet@2 {\n" ">> compatible = \"marvell,mv643xx-port\";\n" ">> interrupts = <6>;\n" ">> interrupt_parent = <&mpic>;\n" ">> phy = <&phy2>;\n" ">> };\n" ">> }\n" - ">> pseudo_eth at 6400 {\n" + ">> pseudo_eth@6400 {\n" ">> #address-cells = <1>;\n" ">> #size-cells = <0>;\n" ">> compatible = \"marvell,mv643xx-shared-eth\"\n" ">> reg = <0x6400 0x1c00>;\n" ">>\n" - ">> ethernet at 0 {\n" + ">> ethernet@0 {\n" ">> compatible = \"marvell,mv643xx-port\";\n" ">> interrupts = <4>;\n" ">> interrupt_parent = <&mpic>;\n" @@ -217,4 +223,4 @@ "\n" -Ian -d8cfe6b5b26589c239b1b89bdb532a49d25da8ff6fa986f15d7bdcb4c546ce05 +9e129a6f2c00147640f6f046a5ab75def0e68e582e6b0a1ecc0b845750db9727
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.