Re: [PATCH v2 1/3] usb: phy: generic: migrate to gpio_desc

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Jan 27, 2015 at 10:40:39PM +0100, Robert Jarzmik wrote:
> Felipe Balbi <balbi@xxxxxx> writes:
> 
> > diff --git a/drivers/usb/phy/phy-generic.c b/drivers/usb/phy/phy-generic.c
> > index 9a826ff31951..34231c31cd1a 100644
> > --- a/drivers/usb/phy/phy-generic.c
> > +++ b/drivers/usb/phy/phy-generic.c
> > @@ -219,11 +219,11 @@ int usb_phy_gen_create_phy(struct device *dev, struct usb_phy_generic *nop,
> >  
> >  		needs_vcc = of_property_read_bool(node, "vcc-supply");
> >  		nop->gpiod_reset = devm_gpiod_get_optional(dev, "reset-gpios");
> > -		err = PTR_ERR(nop->gpiod_reset);
> > +		err = PTR_ERR_OR_ZERO(nop->gpiod_reset);
> >  		if (!err) {
> >  			nop->gpiod_vbus = devm_gpiod_get_optional(dev,
> >  							 "vbus-detect-gpio");
> > -			err = PTR_ERR(nop->gpiod_vbus);
> > +			err = PTR_ERR_OR_ZERO(nop->gpiod_vbus);
> >  		}
> >  	} else if (pdata) {
> >  		type = pdata->type;
> Funny, we ended up on the same solution ... almost, I add the "reset-gpios"
> into "reset" and "vbus-detect-gpio" into "vbus-detect" also. But it's a good
> sign we've ended up on the same code, isn't it ;) ?
> 
> Do you want me to rebase on your next tree and submit ? Or do you want to submit
> the patch yourself ?

your patch is more complete :-) Please send your version.

cheers

-- 
balbi

Attachment: signature.asc
Description: Digital signature


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux