Hello,
On Tue, Dec 04, 2018 at 12:43:21PM +0100, Stefan Wahren wrote:
> Am 04.12.18 um 09:52 schrieb Uwe Kleine-König:
> > But for the review you are right, I added the dt people to Cc for them
> > to comment. In v2 Matthew also noted that he would prefer to handle the
> >
Hello Stefan,
On Tue, Dec 04, 2018 at 09:40:26AM +0100, Stefan Wahren wrote:
> Am 04.12.18 um 09:31 schrieb Uwe Kleine-König:
> > The status quo on i.MX6 is that if "over-current-active-high" is
> > specified in the device tree this is configured as expected. If
&g
The polarity of the over current detection pin isn't configured on i.MX6/7
if it's unspecified in the device tree. So the actual configuration depends
on bootloader behavior which is bad.
So encourage users to fix their device tree by issuing a warning in this
case.
Signed-off-by: Uwe Kleine
.
Signed-off-by: Uwe Kleine-König
---
drivers/usb/chipidea/usbmisc_imx.c | 15 +++
1 file changed, 15 insertions(+)
diff --git a/drivers/usb/chipidea/usbmisc_imx.c
b/drivers/usb/chipidea/usbmisc_imx.c
index 4a8de1b37f67..47eee5cade84 100644
--- a/drivers/usb/chipidea/usbmisc_imx.c
+++ b
n explicit configuration the
bit is kept as is. On i.MX7 over current detection is used unless
disabled in the device tree.
Signed-off-by: Uwe Kleine-König
---
.../devicetree/bindings/usb/ci-hdrc-usb2.txt | 5 ++--
drivers/usb/chipidea/ci_hdrc_imx.c| 16 ---
drivers/us
Hello,
compared to v2 (which starts with Message-Id:
<20181202213325.26017-1-u.kleine-koe...@pengutronix.de>) I adapted patch
1 that the oc polarity is only determined if oc isn't disabled. This
results in the warning that is introduced in patch 2 not to be emitted
in this case.
Best regards
Uwe
larity property only "disable-over-current" is not found at patch 1.
Right, just fixed that up and will send out v3 in a moment.
Best regards
Uwe
--
Pengutronix e.K. | Uwe Kleine-König|
Industrial Linux Solutions | http://www.pengutronix.de/ |
On Mon, Dec 03, 2018 at 04:10:37PM +, Matthew Starr wrote:
> > -Original Message-
> > From: Uwe Kleine-König [mailto:u.kleine-koe...@pengutronix.de]
> > Sent: Sunday, December 02, 2018 3:33 PM
> >
> > The status quo on i.MX6 is that if "over-cu
eep
> > > "over-current-active-high" property unchanging to reduce users effort.
> > > If this property is set, we set OC polarity as high explicitly;
> > > otherwise, we set it as low explicitly.
> > >
> > > Cc: stable
> > > Cc: Peter Chen
ly;
> otherwise, we set it as low explicitly.
>
> Cc: stable
> Cc: Peter Chen
> Cc: Uwe Kleine-König
> Cc: Matthew Starr
> Signed-off-by: Peter Chen
> ---
> drivers/usb/chipidea/ci_hdrc_imx.c | 2 +-
> drivers/usb/chipidea/ci_hdrc_imx.h | 3 ++-
> drivers/usb/c
.
Signed-off-by: Uwe Kleine-König
---
drivers/usb/chipidea/usbmisc_imx.c | 15 +++
1 file changed, 15 insertions(+)
diff --git a/drivers/usb/chipidea/usbmisc_imx.c
b/drivers/usb/chipidea/usbmisc_imx.c
index 4a8de1b37f67..47eee5cade84 100644
--- a/drivers/usb/chipidea/usbmisc_imx.c
+++ b
The polarity of the over current detection pin isn't configured on i.MX6/7
if it's unspecified in the device tree. So the actual configuration depends
on bootloader behavior which is bad.
So encourage users to fix their device tree by issuing a warning in this
case.
Signed-off-by: Uwe Kleine
n explicit configuration the
bit is kept as is. On i.MX7 over current detection is used unless
disabled in the device tree.
Signed-off-by: Uwe Kleine-König
---
.../devicetree/bindings/usb/ci-hdrc-usb2.txt | 5 ++--
drivers/usb/chipidea/ci_hdrc_imx.c| 9 --
drivers/usb/chipidea/ci_
R_DIS);
> + reg |= MX6_BM_OVER_CUR_POLARITY;
> + else {
> reg &= ~(MX6_BM_OVER_CUR_DIS | MX6_BM_OVER_CUR_POLARITY);
You break machines here that use active low signaling but don't use the
new property (yet).
Also it's not nice to inverse the sematic
Hello Fabio,
On Fri, Nov 30, 2018 at 07:16:39PM -0200, Fabio Estevam wrote:
> [Adding Matt]
>
> On Fri, Nov 30, 2018 at 6:53 PM Uwe Kleine-König
> wrote:
> >
> > The status quo is:
> > - on i.MX25 the overcurrent polarity is always explicitly configured as
&g
n't matter if active high is
configured because that's the default or because it is configured
explicitly.
Signed-off-by: Uwe Kleine-König
---
.../devicetree/bindings/usb/ci-hdrc-usb2.txt | 1 +
drivers/usb/chipidea/ci_hdrc_imx.c| 9 -
drivers/usb/chipidea/ci_hdrc_imx.h
g session that led to
39cee200c23e, and I don't have access to the hardware in question any
more.
But your commit logs makes sense, so
Acked-by: Uwe Kleine-König <u.kleine-koe...@pengutronix.de>
Given that it took 2+ years to find this, backporting to stable and a
Fixes: line are p
could keep a per-port is_overcurrent flag. If the port has an
> > > overcurrent status and the flag is clear, log the error and set the
> > > flag. If the port doesn't have an overcurrent status, clear the flag.
> > > In other words, report each overcurrent ev
gic. This could only be
circumvented with another property "over-current-active-low" and
different default handling for imx6/7 vs imx25 if none (or both) of the
properties are present. For simplicity I stick to the easier and better
understandable alternative at the cost of dt incompatibili
ri, Oct 20, 2017 at 05:27:09PM -0200, Fabio Estevam wrote:
> > > > > On Fri, Oct 20, 2017 at 5:20 PM, Uwe Kleine-König
> > > > > <u.kleine-koe...@pengutronix.de> wrote:
> > > > >
> > > > > > It also works. However I wonder if it's
Hello Alan,
On Fri, Oct 20, 2017 at 03:47:37PM -0400, Alan Stern wrote:
> On Fri, 20 Oct 2017, Uwe [iso-8859-1] Kleine-König wrote:
> > On Fri, Oct 20, 2017 at 05:27:09PM -0200, Fabio Estevam wrote:
> > > On Fri, Oct 20, 2017 at 5:20 PM, Uwe Kleine-König
> > > <
Hey Fabio,
On Fri, Oct 20, 2017 at 05:27:09PM -0200, Fabio Estevam wrote:
> On Fri, Oct 20, 2017 at 5:20 PM, Uwe Kleine-König
> <u.kleine-koe...@pengutronix.de> wrote:
>
> > It also works. However I wonder if it's right that I'm spammed by
> > over-current messages no
Hello Alan,
On Fri, Oct 20, 2017 at 01:58:14PM -0400, Alan Stern wrote:
> On Fri, 20 Oct 2017, Uwe Kleine-König wrote:
> > diff --git a/drivers/usb/host/ehci-hub.c b/drivers/usb/host/ehci-hub.c
> > index df169c8e7225..08f59654654b 100644
> > --- a/drivers/usb/host/ehci-hub.c
w such that it doesn't include
manually transfering changes from one tree to another for submission.
Best regards
Uwe
--
Pengutronix e.K. | Uwe Kleine-König|
Industrial Linux Solutions | http://www.pengutronix.de/ |
--
To unsubscribe from th
63c1d7071 ("EHCI: only power off port if over-current is active")
Signed-off-by: Uwe Kleine-König <u.kleine-koe...@pengutronix.de>
---
drivers/usb/host/ehci-hub.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/usb/host/ehci-hub.c b/drivers/usb/host/ehci-hub.c
s
precedence by just overwriting the defaults added here.
Signed-off-by: Uwe Kleine-König <u.kleine-koe...@pengutronix.de>
---
arch/arm/boot/dts/imx25-eukrea-mbimxsd25-baseboard.dts | 2 --
arch/arm/boot/dts/imx25-pdk.dts| 2 --
arch/arm/boot/dts/imx25.dtsi
access the PMC using regmap")
> Reported-by: Uwe Kleine-König <u.kleine-koe...@pengutronix.de>
I played with the AT91 during my non-work time, so please use
u...@kleine-koenig.org as my email address.
> Signed-off-by: Alexandre Belloni <alexandre.bell...@free-electrons.com>
&g
On Tue, Dec 22, 2015 at 12:06:14PM -0600, Felipe Balbi wrote:
> Uwe Kleine-König <u.kleine-koe...@pengutronix.de> writes:
>
> > This is needed to let the omap3 otg controller make use of a generic
> > usb-nop-xceiv phy.
>
> missing SoB
If this is your only
This is needed to let the omap3 otg controller make use of a generic
usb-nop-xceiv phy.
---
drivers/usb/phy/phy-generic.c | 50 ---
drivers/usb/phy/phy-generic.h | 1 +
2 files changed, 48 insertions(+), 3 deletions(-)
diff --git
Todo: binding document, more properties
Signed-off-by: Uwe Kleine-König <u.kleine-koe...@pengutronix.de>
---
drivers/usb/phy/phy-ulpi.c | 129 +
1 file changed, 119 insertions(+), 10 deletions(-)
diff --git a/drivers/usb/phy/phy-ulpi.c b/drive
The phy's init routine must be called before it can be used. Do so in
musb_init_controller and the matching shutdown in musb_remove.
Signed-off-by: Uwe Kleine-König <u.kleine-koe...@pengutronix.de>
---
Note I'm not entirely sure this is the right place, but this made usb
working on my
Hello,
these patches make it possible to use USB on an omap3 machine with an
USB3320 external phy.
Best regards
Uwe
Uwe Kleine-König (3):
usb: musb: core: call init and shutdown for the usb phy
usb: phy-generic: register a struct phy device
usb: phy: ulpi: implement creating a phy via
Hello Stephen,
On Wed, Oct 28, 2015 at 09:55:05PM -0600, Stephen Warren wrote:
> On 10/27/2015 12:18 PM, Uwe Kleine-König wrote:
> > Hello,
> >
> > On Tue, Oct 27, 2015 at 09:08:27AM -0600, Stephen Warren wrote:
> >> On 10/27/2015 01:41 AM, Uwe Kleine-König wrote:
&
n ERR value or
The relevant part is that for a function with "optinal" in it's name not
finding the respective resource isn't an error. So checking for NULL
isn't error handling but handling the 2nd valid case.
Best regards
Uwe
--
Pengutronix e.K. | Uwe Kleine
with_args(np, "phys", "#phy-cells",
index, );
if (ret)
- return ERR_PTR(-ENODEV);
+ return ERR_PTR(ret);
in _of_phy_get (drivers/phy/phy-core.c) to make the phy functions behave
identically?
Best regards
Uwe
--
Pengutronix e.K.
Hello,
On Tue, Oct 27, 2015 at 09:08:27AM -0600, Stephen Warren wrote:
> On 10/27/2015 01:41 AM, Uwe Kleine-König wrote:
> >Hello,
> >
> >On Mon, Oct 26, 2015 at 08:08:58PM -0600, Stephen Warren wrote:
> >>On 10/26/2015 03:28 AM, Alexander Aring wrote:
> &
> an old USB PHY when devm_phy_get returns -ENOENT only. Other values like
> -EPROBE_DEFER need to be returned from the probe function, because it
> could be that a phy is specified but not probed before.
>
> Suggested-by: Uwe Kleine-König <u.kleine-koe...@pengutronix.de>
Acked-
Hello Alex,
On Mon, Oct 26, 2015 at 10:24:58AM +0100, Alexander Aring wrote:
> On Sun, Oct 25, 2015 at 08:00:11PM +0100, Uwe Kleine-König wrote:
> > On Sun, Oct 25, 2015 at 08:57:31AM +0100, Alexander Aring wrote:
> > > This patch adds support to return -EPROBE_DEFER if de
phy = NULL;
> uphy = devm_usb_get_phy(>dev, USB_PHY_TYPE_USB2);
> if (IS_ERR(uphy))
> --
> 2.6.1
>
>
>
--
Pengutronix e.K. | Uwe Kleine-König|
Industrial Linux Solutions | http://www.pengutroni
of struct usb_phy_io_ops's write.
Fixes: ffb865b1e460 ("usb: musb: add ulpi access operations")
Signed-off-by: Uwe Kleine-König <u.kleine-koe...@pengutronix.de>
---
Note that doing this might break callers that got the order wrong, too.
So I'd prefer to have this patch in a release befor
thout following all that fwnode discussion:
gpiod_get et al. should work for you here, doesn't it? It just takes a
struct device * and I'm happy with it.
Best regards
Uwe
--
Pengutronix e.K. | Uwe Kleine-König|
Industrial Linux Solutions | http:/
.
Best regards
Uwe
--
Pengutronix e.K. | Uwe Kleine-König|
Industrial Linux Solutions | http://www.pengutronix.de/ |
--
To unsubscribe from this list: send the line unsubscribe linux-usb in
the body of a message to majord...@vger.kernel.org
.
Thanks
Uwe
Uwe Kleine-König (10):
drm/msm/dp: use flags argument of devm_gpiod_get to set direction
drm/tilcdc: panel: make better use of gpiod API
iio: light: stk3310: use flags argument of devm_gpiod_get
iio: magn: bmc150: use flags argument of devm_gpiod_get
media
...@linux.intel.com
Signed-off-by: Uwe Kleine-König u.kleine-koe...@pengutronix.de
---
drivers/usb/dwc3/dwc3-pci.c | 26 --
1 file changed, 16 insertions(+), 10 deletions(-)
diff --git a/drivers/usb/dwc3/dwc3-pci.c b/drivers/usb/dwc3/dwc3-pci.c
index 27e4fc896e9d
.
To allow dropping this hack convert callers to explictly pass a value
for flags.
Acked-by: Felipe Balbi ba...@ti.com
Acked-by: Robert Jarzmik robert.jarz...@free.fr
Signed-off-by: Uwe Kleine-König u.kleine-koe...@pengutronix.de
---
drivers/usb/gadget/udc/pxa27x_udc.c | 2 +-
drivers/usb/phy/phy-generic.c
gpios.
Simplify driver accordingly.
Signed-off-by: Uwe Kleine-König u.kleine-koe...@pengutronix.de
---
drivers/phy/phy-tusb1210.c | 30 --
1 file changed, 12 insertions(+), 18 deletions(-)
diff --git a/drivers/phy/phy-tusb1210.c b/drivers/phy/phy-tusb1210.c
index
and improve error handling. Also expand the comment to explain
why it's not sensible to switch to devm_gpiod_get and why the gpiod_put
is also necessary.
Furthermore this is one caller less that stops us making the flags
argument to gpiod_get*() mandatory.
Signed-off-by: Uwe Kleine-König u.kleine
Hello,
On Fri, Mar 27, 2015 at 08:30:42PM +0100, Uwe Kleine-König wrote:
Since 39b2bbe3d715 (gpio: add flags argument to gpiod_get*() functions)
which appeared in v3.17-rc1, the gpiod_get* functions take an additional
parameter that allows to specify direction and initial value for output
gpios. Also use devm_gpiod_get* because otherwise the gpio
might be grabbed by a different driver.
Simplify driver accordingly.
Signed-off-by: Uwe Kleine-König u.kleine-koe...@pengutronix.de
---
Hello,
This usage without flags was introduced by commit a89d977cc04c (usb:
dwc3: pci: add quirk
gpios.
Simplify driver accordingly. Furthermore this is one caller less that
stops us making the flags argument to gpiod_get*() mandatory.
Signed-off-by: Uwe Kleine-König u.kleine-koe...@pengutronix.de
---
Hello,
[for the initial submission I forgot linux-usb on Cc, Felipe Balbi
requested
Hello,
On Sat, Mar 28, 2015 at 09:38:12AM +0100, Robert Jarzmik wrote:
Alexandre Courbot acour...@nvidia.com writes:
On 03/28/2015 04:30 AM, Uwe Kleine-König wrote:
Looks good, but can't we also merge the call to gpiod_direction to avoid
using
GPIOD_ASIS? The golden rule
.
To allow dropping this hack convert callers to explictly pass a value
for flags.
Signed-off-by: Uwe Kleine-König u.kleine-koe...@pengutronix.de
---
drivers/usb/gadget/udc/pxa27x_udc.c | 2 +-
drivers/usb/phy/phy-generic.c | 6 --
2 files changed, 5 insertions(+), 3 deletions(-)
diff --git
affected?
Best regards
Uwe
--
Pengutronix e.K. | Uwe Kleine-König|
Industrial Linux Solutions | http://www.pengutronix.de/ |
--
To unsubscribe from this list: send the line unsubscribe linux-usb in
the body of a message to majord...@vger.kernel.org
to handle devm_clk_get returning
-ENOENT and assume this means there is no need for a clock in this case.
(I think devm_clk_get prints an error message in this case. That's ugly
when the error is handled just fine.)
Best regards
Uwe
--
Pengutronix e.K. | Uwe Kleine-König
(USB2_SPEED);
+ /* Depend on boatloader for USB speed to be stated for board */
bootloader?
Uwe
--
Pengutronix e.K. | Uwe Kleine-König|
Industrial Linux Solutions | http://www.pengutronix.de/ |
--
To unsubscribe from this list: send
Hello Peter,
On Thu, Jan 23, 2014 at 09:22:41AM +0800, Peter Chen wrote:
On Wed, Jan 22, 2014 at 10:41:33PM +0100, Uwe Kleine-König wrote:
On Wed, Jan 22, 2014 at 10:49:51AM +0100, Uwe Kleine-König wrote:
On Tue, Dec 03, 2013 at 04:01:50PM +0800, Chris Ruehl wrote:
usb: chipidea
regards
Uwe
--
Pengutronix e.K. | Uwe Kleine-König|
Industrial Linux Solutions | http://www.pengutronix.de/ |
--
To unsubscribe from this list: send the line unsubscribe linux-usb in
the body of a message to majord...@vger.kernel.org
More
Hello,
On Wed, Jan 22, 2014 at 10:49:51AM +0100, Uwe Kleine-König wrote:
On Tue, Dec 03, 2013 at 04:01:50PM +0800, Chris Ruehl wrote:
usb: chipidea: hw_phymode_configure moved before ci_usb_phy_init
hw_phymode_configure configures the PORTSC registers and allow the
following phy_inits
. | Uwe Kleine-König|
Industrial Linux Solutions | http://www.pengutronix.de/ |
arch/arm/boot/dts/imx27-pingrp.h | 14 +
arch/arm/boot/dts/imx27.dtsi | 67
2 files changed, 81 insertions(+)
diff --git a/arch/arm
Hello Alexander,
On Tue, Jan 14, 2014 at 07:30:46AM +0400, Alexander Shiyan wrote:
I'll send you a personal letter with DT configuration.
That would be great. You didn't send it yet, did you?
Best regards
Uwe
--
Pengutronix e.K. | Uwe Kleine-König
Hello Chris,
On Tue, Jan 14, 2014 at 11:53:47AM +0800, Chris Ruehl wrote:
On Tuesday, January 14, 2014 11:30 AM, Alexander Shiyan wrote:
Понедельник, 13 января 2014, 22:31 +01:00 от Uwe Kleine-König
u.kleine-koe...@pengutronix.de:
On Sat, Jan 11, 2014 at 06:01:48PM +0400, Alexander Shiyan
Hello Alexander,
On Tue, Jan 14, 2014 at 02:47:39PM +0100, Uwe Kleine-König wrote:
On Tue, Jan 14, 2014 at 07:30:46AM +0400, Alexander Shiyan wrote:
I'll send you a personal letter with DT configuration.
That would be great. You didn't send it yet, did you?
I take that back, just found
On Sat, Jan 11, 2014 at 06:01:48PM +0400, Alexander Shiyan wrote:
Суббота, 11 января 2014, 13:55 +01:00 от Uwe Kleine-König
u.kleine-koe...@pengutronix.de:
On Mon, Nov 11, 2013 at 11:09:16AM +0400, Alexander Shiyan wrote:
Hello.
On Sunday, November 10, 2013 03:18 PM, Alexander
of definitions in DTS. This will be a next step.
Thanks.
Any news here?
Best regards
Uwe
--
Pengutronix e.K. | Uwe Kleine-König|
Industrial Linux Solutions | http://www.pengutronix.de/ |
--
To unsubscribe from this list: send the line
--
Pengutronix e.K. | Uwe Kleine-König|
Industrial Linux Solutions | http://www.pengutronix.de/ |
--
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
because you don't get a gcc warning :-)
Best regards
Uwe
--
Pengutronix e.K. | Uwe Kleine-König|
Industrial Linux Solutions | http://www.pengutronix.de/ |
--
To unsubscribe from this list: send the line unsubscribe linux-usb in
the body
and if
others are aware of the possible problems. Mark, maybe you can describe
in more detail the downsides you see?
Best regards
Uwe
--
Pengutronix e.K. | Uwe Kleine-König|
Industrial Linux Solutions | http://www.pengutronix.de/ |
--
To unsubscribe
On Wed, Jul 31, 2013 at 04:20:55PM +0800, Peter Chen wrote:
On Wed, Jul 31, 2013 at 09:33:06AM +0200, Uwe Kleine-König wrote:
On Tue, Jul 30, 2013 at 10:04:29PM -0300, Fabio Estevam wrote:
From: Fabio Estevam fabio.este...@freescale.com
By using devm_request_irq() we don't need
[Expanded Cc: a bit]
Hello,
On Wed, Jul 31, 2013 at 10:05:12AM +0100, Mark Brown wrote:
On Wed, Jul 31, 2013 at 10:46:45AM +0200, Uwe Kleine-König wrote:
We're discussing about devm_request_irq and wonder what happens at
remove time when the irq is still active.
OK, so the possible problem
On Wed, Jul 31, 2013 at 05:54:11AM -0400, Tejun Heo wrote:
Hello,
On Wed, Jul 31, 2013 at 11:44:34AM +0200, Uwe Kleine-König wrote:
OK, so the possible problem is that remove is called while the irq is
still active. That means you have to assert that all resources the irq
If your
= devm_clk_get(pdev-dev, NULL);
--
1.7.9.5
--
Pengutronix e.K. | Uwe Kleine-König|
Industrial Linux Solutions | http://www.pengutronix.de/ |
--
To unsubscribe from this list: send the line unsubscribe linux-usb in
the body of a message to majord
On Fri, Apr 26, 2013 at 03:36:42PM +0300, Alexander Shishkin wrote:
Uwe Kleine-König u.kleine-koe...@pengutronix.de writes:
On Wed, Apr 24, 2013 at 02:19:19PM -0300, Fabio Estevam wrote:
From: Fabio Estevam fabio.este...@freescale.com
Currently we always get the following message
72 matches
Mail list logo