diff for duplicates of <20170712094655.GC7472@leverpostej> diff --git a/a/1.txt b/N1/1.txt index e4d8268..b0b7c9c 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,6 +1,6 @@ Hi, -On Wed, Jul 12, 2017 at 11:04:08AM +0200, linux-kernel-dev at beckhoff.com wrote: +On Wed, Jul 12, 2017 at 11:04:08AM +0200, linux-kernel-dev-QonKdJ6Bx35Wk0Htik3J/w@public.gmane.org wrote: > +/dts-v1/; > +#include "imx53.dtsi" > + @@ -29,7 +29,7 @@ No baud-rate or bits configuration? > + compatible = "bhf,emi-ccat"; > + }; > + -> + display0: display at di0 { +> + display0: display@di0 { This unit-address (the bit after the @) isn't valid, as that should match a reg or ranges, but this node has neither. @@ -44,14 +44,14 @@ Just call this display-0. > + pinctrl-0 = <&pinctrl_ipu_disp0>; > + status = "okay"; > + -> + port at 0 { +> + port@0 { > + reg = <0>; > + display0_in: endpoint { > + remote-endpoint = <&ipu_di0_disp0>; > + }; > + }; > + -> + port at 1 { +> + port@1 { > + reg = <1>; > + display0_out: endpoint { > + remote-endpoint = <&panel_in>; @@ -59,7 +59,7 @@ Just call this display-0. > + }; > + }; > + -> + dvi_panel: display at 0 { +> + dvi_panel: display@0 { Likewise you have no reg here, so the unit address isn't valid. @@ -90,7 +90,7 @@ Where has this come from? > + #address-cells = <1>; > + #size-cells = <0>; > + -> + reg_3p2v: regulator at 0 { +> + reg_3p2v: regulator@0 { > + compatible = "regulator-fixed"; > + reg = <0>; @@ -102,7 +102,7 @@ Meaningless reg entry. > + regulator-always-on; > + }; > + -> + reg_usb_vbus: regulator at 1 { +> + reg_usb_vbus: regulator@1 { > + compatible = "regulator-fixed"; > + reg = <1>; @@ -141,3 +141,7 @@ Otherwise, looks fine to me. Thanks, Mark. +-- +To unsubscribe from this list: send the line "unsubscribe devicetree" in +the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org +More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/a/content_digest b/N1/content_digest index 2f42cb9..645a298 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,13 +1,23 @@ "ref\020170712090408.12212-1-linux-kernel-dev@beckhoff.com\0" - "From\0mark.rutland@arm.com (Mark Rutland)\0" - "Subject\0[PATCH] ARM: dts: imx: add CX9020 Embedded PC device tree\0" + "ref\020170712090408.12212-1-linux-kernel-dev-QonKdJ6Bx35Wk0Htik3J/w@public.gmane.org\0" + "From\0Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>\0" + "Subject\0Re: [PATCH] ARM: dts: imx: add CX9020 Embedded PC device tree\0" "Date\0Wed, 12 Jul 2017 10:46:55 +0100\0" - "To\0linux-arm-kernel@lists.infradead.org\0" + "To\0linux-kernel-dev-QonKdJ6Bx35Wk0Htik3J/w@public.gmane.org\0" + "Cc\0robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org" + shawnguo-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org + kernel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org + linux-I+IVW8TIWO2tmTQ+vhA3Yw@public.gmane.org + fabio.estevam-3arQi8VN3Tc@public.gmane.org + linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org + devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org + linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org + " Patrick Br\303\274nn <p.bruenn-QonKdJ6Bx35Wk0Htik3J/w@public.gmane.org>\0" "\00:1\0" "b\0" "Hi,\n" "\n" - "On Wed, Jul 12, 2017 at 11:04:08AM +0200, linux-kernel-dev at beckhoff.com wrote:\n" + "On Wed, Jul 12, 2017 at 11:04:08AM +0200, linux-kernel-dev-QonKdJ6Bx35Wk0Htik3J/w@public.gmane.org wrote:\n" "> +/dts-v1/;\n" "> +#include \"imx53.dtsi\"\n" "> +\n" @@ -36,7 +46,7 @@ "> +\t\tcompatible = \"bhf,emi-ccat\";\n" "> +\t};\n" "> +\n" - "> +\tdisplay0: display at di0 {\n" + "> +\tdisplay0: display@di0 {\n" "\n" "This unit-address (the bit after the @) isn't valid, as that should\n" "match a reg or ranges, but this node has neither.\n" @@ -51,14 +61,14 @@ "> +\t\tpinctrl-0 = <&pinctrl_ipu_disp0>;\n" "> +\t\tstatus = \"okay\";\n" "> +\n" - "> +\t\tport at 0 {\n" + "> +\t\tport@0 {\n" "> +\t\t\treg = <0>;\n" "> +\t\t\tdisplay0_in: endpoint {\n" "> +\t\t\t\tremote-endpoint = <&ipu_di0_disp0>;\n" "> +\t\t\t};\n" "> +\t\t};\n" "> +\n" - "> +\t\tport at 1 {\n" + "> +\t\tport@1 {\n" "> +\t\t\treg = <1>;\n" "> +\t\t\tdisplay0_out: endpoint {\n" "> +\t\t\t\tremote-endpoint = <&panel_in>;\n" @@ -66,7 +76,7 @@ "> +\t\t};\n" "> +\t};\n" "> +\n" - "> +\tdvi_panel: display at 0 {\n" + "> +\tdvi_panel: display@0 {\n" "\n" "Likewise you have no reg here, so the unit address isn't valid.\n" "\n" @@ -97,7 +107,7 @@ "> +\t\t#address-cells = <1>;\n" "> +\t\t#size-cells = <0>;\n" "> +\n" - "> +\t\treg_3p2v: regulator at 0 {\n" + "> +\t\treg_3p2v: regulator@0 {\n" "> +\t\t\tcompatible = \"regulator-fixed\";\n" "> +\t\t\treg = <0>;\n" "\n" @@ -109,7 +119,7 @@ "> +\t\t\tregulator-always-on;\n" "> +\t\t};\n" "> +\n" - "> +\t\treg_usb_vbus: regulator at 1 {\n" + "> +\t\treg_usb_vbus: regulator@1 {\n" "> +\t\t\tcompatible = \"regulator-fixed\";\n" "> +\t\t\treg = <1>;\n" "\n" @@ -147,6 +157,10 @@ "Otherwise, looks fine to me.\n" "\n" "Thanks,\n" - Mark. + "Mark.\n" + "--\n" + "To unsubscribe from this list: send the line \"unsubscribe devicetree\" in\n" + "the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org\n" + More majordomo info at http://vger.kernel.org/majordomo-info.html -abc398a6f2de7b695a0c506f01e471f22d9372877500ce7db2fc3156d9bbbd6f +32a37926f74cc99319bcbd2858d4ef99171cbf2e1f38f91ac6f0a995ac4eff54
diff --git a/a/1.txt b/N2/1.txt index e4d8268..5f41092 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -1,6 +1,6 @@ Hi, -On Wed, Jul 12, 2017 at 11:04:08AM +0200, linux-kernel-dev at beckhoff.com wrote: +On Wed, Jul 12, 2017 at 11:04:08AM +0200, linux-kernel-dev@beckhoff.com wrote: > +/dts-v1/; > +#include "imx53.dtsi" > + @@ -29,7 +29,7 @@ No baud-rate or bits configuration? > + compatible = "bhf,emi-ccat"; > + }; > + -> + display0: display at di0 { +> + display0: display@di0 { This unit-address (the bit after the @) isn't valid, as that should match a reg or ranges, but this node has neither. @@ -44,14 +44,14 @@ Just call this display-0. > + pinctrl-0 = <&pinctrl_ipu_disp0>; > + status = "okay"; > + -> + port at 0 { +> + port@0 { > + reg = <0>; > + display0_in: endpoint { > + remote-endpoint = <&ipu_di0_disp0>; > + }; > + }; > + -> + port at 1 { +> + port@1 { > + reg = <1>; > + display0_out: endpoint { > + remote-endpoint = <&panel_in>; @@ -59,7 +59,7 @@ Just call this display-0. > + }; > + }; > + -> + dvi_panel: display at 0 { +> + dvi_panel: display@0 { Likewise you have no reg here, so the unit address isn't valid. @@ -90,7 +90,7 @@ Where has this come from? > + #address-cells = <1>; > + #size-cells = <0>; > + -> + reg_3p2v: regulator at 0 { +> + reg_3p2v: regulator@0 { > + compatible = "regulator-fixed"; > + reg = <0>; @@ -102,7 +102,7 @@ Meaningless reg entry. > + regulator-always-on; > + }; > + -> + reg_usb_vbus: regulator at 1 { +> + reg_usb_vbus: regulator@1 { > + compatible = "regulator-fixed"; > + reg = <1>; diff --git a/a/content_digest b/N2/content_digest index 2f42cb9..8595828 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -1,13 +1,22 @@ "ref\020170712090408.12212-1-linux-kernel-dev@beckhoff.com\0" - "From\0mark.rutland@arm.com (Mark Rutland)\0" - "Subject\0[PATCH] ARM: dts: imx: add CX9020 Embedded PC device tree\0" + "From\0Mark Rutland <mark.rutland@arm.com>\0" + "Subject\0Re: [PATCH] ARM: dts: imx: add CX9020 Embedded PC device tree\0" "Date\0Wed, 12 Jul 2017 10:46:55 +0100\0" - "To\0linux-arm-kernel@lists.infradead.org\0" + "To\0linux-kernel-dev@beckhoff.com\0" + "Cc\0robh+dt@kernel.org" + shawnguo@kernel.org + kernel@pengutronix.de + linux@armlinux.org.uk + fabio.estevam@nxp.com + linux-arm-kernel@lists.infradead.org + devicetree@vger.kernel.org + linux-kernel@vger.kernel.org + " Patrick Br\303\274nn <p.bruenn@beckhoff.com>\0" "\00:1\0" "b\0" "Hi,\n" "\n" - "On Wed, Jul 12, 2017 at 11:04:08AM +0200, linux-kernel-dev at beckhoff.com wrote:\n" + "On Wed, Jul 12, 2017 at 11:04:08AM +0200, linux-kernel-dev@beckhoff.com wrote:\n" "> +/dts-v1/;\n" "> +#include \"imx53.dtsi\"\n" "> +\n" @@ -36,7 +45,7 @@ "> +\t\tcompatible = \"bhf,emi-ccat\";\n" "> +\t};\n" "> +\n" - "> +\tdisplay0: display at di0 {\n" + "> +\tdisplay0: display@di0 {\n" "\n" "This unit-address (the bit after the @) isn't valid, as that should\n" "match a reg or ranges, but this node has neither.\n" @@ -51,14 +60,14 @@ "> +\t\tpinctrl-0 = <&pinctrl_ipu_disp0>;\n" "> +\t\tstatus = \"okay\";\n" "> +\n" - "> +\t\tport at 0 {\n" + "> +\t\tport@0 {\n" "> +\t\t\treg = <0>;\n" "> +\t\t\tdisplay0_in: endpoint {\n" "> +\t\t\t\tremote-endpoint = <&ipu_di0_disp0>;\n" "> +\t\t\t};\n" "> +\t\t};\n" "> +\n" - "> +\t\tport at 1 {\n" + "> +\t\tport@1 {\n" "> +\t\t\treg = <1>;\n" "> +\t\t\tdisplay0_out: endpoint {\n" "> +\t\t\t\tremote-endpoint = <&panel_in>;\n" @@ -66,7 +75,7 @@ "> +\t\t};\n" "> +\t};\n" "> +\n" - "> +\tdvi_panel: display at 0 {\n" + "> +\tdvi_panel: display@0 {\n" "\n" "Likewise you have no reg here, so the unit address isn't valid.\n" "\n" @@ -97,7 +106,7 @@ "> +\t\t#address-cells = <1>;\n" "> +\t\t#size-cells = <0>;\n" "> +\n" - "> +\t\treg_3p2v: regulator at 0 {\n" + "> +\t\treg_3p2v: regulator@0 {\n" "> +\t\t\tcompatible = \"regulator-fixed\";\n" "> +\t\t\treg = <0>;\n" "\n" @@ -109,7 +118,7 @@ "> +\t\t\tregulator-always-on;\n" "> +\t\t};\n" "> +\n" - "> +\t\treg_usb_vbus: regulator at 1 {\n" + "> +\t\treg_usb_vbus: regulator@1 {\n" "> +\t\t\tcompatible = \"regulator-fixed\";\n" "> +\t\t\treg = <1>;\n" "\n" @@ -149,4 +158,4 @@ "Thanks,\n" Mark. -abc398a6f2de7b695a0c506f01e471f22d9372877500ce7db2fc3156d9bbbd6f +07a7d35436f2be4f4a2acff9fc6ab8ad369f70c17680f69a83c56c8dc9b5122b
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.