HI,
On Tue, Mar 26, 2013 at 09:24:30PM +0100, Michael Grzeschik wrote:
> On Tue, Mar 26, 2013 at 07:55:46PM +0200, Felipe Balbi wrote:
> > Hi,
> >
> > On Tue, Mar 26, 2013 at 05:58:45PM +0100, Michael Grzeschik wrote:
> > > A static count of transfer descriptors was used everywhere in the driver
On Tue, Mar 26, 2013 at 01:37:41PM -0700, Greg KH wrote:
> On Tue, Mar 26, 2013 at 09:25:35PM +0100, Roland Stigge wrote:
> > On 03/26/2013 08:24 PM, Felipe Balbi wrote:
> > > commit ?
> >
> > 1c2088812f095df77f4b3224b65db79d7111a300
>
> Showed up in 3.9-rc3.
>
> Felipe, I can take this directly
On 03/27/2013 02:35 AM, Peter Chen wrote:
> On Tue, Mar 26, 2013 at 12:35:44PM +0200, Alexander Shishkin wrote:
>>
>> I'm thinking now that what Felipe suggests is probably the best at least
>> for the time being. That is, if we don't want host, for example, in the
>> product build, we just compile
Felipe Balbi writes:
> HI,
>
> On Tue, Mar 26, 2013 at 09:24:30PM +0100, Michael Grzeschik wrote:
>> On Tue, Mar 26, 2013 at 07:55:46PM +0200, Felipe Balbi wrote:
>> > Hi,
>> >
>> > On Tue, Mar 26, 2013 at 05:58:45PM +0100, Michael Grzeschik wrote:
>> > > A static count of transfer descriptors w
On Tue, Mar 26, 2013 at 03:50:47PM -0700, Greg KH wrote:
> On Tue, Mar 26, 2013 at 11:57:48PM +0200, Alexander Shishkin wrote:
> > Also, when you resend this patchset, *please* make sure all the patches
> > in it have the same version number, which is greater than *any* of these
> > patches had bef
Hi,
>> Understand, UDC driver will call driver->setup(), and if the return
>> value is negative, UDC driver has to set dev->protocol_stall = 1 and
>> maybe call usb_ep_set_halt(). However, the hardware won't be able to
>> send out zero length response.
>
> Don't be silly; of course it can. Nobody
Hi folks,
> I don't think we are consistent in any way. PowerPC sets up a 32 bit
> DMA mask for all devices during DT probe from arch code, while the
> common code sets up coherent_dma_mask but not dma_mask, except
> for AMBA devices, which also get the 32 bit mask.
>
> The MIPS Octeon and PowerP
On Tue, Mar 26, 2013 at 07:49:07PM +0200, Felipe Balbi wrote:
> Hi,
>
> On Tue, Mar 26, 2013 at 05:58:42PM +0100, Michael Grzeschik wrote:
> > The implementation is derived from the fsl_udc_core code in
> > fsl_ep_enable and makes basic iso handling possible.
> >
> > Signed-off-by: Michael Grzesc
Hi folks,
while going through the dwc2 driver, I've found that there is a lot of
places that need info from the hwcfg registers. Currently, each of these
places do their own bitshifting and masking, but it occurs to me that
the code would become cleaner if all this happened in single place, so
all
Hi
I am observing issue while streaming video from usb camera connected to host
controller based on mentor graphics. The issue is root caused that there are
huge SOF gaps seen between the two isochronous IN token issued by host
controller. This is due to fact, significant amount of time is spen
On 26/03/13 20:23, Stephen Warren wrote:
> On 03/26/2013 10:57 AM, Graeme Gregory wrote:
>> On 26/03/13 16:22, Stephen Warren wrote:
>>> On 03/26/2013 03:27 AM, Graeme Gregory wrote:
>>> ...
If we are tightly coupling as above then using platform_irq is an extra
inefficiency. You both hav
On Tue, Mar 26, 2013 at 02:08:05PM +0200, Alexander Shishkin wrote:
> Michael Grzeschik writes:
> > Hi Alexander, Fabio, Greg,
[...]
> > 618bfec Revert "USB: chipidea: add vbus detect for udc"
>
> This one, though, I will pick.
I miss that patch in your posted series.
--
Pengutronix e.K.
Hi,
On Fri, Mar 22, 2013 at 09:30:17AM +0200, Felipe Balbi wrote:
> this driver compiles fine everywhere which
> means we can use linux-next to compile it
> for us frequently.
>
> By dropping the arch dependency, we also
> ensure driver writers don't add virtual
> arch-depdencies to the driver by
Hi,
On Mon, Mar 25, 2013 at 03:06:51AM -0400, Chao Xie wrote:
> The clock is defined by device, so the driver knows how many
> clocks needed by the device.
> The orignal way that passing the clock name by pdata is not correct.
> The following patches fix it.
>
> V2->V1:
>typo fix
>rebased
Hi,
On Thu, Mar 14, 2013 at 08:12:09PM +0200, Ruslan Bilovol wrote:
> MUSB controller cannot work in DMA mode with misaligned buffers,
> switching in PIO mode.
>
> HCD core has hooks that allow to override the default DMA
> mapping and unmapping routines for host controllers that have
> special D
Hi,
On Tue, Mar 26, 2013 at 03:53:12PM +0100, Tomasz Figa wrote:
> @@ -307,6 +310,7 @@ static int samsung_usb3phy_remove(struct platform_device
> *pdev)
> static struct samsung_usbphy_drvdata usb3phy_exynos5 = {
> .cpu_type = TYPE_EXYNOS5250,
> .devphy_en_mask =
Le 26/03/2013 17:30, Sarah Sharp a écrit :
> On Tue, Mar 26, 2013 at 12:11:13PM +0100, Yann Sionneau wrote:
>> Le 25/03/2013 19:13, Sarah Sharp a écrit :
>>> On Mon, Mar 25, 2013 at 05:43:40PM +0100, Yann Sionneau wrote:
>>> Please compile with CONFIG_USB_DEBUG and CONFIG_USB_XHCI_HCD_DEBUGGING
>>
Hi Felipe,
On Wednesday 27 of March 2013 15:19:58 Felipe Balbi wrote:
> Hi,
>
> On Tue, Mar 26, 2013 at 03:53:12PM +0100, Tomasz Figa wrote:
> > @@ -307,6 +310,7 @@ static int samsung_usb3phy_remove(struct
> > platform_device *pdev)>
> > static struct samsung_usbphy_drvdata usb3phy_exynos5 = {
Hi,
On Wed, Mar 27, 2013 at 02:26:08PM +0100, Tomasz Figa wrote:
> Hi Felipe,
>
> On Wednesday 27 of March 2013 15:19:58 Felipe Balbi wrote:
> > Hi,
> >
> > On Tue, Mar 26, 2013 at 03:53:12PM +0100, Tomasz Figa wrote:
> > > @@ -307,6 +310,7 @@ static int samsung_usb3phy_remove(struct
> > > platf
Hi,
On Thu, Mar 21, 2013 at 01:56:07PM +0530, Bhupesh Sharma wrote:
> This patchset tries to enhance the UVC webcam gadget driver and is based on
> Laurent's git tree available here (head uvc-gadget):
> git://linuxtv.org/pinchartl/uvcvideo.git
>
> Note that to ease review and integration of these
On Wednesday 27 of March 2013 15:31:49 Felipe Balbi wrote:
> Hi,
>
> On Wed, Mar 27, 2013 at 02:26:08PM +0100, Tomasz Figa wrote:
> > Hi Felipe,
> >
> > On Wednesday 27 of March 2013 15:19:58 Felipe Balbi wrote:
> > > Hi,
> > >
> > > On Tue, Mar 26, 2013 at 03:53:12PM +0100, Tomasz Figa wrote:
>
Hi Kuninori,
Renesas USB driver constantly gives me sparse errors:
linux/drivers/usb/renesas_usbhs/common.c:313:17: error: incompatible types in
conditional expression (different base types)
linux/drivers/usb/renesas_usbhs/common.c:322:17: error: incompatible types in
conditional expression (di
Hi,
On Wed, Mar 27, 2013 at 02:39:08PM +0100, Tomasz Figa wrote:
> On Wednesday 27 of March 2013 15:31:49 Felipe Balbi wrote:
> > Hi,
> >
> > On Wed, Mar 27, 2013 at 02:26:08PM +0100, Tomasz Figa wrote:
> > > Hi Felipe,
> > >
> > > On Wednesday 27 of March 2013 15:19:58 Felipe Balbi wrote:
> > >
Michael Grzeschik writes:
> On Tue, Mar 26, 2013 at 02:08:05PM +0200, Alexander Shishkin wrote:
>> Michael Grzeschik writes:
>> > Hi Alexander, Fabio, Greg,
>
> [...]
>
>> > 618bfec Revert "USB: chipidea: add vbus detect for udc"
>>
>> This one, though, I will pick.
>
> I miss that patch in you
On Wed, 27 Mar 2013, victor yeo wrote:
> Thanks, the halt feature is ok now. The SCSI_READ_10 command has a
> problem. The reply value from do_read is -5, which means -EIO. The
> for(;;) loop in do_read() was probably broken at "if (amount_left ==
> 0)". Is this if-statement valid?
Yes, it is. d
Hi Felipe,
On Wednesday 27 March 2013 15:33:15 Felipe Balbi wrote:
> On Thu, Mar 21, 2013 at 01:56:07PM +0530, Bhupesh Sharma wrote:
> > This patchset tries to enhance the UVC webcam gadget driver and is based
> > on Laurent's git tree available here (head uvc-gadget):
> > git://linuxtv.org/pincha
On Wed, Mar 27, 2013 at 10:48:17AM +0100, Matthijs Kooijman wrote:
> Hi folks,
>
> while going through the dwc2 driver, I've found that there is a lot of
> places that need info from the hwcfg registers. Currently, each of these
> places do their own bitshifting and masking, but it occurs to me th
Hi Greg,
> But yes, it does seem to be a good idea,
Ok, thanks.
> as long as these options can't change over time.
It's only read-only registers, or registers whose power-on values
determine their maximum possible value, so they indeed won't change over
time.
Gr.
Matthijs
--
To unsubscribe fro
Its not necessary and also not specified in the datasheet to change the
ZLT flag before every ep_prime. This patch moves this to the ep_enable
and applies it only for non configuration endpoints.
Cc: stable
Signed-off-by: Michael Grzeschik
---
drivers/usb/chipidea/udc.c | 3 ++-
1 file changed,
Hi,
this series solve some issues with the chipidea udc.
The series is based on v3.9-rc4
Michael Grzeschik (6):
usb: chipidea: udc: add attribute aligned(4) to shared memory structs
usb: chipidea: udc: only clear active and halted bits in qhead
usb: chipidea: udc: move ZLT flag change to e
It is not safe to truncate requests to the maximum possible size the
controller can handle with one td and to keep working. That patch fixes
that with proper error handling instead.
Cc: stable
Signed-off-by: Michael Grzeschik
---
drivers/usb/chipidea/udc.c | 4 ++--
1 file changed, 2 insertions
This patch changes the read of the td status to one atomic operation to
analyse coherent bits.
Cc: stable
Signed-off-by: Michael Grzeschik
---
drivers/usb/chipidea/udc.c | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/drivers/usb/chipidea/udc.c b/drivers/usb/chipidea
The datasheet of the synopsys core describes only to overwrite the
active and halted bits in the qhead before priming any endpoint.
Cc: stable
Signed-off-by: Michael Grzeschik
---
drivers/usb/chipidea/udc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/usb/chipidea
The udc uses an shared dma memory space between hard and software. This
memory layout is described in ci13xxx_qh and ci13xxx_td which are marked
with the attribute ((packed)).
The compiler currently does not know about the alignment of the memory
layout, and will create strb and ldrb operations.
There is no need to call ep_queue unlocked inside the own driver. We
move its functionionality into an unlocked version.
This patch removes potential unlocked timeslots inside
isr_setup_status_phase and isr_get_status_response, in which the lock
got released just before acquired again inside usb_e
Michael Grzeschik writes:
> Hi,
>
> this series solve some issues with the chipidea udc.
>
> The series is based on v3.9-rc4
Didn't you forget Felipe's reviewed-bys?
Regards,
--
Alex
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vg
On Wed, Mar 27, 2013 at 06:06:51PM +0200, Alexander Shishkin wrote:
> Michael Grzeschik writes:
>
> > Hi,
> >
> > this series solve some issues with the chipidea udc.
> >
> > The series is based on v3.9-rc4
>
> Didn't you forget Felipe's reviewed-bys?
Yes, i will repost v7 in a minute.
Thanks,
Its not necessary and also not specified in the datasheet to change the
ZLT flag before every ep_prime. This patch moves this to the ep_enable
and applies it only for non configuration endpoints.
Cc: stable
Signed-off-by: Michael Grzeschik
Reviewed-by: Felipe Balbi
---
drivers/usb/chipidea/udc
There is no need to call ep_queue unlocked inside the own driver. We
move its functionionality into an unlocked version.
This patch removes potential unlocked timeslots inside
isr_setup_status_phase and isr_get_status_response, in which the lock
got released just before acquired again inside usb_e
It is not safe to truncate requests to the maximum possible size the
controller can handle with one td and to keep working. That patch fixes
that with proper error handling instead.
Cc: stable
Signed-off-by: Michael Grzeschik
---
drivers/usb/chipidea/udc.c | 4 ++--
1 file changed, 2 insertions
Hi,
this series solve some issues with the chipidea udc.
The series is based on v3.9-rc4.
Michael Grzeschik (6):
usb: chipidea: udc: add attribute aligned(4) to shared memory structs
usb: chipidea: udc: only clear active and halted bits in qhead
usb: chipidea: udc: move ZLT flag change to
The datasheet of the synopsys core describes only to overwrite the
active and halted bits in the qhead before priming any endpoint.
Cc: stable
Signed-off-by: Michael Grzeschik
Reviewed-by: Felipe Balbi
---
drivers/usb/chipidea/udc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff -
The udc uses an shared dma memory space between hard and software. This
memory layout is described in ci13xxx_qh and ci13xxx_td which are marked
with the attribute ((packed)).
The compiler currently does not know about the alignment of the memory
layout, and will create strb and ldrb operations.
This patch changes the read of the td status to one atomic operation to
analyse coherent bits.
Cc: stable
Signed-off-by: Michael Grzeschik
---
drivers/usb/chipidea/udc.c | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/drivers/usb/chipidea/udc.c b/drivers/usb/chipidea
On 26 March 2013 18:50, Alan Stern wrote:
> On Tue, 26 Mar 2013, David Linares wrote:
>
>> >> The device I am working on is an embedded device, therefore the wireless
>> >> adapter has to be connected to the internal hub. This hub and the whole
>> >> device
>> >> are powered via an usb cable.
>>
Michael Grzeschik writes:
> There is no need to call ep_queue unlocked inside the own driver. We
> move its functionionality into an unlocked version.
>
> This patch removes potential unlocked timeslots inside
> isr_setup_status_phase and isr_get_status_response, in which the lock
> got released
On Wed, Mar 27, 2013 at 05:21:13PM +0100, Michael Grzeschik wrote:
> The udc uses an shared dma memory space between hard and software. This
> memory layout is described in ci13xxx_qh and ci13xxx_td which are marked
> with the attribute ((packed)).
>
> The compiler currently does not know about th
Michael Grzeschik writes:
> Hi,
>
> this series solve some issues with the chipidea udc.
>
> The series is based on v3.9-rc4.
>
> Michael Grzeschik (6):
> usb: chipidea: udc: add attribute aligned(4) to shared memory structs
> usb: chipidea: udc: only clear active and halted bits in qhead
>
On 03/27/2013 05:35 PM, Alexander Shishkin wrote:
> Michael Grzeschik writes:
>
>> Hi,
>>
>> this series solve some issues with the chipidea udc.
>>
>> The series is based on v3.9-rc4.
>>
>> Michael Grzeschik (6):
>> usb: chipidea: udc: add attribute aligned(4) to shared memory structs
>> usb
On 03/27/2013 05:37 PM, Marc Kleine-Budde wrote:
> On 03/27/2013 05:35 PM, Alexander Shishkin wrote:
>> Michael Grzeschik writes:
>>
>>> Hi,
>>>
>>> this series solve some issues with the chipidea udc.
>>>
>>> The series is based on v3.9-rc4.
>>>
>>> Michael Grzeschik (6):
>>> usb: chipidea: udc
On Wed, Mar 27, 2013 at 06:35:30PM +0200, Alexander Shishkin wrote:
> Michael Grzeschik writes:
>
> > Hi,
> >
> > this series solve some issues with the chipidea udc.
> >
> > The series is based on v3.9-rc4.
> >
> > Michael Grzeschik (6):
> > usb: chipidea: udc: add attribute aligned(4) to shar
PATCH 1 is for code clean up.
PATCH 2~3 is to introduce usb port power off new feature
PATCH 4 is to fix usb port doesn't register to any bus_type
usb: add usb_enable/disable_remote_wakeup()
usb: introduce usb force power off mechanism
usb: Add usb port system pm support
usb: register usb port to
usb2.0 and usb3.0 devices have different ways to enalbe/disable
remote wakeup. This patch is to put both their operations into
the seperate functions. Otherwise, usb_control_msg() has some
long arguments and are usually nested some indentations. So
encapsulating it into seperate functions would be
This patch is to add usb port system pm support. Add
usb port's system suspend/resume callbacks and call
usb_port_runtime_resume/suspend() to power off these
ports whose pm qos NO_POWER_OFF flag is not set, system
wakeup is disabled and persist is enalbed.
During system pm, usb port should be powe
Usb port isn't assigned to any bus_type. This seems not good from
Greg's comments.
http://marc.info/?l=linux-usb&m=136200364929942&w=2
This patch is to register usb port to usb_bus_type. The usb port's
original name is "portX". This will cause name confilct after adding
usb port to usb_bus
Some devices' firmware will be broken at some points. Power down
and power on device can help device to rework in this case.
This patch is to add ioctl cmd USBDEVFS_POWER_RESET for usbfs node
to repower usb device. First, call hub_port_logical_disconnect() to
disconnect device. Second, Power down
On Wed, Mar 27, 2013 at 09:31:33AM -0700, Greg KH wrote:
> On Wed, Mar 27, 2013 at 05:21:13PM +0100, Michael Grzeschik wrote:
> > The udc uses an shared dma memory space between hard and software. This
> > memory layout is described in ci13xxx_qh and ci13xxx_td which are marked
> > with the attribu
A small tool to test this patch.
#include
#include
#include
#include
#include
#include
#define USBDEVFS_POWER_RESET _IO('U', 28)
int main(int argc, char **argv)
{
const char *filename;
int fd;
int rc;
if (argc != 2) {
fprintf(stderr, "
Hi Bhupesh,
Thanks for the patch. Please see below for a couple of small comments.
On Thursday 21 March 2013 13:56:08 Bhupesh Sharma wrote:
> This patch reworks the videobuffer management logic present in the UVC
> webcam gadget and ports it to use the "more apt" videobuf2 framework for
> video b
Hi Bhupesh,
Thanks for the patch.
On Thursday 21 March 2013 13:56:09 Bhupesh Sharma wrote:
> This patch adds the support for 'get_unmapped_area' in UVC gadget
> which is called when the 'mmap' system call is executed on MMUless
> architectures.
>
> Signed-off-by: Bhupesh Sharma
Acked-by: Laure
Hi Ravi,
On Wednesday 27 March 2013 10:43:24 B, Ravi wrote:
> Hi
>
> I am observing issue while streaming video from usb camera connected to host
> controller based on mentor graphics. The issue is root caused that there
> are huge SOF gaps seen between the two isochronous IN token issued by host
On Wed, 27 Mar 2013, David Linares wrote:
> Thanks Alan. I understand, but the USB cable is plugged into an AC/DC
> adaptor which can provide an output of 1A. After breaking out an USB
> cable, I measured that the consumption of my board is around 130mA
> (nothing connected to the hub). Then I mea
On Thu, 28 Mar 2013, Lan Tianyu wrote:
> usb2.0 and usb3.0 devices have different ways to enalbe/disable
> remote wakeup. This patch is to put both their operations into
> the seperate functions. Otherwise, usb_control_msg() has some
> long arguments and are usually nested some indentations. So
>
On Thu, 28 Mar 2013, Lan Tianyu wrote:
> Some devices' firmware will be broken at some points. Power down
> and power on device can help device to rework in this case.
>
> This patch is to add ioctl cmd USBDEVFS_POWER_RESET for usbfs node
> to repower usb device. First, call hub_port_logical_disc
On Thu, 28 Mar 2013, Lan Tianyu wrote:
> This patch is to add usb port system pm support. Add
> usb port's system suspend/resume callbacks and call
> usb_port_runtime_resume/suspend() to power off these
> ports whose pm qos NO_POWER_OFF flag is not set, system
> wakeup is disabled and persist is e
On Thu, 28 Mar 2013, Lan Tianyu wrote:
> Usb port isn't assigned to any bus_type. This seems not good from
> Greg's comments.
> http://marc.info/?l=linux-usb&m=136200364929942&w=2
>
> This patch is to register usb port to usb_bus_type. The usb port's
> original name is "portX". This will ca
On Wed, 27 Mar 2013, Lan, Tianyu wrote:
> A small tool to test this patch.
>
> #include
> #include
> #include
> #include
> #include
> #include
>
> #define USBDEVFS_POWER_RESET _IO('U', 28)
>
> int main(int argc, char **argv)
> {
> const char *filename;
> int fd;
> in
> From: Matthijs Kooijman [mailto:matth...@stdin.nl]
> Sent: Wednesday, March 27, 2013 8:42 AM
> To: Greg KH
>
> Hi Greg,
>
> > But yes, it does seem to be a good idea,
>
> Ok, thanks.
>
> > as long as these options can't change over time.
> It's only read-only registers, or registers whose pow
Hi again,
the patch works fine on my WM8850 netbook. I didn't get any
unused-function warning on build, most likely because I have CONFIG_PM
set in kernel config.
Regards,
Peter
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel
On 27 March 2013 18:35, Alan Stern wrote:
> On Wed, 27 Mar 2013, David Linares wrote:
>
>> Thanks Alan. I understand, but the USB cable is plugged into an AC/DC
>> adaptor which can provide an output of 1A. After breaking out an USB
>> cable, I measured that the consumption of my board is around 1
This lets us use the ehci-generic driver on platforms without special
requirements for their ehci controllers. In particular, this is true
for the vt8500/wm8x50 platforms, which currently have a separate
driver that causes problems with multiplatform configurations.
Tested-by: Tony Prisk
Tested-b
Good evening,
We are developing a custom piece of hardware that provides the
equivalent of two serial console interfaces. They do not speak the AT
command set. One port provides an interactive console to the user for
configuration purposes (via minicom/etc). The other speaks a proprietary
protocol
On Wed, 27 Mar 2013, David Linares wrote:
> Please correct me if I am wrong.
> On my iMX25 board, I have got a 1-port root_hub which will provide
> 500mA max to its unique child.
You should check that against the iMX25's specs. It's possible that
the root hub has a current limit lower than 500
On Wed, Mar 27, 2013 at 08:45:38PM +0100, Wesley W. Terpstra wrote:
> Good evening,
>
> We are developing a custom piece of hardware that provides the
> equivalent of two serial console interfaces. They do not speak the AT
> command set. One port provides an interactive console to the user for
> c
On Wed, 27 Mar 2013, Arnd Bergmann wrote:
> This lets us use the ehci-generic driver on platforms without special
> requirements for their ehci controllers.
Well, actually the driver's name is "ehci-platform". And the patch
lets us use it on platforms where the DMA mask hasn't been set up, as
o
It seems to be getting more common recently for EHCI host controllers
to be probed after their companion UHCI or OHCI controllers. This may
be caused partly by splitting the ehci-pci driver out from ehci-hcd,
or it may be caused by changes in the way the kernel does driver
probing.
Regardless, it
This patch (as1673) reduces the amount of log spew from the hub driver
by removing a bunch of error messages in the case where the device in
question is already known to have been disconnected. Since the
disconnect event itself appears in the log, there's no need for other
error messages.
Signed-
This patch (as1674) speeds up system sleep transitions by not
suspending each individual device on a USB-1.1 or USB-2 bus. The
devices will automatically go into suspend when their root hubs are
suspended (i.e., stop sending out Start-Of-Frame packets) -- this is
what the USB spec calls "global su
This patch (as1675) removes the CONFIG_USB_SUSPEND option, essentially
replacing it everywhere with CONFIG_PM_RUNTIME (except for one place
in hub.c, where it is replaced with CONFIG_PM because the code needs
to be used in both runtime and system PM). The net result is code
shrinkage and simplific
Michael Grzeschik writes:
> On Wed, Mar 27, 2013 at 06:35:30PM +0200, Alexander Shishkin wrote:
>> Michael Grzeschik writes:
>>
>> > Hi,
>> >
>> > this series solve some issues with the chipidea udc.
>> >
>> > The series is based on v3.9-rc4.
>> >
>> > Michael Grzeschik (6):
>> > usb: chipide
On Wed, 2013-03-27 at 13:06 -0700, Greg KH wrote:
> > Our current prototypes borrow the Sierra VID
> And the USB-IF might revoke your vendor id, if they find you shipping a
> device with a different vendor id than the one you have been assigned.
One of the reasons we borrowed that VID is that we d
On Wed, Mar 27, 2013 at 9:09 PM, Alan Stern wrote:
> On Wed, 27 Mar 2013, Arnd Bergmann wrote:
>
>> This lets us use the ehci-generic driver on platforms without special
>> requirements for their ehci controllers.
>
> Well, actually the driver's name is "ehci-platform". And the patch
> lets us us
On Wed, Mar 27, 2013 at 10:24:04PM +0200, Alexander Shishkin wrote:
> Michael Grzeschik writes:
>
> > On Wed, Mar 27, 2013 at 06:35:30PM +0200, Alexander Shishkin wrote:
> >> Michael Grzeschik writes:
> >>
> >> > Hi,
> >> >
> >> > this series solve some issues with the chipidea udc.
> >> >
> >>
On Wednesday 27 March 2013, Peter Vasil wrote:
> Hi, unfortunately I still can't apply it on 3.9-rc4 :-(
>
> $ git apply --check ehci-dt-bus-glue.patch
> error: removal patch leaves file contents
> error: drivers/usb/host/ehci-vt8500.c: patch does not apply
>
> Looks git really doesn't like ehci-
This lets us use the ehci-platform driver on platforms without special
requirements for their ehci controllers. In particular, this is true
for the vt8500/wm8x50 platforms, which currently have a separate
driver that causes problems with multiplatform configurations.
Tested-by: Tony Prisk
Tested-
[+cc linux-usb]
On Wed, Mar 27, 2013 at 11:37 AM, Udo van den Heuvel wrote:
> Hello,
>
> When my dmesg gives me a growing number of lines like the one below,
> what is going on?
>
> ohci_hcd :00:12.0: urb 88023025c500 path 2 ep1in 6c16 cc 6
> --> status -71
>
> Please let me know!
>
On Wed, Mar 27, 2013 at 10:41 PM, Arnd Bergmann wrote:
> On Wednesday 27 March 2013, Peter Vasil wrote:
>> Hi, unfortunately I still can't apply it on 3.9-rc4 :-(
>>
>> $ git apply --check ehci-dt-bus-glue.patch
>> error: removal patch leaves file contents
>> error: drivers/usb/host/ehci-vt8500.c:
On Wednesday 27 March 2013, Peter Vasil wrote:
> Hm, no idea what the latest versions are :-) I use archlinux, with
> latest updates installed.
> Here are my git and patch versions:
>
> $ git --version
> git version 1.8.2
> $ patch --version
> GNU patch 2.7.1
>
> Btw. my git understands format-pa
On Wed, Mar 27, 2013 at 02:24:24PM +0100, Yann Sionneau wrote:
>
> Le 26/03/2013 17:30, Sarah Sharp a écrit :
> > On Tue, Mar 26, 2013 at 12:11:13PM +0100, Yann Sionneau wrote:
> >> Le 25/03/2013 19:13, Sarah Sharp a écrit :
> >>> On Mon, Mar 25, 2013 at 05:43:40PM +0100, Yann Sionneau wrote:
> >>
This patch changes the read of the td status to one atomic operation to
analyse coherent bits.
Signed-off-by: Michael Grzeschik
---
drivers/usb/chipidea/udc.c | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/drivers/usb/chipidea/udc.c b/drivers/usb/chipidea/udc.c
index
The prepared memory for the qhead needs to be contiguos and 2K aligned.
We change the code from allocating extra buffer for every ep qhead to
one big area. This patch lowers the amount of code to prepare the
memory.
Signed-off-by: Michael Grzeschik
---
drivers/usb/chipidea/ci.h | 9 +++--
The datasheet of the synopsys core describes only to overwrite the
active and halted bits in the qhead before priming any endpoint.
Signed-off-by: Michael Grzeschik
Reviewed-by: Felipe Balbi
---
drivers/usb/chipidea/udc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drive
In hardware_enqueue code adds one extra td with dma_pool_alloc if
mReq->req.zero is true. When _ep_nuke will be called for that endpoint,
dma_pool_free will not be called to free that memory again. That patch
fixes this.
Cc: stable
Signed-off-by: Michael Grzeschik
---
drivers/usb/chipidea/udc.c
There is no need to call ep_queue unlocked inside the own driver. We
move its functionionality into an unlocked version.
This patch removes potential unlocked timeslots inside
isr_setup_status_phase and isr_get_status_response, in which the lock
got released just before acquired again inside usb_e
The udc uses an shared dma memory space between hard and software. This
memory layout is described in ci13xxx_qh and ci13xxx_td which are marked
with the attribute ((packed)).
The compiler currently does not know about the alignment of the memory
layout, and will create strb and ldrb operations.
It is not safe to truncate requests to the maximum possible size the
controller can handle with one td and to keep working. That patch fixes
that with proper error handling instead.
Reported-by: Felipe Balbi
Signed-off-by: Michael Grzeschik
---
drivers/usb/chipidea/udc.c | 4 ++--
1 file change
Hi,
this series solve some issues with the chipidea udc.
The series is based on v3.9-rc4.
Thanks,
Michael
Michael Grzeschik (8):
usb: chipidea: udc: add attribute aligned(4) to shared memory structs
usb: chipidea: udc: only clear active and halted bits in qhead
usb: chipidea: udc: move ZL
Its not necessary and also not specified in the datasheet to change the
ZLT flag before every ep_prime. This patch moves this to the ep_enable
and applies it only for non configuration endpoints.
Signed-off-by: Michael Grzeschik
Reviewed-by: Felipe Balbi
---
drivers/usb/chipidea/udc.c | 3 ++-
On Wed, Mar 27, 2013 at 09:28:11PM +0100, Wesley W. Terpstra wrote:
> On Wed, 2013-03-27 at 13:06 -0700, Greg KH wrote:
> > > Our current prototypes borrow the Sierra VID
> > And the USB-IF might revoke your vendor id, if they find you shipping a
> > device with a different vendor id than the one y
On 2013/3/28 2:47, Alan Stern wrote:
On Thu, 28 Mar 2013, Lan Tianyu wrote
What happens if there's no device plugged in to the port, but the hub
is enabled for remote wakeup? How will the hub be able to detect a
plug-in event if the port isn't powered?
Alan Stern
Hi Alan:
Great thanks
1 - 100 of 122 matches
Mail list logo
