Re: [PATCH 0/3] Fix OMAP EHCI probe & assorted cleanups

2018-09-23 Thread Lee Jones
On Sun, 23 Sep 2018, Lee Jones wrote: > On Sun, 23 Sep 2018, Laurent Pinchart wrote: > > > Hi Tony, > > > > On Tuesday, 11 September 2018 19:25:38 EEST Tony Lindgren wrote: > > > * Laurent Pinchart [180911 16:12]: > > > > On Tuesday, 11 Sep

Re: [PATCH 0/3] Fix OMAP EHCI probe & assorted cleanups

2018-09-23 Thread Lee Jones
> > >> > > >> "[PATCH] mfd: omap-usb-host: Fix dts probe of children" > > > > > > This fixes the issue for me. > > > > > > Tested-by: Laurent Pinchart > > > > OK good to hear. > > The fix is still not in v4.19-rc4

Re: [Question] MFD driver that handles clocks/resets and populates child nodes

2018-04-03 Thread Lee Jones
On Tue, 03 Apr 2018, Masahiro Yamada wrote: > 2018-04-03 17:03 GMT+09:00 Lee Jones : > > On Mon, 02 Apr 2018, Andrew Lunn wrote: > > > >> On Mon, Apr 02, 2018 at 10:21:01PM +0900, Masahiro Yamada wrote: > >> > 2018-04-02 21:04 GMT+09:00 Andrew Lunn : > >&

Re: [Question] MFD driver that handles clocks/resets and populates child nodes

2018-04-03 Thread Lee Jones
it up, you don't need an MFD. If > however the registers are in overlapping areas, you do need an > MFD. The MFD core provides access to the registers, while its children > implement PHY, reset, regulator etc. This device certainly sounds like an MFD to me. Can you share the

[no subject]

2018-02-03 Thread Jones
This is in regards to an inheritance on your surname, reply back using your email address, stating your full name for more details. Reply to email for info. Email me here ( ger...@dr.com ) -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord..

Re: [PATCH] extcon: Split out extcon header file for consumer and provider device

2017-10-04 Thread Lee Jones
on device instance > : Register/unregister extcon device > : Set the state of each external connector > : Set the property value of each external connector > : Set the property capability of each external connector > > Cc: Felipe Balbi > Cc: Kishon Vijay Abraham I >

Re: [PATCH v2 8/9] atmel_flexcom: Support backup mode

2017-09-19 Thread Lee Jones
t; Regards, > > > --- > > drivers/mfd/atmel-flexcom.c | 65 > > ++--- > > 1 file changed, 50 insertions(+), 15 deletions(-) This is the first time I've seen this patch. Why's that? -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software

Re: [PATCH 2/3] mfd: tps65010: move header file out of I2C realm

2017-08-14 Thread Lee Jones
On Mon, 14 Aug 2017, Lee Jones wrote: > On Sun, 13 Aug 2017, Wolfram Sang wrote: > > > On Tue, May 23, 2017 at 08:18:19AM +0100, Lee Jones wrote: > > > On Mon, 22 May 2017, Wolfram Sang wrote: > > > > > > > include/linux/i2c is not for client devic

Re: [PATCH 2/3] mfd: tps65010: move header file out of I2C realm

2017-08-13 Thread Lee Jones
On Sun, 13 Aug 2017, Wolfram Sang wrote: > On Tue, May 23, 2017 at 08:18:19AM +0100, Lee Jones wrote: > > On Mon, 22 May 2017, Wolfram Sang wrote: > > > > > include/linux/i2c is not for client devices. Move the header file to a > > > more appropriate location. &g

Re: [PATCH 3/3] mfd: twl: move header file out of I2C realm

2017-08-13 Thread Lee Jones
On Sun, 13 Aug 2017, Wolfram Sang wrote: > On Thu, Jul 06, 2017 at 08:03:52AM +0100, Lee Jones wrote: > > On Thu, 06 Jul 2017, Thierry Reding wrote: > > > > > On Mon, May 22, 2017 at 12:02:10AM +0200, Wolfram Sang wrote: > > > > include/linux/i2c is not for

Re: [PATCH v3 3/3] power: wm831x_power: Support USB charger current limit management

2017-07-25 Thread Lee Jones
hout drawing more current than specified from others. > > Signed-off-by: Mark Brown > Signed-off-by: Baolin Wang > --- > Documentation/devicetree/bindings/mfd/wm831x.txt | 1 + Acked-by: Lee Jones -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open s

Re: [PATCH 3/3] mfd: twl: move header file out of I2C realm

2017-07-06 Thread Lee Jones
eletions(-) > > rename include/linux/{i2c => mfd}/twl.h (100%) > > I didn't see this get applied yet, so just in case anyone was waiting > for me (this is trivial, so I don't think there's a need): You're not the last. :) -- Lee Jones Linaro S

[GIT PULL] Immutable branch between MFD and X86 due for the v4.13 merge window

2017-06-19 Thread Lee Jones
+-- drivers/platform/x86/intel_bxtwc_tmu.c | 4 - drivers/thermal/intel_bxt_pmic_thermal.c | 2 +- drivers/usb/typec/typec_wcove.c | 2 +- include/linux/mfd/intel_soc_pmic.h | 5 +- 6 files changed, 175 insertions(+), 84 deletions(-) -- Lee Jones Linaro STMicroelectronics

Re: [PATCH 2/3] mfd: tps65010: move header file out of I2C realm

2017-05-23 Thread Lee Jones
0.c | 2 +- For my own reference: Acked-for-MFD-by: Lee Jones > drivers/usb/host/ohci-omap.c| 2 +- > drivers/usb/phy/phy-isp1301-omap.c | 2 +- > drivers/video/fbdev/omap/lcd_h3.c | 2 +- > include/linux/{i2c => mfd}/tps65010.h | 2

Re: [PATCH 3/3] mfd: twl: move header file out of I2C realm

2017-05-23 Thread Lee Jones
twl4030-vibra.c | 2 +- > drivers/mfd/twl-core.c | 6 +++--- > drivers/mfd/twl4030-audio.c | 2 +- > drivers/mfd/twl4030-irq.c | 2 +- > drivers/mfd/twl4030-power.c | 2 +- > drivers/mfd/twl6030-irq.c | 2 +- F

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

2017-03-14 Thread Lee Jones
On Tue, 14 Mar 2017, Baolin Wang wrote: > Hi, > > On 14 March 2017 at 17:57, Lee Jones wrote: > > On Mon, 20 Feb 2017, Baolin Wang wrote: > > > > [...] > > > >> drivers/power/supply/wm831x_power.c | 63 +++ > >> drivers/usb/gadget/Kco

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

2017-03-14 Thread Lee Jones
sb/gadget/udc/charger.c > create mode 100644 include/linux/usb/charger.h > create mode 100644 include/uapi/linux/usb/charger.h Why have you sent this set to me? Please be careful when submitting patches, since a lot of us have enough mail to contend with already. -- Lee Jones Lina

[PATCH] USB: serial: cp210x: Add ID for a Juniper console

2016-09-23 Thread Kyle Jones
Signed-off-by: Kyle Jones --- Bus 004 Device 002: ID 10c4:8470 Cygnal Integrated Products, Inc. Device Descriptor: bLength18 bDescriptorType 1 bcdUSB 1.10 bDeviceClass0 bDeviceSubClass 0 bDeviceProtocol 0

Re: [PATCH 1/1] mfd: rtsx_usb: avoid setting ucr->current_sg.status

2016-08-30 Thread Lee Jones
if (act_len) > *act_len = ucr->current_sg.bytes; > > - return ucr->current_sg.status; > + return ret; > } > > int rtsx_usb_transfer_data(struct rtsx_ucr *ucr, unsigned int pipe, -- Lee Jones Linaro STMicroelectronics Landing Team Lead Lin

Re: [PATCHv6 3/3] mfd: intel_soc_pmic_bxtwc: add support for USB Type-C PHY on WhiskeyCove

2016-08-30 Thread Lee Jones
On Tue, 30 Aug 2016, Lee Jones wrote: > On Mon, 29 Aug 2016, Heikki Krogerus wrote: > > > Intel WhiskeyCove PMIC has also a USB Type-C PHY, so let's > > create a device for it. > > > > Signed-off-by: Heikki Krogerus > > Cc: Lee Jones > > -

Re: [PATCHv6 3/3] mfd: intel_soc_pmic_bxtwc: add support for USB Type-C PHY on WhiskeyCove

2016-08-30 Thread Lee Jones
On Mon, 29 Aug 2016, Heikki Krogerus wrote: > Intel WhiskeyCove PMIC has also a USB Type-C PHY, so let's > create a device for it. > > Signed-off-by: Heikki Krogerus > Cc: Lee Jones > --- > drivers/mfd/intel_soc_pmic_bxtwc.c | 11 +++ > 1 file chang

Re: [PATCH v4 1/2] mfd: ti-smusbdig: Add support for the TI SM-USB-DIG

2016-08-17 Thread Lee Jones
On Tue, 16 Aug 2016, Andrew F. Davis wrote: > On 08/09/2016 10:27 AM, Lee Jones wrote: > > On Tue, 09 Aug 2016, Andrew F. Davis wrote: > > > >> The TI SM-USB-DIG is a USB to SPI/I2C/1Wire/GPIO adapter. > >> Add MFD core support. > >> > >> Sig

Re: [PATCH v4 1/2] mfd: ti-smusbdig: Add support for the TI SM-USB-DIG

2016-08-09 Thread Lee Jones
ill requires a DT Ack. Please ping the relevant guys. -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a me

Re: [PATCH 4/7] reset: Supply *_shared variant calls when using *_optional APIs

2016-06-29 Thread Lee Jones
On Wed, 29 Jun 2016, Philipp Zabel wrote: > Am Mittwoch, den 29.06.2016, 09:06 +0100 schrieb Lee Jones: > > On Wed, 29 Jun 2016, Philipp Zabel wrote: > > > Am Dienstag, den 28.06.2016, 09:56 +0100 schrieb Lee Jones: > > > > Philipp, > > > > &

Re: [PATCH 4/7] reset: Supply *_shared variant calls when using *_optional APIs

2016-06-29 Thread Lee Jones
On Wed, 29 Jun 2016, Philipp Zabel wrote: > Am Dienstag, den 28.06.2016, 09:56 +0100 schrieb Lee Jones: > > Philipp, > > > > I need this to go into the -rcs too. > > > > Can I add it with your Ack please? > > I have already added your patches to my branch

[PATCH 1/1] usb: dwc3: st: Inform the reset framework that our reset line may be shared

2016-06-28 Thread Lee Jones
On the STiH410 B2120 development board the MiPHY28lp shares its reset line with the Synopsys DWC3 SuperSpeed (SS) USB 3.0 Dual-Role-Device (DRD). New functionality in the reset subsystems forces consumers to be explicit when requesting shared/exclusive reset lines. Signed-off-by: Lee Jones

Re: [PATCH 4/7] reset: Supply *_shared variant calls when using *_optional APIs

2016-06-28 Thread Lee Jones
s users of the optional_* API > in particular to specify that their request is for a 'shared' line. > > Signed-off-by: Lee Jones > --- > include/linux/reset.h | 12 > 1 file changed, 12 insertions(+) > > diff --git a/include/linux/reset.h b/includ

Re: [PATCH 9/10] usb: host: ehci-st: Inform the reset framework that our reset line may be shared

2016-06-28 Thread Lee Jones
On Tue, 28 Jun 2016, Lee Jones wrote: > On Mon, 06 Jun 2016, Alan Stern wrote: > > > On Mon, 6 Jun 2016, Lee Jones wrote: > > > > > On the STiH410 B2120 development board the ST EHCI IP shares its reset > > > line with the OHCI IP. New function

Re: [PATCH 9/10] usb: host: ehci-st: Inform the reset framework that our reset line may be shared

2016-06-28 Thread Lee Jones
On Mon, 06 Jun 2016, Alan Stern wrote: > On Mon, 6 Jun 2016, Lee Jones wrote: > > > On the STiH410 B2120 development board the ST EHCI IP shares its reset > > line with the OHCI IP. New functionality in the reset subsystems forces > > consumers to be explicit when req

Re: [PATCH 6/7] phy: miphy28lp: Inform the reset framework that our reset line may be shared

2016-06-28 Thread Lee Jones
On Tue, 07 Jun 2016, Kishon Vijay Abraham I wrote: > > > On Monday 06 June 2016 09:26 PM, Lee Jones wrote: > > On the STiH410 B2120 development board the MiPHY28lp shares its reset > > line with the Synopsys DWC3 SuperSpeed (SS) USB 3.0 Dual-Role-Device > > (DRD)

Re: [PATCH v3 1/2] mfd: ti-smusbdig: Add support for the TI SM-USB-DIG

2016-06-16 Thread Lee Jones
On Wed, 15 Jun 2016, Andrew F. Davis wrote: > On 06/15/2016 10:43 AM, Lee Jones wrote: > > This requires a USB Ack. > > > >> The TI SM-USB-DIG is a USB to SPI/I2C/1Wire/GPIO adapter. > >> Add MFD core support. > >> > >> Signed-off-by: Andrew

Re: [PATCH v3 1/2] mfd: ti-smusbdig: Add support for the TI SM-USB-DIG

2016-06-15 Thread Lee Jones
tions(+) > create mode 100644 drivers/mfd/ti-smusbdig.c > create mode 100644 include/linux/mfd/ti-smusbdig.h -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the li

Re: [RESEND PATCH v11 4/4] power: wm831x_power: Support USB charger current limit management

2016-06-15 Thread Lee Jones
hout drawing more current than specified from others. > > Signed-off-by: Mark Brown > Signed-off-by: Baolin Wang > Acked-by: Lee Jones > Acked-by: Charles Keepax > Acked-by: Peter Chen > Acked-by: Sebastian Reichel > --- >

Re: [STLinux Kernel] [PATCH 0/7] reset: Consumers to explicitly request 'exclusive' or 'shared' lines

2016-06-07 Thread Lee Jones
On Tue, 07 Jun 2016, Lee Jones wrote: > On Tue, 07 Jun 2016, Peter Griffin wrote: > > > Hi, > > > > On Mon, 06 Jun 2016, Lee Jones wrote: > > > > > Phasing out generic reset line requests enables us to make some better > > > decisions on when and

Re: [STLinux Kernel] [PATCH 0/7] reset: Consumers to explicitly request 'exclusive' or 'shared' lines

2016-06-07 Thread Lee Jones
On Tue, 07 Jun 2016, Peter Griffin wrote: > Hi, > > On Mon, 06 Jun 2016, Lee Jones wrote: > > > Phasing out generic reset line requests enables us to make some better > > decisions on when and how to (de)assert said lines. If an 'exclusive' > > line is

[PATCH 8/10] phy: phy-stih407-usb: Inform the reset framework that our reset line may be shared

2016-06-06 Thread Lee Jones
On the STiH410 B2120 development board the ports on the Generic PHY share their reset lines with each other. New functionality in the reset subsystems forces consumers to be explicit when requesting shared/exclusive reset lines. Signed-off-by: Lee Jones --- drivers/phy/phy-stih407-usb.c | 4

[PATCH 9/10] usb: host: ehci-st: Inform the reset framework that our reset line may be shared

2016-06-06 Thread Lee Jones
On the STiH410 B2120 development board the ST EHCI IP shares its reset line with the OHCI IP. New functionality in the reset subsystems forces consumers to be explicit when requesting shared/exclusive reset lines. Signed-off-by: Lee Jones --- drivers/usb/host/ehci-st.c | 4 ++-- 1 file changed

[PATCH 10/10] usb: host: ohci-st: Inform the reset framework that our reset line may be shared

2016-06-06 Thread Lee Jones
On the STiH410 B2120 development board the ST EHCI IP shares its reset line with the OHCI IP. New functionality in the reset subsystems forces consumers to be explicit when requesting shared/exclusive reset lines. Signed-off-by: Lee Jones --- drivers/usb/host/ohci-st.c | 4 ++-- 1 file changed

[PATCH 3/7] reset: Supply *_shared variant calls when using of_* API

2016-06-06 Thread Lee Jones
Consumers need to be able to specify whether they are requesting an 'exclusive' or 'shared' reset line no matter which API (of_*, devm_*, etc) they are using. This change allows users of the of_* API in particular to specify that their request is for a 'shared' l

[PATCH 1/7] reset: Reorder inline reset_control_get*() wrappers

2016-06-06 Thread Lee Jones
index() Signed-off-by: Lee Jones --- include/linux/reset.h | 42 +- 1 file changed, 21 insertions(+), 21 deletions(-) diff --git a/include/linux/reset.h b/include/linux/reset.h index ec0306ce..33eaf11 100644 --- a/include/linux/reset.h +++ b/include/linux/re

[PATCH 7/7] usb: dwc3: st: Inform the reset framework that our reset line may be shared

2016-06-06 Thread Lee Jones
On the STiH410 B2120 development board the MiPHY28lp shares its reset line with the Synopsys DWC3 SuperSpeed (SS) USB 3.0 Dual-Role-Device (DRD). New functionality in the reset subsystems forces consumers to be explicit when requesting shared/exclusive reset lines. Signed-off-by: Lee Jones

[PATCH 2/7] reset: Ensure drivers are explicit when requesting reset lines

2016-06-06 Thread Lee Jones
l consumers to move to the API immediately, causing administration headache between subsystems, this patch adds some temporary stand-in shim-calls. This will ease the burden at merge time and allow subsystems to migrate over to the new API in a more realistic time-frame. Signed-off-by: Lee Jon

[PATCH 4/7] reset: Supply *_shared variant calls when using *_optional APIs

2016-06-06 Thread Lee Jones
x27; line. Signed-off-by: Lee Jones --- include/linux/reset.h | 12 1 file changed, 12 insertions(+) diff --git a/include/linux/reset.h b/include/linux/reset.h index fd69240..c358106 100644 --- a/include/linux/reset.h +++ b/include/linux/reset.h @@ -141,6 +141,12 @@ stat

[PATCH 5/7] reset: TRIVIAL: Add line break at same place for similar APIs

2016-06-06 Thread Lee Jones
Standardise the way inline functions: devm_reset_control_get_shared_by_index devm_reset_control_get_exclusive_by_index ... are formatted. Signed-off-by: Lee Jones --- include/linux/reset.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/include/linux/reset.h b

[PATCH 6/7] phy: miphy28lp: Inform the reset framework that our reset line may be shared

2016-06-06 Thread Lee Jones
On the STiH410 B2120 development board the MiPHY28lp shares its reset line with the Synopsys DWC3 SuperSpeed (SS) USB 3.0 Dual-Role-Device (DRD). New functionality in the reset subsystems forces consumers to be explicit when requesting shared/exclusive reset lines. Signed-off-by: Lee Jones

[PATCH 0/7] reset: Consumers to explicitly request 'exclusive' or 'shared' lines

2016-06-06 Thread Lee Jones
l consumers to move to the API immediately, causing administration headache between subsystems, this patch adds some temporary stand-in shim-calls. This will ease the burden at merge time and allow subsystems to migrate over to the new API in a more realistic time-frame. Lee Jones (7): reset: Re

Re: [PATCH v8 5/7] mfd: intel_vuport: Add Intel virtual USB port MFD Driver

2016-05-09 Thread Lee Jones
| 1 + > drivers/mfd/intel-vuport.c | 89 > ++ > 3 files changed, 98 insertions(+) > create mode 100644 drivers/mfd/intel-vuport.c Acked-by: Lee Jones > diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig > index eea61e3..7e115ab

Re: [PATCH v4 1/7] mfd: da8xx-cfgchip: New header file for CFGCHIP registers.

2016-04-25 Thread Lee Jones
> include/linux/mfd/da8xx-cfgchip.h | 153 > ++ > include/linux/platform_data/usb-davinci.h | 22 - Works for me. Acked-by: Lee Jones Since this is a new file, I'm happy for it to be taken in via another tree. > 2 files changed, 153 insertions(+), 22 d

Re: [PATCH] usb: host: xhci-plat: Make enum xhci_plat_type start at a non zero value

2016-03-29 Thread Lee Jones
> >> > > >> > This patch adds struct xhci_plat_priv to simplify the code to match > >> > platform specific variables. For now, this patch adds a member "type" in > >> > the structure > >> > >> that's fine but the answ

Re: [PATCH v5 6/6] mfd: intel_vuport: Add Intel virtual USB port MFD Driver

2016-03-18 Thread Lee Jones
vers. > > Signed-off-by: Lu Baolu > Suggested-by: David Cohen This should be at the top. You couldn't have written the patch before it was suggested. > Reviewed-by: Felipe Balbi > Signed-off-by: Fengguang Wu What is this sign-off meant to indicate? > Reviewed-by: L

Re: [PATCH v2 7/7] mfd: intel_vuport: Add Intel virtual USB port MFD Driver

2016-03-07 Thread Lee Jones
orm_device *pdev) > +{ > + mfd_remove_devices(&pdev->dev); > + acpi_dev_remove_driver_gpios(ACPI_COMPANION(&pdev->dev)); > + > + return 0; > +} > + > +static struct acpi_device_id vuport_acpi_match[] = { > + { "INT3496" }, > + { } > +}; >

Re: mass storage behaviour

2015-10-17 Thread Paul Jones
On 17 Oct 2015, at 16:06, Alan Stern wrote: > On Sat, 17 Oct 2015, Paul Jones wrote: > >> On 17 Oct 2015, at 12:30, Alan Stern wrote: >> >>> Paul, can you tell your email client to wrap lines after 72 columns or >>> so? It would make replying a lot easi

Re: mass storage behaviour

2015-10-17 Thread Paul Jones
On 17 Oct 2015, at 12:30, Alan Stern wrote: > Paul, can you tell your email client to wrap lines after 72 columns or > so? It would make replying a lot easier… Mac Mail is not very friendly in this respect, but I can pay attention to not make long lines :) > ... >>> One thing you did not sho

net2280 crash during testusb

2015-10-16 Thread Paul Jones
Youssaf, if I run test 13 of testusb I get the following crash: [ 963.504860] WARNING: CPU: 1 PID: 0 at drivers/usb/gadget/udc/net2280.c:893 start_dma+0x218/0x220 [net2280]() [ 963.504861] Modules linked in: g_zero usbtest usb_f_ss_lb libcomposite configfs ccm arc4 iwlmvm i915 mac80211 iwlwifi

Re: mass storage behaviour

2015-10-16 Thread Paul Jones
On 16 Oct 2015, at 18:06, Felipe Balbi wrote: > > Hi, > > Paul Jones writes: >> First DMA to first IRQ = 116us. >> >> Second DMA to second IRQ = 107us. >> >> These seem fairly stable as the last request in a 500+MB transfer had >> exactly th

Re: mass storage behaviour

2015-10-16 Thread Paul Jones
On 16 Oct 2015, at 15:59, Alan Stern wrote: > On Fri, 16 Oct 2015, Paul Jones wrote: > >> Added some debugging statements in f_mass_storage/net2280 to get an idea of >> what is going on on the wire (as I unfortunately don’t have any tools to >> figure it out any o

Re: mass storage behaviour

2015-10-16 Thread Paul Jones
On 06 Oct 2015, at 12:34, Paul Jones wrote: > On 06 Oct 2015, at 16:44, Alan Stern wrote: > >> On Tue, 6 Oct 2015, Paul Jones wrote: >> >>> On 05 Oct 2015, at 23:09, Alan Stern wrote: >>> >>>> On Mon, 5 Oct 2015, Paul Jones wrote: >>

Re: Crash in usb_f_mass_storage

2015-10-16 Thread Paul Jones
On 16 Oct 2015, at 05:55, Kaukab, Yousaf wrote: >> -Original Message- >> From: Paul Jones [mailto:p.jo...@teclyn.com] >> Sent: Friday, October 16, 2015 3:40 AM >> To: Kaukab, Yousaf >> Cc: Alan Stern; Felipe Balbi; Linux USB Mailing List >> Subj

Re: Crash in usb_f_mass_storage

2015-10-15 Thread Paul Jones
On 15 Oct 2015, at 04:12, Kaukab, Yousaf wrote: >> -Original Message- >> From: Paul Jones [mailto:p.jo...@teclyn.com] >> Sent: Thursday, October 15, 2015 12:30 AM >> To: Alan Stern >> Cc: Kaukab, Yousaf; Felipe Balbi; Linux USB Mailing List >> Subj

Re: Crash in usb_f_mass_storage

2015-10-15 Thread Paul Jones
On 15 Oct 2015, at 17:18, Paul Jones wrote: > On 15 Oct 2015, at 04:12, Kaukab, Yousaf wrote: > >>> -Original Message- >>> From: Paul Jones [mailto:p.jo...@teclyn.com] >>> Sent: Thursday, October 15, 2015 12:30 AM >>> To: Alan Stern >&

Re: Crash in usb_f_mass_storage

2015-10-15 Thread Paul Jones
On 15 Oct 2015, at 04:12, Kaukab, Yousaf wrote: >> -Original Message- >> From: Paul Jones [mailto:p.jo...@teclyn.com] >> Sent: Thursday, October 15, 2015 12:30 AM >> To: Alan Stern >> Cc: Kaukab, Yousaf; Felipe Balbi; Linux USB Mailing List >> Subj

Re: Crash in usb_f_mass_storage

2015-10-14 Thread Paul Jones
On 14 Oct 2015, at 15:37, Alan Stern wrote: > On Wed, 14 Oct 2015, Paul Jones wrote: > >> On 12 Oct 2015, at 14:16, Felipe Balbi wrote: >> >>> >>> Hi, >>> >>> Paul Jones writes: >>>> On 10 Oct 2015, at 16:32, Paul Jones wr

Re: Crash in usb_f_mass_storage

2015-10-14 Thread Paul Jones
On 14 Oct 2015, at 15:37, Alan Stern wrote: > On Wed, 14 Oct 2015, Paul Jones wrote: > >> On 12 Oct 2015, at 14:16, Felipe Balbi wrote: >> >>> >>> Hi, >>> >>> Paul Jones writes: >>>> On 10 Oct 2015, at 16:32, Paul Jones wr

Re: Crash in usb_f_mass_storage

2015-10-14 Thread Paul Jones
On 14 Oct 2015, at 14:57, Felipe Balbi wrote: > > Hi, > > Paul Jones writes: >> Hi, >> >> On 12 Oct 2015, at 14:16, Felipe Balbi wrote: >> >>> >>> Hi, >>> >>> Paul Jones writes: >>>> On 10 Oct 2015, at

Re: Crash in usb_f_mass_storage

2015-10-14 Thread Paul Jones
On 12 Oct 2015, at 14:16, Felipe Balbi wrote: > > Hi, > > Paul Jones writes: >> On 10 Oct 2015, at 16:32, Paul Jones wrote: >> >>> I came across the following kernel message on the latest 4.3-rc4 whilst >>> performance testing on a

Re: Crash in usb_f_mass_storage

2015-10-10 Thread Paul Jones
On 10 Oct 2015, at 16:32, Paul Jones wrote: > I came across the following kernel message on the latest 4.3-rc4 whilst > performance testing on a USB3380 device connected to a Mac (10.9.5): > > [ 51.613838] WARNING: CPU: 2 PID: 0 at > drivers/usb/gadget/function/f_mas

Re: mass storage behaviour

2015-10-10 Thread Paul Jones
On 10 Oct 2015, at 16:34, Alan Stern wrote: > On Sat, 10 Oct 2015, Paul Jones wrote: > >>>> Why is Windows so much faster? Or to put it another way, why is Linux >>>> slow? How can we improve things? >>> >>> I don't know. We were doing

Crash in usb_f_mass_storage

2015-10-10 Thread Paul Jones
I came across the following kernel message on the latest 4.3-rc4 whilst performance testing on a USB3380 device connected to a Mac (10.9.5): [ 51.613838] WARNING: CPU: 2 PID: 0 at drivers/usb/gadget/function/f_mass_storage.c:346 fsg_setup+0x12a/0x170 [usb_f_mass_storage]() [ 51.613838] Modu

Re: mass storage behaviour

2015-10-10 Thread Paul Jones
On 06 Oct 2015, at 13:26, Paul Zimmerman wrote: > On Tue, Oct 6, 2015 at 10:01 AM, Alan Stern wrote: >> On Tue, 6 Oct 2015, Felipe Balbi wrote: >> > In my experience, you need to do at least the following to get max > performance from the mass storage gadget: > > - Use Windows

Re: mass storage behaviour

2015-10-07 Thread Paul Jones
On 07 Oct 2015, at 10:13, Greg KH wrote: > On Tue, Oct 06, 2015 at 10:26:08AM -0700, Paul Zimmerman wrote: >> On Tue, Oct 6, 2015 at 10:01 AM, Alan Stern >> wrote: >>> On Tue, 6 Oct 2015, Felipe Balbi wrote: >>> >> In my experience, you need to do at least the following to get max >>

Re: mass storage behaviour

2015-10-06 Thread Paul Jones
On 06 Oct 2015, at 16:44, Alan Stern wrote: > On Tue, 6 Oct 2015, Paul Jones wrote: > >> On 05 Oct 2015, at 23:09, Alan Stern wrote: >> >>> On Mon, 5 Oct 2015, Paul Jones wrote: >>> >>>>> Increasing the max_sectors_kb value, on the other han

Re: mass storage behaviour

2015-10-06 Thread Paul Jones
On 05 Oct 2015, at 23:09, Alan Stern wrote: > On Mon, 5 Oct 2015, Paul Jones wrote: > >>> Increasing the max_sectors_kb value, on the other hand, might remove >>> overhead by allowing a higher percentage of the transfer to consist of >>> real data as oppo

Re: mass storage behaviour

2015-10-05 Thread Paul Jones
On 05 Oct 2015, at 20:54, Alan Stern wrote: > On Mon, 5 Oct 2015, Paul Jones wrote: > >>> g_mass_storage, by default, uses 2 struct usb_request, try increasing that >>> to 4 >>> (can be done from make menuconfig itself) and see if anything changes. >>

Re: mass storage behaviour

2015-10-05 Thread Paul Jones
On 05 Oct 2015, at 20:29, Alan Stern wrote: > On Mon, 5 Oct 2015, Paul Jones wrote: > >> I�m investigating the (lack of) performance (around 150MB/s) of the USB3380 >> gadget in mass storage mode. >> Whilst tracing on a Linux 4.1 host I noticed that the Linux m

Re: mass storage behaviour

2015-10-05 Thread Paul Jones
On 05 Oct 2015, at 20:08, Felipe Balbi wrote: > On Mon, Oct 05, 2015 at 07:30:05PM +0200, Paul Jones wrote: >> I’m investigating the (lack of) performance (around 150MB/s) of the USB3380 >> gadget in mass storage mode. Whilst tracing on a Linux 4.1 host I noticed >> that

mass storage behaviour

2015-10-05 Thread Paul Jones
I’m investigating the (lack of) performance (around 150MB/s) of the USB3380 gadget in mass storage mode. Whilst tracing on a Linux 4.1 host I noticed that the Linux max storage driver is requesting 240 blocks, 16 blocks, 240 blocks, 16 blocks, etc. when doing a dd directly on the device:

Re: [PATCH v4 5/5] power: wm831x_power: Support USB charger current limit management

2015-08-19 Thread Lee Jones
t; + } > + > for (i = 0; i < ARRAY_SIZE(wm831x_bat_irqs); i++) { > irq = wm831x_irq(wm831x, >platform_get_irq_byname(pdev, > diff --git a/include/linux/mfd/wm831x/pdata.h > b/include/linux/mfd/wm831x/pdata.h > index dcc

Re: [PATCH v3 3/3] power: wm831x_power: Support USB charger current limit management

2015-08-18 Thread Lee Jones
guys got a bunch of bounces, we had a bit of an > email fiasco at this end. Hopefully all fixed now. Yes, I sent you guys a mail about that. Glad it's all sorted now. -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Fa

Re: [PATCH v3 3/3] power: wm831x_power: Support USB charger current limit management

2015-08-18 Thread Lee Jones
hout drawing more current than specified from others. > > Signed-off-by: Mark Brown > Signed-off-by: Baolin Wang > --- > drivers/power/wm831x_power.c | 69 > ++ > include/linux/mfd/wm831x/pdata.h | 3 ++ MFD: Acked-by: Lee Jones

Re: [PATCH v2 3/3] power: wm831x_power: Support USB charger current limit management

2015-08-14 Thread Lee Jones
hout drawing more current than specified from others. > > Signed-off-by: Mark Brown > Signed-off-by: Baolin Wang > --- > drivers/power/wm831x_power.c | 69 > ++ > include/linux/mfd/wm831x/pdata.h |3 ++ Acked-by: Lee Jones

Re: net2280 and UAS not working

2015-07-06 Thread Paul Jones
Hi, On 05 Jul 2015, at 20:43, Felipe Balbi wrote: > Hi, > > On Sun, Jul 05, 2015 at 06:56:29PM +0200, Paul Jones wrote: >> Ricardo, >> >> I’m trying to get the 3380 to work in UAS mode on the 4.0.1 and/or 4.1.0-rc8 >> kernels. >> I’m using the followin

net2280 and UAS not working

2015-07-05 Thread Paul Jones
Ricardo, I’m trying to get the 3380 to work in UAS mode on the 4.0.1 and/or 4.1.0-rc8 kernels. I’m using the following script (derived from Sebastian’s post): modprobe tcm_usb_gadget mount -t configfs none /sys/kernel/config CONFIGFS=/sys/kernel/config/ TARGET=$CO

Re: [PATCH v8 4/9] mfd: Add binding document for NVIDIA Tegra XUSB

2015-05-26 Thread Lee Jones
On Thu, 21 May 2015, Thierry Reding wrote: > On Thu, May 21, 2015 at 09:40:01AM +0100, Lee Jones wrote: > > On Wed, 20 May 2015, Thierry Reding wrote: > > > On Wed, May 20, 2015 at 07:35:51AM +0100, Lee Jones wrote: > > > > On Tue, 19 May 2015, Andrew Bresticker w

Re: [PATCH v8 4/9] mfd: Add binding document for NVIDIA Tegra XUSB

2015-05-21 Thread Lee Jones
On Wed, 20 May 2015, Thierry Reding wrote: > On Wed, May 20, 2015 at 07:35:51AM +0100, Lee Jones wrote: > > On Tue, 19 May 2015, Andrew Bresticker wrote: > > > On Thu, May 14, 2015 at 10:38 AM, Andrew Bresticker > > > wrote: > > > > On Thu, May 14, 2015

Unhandled IRQ 16 from ehci_hcd makes mouse unusable

2015-04-23 Thread Kynn Jones
NB: This is an updated version of https://bugzilla.kernel.org/show_bug.cgi?id=97131. I'm reposting here by the request of that bug's assignee. In this update I've added the output of some more diagnostic commands (at the end of the post). The list of diagnostic commands run is now this: $ c

Re: [PATCH 3/4] mfd: dln2: add support for ACPI

2015-01-20 Thread Lee Jones
i->dev = NULL; > + /* we can't release firmware see comment in dln2_probe_acpi */ > + ai->fw = NULL; > + } > + mutex_unlock(&dln2_acpi_lock); > +} > +#else > +static void dln2_probe_acpi(struct dln2_dev *dln2) > +{ > +} > + >

Maximum bitrate for CDC ECM

2015-01-20 Thread Paul Jones
wondering what that calculation is based on? The 1024/512 seem to be the IN/OUT buffer size being used. Technically USB3 allows for a native bitrate (after 8/10b coding) of 4Gbit/s. Regards, Paul. On 18 Jan 2015, at 16:49, Paul Jones wrote: > Ricardo, > > I think I figured out th

Re: [PATCH v3 1/2] mfd: dln2: add start/stop RX URBs helpers

2015-01-20 Thread Lee Jones
out_stop_rx; > > ret = dln2_hw_init(dln2); > if (ret < 0) { > dev_err(dev, "failed to initialize hardware\n"); > - goto out_cleanup; > + goto out_stop_rx; > } > > ret = mfd_add_hotplug_devices

Re: [PATCH v3 2/2] mfd: dln2: add suspend/resume functionality

2015-01-20 Thread Lee Jones
er = { > .probe = dln2_probe, > .disconnect = dln2_disconnect, > .id_table = dln2_table, > + .suspend = dln2_suspend, > + .resume = dln2_resume, > }; > > module_usb_driver(dln2_driver); -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source

Re: [PATCH 01/12] mfd: syscon: Add atmel-matrix registers definition

2015-01-19 Thread Lee Jones
On Sun, 18 Jan 2015, Boris Brezillon wrote: > Hi Lee, > > On Sun, 18 Jan 2015 12:52:59 +0000 > Lee Jones wrote: > > > On Wed, 14 Jan 2015, Alexandre Belloni wrote: > > > > > From: Boris Brezillon > > > > > > AT91 SoCs have a memory range

Re: [PATCH v2 3/4] mfd: dln2: add start/stop RX URBs helpers

2015-01-18 Thread Lee Jones
dev_err(dev, "failed to initialize hardware\n"); > - goto out_cleanup; > + goto out_stop_rx; > } > > ret = mfd_add_hotplug_devices(dev, dln2_devs, ARRAY_SIZE(dln2_devs)); > if (ret != 0) { > dev_err(dev, "fa

Re: hard lockup with USB3380

2015-01-18 Thread Paul Jones
higher performance? Regards, Paul. On 25 Nov 2014, at 16:30, Paul Jones wrote: > Ricardo, > > Unfortunately your latest change gives similar results, cycling errors in the > log: > [ 201.287706] [ cut here ] > [ 201.288328] WARNING: CPU: 3 PID: 1500

Re: [PATCH 03/12] mfd: syscon: Add atmel-smc registers definition

2015-01-18 Thread Lee Jones
gt; Signed-off-by: Boris Brezillon > Acked-by: Lee Jones > --- > include/linux/mfd/syscon/atmel-smc.h | 173 > +++ > 1 file changed, 173 insertions(+) > create mode 100644 include/linux/mfd/syscon/atmel-smc.h Applied, thanks. > diff --gi

Re: [PATCH 04/12] mfd: syscon: Add Atmel SMC binding doc

2015-01-18 Thread Lee Jones
egion as a syscon, so that different drivers can > configure the SMC interface (mostly timing configuration) according to > their need. > > Signed-off-by: Boris Brezillon > Acked-by: Lee Jones > --- > Documentation/devicetree/bindings/mfd/atmel-smc.txt | 19 ++

Re: [PATCH 02/12] mfd: syscon: Add Atmel Matrix bus DT binding documentation

2015-01-18 Thread Lee Jones
lare this register set as a > syscon device. > > Signed-off-by: Boris Brezillon > Acked-by: Lee Jones > --- > .../devicetree/bindings/mfd/atmel-matrix.txt | 24 > ++ > 1 file changed, 24 insertions(+) > create mode 100644 Documentation/devi

Re: [PATCH 01/12] mfd: syscon: Add atmel-matrix registers definition

2015-01-18 Thread Lee Jones
Brezillon > Acked-by: Lee Jones > --- > include/linux/mfd/syscon/atmel-matrix.h | 117 > > 1 file changed, 117 insertions(+) > create mode 100644 include/linux/mfd/syscon/atmel-matrix.h Applied, thanks. > diff --git a/include/linux/mfd/sy

Re: [PATCH v4 1/3] mfd: add support for Cypress CYUSBS234 USB Serial Bridge controller

2014-12-01 Thread Lee Jones
r the terms of the GNU General Public License version 2 as published > by > + * the Free Software Foundation. > + */ > + > +#ifndef __MFD_CYUSBS23X_H__ > +#define __MFD_CYUSBS23X_H__ > + > +#include > +#include > + > +/* Structure to hold all device specific stuff */ >

Re: hard lockup with USB3380

2014-11-25 Thread Paul Jones
MPTY))) { > > have this: > > if ( t & BIT(NAK_OUT_PACKETS)){ > count = readl(&ep->dma->dmacount); > count &= DMA_BYTE_COUNT_MASK; > break; > } > > if (likely(t & BIT(FIFO_EMPTY))) { > > > > On Tue, Nov 25, 2014 a

Re: hard lockup with USB3380

2014-11-25 Thread Paul Jones
Ricardo, it no longer locks up but if I try to write to the drive, I get cycles of: [ 2334.127653] [ cut here ] [ 2334.128318] WARNING: CPU: 0 PID: 2140 at drivers/usb/gadget/udc/net2280.c:816 start_dma+0x153/0x160 [net2280]() [ 2334.129105] Modules linked in: g_mass_stora

Re: hard lockup with USB3380

2014-11-25 Thread Paul Jones
ma->dmastat > ep->dma->dmacount > ep->dma->dmadesc > req->td_dma > ep->regs->ep_stat > > > > Thanks > > On Tue, Nov 25, 2014 at 1:58 PM, Paul Jones wrote: >> Ricardo, >> >> count = 0, req->valid = 1, le32_to_cpu(req-&g

  1   2   3   >