From: Peter Chen <hzpeterchen@gmail.com>
To: Vaibhav Hiremath <vaibhav.hiremath@linaro.org>
Cc: Peter Chen <peter.chen@nxp.com>,
gregkh@linuxfoundation.org, stern@rowland.harvard.edu,
ulf.hansson@linaro.org, broonie@kernel.org, sre@kernel.org,
robh+dt@kernel.org, shawnguo@kernel.org, dbaryshkov@gmail.com,
dwmw3@infradead.org, mark.rutland@arm.com,
devicetree@vger.kernel.org, k.kozlowski@samsung.com,
stephen.boyd@linaro.org, oscar@naiandei.net, arnd@arndb.de,
pawel.moll@arm.com, linux-pm@vger.kernel.org,
linux-kernel@vger.kernel.org, s.hauer@pengutronix.de,
linux-usb@vger.kernel.org, mail@maciej.szmigiero.name,
troy.kisky@boundarydevices.com, stillcompiling@gmail.com,
p.zabel@pengutronix.de, festevam@gmail.com, mka@chromium.org,
linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v6 4/8] usb: core: add power sequence handling for USB devices
Date: Fri, 2 Sep 2016 09:30:55 +0800 [thread overview]
Message-ID: <20160902013055.GD11262@shlinux2> (raw)
In-Reply-To: <811390f7-a227-2a1f-986e-68d4668ee0cb@linaro.org>
On Thu, Sep 01, 2016 at 01:32:49PM +0530, Vaibhav Hiremath wrote:
>
>
> On Monday 15 August 2016 02:43 PM, Peter Chen wrote:
> >Some hard-wired USB devices need to do power sequence to let the
> >device work normally, the typical power sequence like: enable USB
> >PHY clock, toggle reset pin, etc. But current Linux USB driver
> >lacks of such code to do it, it may cause some hard-wired USB devices
> >works abnormal or can't be recognized by controller at all.
> >
> >In this patch, it calls power sequence library APIs to finish
> >the power sequence events. At first, it calls pwrseq_alloc_generic
> >to create a generic power sequence instance, then it will do power
> >on sequence at hub's probe for all devices under this hub
> >(includes root hub).
> >
> >At hub_disconnect, it will do power off sequence which is at powered
> >on list.
> >
> >Signed-off-by: Peter Chen <peter.chen@nxp.com>
> >Tested-by Joshua Clayton <stillcompiling@gmail.com>
> >---
> > drivers/usb/core/Makefile | 1 +
> > drivers/usb/core/hub.c | 12 ++++--
> > drivers/usb/core/hub.h | 12 ++++++
> > drivers/usb/core/pwrseq.c | 100 ++++++++++++++++++++++++++++++++++++++++++++++
> > 4 files changed, 122 insertions(+), 3 deletions(-)
> > create mode 100644 drivers/usb/core/pwrseq.c
> >
> >diff --git a/drivers/usb/core/Makefile b/drivers/usb/core/Makefile
> >index 9780877..39f2149 100644
> >--- a/drivers/usb/core/Makefile
> >+++ b/drivers/usb/core/Makefile
> >@@ -9,5 +9,6 @@ usbcore-y += port.o of.o
> > usbcore-$(CONFIG_PCI) += hcd-pci.o
> > usbcore-$(CONFIG_ACPI) += usb-acpi.o
> >+usbcore-$(CONFIG_PWRSEQ_GENERIC) += pwrseq.o
> > obj-$(CONFIG_USB) += usbcore.o
> >diff --git a/drivers/usb/core/hub.c b/drivers/usb/core/hub.c
> >index bee1351..a346a8b 100644
> >--- a/drivers/usb/core/hub.c
> >+++ b/drivers/usb/core/hub.c
> >@@ -1700,6 +1700,7 @@ static void hub_disconnect(struct usb_interface *intf)
> > hub->error = 0;
> > hub_quiesce(hub, HUB_DISCONNECT);
> >+ hub_pwrseq_off(hub);
> > mutex_lock(&usb_port_peer_mutex);
> > /* Avoid races with recursively_mark_NOTATTACHED() */
> >@@ -1733,6 +1734,7 @@ static int hub_probe(struct usb_interface *intf, const struct usb_device_id *id)
> > struct usb_endpoint_descriptor *endpoint;
> > struct usb_device *hdev;
> > struct usb_hub *hub;
> >+ int ret = -ENODEV;
> > desc = intf->cur_altsetting;
> > hdev = interface_to_usbdev(intf);
> >@@ -1839,6 +1841,7 @@ descriptor_error:
> > INIT_DELAYED_WORK(&hub->leds, led_work);
> > INIT_DELAYED_WORK(&hub->init_work, NULL);
> > INIT_WORK(&hub->events, hub_event);
> >+ INIT_LIST_HEAD(&hub->pwrseq_on_list);
> > usb_get_intf(intf);
> > usb_get_dev(hdev);
> >@@ -1852,11 +1855,14 @@ descriptor_error:
> > if (id->driver_info & HUB_QUIRK_CHECK_PORT_AUTOSUSPEND)
> > hub->quirk_check_port_auto_suspend = 1;
> >- if (hub_configure(hub, endpoint) >= 0)
> >- return 0;
> >+ if (hub_configure(hub, endpoint) >= 0) {
> >+ ret = hub_pwrseq_on(hub);
> >+ if (!ret)
> >+ return 0;
> >+ }
> > hub_disconnect(intf);
> >- return -ENODEV;
> >+ return ret;
> > }
> > static int
> >diff --git a/drivers/usb/core/hub.h b/drivers/usb/core/hub.h
> >index 34c1a7e..9473f6f 100644
> >--- a/drivers/usb/core/hub.h
> >+++ b/drivers/usb/core/hub.h
> >@@ -78,6 +78,7 @@ struct usb_hub {
> > struct delayed_work init_work;
> > struct work_struct events;
> > struct usb_port **ports;
> >+ struct list_head pwrseq_on_list; /* powered pwrseq node list */
> > };
> > /**
> >@@ -166,3 +167,14 @@ static inline int hub_port_debounce_be_stable(struct usb_hub *hub,
> > {
> > return hub_port_debounce(hub, port1, false);
> > }
> >+
> >+#if IS_ENABLED(CONFIG_PWRSEQ_GENERIC)
> >+int hub_pwrseq_on(struct usb_hub *hub);
> >+void hub_pwrseq_off(struct usb_hub *hub);
> >+#else
> >+static inline int hub_pwrseq_on(struct usb_hub *hub)
> >+{
> >+ return 0;
> >+}
> >+static inline void hub_pwrseq_off(struct usb_hub *hub) {}
> >+#endif /* CONFIG_PWRSEQ_GENERIC */
> >diff --git a/drivers/usb/core/pwrseq.c b/drivers/usb/core/pwrseq.c
> >new file mode 100644
> >index 0000000..837fe66
> >--- /dev/null
> >+++ b/drivers/usb/core/pwrseq.c
> >@@ -0,0 +1,100 @@
> >+/*
> >+ * pwrseq.c USB device power sequence management
> >+ *
> >+ * Copyright (C) 2016 Freescale Semiconductor, Inc.
> >+ * Author: Peter Chen <peter.chen@nxp.com>
> >+ *
> >+ * This program is free software: you can redistribute it and/or modify
> >+ * it under the terms of the GNU General Public License version 2 of
> >+ * the License as published by the Free Software Foundation.
> >+ *
> >+ * This program is distributed in the hope that it will be useful,
> >+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
> >+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> >+ * GNU General Public License for more details.
> >+ *
> >+ * You should have received a copy of the GNU General Public License
> >+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
> >+ */
> >+
> >+#include <linux/list.h>
> >+#include <linux/of.h>
> >+#include <linux/power/pwrseq.h>
> >+#include <linux/slab.h>
> >+#include <linux/usb.h>
> >+#include <linux/usb/hcd.h>
> >+
> >+#include "hub.h"
> >+
> >+struct usb_pwrseq_node {
> >+ struct pwrseq *pwrseq;
> >+ struct list_head list;
> >+};
> >+
> >+static int hub_of_pwrseq_on(struct device_node *np, struct usb_hub *hub)
> >+{
> >+ struct pwrseq *pwrseq;
> >+ struct usb_pwrseq_node *pwrseq_node;
> >+ int ret;
> >+
> >+ pwrseq = pwrseq_alloc_generic();
> >+ if (IS_ERR(pwrseq))
> >+ return PTR_ERR(pwrseq);
> >+
> >+ ret = pwrseq_get(np, pwrseq);
> >+ if (ret)
> >+ goto pwr_free;
> >+
> >+ ret = pwrseq_on(np, pwrseq);
> >+ if (ret)
> >+ goto pwr_put;
> >+
> >+ pwrseq_node = kzalloc(sizeof(*pwrseq_node), GFP_KERNEL);
>
> You probably want to check the return value here.
>
> I think someone already commented on this, but still for the record
> providing it.
>
Thanks, I will.
--
Best Regards,
Peter Chen
next prev parent reply other threads:[~2016-09-02 1:41 UTC|newest]
Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-08-15 9:13 [PATCH v6 0/8] power: add power sequence library Peter Chen
2016-08-15 9:13 ` [PATCH v6 1/8] binding-doc: power: pwrseq-generic: add binding doc for generic " Peter Chen
2016-09-01 8:03 ` Vaibhav Hiremath
2016-09-02 1:00 ` Peter Chen
2016-09-06 6:04 ` Vaibhav Hiremath
2016-08-15 9:13 ` [PATCH v6 2/8] power: add " Peter Chen
2016-08-22 6:51 ` Peter Chen
2016-08-22 10:23 ` Sebastian Reichel
2016-08-23 1:25 ` Peter Chen
2016-09-01 8:02 ` Vaibhav Hiremath
2016-09-02 1:29 ` Peter Chen
2016-08-15 9:13 ` [PATCH v6 3/8] binding-doc: usb: usb-device: add optional properties for power sequence Peter Chen
2016-08-15 9:13 ` [PATCH v6 4/8] usb: core: add power sequence handling for USB devices Peter Chen
2016-08-22 6:53 ` Peter Chen
2016-08-22 16:09 ` Alan Stern
2016-08-23 3:10 ` Peter Chen
2016-08-23 15:37 ` Alan Stern
2016-09-01 8:02 ` Vaibhav Hiremath
2016-09-02 1:30 ` Peter Chen [this message]
2016-08-15 9:13 ` [PATCH v6 5/8] usb: chipidea: let chipidea core device of_node equal's glue layer device of_node Peter Chen
2016-08-15 9:13 ` [PATCH v6 6/8] ARM: dts: imx6qdl: Enable usb node children with <reg> Peter Chen
2016-08-15 9:13 ` [PATCH v6 7/8] ARM: dts: imx6qdl-udoo.dtsi: fix onboard USB HUB property Peter Chen
2016-08-15 9:13 ` [PATCH v6 8/8] ARM: dts: imx6q-evi: Fix onboard hub reset line Peter Chen
2016-08-23 10:32 ` [PATCH v6 0/8] power: add power sequence library Vaibhav Hiremath
2016-08-24 8:53 ` Peter Chen
2016-08-29 11:10 ` Peter Chen
2016-08-31 8:16 ` Vaibhav Hiremath
2016-08-31 9:52 ` Peter Chen
2016-08-31 16:58 ` Vaibhav Hiremath
2016-09-02 1:10 ` Peter Chen
2016-09-06 10:18 ` Vaibhav Hiremath
2016-09-09 8:47 ` Ulf Hansson
2016-09-19 7:39 ` Vaibhav Hiremath
2016-09-19 7:46 ` Peter Chen
2016-09-19 8:17 ` Vaibhav Hiremath
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=20160902013055.GD11262@shlinux2 \
--to=hzpeterchen@gmail.com \
--cc=arnd@arndb.de \
--cc=broonie@kernel.org \
--cc=dbaryshkov@gmail.com \
--cc=devicetree@vger.kernel.org \
--cc=dwmw3@infradead.org \
--cc=festevam@gmail.com \
--cc=gregkh@linuxfoundation.org \
--cc=k.kozlowski@samsung.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=mail@maciej.szmigiero.name \
--cc=mark.rutland@arm.com \
--cc=mka@chromium.org \
--cc=oscar@naiandei.net \
--cc=p.zabel@pengutronix.de \
--cc=pawel.moll@arm.com \
--cc=peter.chen@nxp.com \
--cc=robh+dt@kernel.org \
--cc=s.hauer@pengutronix.de \
--cc=shawnguo@kernel.org \
--cc=sre@kernel.org \
--cc=stephen.boyd@linaro.org \
--cc=stern@rowland.harvard.edu \
--cc=stillcompiling@gmail.com \
--cc=troy.kisky@boundarydevices.com \
--cc=ulf.hansson@linaro.org \
--cc=vaibhav.hiremath@linaro.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).