Re: usb: dwc2: gadget: Do not fail probe if there isn't a clock node

2014-11-24 Thread Dinh Nguyen
+CC: Felipe Balbi On 11/24/2014 04:48 AM, Dan Carpenter wrote: > On Mon, Nov 24, 2014 at 01:46:56PM +0300, Dan Carpenter wrote: >> Hello Dinh Nguyen, >> >> The patch 8d736d8a9c44: "usb: dwc2: gadget: Do not fail probe if >> there isn't a clock node"

Re: usb: dwc2: gadget: Do not fail probe if there isn't a clock node

2014-11-24 Thread Felipe Balbi
On Mon, Nov 24, 2014 at 10:49:57AM -0600, Dinh Nguyen wrote: > +CC: Felipe Balbi > > On 11/24/2014 04:48 AM, Dan Carpenter wrote: > > On Mon, Nov 24, 2014 at 01:46:56PM +0300, Dan Carpenter wrote: > >> Hello Dinh Nguyen, > >> > >> The patch 8d736d8a9c44

Re: usb: dwc2: gadget: Do not fail probe if there isn't a clock node

2014-11-24 Thread Dan Carpenter
On Mon, Nov 24, 2014 at 01:46:56PM +0300, Dan Carpenter wrote: > Hello Dinh Nguyen, > > The patch 8d736d8a9c44: "usb: dwc2: gadget: Do not fail probe if > there isn't a clock node" from Nov 11, 2014, leads to the following > static checker warning: > >

re: usb: dwc2: gadget: Do not fail probe if there isn't a clock node

2014-11-24 Thread Dan Carpenter
Hello Dinh Nguyen, The patch 8d736d8a9c44: "usb: dwc2: gadget: Do not fail probe if there isn't a clock node" from Nov 11, 2014, leads to the following static checker warning: drivers/usb/dwc2/gadget.c:3436 dwc2_gadget_init() warn: passing zero to 'PTR_ER

Re: [PATCHv7 6/8] usb: dwc2: gadget: Do not fail probe if there isn't a clock node

2014-11-14 Thread Felipe Balbi
On Fri, Nov 14, 2014 at 09:01:49PM +, Paul Zimmerman wrote: > > From: dingu...@opensource.altera.com [mailto:dingu...@opensource.altera.com] > > Sent: Tuesday, November 11, 2014 9:14 AM > > > > From: Dinh Nguyen > > > > Since the dwc2 hcd driver is currently not looking for a clock node duri

RE: [PATCHv7 6/8] usb: dwc2: gadget: Do not fail probe if there isn't a clock node

2014-11-14 Thread Paul Zimmerman
> From: dingu...@opensource.altera.com [mailto:dingu...@opensource.altera.com] > Sent: Tuesday, November 11, 2014 9:14 AM > > From: Dinh Nguyen > > Since the dwc2 hcd driver is currently not looking for a clock node during > init, we should not completely fail if there isn't a clock provided. >

[PATCHv7 6/8] usb: dwc2: gadget: Do not fail probe if there isn't a clock node

2014-11-11 Thread dinguyen
From: Dinh Nguyen Since the dwc2 hcd driver is currently not looking for a clock node during init, we should not completely fail if there isn't a clock provided. By assigning clk = NULL, this allows the driver, when configured for dual-role mode, to be able to continue loading the host portion of

Re: [PATCHv6 6/8] usb: dwc2: gadget: Do not fail probe if there isn't a clock node

2014-11-03 Thread Felipe Balbi
Hi, On Fri, Oct 31, 2014 at 02:31:31PM -0500, Dinh Nguyen wrote: > On 10/31/2014 12:42 PM, Felipe Balbi wrote: > > Hi, > > > > On Fri, Oct 31, 2014 at 10:20:06AM -0500, Dinh Nguyen wrote: > @@ -339,7 +339,8 @@ static void dwc2_handle_wakeup_detected_intr(struct > dwc2_hsotg *hsotg) > >

Re: [PATCHv6 6/8] usb: dwc2: gadget: Do not fail probe if there isn't a clock node

