[PATCH v12 1/9] binding-doc: power: pwrseq-generic: add binding doc for generic power sequence library

Add binding doc for generic power sequence library. Signed-off-by: Peter Chen <peter.c...@nxp.com> Acked-by: Philipp Zabel <p.za...@pengutronix.de> Acked-by: Rob Herring <r...@kernel.org> --- .../bindings/power/pwrseq/pwrseq-generic.txt | 48 ++

[PATCH v12 6/9] usb: xhci: use bus->sysdev for DMA configuration

From: Arnd Bergmann For xhci-hcd platform device, all the DMA parameters are not configured properly, notably dma ops for dwc3 devices. So, set the dma for xhci from sysdev. sysdev is pointing to device that is known to the system firmware or hardware. Signed-off-by: Arnd

[PATCH v12 7/9] usb: ehci: use bus->sysdev for DMA configuration

Set the dma for ehci from sysdev. The sysdev is pointing to device that is known to the system firmware or hardware. Cc: Arnd Bergmann <a...@arndb.de> Cc: Sriram Dash <sriram.d...@nxp.com> Signed-off-by: Peter Chen <peter.c...@nxp.com> --- drivers/usb/ho

[PATCH v12 3/9] usb: separate out sysdev pointer from usb_bus

m> Cc: Stephen Boyd <sb...@codeaurora.org> Cc: Bjorn Andersson <bjorn.anders...@linaro.org> Cc: Ming Lei <tom.leim...@gmail.com> Cc: Jon Masters <j...@redhat.com> Cc: Dann Frazier <dann.fraz...@canonical.com> Cc: Peter Chen

[PATCH v12 8/9] binding-doc: usb: usb-device: add optional properties for power sequence

Add optional properties for power sequence. Signed-off-by: Peter Chen <peter.c...@nxp.com> Acked-by: Rob Herring <r...@kernel.org> --- Documentation/devicetree/bindings/usb/usb-device.txt | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/Documentatio

[PATCH v12 9/9] usb: core: add power sequence handling for USB devices

-off-by: Peter Chen <peter.c...@nxp.com> Tested-by Joshua Clayton <stillcompil...@gmail.com> Tested-by: Maciej S. Szmigiero <m...@maciej.szmigiero.name> Reviewed-by: Vaibhav Hiremath <hvaibhav.li...@gmail.com> --- drivers/usb/Kconfig| 1 + dri

[PATCH v12 4/9] usb: chipidea: use bus->sysdev for DMA configuration

From: Arnd Bergmann <a...@arndb.de> Set the dma for chipidea from sysdev. This is inherited from its parent node. Also, do not set dma mask for child as it is not required now. Signed-off-by: Arnd Bergmann <a...@arndb.de> Signed-off-by: Sriram Dash <sriram.d...@nxp.com> Ac

[PATCH v12 2/9] power: add power sequence library

instead (eg, USB hub driver). For new power sequence library, it can add its compatible string to pwrseq_of_match_table, then the pwrseq core will match it with DT's, and choose this library at runtime. Signed-off-by: Peter Chen <peter.c...@nxp.com> Tested-by: Maciej S. Szmigi

[PATCH v12 0/9] power: add power sequence library

usb: chipidea: use bus->sysdev for DMA configuration usb: ehci: fsl: use bus->sysdev for DMA configuration usb: xhci: use bus->sysdev for DMA configuration Peter Chen (5): binding-doc: power: pwrseq-generic: add binding doc for generic power sequence library power: add power

[GIT PULL] usb: chipidea changes for v4.11-rc1

): usb: chipdata: Replace the extcon API Jisheng Zhang (2): usb: chipidea: delete an useless header include usb: chipidea: usb2: delete the redundant setting default DMA mask code Peter Chen (1): usb: chipidea: vbus event may exist before starting gadget Stephen Boyd (22

Re: [PATCH 2/8] power: add power sequence library

On Fri, Feb 03, 2017 at 10:14:31PM +0100, Rafael J. Wysocki wrote: > On Friday, February 03, 2017 04:16:15 PM Peter Chen wrote: > > On Wed, Feb 01, 2017 at 09:08:17AM +0100, Greg Kroah-Hartman wrote: > > > On Wed, Feb 01, 2017 at 12:10:17AM +0100, Rafael J. Wysocki wrote: >

Re: [PATCH v8 0/3] Support qcom's HSIC USB and rewrite USB2 HS support

t; + > > drivers/phy/phy-qcom-usb-hsic.c | 160 +++ > > drivers/usb/chipidea/core.c| 20 +- > > 7 files changed, 636 insertions(+), 6 deletions(-) > > create mode 100644 > > Documentation/devicetree/bindings/phy/qcom,usb-hs-phy.txt > > create mode 100644 > > Documentation/devicetree/bindings/phy/qcom,usb-hsic-phy.txt > > create mode 100644 drivers/phy/phy-qcom-usb-hs.c > > create mode 100644 drivers/phy/phy-qcom-usb-hsic.c > > > > ___ > linux-arm-kernel mailing list > linux-arm-ker...@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- Best Regards, Peter Chen -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH 2/8] power: add power sequence library

