The patch below removes an extra "l" in the word.
Signed-off-by: Justin P. Mattock
---
drivers/staging/comedi/drivers/pcl816.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/staging/comedi/drivers/pcl816.c
b/drivers/staging/comedi/drivers/pcl816.c
index 3d0f0
The patch below removes an extra "l" in the word.
Signed-off-by: Justin P. Mattock
---
drivers/staging/vt6655/device_main.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/staging/vt6655/device_main.c
b/drivers/staging/vt6655/device_main.c
index f5028d9..8950c
The patch below removes an extra "l" in the word.
Signed-off-by: Justin P. Mattock
---
drivers/staging/rtl8712/rtl871x_pwrctrl.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/staging/rtl8712/rtl871x_pwrctrl.c
b/drivers/staging/rtl8712/rtl871x_pwrctrl.c
index
The patch below removes an extra "l" in the word.
Signed-off-by: Justin P. Mattock
---
drivers/staging/comedi/drivers/pcl818.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/comedi/drivers/pcl818.c
b/drivers/staging/comedi/drivers/pcl818.c
index d2b
The patch below removes an extra "l" in the word.
Signed-off-by: Justin P. Mattock
---
drivers/staging/vt6656/main_usb.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/staging/vt6656/main_usb.c
b/drivers/staging/vt6656/main_usb.c
index 7cc3d24..7b5b7c5 100644
> -Original Message-
> From: Greg KH [mailto:gre...@suse.de]
> Sent: Thursday, February 24, 2011 7:32 PM
> To: KY Srinivasan
> Cc: linux-ker...@vger.kernel.org; devel@linuxdriverproject.org;
> virtualizat...@lists.osdl.org; Haiyang Zhang; Hank Janssen
> Subject: Re: [PATCH ] Staging: hv:
On Thu, Feb 24, 2011 at 08:58:52PM +0100, Belisko Marek wrote:
> Hi Greg,
>
> On Thu, Feb 24, 2011 at 6:12 PM, Greg KH wrote:
> > On Thu, Feb 24, 2011 at 10:33:18AM +0100, Marek Belisko wrote:
> >> This patch will fix following compilation error:
> >> drivers/staging/gma500/psb_drv.c:1635: error:
On Fri, Feb 25, 2011 at 12:24:57AM +, KY Srinivasan wrote:
>
>
> > -Original Message-
> > From: Greg KH [mailto:gre...@suse.de]
> > Sent: Thursday, February 24, 2011 6:46 PM
> > To: KY Srinivasan
> > Cc: linux-ker...@vger.kernel.org; devel@linuxdriverproject.org;
> > virtualizat...@li
> -Original Message-
> From: Greg KH [mailto:gre...@suse.de]
> Sent: Thursday, February 24, 2011 6:46 PM
> To: KY Srinivasan
> Cc: linux-ker...@vger.kernel.org; devel@linuxdriverproject.org;
> virtualizat...@lists.osdl.org; Haiyang Zhang; Hank Janssen
> Subject: Re: [PATCH ] Staging: hv:
On Thu, Feb 24, 2011 at 03:20:58PM -0800, K. Y. Srinivasan wrote:
> This patch cleans up (a lot of the) naming issues that
> various reviewers have noted. It also gets rid of
> some unnecessary layering in the code.
Whenever you have a patch description that says "It also..." you know
you need to
This error was spotted by cppcheck:
drivers/staging/brcm80211/phy/wlc_phy_lcn.c:4053: error: Memory leak: ptr
v2:
Rebased patch on linux-next (file was moved to a new directory).
Cc: Brett Rudley
Cc: Henry Ptasinski
Cc: Greg Kroah-Hartman
Cc: linux-wirel...@vger.kernel.org
Cc: de...@driverdev
Am 23.02.2011 23:05, schrieb Greg KH:
On Sun, Feb 20, 2011 at 12:29:11AM +0100, Stefan Weil wrote:
This error was spotted by cppcheck:
drivers/staging/brcm80211/phy/wlc_phy_lcn.c:4053: error: Memory leak: ptr
Cc: Brett Rudley
Cc: Henry Ptasinski
Cc: linux-wirel...@vger.kernel.org
Cc: de...@d
Hi Greg,
On Thu, Feb 24, 2011 at 6:12 PM, Greg KH wrote:
> On Thu, Feb 24, 2011 at 10:33:18AM +0100, Marek Belisko wrote:
>> This patch will fix following compilation error:
>> drivers/staging/gma500/psb_drv.c:1635: error: unknown field ‘pci_driver’
>> specified in initializer
>> drivers/staging
On Thu, 2011-02-24 at 19:08 +, Ilia Mirkin wrote:
> FWIW my reasoning for doing this was that generally the reason you're
> freeing something is because you allocated it, so kfree(NULL) happens
> rarely -- error paths, conditional features, etc. If you actually
> expect the argument to be NULL
On Thu, Feb 24, 2011 at 5:25 PM, Steven Rostedt wrote:
> On Thu, 2011-02-24 at 09:06 -0800, Greg KH wrote:
>> On Thu, Feb 24, 2011 at 11:08:29AM -0500, Steven Rostedt wrote:
>> > On Tue, Feb 22, 2011 at 03:53:40AM -0500, Ilia Mirkin wrote:
>> > > This patch was created with the following semantic
On Thu, Feb 24, 2011 at 10:50:28AM -0800, Brett Rudley wrote:
> > So please work on removing all of them from the driver, it's one of the
> > requirements to move out of the staging tree.
> >
>
> We will. What are others? The more we know the more we can cleanup.
Isn't the list in the TODO f
> So please work on removing all of them from the driver, it's one of the
> requirements to move out of the staging tree.
>
We will. What are others? The more we know the more we can cleanup.
Thanks
brett
___
devel mailing list
devel@linuxdriverp
On Thu, 2011-02-24 at 09:06 -0800, Greg KH wrote:
> On Thu, Feb 24, 2011 at 11:08:29AM -0500, Steven Rostedt wrote:
> > On Tue, Feb 22, 2011 at 03:53:40AM -0500, Ilia Mirkin wrote:
> > > This patch was created with the following semantic patch:
> > >
> > > //
> > > @@
> > > expression E;
> > > @@
On Thu, Feb 24, 2011 at 10:33:18AM +0100, Marek Belisko wrote:
> This patch will fix following compilation error:
> drivers/staging/gma500/psb_drv.c:1635: error: unknown field ‘pci_driver’
> specified in initializer
> drivers/staging/gma500/psb_drv.c:1636: error: unknown field ‘name’ specified
>
On Thu, Feb 24, 2011 at 11:30:53AM +0100, Roland Vossen wrote:
> Hello Dan,
>
> >Yeah. We should delete the assert statement too because that's the
> >easiest way to get rid of the warning.
>
> I use a little script that I wrote myself that checks a patch series
> before submitting it. The scrip
On Thu, Feb 24, 2011 at 11:08:29AM -0500, Steven Rostedt wrote:
> On Tue, Feb 22, 2011 at 03:53:40AM -0500, Ilia Mirkin wrote:
> > This patch was created with the following semantic patch:
> >
> > //
> > @@
> > expression E;
> > @@
> >
> > - if (E != NULL) kfree(E);
> > + kfree(E);
> > //
>
>
On Tue, Feb 22, 2011 at 03:53:40AM -0500, Ilia Mirkin wrote:
> This patch was created with the following semantic patch:
>
> //
> @@
> expression E;
> @@
>
> - if (E != NULL) kfree(E);
> + kfree(E);
> //
OK, so when will we be removing the unlikely() from the implementations
of kfree()?
Dear partner,
May i confide in you?,and If yes, please i do have a reason for both of us, to
enshrine total trust and confidence in my quest for you to be my foreign
benefactor in this good live changing business.
Honestly I want you to keep the payment of my 10% contract commission bill
fro
On Thu, 24 Feb 2011 10:33:18 +0100
Marek Belisko wrote:
> This patch will fix following compilation error:
The joys of the DRM layer API of the week.
Acked-by: Alan Cox
___
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverp
A warning was generated if CONFIG_BRCMDBG=n. Fixed this by introducing an
function.
Signed-off-by: Roland Vossen
---
drivers/staging/brcm80211/brcmsmac/wl_mac80211.c |3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/drivers/staging/brcm80211/brcmsmac/wl_mac80211.c
b/dri
Hello Dan,
Yeah. We should delete the assert statement too because that's the
easiest way to get rid of the warning.
I use a little script that I wrote myself that checks a patch series
before submitting it. The script builds each commit in the patch series
in different configurations (eg,
This patch will fix following compilation error:
drivers/staging/gma500/psb_drv.c:1635: error: unknown field ‘pci_driver’
specified in initializer
drivers/staging/gma500/psb_drv.c:1636: error: unknown field ‘name’ specified in
initializer
drivers/staging/gma500/psb_drv.c:1636: warning: initializa
27 matches
Mail list logo