diff for duplicates of <20151222113742.GA26290@tiger> diff --git a/a/1.txt b/N1/1.txt index 0a7344a..9fc4658 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -86,7 +86,7 @@ find a plenty of examples. > + enable-gpios = <&gpio4 15 GPIO_ACTIVE_HIGH>; > + }; > + -> + reg_1p8v: regulator at 1 { +> + reg_1p8v: regulator@1 { @unit-address should only present for nodes which have 'reg' property. As we choose to put these regulators directly under root node, we would @@ -104,7 +104,7 @@ probably want to name them like: > + regulator-always-on; > + }; > + -> + reg_3p3v: regulator at 2 { +> + reg_3p3v: regulator@2 { > + compatible = "regulator-fixed"; > + regulator-name = "3P3V"; > + regulator-min-microvolt = <3300000>; @@ -112,7 +112,7 @@ probably want to name them like: > + regulator-always-on; > + }; > + -> + reg_lvds: regulator at 3 { +> + reg_lvds: regulator@3 { > + compatible = "regulator-fixed"; > + regulator-name = "lvds_ppen"; > + regulator-min-microvolt = <3300000>; @@ -122,14 +122,14 @@ probably want to name them like: > + enable-active-high; > + }; > + -> + reg_usb_otg_vbus: regulator at 4 { +> + reg_usb_otg_vbus: regulator@4 { > + compatible = "regulator-fixed"; > + regulator-name = "usb_otg_vbus"; > + regulator-min-microvolt = <5000000>; > + regulator-max-microvolt = <5000000>; > + }; > + -> + reg_usb_h1_vbus: regulator at 5 { +> + reg_usb_h1_vbus: regulator@5 { > + compatible = "regulator-fixed"; > + regulator-name = "usb_h1_vbus"; > + regulator-min-microvolt = <5000000>; @@ -150,7 +150,7 @@ probably want to name them like: > + pinctrl-0 = <&pinctrl_ecspi1>; > + status = "okay"; > + -> + flash: n25q032 at 0 { +> + flash: n25q032@0 { > + compatible = "jedec,spi-nor"; > + spi-max-frequency = <20000000>; > + reg = <0>; @@ -161,21 +161,21 @@ Have a new line between properties and sub-nodes. > + #address-cells = <1>; > + #size-cells = <1>; > + -> + partition at 0 { +> + partition@0 { > + label = "U-Boot"; > + reg = <0x0 0xC0000>; Please consistently use lowercase for hex values. > + }; -> + partition at C0000 { +> + partition@C0000 { > + label = "env"; > + reg = <0xC0000 0x10000>; > + }; Have a new line between nodes. -> + partition at D0000 { +> + partition@D0000 { > + label = "spare"; > + reg = <0xD0000 0x130000>; > + }; @@ -215,7 +215,7 @@ Have a new line between nodes. > + pinctrl-0 = <&pinctrl_i2c3>; > + status = "okay"; > + -> + pmic at 58 { +> + pmic@58 { > + compatible = "dlg,da9063"; > + reg = <0x58>; > + pinctrl-names = "default"; @@ -325,7 +325,7 @@ Have a new line between nodes. > + }; > + }; > + -> + rtc at 32 { +> + rtc@32 { > + compatible = "epson,rx8010"; Is this compatible string documented somewhere? diff --git a/a/content_digest b/N1/content_digest index f185e76..edecd3f 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,9 +1,19 @@ "ref\01449177036-13317-1-git-send-email-akshay.bhat@timesys.com\0" "ref\01449177036-13317-4-git-send-email-akshay.bhat@timesys.com\0" - "From\0shawnguo@kernel.org (Shawn Guo)\0" - "Subject\0[PATCH v5 3/7] ARM: dts: imx: Add Advantech BA-16 Qseven module\0" + "From\0Shawn Guo <shawnguo@kernel.org>\0" + "Subject\0Re: [PATCH v5 3/7] ARM: dts: imx: Add Advantech BA-16 Qseven module\0" "Date\0Tue, 22 Dec 2015 19:40:14 +0800\0" - "To\0linux-arm-kernel@lists.infradead.org\0" + "To\0Akshay Bhat <akshay.bhat@timesys.com>\0" + "Cc\0mark.rutland@arm.com" + devicetree@vger.kernel.org + linux@arm.linux.org.uk + ijc+devicetree@hellion.org.uk + robh+dt@kernel.org + kernel@pengutronix.de + galak@codeaurora.org + justin.waters@timesys.com + linux-arm-kernel@lists.infradead.org + " l.stach@pengutronix.de\0" "\00:1\0" "b\0" "On Thu, Dec 03, 2015 at 04:10:32PM -0500, Akshay Bhat wrote:\n" @@ -94,7 +104,7 @@ "> +\t\tenable-gpios = <&gpio4 15 GPIO_ACTIVE_HIGH>;\n" "> +\t};\n" "> +\n" - "> +\treg_1p8v: regulator at 1 {\n" + "> +\treg_1p8v: regulator@1 {\n" "\n" "@unit-address should only present for nodes which have 'reg' property.\n" "As we choose to put these regulators directly under root node, we would\n" @@ -112,7 +122,7 @@ "> +\t\tregulator-always-on;\n" "> +\t};\n" "> +\n" - "> +\treg_3p3v: regulator at 2 {\n" + "> +\treg_3p3v: regulator@2 {\n" "> +\t\tcompatible = \"regulator-fixed\";\n" "> +\t\tregulator-name = \"3P3V\";\n" "> +\t\tregulator-min-microvolt = <3300000>;\n" @@ -120,7 +130,7 @@ "> +\t\tregulator-always-on;\n" "> +\t};\n" "> +\n" - "> +\treg_lvds: regulator at 3 {\n" + "> +\treg_lvds: regulator@3 {\n" "> +\t\tcompatible = \"regulator-fixed\";\n" "> +\t\tregulator-name = \"lvds_ppen\";\n" "> +\t\tregulator-min-microvolt = <3300000>;\n" @@ -130,14 +140,14 @@ "> +\t\tenable-active-high;\n" "> +\t};\n" "> +\n" - "> +\treg_usb_otg_vbus: regulator at 4 {\n" + "> +\treg_usb_otg_vbus: regulator@4 {\n" "> +\t\tcompatible = \"regulator-fixed\";\n" "> +\t\tregulator-name = \"usb_otg_vbus\";\n" "> +\t\tregulator-min-microvolt = <5000000>;\n" "> +\t\tregulator-max-microvolt = <5000000>;\n" "> +\t};\n" "> +\n" - "> +\treg_usb_h1_vbus: regulator at 5 {\n" + "> +\treg_usb_h1_vbus: regulator@5 {\n" "> +\t\tcompatible = \"regulator-fixed\";\n" "> +\t\tregulator-name = \"usb_h1_vbus\";\n" "> +\t\tregulator-min-microvolt = <5000000>;\n" @@ -158,7 +168,7 @@ "> +\tpinctrl-0 = <&pinctrl_ecspi1>;\n" "> +\tstatus = \"okay\";\n" "> +\n" - "> +\tflash: n25q032 at 0 {\n" + "> +\tflash: n25q032@0 {\n" "> +\t\tcompatible = \"jedec,spi-nor\";\n" "> +\t\tspi-max-frequency = <20000000>;\n" "> +\t\treg = <0>;\n" @@ -169,21 +179,21 @@ "> +\t\t\t#address-cells = <1>;\n" "> +\t\t\t#size-cells = <1>;\n" "> +\n" - "> +\t\t\tpartition at 0 {\n" + "> +\t\t\tpartition@0 {\n" "> +\t\t\t\tlabel = \"U-Boot\";\n" "> +\t\t\t\treg = <0x0 0xC0000>;\n" "\n" "Please consistently use lowercase for hex values.\n" "\n" "> +\t\t\t};\n" - "> +\t\t\tpartition at C0000 {\n" + "> +\t\t\tpartition@C0000 {\n" "> +\t\t\t\tlabel = \"env\";\n" "> +\t\t\t\treg = <0xC0000 0x10000>;\n" "> +\t\t\t};\n" "\n" "Have a new line between nodes.\n" "\n" - "> +\t\t\tpartition at D0000 {\n" + "> +\t\t\tpartition@D0000 {\n" "> +\t\t\t\tlabel = \"spare\";\n" "> +\t\t\t\treg = <0xD0000 0x130000>;\n" "> +\t\t\t};\n" @@ -223,7 +233,7 @@ "> +\tpinctrl-0 = <&pinctrl_i2c3>;\n" "> +\tstatus = \"okay\";\n" "> +\n" - "> +\tpmic at 58 {\n" + "> +\tpmic@58 {\n" "> +\t\tcompatible = \"dlg,da9063\";\n" "> +\t\treg = <0x58>;\n" "> +\t\tpinctrl-names = \"default\";\n" @@ -333,7 +343,7 @@ "> +\t\t};\n" "> +\t};\n" "> +\n" - "> +\trtc at 32 {\n" + "> +\trtc@32 {\n" "> +\t\tcompatible = \"epson,rx8010\";\n" "\n" "Is this compatible string documented somewhere?\n" @@ -666,4 +676,4 @@ "> \n" > -0d5b37d215f1e4206b7a293faeec8c3ec0fcb69242526837b8a57b84063ccd19 +ddfbdffc949b940f1cb07e06d12ccf31892dc194082f8c5afefcdcfebdee6c1c
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.