2014-10-31 Thread Dinh Nguyen
On 10/31/2014 02:31 PM, Dinh Nguyen wrote: > On 10/31/2014 12:42 PM, Felipe Balbi wrote: >> Hi, >> >> On Fri, Oct 31, 2014 at 10:20:06AM -0500, Dinh Nguyen wrote: > @@ -339,7 +339,8 @@ static void dwc2_handle_wakeup_detected_intr(struct > dwc2_hsotg *hsotg) > } >

Re: [PATCHv6 6/8] usb: dwc2: gadget: Do not fail probe if there isn't a clock node

2014-10-31 Thread Dinh Nguyen
On 10/31/2014 12:42 PM, Felipe Balbi wrote: > Hi, > > On Fri, Oct 31, 2014 at 10:20:06AM -0500, Dinh Nguyen wrote: @@ -339,7 +339,8 @@ static void dwc2_handle_wakeup_detected_intr(struct dwc2_hsotg *hsotg) } /* Change to L0 state */ hsotg-

Re: [PATCHv6 6/8] usb: dwc2: gadget: Do not fail probe if there isn't a clock node

2014-10-31 Thread Felipe Balbi
Hi, On Fri, Oct 31, 2014 at 10:20:06AM -0500, Dinh Nguyen wrote: > >> @@ -339,7 +339,8 @@ static void dwc2_handle_wakeup_detected_intr(struct > >> dwc2_hsotg *hsotg) > >>} > >>/* Change to L0 state */ > >>hsotg->lx_state = DWC2_L0; > >> - call_gadget(h

Re: [PATCHv6 6/8] usb: dwc2: gadget: Do not fail probe if there isn't a clock node

2014-10-31 Thread Dinh Nguyen
On 10/30/2014 09:04 AM, Felipe Balbi wrote: > Hi, > > On Tue, Oct 28, 2014 at 06:25:47PM -0500, dingu...@opensource.altera.com > wrote: >> From: Dinh Nguyen >> >> Since the dwc2 hcd driver is currently not looking for a clock node during >> init, we should not completely fail if there isn't a cl

Re: [PATCHv6 6/8] usb: dwc2: gadget: Do not fail probe if there isn't a clock node

2014-10-31 Thread Felipe Balbi
On Fri, Oct 31, 2014 at 10:38:28AM +0800, Kever Yang wrote: > Hi Dinh, > > On 10/29/2014 07:25 AM, dingu...@opensource.altera.com wrote: > >From: Dinh Nguyen > > > >Since the dwc2 hcd driver is currently not looking for a clock node during > >init, we should not completely fail if there isn't a c

Re: [PATCHv6 6/8] usb: dwc2: gadget: Do not fail probe if there isn't a clock node

2014-10-30 Thread Kever Yang
Hi Dinh, On 10/29/2014 07:25 AM, dingu...@opensource.altera.com wrote: From: Dinh Nguyen Since the dwc2 hcd driver is currently not looking for a clock node during init, we should not completely fail if there isn't a clock provided. For dual-role mode, we will only fail init for a non-clock no

Re: [PATCHv6 6/8] usb: dwc2: gadget: Do not fail probe if there isn't a clock node

2014-10-30 Thread Felipe Balbi
Hi, On Tue, Oct 28, 2014 at 06:25:47PM -0500, dingu...@opensource.altera.com wrote: > From: Dinh Nguyen > > Since the dwc2 hcd driver is currently not looking for a clock node during > init, we should not completely fail if there isn't a clock provided. > For dual-role mode, we will only fail in

RE: [PATCHv6 6/8] usb: dwc2: gadget: Do not fail probe if there isn't a clock node

2014-10-28 Thread Paul Zimmerman
> From: dingu...@opensource.altera.com [mailto:dingu...@opensource.altera.com] > Sent: Tuesday, October 28, 2014 4:26 PM > > Since the dwc2 hcd driver is currently not looking for a clock node during > init, we should not completely fail if there isn't a clock provided. > For dual-role mode, we wi

[PATCHv6 6/8] usb: dwc2: gadget: Do not fail probe if there isn't a clock node

2014-10-28 Thread dinguyen
From: Dinh Nguyen Since the dwc2 hcd driver is currently not looking for a clock node during init, we should not completely fail if there isn't a clock provided. For dual-role mode, we will only fail init for a non-clock node error. We then update the HCD to only call gadget funtions if there is

[PATCHv5 6/7] usb: dwc2: gadget: Do not fail probe if there isn't a clock node

2014-10-20 Thread dinguyen
From: Dinh Nguyen Since the dwc2 hcd driver is currently not looking for a clock node during init, we should not completely fail if there isn't a clock provided. For dual-role mode, we will only fail init for a non-clock node error. We then update the HCD to only call gadget funtions if there is

Re: [PATCHv4 08/12] usb: dwc2: gadget: Do not fail probe if there isn't a clock node

2014-09-19 Thread Dinh Nguyen
On 9/12/14, 11:28 AM, Bartlomiej Zolnierkiewicz wrote: > > [ added linux-kernel ML to cc: ] > > Hi, > > On Tuesday, August 26, 2014 11:19:59 AM dingu...@opensource.altera.com wrote: >> From: Dinh Nguyen >> >> Since the dwc2 hcd driver is currently not looking for a clock node during >> init,

Re: [PATCHv4 08/12] usb: dwc2: gadget: Do not fail probe if there isn't a clock node

2014-09-12 Thread Bartlomiej Zolnierkiewicz
[ added linux-kernel ML to cc: ] Hi, On Tuesday, August 26, 2014 11:19:59 AM dingu...@opensource.altera.com wrote: > From: Dinh Nguyen > > Since the dwc2 hcd driver is currently not looking for a clock node during > init, we should not completely fail if there isn't a clock provided. > Add a c

[PATCHv4 08/12] usb: dwc2: gadget: Do not fail probe if there isn't a clock node

2014-08-26 Thread dinguyen
From: Dinh Nguyen Since the dwc2 hcd driver is currently not looking for a clock node during init, we should not completely fail if there isn't a clock provided. Add a check for a valid clock before calling clock functions. Signed-off-by: Dinh Nguyen Acked-by: Paul Zimmerman --- drivers/usb/d

[PATCHv3 08/12] usb: dwc2: gadget: Do not fail probe if there isn't a clock node

2014-08-08 Thread dinguyen
From: Dinh Nguyen Since the dwc2 hcd driver is currently not looking for a clock node during init, we should not completely fail if there isn't a clock provided. Add a check for a valid clock before calling clock functions. Signed-off-by: Dinh Nguyen --- drivers/usb/dwc2/gadget.c | 36

[PATCHv2 09/13] usb: dwc2: gadget: Do not fail probe if there isn't a clock node

2014-07-30 Thread dinguyen
From: Dinh Nguyen Since the dwc2 hcd driver is currently not looking for a clock node during init, we should not completely fail if there isn't a clock provided. Add a check for a valid clock before calling clock functions. Signed-off-by: Dinh Nguyen --- drivers/usb/dwc2/gadget.c | 37 ++

[PATCH 08/12] usb: dwc2: gadget: Do not fail probe if there isn't a clock node

2014-07-16 Thread dinguyen
From: Dinh Nguyen Since the dwc2 hcd driver is currently not looking for a clock node during init, we should not completely fail if there isn't a clock provided. Add a check for a valid clock before calling clock functions. Signed-off-by: Dinh Nguyen --- drivers/usb/dwc2/gadget.c | 15 ++

[RESEND PATCH 07/15] usb: dwc2: gadget: Do not fail probe if there isn't a clock node

2014-06-20 Thread dinguyen
From: Dinh Nguyen Since the dwc2 hcd driver is currently not looking for a clock node during init, we should not completely fail if there isn't a clock provided. Add a check for a valid clock before calling clock functions. Signed-off-by: Dinh Nguyen --- drivers/usb/dwc2/gadget.c | 15 ++

[PATCH 07/15] usb: dwc2: gadget: Do not fail probe if there isn't a clock node

2014-06-20 Thread dinguyen
From: Dinh Nguyen Since the dwc2 hcd driver is currently not looking for a clock node during init, we should not completely fail if there isn't a clock provided. Add a check for a valid clock before calling clock functions. Signed-off-by: Dinh Nguyen --- drivers/usb/dwc2/gadget.c | 15 ++