From: Li Jun
Hi Felipe,
This patchset adds otg HNP polling common part for otg fsm.
Li Jun (3):
usb: common: otg-fsm: start HNP polling timer in host state
usb: common: otg-fsm: add HNP polling request sending funciton
usb: common: otg-fsm: add host_request_flag in usb_gadget
drivers/us
From: Li Jun
This patch adds host_request_flag in usb_gadget to store host request from
application, clear the flag when otg leaves host state.
Signed-off-by: Li Jun
---
drivers/usb/common/usb-otg-fsm.c |4
include/linux/usb/gadget.h |1 +
2 files changed, 5 insertions(+)
d
From: Li Jun
This patch starts HNP polling timer when otg is set to be a_host
or b_host.
Signed-off-by: Li Jun
---
drivers/usb/common/usb-otg-fsm.c |2 ++
include/linux/usb/otg-fsm.h |1 +
2 files changed, 3 insertions(+)
diff --git a/drivers/usb/common/usb-otg-fsm.c b/drivers/us
From: Li Jun
This patch adds OTG status selector request sending function, can be called
by HNP polling timer function after OTG port is in host state.
Signed-off-by: Li Jun
---
drivers/usb/common/usb-otg-fsm.c | 47 ++
include/linux/usb/otg-fsm.h |
Peripheral answers OTG status selector request from host according to
host request flag of gadget, this flag may be set by application via
kernel interface.
Signed-off-by: Li Jun
---
drivers/usb/chipidea/udc.c | 12 ++--
1 file changed, 10 insertions(+), 2 deletions(-)
diff --git a/dr
From: Li Jun
Hi Peter,
This patchset adds chipidea driver part of otg HNP polling support.
Li Jun (3):
usb: chipidea: udc: add OTG status request handling.
usb: chipidea: otg_fsm: add HNP polling support
usb: chipidea: set host request flag via sys input
drivers/usb/chipidea/ci.h |
From: Li Jun
This patch sets host request flag in sys input when a A device
set a_bus_req or a B device set b_bus_req in peripheral role.
Signed-off-by: Li Jun
---
drivers/usb/chipidea/otg_fsm.c | 13 -
1 file changed, 12 insertions(+), 1 deletion(-)
diff --git a/drivers/usb/chi
From: Li Jun
This patch adds HNP polling support for chipidea otg fsm driver, which
adds a SW timer to send HNP polling request.
Signed-off-by: Li Jun
---
drivers/usb/chipidea/ci.h |2 ++
drivers/usb/chipidea/otg_fsm.c | 32 ++--
drivers/usb/chipidea/otg_
The upper timer_interval limit is arbitrary and much higher
than anything usable in the real world. Reducing it from 15s
to ~4s to make the timer_interval fit in an u32 does not make
much difference. The limit is still outside the practical
bounds.
This eliminates the need for a 64bit timer_inte
From: Li Jun
This patch adds HNP polling support for chipidea otg fsm driver, which
adds a SW timer to send HNP polling request.
Signed-off-by: Li Jun
---
drivers/usb/chipidea/ci.h |2 ++
drivers/usb/chipidea/otg_fsm.c | 32 ++--
drivers/usb/chipidea/otg_
Change for v2:
- Replace gadget_is_otg() with ci_otg_is_fsm_mode() as condition when
handling OTG status selector request.
Peripheral answers OTG status selector request from host according to
host request flag of gadget, this flag may be set by application via
kernel interface.
Signed-off-by:
From: Li Jun
This patch sets host request flag in sys input when a A device
set a_bus_req or a B device set b_bus_req in peripheral role.
Signed-off-by: Li Jun
---
drivers/usb/chipidea/otg_fsm.c | 13 -
1 file changed, 12 insertions(+), 1 deletion(-)
diff --git a/drivers/usb/chi
Lars Melin writes:
> Your target audience is embedded systems with limited cpu power and
> buffer memory, right?
> If so, then you can't expect them to have ethtool included and their
> developers are not likely to be happy over having to "waste" another
> 100KB in order to tune a 20KB driver.
>
From: Li Jun
Hi Peter,
This patchset adds chipidea driver part of otg HNP polling support.
Li Jun (3):
usb: chipidea: udc: add OTG status request handling.
usb: chipidea: otg_fsm: add HNP polling support
usb: chipidea: set host request flag via sys input
drivers/usb/chipidea/ci.h |
On Fri, 2014-05-16 at 18:40 +0100, Catalin Marinas wrote:
> On Fri, May 16, 2014 at 06:08:45PM +0100, Jon Medhurst (Tixy) wrote:
> > On Fri, 2014-05-16 at 13:55 +0100, Catalin Marinas wrote:
> > [...]
> > > > It could if arm64 would restrict the DMA addresses to 32-bit, but it
> > > > doesn't
> >
On Friday 16 May 2014 13:55:01 Catalin Marinas wrote:
> On Thu, May 15, 2014 at 05:53:53PM +0100, Liviu Dudau wrote:
> > On Thu, May 15, 2014 at 04:36:25PM +0100, Alan Stern wrote:
> > > On Thu, 15 May 2014, Liviu Dudau wrote:
> > > > On Thu, May 15, 2014 at 03:11:48PM +0100, Alan Stern wrote:
> >
The dwc3 wrapper driver should not be fiddling with the core interrupts.
Disabling the core interrupts in prepare stops xhci from proper operation.
So remove disable/enable of core interrupts from prepare/complete.
Signed-off-by: George Cherian
---
drivers/usb/dwc3/dwc3-omap.c | 15 +
Move the extcon related code to its own function.
Improve code readability, decrease the dwc3_probe() size.
Signed-off-by: George Cherian
---
drivers/usb/dwc3/dwc3-omap.c | 65 ++--
1 file changed, 39 insertions(+), 26 deletions(-)
diff --git a/drivers/us
Following crash is seen on dwc3_omap removal
Unable to handle kernel NULL pointer dereference at virtual address 0018
pgd = ec098000
[0018] *pgd=ad1f9831, *pte=, *ppte=
Internal error: Oops: 17 [#1] SMP ARM
Modules linked in: usb_f_ss_lb g_zero usb_f_acm u_serial usb_f_ecm u
Move map offset to its own seperate function.
Improve code readability, decrease the dwc3_probe() size.
Signed-off-by: George Cherian
---
drivers/usb/dwc3/dwc3-omap.c | 31 +++
1 file changed, 19 insertions(+), 12 deletions(-)
diff --git a/drivers/usb/dwc3/dwc3-omap.
Move find and set the utmi mode to its own seperate function.
Improve code readability, decrease the dwc3_probe() size.
Signed-off-by: George Cherian
---
drivers/usb/dwc3/dwc3-omap.c | 44 +---
1 file changed, 25 insertions(+), 19 deletions(-)
diff --git
additional information:
I build a customized Kernel 3.14.4 based on Arch Stock kernel; with
only ONE change:
I deleted line 138 (NOT line 121)
xhci->quirks |= XHCI_SPURIOUS_REBOOT;
in drivers/usb/host/xhci-pci.c.
and issue disappears -> everything works fine now
---
Remove the x_major calculation logic from the wrapper revision register
to differentiate between OMAP5 and AM437x. This was done to find the
register offsets of wrapper register. Now that We do it using dt
compatible, remove the whole logic.
Signed-off-by: George Cherian
---
drivers/usb/dwc3/dwc
The series does some refactoring on dwc3_probe()
Patch 1 - Now that we use driver compatible for revision check, remove the
unnecessary logic.
Patch 2-4 - reduce the size of dwc3_probe()
Patch 5 - Fix the crash on dwc3_omap removal
Patch 6 - Addresses the issue of xhci hang while resuming from
Hi Bin,
On 5/15/2014 8:49 PM, Bin Liu wrote:
George,
On Thu, May 15, 2014 at 1:28 AM, George Cherian wrote:
Hi Bin,
On 5/14/2014 10:13 PM, Bin Liu wrote:
George,
On Wed, May 14, 2014 at 9:34 AM, Bin Liu wrote:
George,
On Wed, May 14, 2014 at 12:37 AM, George Cherian
wrote:
On 5/14/20
On Mon, May 19, 2014 at 09:32:43AM +0100, Arnd Bergmann wrote:
> On Friday 16 May 2014 13:55:01 Catalin Marinas wrote:
> > On Thu, May 15, 2014 at 05:53:53PM +0100, Liviu Dudau wrote:
> > > On Thu, May 15, 2014 at 04:36:25PM +0100, Alan Stern wrote:
> > > > On Thu, 15 May 2014, Liviu Dudau wrote:
>
From: Of Parag Warudkar
> I see a continuous flood of below messages on plugging in/using my USB
> token. (The comp code wasn't in the original message - I added it.) From
> what I can tell the device continues to work as expected.
>
> Should the warning be disabled for COMP_SHORT_TX like it is fo
On Wed, 2014-05-14 at 16:54 +0300, Dan Carpenter wrote:
> There are several issues here:
> 1) platform_get_resource() can return NULL and that wasn't handled.
> 2) We should request the memory before we remap it, and
>devm_ioremap_resource() does that.
> 3) devm_ioremap() returns a NULL but we
Vivek Gautam wrote:
>
> This patch adds sysreg-syscon node to exynos5250 and exynos5420 device
> tree, to access System Register's registers using syscon driver.
>
> Signed-off-by: Kamil Debski
> [gautam.vi...@samsung.com: Split this syreg-syscon dts entry from
> dts: Add usb2phy to Exynos 5250
Hi All,
I'm happy to announce the libusb-1.0.19-rc1 release.
The big feature of this release is support for bulk-streams on
Mac OS X and Linux (using the new usbfs API for this from 3.15).
Changelog:
* Add support for USB bulk streams on Linux and Mac OS X (#11)
* Windows: Add AMD and Intel USB-
On Mon, May 19, 2014 at 12:19:10PM +0300, Ivan T. Ivanov wrote:
> On Wed, 2014-05-14 at 16:54 +0300, Dan Carpenter wrote:
> > There are several issues here:
> > 1) platform_get_resource() can return NULL and that wasn't handled.
> > 2) We should request the memory before we remap it, and
> >dev
On Monday 19 May 2014 10:03:40 Catalin Marinas wrote:
> On Mon, May 19, 2014 at 09:32:43AM +0100, Arnd Bergmann wrote:
> > On Friday 16 May 2014 13:55:01 Catalin Marinas wrote:
> > > On Thu, May 15, 2014 at 05:53:53PM +0100, Liviu Dudau wrote:
> > > > On Thu, May 15, 2014 at 04:36:25PM +0100, Alan
Hello Joe
Thanks again for your comments. Since some the of the issues that you
point are also in the other parts of the file I will fix them in a
follow up patch for the whole driver.
On Thu, May 15, 2014 at 8:40 PM, Joe Perches wrote:
> On Thu, 2014-05-15 at 14:28 +0200, Ricardo Ribalda Delg
Some platforms (such as the Renesas R-Car) need to initialize some specific
registers after xhci driver calls usb_add_hcd() and before the driver calls
xhci_run(). So, this patch adds the xhci_plat_start() function.
Signed-off-by: Yoshihiro Shimoda
---
drivers/usb/host/xhci-plat.c |7 ++-
This patch set adds the USB xHCI support for the R-Car H2 and M2 SoCs.
These SoCs use an xHCI but still need specific initialization, mainly
to setup the firmware downloading and the specific registers.
This patch set depends on the following patches:
http://marc.info/?l=linux-usb&m=140014
The R-Car H2 and M2 SoCs come with an xHCI controller that requires
some specific initilization related to the firmware downloading and
some specific registers. This patch adds the support for this special
configuration as an xHCI quirk executed during probe and start.
Signed-off-by: Yoshihiro Shi
This commit extends the compatible string list of the xhci-platform
binding with the new "renesas,r8a7790-xhci" and "renesas,r8a7791-xhci"
compatible strings. It is used to describe the xHCI controller which
is available in the R-Car H2 and M2 SoCs.
Signed-off-by: Yoshihiro Shimoda
---
Documentat
Signed-off-by: Ricardo Ribalda Delgado
---
drivers/usb/gadget/net2280.c | 19 +--
1 file changed, 5 insertions(+), 14 deletions(-)
diff --git a/drivers/usb/gadget/net2280.c b/drivers/usb/gadget/net2280.c
index b43725a..bd851de 100644
--- a/drivers/usb/gadget/net2280.c
+++ b/drive
For bool variables
Signed-off-by: Ricardo Ribalda Delgado
---
drivers/usb/gadget/net2280.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/usb/gadget/net2280.c b/drivers/usb/gadget/net2280.c
index 5b9368d..b43725a 100644
--- a/drivers/usb/gadget/net2280.c
+++
Improves readability of the code
Signed-off-by: Ricardo Ribalda Delgado
---
drivers/usb/gadget/net2280.c | 572 +--
drivers/usb/gadget/net2280.h | 65 ++---
2 files changed, 318 insertions(+), 319 deletions(-)
diff --git a/drivers/usb/gadget/net2280.c b/
Instead of using magic numbers use #defines
Signed-off-by: Ricardo Ribalda Delgado
---
drivers/usb/gadget/net2280.c | 68 +++-
drivers/usb/gadget/net2280.h | 1 +
2 files changed, 36 insertions(+), 33 deletions(-)
diff --git a/drivers/usb/gadget/net2280.
From: Ricardo Ribalda Delgado
> Improves readability of the code
Personally I don't find macros like BIT() improve the readability at all.
What next? Maybe:
#define ADD(a, b) ((a) + (b))
David
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a mes
Hi Shimoda-san,
Thanks for your patches, I did however find one typo below:
On Mon, May 19, 2014 at 7:08 PM, Yoshihiro Shimoda
wrote:
> The R-Car H2 and M2 SoCs come with an xHCI controller that requires
> some specific initilization related to the firmware downloading and
> some specific regist
On 05/19/2014 08:46 AM, lin du wrote:
> Add linux-usb in loop.
>
> Hello,
>
> When reload the xhci_hcd.ko continuously as following, we will see
> many error messages, even HW hang! (almost < 300 times;
> if no sleep between rmmod and modprobe driver, the value should be smaller).
>
Hi
And thanks
Hello David
I dont want to start a flame war here :) but In the particular case
of hardware registers I believe it does.
This patch has been posted because on a previous patch to support the
usb3380, one of the requested changes was to use this macro. In order
to have consistent code in the file
Hello.
On 19-05-2014 10:18, Li Jun wrote:
From: Li Jun
This patch sets host request flag in sys input when a A device
set a_bus_req or a B device set b_bus_req in peripheral role.
Signed-off-by: Li Jun
---
drivers/usb/chipidea/otg_fsm.c | 13 -
1 file changed, 12 insert
Hello.
On 19-05-2014 12:32, George Cherian wrote:
Remove the x_major calculation logic from the wrapper revision register
to differentiate between OMAP5 and AM437x. This was done to find the
register offsets of wrapper register. Now that We do it using dt
compatible, remove the whole logic.
ned-off-by: Gregory CLEMENT
> > Signed-off-by: Thomas Petazzoni
> > ---
> > arch/arm/mach-mvebu/Kconfig | 1 +
> > 1 file changed, 1 insertion(+)
>
> Patches 10 and 11 applied to mvebu/soc
These two just hit linux-next (in next-20140519). They're pointless, as
con
Based on 'for-next' branch of Kgene's linux-samsung tree.
These patches are as per discussions on the driver side patches
which have already been acked. [1]
Changes from v2:
- Addressed kukjin's comments to update DT binding documentation for
sysreg.
- removed the version for each
From: Vivek Gautam
This patch adds sysreg-syscon node to exynos5250 and exynos5420 device
tree, to access System Register's registers using syscon driver.
Signed-off-by: Kamil Debski
[gautam.vi...@samsung.com: Split this syreg-syscon dts entry from
dts: Add usb2phy to Exynos 5250 patch]
[gautam
From: Vivek Gautam
Add required device node for usb2phy to let enable USB 2.0
support.
Signed-off-by: Vivek Gautam
---
arch/arm/boot/dts/exynos5420.dtsi | 10 ++
1 file changed, 10 insertions(+)
diff --git a/arch/arm/boot/dts/exynos5420.dtsi
b/arch/arm/boot/dts/exynos5420.dtsi
inde
From: Vivek Gautam
Add required device node for ehci and ohci controllers to
enable USB 2.0 support.
Signed-off-by: Vivek Gautam
---
arch/arm/boot/dts/exynos5420.dtsi | 30 ++
1 file changed, 30 insertions(+)
diff --git a/arch/arm/boot/dts/exynos5420.dtsi
b/arch
From: Kamil Debski
Add support to PHY of USB2 of the Exynos 5250 SoC.
Signed-off-by: Kamil Debski
[gautam.vi...@samsung.com: Split the usb phy entries from
syscon entries from earlier patch: dts: Add usb2phy to Exynos 5250]
[gautam.vi...@samsung.com: Added phy entry for OHCI also along with EHC
Vivek Gautam wrote:
>
> Add required fixed-regulator for VBUS supply for USB 3.0
> controller phy.
>
> Signed-off-by: Vivek Gautam
> ---
> arch/arm/boot/dts/exynos5250-snow.dts | 22 ++
> 1 file changed, 22 insertions(+)
>
> diff --git a/arch/arm/boot/dts/exynos5250-snow.
On Fri, May 16, 2014 at 10:47:18PM +0800, Zhuang Jin Can wrote:
> Hi,
> On Fri, May 16, 2014 at 07:41:06AM -0500, Felipe Balbi wrote:
> > On Fri, May 16, 2014 at 11:50:13PM +0800, Zhuang Jin Can wrote:
> > > > On Fri, May 16, 2014 at 05:57:57AM +0800, Zhuang Jin Can wrote:
> > > > > In ISOC transfe
From: Oliver Neukum
Parsing device descriptors can fail due to a failed memory
allocation. The error needs to be properly propagated to the
upper layers.
Signed-off-by: Oliver Neukum
---
drivers/gpu/drm/udl/udl_main.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/driver
From: Oliver Neukum
There's a window during which read() would return 0 instead
of a correct error for no data yet. Reorder initialization
to fix the race.
Signed-off-by: Oliver Neukum
---
drivers/usb/misc/yurex.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/us
From: Oliver Neukum
The workqueue handler may call appledisplay_bl_get_brightness() while
user space calls appledisplay_bl_update_status(). As they share a
buffer that must not happen. Use a mutex for mutual exclusion.
Signed-off-by: Oliver Neukum
---
drivers/usb/misc/appledisplay.c | 13 +
From: Oliver Neukum
send_request_dev_dep_msg_in() use a buffer allocated on the stack.
Fix by kmalloc()ing the buffer.
Signed-off-by: Oliver Neukum
---
drivers/usb/class/usbtmc.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/drivers/usb/class/usbtmc.c b/drivers/usb/c
From: Oliver Neukum
Converting the header to BIT for readability. No functional
change.
Signed-off-by: Oliver Neukum
---
drivers/usb/class/cdc-acm.h | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/usb/class/cdc-acm.h b/drivers/usb/class/cdc-acm.h
index
Hi Shimoda-san,
On Mon, May 19, 2014 at 12:08 PM, Yoshihiro Shimoda
wrote:
> The R-Car H2 and M2 SoCs come with an xHCI controller that requires
> some specific initilization related to the firmware downloading and
> some specific registers. This patch adds the support for this special
> configur
On Mon, May 19, 2014 at 12:08 PM, Yoshihiro Shimoda
wrote:
> This commit extends the compatible string list of the xhci-platform
> binding with the new "renesas,r8a7790-xhci" and "renesas,r8a7791-xhci"
> compatible strings. It is used to describe the xHCI controller which
> is available in the R-C
On Mon, May 19, 2014 at 12:07 PM, Yoshihiro Shimoda
wrote:
> Some platforms (such as the Renesas R-Car) need to initialize some specific
> registers after xhci driver calls usb_add_hcd() and before the driver calls
> xhci_run(). So, this patch adds the xhci_plat_start() function.
>
> Signed-off-by
Hello.
On 19-05-2014 14:08, Yoshihiro Shimoda wrote:
The R-Car H2 and M2 SoCs come with an xHCI controller that requires
some specific initilization related to the firmware downloading and
some specific registers. This patch adds the support for this special
configuration as an xHCI quirk execu
On Sat, 2014-05-17 at 05:08 +0700, Alif Mubarak Ahmad wrote:
> This device vendor and product id is 1c9e:9800
> It is working as serial interface with generic usbserial driver.
> I thought it is more suitable to use usbserial option driver, which
> has better capability distinguishing between modem
Find whether we are running on newer silicon. The babble control
register reads 0x4 by default in newer silicon as opposed to 0
in old versions of AM335x. Based on this enable the sw babble
control logic.
Signed-off-by: George Cherian
---
drivers/usb/musb/musb_dsps.c | 37 +++
For DSPS platform usb_phy_vbus(_off/_on) are NOPs.
So during musb_platform_reset() call usb_phy(_shutdown/_init)
Signed-off-by: George Cherian
---
drivers/usb/musb/musb_dsps.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/drivers/usb/musb/musb_dsps.c b/drivers/usb/musb
Add sw_babble_control() logic to differentiate between transient
babble and real babble condition. Also add the SW babble control
register definitions.
Babble control register logic is implemented in the latest
revision of AM335x.
Signed-off-by: George Cherian
---
drivers/usb/musb/musb_dsps.c |
Subject: [PATCH v3 0/5] Add support for SW babble Control
Series add support for SW babble control logic found in
new silicon versions of AM335x. Runtime differentiation of
silicon version is done by checking the BABBLE_CTL register.
For newer silicon the register default value read is 0x4 and
f
During babble condition both first disconnect of devices are
initiated. Make sure MUSB controller is reset and re-initialized
after all disconnects.
To acheive this schedule a delayed work for babble rrecovery.
While at that convert udelay to usleep_range.
Refer Documentation/timers/timers-howto.
Currently musb_platform_reset() is only used by dsps.
In case of BABBLE interrupt for other platforms the musb_platform_reset()
is a NOP. In such situations no need to re-initialize the endpoints.
Also in the latest silicon revision of AM335x, we do have a babble recovery
mechanism without resetti
BABBLE and RESET share the same interrupt. The interrupt
is considered to be RESET if MUSB is in peripheral mode and
as a BABBLE if MUSB is in HOST mode.
Handle babble condition iff MUSB is in HOST mode.
Signed-off-by: George Cherian
---
drivers/usb/musb/musb_core.c | 2 +-
1 file changed, 1 in
Hi George,
On Mon, May 19, 2014 at 3:40 AM, George Cherian wrote:
> Hi Bin,
>
>
> On 5/15/2014 8:49 PM, Bin Liu wrote:
>>
>> George,
>>
>> On Thu, May 15, 2014 at 1:28 AM, George Cherian
>> wrote:
>>>
>>> Hi Bin,
>>>
>>>
>>> On 5/14/2014 10:13 PM, Bin Liu wrote:
George,
On We
On Mon, 19 May 2014, Arnd Bergmann wrote:
> > dma_alloc_coherent() on arm64 should return 32-bit addresses if the
> > coherent_dma_mask is set to 32-bit. Which kernel version is this?
>
> The more important question is what happens to high buffers allocated
> elsewhere
> that get passed into dma
Hi Peter,
W dniu 16.05.2014 11:00, Peter Chen pisze:
Hi Felipe & Alan,
To continue with topic discussed at
http://www.spinics.net/lists/linux-usb/msg105279.html,
I implement the gadget bus to bind udc to gadget driver follow
most ideas from your two.
While the idea is interesting I think som
On Mon, May 19, 2014 at 05:40:29PM +0200, Andrzej Pietrasiewicz wrote:
> Hi Peter,
>
> W dniu 16.05.2014 11:00, Peter Chen pisze:
> >Hi Felipe & Alan,
> >
> >To continue with topic discussed at
> >http://www.spinics.net/lists/linux-usb/msg105279.html,
> >I implement the gadget bus to bind udc to g
Hi,
On Mon, May 19, 2014 at 8:39 AM, George Cherian wrote:
> BABBLE and RESET share the same interrupt. The interrupt
> is considered to be RESET if MUSB is in peripheral mode and
> as a BABBLE if MUSB is in HOST mode.
>
> Handle babble condition iff MUSB is in HOST mode.
>
> Signed-off-by: Georg
On Mon, May 19, 2014 at 10:44:51AM +0100, Arnd Bergmann wrote:
> On Monday 19 May 2014 10:03:40 Catalin Marinas wrote:
> > On Mon, May 19, 2014 at 09:32:43AM +0100, Arnd Bergmann wrote:
> > > The more important question is what happens to high buffers allocated
> > > elsewhere
> > > that get passe
On Mon, 19 May 2014, Felipe Balbi wrote:
> On Mon, May 19, 2014 at 05:40:29PM +0200, Andrzej Pietrasiewicz wrote:
> > Hi Peter,
> >
> > W dniu 16.05.2014 11:00, Peter Chen pisze:
> > >Hi Felipe & Alan,
> > >
> > >To continue with topic discussed at
> > >http://www.spinics.net/lists/linux-usb/msg1
Hi,
On Mon, May 19, 2014 at 8:39 AM, George Cherian wrote:
> Currently musb_platform_reset() is only used by dsps.
> In case of BABBLE interrupt for other platforms the musb_platform_reset()
> is a NOP. In such situations no need to re-initialize the endpoints.
> Also in the latest silicon revis
On Monday 19 May 2014 10:16:57 Alan Stern wrote:
> On Mon, 19 May 2014, Arnd Bergmann wrote:
>
> > > dma_alloc_coherent() on arm64 should return 32-bit addresses if the
> > > coherent_dma_mask is set to 32-bit. Which kernel version is this?
> >
> > The more important question is what happens to h
On Monday 19 May 2014 16:56:08 Catalin Marinas wrote:
> On Mon, May 19, 2014 at 10:44:51AM +0100, Arnd Bergmann wrote:
> > On Monday 19 May 2014 10:03:40 Catalin Marinas wrote:
> > > On Mon, May 19, 2014 at 09:32:43AM +0100, Arnd Bergmann wrote:
> > > > The more important question is what happens t
On Sat, May 17, 2014 at 10:43:14AM -0400, Alan Stern wrote:
> On Fri, 16 May 2014, Todd E Brandt wrote:
>
> > This patch creates a separate instance of the usb_address0 mutex for each
> > USB
> > bus, and attaches it to the usb_bus device struct. This allows devices on
> > separate buses to be en
This patch creates a separate instance of the usb_address0 mutex for each USB
bus, and attaches it to the usb_bus device struct. This allows devices on
separate buses to be enumerated in parallel; saving time.
In the current code, there is a single, global instance of the usb_address0
mutex which
Move logical continuations to end of line
Signed-off-by: Ricardo Ribalda Delgado
---
drivers/usb/gadget/net2280.c | 145 +--
drivers/usb/gadget/net2280.h | 4 +-
2 files changed, 73 insertions(+), 76 deletions(-)
diff --git a/drivers/usb/gadget/net2280.
Replace a long and ugly expresion with an already available function.
Signed-off-by: Ricardo Ribalda Delgado
---
drivers/usb/gadget/net2280.c | 10 +-
1 file changed, 1 insertion(+), 9 deletions(-)
diff --git a/drivers/usb/gadget/net2280.c b/drivers/usb/gadget/net2280.c
index bd851de..c
Driver was using custom functions WARNING, ERROR, DEBUG, instead of
pr_err, pr_dgb...
VDEBUG has also been replaced by DEBUG because the same functionality
(or even better) can be achieved with dynamic printk.
Signed-off-by: Ricardo Ribalda Delgado
---
drivers/usb/gadget/net2280.c | 130 ++
net2280 was developed before the CodeStyle was enforced. This set of patches
intend to improve the readability of the code.
This set of patches should be applied after:
[PATCH 5/5] usb: gadget: net2280: Use module_pci_driver macro
In order to avoid mail list spamming I have not resubmitted the
Instead of octal notation, to be consistent around the file.
Signed-off-by: Ricardo Ribalda Delgado
---
drivers/usb/gadget/net2280.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/usb/gadget/net2280.c b/drivers/usb/gadget/net2280.c
index c3205ec..5b2ab68 100644
-
On Mon, 2014-05-19 at 21:11 +0200, Ricardo Ribalda Delgado wrote:
> VDEBUG has also been replaced by DEBUG because the same functionality
> (or even better) can be achieved with dynamic printk.
dev_vdbg exists for that purpose.
--
To unsubscribe from this list: send the line "unsubscribe l
On Mon, 19 May 2014, Ricardo Ribalda Delgado wrote:
> Instead of octal notation, to be consistent around the file.
>
> Signed-off-by: Ricardo Ribalda Delgado
> ---
> drivers/usb/gadget/net2280.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/usb/gadget/net
Hello Joe
I think dev_vdg cannot be controlled with DYNAMIC_DEBUG, therefore I
replaced it with dev_dbg.
But if you prefer to use dev_dbg, I will modify it to use dev_dbg
Thanks!
On Mon, May 19, 2014 at 9:16 PM, Joe Perches wrote:
> On Mon, 2014-05-19 at 21:11 +0200, Ricardo Ribalda Delgado w
Hello Alan
What about using octal notation in all the file and merge the patch
with the other big patch of the series? I am trying to have some
consistency in the file :)
Thanks!
On Mon, May 19, 2014 at 9:30 PM, Alan Stern wrote:
> On Mon, 19 May 2014, Ricardo Ribalda Delgado wrote:
>
>> Inste
On Mon, 19 May 2014, Ricardo Ribalda Delgado wrote:
> Fix Code Style using checkpatch.pl criteria
>
> Signed-off-by: Ricardo Ribalda Delgado
Not really necessary, although this doesn't hurt.
It's notable that scattered in amongst all the checkpatch-related
changes are a few things that actuall
On Mon, 2014-05-19 at 21:33 +0200, Ricardo Ribalda Delgado wrote:
> I think dev_vdg cannot be controlled with DYNAMIC_DEBUG, therefore I
> replaced it with dev_dbg.
>
> But if you prefer to use dev_dbg, I will modify it to use dev_dbg
s/VDEBUG/dev_vdbg/
dev_vdbg can be controlled by DYNAMIC_DEBU
On Mon, 19 May 2014, Ricardo Ribalda Delgado wrote:
> Move logical continuations to end of line
This is almost totally unnecessary. Some people prefer the
operators at the end of the first line, other people prefer them at the
start of the next line. It's just a matter of taste and it doesn't
Hello Joe
OK, then I will use dev_vdbg. Thanks!
On Mon, May 19, 2014 at 9:42 PM, Joe Perches wrote:
> On Mon, 2014-05-19 at 21:33 +0200, Ricardo Ribalda Delgado wrote:
>> I think dev_vdg cannot be controlled with DYNAMIC_DEBUG, therefore I
>> replaced it with dev_dbg.
>>
>> But if you prefer
On Mon, 19 May 2014, Ricardo Ribalda Delgado wrote:
> Driver was using custom functions WARNING, ERROR, DEBUG, instead of
> pr_err, pr_dgb...
What's wrong with that, and why bother to change it?
> diff --git a/drivers/usb/gadget/net2280.c b/drivers/usb/gadget/net2280.c
> index e67ae96..11800bb
Hello Alan
I am trying to be consistent around the file. On a previous patch I
was asked to move the operators at the end. I did so, and now I am
modifying the file to look alike.
On Mon, May 19, 2014 at 9:43 PM, Alan Stern wrote:
> On Mon, 19 May 2014, Ricardo Ribalda Delgado wrote:
>
>> Mov
On Mon, 19 May 2014, Ricardo Ribalda Delgado wrote:
> Hello Alan
>
>
> I am trying to be consistent around the file. On a previous patch I
> was asked to move the operators at the end. I did so, and now I am
> modifying the file to look alike.
Changing other people's code to be more consistent
1 - 100 of 124 matches
Mail list logo
