From mboxrd@z Thu Jan  1 00:00:00 1970
From: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
Subject: Re: [PATCH v5 2/2] ARM: shmobile: lager: enable internal PCI
Date: Thu, 26 Jun 2014 11:48:17 +0900
Message-ID: <53AB89F1.8040509@renesas.com>
References: <201406242157.05973.sergei.shtylyov@cogentembedded.com> <201406242202.22807.sergei.shtylyov@cogentembedded.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="ISO-8859-1"
Content-Transfer-Encoding: 7bit
Return-path: <linux-sh-owner@vger.kernel.org>
In-Reply-To: <201406242202.22807.sergei.shtylyov@cogentembedded.com>
Sender: linux-sh-owner@vger.kernel.org
To: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>, "horms@verge.net.au" <horms@verge.net.au>, "linux-sh@vger.kernel.org" <linux-sh@vger.kernel.org>, "robh+dt@kernel.org" <robh+dt@kernel.org>, "pawel.moll@arm.com" <pawel.moll@arm.com>, "mark.rutland@arm.com" <mark.rutland@arm.com>, "ijc+devicetree@hellion.org.uk" <ijc+devicetree@hellion.org.uk>, "galak@codeaurora.org" <galak@codeaurora.org>, "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>
Cc: "magnus.damm@gmail.com" <magnus.damm@gmail.com>, "linux@arm.linux.org.uk" <linux@arm.linux.org.uk>, "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org>, "ben.dooks@codethink.co.uk" <ben.dooks@codethink.co.uk>
List-Id: devicetree@vger.kernel.org

Hi Sergei,

(2014/06/25 3:02), Sergei Shtylyov wrote:
> From: Ben Dooks <ben.dooks@codethink.co.uk>
> 
> Enable internal AHB-PCI bridges for the USB EHCI/OHCI controllers attached to
> them.
> 
> Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk>
> Reviewed-by: Ian Molton <ian.molton@codethink.co.uk>
> [Sergei: enabled PCI0]
> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>

I also tested this patch, and it works.

Tested-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>

Best regards,
Yoshihiro shimoda

> ---
> Changes in version 4:
> - refreshed the patch.
> 
> Changes in version 3:
> - refreshed the patch.
> 
> Changes in version 2:
> - enabled PCI0 device;
> - reworded summary and changelog;
> - refreshed the patch.
> 
>  arch/arm/boot/dts/r8a7790-lager.dts |   33 +++++++++++++++++++++++++++++++++
>  1 file changed, 33 insertions(+)
> 
> Index: renesas/arch/arm/boot/dts/r8a7790-lager.dts
> ===================================================================
> --- renesas.orig/arch/arm/boot/dts/r8a7790-lager.dts
> +++ renesas/arch/arm/boot/dts/r8a7790-lager.dts
> @@ -219,6 +219,21 @@
>  		renesas,groups = "i2c3";
>  		renesas,function = "i2c3";
>  	};
> +
> +	usb0_pins: usb0 {
> +		renesas,groups = "usb0";
> +		renesas,function = "usb0";
> +	};
> +
> +	usb1_pins: usb1 {
> +		renesas,groups = "usb1";
> +		renesas,function = "usb1";
> +	};
> +
> +	usb2_pins: usb2 {
> +		renesas,groups = "usb2";
> +		renesas,function = "usb2";
> +	};
>  };
>  
>  &ether {
> @@ -368,3 +383,21 @@
>  		regulator-always-on;
>  	};
>  };
> +
> +&pci0 {
> +	status = "okay";
> +	pinctrl-0 = <&usb0_pins>;
> +	pinctrl-names = "default";
> +};
> +
> +&pci1 {
> +	status = "okay";
> +	pinctrl-0 = <&usb1_pins>;
> +	pinctrl-names = "default";
> +};
> +
> +&pci2 {
> +	status = "okay";
> +	pinctrl-0 = <&usb2_pins>;
> +	pinctrl-names = "default";
> +};
> --
> 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
>