Hi Tony,
On Wednesday 19 November 2014 08:58 PM, Heikki Krogerus wrote:
The driver does no use it any more.
Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com
This version is slightly different from the one you gave your Acked-by.
So are you okay if I merge this with your
Hi Greg,
On Wednesday 19 November 2014 08:58 PM, Heikki Krogerus wrote:
The device name is usually required when assigning resources
like clocks to platform devices. The problem is that the
device name is not know before platform_device_add is called
and that can be too late as the drivers
Hi,
On Wed, Nov 19, 2014 at 8:58 PM, Heikki Krogerus
heikki.kroge...@linux.intel.com wrote:
This set will in practice just separate the creation of a phy and
binding of it to the consumer. Mapping phys to consumers can be now
done with lookups similarly how clocks can be mapped in clkdev.c.
On Wed, 2014-11-19 at 19:22 -0600, Ahmed Tamrawi wrote:
Bug Report Filed: https://bugzilla.kernel.org/show_bug.cgi?id=88521
Linux Version [3.17-rc1]
Configuration: Default configuration for x86
Function Source:
http://lxr.free-electrons.com/source/drivers/usb/class/cdc-acm.c#L1404
In
On Thu, Nov 20, 2014 at 01:45:05PM +0100, Oliver Neukum wrote:
On Wed, 2014-11-19 at 19:22 -0600, Ahmed Tamrawi wrote:
Bug Report Filed: https://bugzilla.kernel.org/show_bug.cgi?id=88521
Linux Version [3.17-rc1]
Configuration: Default configuration for x86
Function Source:
On Thu, 2014-11-20 at 14:27 +0100, Johan Hovold wrote:
It looks like it's never freed in the alloc_fail8 error path of probe,
though.
Indeed. A case of the coder being blind to his own code.
Patch is forthcoming.
Thanks
Oliver
--
To unsubscribe from this list: send
If probe() fails not only the attributes need to be removed
but also the memory freed.
Reported-by: Ahmed Tamrawi ahmedtamr...@gmail.com
Signed-off-by: Oliver Neukum oneu...@suse.de
CC: sta...@vger.kernel.org
---
drivers/usb/class/cdc-acm.c | 1 +
1 file changed, 1 insertion(+)
diff --git
Oliver Neukum oneu...@suse.de writes:
On Thu, 2014-11-20 at 14:27 +0100, Johan Hovold wrote:
It looks like it's never freed in the alloc_fail8 error path of probe,
though.
Indeed. A case of the coder being blind to his own code.
Patch is forthcoming.
Why does the driver have to allocate
On Thu, 2014-11-20 at 15:09 +0100, Bjørn Mork wrote:
Oliver Neukum oneu...@suse.de writes:
On Thu, 2014-11-20 at 14:27 +0100, Johan Hovold wrote:
It looks like it's never freed in the alloc_fail8 error path of probe,
though.
Indeed. A case of the coder being blind to his own code.
Oliver Neukum oneu...@suse.de writes:
On Thu, 2014-11-20 at 15:09 +0100, Bjørn Mork wrote:
Oliver Neukum oneu...@suse.de writes:
On Thu, 2014-11-20 at 14:27 +0100, Johan Hovold wrote:
It looks like it's never freed in the alloc_fail8 error path of probe,
though.
Indeed. A case of the
On Thu, 2014-11-20 at 15:36 +0100, Bjørn Mork wrote:
Oliver Neukum oneu...@suse.de writes:
On Thu, 2014-11-20 at 15:09 +0100, Bjørn Mork wrote:
Oliver Neukum oneu...@suse.de writes:
On Thu, 2014-11-20 at 14:27 +0100, Johan Hovold wrote:
It looks like it's never freed in the
On Thu, 20 Nov 2014, Oliver Neukum wrote:
On Thu, 2014-11-20 at 15:36 +0100, Bjørn Mork wrote:
Oliver Neukum oneu...@suse.de writes:
On Thu, 2014-11-20 at 15:09 +0100, Bjørn Mork wrote:
Oliver Neukum oneu...@suse.de writes:
On Thu, 2014-11-20 at 14:27 +0100, Johan Hovold wrote:
On Wed, 19 Nov 2014, Thomas Petazzoni wrote:
Dear Marcin Wojtas,
On Thu, 13 Nov 2014 00:49:50 +0100, Marcin Wojtas wrote:
This commit fixes ehci-orion operation in big-endian mode by enabling
byteswap
when accessing registers using 'rdl' and 'wrl' macros.
Signed-off-by: Grzegorz
From: Markus Elfring elfr...@users.sourceforge.net
Date: Thu, 20 Nov 2014 16:11:56 +0100
The kfree() function tests whether its argument is NULL and then
returns immediately. Thus the test around the call is not needed.
This issue was detected by using the Coccinelle software.
Signed-off-by:
When __of_usb_find_phy() fails, it returns -ENODEV - its
error code has to be returned by devm_usb_get_phy_by_phandle().
Only when the former function succeeds and try_module_get()
fails should -EPROBE_DEFER be returned.
Signed-off-by: Arjun Sreedharan arjun...@gmail.com
---
On Thu, Nov 13, 2014 at 12:12 PM, Felipe Balbi ba...@ti.com wrote:
Hi,
(please don't top-post)
On Thu, Nov 13, 2014 at 11:26:50AM -0700, Ashwini Pahuja wrote:
Hi Felipe,
I Just sent you the V3 PATCH, I think it should be good for your next
submission to Greg for 3.18-rc6, I guess it's too
There was another instance where we were
holding pointers which could be long gone.
Fix that by caching only values pointed to
by such pointer.
Because no crash has been observed, this patch
will be sent on v3.19 merge window, instead of
-rc.
Signed-off-by: Felipe Balbi ba...@ti.com
---
From: Julia Lawall julia.law...@lip6.fr
Return a negative error code on failure.
A simplified version of the semantic match that finds this problem is as
follows: (http://coccinelle.lip6.fr/)
// smpl
@@
identifier ret; expression e1,e2;
@@
(
if (\(ret 0\|ret != 0\))
{ ... return ret; }
|
ret
The complate semantic patch that finds this problem is as follows:
(http://coccinelle.lip6.fr/)
// smpl
@ok exists@
identifier f,ret,i;
expression e;
constant c;
@@
// identify a function that returns a negative return value at least once.
f(...) {
... when any
(
return -c@i;
|
ret = -c@i;
...
From: Julia Lawall julia.law...@lip6.fr
Return a negative error code on failure.
A simplified version of the semantic match that finds this problem is as
follows: (http://coccinelle.lip6.fr/)
// smpl
@@
identifier ret; expression e1,e2;
@@
(
if (\(ret 0\|ret != 0\))
{ ... return ret; }
|
ret
On Thu, Nov 20, 2014 at 09:23:36PM +0530, Arjun Sreedharan wrote:
When __of_usb_find_phy() fails, it returns -ENODEV - its
error code has to be returned by devm_usb_get_phy_by_phandle().
Only when the former function succeeds and try_module_get()
fails should -EPROBE_DEFER be returned.
On Tue, Nov 18, 2014 at 03:11:54PM -0500, Jorge Ramirez-Ortiz wrote:
Add SS descriptors to support the capabilities provided by USB3 controller
drivers; unit tests run using a PLX 3380 [max transfer speed measured of
1Gbps]
This driver shall fallback to lower operating modes when the higher
On Tue, Nov 18, 2014 at 11:21:17PM +0100, Daniel Mack wrote:
f_midi_register_card() uses midi-id and midi-index, so they need to
be assigned before the function is called from f_midi_bind_config().
Move the assigment of midi-buflen and midi-qlen as well so they are
all grouped.
Compile
On Wed, Nov 19, 2014 at 04:37:27PM +0100, Nicolas Ferre wrote:
From: Ronald Wahl ronald.w...@raritan.com
Commit 7628083227b6bc4a7e33d7c381d7a4e558424b6b (usb: gadget: at91_udc:
prepare clk before calling enable) added clock preparation in interrupt
context. This is not allowed as it might
On Fri, Oct 31, 2014 at 11:12:33AM +0100, Marek Szyprowski wrote:
This patch adds mutex, which protects initialization and
deinitialization procedures against suspend/resume methods.
Signed-off-by: Marek Szyprowski m.szyprow...@samsung.com
doesn't apply either:
checking file
On Wed, Nov 19, 2014 at 08:36:22AM -0800, Todd Poynor wrote:
On Wed, Nov 19, 2014 at 2:43 AM, Kiran Kumar Raparthy
kiran.ku...@linaro.org wrote:
From: Todd Poynor toddpoy...@google.com
usb: phy: introduce usb_phy_set_event interface
Hi Kiran, this is new stuff that is all your own work
Hi community,
As Windows support UAC1 only, I'd presume that anyone who is serious
about UAC will consider Windows Host support.
A few months back, I got UAC1 working for Windows 7/8 Host on my
Wandboard (Freescale i.MX6 based), on kernel 3.10.17.
I quickly checked the latest Linux kernel,
Hi Felipe,
On 21 November 2014 01:24, Felipe Balbi ba...@ti.com wrote:
On Wed, Nov 19, 2014 at 08:36:22AM -0800, Todd Poynor wrote:
On Wed, Nov 19, 2014 at 2:43 AM, Kiran Kumar Raparthy
kiran.ku...@linaro.org wrote:
From: Todd Poynor toddpoy...@google.com
usb: phy: introduce
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 11/20/2014 08:49 PM, Felipe Balbi wrote:
On Tue, Nov 18, 2014 at 11:21:17PM +0100, Daniel Mack wrote:
f_midi_register_card() uses midi-id and midi-index, so they
need to be assigned before the function is called from
f_midi_bind_config(). Move
Hi Dinh:
On 2014/11/12 1:13, dingu...@opensource.altera.com wrote:
From: Dinh Nguyen dingu...@opensource.altera.com
Adds the gadget data structure and appropriate data structure pointers
to the common dwc2_hsotg data structure. To keep the driver data
dereference code looking clean, the
Hi community,
As Windows support UAC1 only, I'd presume that anyone who is serious about
UAC will consider Windows Host support.
A few months back, I got UAC1 working for Windows 7/8 Host on my
Wandboard (Freescale i.MX6 based), on kernel 3.10.17.
I quickly checked the latest Linux
usb: phy: Handle per-PHY event for connnect and disconnect events
When usb is connected and enumerated in device mode or when usb is
disconnected,call usb_phy_set_event from phy drivers to handle per-PHY event.
Cc: Felipe Balbi ba...@ti.com
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
Cc:
usb: phy: introduce usb_phy_set_event interface
PHY drivers require a generic interface to handle per-PHY events.
usb_phy_set_event interface sets event to phy event.
PHY drivers call this interface for each phy event.
Cc: Felipe Balbi ba...@ti.com
Cc: Greg Kroah-Hartman
usb: phy: hold wakeupsource when USB is enumerated in peripheral mode
Some systems require a mechanism to prevent system to enter into suspend
state when USB is connected and enumerated in peripheral mode.
This patch provides an interface to hold a wakeupsource to prevent suspend.
PHY drivers
34 matches
Mail list logo