On Wed, Feb 01, 2017 at 09:08:17AM +0100, Greg Kroah-Hartman wrote: > On Wed, Feb 01, 2017 at 12:10:17AM +0100, Rafael J. Wysocki wrote: > > On Tue, Jan 3, 2017 at 7:33 AM, Peter Chen <peter.c...@nxp.com> wrote: > > > We have an well-known problem that the devic

Re: [RESEND PATCH] usb: chipidea: usb2: delete the redundant setting default DMA mask code

} > } > > - ret = dma_set_mask_and_coherent(dev, DMA_BIT_MASK(32)); > - if (ret) > - goto clk_err; > - > ci_pdata->name = dev_name(dev); > > priv->ci_pdev = ci_hdrc_add_device(dev, pdev->resource, Applied,

Re: [RESEND PATCH] usb: chipidea: delete an useless header include

/usb/chipidea/core.c b/drivers/usb/chipidea/core.c > index 3dbb4a21ab44..77078083e9fb 100644 > --- a/drivers/usb/chipidea/core.c > +++ b/drivers/usb/chipidea/core.c > @@ -62,7 +62,6 @@ > #include > #include > #include > -#include > #include > #include > Ap

Re: [PATCH v11 2/8] power: add power sequence library

On Fri, Jan 20, 2017 at 11:21:27AM +0100, Rafael J. Wysocki wrote: > On Fri, Jan 20, 2017 at 8:52 AM, Peter Chen <hzpeterc...@gmail.com> wrote: > > On Tue, Jan 10, 2017 at 03:02:41PM +0800, Peter Chen wrote: > >> On Sat, Jan 07, 2017 at 10:54:56AM +0200, Krzysztof Kozlow

Re: [PATCH v7 2/5] usb: chipidea: msm: Configure phy for appropriate mode

phy_set_mode(ci->phy, PHY_MODE_USB_HOST); > } > break; > case CI_HDRC_CONTROLLER_STOPPED_EVENT: > -- > 2.10.0.297.gf6727b0 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-usb" in >

RE: [PATCH v7 4/5] usb: chipidea: Signal vbus state to phy

> > drivers/usb/chipidea/otg.c | 7 +-- > 1 file changed, 5 insertions(+), 2 deletions(-) > >diff --git a/drivers/usb/chipidea/otg.c b/drivers/usb/chipidea/otg.c index >10236fe71522..6ea702beed48 100644 >--- a/drivers/usb/chipidea/otg.c >+++ b/drivers/usb/chipidea/otg.c >@@ -134,10 +134,13 @@

RE: [PATCH v7 3/5] phy: Add set_vbus callback

