diff mbox series

[01/15] ARM: dts: alpine: fix PCIe node name

Message ID 20170322020313.24338-1-robh@kernel.org
State New
Headers show
Series [01/15] ARM: dts: alpine: fix PCIe node name | expand

Commit Message

Rob Herring March 22, 2017, 2:02 a.m. UTC
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

Comments

Antoine Tenart March 22, 2017, 8:01 a.m. UTC | #1
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
Arnd Bergmann March 24, 2017, 5 p.m. UTC | #2
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
Antoine Tenart March 24, 2017, 6:25 p.m. UTC | #3
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 mbox series

Patch

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>;