diff mbox

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

Message ID 20170322020313.24338-1-robh@kernel.org (mailing list archive)
State New, archived
Headers show

Commit Message

Rob Herring (Arm) 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(-)

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
>
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
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
diff mbox

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