[PATCH] usb: imx21-hcd: Include missing linux/module.h

2012-12-09 Thread Fabio Estevam
From: Fabio Estevam Include , so that the following errors are fixed: drivers/usb/host/imx21-hcd.c:1929:20: error: expected declaration specifiers or '...' before string constant drivers/usb/host/imx21-hcd.c:1930:15: error: expected declaration specifiers or '...' be

Re: [PATCH] usb: imx21-hcd: Include missing linux/module.h

2012-12-10 Thread Fabio Estevam
Hi Greg, On Mon, Dec 10, 2012 at 4:16 PM, Greg KH wrote: > What tree needs this fixed, 3.7-rc8? Or is this showing up in > linux-next and can wait for 3.8-rc1? Just tried it on top of tree linus' tree and the problem is also there. Thanks, Fabio Estevam -- To unsubscribe fro

Re: 3.7 kernel hangs when doing scp

2012-12-19 Thread Fabio Estevam
Hi Greg, On Tue, Dec 18, 2012 at 12:43 AM, Greg KH wrote: > On Tue, Dec 18, 2012 at 12:01:35AM -0200, Fabio Estevam wrote: >> Hi, >> >> Booting a 3.7 kernel on a mx6qsabrelite board via NFS and using an >> asix USB/Ethernet adapter and then I try to transfer a 10MB fil

Re: 3.7 kernel hangs when doing scp

2012-12-20 Thread Fabio Estevam
efine USBMODE_SDIS(1<<3) /* Stream disable */ , which is at a different bit position from the i.MX6 Reference Manual: 4 SDIS Stream Disable Mode. (0 - Inactive [default]; 1 - Active) Also interested to know more details at to why disabling stream mode make thi

Re: 3.7 kernel hangs when doing scp

2012-12-20 Thread Fabio Estevam
On Thu, Dec 20, 2012 at 8:20 AM, Fabio Estevam wrote: > Hi Peter, > > On Thu, Dec 20, 2012 at 2:39 AM, Peter Chen wrote: > >> Please to see if the Stream mode disable (bit4, usbmode) is set or not? >> If it is not, please try below patch >> >> diff --git a/d

Re: 3.7 kernel hangs when doing scp

2012-12-20 Thread Fabio Estevam
On Thu, Dec 20, 2012 at 8:20 AM, Fabio Estevam wrote: > Excellent! Your suggestion made scp work with mx6 on 3.7. > > I assume that this "fix" or workaround would need a better place, right? > > Should this fix go into the chipidea host driver instead? Also, w

Re: 3.7 kernel hangs when doing scp

