Message ID | 20170322020313.24338-1-robh@kernel.org |
---|---|
State | New |
Headers | show |
Series | [01/15] ARM: dts: alpine: fix PCIe node name | expand |
Hi Rob, On Tue, Mar 21, 2017 at 09:02:59PM -0500, Rob Herring wrote: > PCIe bridges should have a node name of 'pcie'. > > Signed-off-by: Rob Herring <robh@kernel.org> Acked-by: Antoine Tenart <antoine.tenart@free-electrons.com> This can go through arm-soc directly. If not I'll take it. Thanks! Antoine > Cc: Tsahee Zidenberg <tsahee@annapurnalabs.com> > Cc: Antoine Tenart <antoine.tenart@free-electrons.com> > --- > Sub-arch maintainers, please apply to your trees unless arm-soc wants > to take the whole lot. > > arch/arm/boot/dts/alpine.dtsi | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm/boot/dts/alpine.dtsi b/arch/arm/boot/dts/alpine.dtsi > index d0eefc3b886c..f09f9c8e178a 100644 > --- a/arch/arm/boot/dts/alpine.dtsi > +++ b/arch/arm/boot/dts/alpine.dtsi > @@ -139,7 +139,7 @@ > }; > > /* Internal PCIe Controller */ > - pcie-internal@0xfbc00000 { > + pcie@fbc00000 { > compatible = "pci-host-ecam-generic"; > device_type = "pci"; > #size-cells = <2>; > -- > 2.10.1 > -- Antoine Ténart, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com
On Wed, Mar 22, 2017 at 9:01 AM, Antoine Tenart <antoine.tenart@free-electrons.com> wrote: > Hi Rob, > > On Tue, Mar 21, 2017 at 09:02:59PM -0500, Rob Herring wrote: >> PCIe bridges should have a node name of 'pcie'. >> >> Signed-off-by: Rob Herring <robh@kernel.org> > > Acked-by: Antoine Tenart <antoine.tenart@free-electrons.com> > > This can go through arm-soc directly. If not I'll take it. > Thanks! I have a mild preference for this kind of patch to get picked up by platform maintainers. Rob, once replies stop coming in, please send a pull request for anything that did not get picked up by anyone else. Arnd -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi Arnd, On Fri, Mar 24, 2017 at 06:00:30PM +0100, Arnd Bergmann wrote: > On Wed, Mar 22, 2017 at 9:01 AM, Antoine Tenart > <antoine.tenart@free-electrons.com> wrote: > > On Tue, Mar 21, 2017 at 09:02:59PM -0500, Rob Herring wrote: > >> PCIe bridges should have a node name of 'pcie'. > >> > >> Signed-off-by: Rob Herring <robh@kernel.org> > > > > Acked-by: Antoine Tenart <antoine.tenart@free-electrons.com> > > > > This can go through arm-soc directly. If not I'll take it. > > Thanks! > > I have a mild preference for this kind of patch to get picked up by > platform maintainers. Sure. Applied to alpine/dt. Thanks! Antoine -- Antoine Ténart, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com
diff --git a/arch/arm/boot/dts/alpine.dtsi b/arch/arm/boot/dts/alpine.dtsi index d0eefc3b886c..f09f9c8e178a 100644 --- a/arch/arm/boot/dts/alpine.dtsi +++ b/arch/arm/boot/dts/alpine.dtsi @@ -139,7 +139,7 @@ }; /* Internal PCIe Controller */ - pcie-internal@0xfbc00000 { + pcie@fbc00000 { compatible = "pci-host-ecam-generic"; device_type = "pci"; #size-cells = <2>;
PCIe bridges should have a node name of 'pcie'. Signed-off-by: Rob Herring <robh@kernel.org> Cc: Tsahee Zidenberg <tsahee@annapurnalabs.com> Cc: Antoine Tenart <antoine.tenart@free-electrons.com> --- Sub-arch maintainers, please apply to your trees unless arm-soc wants to take the whole lot. arch/arm/boot/dts/alpine.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- 2.10.1 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html