> * @set_mode: set the mode of the phy >+ * @set_vbus: enable/disable vbus in the phy (USB) > * @reset: resetting the phy > * @owner: the module owner containing the ops > */ >@@ -45,6 +46,7 @@ struct phy_ops { > int (*power_on)(struct phy *phy); > int (*power_off)(struct

Re: [PATCH v11 2/8] power: add power sequence library

On Tue, Jan 10, 2017 at 03:02:41PM +0800, Peter Chen wrote: > On Sat, Jan 07, 2017 at 10:54:56AM +0200, Krzysztof Kozlowski wrote: > > On Thu, Jan 05, 2017 at 02:01:53PM +0800, Peter Chen wrote: > > > We have an well-known problem that the device needs to do some power > >

Re: [PATCH 2/2] usb: gadget: uac2: add req_number as parameter

On Tue, Jan 17, 2017 at 05:57:50PM +0800, Peter Chen wrote: > On Tue, Jan 17, 2017 at 11:23:55AM +0200, Felipe Balbi wrote: > > > > Hi, > > > > Peter Chen <hzpeterc...@gmail.com> writes: > > > On Mon, Jan 16, 2017 at 12:40:06PM +0200, Felipe Balbi wrot

Re: [PATCH v6 00/25] Support qcom's HSIC USB and rewrite USB2 HS support

sequence before the HSIC controller enumerates or HSIC doesn't work. >Right now I have a hack to force the controller to probe defer >once so that usb4604 probes first. This needs a more proper solution >like having the DT describe a linkage between the controller and >the

Re: [PATCH v6 03/25] usb: ulpi: Support device discovery via DT

bus; > ulpi->dev.type = _dev_type; > dev_set_name(>dev, "%s.ulpi", dev_name(dev)); > > ACPI_COMPANION_SET(>dev, ACPI_COMPANION(dev)); > > - request_module("ulpi:v%04xp%04x", ulpi->id.vendor, ulpi->id.product); > + ret = ulpi_of_register(ulpi); > + if (ret) > + return ret; > + > + ret = ulpi_read_id(ulpi); > + if (ret) > + return ret; > > ret = device_register(>dev); > if (ret) > @@ -234,6 +300,7 @@ EXPORT_SYMBOL_GPL(ulpi_register_interface); > */ > void ulpi_unregister_interface(struct ulpi *ulpi) > { > + of_node_put(ulpi->dev.of_node); > device_unregister(>dev); > } > EXPORT_SYMBOL_GPL(ulpi_unregister_interface); > -- > 2.10.0.297.gf6727b0 > > > ___ > linux-arm-kernel mailing list > linux-arm-ker...@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- Best Regards, Peter Chen -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH v6 23/25] usb: chipidea: Pullup D+ in device mode via phy APIs

On Wed, Jan 18, 2017 at 02:57:27PM -0600, Rob Herring wrote: > On Wed, Jan 18, 2017 at 2:54 PM, Stephen Boyd <stephen.b...@linaro.org> wrote: > > Quoting Peter Chen (2017-01-17 23:34:32) > >> On Tue, Jan 17, 2017 at 09:58:33AM -0800, Stephen Boyd wrote: > >> >

Re: [PATCH v6 23/25] usb: chipidea: Pullup D+ in device mode via phy APIs

On Tue, Jan 17, 2017 at 09:58:33AM -0800, Stephen Boyd wrote: > Quoting Peter Chen (2017-01-15 19:45:51) > > > > So, you need to call phy_set_mode when switching between host and device. > > Besides, you also need to toggle VBUSVLDEXT when the external vbus > >

Re: [PATCH 2/2] usb: gadget: uac2: add req_number as parameter

On Tue, Jan 17, 2017 at 11:23:55AM +0200, Felipe Balbi wrote: > > Hi, > > Peter Chen <hzpeterc...@gmail.com> writes: > > On Mon, Jan 16, 2017 at 12:40:06PM +0200, Felipe Balbi wrote: > >> > >> Hi, > >> > >> Peter Chen <peter.c.

Re: [PATCH 2/2] usb: gadget: uac2: add req_number as parameter

On Mon, Jan 16, 2017 at 12:40:06PM +0200, Felipe Balbi wrote: > > Hi, > > Peter Chen <peter.c...@nxp.com> writes: > > There are only two requests for uac2, it may not be enough at high > > loading system which usb interrupt handler can't be serviced on > > ti

Re: [PATCH 2/2] usb: gadget: uac2: add req_number as parameter

On Wed, Jan 04, 2017 at 10:19:23AM +0800, Peter Chen wrote: > There are only two requests for uac2, it may not be enough at high > loading system which usb interrupt handler can't be serviced on > time, then the data will be lost since it is isoc transfer for audio. > > In

Re: [PATCH 1/2] usb: gadget: f_uac2: improve error handling

On Wed, Jan 04, 2017 at 10:19:22AM +0800, Peter Chen wrote: > If it is out of memory, we should return -ENOMEM; > > Signed-off-by: Peter Chen <peter.c...@nxp.com> > --- > drivers/usb/gadget/function/f_uac2.c | 10 ++ > 1 file changed, 6 insertions(+), 4 dele

Re: [PATCH v6 23/25] usb: chipidea: Pullup D+ in device mode via phy APIs

On Fri, Jan 13, 2017 at 12:03:00PM -0800, Stephen Boyd wrote: > Quoting Peter Chen (2017-01-12 19:35:36) > > On Thu, Jan 12, 2017 at 02:49:51PM -0800, Stephen Boyd wrote: > > > > > > With the boards I have, vbus is not routed to the phy. Instead, there's > > >

Re: [PATCH v6 23/25] usb: chipidea: Pullup D+ in device mode via phy APIs

On Thu, Jan 12, 2017 at 02:49:51PM -0800, Stephen Boyd wrote: > Quoting Peter Chen (2017-01-12 01:50:40) > > On Wed, Jan 11, 2017 at 04:19:53PM -0800, Stephen Boyd wrote: > > > Quoting Peter Chen (2017-01-02 22:53:19) > > > > On Wed, Dec 28, 2016 at 02:57:

Re: [PATCH v6 11/25] usb: chipidea: vbus event may exist before starting gadget

On Wed, Jan 11, 2017 at 04:32:34PM -0800, Stephen Boyd wrote: > Quoting Peter Chen (2017-01-03 00:00:31) > > On Wed, Dec 28, 2016 at 02:56:57PM -0800, Stephen Boyd wrote: > > > From: Peter Chen <peter.c...@nxp.com> > > > > > > At some situations, the

Re: [PATCH v6 23/25] usb: chipidea: Pullup D+ in device mode via phy APIs

On Wed, Jan 11, 2017 at 04:19:53PM -0800, Stephen Boyd wrote: > Quoting Peter Chen (2017-01-02 22:53:19) > > On Wed, Dec 28, 2016 at 02:57:09PM -0800, Stephen Boyd wrote: > > > If the phy supports it, call phy_set_mode() to pull up D+ when > > > required by setting the

Re: [PATCH v11 2/8] power: add power sequence library

On Sat, Jan 07, 2017 at 10:54:56AM +0200, Krzysztof Kozlowski wrote: > On Thu, Jan 05, 2017 at 02:01:53PM +0800, Peter Chen wrote: > > We have an well-known problem that the device needs to do some power > > sequence before it can be recognized by related host, the typical > &

[PATCH v11 1/8] binding-doc: power: pwrseq-generic: add binding doc for generic power sequence library

Add binding doc for generic power sequence library. Signed-off-by: Peter Chen <peter.c...@nxp.com> Acked-by: Philipp Zabel <p.za...@pengutronix.de> Acked-by: Rob Herring <r...@kernel.org> --- .../bindings/power/pwrseq/pwrseq-generic.txt | 48 ++

[PATCH v11 0/8] power: add power sequence library

ce chipidea core has no of_node, let core's of_node equals glue layer's at core's probe Joshua Clayton (2): ARM: dts: imx6qdl: Enable usb node children with ARM: dts: imx6q-evi: Fix onboard hub reset line Peter Chen (6): binding-doc: power: pwrseq-generic: add binding doc for generic power

[PATCH v11 7/8] ARM: dts: imx6qdl-udoo.dtsi: fix onboard USB HUB property

The current dts describes USB HUB's property at USB controller's entry, it is improper. The USB HUB should be the child node under USB controller, and power sequence properties are under it. Besides, using gpio pinctrl setting for USB2415's reset pin. Signed-off-by: Peter Chen <peter.c...@nxp.

[PATCH v11 6/8] ARM: dts: imx6qdl: Enable usb node children with

From: Joshua Clayton <stillcompil...@gmail.com> Give usb nodes #address and #size attributes, so that a child node representing a permanently connected device such as an onboard hub may be addressed with a attribute Signed-off-by: Joshua Clayton <stillcompil...@gmail.com> Signed-o

[PATCH v11 5/8] usb: chipidea: let chipidea core device of_node equal's glue layer device of_node

From: Peter Chen <peter.c...@freescale.com> At device tree, we have no device node for chipidea core, the glue layer's node is the parent node for host and udc device. But in related driver, the parent device is chipidea core. So, in order to let the common driver get parent's node,

[PATCH v11 3/8] binding-doc: usb: usb-device: add optional properties for power sequence

Add optional properties for power sequence. Signed-off-by: Peter Chen <peter.c...@nxp.com> Acked-by: Rob Herring <r...@kernel.org> --- Documentation/devicetree/bindings/usb/usb-device.txt | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/Documentatio

[PATCH v11 8/8] ARM: dts: imx6q-evi: Fix onboard hub reset line

layton <stillcompil...@gmail.com> Signed-off-by: Peter Chen <peter.c...@nxp.com> --- arch/arm/boot/dts/imx6q-evi.dts | 25 +++-- 1 file changed, 7 insertions(+), 18 deletions(-) diff --git a/arch/arm/boot/dts/imx6q-evi.dts b/arch/arm/boot/dts/imx6q-evi.dts index 7c7c1

[PATCH v11 4/8] usb: core: add power sequence handling for USB devices

-off-by: Peter Chen <peter.c...@nxp.com> Tested-by Joshua Clayton <stillcompil...@gmail.com> Tested-by: Maciej S. Szmigiero <m...@maciej.szmigiero.name> Reviewed-by: Vaibhav Hiremath <hvaibhav.li...@gmail.com> --- drivers/usb/Kconfig| 1 + dri

[PATCH v11 2/8] power: add power sequence library

instead (eg, USB hub driver). For new power sequence library, it can add its compatible string to pwrseq_of_match_table, then the pwrseq core will match it with DT's, and choose this library at runtime. Signed-off-by: Peter Chen <peter.c...@nxp.com> Tested-by: Maciej S. Szmigi

[PATCH 2/2] usb: gadget: uac2: add req_number as parameter

override it if current number for request is not enough for his/her use case. Besides, update this parameter for legacy audio gadget and documentation. Signed-off-by: Peter Chen <peter.c...@nxp.com> --- Documentation/usb/gadget-testing.txt | 2 ++ drivers/usb/gadget/function/f_uac2.

[PATCH 1/2] usb: gadget: f_uac2: improve error handling

If it is out of memory, we should return -ENOMEM; Signed-off-by: Peter Chen <peter.c...@nxp.com> --- drivers/usb/gadget/function/f_uac2.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/drivers/usb/gadget/function/f_uac2.c b/drivers/usb/gadget/function/f_

[PATCH 1/1] net: usb: asix_devices: add .reset_resume for USB PM

this commit. Signed-off-by: Peter Chen <peter.c...@nxp.com> --- drivers/net/usb/asix_devices.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/net/usb/asix_devices.c b/drivers/net/usb/asix_devices.c index 6c646e2..6e98ede 100644 --- a/drivers/net/usb/asix_devices.c +++ b/drivers/n

Re: [PATCH v6 11/25] usb: chipidea: vbus event may exist before starting gadget

On Wed, Dec 28, 2016 at 02:56:57PM -0800, Stephen Boyd wrote: > From: Peter Chen <peter.c...@nxp.com> > > At some situations, the vbus may already be there before starting > gadget. So we need to check vbus event after switch to gadget in > order to handle missing vbus e

Re: [PATCH v6 23/25] usb: chipidea: Pullup D+ in device mode via phy APIs

On Wed, Dec 28, 2016 at 02:57:09PM -0800, Stephen Boyd wrote: > If the phy supports it, call phy_set_mode() to pull up D+ when > required by setting the mode to PHY_MODE_USB_DEVICE. If we want > to remove the pullup, set the mode to PHY_MODE_USB_HOST. > > Cc: Peter Chen <peter.

Re: [PATCH v6 22/25] usb: chipidea: Drop lock across event_notify during gadget stop

On Wed, Dec 28, 2016 at 02:57:08PM -0800, Stephen Boyd wrote: > The CI_HDRC_CONTROLLER_STOPPED_EVENT may want to call sleeping > APIs similar to how _gadget_stop_activity() may. Let's drop the > lock across the event so that glue drivers can make sleeping > calls. > > Cc: P

[PATCH 2/8] power: add power sequence library

instead (eg, USB hub driver). For new power sequence library, it can add its compatible string to pwrseq_of_match_table, then the pwrseq core will match it with DT's, and choose this library at runtime. Signed-off-by: Peter Chen <peter.c...@nxp.com> Tested-by: Maciej S. Szmigi

[PATCH 5/8] usb: chipidea: let chipidea core device of_node equal's glue layer device of_node

From: Peter Chen <peter.c...@freescale.com> At device tree, we have no device node for chipidea core, the glue layer's node is the parent node for host and udc device. But in related driver, the parent device is chipidea core. So, in order to let the common driver get parent's node,

[PATCH 8/8] ARM: dts: imx6q-evi: Fix onboard hub reset line

layton <stillcompil...@gmail.com> Signed-off-by: Peter Chen <peter.c...@nxp.com> --- arch/arm/boot/dts/imx6q-evi.dts | 25 +++-- 1 file changed, 7 insertions(+), 18 deletions(-) diff --git a/arch/arm/boot/dts/imx6q-evi.dts b/arch/arm/boot/dts/imx6q-evi.dts index 7c7c1

[PATCH 4/8] usb: core: add power sequence handling for USB devices

-off-by: Peter Chen <peter.c...@nxp.com> Tested-by Joshua Clayton <stillcompil...@gmail.com> Tested-by: Maciej S. Szmigiero <m...@maciej.szmigiero.name> Reviewed-by: Vaibhav Hiremath <hvaibhav.li...@gmail.com> --- drivers/usb/Kconfig| 1 + dri

[PATCH 7/8] ARM: dts: imx6qdl-udoo.dtsi: fix onboard USB HUB property

The current dts describes USB HUB's property at USB controller's entry, it is improper. The USB HUB should be the child node under USB controller, and power sequence properties are under it. Besides, using gpio pinctrl setting for USB2415's reset pin. Signed-off-by: Peter Chen <peter.c...@nxp.

[PATCH 6/8] ARM: dts: imx6qdl: Enable usb node children with

From: Joshua Clayton <stillcompil...@gmail.com> Give usb nodes #address and #size attributes, so that a child node representing a permanently connected device such as an onboard hub may be addressed with a attribute Signed-off-by: Joshua Clayton <stillcompil...@gmail.com> Signed-o

[PATCH 3/8] binding-doc: usb: usb-device: add optional properties for power sequence

Add optional properties for power sequence. Signed-off-by: Peter Chen <peter.c...@nxp.com> Acked-by: Rob Herring <r...@kernel.org> --- Documentation/devicetree/bindings/usb/usb-device.txt | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/Documentatio

[PATCH 0/8] power: add power sequence library

ce chipidea core has no of_node, let core's of_node equals glue layer's at core's probe Joshua Clayton (2): ARM: dts: imx6qdl: Enable usb node children with ARM: dts: imx6q-evi: Fix onboard hub reset line Peter Chen (6): binding-doc: power: pwrseq-generic: add binding doc for generic power

[PATCH 1/8] binding-doc: power: pwrseq-generic: add binding doc for generic power sequence library

Add binding doc for generic power sequence library. Signed-off-by: Peter Chen <peter.c...@nxp.com> Acked-by: Philipp Zabel <p.za...@pengutronix.de> Acked-by: Rob Herring <r...@kernel.org> --- .../bindings/power/pwrseq/pwrseq-generic.txt | 48 ++

Re: [PATCH v10 0/8] power: add power sequence library

On Mon, Dec 19, 2016 at 09:15:04PM +0200, Krzysztof Kozlowski wrote: > On Mon, Nov 14, 2016 at 09:35:51AM +0800, Peter Chen wrote: > > Hi all, > > > > This is a follow-up for my last power sequence framework patch set [1]. > > According to Rob Herring and Ulf Hanss

Re: RE: [PATCH] usb: chipidea: use better pattern with WARN_ON()

ave been instructed by greg k-h to first try to submit > patches in staging drivers. So I am trying to submit some patches there. I > had sent you the patch to review before greg asked me to do so. > Please wrap up 75 characters. > Maybe the patch I sent could be base6

Re: [PATCH 05/12] usb: chipdata: Replace the extcon API

FINE_IDA(ci_ida); > > struct platform_device *ci_hdrc_add_device(struct device *dev, > @@ -1053,7 +1037,6 @@ static int ci_hdrc_probe(struct platform_device *pdev) > if (!ret) > return 0; > > - ci_extcon_unregister(ci); > stop: > ci_role_d

Re: [PATCH v10 2/8] power: add power sequence library

On Thu, Dec 01, 2016 at 10:57:24PM +0100, Rafael J. Wysocki wrote: > On Tue, Nov 22, 2016 at 4:53 AM, Peter Chen <hzpeterc...@gmail.com> wrote: > > On Tue, Nov 22, 2016 at 03:23:12AM +0100, Rafael J. Wysocki wrote: > >> > @@ -0,0 +1,237 @@ > >> > +/* > >

[GIT PULL] usb: chipidea changes for v4.10-rc1

deletions(-) -- Best Regards, Peter Chen -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH v10 2/8] power: add power sequence library

On Tue, Nov 22, 2016 at 03:23:12AM +0100, Rafael J. Wysocki wrote: > > @@ -0,0 +1,237 @@ > > +/* > > + * core.c power sequence core file > > + * > > + * Copyright (C) 2016 Freescale Semiconductor, Inc. > > + * Author: Peter Chen <peter.c...@nxp.co

Re: [PATCH v10 2/8] power: add power sequence library

On Mon, Nov 14, 2016 at 09:35:53AM +0800, Peter Chen wrote: > We have an well-known problem that the device needs to do some power > sequence before it can be recognized by related host, the typical > example like hard-wired mmc devices and usb devices. > > This power s

Re: [PATCH 2/2] usb: gadget: serial: fix possible Oops caused by calling kthread_stop(NULL)

ference problem after enable g_serial console, and the issue is fixed after applying your patch. So, Tested-by: Peter Chen <peter.c...@nxp.com> However, but there is no console enabled information, but console disabled information is existed like below, I am wondering if it is a problem.

Re: [GIT PULL] USB fixes for v4.9-rc4

On Tue, Nov 15, 2016 at 10:54:09PM -0500, David Miller wrote: > From: Peter Chen <hzpeterc...@gmail.com> > Date: Wed, 16 Nov 2016 11:41:15 +0800 > > > I just notice that you submitted the "usb: gadget: u_ether: remove > > interrupt throttling", and

Re: [GIT PULL] USB fixes for v4.9-rc4

s problem, you may add this to -next tree first, and test one or two revisions, this feature itself is not a bug in all, just some controllers may can't use it. [1] https://www.spinics.net/lists/linux-usb/msg148762.html -- Best Regards, Peter Chen -- To unsubscribe from this list: send t

Re: PROBLEM: Oops when deactivating gadget serial driver

On Wed, Nov 16, 2016 at 12:15:34AM +0100, Felix Hädicke wrote: > Hello, > > In 4.8.6 and 4.9-rc5, the gadget serial driver crashes during > deinitialisation when compiled with CONFIG_U_SERIAL_CONSOLE. > > Steps to reproduce: > modprobe g-serial > rmmod g-serial > > The problem also occurs when

[PATCH v3 1/1] usb: chipidea: move the lock initialization to core file

("USB: Chipidea: rename struct ci13xxx variables from udc to ci") Signed-off-by: Peter Chen <peter.c...@nxp.com> --- Changes for v3: - Delete the wrong kernel version for stable tree, and using real "Fixes" instead. Changes for v2: - Clean up dump messa

Re: [PATCH v2 1/1] usb: chipidea: move the lock initialization to core file

On Tue, Nov 15, 2016 at 11:32:02AM +0200, Felipe Balbi wrote: > > Hi, > > Peter Chen <hzpeterc...@gmail.com> writes: > > On Tue, Nov 15, 2016 at 07:58:16AM +0100, Greg KH wrote: > >> On Tue, Nov 15, 2016 at 02:02:47PM +0800, Peter Chen wrote: > >&g

Re: [PATCH v2 1/1] usb: chipidea: move the lock initialization to core file

On Tue, Nov 15, 2016 at 07:58:16AM +0100, Greg KH wrote: > On Tue, Nov 15, 2016 at 02:02:47PM +0800, Peter Chen wrote: > > This can fix below dump when the lock is accessed at host > > mode due to it is not initialized. > > > > [ 46.119638] INFO: trying

Re: [GIT PULL] USB: chipidea fixes for v4.9

On Tue, Nov 15, 2016 at 07:59:51AM +0100, Greg KH wrote: > On Tue, Nov 15, 2016 at 02:05:49PM +0800, Peter Chen wrote: > > On Mon, Nov 14, 2016 at 12:48:45PM +0100, Greg KH wrote: > > > On Mon, Nov 14, 2016 at 10:01:53AM +0800, Peter Chen wrote: > > > > The

[PATCH v2 1/1] usb: chipidea: move the lock initialization to core file

:b6de3000 [ 46.382667] r5:ee1c6a00 [ 46.385222] r4:ee1c6a00 [ 46.387817] [] (SyS_read) from [] (ret_fast_syscall+0x0/0x1c) [ 46.395314] r7:0003 [ 46.397687] r6:b6de3000 [ 46.400243] r5:0002 [ 46.401310] r4:0002 Cc: <sta...@vger.kernel.org> #v4.1+ Signed-off-by:

Re: [GIT PULL] USB: chipidea fixes for v4.9

On Mon, Nov 14, 2016 at 12:48:45PM +0100, Greg KH wrote: > On Mon, Nov 14, 2016 at 10:01:53AM +0800, Peter Chen wrote: > > The following changes since commit 18266403f3fe507f0246faa1d5432333a2f139ca: > > > > USB: cdc-acm: fix TIOCMIWAIT (2016-11-10 13:12:59 +0100) &

Re: [PATCH v18 0/4] Introduce usb charger framework to deal with the usb gadget power negotation

trying to keep these two notifications at an new framework, and power client gets refined notification from this new framework. The biggest problem I concern about your solution is extcon device, it may not be an universal solution, does current frameworks have a way to get cable type (usb charger type)

[GIT PULL] USB: chipidea fixes for v4.9

Peter Chen (1): usb: chipidea: move the lock initialization to core file drivers/usb/chipidea/core.c | 1 + drivers/usb/chipidea/udc.c | 2 -- 2 files changed, 1 insertion(+), 2 deletions(-) -- Best Regards, Peter Chen -- To unsubscribe from this list: send the line

[PATCH v10 3/8] binding-doc: usb: usb-device: add optional properties for power sequence

Add optional properties for power sequence. Signed-off-by: Peter Chen <peter.c...@nxp.com> Acked-by: Rob Herring <r...@kernel.org> --- Documentation/devicetree/bindings/usb/usb-device.txt | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/Documentatio

[PATCH v10 6/8] ARM: dts: imx6qdl: Enable usb node children with

From: Joshua Clayton <stillcompil...@gmail.com> Give usb nodes #address and #size attributes, so that a child node representing a permanently connected device such as an onboard hub may be addressed with a attribute Signed-off-by: Joshua Clayton <stillcompil...@gmail.com> Signed-o

Re: [PATCH v3 5/6] usb: dwc3: use bus->sysdev for DMA configuration

r otherwise), and the phy lookup in has to be done > through sysdev as well because the platform cannot add it to > the child device it has no knowledge of. > > When we set the sysdev to the parent, the phy lookup has to be > added to that device as well, rather than the child device. >

[PATCH v10 7/8] ARM: dts: imx6qdl-udoo.dtsi: fix onboard USB HUB property

The current dts describes USB HUB's property at USB controller's entry, it is improper. The USB HUB should be the child node under USB controller, and power sequence properties are under it. Besides, using gpio pinctrl setting for USB2415's reset pin. Signed-off-by: Peter Chen <peter.c...@nxp.

[PATCH v10 8/8] ARM: dts: imx6q-evi: Fix onboard hub reset line

layton <stillcompil...@gmail.com> Signed-off-by: Peter Chen <peter.c...@nxp.com> --- arch/arm/boot/dts/imx6q-evi.dts | 25 +++-- 1 file changed, 7 insertions(+), 18 deletions(-) diff --git a/arch/arm/boot/dts/imx6q-evi.dts b/arch/arm/boot/dts/imx6q-evi.dts index 6de21

[PATCH v10 5/8] usb: chipidea: let chipidea core device of_node equal's glue layer device of_node

From: Peter Chen <peter.c...@freescale.com> At device tree, we have no device node for chipidea core, the glue layer's node is the parent node for host and udc device. But in related driver, the parent device is chipidea core. So, in order to let the common driver get parent's node,

[PATCH v10 4/8] usb: core: add power sequence handling for USB devices

-off-by: Peter Chen <peter.c...@nxp.com> Tested-by Joshua Clayton <stillcompil...@gmail.com> Tested-by: Maciej S. Szmigiero <m...@maciej.szmigiero.name> Reviewed-by: Vaibhav Hiremath <hvaibhav.li...@gmail.com> --- drivers/usb/Kconfig| 1 + dri

[PATCH v10 0/8] power: add power sequence library

ce chipidea core has no of_node, let core's of_node equals glue layer's at core's probe Joshua Clayton (2): ARM: dts: imx6qdl: Enable usb node children with ARM: dts: imx6q-evi: Fix onboard hub reset line Peter Chen (6): binding-doc: power: pwrseq-generic: add binding doc for generic power

[PATCH v10 2/8] power: add power sequence library

instead (eg, USB hub driver). For new power sequence library, it can add its compatible string to pwrseq_of_match_table, then the pwrseq core will match it with DT's, and choose this library at runtime. Signed-off-by: Peter Chen <peter.c...@nxp.com> Tested-by: Maciej S. Szmigi

[PATCH v10 1/8] binding-doc: power: pwrseq-generic: add binding doc for generic power sequence library

Add binding doc for generic power sequence library. Signed-off-by: Peter Chen <peter.c...@nxp.com> Acked-by: Philipp Zabel <p.za...@pengutronix.de> Acked-by: Rob Herring <r...@kernel.org> --- .../bindings/power/pwrseq/pwrseq-generic.txt | 48 ++

Re: [PATCH v5 00/23] Support qcom's HSIC USB and rewrite USB2 HS support

On Tue, Oct 18, 2016 at 01:51:39PM -0700, Stephen Boyd wrote: > Quoting Peter Chen (2016-10-18 02:31:40) > > On Mon, Oct 17, 2016 at 06:56:13PM -0700, Stephen Boyd wrote: > > > I've also sent separate patches for other minor pieces to make this > > > all work. The fu

Re: [PATCH v9 2/8] power: add power sequence library

On Fri, Nov 11, 2016 at 02:05:01AM +0100, Rafael J. Wysocki wrote: > On Tue, Nov 8, 2016 at 3:51 AM, Peter Chen <peter.c...@nxp.com> wrote: > > We have an well-known problem that the device needs to do some power > > sequence before it can be recognized by related host, th

Re: [PATCH v4] phy: rcar-gen3-usb2: add sysfs for usb role swap

eck_id(ch); > + is_host = rcar_gen3_is_host(ch); > + if (!strncmp(buf, "host", strlen("host"))) > + new_mode_is_host = true; > + else if (!strncmp(buf, "peripheral", strlen("peripheral"))) > + new_mode_is_host = false; > + else

Re: [PATCH v18 0/4] Introduce usb charger framework to deal with the usb gadget power negotation

On Wed, Nov 09, 2016 at 07:38:36AM +1100, NeilBrown wrote: > On Tue, Nov 08 2016, Peter Chen wrote: > > > On Thu, Nov 03, 2016 at 12:25:42PM +1100, NeilBrown wrote: > >> > >> > >> > >> 2/ Change all usb phys to register an extcon and to send app

Re: [PATCH v3] phy: rcar-gen3-usb2: add sysfs for usb role swap

rent and new mode is the same, this returns the error */ > + if (is_host == new_mode_is_host) > + return -EINVAL; > + > + if (new_mode_is_host) { > + if (!is_b_device && !is_host) /* A-Peripheral */ > + rcar_gen3_init_from_a_peri_to_a_host(ch); > + if (

Re: [PATCH v18 0/4] Introduce usb charger framework to deal with the usb gadget power negotation

tery charger registers to >know what sort of cable was connected (which I believe is the case >for the chips you are interested in), then it should do that. Not only USB PHY to register an extcon, but also for the drivers which can detect USB charger type, it may be USB controller drive

[PATCH v9 5/8] usb: chipidea: let chipidea core device of_node equal's glue layer device of_node

From: Peter Chen <peter.c...@freescale.com> At device tree, we have no device node for chipidea core, the glue layer's node is the parent node for host and udc device. But in related driver, the parent device is chipidea core. So, in order to let the common driver get parent's node,

[PATCH v9 3/8] binding-doc: usb: usb-device: add optional properties for power sequence

Add optional properties for power sequence. Signed-off-by: Peter Chen <peter.c...@nxp.com> Acked-by: Rob Herring <r...@kernel.org> --- Documentation/devicetree/bindings/usb/usb-device.txt | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/Documentatio

[PATCH v9 7/8] ARM: dts: imx6qdl-udoo.dtsi: fix onboard USB HUB property

The current dts describes USB HUB's property at USB controller's entry, it is improper. The USB HUB should be the child node under USB controller, and power sequence properties are under it. Besides, using gpio pinctrl setting for USB2415's reset pin. Signed-off-by: Peter Chen <peter.c...@nxp.

[PATCH v9 6/8] ARM: dts: imx6qdl: Enable usb node children with

From: Joshua Clayton <stillcompil...@gmail.com> Give usb nodes #address and #size attributes, so that a child node representing a permanently connected device such as an onboard hub may be addressed with a attribute Signed-off-by: Joshua Clayton <stillcompil...@gmail.com> Signed-o

[PATCH v9 2/8] power: add power sequence library

instead (eg, USB hub driver). For new power sequence library, it can add its compatible string to pwrseq_of_match_table, then the pwrseq core will match it with DT's, and choose this library at runtime. Signed-off-by: Peter Chen <peter.c...@nxp.com> Tested-by: Maciej S. Szmigi

[PATCH v9 8/8] ARM: dts: imx6q-evi: Fix onboard hub reset line

layton <stillcompil...@gmail.com> Signed-off-by: Peter Chen <peter.c...@nxp.com> --- arch/arm/boot/dts/imx6q-evi.dts | 25 +++-- 1 file changed, 7 insertions(+), 18 deletions(-) diff --git a/arch/arm/boot/dts/imx6q-evi.dts b/arch/arm/boot/dts/imx6q-evi.dts index 6de21

[PATCH v9 4/8] usb: core: add power sequence handling for USB devices

-off-by: Peter Chen <peter.c...@nxp.com> Tested-by Joshua Clayton <stillcompil...@gmail.com> Tested-by: Maciej S. Szmigiero <m...@maciej.szmigiero.name> Reviewed-by: Vaibhav Hiremath <hvaibhav.li...@gmail.com> --- drivers/usb/core/hub.c | 41 +++

<    1   2   3   4   5   6   7   8   9   10   >