Hi Peter,
On 07/05/2014 08:25 AM, Peter Griffin wrote:
Signed-off-by: Peter Griffin
Acked-by: Maxime Coquelin
Thanks,
Maxime
---
MAINTAINERS | 1 +
1 file changed, 1 insertion(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 702ca10..269ad3b 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@
On Tue, 08 Jul 2014, Maxime Coquelin wrote:
> On 07/05/2014 08:25 AM, Peter Griffin wrote:
> >Signed-off-by: Peter Griffin
>
> Acked-by: Maxime Coquelin
Maxime,
Once Acked by Felipe, I think this should go in via your tree to
avoid conflicts with other ARM/STI ARCHITECTURE additions.
> >---
* Nick Krause [140707 10:42]:
> Hey Tony,
> This seems to be a false positive then.Please reply if it is so I can remove
> the
> Fix me message.
Yes, just replace the FIXME with a comment please.
Regards,
Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body
Dear Matt and List,
Development of libusbg is going forward but there has been no review or
merge from Matt since April. There are three pending pull requests which
has been reviewed by list and since at least 4 weeks there was no
remarks for them.
Matt, I know that you are quite a busy guy and y
From: Julius Werner
...
> Also, really the only reason later kernels work for that controller is
> because we don't handle the case where a TD wraps all the way around a
> whole multi-segment ring back to the same segment.
That is allowed because it would then be impossible to add another
ring seg
From: David Laight
> From: Julius Werner
> ...
> > Also, really the only reason later kernels work for that controller is
> > because we don't handle the case where a TD wraps all the way around a
> > whole multi-segment ring back to the same segment.
>
> That is allowed because it would then be i
On 07/08/2014 09:53 AM, Lee Jones wrote:
On Tue, 08 Jul 2014, Maxime Coquelin wrote:
On 07/05/2014 08:25 AM, Peter Griffin wrote:
Signed-off-by: Peter Griffin
Acked-by: Maxime Coquelin
Maxime,
Once Acked by Felipe, I think this should go in via your tree to
avoid conflicts with other
Some of function attributes are read only on configfs, so library
should not try to set them and accept only empty values in functions
which set multiple attributes at single call.
Signed-off-by: Krzysztof Opasiak
---
src/usbg.c | 25 ++---
1 file changed, 18 insertions(+),
Const qualifers should be placed in all functions which
don't need to modify provided buffers to allow caller to
pass literals or pointers to non-modifable space.
Signed-off-by: Krzysztof Opasiak
---
include/usbg/usbg.h | 28 +++---
src/usbg.c | 79 ++
ether_aton() and ether_ntoa() use static buffers which are
overwritten by next call of this function so those functions
should not be used in multithread environment.
Signed-off-by: Krzysztof Opasiak
---
include/usbg/usbg.h |1 +
src/usbg.c | 16 ++--
2 files changed,
Dear Matt,
This is a small series which fix usbg_set_function_attrs()
which previously tried to set attributes which are read only
on configfs.
I have also went through code and marked attributes which should not
change during function calls with const qualifiers.
Third patch is a small enhancem
On Wed, Jul 2, 2014 at 3:28 PM, Yegor Yefremov
wrote:
> On Tue, Jul 1, 2014 at 5:37 PM, Ezequiel GarcĂa
> wrote:
>> On 1 July 2014 12:07, Yegor Yefremov wrote:
>> [..]
>>>
>>> What can be done with these error messages:
>>>
>>> of_get_named_gpiod_flags: can't parse gpios property of node
>>> '/o
On Mon, Jul 07, 2014 at 04:22:08PM +0500, Abbas Raza wrote:
> On 07/07/2014 02:37 PM, Michael Grzeschik wrote:
> >On Sun, Jul 06, 2014 at 11:47:50PM +0500, Abbas Raza wrote:
> >>On 07/06/2014 11:01 PM, Greg KH wrote:
> >>>On Sun, Jul 06, 2014 at 07:52:41PM +0200, Michael Grzeschik wrote:
> Hi,
-ENODEV is interpreted by the generic driver probing function as a
non-matching driver. This leads to a missing probe failure message.
Also a missing USB PHY is more of an invalid configuration of the usb
driver because it is necessary.
This patch returns -EINVAL if devm_usb_get_phy_by_phandle()
fsl,usbphy is no optional property. This patch moves it to the list of
required properties.
Signed-off-by: Markus Pargmann
---
Documentation/devicetree/bindings/usb/ci-hdrc-imx.txt | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/usb/ci-hdrc-i
Hello Himangi Saraogi,
The patch 9f7b23ce88e9: "usb: phy: phy-gpio-vbus-usb: use devm_
functions" from Jun 4, 2014, leads to the following static checker
warning:
drivers/usb/phy/phy-gpio-vbus-usb.c:264 gpio_vbus_probe()
warn: passing devm_ allocated variable to kfree. 'gpio_vbus'
Use SIMPLE_DEV_PM_OPS macro and remove DEV_PM_OPS macro, in order
to make the code simpler.
Signed-off-by: Jingoo Han
---
drivers/usb/phy/phy-am335x.c | 12 +++-
1 file changed, 3 insertions(+), 9 deletions(-)
diff --git a/drivers/usb/phy/phy-am335x.c b/drivers/usb/phy/phy-am335x.c
in
Dear Matt,
This quite big series adds new part of libusbg API which allows to
import/export gadget/function/configuration from/to file.
Motivation:
Libusbg allows to create a binary file which set up custom
gadget. This is useful but if we have to create custom binary for
each gadget we wast a l
Changes sent to libcofngi has not been merged yet,
so add compatibility defines to make all compile and work.
Signed-off-by: Krzysztof Opasiak
---
src/usbg.c |9 +
1 file changed, 9 insertions(+)
diff --git a/src/usbg.c b/src/usbg.c
index b17a707..e6e97bf 100644
--- a/src/usbg.c
+++
This library is used to import and export
gadget/function/config to and from file.
Signed-off-by: Krzysztof Opasiak
---
configure.ac|1 +
libusbg.pc.in |2 +-
src/Makefile.am |4 +++-
3 files changed, 5 insertions(+), 2 deletions(-)
diff --git a/configure.ac b/configure.ac
ind
Function settings may be complicated and their configuration
may take a long while. To save time it would be convenient
to allow for storing function settings in a file.
Signed-off-by: Krzysztof Opasiak
---
include/usbg/usbg.h | 11
src/usbg.c | 159 +
Add some field where additional label for function can be stored.
Signed-off-by: Krzysztof Opasiak
---
src/usbg.c |5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/src/usbg.c b/src/usbg.c
index e6e97bf..5730854 100644
--- a/src/usbg.c
+++ b/src/usbg.c
@@ -84,7 +84,8 @@ s
Whole gadget setting process take a lot of simple
commands (or lines of code). Those shell commands
may take a while or require dedicated script and
hard-coding gadget configuration. To avoid such
situation add to library ability to export a whole
gadget into file.
Signed-off-by: Krzysztof Opasiak
Configuration may have several functions if we add
strings and some attributes it gives a few shell commands
to set it up. To avoid this add export cofiguration
which allows to store usb configuration in a file.
Signed-off-by: Krzysztof Opasiak
---
include/usbg/usbg.h |8 ++
src/usbg.c
Add sample C code which shows how to use new functionality
of libusbg - gadget export. This program allows to export
chosen gadget from configfs to a file.
Signed-off-by: Krzysztof Opasiak
---
examples/Makefile.am |3 +-
examples/gadget-export.c | 81 +++
Signed-off-by: Krzysztof Opasiak
---
include/usbg/usbg.h |4
src/usbg.c | 24
2 files changed, 28 insertions(+)
diff --git a/include/usbg/usbg.h b/include/usbg/usbg.h
index 3a66ae3..3d3cba0 100644
--- a/include/usbg/usbg.h
+++ b/include/usbg/usbg.h
@@
If error occurred during parsing user should have
an oportunity to get details about place and type
of error.
Signed-off-by: Krzysztof Opasiak
---
src/usbg.c |8
1 file changed, 8 insertions(+)
diff --git a/src/usbg.c b/src/usbg.c
index f6841f4..62e66d8 100644
--- a/src/usbg.c
+++
Functions can be exported to file using usbg_export_function().
This commit adds complementary API function usbg_import_function()
which allows to import function from file to configfs.
Signed-off-by: Krzysztof Opasiak
---
include/usbg/usbg.h | 13
src/usbg.c | 187 +
Configurations can be exported to file using usbg_export_config().
This commit adds complementary API function usbg_import_config()
which allows to import configuration (with attrs, strings and
bindings) from file to chosen gadget.
Signed-off-by: Krzysztof Opasiak
---
include/usbg/usbg.h | 11
Whole gadget can be exported to file using usbg_export_gadget().
This commit adds complementary API function usbg_import_gadget()
which allows to import whole gadget (including attrs, strings,
configs, functions and bindings) from file to configfs.
Signed-off-by: Krzysztof Opasiak
---
include/us
If error occurred during parsing user should have
an opportunity to get details about place and type
of error.
Signed-off-by: Krzysztof Opasiak
---
src/usbg.c |7 +++
1 file changed, 7 insertions(+)
diff --git a/src/usbg.c b/src/usbg.c
index 9ba92fe..79e7ab4 100644
--- a/src/usbg.c
+++
Signed-off-by: Krzysztof Opasiak
---
include/usbg/usbg.h | 43 +++
src/usbg.c | 48
2 files changed, 91 insertions(+)
diff --git a/include/usbg/usbg.h b/include/usbg/usbg.h
index 23783eb..a934a7
Signed-off-by: Krzysztof Opasiak
---
examples/Makefile.am |3 +-
examples/gadget-import.c | 79 ++
src/usbg.c |6 ++--
3 files changed, 84 insertions(+), 4 deletions(-)
create mode 100644 examples/gadget-import.c
diff --git
Add document which clarify reasons of implementing
gadget schemes and also describes syntax of such files.
Signed-off-by: Krzysztof Opasiak
---
doc/gadget_schemes.txt | 301
1 file changed, 301 insertions(+)
create mode 100644 doc/gadget_schemes
This patch adds PID 0x0003 to the VID 0x128d (Testo). At least the
Testo 435-4 uses this, likely other gear as well.
Signed-off-by: Bert Vermeulen
---
drivers/usb/serial/ftdi_sio.c | 3 ++-
drivers/usb/serial/ftdi_sio_ids.h | 3 ++-
2 files changed, 4 insertions(+), 2 deletions(-)
diff --gi
On Mon, 7 Jul 2014, Steve Calfee wrote:
> You are right, the device will get 5 volts. However it is still part
> of the spec that 2ms without a SOF will force the device into suspend
> mode, so it will use less than 2ma. So depending on your definition of
> function, there will be power, but no ac
On 07.07.2014 11:39, Marek Szyprowski wrote:
> This patch adds support for Exynos3250 SoC to Exynos2USB PHY driver.
> Although Exynos3250 has only one device phy interface, the register
> layout and all operations that are required to get it enabled are almost
> same as on Exynos4x12. The only diff
The commit 5b7d70c6dbf2 ("USB: Gadget driver for Samsung HS/OtG
block") which added this driver also introduced the inactive DMA
support (due to hardware limitations of DMA buffers alignment).
It has been a dead code for over 5 years and should be removed.
We don't keep the unused/untested feature
hi Alan:
2014-07-08 1:02 GMT+08:00 Alan Stern :
> On Tue, 8 Jul 2014, loody wrote:
>
>> > An alternative is to write a positive value, such as 24, to
>> >
>> > /sys/block/sdX/events_poll_msecs
>> >
>> > where X is replaced with the proper drive letter. (24 ms = 240 s =
>> > 4 m.)
In preparation for DT conversion to reduce reliance on platform device
callbacks.
Signed-off-by: Ulrich Hecht
---
drivers/usb/renesas_usbhs/Makefile | 2 +-
drivers/usb/renesas_usbhs/common.c | 39 ---
drivers/usb/renesas_usbhs/rcar2.c | 76 +
Let the driver handle initialization and power control.
Signed-off-by: Ulrich Hecht
---
arch/arm/mach-shmobile/board-lager.c | 109 +--
1 file changed, 15 insertions(+), 94 deletions(-)
diff --git a/arch/arm/mach-shmobile/board-lager.c
b/arch/arm/mach-shmobile/b
On Tue, 8 Jul 2014, loody wrote:
> You are correct.
> i can see events_poll_msecs when I change my kernel to 3.8.0
> But the device still disconnect.
> I have some question.
> 1. when I cat /sys/block/sdX/events, I get the answer as media_change.
> is there any events type we can choose, such
OS String descriptor can be provided by using FunctionFS with configfs.
In order for FunctionFS to support Feature Descriptors, corresponding
Extended Compatibility and Extended Properties descriptors must be
passed to ep0. This series adds parsing the said descriptors and
setting up appropriate in
Add support for OS descriptors. The new format of descriptors is used,
because the "flags" field is required for extensions. os_count gives
the number of OSDesc[] elements.
The format of descriptors is given in include/uapi/linux/usb/functionfs.h.
For extended properties descriptor the usb_ext_pro
ffs_do_desc() handles one descriptor, while ffs_do_descs() handles a number
of descriptors. The tho names are so similar that it causes confusion.
Rename to reflect their purpose better.
Signed-off-by: Andrzej Pietrasiewicz
Acked-by: Michal Nazarewicz
---
drivers/usb/gadget/function/f_fs.c | 7
Provide helper functions to get pointers to particular locations within
a buffer holding an extended properties descriptor.
Signed-off-by: Andrzej Pietrasiewicz
---
drivers/usb/gadget/u_os_desc.h | 59 --
1 file changed, 46 insertions(+), 13 deletions(-)
Hello.
On 07/08/2014 02:14 PM, Markus Pargmann wrote:
fsl,usbphy is no optional property. This patch moves it to the list of
required properties.
Signed-off-by: Markus Pargmann
---
Documentation/devicetree/bindings/usb/ci-hdrc-imx.txt | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
On Tue, Jul 08 2014, Andrzej Pietrasiewicz wrote:
> Thank you for your review. One more chance for a better code ;)
> A v3 series is in order.
You're welcome. I think v3 may be the one. ;)
--
Best regards, _ _
.o. | Liege of Serenely Enlightened Maje
Hello.
On 07/08/2014 04:53 PM, Krzysztof Opasiak wrote:
Signed-off-by: Krzysztof Opasiak
[...]
diff --git a/src/usbg.c b/src/usbg.c
index 764cc03..f6841f4 100644
--- a/src/usbg.c
+++ b/src/usbg.c
[...]
@@ -272,6 +284,18 @@ const char *usbg_strerror(usbg_error e)
case USBG_ERROR_PA
On Tue, Jul 08 2014, Andrzej Pietrasiewicz wrote:
> Add support for OS descriptors. The new format of descriptors is used,
> because the "flags" field is required for extensions. os_count gives
> the number of OSDesc[] elements.
> The format of descriptors is given in include/uapi/linux/usb/functi
>> You are right, the device will get 5 volts. However it is still part
>> of the spec that 2ms without a SOF will force the device into suspend
>> mode, so it will use less than 2ma. So depending on your definition of
>> function, there will be power, but no activity. If it uses more than
>> 2ma,
On Tue, 8 Jul 2014, Grant wrote:
> >> You are right, the device will get 5 volts. However it is still part
> >> of the spec that 2ms without a SOF will force the device into suspend
> >> mode, so it will use less than 2ma. So depending on your definition of
> >> function, there will be power, but
From: Hayes Wang
Date: Thu, 3 Jul 2014 11:55:48 +0800
> When the system is too busy to complete the urb, the tx timout function
> would be called. This causes the other tx urbs would be killed, too.
> Increase the tx timeout to avoid it.
>
> Signed-off-by: Hayes Wang
Applied, thank you.
--
To
Commit 1f81b6d22 "usb: xhci: Prefer endpoint context dequeue pointer
over stopped_trb" changed the logic in xhci_find_new_dequeue_state() to
only use the Endpoint Context's TR Dequeue Pointer instead of the last
Event TRB's TRB Pointer as a basis from which to infer the host
controller state. This
hi Alan:
2014-07-09 0:00 GMT+08:00 Alan Stern :
> On Tue, 8 Jul 2014, loody wrote:
>
>> You are correct.
>> i can see events_poll_msecs when I change my kernel to 3.8.0
>> But the device still disconnect.
>> I have some question.
>> 1. when I cat /sys/block/sdX/events, I get the answer as media_c
On Wed, 9 Jul 2014, loody wrote:
> there is one thing pop in my mind.
> if events_poll_msecs is used for media_change, shouldn't we wrap is READ10?
No. Why should media change polling use READ(10)? TEST UNIT READY
does a good job of detecting media changes.
> the difference seems
> 1. change
> From: Bartlomiej Zolnierkiewicz [mailto:b.zolnier...@samsung.com]
> Sent: Tuesday, July 08, 2014 7:55 AM
>
> The commit 5b7d70c6dbf2 ("USB: Gadget driver for Samsung HS/OtG
> block") which added this driver also introduced the inactive DMA
> support (due to hardware limitations of DMA buffers al
I'm no longer the USB 3.0 maintainer. Please contact Mathias Nyman and
the Linux USB mailing list for help. They are on Cc.
Thanks,
Sarah Sharp
On Tue, Jun 24, 2014 at 11:14:50AM +0200, Carsten Koepp wrote:
> Hi Sarah,
>
> from what I've found on the internet your are the maintainer of the
> x
On Fri, Jun 20, 2014 at 9:58 AM, Julius Werner wrote:
>> +static const struct hc_driver tegra_xhci_hc_driver = {
>> + .description = "tegra-xhci-hcd",
>> + .product_desc = "Tegra xHCI Host Controller",
>> + .hcd_priv_size =sizeof(struct xhci_hcd *),
>> +
From: Olivier Sobrie
Date: Mon, 7 Jul 2014 11:06:07 +0200
> When the module sends bursts of data, sometimes a deadlock happens in
> the hso driver when the tty buffer doesn't get the chance to be flushed
> quickly enough.
>
> To avoid this, first, we remove the endless while loop in
> put_rx_bu
Hi Ulrich
Thank you for your patch
> In preparation for DT conversion to reduce reliance on platform device
> callbacks.
>
> Signed-off-by: Ulrich Hecht
> ---
(snip)
> @@ -186,6 +192,8 @@ struct renesas_usbhs_platform_info {
>* driver use these param for some register
>*/
>
This patch 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 is based on the usb-next branch of Greg usb.git tree.
(commit id = f8dd7a2a3a572dd93
The R-Car H2 and M2 SoCs come with an xHCI controller that requires
some specific initializations 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 S
This patch removes a fixme message in this file:wq for setting the usb 2
speed on the board to the correct level. We need to depend on the
bootloader for doing this as the wires may be shared for the other
things on the board with the usb chipset.
Signed-off-by: Nicholas Krause
---
arch/arm/mach
From: Hayes Wang
Date: Tue, 8 Jul 2014 14:49:28 +0800
> The device should be waked up from runtime suspend before dumping
> the hw counter.
>
> Signed-off-by: Hayes Wang
Applied, thanks.
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord.
hi Alan:
2014-07-09 3:26 GMT+08:00 Alan Stern :
> On Wed, 9 Jul 2014, loody wrote:
>
>> there is one thing pop in my mind.
>> if events_poll_msecs is used for media_change, shouldn't we wrap is READ10?
>
> No. Why should media change polling use READ(10)? TEST UNIT READY
> does a good job of det
On Tue, Jul 08, 2014 at 09:58:39PM -0400, Nicholas Krause wrote:
> This patch removes a fixme message in this file:wq for setting the usb 2
I don't think you did this correctly :)
> speed on the board to the correct level. We need to depend on the
> bootloader for doing this as the wires may be s
67 matches
Mail list logo