lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Thu, 18 Feb 2016 17:39:48 +0100 From: Marek Vasut <marex@...x.de> To: Alan Stern <stern@...land.harvard.edu>, Antony Pavlov <antonynpavlov@...il.com> CC: linux-mips@...ux-mips.org, Wills Wang <wills.wang@...e.com>, Daniel Schwierzeck <daniel.schwierzeck@...il.com>, Alban Bedel <albeu@...e.fr>, Greg Kroah-Hartman <gregkh@...uxfoundation.org>, linux-usb@...r.kernel.org, linux-kernel@...r.kernel.org Subject: Re: [RFC v5 07/15] usb: ehci: add vbus-gpio parameter On 02/18/2016 05:12 PM, Alan Stern wrote: > On Tue, 9 Feb 2016, Antony Pavlov wrote: > >> This patch retrieves and configures the vbus control gpio via >> the device tree. >> >> This patch is based on a ehci-s5p.c commit fd81d59c90d38661 >> ("USB: ehci-s5p: Add vbus setup function to the s5p ehci glue layer"). >> >> Signed-off-by: Antony Pavlov <antonynpavlov@...il.com> >> Cc: Alan Stern <stern@...land.harvard.edu> >> Cc: Greg Kroah-Hartman <gregkh@...uxfoundation.org> >> Cc: linux-usb@...r.kernel.org >> Cc: linux-kernel@...r.kernel.org >> --- >> drivers/usb/host/ehci-platform.c | 22 ++++++++++++++++++++++ >> 1 file changed, 22 insertions(+) >> >> diff --git a/drivers/usb/host/ehci-platform.c b/drivers/usb/host/ehci-platform.c >> index bd7082f2..0d95ced 100644 >> --- a/drivers/usb/host/ehci-platform.c >> +++ b/drivers/usb/host/ehci-platform.c >> @@ -28,6 +28,7 @@ >> #include <linux/io.h> >> #include <linux/module.h> >> #include <linux/of.h> >> +#include <linux/of_gpio.h> >> #include <linux/phy/phy.h> >> #include <linux/platform_device.h> >> #include <linux/reset.h> >> @@ -142,6 +143,25 @@ static struct usb_ehci_pdata ehci_platform_defaults = { >> .power_off = ehci_platform_power_off, >> }; >> >> +static void setup_vbus_gpio(struct device *dev) >> +{ >> + int err; >> + int gpio; >> + >> + if (!dev->of_node) >> + return; >> + >> + gpio = of_get_named_gpio(dev->of_node, "vbus-gpio", 0); >> + if (!gpio_is_valid(gpio)) >> + return; >> + >> + err = devm_gpio_request_one(dev, gpio, >> + GPIOF_OUT_INIT_HIGH | GPIOF_EXPORT_DIR_FIXED, >> + "ehci_vbus_gpio"); >> + if (err) >> + dev_err(dev, "can't request ehci vbus gpio %d", gpio); > > I don't understand this. If you get an error here, what's the point of > allowing the probe to continue? Shouldn't you return an error code so > the probe will fail? The idea is I believe that if there is no vbus gpio specified, the port might just not have vbus control, so the probe can continue. But this patch is irrelevant anyway, since Alexey will switch to CI HDRC driver and use standard regulator, as it should be done.
Powered by blists - more mailing lists