All of lore.kernel.org
 help / color / mirror / Atom feed
From: Roger Quadros <rogerq@ti.com>
To: tony@atomide.com
Cc: Roger Quadros <rogerq@ti.com>,
	balbi@ti.com, bcousson@baylibre.com, p.zabel@pengutronix.de,
	broonie@kernel.org, linux-omap@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org, linux-usb@vger.kernel.org,
	linux-kernel@vger.kernel.org, devicetree@vger.kernel.org
Subject: Re: [PATCH v3 04/10] ARM: OMAP2+: usb-host: Adapt to USB phy-nop RESET line changes
Date: Wed, 2 Oct 2013 13:20:04 +0300	[thread overview]
Message-ID: <524BF354.3000502@ti.com> (raw)
In-Reply-To: <1380012836-29900-5-git-send-email-rogerq@ti.com>

Hi Tony,

On 09/24/2013 11:53 AM, Roger Quadros wrote:
> The USB phy-nop nop driver expects the RESET line information
> to be sent as a GPIO number via platform data. Adapt to that.
> 
> Signed-off-by: Roger Quadros <rogerq@ti.com>

Need your Ack on this one as well.

cheers,
-roger

> ---
>  arch/arm/mach-omap2/usb-host.c |   11 +----------
>  1 files changed, 1 insertions(+), 10 deletions(-)
> 
> diff --git a/arch/arm/mach-omap2/usb-host.c b/arch/arm/mach-omap2/usb-host.c
> index 78ac1c2..10855eb 100644
> --- a/arch/arm/mach-omap2/usb-host.c
> +++ b/arch/arm/mach-omap2/usb-host.c
> @@ -460,8 +460,7 @@ int usbhs_init_phys(struct usbhs_phy_data *phy, int num_phys)
>  		memset(&nop_pdata, 0, sizeof(nop_pdata));
>  		if (gpio_is_valid(phy->vcc_gpio))
>  			nop_pdata.needs_vcc = true;
> -		if (gpio_is_valid(phy->reset_gpio))
> -			nop_pdata.needs_reset = true;
> +		nop_pdata.gpio_reset = phy->reset_gpio;
>  		nop_pdata.type = USB_PHY_TYPE_USB2;
>  
>  		/* create a NOP PHY device */
> @@ -483,14 +482,6 @@ int usbhs_init_phys(struct usbhs_phy_data *phy, int num_phys)
>  
>  		usb_bind_phy("ehci-omap.0", phy->port - 1, phy_id);
>  
> -		/* Do we need RESET regulator ? */
> -		if (gpio_is_valid(phy->reset_gpio)) {
> -			scnprintf(rail_name, MAX_STR,
> -					"hsusb%d_reset", phy->port);
> -			usbhs_add_regulator(rail_name, phy_id, "reset",
> -						phy->reset_gpio, 1);
> -		}
> -
>  		/* Do we need VCC regulator ? */
>  		if (gpio_is_valid(phy->vcc_gpio)) {
>  			scnprintf(rail_name, MAX_STR, "hsusb%d_vcc", phy->port);
> 

WARNING: multiple messages have this Message-ID (diff)
From: rogerq@ti.com (Roger Quadros)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v3 04/10] ARM: OMAP2+: usb-host: Adapt to USB phy-nop RESET line changes
Date: Wed, 2 Oct 2013 13:20:04 +0300	[thread overview]
Message-ID: <524BF354.3000502@ti.com> (raw)
In-Reply-To: <1380012836-29900-5-git-send-email-rogerq@ti.com>

Hi Tony,

On 09/24/2013 11:53 AM, Roger Quadros wrote:
> The USB phy-nop nop driver expects the RESET line information
> to be sent as a GPIO number via platform data. Adapt to that.
> 
> Signed-off-by: Roger Quadros <rogerq@ti.com>

Need your Ack on this one as well.

cheers,
-roger

> ---
>  arch/arm/mach-omap2/usb-host.c |   11 +----------
>  1 files changed, 1 insertions(+), 10 deletions(-)
> 
> diff --git a/arch/arm/mach-omap2/usb-host.c b/arch/arm/mach-omap2/usb-host.c
> index 78ac1c2..10855eb 100644
> --- a/arch/arm/mach-omap2/usb-host.c
> +++ b/arch/arm/mach-omap2/usb-host.c
> @@ -460,8 +460,7 @@ int usbhs_init_phys(struct usbhs_phy_data *phy, int num_phys)
>  		memset(&nop_pdata, 0, sizeof(nop_pdata));
>  		if (gpio_is_valid(phy->vcc_gpio))
>  			nop_pdata.needs_vcc = true;
> -		if (gpio_is_valid(phy->reset_gpio))
> -			nop_pdata.needs_reset = true;
> +		nop_pdata.gpio_reset = phy->reset_gpio;
>  		nop_pdata.type = USB_PHY_TYPE_USB2;
>  
>  		/* create a NOP PHY device */
> @@ -483,14 +482,6 @@ int usbhs_init_phys(struct usbhs_phy_data *phy, int num_phys)
>  
>  		usb_bind_phy("ehci-omap.0", phy->port - 1, phy_id);
>  
> -		/* Do we need RESET regulator ? */
> -		if (gpio_is_valid(phy->reset_gpio)) {
> -			scnprintf(rail_name, MAX_STR,
> -					"hsusb%d_reset", phy->port);
> -			usbhs_add_regulator(rail_name, phy_id, "reset",
> -						phy->reset_gpio, 1);
> -		}
> -
>  		/* Do we need VCC regulator ? */
>  		if (gpio_is_valid(phy->vcc_gpio)) {
>  			scnprintf(rail_name, MAX_STR, "hsusb%d_vcc", phy->port);
> 

WARNING: multiple messages have this Message-ID (diff)
From: Roger Quadros <rogerq@ti.com>
To: <tony@atomide.com>
Cc: Roger Quadros <rogerq@ti.com>, <balbi@ti.com>,
	<bcousson@baylibre.com>, <p.zabel@pengutronix.de>,
	<broonie@kernel.org>, <linux-omap@vger.kernel.org>,
	<linux-arm-kernel@lists.infradead.org>,
	<linux-usb@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
	<devicetree@vger.kernel.org>
Subject: Re: [PATCH v3 04/10] ARM: OMAP2+: usb-host: Adapt to USB phy-nop RESET line changes
Date: Wed, 2 Oct 2013 13:20:04 +0300	[thread overview]
Message-ID: <524BF354.3000502@ti.com> (raw)
In-Reply-To: <1380012836-29900-5-git-send-email-rogerq@ti.com>

Hi Tony,

On 09/24/2013 11:53 AM, Roger Quadros wrote:
> The USB phy-nop nop driver expects the RESET line information
> to be sent as a GPIO number via platform data. Adapt to that.
> 
> Signed-off-by: Roger Quadros <rogerq@ti.com>

Need your Ack on this one as well.

cheers,
-roger

> ---
>  arch/arm/mach-omap2/usb-host.c |   11 +----------
>  1 files changed, 1 insertions(+), 10 deletions(-)
> 
> diff --git a/arch/arm/mach-omap2/usb-host.c b/arch/arm/mach-omap2/usb-host.c
> index 78ac1c2..10855eb 100644
> --- a/arch/arm/mach-omap2/usb-host.c
> +++ b/arch/arm/mach-omap2/usb-host.c
> @@ -460,8 +460,7 @@ int usbhs_init_phys(struct usbhs_phy_data *phy, int num_phys)
>  		memset(&nop_pdata, 0, sizeof(nop_pdata));
>  		if (gpio_is_valid(phy->vcc_gpio))
>  			nop_pdata.needs_vcc = true;
> -		if (gpio_is_valid(phy->reset_gpio))
> -			nop_pdata.needs_reset = true;
> +		nop_pdata.gpio_reset = phy->reset_gpio;
>  		nop_pdata.type = USB_PHY_TYPE_USB2;
>  
>  		/* create a NOP PHY device */
> @@ -483,14 +482,6 @@ int usbhs_init_phys(struct usbhs_phy_data *phy, int num_phys)
>  
>  		usb_bind_phy("ehci-omap.0", phy->port - 1, phy_id);
>  
> -		/* Do we need RESET regulator ? */
> -		if (gpio_is_valid(phy->reset_gpio)) {
> -			scnprintf(rail_name, MAX_STR,
> -					"hsusb%d_reset", phy->port);
> -			usbhs_add_regulator(rail_name, phy_id, "reset",
> -						phy->reset_gpio, 1);
> -		}
> -
>  		/* Do we need VCC regulator ? */
>  		if (gpio_is_valid(phy->vcc_gpio)) {
>  			scnprintf(rail_name, MAX_STR, "hsusb%d_vcc", phy->port);
> 


  reply	other threads:[~2013-10-02 10:20 UTC|newest]

Thread overview: 71+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-09-24  8:53 [PATCH v3 00/10] USB: phy: phy-nop: Manage RESET GPIO in the driver Roger Quadros
2013-09-24  8:53 ` Roger Quadros
2013-09-24  8:53 ` Roger Quadros
2013-09-24  8:53 ` [PATCH v3 02/10] usb: phy: generic: Don't use regulator framework for RESET line Roger Quadros
2013-09-24  8:53   ` Roger Quadros
2013-09-24  8:53   ` Roger Quadros
     [not found] ` <1380012836-29900-1-git-send-email-rogerq-l0cyMroinI0@public.gmane.org>
2013-09-24  8:53   ` [PATCH v3 01/10] usb: phy: generic: Add gpio_reset to platform data Roger Quadros
2013-09-24  8:53     ` Roger Quadros
2013-09-24  8:53     ` Roger Quadros
2013-09-24  8:53   ` [PATCH v3 03/10] ARM: OMAP2+: omap-usb-host: Get rid of platform_data from struct usbhs_phy_data Roger Quadros
2013-09-24  8:53     ` Roger Quadros
2013-09-24  8:53     ` Roger Quadros
     [not found]     ` <1380012836-29900-4-git-send-email-rogerq-l0cyMroinI0@public.gmane.org>
2013-10-02 10:19       ` Roger Quadros
2013-10-02 10:19         ` Roger Quadros
2013-10-02 10:19         ` Roger Quadros
2013-10-03  4:03         ` Tony Lindgren
2013-10-03  4:03           ` Tony Lindgren
2013-09-24  8:53   ` [PATCH v3 04/10] ARM: OMAP2+: usb-host: Adapt to USB phy-nop RESET line changes Roger Quadros
2013-09-24  8:53     ` Roger Quadros
2013-09-24  8:53     ` Roger Quadros
2013-10-02 10:20     ` Roger Quadros [this message]
2013-10-02 10:20       ` Roger Quadros
2013-10-02 10:20       ` Roger Quadros
     [not found]       ` <524BF354.3000502-l0cyMroinI0@public.gmane.org>
2013-10-03  4:03         ` Tony Lindgren
2013-10-03  4:03           ` Tony Lindgren
2013-10-03  4:03           ` Tony Lindgren
2013-09-24  8:53   ` [PATCH v3 05/10] ARM: dts: omap3-beagle: Use reset-gpios for hsusb2_reset Roger Quadros
2013-09-24  8:53     ` Roger Quadros
2013-09-24  8:53     ` Roger Quadros
2013-10-03 10:34     ` Roger Quadros
2013-10-03 10:34       ` Roger Quadros
2013-10-03 10:34       ` Roger Quadros
     [not found]       ` <524D483D.9040907-l0cyMroinI0@public.gmane.org>
2013-10-03 12:05         ` Benoit Cousson
2013-10-03 12:05           ` Benoit Cousson
2013-10-03 12:05           ` Benoit Cousson
2013-10-03 13:44           ` Benoit Cousson
2013-10-03 13:44             ` Benoit Cousson
     [not found]             ` <524D74A5.7010604-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
2013-10-03 13:58               ` Roger Quadros
2013-10-03 13:58                 ` Roger Quadros
2013-10-03 13:58                 ` Roger Quadros
     [not found]                 ` <524D780C.1090708-l0cyMroinI0@public.gmane.org>
2013-10-03 14:25                   ` Benoit Cousson
2013-10-03 14:25                     ` Benoit Cousson
2013-10-03 14:25                     ` Benoit Cousson
2013-09-24  8:53   ` [PATCH v3 07/10] ARM: dts: omap5-uevm: Use reset-gpios for hsusb2/3_reset Roger Quadros
2013-09-24  8:53     ` Roger Quadros
2013-09-24  8:53     ` Roger Quadros
2013-09-24  8:53 ` [PATCH v3 06/10] ARM: dts: omap4-panda: Use reset-gpios for hsusb1_reset Roger Quadros
2013-09-24  8:53   ` Roger Quadros
2013-09-24  8:53   ` Roger Quadros
2013-09-24  8:53 ` [PATCH v3 08/10] ARM: dts: omap3-beagle: Make USB host pin naming consistent Roger Quadros
2013-09-24  8:53   ` Roger Quadros
2013-09-24  8:53   ` Roger Quadros
2013-09-24  8:53 ` [PATCH v3 09/10] ARM: dts: omap3-beagle-xm: Add USB Host support Roger Quadros
2013-09-24  8:53   ` Roger Quadros
2013-09-24  8:53   ` Roger Quadros
     [not found]   ` <1380012836-29900-10-git-send-email-rogerq-l0cyMroinI0@public.gmane.org>
2013-11-19 21:05     ` Nishanth Menon
2013-11-19 21:05       ` Nishanth Menon
2013-11-19 21:05       ` Nishanth Menon
2013-11-20 10:32       ` Roger Quadros
2013-11-20 10:32         ` Roger Quadros
2013-11-20 10:32         ` Roger Quadros
2013-11-26 21:04         ` Tony Lindgren
2013-11-26 21:04           ` Tony Lindgren
2013-11-26 22:15           ` Nishanth Menon
2013-11-26 22:15             ` Nishanth Menon
2013-09-24  8:53 ` [PATCH v3 10/10] ARM: dts: omap3-beagle: Add USB OTG PHY details Roger Quadros
2013-09-24  8:53   ` Roger Quadros
2013-09-24  8:53   ` Roger Quadros
2013-10-03 10:32 ` [PATCH v3 00/10] USB: phy: phy-nop: Manage RESET GPIO in the driver Roger Quadros
2013-10-03 10:32   ` Roger Quadros
2013-10-03 10:32   ` Roger Quadros

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=524BF354.3000502@ti.com \
    --to=rogerq@ti.com \
    --cc=balbi@ti.com \
    --cc=bcousson@baylibre.com \
    --cc=broonie@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=p.zabel@pengutronix.de \
    --cc=tony@atomide.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.