This patch fixes the following checkpatch.pl warning:
braces {} are not necessary for single statement blocks
Signed-off-by: Salah Triki salah.tr...@acm.org
---
drivers/staging/dgnc/dgnc_tty.c | 4
1 file changed, 4 deletions(-)
diff --git a/drivers/staging/dgnc/dgnc_tty.c
On Sat, Feb 21, 2015 at 06:53:30PM -0800, Joe Perches wrote:
The seq_printf return value, because it's frequently misused,
will eventually be converted to void.
See: commit 1f33c41c03da (seq_file: Rename seq_overflow() to
seq_has_overflowed() and make public)
Convert these uses to:
On Wed, Feb 18, 2015 at 05:37:55PM -0300, Mario J. Rugiero wrote:
This fixes all appearances of the warning but one, as that one seems to be
intentional:
WARNING: Single statement macros should not use a do {} while (0) loop
#1221: FILE: lustre/include/lustre_dlm.h:1221:
+#define
On Mon, Mar 02, 2015 at 12:38:35AM +0100, Salah Triki wrote:
This patch removes unnecessary checks of unit magic, channel magic, board
magic and board state.
Signed-off-by: Salah Triki salah.tr...@acm.org
---
drivers/staging/dgnc/dgnc_sysfs.c | 14 --
1 file changed, 14
On Sat, Feb 28, 2015 at 06:37:30PM +0100, Matteo Semenzato wrote:
From: Matteo Semenzato mattew8...@gmail.com
This patch fixes the following error:
space required after that ';'
Signed-off-by: Matteo Semenzato mattew8...@gmail.com
---
drivers/staging/fbtft/fbtft_device.c | 145
On Mon, Feb 23, 2015 at 05:24:51PM +0100, Matteo Semenzato wrote:
From: Matteo Semenzato mattew8...@gmail.com
This patch fixes the following error:
space prohibited before that ','
Signed-off-by: Matteo Semenzato mattew8...@gmail.com
---
drivers/staging/fbtft/fb_ra8875.c | 150
On Sat, Feb 28, 2015 at 03:51:47PM +0100, Matteo Semenzato wrote:
From: Matteo Semenzato mattew8...@gmail.com
This patch fixes the following error:
code indent should use tabs where possible
Signed-off-by: Matteo Semenzato mattew8...@gmail.com
---
drivers/staging/fbtft/flexfb.c | 9
On Fri, Feb 27, 2015 at 09:44:42PM -0800, Tolga Ceylan wrote:
Replaced C99 '//' comments with C89 '/**/'
Signed-off-by: Tolga Ceylan tolga.cey...@gmail.com
---
drivers/staging/fbtft/fb_ssd1331.c | 30 +++---
1 file changed, 15 insertions(+), 15 deletions(-)
Doesn't
On Fri, Feb 27, 2015 at 03:08:58AM +, Sheng Yong wrote:
For value which has size_t type, use %zu to avoid compile warning of
printk format.
Signed-off-by: Sheng Yong shengyo...@huawei.com
---
drivers/staging/fbtft/fbtft-core.c | 2 +-
drivers/staging/fbtft/fbtft-io.c | 4 ++--
2
On Tue, Feb 10, 2015 at 07:24:40PM +0800, Matt Tyler wrote:
module.c was previously the sole exporter of symbols from module.c
This patch removes the global symbols by merging the two files.
Signed-off-by: Matthew Tyler matt.ty...@flashics.com
---
On Tue, Feb 10, 2015 at 08:36:21PM +0200, Tal Shorer wrote:
following coding style error in
drivers/staging/lustre/lustre/libcfs/linux/linux-proc.c:
missing spaces around '='
Signed-off-by: Tal Shorer tal.sho...@gmail.com
---
drivers/staging/lustre/lustre/libcfs/linux/linux-proc.c | 2 +-
On Tue, Feb 10, 2015 at 08:36:19PM +0200, Tal Shorer wrote:
fix the following coding style error in
drivers/staging/lustre/lustre/libcfs/linux/linux-proc.c:
initialization of lnet_table_header (static pointer) to NULL
Signed-off-by: Tal Shorer tal.sho...@gmail.com
---
On Tue, Feb 10, 2015 at 08:36:20PM +0200, Tal Shorer wrote:
fix the following coding style error in
drivers/staging/lustre/lustre/libcfs/linux/linux-proc.c:
initialization of min_watchdog_ratelimit (static int) to 0
for clarity's sake, the = 0 is kept as part of a comment
Signed-off-by:
From: Matteo Semenzato mattew8...@gmail.com
This patch fixes the following warning:
Use of EXTRA_CFLAGS is deprecated, please use `ccflags-y instead.
Signed-off-by: Matteo Semenzato mattew8...@gmail.com
---
drivers/staging/dgnc/Makefile | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
From: Matteo Semenzato mattew8...@gmail.com
This patch fixes the following warning:
Prefer [subsystem eg: netdev]_err([subsystem]dev, ... then dev_err(dev,
... then pr_err(... to printk(KERN_ERR ...
Signed-off-by: Matteo Semenzato mattew8...@gmail.com
---
drivers/staging/dgnc/dgnc_sysfs.c | 4
This patch removes unnecessary checks of unit magic, channel magic, board
magic and board state.
Signed-off-by: Salah Triki salah.tr...@acm.org
---
drivers/staging/dgnc/dgnc_sysfs.c | 14 --
1 file changed, 14 deletions(-)
diff --git a/drivers/staging/dgnc/dgnc_sysfs.c
On Sat, Feb 21, 2015 at 09:36:49PM +0100, Matteo Semenzato wrote:
From: Matteo Semenzato mattew8...@gmail.com
This patch fixes the following warnings:
space prohibited before that close parenthesis ')'
space prohibited after that open parenthesis '('
Signed-off-by: Matteo Semenzato
-Original Message-
From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com]
Sent: Friday, February 27, 2015 8:14 AM
To: KY Srinivasan; de...@linuxdriverproject.org
Cc: Haiyang Zhang; linux-ker...@vger.kernel.org; Dexuan Cui; Radim Krčmář;
Greg Kroah-Hartman; linux-...@vger.kernel.org
-Original Message-
From: Greg KH [mailto:gre...@linuxfoundation.org]
Sent: Sunday, March 1, 2015 7:34 PM
To: Jake Oshins
Cc: rafael.j.wyso...@intel.com; KY Srinivasan; linux-ker...@vger.kernel.org;
de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com;
hey greg, I've updated to the latest linux-next, applied the patch and compiled
with no issues.
I think it might be due to clashes with another patch that has been proposed by
Tal Shorer to fix style violations in linux-proc.c (which my patch merges into
module.c)
Tal - do you want to take my
On Tue, Feb 17, 2015 at 11:41:49AM -0800, Jake Oshins wrote:
This patch adds some wrapper functions in the pnp layer. The intent is
to allow memory address space claims by devices which are descendants
(a child or grandchild of) a device which is already part of the pnp
layer. This allows a
On Sun, Mar 1, 2015 at 5:18 PM, Greg Kroah-Hartman
gre...@linuxfoundation.org wrote:
On Fri, Feb 27, 2015 at 09:44:42PM -0800, Tolga Ceylan wrote:
Replaced C99 '//' comments with C89 '/**/'
Signed-off-by: Tolga Ceylan tolga.cey...@gmail.com
---
drivers/staging/fbtft/fb_ssd1331.c | 30
A: No.
Q: Should I include quotations after my reply?
http://daringfireball.net/2007/07/on_top
On Mon, Mar 02, 2015 at 11:18:09AM +0800, Matt Tyler wrote:
hey greg, I've updated to the latest linux-next, applied the patch and
compiled with no issues.
Please work against my staging-testing
On Sat, Feb 07, 2015 at 08:19:01PM -0800, Shirish Gajera wrote:
This patch fixes the checkpatch.pl warning:
WARNING: line over 80 characters
All line over 80 characters in driver/staging/speakup/* are fixed.
Signed-off-by: Shirish Gajera gajerashir...@gmail.com
---
drivers/hv/vmbus_drv.c:51:5: sparse: symbol 'hyperv_panic_event' was not
declared. Should it be static?
drivers/hv/vmbus_drv.c:51:5: sparse: symbol 'hyperv_panic_event' was not
declared. Should it be static?
drivers/hv/vmbus_drv.c:51:5: sparse: symbol 'hyperv_panic_event' was not
declared.
tree: git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git
char-misc-testing
head: b3de8e3719e582f3182bb504295e4a8e43c8c96f
commit: 96c1d0581d00f7abe033350edb021a9d947d8d81 [25/45] Drivers: hv: vmbus:
Add support for VMBus panic notifier handler
reproduce:
# apt-get install
From: Oleg Drokin gr...@linuxhacker.ru
A recent crash report with CONFIG_CPUMASK_OFFSTACK enabled
set off a chain of emails through which I learned that cpus_* functions
are deprecated and should not be really used to operate on cpumasks.
These two patches do just that, replacing cpus_* and
From: Oleg Drokin gr...@linuxhacker.ru
Rusty Russel advises that cpus_* functions are deprecated to work
on cpumasks and cpumask_* functions should be called instead,
otherwise problems with CPUMASK_OFFSTACK arise.
Signed-off-by: Oleg Drokin gr...@linuxhacker.ru
---
From: Oleg Drokin gr...@linuxhacker.ru
As per Rusty Russel, cpus_* functions are deprecated.
When mixing cpumask_copy with cpus_weight, they operate on different
sized masks if CPUMASK_OFFSTACK is enabled, causing an
immediate assertion failure.
Copying of cpumasks by assignment is also not
On Sun, Mar 01, 2015 at 09:41:26AM +0100, Matteo Semenzato wrote:
From: Matteo Semenzato mattew8...@gmail.com
This patch fixes the following warning:
Use of EXTRA_CFLAGS is deprecated, please use `ccflags-y instead.
Signed-off-by: Matteo Semenzato mattew8...@gmail.com
---
On Sun, Mar 01, 2015 at 09:31:41AM +0100, Matteo Semenzato wrote:
From: Matteo Semenzato mattew8...@gmail.com
This patch fixes the following warning:
Prefer [subsystem eg: netdev]_err([subsystem]dev, ... then dev_err(dev,
... then pr_err(... to printk(KERN_ERR ...
Signed-off-by: Matteo
From: Matteo Semenzato mattew8...@gmail.com
This patch removes EXTRA_CFLAGS from the Makefile and defines DDG_PART
in dgnc_driver.h
Signed-off-by: Matteo Semenzato mattew8...@gmail.com
---
drivers/staging/dgnc/Makefile | 2 --
drivers/staging/dgnc/dgnc_driver.h | 1 +
2 files changed, 1
On Sun, Mar 01, 2015 at 06:19:57PM +0100, Matteo Semenzato wrote:
From: Matteo Semenzato mattew8...@gmail.com
This patch removes EXTRA_CFLAGS from the Makefile and defines DDG_PART
in dgnc_driver.h
Signed-off-by: Matteo Semenzato mattew8...@gmail.com
---
drivers/staging/dgnc/Makefile
On 27 February 2015 at 23:14, Andrey Ryabinin a.ryabi...@samsung.com wrote:
binder_update_page_range() initializes only addr and size
fields in 'struct vm_struct tmp_area;' and passes it to
map_vm_area().
Before 71394fe50146 (mm: vmalloc: add flag preventing guard hole allocation)
this was
34 matches
Mail list logo