2012-12-20 Thread Fabio Estevam
On Thu, Dec 20, 2012 at 8:45 AM, Fabio Estevam wrote: > Would this be the correct fix? > > --- a/include/linux/usb/chipidea.h > +++ b/include/linux/usb/chipidea.h > @@ -18,7 +18,7 @@ struct ci13xxx_platform_data { > #define CI13XXX_REGS_SHARED

Re: 3.7 kernel hangs when doing scp

2012-12-21 Thread Fabio Estevam
Hi Peter, On Fri, Dec 21, 2012 at 12:22 AM, Peter Chen wrote: > Current chipidea driver only considers disable stream mode at device > mode, in fact, it may be related to below chipidea bug, and needs > to consider all usb modes. > > STAR 9000378958 > Title: Non-Double Word Aligned Buffer Addres

Re: 3.7 kernel hangs when doing scp

2012-12-21 Thread Fabio Estevam
. hw_write() is now used by ci_role_start(), so I re-ordered so that hw_write comes first in the code and compiler does not complain. Regards, Fabio Estevam -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kerne

[PATCH] usb: chipidea: Allow disabling streaming not only in udc mode

2012-12-21 Thread Fabio Estevam
From: Fabio Estevam When running a scp transfer using a USB/Ethernet adapter the following crash happens: $ scp test.tar.gz fabio@192.168.1.100:/home/fabio fabio@192.168.1.100's password: test.tar.gz 0%0 0.0KB/s --:-- ETA [ cut

[PATCH v2] usb: chipidea: Allow disabling streaming not only in udc mode

2012-12-22 Thread Fabio Estevam
From: Fabio Estevam When running a scp transfer using a USB/Ethernet adapter the following crash happens: $ scp test.tar.gz fabio@192.168.1.100:/home/fabio fabio@192.168.1.100's password: test.tar.gz 0%0 0.0KB/s --:-- ETA [ cut

Re: [PATCH v2] usb: chipidea: Allow disabling streaming not only in udc mode

2012-12-30 Thread Fabio Estevam
Hi Peter, On Sat, Dec 22, 2012 at 9:24 AM, Fabio Estevam wrote: > From: Fabio Estevam > > When running a scp transfer using a USB/Ethernet adapter the following crash > happens: > > $ scp test.tar.gz fabio@192.168.1.100:/home/fabio > fabio@192.168.1.100'

Re: [PATCH v4 0/6] support other fsl SoCs with usbmisc + small fixes

2013-01-07 Thread Fabio Estevam
Hi Alexander, On Wed, Nov 28, 2012 at 12:52 AM, Peter Chen wrote: > I have tested it at i.mx6q sabrelite board, it works good. > > I have pushed your commit to my git, please cc me > your coming chipidea patches, thanks. > > Alex, please add: > > Reviewed-by: Peter Chen > Tested-by: Peter Chen

Re: [PATCH 1/2] usb: host: imx21: fix compile error

2013-01-10 Thread Fabio Estevam
fore string constant > > by including > > Signed-off-by: Felipe Balbi I have already sent this fix: http://www.spinics.net/lists/linux-usb/msg75850.html Regards, Fabio Estevam -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a m

Re: [PATCH v6 1/3] usb: fsl-mxc-udc: replace cpu_is_xxx() with platform_device_id

2013-01-17 Thread Fabio Estevam
On Thu, Jan 17, 2013 at 9:25 AM, Lothar Waßmann wrote: > The equivalent of !cpu_is_mx51() would be > strcmp(pdev->id_entry->name, "imx-udc-mx51") (without the '!') meaning > id_entry->name is different from "imx-udc-mx51". Yes, agree. strcmp(pdev->id_entry->name, "imx-udc-mx51") is also better

[PATCH 3/3] usb: chipidea: udc: Use the preferred form for passing a size of a struct

2016-09-08 Thread Fabio Estevam
From: Fabio Estevam According to Documentation/CodingStyle: "The preferred form for passing a size of a struct is the following: p = kmalloc(sizeof(*p), ...); " , so do as suggested to improve readability. Signed-off-by: Fabio Estevam --- drivers/usb/chipidea/udc.c | 2

[PATCH 2/3] usb: chipidea: udc: Fit into a single line

2016-09-08 Thread Fabio Estevam
From: Fabio Estevam No need to split the dma_pool_zalloc() line into two as it can perfectly fit into a single line. Signed-off-by: Fabio Estevam --- drivers/usb/chipidea/udc.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/usb/chipidea/udc.c b/drivers/usb

[PATCH 1/3] usb: chipidea: udc: Use dma_pool_zalloc()

2016-09-08 Thread Fabio Estevam
From: Fabio Estevam We can make the code simpler by using dma_pool_zalloc() instead of calling dma_pool_zalloc() and then a memset(). Signed-off-by: Fabio Estevam --- drivers/usb/chipidea/udc.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/usb/chipidea

[PATCH v2 3/3] usb: chipidea: udc: Use the preferred form for passing a size of a struct

2016-09-09 Thread Fabio Estevam
From: Fabio Estevam According to Documentation/CodingStyle: "The preferred form for passing a size of a struct is the following: p = kmalloc(sizeof(*p), ...); " , so do as suggested to improve readability. Signed-off-by: Fabio Estevam --- Changes since v1: - None d

[PATCH v2 2/3] usb: chipidea: udc: Fit into a single line

2016-09-09 Thread Fabio Estevam
From: Fabio Estevam No need to split the dma_pool_zalloc() line in two lines as it can perfectly fit into a single one. Signed-off-by: Fabio Estevam --- Changes since v1: - Add a missing 'line' in the commit log. drivers/usb/chipidea/udc.c | 3 +-- 1 file changed, 1 insertion(+), 2

[PATCH v2 1/3] usb: chipidea: udc: Use dma_pool_zalloc()

2016-09-09 Thread Fabio Estevam
From: Fabio Estevam We can make the code simpler by using dma_pool_zalloc() instead of calling dma_pool_alloc() followed by memset(). Signed-off-by: Fabio Estevam --- Changes since v1: - Fix a typo in commit log (Sergei) drivers/usb/chipidea/udc.c | 6 ++ 1 file changed, 2 insertions

Re: [PATCH 1/3] usb: chipidea: udc: Use dma_pool_zalloc()

2016-09-09 Thread Fabio Estevam
Hi Sergei, On Fri, Sep 9, 2016 at 9:47 AM, Sergei Shtylyov wrote: >> We can make the code simpler by using dma_pool_zalloc() instead >> of calling dma_pool_zalloc() and then a memset(). > > >dma_pool_alloc(). Thanks, fixed in v2. -- To unsubscribe from this list: send the line "unsubscribe

Re: [PATCH 2/3] doc: dt-binding: generic onboard USB HUB

2015-12-07 Thread Fabio Estevam
On Mon, Dec 7, 2015 at 11:37 PM, Peter Chen wrote: > Add dt-binding documentation for generic onboard USB HUB. > > Signed-off-by: Peter Chen > --- > .../bindings/usb/generic-onboard-hub.txt | 28 > ++ > 1 file changed, 28 insertions(+) > create mode 100644 > Docu

Re: [PATCH v2 0/3] USB: add generic onboard USB HUB driver

2015-12-14 Thread Fabio Estevam
Hi Peter, On Mon, Dec 14, 2015 at 5:26 AM, Peter Chen wrote: > Hi all, > > There is a known issue that the USB code can't handle USB HUB's > external pins well, in that case, it may cause some onboard > USB HUBs can't work since their PHY's clock or reset pin needs to > operate. > > The user rep

Re: [PATCH v2 0/3] USB: add generic onboard USB HUB driver

2015-12-15 Thread Fabio Estevam
On Tue, Dec 15, 2015 at 4:28 AM, Peter Chen wrote: > Thanks, Fabio. > > I am afraid I forget to set gpio as output, would you please apply > below patch against my original ones: Same error happens with these changes applied. Here are more details: if I run a pure 4.3.2 then I do see the USB st

Re: [PATCH] Revert "usb: chipidea: usbmisc_imx: delete clock information"

2015-12-16 Thread Fabio Estevam
Hi Maciej, On Tue, Dec 15, 2015 at 6:19 PM, Maciej S. Szmigiero wrote: > Can we now use this change for repairing the USB support on UDOO board? > > This seems to work fine if not 100% correct: > --- a/arch/arm/boot/dts/imx6qdl-udoo.dtsi > +++ b/arch/arm/boot/dts/imx6qdl-udoo.dtsi > @@ -122,7 +1

Re: [PATCH v2 0/3] USB: add generic onboard USB HUB driver

2015-12-16 Thread Fabio Estevam
Hi Peter, On Wed, Dec 16, 2015 at 2:11 AM, Peter Chen wrote: > Thanks, Fabio, but I am curious how things like that? The USBOH3 clock > is not opened, the usb driver will hang when it tries to access > registers. If this clock is always on, then, why the system will > hang later? I found the is

[PATCH] chipidea: imx: Enable PM runtime for mx28

2015-12-22 Thread Fabio Estevam
usb1-port1: cannot reset (err = -32) usb usb1-port1: cannot reset (err = -32) usb usb1-port1: Cannot enable. Maybe the USB cable is bad? Passing CI_HDRC_SUPPORTS_RUNTIME_PM flag fixes the problem. Tested on a mx28evk and also on a mx28 custom board. Signed-off-by: Fabio Estevam --- drivers/usb

MX28 with hub cannot reset

2015-12-22 Thread Fabio Estevam
6aa33221 ("usb: phy: mxs: Add VF610 USB PHY support") shows the same error messages in the commit log, but trying the same approach of adding the MXS_PHY_NEED_IP_FIX flag does not help on mx28. Any suggestions? Thanks, Fabio Estevam -- To unsubscribe from this list: send the line "un

Re: MX28 with hub cannot reset

2015-12-23 Thread Fabio Estevam
Hi Peter, On Wed, Dec 23, 2015 at 7:49 AM, Peter Chen wrote: > Fabio, there is a bug for imx28 usb low power mode, and we have no > good way to implement > workaround using current USB PHY framework, would you please test by > add "usbcore.autosuspend = -1" > at bootargs (no CI_HDRC_SUPPORTS_RUN

Re: MX28 with hub cannot reset

2015-12-24 Thread Fabio Estevam
hub. > Is i.MX23 affected, too? As far as I know mx23 olinuxino maxi has a USB hub and the issue is not present there. Maybe Peter can confirm. Regards, Fabio Estevam -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger

[PATCH] usb: phy: mxs: Remove unused flags

2015-12-28 Thread Fabio Estevam
From: Fabio Estevam MXS_PHY_ABNORMAL_IN_SUSPEND and MXS_PHY_SENDING_SOF_TOO_FAST flags are never used, so let's get rid of them. Reported-by: Stefan Wahren Signed-off-by: Fabio Estevam --- drivers/usb/phy/phy-mxs-usb.c | 31 +++ 1 file changed, 3 inser

Re: [PATCH] usb: phy: mxs: Remove unused flags

2015-12-29 Thread Fabio Estevam
On Tue, Dec 29, 2015 at 6:15 AM, Peter Chen wrote: > I prefer to keep it, and it let us know the existing issues although > we have not implemented workaround for them. In future, we may have > solutions for how to implement them. Ok, what about adding the recommendation below? --- a/drivers/us

Re: MX28 with hub cannot reset

2015-12-29 Thread Fabio Estevam
n't know that the issue comes really from the mx28 instead of the > hub. It seems there are other users that reported this same issue at community.freescale.com. Regards, Fabio Estevam -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a

[PATCH] usb: phy: mxs: Suggest passing "usbcore.autosuspend=-1" for mx23/mx28

2015-12-30 Thread Fabio Estevam
From: Fabio Estevam On a mx28 board with a USB hub the following error is observed: hub 1-1:1.0: USB hub found hub 1-1:1.0: 2 ports detected usb 1-1: USB disconnect, device number 2 usb usb1-port1: cannot reset (err = -32) usb usb1-port1: cannot reset (err = -32) usb usb1-port1: cannot reset

Wakeup from USB on mx6

2016-02-03 Thread Fabio Estevam
ces/usb1/power/wakeup $ echo enabled > /sys/bus/usb/devices/1-1/power/wakeup - Put the system in suspend $ echo mem > /sys/power/state - Clicking in the mouse button does not wakeup the system. Any ideas? Thanks, Fabio Estevam -- To unsubscribe from this list: send the line "unsubscribe

Re: Wakeup from USB on mx6

2016-02-03 Thread Fabio Estevam
On Wed, Feb 3, 2016 at 1:21 PM, Fabio Estevam wrote: > Hi Peter, > > I am running imx6sl-evk with linux-next 20160203 and I am trying to > make it wake up via a USB mouse click. > > These are the steps I am doing: > > - Insert the USB mouse in the USB host port: > &

Re: Wakeup from USB on mx6

2016-02-03 Thread Fabio Estevam
On Wed, Feb 3, 2016 at 1:21 PM, Fabio Estevam wrote: > Hi Peter, > > I am running imx6sl-evk with linux-next 20160203 and I am trying to > make it wake up via a USB mouse click. > > These are the steps I am doing: > > - Insert the USB mouse in the USB host port: > &

Re: Wakeup from USB on mx6

2016-02-03 Thread Fabio Estevam
00.usb/power/wakeup /sys/devices/soc0/soc/210.aips-bus/2184200.usb/ci_hdrc.1/usb1/1-1/power/wakeup /sys/devices/soc0/soc/210.aips-bus/2184200.usb/ci_hdrc.1/usb1/power/wakeup /sys/devices/soc0/soc/210.aips-bus/2184200.usb/ci_hdrc.1/power/wakeup Regards, Fabio Estevam -- To unsubscribe fro

Re: Wakeup from USB on mx6

2016-02-03 Thread Fabio Estevam
On Wed, Feb 3, 2016 at 2:58 PM, Alan Stern wrote: > On Wed, 3 Feb 2016, Fabio Estevam wrote: > >> Hi Alan, >> >> On Wed, Feb 3, 2016 at 2:29 PM, Alan Stern wrote: >> >> > It seems likely that this script does exactly the same thing that you >> > di

[PATCH RESEND] usb: phy: mxs: Suggest passing "usbcore.autosuspend=-1" for mx23/mx28

2016-02-19 Thread Fabio Estevam
From: Fabio Estevam On a mx28 board with a USB hub the following error is observed: hub 1-1:1.0: USB hub found hub 1-1:1.0: 2 ports detected usb 1-1: USB disconnect, device number 2 usb usb1-port1: cannot reset (err = -32) usb usb1-port1: cannot reset (err = -32) usb usb1-port1: cannot reset

Re: [PATCH RESEND] usb: phy: mxs: Suggest passing "usbcore.autosuspend=-1" for mx23/mx28

2016-02-19 Thread Fabio Estevam
Hi Felipe, On Fri, Feb 19, 2016 at 10:05 AM, Felipe Balbi wrote: > how about detecting that you're running on mx23/mx28 and returning > -EBUSY on your runtime_idle implementation ? You already have the basics > done for it. Care to test below and tell me whether it helps ? Thanks for the sugges

Re: [PATCH RESEND] usb: phy: mxs: Suggest passing "usbcore.autosuspend=-1" for mx23/mx28

2016-02-19 Thread Fabio Estevam
On Fri, Feb 19, 2016 at 11:33 AM, Felipe Balbi wrote: > alright, in which sense doesn't it help ? Which platform did you use ? USB devices are not recognized with your patch applied on a mx28 board with an on-board USB hub and if I remove 'usbcore.autosuspend=-1' from the kernel command line. >

Re: [PATCH RESEND] usb: phy: mxs: Suggest passing "usbcore.autosuspend=-1" for mx23/mx28

2016-02-19 Thread Fabio Estevam
On Fri, Feb 19, 2016 at 12:13 PM, Felipe Balbi wrote: > h, okay. So you have another problem, actually. Seems like ci_hdrc > just shouldn't call your phy->suspend(), or your phy->suspend() > shouldn't do anything. How about below? > > @@ -370,6 +370,9 @@ static int mxs_phy_suspend(struct usb_

Re: [PATCH RESEND] usb: phy: mxs: Suggest passing "usbcore.autosuspend=-1" for mx23/mx28

2016-02-23 Thread Fabio Estevam
Hi Peter, On Sun, Feb 21, 2016 at 10:59 PM, Peter Chen wrote: > Fabio, Felipe is correct. The mx23 and mx28 should NOT call mxs phy's > suspend API > since the runtime suspend is not enabled for mx23 and mx28 at chipidea driver. > Would you please check if CI_HDRC_SUPPORTS_RUNTIME_PM is set wron

Re: [PATCH RESEND] usb: phy: mxs: Suggest passing "usbcore.autosuspend=-1" for mx23/mx28

2016-02-23 Thread Fabio Estevam
On Tue, Feb 23, 2016 at 4:47 PM, Fabio Estevam wrote: > Hi Peter, > > On Sun, Feb 21, 2016 at 10:59 PM, Peter Chen wrote: > >> Fabio, Felipe is correct. The mx23 and mx28 should NOT call mxs phy's >> suspend API >> since the runtime suspend is not enabled for m

Re: [PATCH RESEND] usb: phy: mxs: Suggest passing "usbcore.autosuspend=-1" for mx23/mx28

2016-02-23 Thread Fabio Estevam
On Tue, Feb 23, 2016 at 10:20 PM, Peter Chen wrote: > Oh, you said it is called before. > > http://www.spinics.net/lists/linux-usb/msg136714.html Yes, I have enabled runtime pm previously. Sorry for the confusion. > If you make sure the neither .runtime_suspend at chipidea driver > nor mxs_phy_

Re: [PATCH RESEND] usb: phy: mxs: Suggest passing "usbcore.autosuspend=-1" for mx23/mx28

2016-02-23 Thread Fabio Estevam
On Tue, Feb 23, 2016 at 11:44 PM, Peter Chen wrote: > Just plug in an external USB HUB at i.mx28 evk host port without pass > ''usbcore.autosuspend=-1' at bootargs to see if it works. Well, the board I have is based on mx28 and has a USB hub onboard. It does not work without ''usbcore.autosuspen

Re: [PATCH RESEND] usb: phy: mxs: Suggest passing "usbcore.autosuspend=-1" for mx23/mx28

2016-02-24 Thread Fabio Estevam
Hi Felipe, On Wed, Feb 24, 2016 at 7:47 AM, Felipe Balbi wrote: > Then what DOES get called ? If we don't reach runtime_pm of the PHY, we > must reach runtime_pm of chipidea. In that case, detect _there_ if > you're running on one of the known broken ones and return -EBUSY or > something like th

Re: [PATCH RESEND] usb: phy: mxs: Suggest passing "usbcore.autosuspend=-1" for mx23/mx28

2016-02-24 Thread Fabio Estevam
On Wed, Feb 24, 2016 at 8:32 AM, Felipe Balbi wrote: > Then that's the problem. You should _always_ implement your runtime_pm > callbacks otherwise driver model will assume you don't need to do > ANYTHING for runtime pm and runtime suspend you unconditionally. > > As a test, try setting the flag

Re: [PATCH RESEND] usb: phy: mxs: Suggest passing "usbcore.autosuspend=-1" for mx23/mx28

2016-02-24 Thread Fabio Estevam
On Wed, Feb 24, 2016 at 8:48 AM, Fabio Estevam wrote: > and this is the result: I missed to post the first print. Here is the complete log: [2.375588] usbcore: registered new interface driver usb-storage [2.405944] ** Calling ci_hdrc_imx_runtime_suspend [2.412

Re: [PATCH RESEND] usb: phy: mxs: Suggest passing "usbcore.autosuspend=-1" for mx23/mx28

2016-02-24 Thread Fabio Estevam
On Wed, Feb 24, 2016 at 10:29 AM, Felipe Balbi wrote: >> [2.814449] usb 1-1: new high-speed USB device number 2 using ci_hdrc >> [2.857129] fec 800f.ethernet eth0: Freescale FEC PHY driver >> [SMSC LAN8710/LAN8720] (mii_bus:phy_addr=800f.etherne:00, irq) >> [2.993879] hub 1-1:

Re: [PATCH RESEND] usb: phy: mxs: Suggest passing "usbcore.autosuspend=-1" for mx23/mx28

2016-02-25 Thread Fabio Estevam
On Wed, Feb 24, 2016 at 10:20 PM, Peter Chen wrote: > Would you get any chances to test i.mx28 evk with an external HUB? > I just would like to know it is not a HUB issue which can't be > suspended. Just connected an external USB hub on a mx28evk and did not see any issue. -- To unsubscribe from

Re: [PATCH v3 0/6] power: add power sequence library

2016-07-28 Thread Fabio Estevam
#x27;usbcore.autosuspend=-1' in the kernel command line? Regards, Fabio Estevam -- 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 v3 0/6] power: add power sequence library

2016-08-01 Thread Fabio Estevam
On Mon, Aug 1, 2016 at 11:55 AM, Joshua Clayton wrote: > Thanks a million, Fabio! > > 'usbcore.autosuspend=-1' quiets the errors. > > ~joshua > > P.S. I guess this technically is a bug in chipidea usb. > I'll give it a quick once over, though I am not very familiar > with USB core or the CI drive

[PATCH] usb: phy: phy-generic: Check clk_prepare_enable() error

2016-08-22 Thread Fabio Estevam
From: Fabio Estevam clk_prepare_enable() may fail, so we should better check its return value and propagate it in the case of failure. Signed-off-by: Fabio Estevam --- drivers/usb/phy/phy-generic.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/usb/phy/phy

Re: [PATCH] HID: usbkbd: return proper error code

2016-08-31 Thread Fabio Estevam
On Wed, Aug 31, 2016 at 1:28 PM, Sudip Mukherjee wrote: > Use proper error code instead of using -1 on failure to allocate > memory. We may use the error code later in the caller. > > Signed-off-by: Sudip Mukherjee > --- > drivers/hid/usbhid/usbkbd.c | 10 +- > 1 file changed, 5 insertio

Re: [PATCH] ARM: imx25: set default phy_type and dr_mode for usbotg port

2017-02-16 Thread Fabio Estevam
o now they had > to specify these two properties in the machine.dts which still takes > precedence by just overwriting the defaults added here. > > Signed-off-by: Uwe Kleine-König Reviewed-by: Fabio Estevam -- To unsubscribe from this list: send the line "unsubscribe linux-usb"

Re: Choppy USB audio on i.MX53 chipidea with 4.4 kernel. Ok in 4.10

2017-02-21 Thread Fabio Estevam
Hi Martin, On Tue, Feb 21, 2017 at 1:06 PM, Martin Fuzzey wrote: > Hi, > > I ran into a problem of "choppy" audio when using a USB codec on a i.MX53 > (with the chipidea driver) > The problem occurs on 4.4.47 but not on an old 3.19 > It also does not occur on the latest 4.10. > > The codec is :

Re: Patch for drivers/usb/serial/ch341.c enabling parity

2017-04-02 Thread Fabio Estevam
On Sun, Apr 2, 2017 at 3:00 PM, Leif Neland wrote: > There is a patch > > https://github.com/karlp/ch341-linux/blob/master/0001-usb-serial-ch341-Add-parity-support.patch > > which enables parity selection for the ch341 USB-RS485 adapter. > > From: Karl Palsson > Date: Tue, 18 Mar 2014 23:33:27 +0

[PATCH] chipidea: ci13xxx_imx: Remove sparse warning

2013-01-27 Thread Fabio Estevam
From: Fabio Estevam Remove the following sparse warning: drivers/usb/chipidea/ci13xxx_imx.h:22:25: error: dubious one-bit signed bitfield drivers/usb/chipidea/ci13xxx_imx.h:22:25: error: dubious one-bit signed bitfield Signed-off-by: Fabio Estevam --- drivers/usb/chipidea/ci13xxx_imx.h

[PATCH] usb: chipidea: usbmisc_imx6q: Staticize usbmisc_imx6q_drv_init/exit()

2013-02-20 Thread Fabio Estevam
From: Fabio Estevam Staticize usbmisc_imx6q_drv_init/exit() to fix the following sparse warnings: drivers/usb/chipidea/usbmisc_imx6q.c:147:12: warning: symbol 'usbmisc_imx6q_drv_init' was not declared. Should it be static? drivers/usb/chipidea/usbmisc_imx6q.c:153:13: warni

[PATCH 1/2] usb: host: ehci-mxc: Remove unneeded header file

2013-02-20 Thread Fabio Estevam
From: Fabio Estevam Since commit c0304996b (USB: ehci-mxc: remove Efika MX-specific CHRGVBUS hack) there is no need to include , so remove it. Signed-off-by: Fabio Estevam --- drivers/usb/host/ehci-mxc.c |4 1 file changed, 4 deletions(-) diff --git a/drivers/usb/host/ehci-mxc.c b

[PATCH 2/2] usb: host: ehci-mxc: Indicate succesful probe

2013-02-20 Thread Fabio Estevam
From: Fabio Estevam Rather than reporting that the controller is about to be initialized, it is more useful to indicate a succesful probe instead. Signed-off-by: Fabio Estevam --- drivers/usb/host/ehci-mxc.c |3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/usb

[PATCH v2 1/2] usb: host: ehci-mxc: Remove unneeded header file

2013-02-20 Thread Fabio Estevam
From: Fabio Estevam Since commit c0304996b (USB: ehci-mxc: remove Efika MX-specific CHRGVBUS hack) there is no need to include , so remove it. Signed-off-by: Fabio Estevam Acked-by: Alan Stern --- Changes since v1: - Added Alan's ack drivers/usb/host/ehci-mxc.c |4 1 file ch

[PATCH v2 2/2] usb: host: ehci-mxc: Remove dev_info on probe

2013-02-20 Thread Fabio Estevam
From: Fabio Estevam It is not very useful to indicate the the driver is about to be probed. Quoting Alan Stern [1]: "Plenty of drivers don't include any message like this at all. You might as well get rid of it entirely." Remove such dev_info(). [1] http://marc.info

mx28 usb host fails on 3.8

2013-02-27 Thread Fabio Estevam
re any pending patch to fix it. Thanks, Fabio Estevam -- 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: mx28 usb host fails on 3.8

2013-02-27 Thread Fabio Estevam
On Wed, Feb 27, 2013 at 11:53 AM, Fabio Estevam wrote: > Hi, > > I noticed that USB host is no longer functional on mx28evk running 3.8 > kernel. It worked well until 3.7.9: Ok, I managed to get USB host working now. commit 09f6ffde2e (USB: EHCI: fix build error by making ChipIdea h

Re: [PATCH 0/5] usb: chipidea: udc: fixes, iso ep and multi td support

2013-03-01 Thread Fabio Estevam
Hi Steffen, On Fri, Mar 1, 2013 at 12:39 PM, Steffen Trumtrar wrote: > This series for imx28: > > Tested-by: Steffen Trumtrar May I see your imx28-evk.dts with USB udc support, please? Thanks, Fabio Estevan -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in th

[PATCH] usb: core: Use 'unsigned int' for indexes

2013-03-19 Thread Fabio Estevam
From: Fabio Estevam Use 'unsigned int' for indexes, in order to fix the following build warnings with W=1 option: drivers/usb/core/usb.c: In function 'usb_find_alt_setting': drivers/usb/core/usb.c:89:16: warning: comparison between signed and unsigned integer express

[PATCH] usb: ulpi: Define a *otg_ulpi_create no-op

2013-03-20 Thread Fabio Estevam
From: Fabio Estevam Building a kernel for imx_v4_v5_defconfig with CONFIG_USB_ULPI disabled, results in the following error: arch/arm/mach-imx/built-in.o: In function 'pca100_init': platform-mx2-emma.c:(.init.text+0x6788): undefined reference to 'otg_ulpi_create'

Re: [PATCH] usb: ulpi: Define a *otg_ulpi_create no-op

2013-03-20 Thread Fabio Estevam
Hi Igor, On Wed, Mar 20, 2013 at 11:14 AM, Igor Grinberg wrote: > On 03/20/13 15:35, Fabio Estevam wrote: >> From: Fabio Estevam >> >> Building a kernel for imx_v4_v5_defconfig with CONFIG_USB_ULPI disabled, >> results >> in the following error: >> >&

Re: [PATCH v6 00/14] add imx usb driver based on Greg next tree

2012-07-30 Thread Fabio Estevam
On Mon, Jun 25, 2012 at 11:37 PM, Shawn Guo wrote: > On Mon, Jun 25, 2012 at 01:18:22PM -0300, Fabio Estevam wrote: >> Any suggestions as to how to make the driver work on mx23? >> > I won't be able to test it on my imx23-evk board, because the board > design is somehow

Re: mxs-phy on i.MX233 not enumerating

2012-08-13 Thread Fabio Estevam
related to this hack in FSL kernel? http://opensource.freescale.com/git?p=imx/linux-2.6-imx.git;a=commitdiff;h=c44dc00d2aecfcbf30336180f5ecd6a3a633545d What would be the proper way to handle HOSTDISCONDETECT so that it can work for mx23 as well? Thanks, Fabio Estevam -- 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: mxs-phy on i.MX233 not enumerating

2012-08-13 Thread Fabio Estevam
, including mx23, and would like to get some feedback from the FSL guys in Cc. Thanks, Fabio Estevam -- 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: mxs-phy on i.MX233 not enumerating

2012-08-13 Thread Fabio Estevam
e reset and speed negotiation period. 3. Do not set HW_USBPHY_CTRL.ENHOSTDISCONDETECT during host suspend/resume sequence." and the same is not present on mx28 reference manual. How can we make sure we are not violating the point 2 above? Thanks, Fabio Estevam -- To unsubscribe from this lis

Re: mxs-phy on i.MX233 not enumerating

2012-08-14 Thread Fabio Estevam
On 8/14/12, Chen Peter-B29397 wrote: > HW_USBPHY_CTRL.ENHOSTDISCONDETECT should be set after bus reset is > finished. I agree. How do we guarantee that ENHOSTDISCONDETECT is set after bus reset? Thanks, Fabio Estevam -- To unsubscribe from this list: send the line "unsubscribe li

[PATCH] usb: otg: mxs-phy: Allow USB host to work on mx23

2012-08-20 Thread Fabio Estevam
From: Fabio Estevam Differently from mx28 and mx6q, the mx23 SoC has the following description about ENHOSTDISCONDETECT bit in its Reference Manual: "Due to a on chip issue (Errata #2791), software must pay attention to when to assert the ENHOSTDISCONDETECT bit in HW_USBPHY_CTRL regist

Re: [PATCH] usb: otg: mxs-phy: Allow USB host to work on mx23

2012-08-20 Thread Fabio Estevam
ork on your first suggestion was that you only commented out ENHOSTDISCONDETECT from mxs_phy_on_connect and still kept it inside mxs_phy_on_disconnect. Regards, Fabio Estevam -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...

Re: [PATCH] usb: otg: mxs-phy: Allow USB host to work on mx23

2012-08-28 Thread Fabio Estevam
On Tue, Aug 21, 2012 at 1:04 AM, Fabio Estevam wrote: > From: Fabio Estevam > > Differently from mx28 and mx6q, the mx23 SoC has the following description > about ENHOSTDISCONDETECT bit in its Reference Manual: > > "Due to a on chip issue (Errata #2791), software must pay

[PATCH] usb: otg: mxs-phy: Fix mx23 operation

2012-08-30 Thread Fabio Estevam
connection processing from completing and not setting it prevents disconnect from being detected. No delay is needed for clearing of ENHOSTDISCONDETECT. Successfully tested on mx23-olinuxino (micro, mini and maxi variants) and mx28evk. Signed-off-by: Mike Thompson Signed-off-by: Fabio Estevam

[PATCH 2/2] usb: otg: mxs-phy: Improve mxs phy initialization

2012-08-30 Thread Fabio Estevam
-off-by: Fabio Estevam --- drivers/usb/otg/mxs-phy.c | 57 - 1 file changed, 51 insertions(+), 6 deletions(-) diff --git a/drivers/usb/otg/mxs-phy.c b/drivers/usb/otg/mxs-phy.c index 8188380..93a48ce 100644 --- a/drivers/usb/otg/mxs-phy.c +++ b/drivers

[PATCH 1/2] usb: otg: mxs-phy: Fix mx23 operation

2012-08-30 Thread Fabio Estevam
connection processing from completing and not setting it prevents disconnect from being detected. No delay is needed for clearing of ENHOSTDISCONDETECT. Successfully tested on mx23-olinuxino (micro, mini and maxi variants) and mx28evk. Signed-off-by: Mike Thompson Signed-off-by: Fabio Estevam

Re: [PATCH 1/2] usb: otg: mxs-phy: Fix mx23 operation

2012-08-30 Thread Fabio Estevam
tdiscondetect_work, >> + mxs_phy_enhostdiscondetect_delay); >> + schedule_delayed_work(&mxs_phy->enhostdiscondetect_work, >> + msecs_to_jiffies(MXY_PHY_ENHOSTDISCONDETECT_DELAY)); > > Isn't this mx23 specific? Checked with Pet

[PATCH v2 1/2] usb: otg: mxs-phy: Fix mx23 operation

2012-08-30 Thread Fabio Estevam
connection processing from completing and not setting it prevents disconnect from being detected. No delay is needed for clearing of ENHOSTDISCONDETECT. Successfully tested on mx23-olinuxino (micro, mini and maxi variants) and mx28evk. Signed-off-by: Mike Thompson Signed-off-by: Fabio Estevam

Re: [PATCH 1/2] usb: otg: mxs-phy: Fix mx23 operation

2012-09-06 Thread Fabio Estevam
Hi Felipe, On Thu, Sep 6, 2012 at 11:30 AM, Felipe Balbi wrote: > I need a final version for this series or it won't make it into v3.7 I sent a v2 for this last week: http://www.spinics.net/lists/linux-usb/msg69955.html Thanks, Fabio Estevam -- To unsubscribe from this list: send

Re: [PATCH v2 1/2] usb: otg: mxs-phy: Fix mx23 operation

2012-09-10 Thread Fabio Estevam
Hi Felipe, On Thu, Aug 30, 2012 at 6:26 PM, Fabio Estevam wrote: > From: Mike Thompson > > Currently mx23 fails to enumerate a USB device: > > [ 1.30] hub 1-0:1.0: unable to enumerate USB device on port 1 > [ 1.52] hub 1-0:1.0: unable to enumerate USB device on port 1

[PATCH] chipidea: ci13xxx_imx: Access phy via private data

2013-06-24 Thread Fabio Estevam
reported) - Insert the USB thumb again (connection is not detected) Fix this problem by accessing the usb_phy structure using the private data instead of accessing a local structure. Tested on a mx28evk board. Signed-off-by: Fabio Estevam --- drivers/usb/chipidea/ci13xxx_imx.c | 9 - 1 file

Re: Chipidea usb otg support for IMX/MXS (device functionality)

2013-06-25 Thread Fabio Estevam
On Mon, Jun 24, 2013 at 10:51 AM, Marek Vasut wrote: > Fabio, can you possibly test on MX23EVK please? I never used USB gadget with chipidea driver. Could you please explain what are the changes I need to do in the dts file (I want to try on mx28evk first) and defconfig in order to be able to t

Re: Chipidea usb otg support for IMX/MXS (device functionality)

2013-06-25 Thread Fabio Estevam
On Tue, Jun 25, 2013 at 11:23 AM, Fabio Estevam wrote: > On Mon, Jun 24, 2013 at 10:51 AM, Marek Vasut wrote: > >> Fabio, can you possibly test on MX23EVK please? > > I never used USB gadget with chipidea driver. > > Could you please explain what are the changes I need to

[PATCH] chipidea: bits: Cast PORTSC_PTS and DEVLC_PTS macros

2013-06-26 Thread Fabio Estevam
truncated to unsigned type [-Woverflow] drivers/usb/chipidea/core.c:243:3: warning: large integer implicitly truncated to unsigned type [-Woverflow] drivers/usb/chipidea/core.c:246:3: warning: large integer implicitly truncated to unsigned type [-Woverflow] Reported-by: Felipe Balbi Signed-off

Re: Chipidea usb otg support for IMX/MXS (device functionality)

2013-06-28 Thread Fabio Estevam
On Fri, Jun 28, 2013 at 10:04 AM, Chen Peter-B29397 wrote: > > >> > >> > Just measure the voltage of your otg vbus pin. >> >> Ok, I see it's 2.25 volts after I disconnect the microB-A thing . I >> should see >> 0.8V constantly with nothing connected on the port? >> > > You should see the voltage o

[PATCH] usb: phy: phy-mxs-usb: Check the return value from stmp_reset_block()

2013-07-03 Thread Fabio Estevam
From: Fabio Estevam stmp_reset_block() may fail, so let's check its return value and propagate it in the case of error. Signed-off-by: Fabio Estevam --- drivers/usb/phy/phy-mxs-usb.c | 13 - 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/drivers/usb/phy/ph

Re: [PATCH] usb: phy: samsung-usb2: Toggle HSIC GPIO from device tree

2013-07-10 Thread Fabio Estevam
Hi Julius, On Wed, Jul 10, 2013 at 2:42 PM, Julius Werner wrote: > Hi Felipe, > > This is intended to pull down a reset signal line, not to switch power > to the device. I could implement that with the regulator framework > too, but I think that would just be confusing and harder to understand >

Re: [PATCH v12 00/13] Add tested id switch and vbus connect detect support for Chipidea

2013-07-19 Thread Fabio Estevam
Hi Alexander, On Thu, Jul 11, 2013 at 3:27 AM, Peter Chen wrote: > This patchset adds tested otg id switch function and > vbus connect and disconnect detection for chipidea driver. > And fix kinds of bugs found at chipidea drivers after enabling > id and vbus detection. > > This patch is fully te

Re: [PATCH v2 6/8] usb: host: ohci-s3c2410 Use clk_prepare_enable/clk_disable_unprepare

2013-07-22 Thread Fabio Estevam
On Mon, Jul 22, 2013 at 8:49 PM, Tomasz Figa wrote: > dev_dbg(&dev->dev, "s3c2410_start_hc:\n"); > > - clk_enable(usb_clk); > + clk_prepare_enable(usb_clk); clk_prepare_enable may fail, so you would better check its return value. -- To unsubscribe from this list: send the li

Re: EHCI driver breaks at 6 endpoints

2013-07-23 Thread Fabio Estevam
application uses Please try it on a 3.10.2 kernel and let us know the result. Regards, Fabio Estevam -- 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: EHCI driver breaks at 6 endpoints

2013-07-23 Thread Fabio Estevam
ll you need to do is to add the USB nodes in device tree. You can refer to this commit: https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/arch/arm/boot/dts/imx53-qsb.dts?id=502d26a26ad745b7b65f7f0e4518eab7aa79a3ee Regards, Fabio Estevam -- To unsubscribe from this list: send th

[PATCH] chipidea: ci_hdrc_imx: Remove unused variable 'res'

2013-07-24 Thread Fabio Estevam
From: Fabio Estevam 'res' is not used anywhere, so let's get rid of it. Signed-off-by: Fabio Estevam --- drivers/usb/chipidea/ci_hdrc_imx.c | 7 --- 1 file changed, 7 deletions(-) diff --git a/drivers/usb/chipidea/ci_hdrc_imx.c b/drivers/usb/chipidea/ci_hdrc_imx.c index 1

[PATCH] chipidea: core: Move hw_phymode_configure() into probe

2013-07-25 Thread Fabio Estevam
port on mx53qsb board is functional. Signed-off-by: Fabio Estevam --- drivers/usb/chipidea/core.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/usb/chipidea/core.c b/drivers/usb/chipidea/core.c index a5df24c..a5b3774 100644 --- a/drivers/usb/chipidea/core.c

  1   2   3   4   >