Hi,
On Sat, Sep 27, 2014 at 01:05:46AM +, Paul Zimmerman wrote:
> > From: Felipe Balbi [mailto:ba...@ti.com]
> > Sent: Friday, September 26, 2014 5:54 PM
> >
> > On Fri, Sep 26, 2014 at 11:18:48PM +, Paul Zimmerman wrote:
> > > > From: Felipe Balbi [mailto:ba...@ti.com]
> > > > Sent: Frid
> From: Felipe Balbi [mailto:ba...@ti.com]
> Sent: Friday, September 26, 2014 5:54 PM
>
> On Fri, Sep 26, 2014 at 11:18:48PM +, Paul Zimmerman wrote:
> > > From: Felipe Balbi [mailto:ba...@ti.com]
> > > Sent: Friday, September 26, 2014 2:40 PM
> > >
> > > On Fri, Sep 26, 2014 at 08:57:19PM +00
Hi,
On Fri, Sep 26, 2014 at 11:18:48PM +, Paul Zimmerman wrote:
> > From: Felipe Balbi [mailto:ba...@ti.com]
> > Sent: Friday, September 26, 2014 2:40 PM
> >
> > On Fri, Sep 26, 2014 at 08:57:19PM +, Paul Zimmerman wrote:
> > > > > diff --git a/drivers/usb/dwc3/gadget.c b/drivers/usb/dwc3
> From: Felipe Balbi [mailto:ba...@ti.com]
> Sent: Friday, September 26, 2014 2:40 PM
>
> On Fri, Sep 26, 2014 at 08:57:19PM +, Paul Zimmerman wrote:
> > > > diff --git a/drivers/usb/dwc3/gadget.c b/drivers/usb/dwc3/gadget.c
> > > > index 0fcc0a3..8277065 100644
> > > > --- a/drivers/usb/dwc3/
Hi,
On Fri, Sep 26, 2014 at 08:57:19PM +, Paul Zimmerman wrote:
> > > diff --git a/drivers/usb/dwc3/gadget.c b/drivers/usb/dwc3/gadget.c
> > > index 0fcc0a3..8277065 100644
> > > --- a/drivers/usb/dwc3/gadget.c
> > > +++ b/drivers/usb/dwc3/gadget.c
> > > @@ -2635,6 +2635,7 @@ static irqreturn_
Enable USB PHY device for the Henninger board.
Signed-off-by: Sergei Shtylyov
---
Changes in version 4:
- refreshed the patch.
Changes in version 3:
- refreshed the patch.
arch/arm/boot/dts/r8a7791-henninger.dts |4
1 file changed, 4 insertions(+)
Index: renesas/arch/arm/boot/dts/r8
Enable USB PHY device for the Koelsch board.
Signed-off-by: Sergei Shtylyov
---
Changes in version 4:
- refreshed the patch.
Changes in version 3:
- refreshed the patch.
Changes in version 2:
- fixed the wrong board name in the changelog;
- refreshed the patch.
arch/arm/boot/dts/r8a7791-koel
Define the R8A7791 generic part of the USB PHY device node. It is up to the
board file to enable the device.
Signed-off-by: Sergei Shtylyov
---
Changes in version 4:
- refreshed the patch.
Changes in version 3:
- changed subnodes of the USB PHY node to be per-channel;
- changed "renesas,phy-sel
Hello.
Here's the set of 3 patches against Simon Horman's 'renesas.git' repo,
'renesas-devel-20140924-v3.17-rc6' tag. Here we add the USB PHY device tree
support on the R8A7791/Koelsch/Henninger boards. The patchset requires the USB
PHY driver already merged by Kishon and Greg in order to work,
Enable USB PHY device for the Lager board.
Signed-off-by: Sergei Shtylyov
---
Changes in version 4:
- refreshed the patch.
Changes in version 3:
- refreshed the patch.
Changes in version 2:
- refreshed the patch.
arch/arm/boot/dts/r8a7790-lager.dts |4
1 file changed, 4 insertions(
Define the R8A7790 generic part of the USB PHY device node. It is up to the
board file to enable the device.
Signed-off-by: Sergei Shtylyov
---
Changes in version 4:
- refreshed the patch.
Changes in version 3:
- changed subnodes of the USB PHY node to be per-channel;
- changed "renesas,phy-sel
From: linux-usb-ow...@vger.kernel.org [mailto:linux-usb-ow...@vger.kernel.org]
On Behalf Of Felipe Balbi
Sent: Thursday, September 25, 2014 7:51 AM
> On Thu, Sep 25, 2014 at 03:21:46PM +0800, Huang Rui wrote:
> > diff --git a/drivers/usb/dwc3/gadget.c b/drivers/usb/dwc3/gadget.c
> > index 0fcc0a3
Hello.
Here's the set of 2 patches against Simon Horman's 'renesas.git' repo,
'renesas-devel-20140924-v3.17-rc6' tag. Here we add the USB PHY device tree
support on the R8A7790/Lager reference board. The patchset requires the USB PHY
driver already merged by Kishon and Greg in order to work, so
From: Hayes Wang
Date: Tue, 23 Sep 2014 16:31:47 +0800
> netif_carrier_off would be called when autoresuming, even though
> the cable is plugged. This causes some applications do relative
> actions when detecting the carrier off. Keep the status of the
> carrier, and let it be modified when the l
Add structure to store informations about available udcs
instead of using their names as a string.
Signed-off-by: Krzysztof Opasiak
---
include/usbg/usbg.h | 14
src/usbg.c | 94 +++
2 files changed, 108 insertions(+)
diff --
Add usbg_get_udc_gadget() which allows user to learn
which gadget is attached to selected UDC.
Signed-off-by: Krzysztof Opasiak
---
include/usbg/usbg.h |7 +++
src/usbg.c | 28
2 files changed, 35 insertions(+)
diff --git a/include/usbg/usbg.h b/i
Library allows to iterate over each gadget using loop.
To be consistent with this convention add similar loop
for UDCs.
Signed-off-by: Krzysztof Opasiak
---
include/usbg/usbg.h | 31 ---
src/usbg.c | 49 +++--
2
Kernel may decide to detach our gadget in some cases.
A good example is when FFS daemon received a SIGSEGV
and all descriptors has been closed.
To avoid returning false informations to user we have
to refresh UDC attribute before returning cached pointer
to usbg_udc.
Signed-off-by: Krzysztof Opas
This example shows how to learn what udcs are available
in system and also how to find out what gadgets are
enabled on them.
Signed-off-by: Krzysztof Opasiak
---
examples/Makefile.am |3 ++-
examples/show-udcs.c | 61 ++
2 files changed, 63 i
Using string as udc identifier provides a lot
of troubles. To be more consistent with rest of
API rework it to start using usbg_udc structure
instead of using char *.
Signed-off-by: Krzysztof Opasiak
---
examples/gadget-vid-pid-remove.c | 12 ++---
examples/show-gadgets.c | 15 -
Dear Matt,
This series adds usbg_udc structure to remove direct udc name usage.
Before this series UDC didn't have its own structure. Gadget enable
took const char * as parameter. Such approach is inconsisten with rest
of API.
This series introduce structure usbg_udc as abstraction of UDC. This
Users of library may often receive function type as a string.
Instead of implementing parsing of such string in each program
it would be more convenient to provide such support in library.
Signed-off-by: Krzysztof Opasiak
---
include/usbg/usbg.h |7 +++
src/usbg.c |2 +-
2 f
Signed-off-by: Krzysztof Opasiak
---
include/usbg/usbg.h | 40 +++-
1 file changed, 19 insertions(+), 21 deletions(-)
diff --git a/include/usbg/usbg.h b/include/usbg/usbg.h
index a934a7b..fe0e019 100644
--- a/include/usbg/usbg.h
+++ b/include/usbg/usbg.h
@@
Having specialized functions for each attribute may be
sometime inconvenient to use. Provide also function
whihc allows to set attribute selected by parameter.
Signed-off-by: Krzysztof Opasiak
---
include/usbg/usbg.h | 53 +
src/usbg.c | 73 ++
Dear Matt,
This series fixes a lot of bugs and makes API usage easier.
First two commits contains contains a difference between
libubsg/libusbg and kopasiak/libusbg repository.
List of fixed issues:
- doxygen typos
- Off-by-one error in usbg_get_function_type_str()
- remove unused variables
- Al
From: Philippe De Swert
Signed-off-by: Philippe De Swert
Acked-by: Krzysztof Opasiak
Signed-off-by: Krzysztof Opasiak
Update commit message.
Signed-off-by: Krzysztof Opasiak
---
src/usbg.c |4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/usbg.c b/src/usbg.c
ind
Replace ERRORNO() macro with ERROR(). Difference betwen
them is that ERRORNO() prints also standard error string
based on errno value. This variable should not be accessed
when called function doesn't directly set it.
Signed-off-by: Krzysztof Opasiak
---
src/usbg.c |8
1 file change
Signed-off-by: Krzysztof Opasiak
---
src/usbg.c | 13 ++---
1 file changed, 2 insertions(+), 11 deletions(-)
diff --git a/src/usbg.c b/src/usbg.c
index f12a4b3..9c9b51f 100644
--- a/src/usbg.c
+++ b/src/usbg.c
@@ -3125,7 +3125,6 @@ static int usbg_export_gadget_prep(usbg_gadget *g,
co
strncpy() may not append trailing '\0' so let's append
it always at end of string to avoid getting into troubles.
Signed-off-by: Krzysztof Opasiak
---
src/usbg.c | 72 ++--
1 file changed, 36 insertions(+), 36 deletions(-)
diff --git a/s
Signed-off-by: Krzysztof Opasiak
---
src/usbg.c |9 +++--
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/src/usbg.c b/src/usbg.c
index cd2179e..4d74a0d 100644
--- a/src/usbg.c
+++ b/src/usbg.c
@@ -316,18 +316,15 @@ static int usbg_lookup_function_type(const char *name)
Signed-off-by: Krzysztof Opasiak
---
src/usbg.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/usbg.c b/src/usbg.c
index bbba689..d2ec246 100644
--- a/src/usbg.c
+++ b/src/usbg.c
@@ -2058,7 +2058,7 @@ int usbg_create_config(usbg_gadget *g, int id, const char
*label,
Signed-off-by: Krzysztof Opasiak
---
src/usbg.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/usbg.c b/src/usbg.c
index d2ec246..f12a4b3 100644
--- a/src/usbg.c
+++ b/src/usbg.c
@@ -319,7 +319,7 @@ static int usbg_lookup_function_type(const char *name)
const const
Signed-off-by: Krzysztof Opasiak
---
AUTHORS |2 ++
1 file changed, 2 insertions(+)
diff --git a/AUTHORS b/AUTHORS
index f9b069c..6fea705 100644
--- a/AUTHORS
+++ b/AUTHORS
@@ -1 +1,3 @@
Matt Porter
+Krzysztof Opasiak
+
--
1.7.9.5
--
To unsubscribe from this list: send the line "unsubsc
Signed-off-by: Krzysztof Opasiak
Changes since v1:
Fix show-gadgets example: get attributes before dereferencing them
Reported-by: Philippe De Swert
Signed-off-by: Krzysztof Opasiak
---
examples/show-gadgets.c | 42
include/usbg/usbg.h | 70 +++
Hi,
On Fri, Sep 26, 2014 at 11:15:03AM -0500, Felipe Balbi wrote:
> On Fri, Sep 26, 2014 at 12:06:00PM -0400, Alan Stern wrote:
> > > Any hints will be appreciated.
> >
> > We could implement READ CAPACITY(16) if necessary. But since we don't
> > implement READ(16) or WRITE(16), it seems rather
On Fri, Sep 26, 2014 at 12:06:00PM -0400, Alan Stern wrote:
> > Any hints will be appreciated.
>
> We could implement READ CAPACITY(16) if necessary. But since we don't
> implement READ(16) or WRITE(16), it seems rather pointless.
>
> Are you certain you are interpreting the USBCV output correc
On Fri, 26 Sep 2014, Felipe Balbi wrote:
> Hi Alan,
>
> it seems like newer versions of USB20CV will fail us on the MSC tests
> unless we support READ_CAPACITY(16).
How is that possible? The USB Mass-Storage Class spec doesn't require
the device to support any particular set of SCSI commands.
On Thu, Sep 25, 2014 at 06:12:51PM -0500, Felipe Balbi wrote:
> Hi,
>
> On Thu, Sep 25, 2014 at 09:57:59PM +, Paul Zimmerman wrote:
> > > From: Felipe Balbi [mailto:ba...@ti.com]
> > >
> > > On Thu, Sep 25, 2014 at 07:08:12PM +, Paul Zimmerman wrote:
> > > > > From: Alan Stern [mailto:st.
Hi,
On Fri, Sep 26, 2014 at 09:20:54AM +0200, Arnd Bergmann wrote:
> On Thursday 25 September 2014 19:39:34 Felipe Balbi wrote:
> > >
> > > why would a glue layer need to access registers from the core ? That
> > > sounds very odd. I haven't seen that and will, definitely, NACK such a
> > > patch
Hi Alan,
it seems like newer versions of USB20CV will fail us on the MSC tests
unless we support READ_CAPACITY(16).
There is one detail though, looking at the sniffer, CBW comes as:
55 53 42 43 43 59 45 4C 00 00 00 00 00 00 0A 2F 00 00
00 00 00 00 00 80 00 00 00 00 00 00 00
so SCSI cmd opcode i
Hi,
On Fri, Sep 26, 2014 at 09:35:21AM -0500, Felipe Balbi wrote:
> This is just because I don't have access to the HW myself, so I can't
> verify your patches. One thing I can tell you, with my testing/next,
> dwc3 is really stable. I have every test passing except for Halt
> Endpoint which I'm d
On Fri, 26 Sep 2014, Michael Grzeschik wrote:
> The current EHCI implementation is prepared to toggle the
> PORT_POWER bit to enable or disable a USB-Port. In some
> cases this port power can not be toggled by the PORT_POWER
> bit but instead i.e. by an external GPIO.
> This patch adds an callback
On Fri, Sep 26, 2014 at 04:50:26PM +0800, Huang Rui wrote:
> On Thu, Sep 25, 2014 at 09:50:32AM -0500, Felipe Balbi wrote:
> > Hi,
> >
> > On Thu, Sep 25, 2014 at 03:21:46PM +0800, Huang Rui wrote:
> > > diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c
> > > index b0f4d52..6138c5d 10
Hi,
On Fri, Sep 26, 2014 at 05:22:09PM +0800, Huang Rui wrote:
> > What you need here is something to talk to usbcore and udc-core and
> > orchestrate the mode change through usb_add_hcd()/usb_add_gadget_udc()
> > and their counterparts.
> >
> > George Cherian is already working on a version of t
This patch adds an external portpower override callback.
It can be used by the ehci-hub to toggle the PORT_POWER
by the registered regulator. That is needed when an external
GPIO is used instead of the default PORT_POWER bit.
Signed-off-by: Michael Grzeschik
---
drivers/usb/chipidea/host.c | 69
This series adds an portpower callback that can be triggered additionaly
to the PORT_POWER ehci bit. With that we are able to toggle an external
registered regulator i.e. fixed GPIO.
The first user is the chipidea-host to toggle the regulator.
This series is based on v3.17-rc6.
Thanks,
Michael
The current EHCI implementation is prepared to toggle the
PORT_POWER bit to enable or disable a USB-Port. In some
cases this port power can not be toggled by the PORT_POWER
bit but instead i.e. by an external GPIO.
This patch adds an callback to be triggered as well.
The host needs to assign a capa
On Fri, 26 Sep 2014, Johan Hovold wrote:
> Use mfd_add_hotplug_devices helper to register the subdevices.
>
> Compile-only tested.
>
> Signed-off-by: Johan Hovold
As I expect this to be going through MFD tree together with the rest of
the patchset
Acked-by: Jiri Kosina
> ---
> driv
Allow more than one viperboard to be connected by registering with
PLATFORM_DEVID_AUTO instead of PLATFORM_DEVID_NONE.
The subdevices are currently registered with PLATFORM_DEVID_NONE, which
will cause a name collision on the platform bus when a second viperboard
is plugged in:
viperboard 1-2.4:1
Use mfd_add_hotplug_devices helper to register the subdevices.
Compile-only tested.
Signed-off-by: Johan Hovold
---
drivers/hid/hid-sensor-hub.c | 8 +++-
1 file changed, 3 insertions(+), 5 deletions(-)
diff --git a/drivers/hid/hid-sensor-hub.c b/drivers/hid/hid-sensor-hub.c
index 2ac25760
Hot-pluggable multi-function devices should always be registered with
PLATFORM_DEVID_AUTO to avoid name collisions on the platform bus. This
helper also hides the memory map and irq parameters, which aren't used
by hot-pluggable (e.g. USB-based) devices.
Signed-off-by: Johan Hovold
---
include/l
Use mfd_add_hotplug_devices helper to register the subdevices.
Signed-off-by: Johan Hovold
---
drivers/mfd/rtsx_usb.c | 4 ++--
drivers/mfd/viperboard.c | 5 ++---
2 files changed, 4 insertions(+), 5 deletions(-)
diff --git a/drivers/mfd/rtsx_usb.c b/drivers/mfd/rtsx_usb.c
index 78073e4b87e4.
Hot-pluggable multi-function devices should use PLATFORM_DEVID_AUTO to
avoid name collisions on the platform bus.
This driver currently uses the USB-device address as an id. This makes
name collisions unlikely, but it could still happen if two devices are
connected to separate buses and gets assig
Make sure to always honour multi-function devices registered with
PLATFORM_DEVID_NONE (-1) or PLATFORM_DEVID_AUTO (-2) as id base. In this
case it does not make sense to append the cell id to the mfd-id base and
potentially change the requested behaviour.
Specifically this will allow multi-functio
Hot-pluggable multi-function devices should always be registered with
PLATFORM_DEVID_AUTO to avoid name collisions on the platform bus.
This series fix the two mfd drivers that currently fail to get this
right, and also adds a new helper function to assist any future driver
authors.
Included is a
On Fri, Sep 26, 2014 at 12:26:05AM -0500, Felipe Balbi wrote:
> Hi,
>
> On Fri, Sep 26, 2014 at 12:53:21PM +0800, Huang Rui wrote:
> > On Thu, Sep 25, 2014 at 09:53:55AM -0500, Felipe Balbi wrote:
> > > On Thu, Sep 25, 2014 at 03:21:44PM +0800, Huang Rui wrote:
> > > > Add PCI device ID of AMD NL.
On Thu, Sep 25, 2014 at 09:53:20AM -0500, Felipe Balbi wrote:
> Hi,
>
> On Thu, Sep 25, 2014 at 03:21:47PM +0800, Huang Rui wrote:
> > This patch implemented a feature to dynamic switch to host or device
> > role under debugfs for some physical limitation that unable to
> > leverage connector A/B
On Thu, Sep 25, 2014 at 09:50:32AM -0500, Felipe Balbi wrote:
> Hi,
>
> On Thu, Sep 25, 2014 at 03:21:46PM +0800, Huang Rui wrote:
> > diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c
> > index b0f4d52..6138c5d 100644
> > --- a/drivers/usb/dwc3/core.c
> > +++ b/drivers/usb/dwc3/core.
On Thursday 25 September 2014 19:39:34 Felipe Balbi wrote:
> >
> > why would a glue layer need to access registers from the core ? That
> > sounds very odd. I haven't seen that and will, definitely, NACK such a
> > patch
> >
> > can you further describe why you think a glue layer might need to a
On Friday 26 September 2014 08:23:40 Peter Chen wrote:
> In current chipidea structure, the parent (glue layer) driver will not be
> used for dma, udc/host driver uses dma mask from child (core layer), at core
> layer we will do:
>
>
> pdev->dev.dma_mask = dev->dma_mask; /* this device is
60 matches
Mail list logo
