Re: [PATCH 2/6] staging: rtl8188eu: hal: removed code indent error

2015-02-10 Thread Sudip Mukherjee
On Tue, Feb 10, 2015 at 03:27:11PM +0100, Bas Peters wrote: @@ -101,8 +101,7 @@ void rtl88eu_phy_rf6052_set_cck_txpower(struct adapter *adapt, u8 *powerlevel) ptr++; } } - rtl88eu_dm_txpower_track_adjust(hal_data-odmpriv, 1, direction, -

Re: [PATCH v2 1/1] staging: ozwpan: Remove allocation from delaration line

2015-02-10 Thread Joe Perches
On Tue, 2015-02-10 at 13:13 +0300, Dan Carpenter wrote: diff --git a/drivers/staging/ozwpan/ozhcd.c b/drivers/staging/ozwpan/ozhcd.c [] @@ -280,8 +280,9 @@ static void oz_free_urb_link(struct oz_urb_link *urbl) */ static struct oz_endpoint *oz_ep_alloc(int buffer_size, gfp_t mem_flags)

Re: [PATCH 2/6] staging: rtl8188eu: hal: removed code indent error

2015-02-10 Thread Bas Peters
@@ -101,8 +101,7 @@ void rtl88eu_phy_rf6052_set_cck_txpower(struct adapter *adapt, u8 *powerlevel) ptr++; } } - rtl88eu_dm_txpower_track_adjust(hal_data-odmpriv, 1, direction, - pwrtrac_value); +

[PATCH] staging: lustre: resolves sparse warnings using static declaration

2015-02-10 Thread Le Tan
This patch resolves sparse warnings about non-declared symbol in staging/lustre/lustre/lov by adding static declaration. These warnings are like this: warning: symbol 'fiemap_calc_fm_end_offset' was not declared. Should it be static? Signed-off-by: Le Tan tam...@live.com ---

Fwd:Чем порадовать?=Классный подарок на 14 февраля=

2015-02-10 Thread Неляна
=Чем порадовать?=Крутой подарок к 14 февраля= http://www.google.com/url?q=htt%70%3A%2F%2Fs%65%72gyv%70%6fda%72%6fk%2e%72%75su%62est%2e%72u%2F%23%30%35%34%31%322%334%362054%3122346%32sa=Dsntz=1usg=AFQjCNHQQm8l6YWW5otIPIOnsNAjWLwlzw ___ devel mailing

Re: doubt about sm7xxfb

2015-02-10 Thread Greg Kroah-Hartman
On Tue, Feb 10, 2015 at 07:37:47PM +0530, Sudip Mukherjee wrote: Hi Greg, we have this sm7xxfb in drivers/staging now which is supporting SM710, SM712, SM721 and SM722. I am also working on another new hardware SM750, which will be ready for staging in next 1 -2 weeks. this SM750 is entirely

Re: [PATCH 2/2] staging: panel: initialize lcd if lcd enabled

2015-02-10 Thread Willy Tarreau
On Tue, Feb 10, 2015 at 05:26:03PM +0530, Sudip Mukherjee wrote: initialiaze lcd parameters only if lcd is enabled. Signed-off-by: Sudip Mukherjee su...@vectorindia.org One minor comment below (cosmetic), but after this it's OK. Acked-by: Willy Tarreau w...@1wt.eu + if (lcd.enabled) {

Re: [PATCH 1/2] staging: panel: register driver after checking device

2015-02-10 Thread Willy Tarreau
On Tue, Feb 10, 2015 at 05:26:02PM +0530, Sudip Mukherjee wrote: register the driver only if lcd or keypad has been enabled and if both are disabled then just exit. Signed-off-by: Sudip Mukherjee su...@vectorindia.org Acked-by: Willy Tarreau w...@1wt.eu Willy

[PATCH] staging: lustre: lustre: lov: lov_dev: fix sparse warning of static declaration

2015-02-10 Thread Mohammad Jamal
This patch adds a static keyword to cl_lov_device_mutex_class variable to suppress the warning of static declaration Signed-off-by: Mohammad Jamal md.jamalmohiud...@gmail.com --- drivers/staging/lustre/lustre/lov/lov_dev.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 25/30] staging: unisys: vmcall_io_controlvm_addr_params pragma

2015-02-10 Thread Benjamin Romer
From: Jeffrey Brown jeffrey.br...@unisys.com Replaced pragma code for the struct vmcall_io_controlvm_addr_params with __packed Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com ---

[PATCH 23/30] staging: unisys: vbusdeviceinfo pragma

2015-02-10 Thread Benjamin Romer
From: Jeffrey Brown jeffrey.br...@unisys.com Replaced pragma code in vbusdeviceinfo.h with __packed for the struct ultra_vbus_deviceinfo Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com ---

[PATCH 22/30] staging: unisys: Remove unnecessary gotos from info_debugfs_read_helper

2015-02-10 Thread Benjamin Romer
Get rid of all the useless gotos in info_debugfs_read_helper() and just return the correct values straight away. Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- drivers/staging/unisys/uislib/uislib.c | 13 ++--- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git

[PATCH 04/30] staging: unisys: change variable registered from bool to int

2015-02-10 Thread Benjamin Romer
From: Erik Arfvidson erik.arfvid...@unisys.com this patch changes variable registered from bool to int Signed-off-by: Erik Arfvidson erik.arfvid...@unisys.com Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- drivers/staging/unisys/visorchipset/file.c | 8 1 file changed, 4

[PATCH 19/30] staging: unisys: remove #definition from timskmod.h

2015-02-10 Thread Benjamin Romer
From: Erik Arfvidson erik.arfvid...@unisys.com this patch removes #definitions from timksmod.h Signed-off-by: Erik Arfvidson erik.arfvid...@unisys.com Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- drivers/staging/unisys/include/timskmod.h| 5 -

[PATCH 21/30] staging: unisys: remove unnecessary goto in parser_param_start()

2015-02-10 Thread Benjamin Romer
The goto isn't necessary here so replace it with a simple return statement. Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- drivers/staging/unisys/visorchipset/parser.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git

[PATCH 08/30] staging: unisys: changes needs_lock from bool to int

2015-02-10 Thread Benjamin Romer
From: Erik Arfvidson erik.arfvid...@unisys.com This patch changes needs_lock from bool to int. As a result this changes variable needs_lock inside function visorchannel_create_guts. Signed-off-by: Erik Arfvidson erik.arfvid...@unisys.com Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com

[PATCH 13/30] staging: unisys: change charqueue.[ch] variables from bool to int

2015-02-10 Thread Benjamin Romer
From: Erik Arfvidson erik.arfvid...@unisys.com This patch changes charqueue.[ch] functions and variables from bool to int including their return values. Signed-off-by: Erik Arfvidson erik.arfvid...@unisys.com Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com ---

[PATCH 28/30] staging: unisys: channel.h pragma

2015-02-10 Thread Benjamin Romer
From: Jeffrey Brown jeffrey.br...@unisys.com Replaced pragma code surrounding the structs signal_queue_header and channel_header with __packed for both of them Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com ---

[PATCH 15/30] staging: unisys: change memregion_direct.c variables type bool to int.

2015-02-10 Thread Benjamin Romer
From: Erik Arfvidson erik.arfvid...@unisys.com This patch changes all variable types bool to int in memregion_direct.c Signed-off-by: Erik Arfvidson erik.arfvid...@unisys.com Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- .../staging/unisys/visorutil/memregion_direct.c| 30

[PATCH 10/30] staging: unisys: change periodic_work.[ch] functions from bool to int

2015-02-10 Thread Benjamin Romer
From: Erik Arfvidson erik.arfvid...@unisys.com This patch changes the following functions from bool to int: visor_periodic_work_nextperiod visor_periodic_work_destroy visor_periodic_work_stop visor_periodic_work_start Also change struct periodic_work variables from bool to int Signed-off-by:

[PATCH v2 1/1] staging: ozwpan: Remove allocation from delaration line

2015-02-10 Thread Quentin Lambert
This patch removes allocation from declaration line because people are known to gloss over declarations. Signed-off-by: Quentin Lambert lambert.quen...@gmail.com --- Changes since v1: - Remove the blank line between allocation and NULL check drivers/staging/ozwpan/ozhcd.c | 5 +++--

[PATCH 1/1] staging: ozwpan: Remove allocation from delaration line

2015-02-10 Thread Quentin Lambert
This patch removes allocation from declaration line because people are known to gloss over declarations. Signed-off-by: Quentin Lambert lambert.quen...@gmail.com --- This patch was inspired by http://www.spinics.net/lists/linux-usb/msg44389.html and https://lkml.org/lkml/2015/2/7/47 but I am

[PATCH 5/7] staging: rtl8188eu: core: remove indentation warning

2015-02-10 Thread Pushpendra Singh
remove checckpatch.pl warning WARNINGROR: switch and case should be at the same indent Signed-off-by: Pushpendra Singh pushpendra.si...@smartplayin.com --- drivers/staging/rtl8188eu/core/rtw_xmit.c | 32 +-- 1 files changed, 17 insertions(+), 17 deletions(-) diff

[PATCH 6/7] staging: rtl8188eu: core: removed unnecessary braces

2015-02-10 Thread Pushpendra Singh
remove checkpatch.pl warning WARNING: braces {} are not necessary for single statement blocks Signed-off-by: Pushpendra Singh pushpendra.si...@smartplayin.com --- drivers/staging/rtl8188eu/core/rtw_efuse.c| 3 +-- drivers/staging/rtl8188eu/core/rtw_mlme_ext.c | 14 +-

[PATCH 7/7] staging: rtl8188eu: core : remove space before tabs

2015-02-10 Thread Pushpendra Singh
remove below checkpatch.pl warning WARNING: please, no space before tabs Signed-off-by: Pushpendra Singh pushpendra.si...@smartplayin.com --- drivers/staging/rtl8188eu/core/rtw_xmit.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git

Re: [PATCH 1/1] staging: ozwpan: Remove allocation from delaration line

2015-02-10 Thread Dan Carpenter
On Tue, Feb 10, 2015 at 09:55:03AM +0100, Quentin Lambert wrote: static struct oz_endpoint *oz_ep_alloc(int buffer_size, gfp_t mem_flags) { - struct oz_endpoint *ep = - kzalloc(sizeof(struct oz_endpoint)+buffer_size, mem_flags); + struct oz_endpoint *ep; + + ep =

[PATCH 1/1] staging: ozwpan: Move code from success handling to error handling

2015-02-10 Thread Quentin Lambert
The original version was success handling rather than error handling, therefore this patch reduces nesting. Signed-off-by: Quentin Lambert lambert.quen...@gmail.com --- drivers/staging/ozwpan/ozhcd.c | 18 +++--- drivers/staging/ozwpan/ozpd.c | 53

Re: Ralink 6570 / Mediatek MT7601STA (was Re: Addition of a wifi kernel module to the linux source tree)

2015-02-10 Thread Parth Sane
Hello everone! A few months earlier I had requested addition of the MT7601 WiFi dongle module into the kernel. I was told that the needful would be done in the rt2x00 driver. Could you guys update me with the status? Thanks a lot! -Parth Sane On 4 December 2014 at 20:06, Stanislaw Gruszka

[PATCH 2/7] staging: rtl8188eu: core: removing unnecessary space

2015-02-10 Thread Pushpendra Singh
Removed unnecessary space before that ',' ERROR: space prohibited before that ',' (ctx:WxE) Signed-off-by: Pushpendra Singh pushpendra.si...@smartplayin.com --- drivers/staging/rtl8188eu/core/rtw_ieee80211.c | 8 +++--- drivers/staging/rtl8188eu/core/rtw_mlme.c | 18 ++--

[PATCH 4/7] staging: rtl8188eu: core: remove unnecessary parentheses

2015-02-10 Thread Pushpendra Singh
remove checkpatch.pl warning WARNING: Unnecessary parentheses - maybe == should be = ? Signed-off-by: Pushpendra Singh pushpendra.si...@smartplayin.com --- drivers/staging/rtl8188eu/core/rtw_ioctl_set.c | 4 ++-- drivers/staging/rtl8188eu/core/rtw_mlme.c | 2 +-

[PATCH 1/7] staging: rtl8188eu: core: added blank line after declarations

2015-02-10 Thread Pushpendra Singh
Removed following checkpatch.pl warning from mutiple files WARNING: Missing a blank line after declarations Signed-off-by: Pushpendra Singh pushpendra.si...@smartplayin.com --- drivers/staging/rtl8188eu/core/rtw_debug.c | 3 ++- drivers/staging/rtl8188eu/core/rtw_efuse.c | 12

[PATCH 3/7] staging: rtl8188eu: core: remove spaces at start of line

2015-02-10 Thread Pushpendra Singh
remove checkpatch.pl warning WARNING: please, no spaces at the start of a line Signed-off-by: Pushpendra Singh pushpendra.si...@smartplayin.com --- drivers/staging/rtl8188eu/core/rtw_led.c | 2 +- drivers/staging/rtl8188eu/core/rtw_mlme_ext.c | 2 +-

Re: [PATCH v2 1/1] staging: ozwpan: Remove allocation from delaration line

2015-02-10 Thread Dan Carpenter
Looks good. Thanks! diff --git a/drivers/staging/ozwpan/ozhcd.c b/drivers/staging/ozwpan/ozhcd.c index 8543bb2..96e95bf 100644 --- a/drivers/staging/ozwpan/ozhcd.c +++ b/drivers/staging/ozwpan/ozhcd.c @@ -280,8 +280,9 @@ static void oz_free_urb_link(struct oz_urb_link *urbl) */ static

[PATCH v2 1/2] staging: unisys: remove unused variable

2015-02-10 Thread Sudip Mukherjee
we were getting lots of warnings about _tempresult set but not used. _tempresult was used in the macro ISSUE_IO_VMCALL_POSTCODE_SEVERITY which was again using another macro ISSUE_IO_EXTENDED_VMCALL. but the value assigned to it was never used. Signed-off-by: Sudip Mukherjee su...@vectorindia.org

[PATCH v2 2/2] staging: unisys: fix directory warning

2015-02-10 Thread Sudip Mukherjee
we were getting three warnings about timskmod and sparstopdriver and channels. These warnings were about no such file or directory. These directory names were included in the Makefile, but the directories were not existing. Signed-off-by: Sudip Mukherjee su...@vectorindia.org --- make W=1 is

Re: [PATCH 1/1] staging: ozwpan: Move code from success handling to error handling

2015-02-10 Thread Dan Carpenter
On Tue, Feb 10, 2015 at 11:42:08AM +0100, Quentin Lambert wrote: The original version was success handling rather than error handling, therefore this patch reduces nesting. Signed-off-by: Quentin Lambert lambert.quen...@gmail.com Fantastic. :) Thanks! regards, dan carpenter

Re: [PATCH 1/1] staging: ozwpan: Move code from success handling to error handling

2015-02-10 Thread Quentin Lambert
On 10/02/2015 12:04, Dan Carpenter wrote: On Tue, Feb 10, 2015 at 11:42:08AM +0100, Quentin Lambert wrote: The original version was success handling rather than error handling, therefore this patch reduces nesting. Signed-off-by: Quentin Lambert lambert.quen...@gmail.com Fantastic. :)

[PATCH 1/1] staging: android: Remove allocation from declaration line

2015-02-10 Thread Quentin Lambert
This patch removes allocation from declaration line because people are known to gloss over declarations. Signed-off-by: Quentin Lambert lambert.quen...@gmail.com --- drivers/staging/android/ion/ion_page_pool.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git

Re: [PATCH 1/1] staging: ozwpan: Move code from success handling to error handling

2015-02-10 Thread Dan Carpenter
On Tue, Feb 10, 2015 at 12:08:14PM +0100, Quentin Lambert wrote: On 10/02/2015 12:04, Dan Carpenter wrote: On Tue, Feb 10, 2015 at 11:42:08AM +0100, Quentin Lambert wrote: The original version was success handling rather than error handling, therefore this patch reduces nesting.

Re: [PATCH] staging:lustre:libcfs: Merge linux-proc.c into module.c

2015-02-10 Thread Matt Tyler
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 --- drivers/staging/lustre/lustre/libcfs/Makefile | 2 +- .../lustre/lustre/libcfs/linux/linux-proc.c

[PATCH 1/2] staging: panel: register driver after checking device

2015-02-10 Thread Sudip Mukherjee
register the driver only if lcd or keypad has been enabled and if both are disabled then just exit. Signed-off-by: Sudip Mukherjee su...@vectorindia.org --- drivers/staging/panel/panel.c | 18 ++ 1 file changed, 6 insertions(+), 12 deletions(-) diff --git

[PATCH 2/2] staging: panel: initialize lcd if lcd enabled

2015-02-10 Thread Sudip Mukherjee
initialiaze lcd parameters only if lcd is enabled. Signed-off-by: Sudip Mukherjee su...@vectorindia.org --- drivers/staging/panel/panel.c | 41 ++--- 1 file changed, 22 insertions(+), 19 deletions(-) diff --git a/drivers/staging/panel/panel.c

[PATCH] staging: sm7xxfb: make vgamode static

2015-02-10 Thread Sudip Mukherjee
the variable vgamode is used only in this file and hence can be safely made as static. Signed-off-by: Sudip Mukherjee su...@vectorindia.org --- drivers/staging/sm7xxfb/sm7xx.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/sm7xxfb/sm7xx.h

Re: [PATCH 1/1] staging: unisys: Remove allocation from declaration line

2015-02-10 Thread Sudip Mukherjee
On Tue, Feb 10, 2015 at 02:02:14PM +0100, Quentin Lambert wrote: This patch removes allocation from declaration line because people are known to gloss over declarations. Signed-off-by: Quentin Lambert lambert.quen...@gmail.com --- drivers/staging/unisys/visorchipset/visorchipset_main.c | 6

[PATCH 06/30] staging: unisys: changes function visorchannel_signalinsert from bool to int

2015-02-10 Thread Benjamin Romer
From: Erik Arfvidson erik.arfvid...@unisys.com This patch changes function from visorchannel_signalinsert from bool to int. Also, changes to signalinsert_inner had to be change since visorchannel_signalinsert call that other function which return value was bool so it was change to int as well.

[PATCH 14/30] staging: unisys: remove #define bool in visorchannel.h

2015-02-10 Thread Benjamin Romer
From: Erik Arfvidson erik.arfvid...@unisys.com This patche removes pound define from visorchannel.h. Signed-off-by: Erik Arfvidson erik.arfvid...@unisys.com Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- drivers/staging/unisys/visorchannel/visorchannel.h | 3 --- 1 file changed, 3

[PATCH 11/30] staging: unisys: changes visorchipset.[ch] functions from bool to int

2015-02-10 Thread Benjamin Romer
From: Erik Arfvidson erik.arfvid...@unisys.com This patch changes the following functions in visorchipset.[ch] from bool to int: visorchipset_cache_alloc visorchipset_get_bus_info visorchipset_get_device_info visorchipset_set_bus_context Signed-off-by: Erik Arfvidson erik.arfvid...@unisys.com

[PATCH 12/30] staging: unisys: change visorchipset/parse.[ch] from bool to int

2015-02-10 Thread Benjamin Romer
From: Erik Arfvidson erik.arfvid...@unisys.com this patch changes visorchipset variables from bool to int. Signed-off-by: Erik Arfvidson erik.arfvid...@unisys.com Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- drivers/staging/unisys/visorchipset/parser.c | 24

[PATCH 27/30] staging: unisys: vmcall_channel_version_mismatch

2015-02-10 Thread Benjamin Romer
From: Jeffrey Brown jeffrey.br...@unisys.com Replaced pragma pack(push, 1) and pragma pack(pop) with __packed Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- drivers/staging/unisys/common-spar/include/vmcallinterface.h | 7

[PATCH 30/30] staging: unisys: vbuschannel.h pragma

2015-02-10 Thread Benjamin Romer
From: Jeffrey Brown jeffrey.br...@unisys.com Replaced pragma statements that was surrounding the structs spar_vbus_headerinfo and spar_vbus_channel_protocol with __packed Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com ---

[PATCH v4 1/3] staging: lustre: fix coding style errors

2015-02-10 Thread Tal Shorer
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 --- drivers/staging/lustre/lustre/libcfs/linux/linux-proc.c | 2 +- 1 file changed, 1

[PATCH 24/30] staging: unisys: vmcallintferace phys_info pragma

2015-02-10 Thread Benjamin Romer
From: Jeffrey Brown jeffrey.br...@unisys.com Changed pragma code surrounding the struct phys_info with a __packed statement Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com ---

[PATCH 20/30] staging: unisys: remove goto from sig_do_data()

2015-02-10 Thread Benjamin Romer
The gotos in this function are unnecessary, so just return. Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- drivers/staging/unisys/visorchannel/visorchannel_funcs.c | 15 +-- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git

[PATCH 05/30] staging: unisys: fix function visorchannel_signalremove from bool to int

2015-02-10 Thread Benjamin Romer
From: Erik Arfvidson erik.arfvid...@unisys.com this patch changes visorchannel_signalremove from bool to int. Also modified signalremove_inner since the return value was bool and change it to int. Signed-off-by: Erik Arfvidson erik.arfvid...@unisys.com Signed-off-by: Benjamin Romer

[PATCH 07/30] staging: unisys: changes sig_read_header function from bool to int

2015-02-10 Thread Benjamin Romer
From: Erik Arfvidson erik.arfvid...@unisys.com This patch changes function sig_read_header return value from bool to int Signed-off-by: Erik Arfvidson erik.arfvid...@unisys.com Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- drivers/staging/unisys/visorchannel/visorchannel_funcs.c |

[PATCH 18/30] staging: unisys: change uislib.c variables from type bool to int

2015-02-10 Thread Benjamin Romer
From: Erik Arfvidson erik.arfvid...@unisys.com this patch changes uislib.c variables from type bool to int Signed-off-by: Erik Arfvidson erik.arfvid...@unisys.com Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- drivers/staging/unisys/uislib/uislib.c | 16 1 file

[PATCH 02/30] staging: unisys: change serverchangingstate variable bool to int

2015-02-10 Thread Benjamin Romer
From: Erik Arfvidson erik.arfvid...@unisys.com This patch changes serverchangingstate variable from bool to int Signed-off-by: Erik Arfvidson erik.arfvid...@unisys.com Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- drivers/staging/unisys/virthba/virthba.c | 12 ++-- 1 file

[PATCH 17/30] staging: unisys: change visorkmodutils.c variable from bool to int.

2015-02-10 Thread Benjamin Romer
From: Erik Arfvidson erik.arfvid...@unisys.com this patch changes the value of variable unisys_spar_plataform from bool to int. Signed-off-by: Erik Arfvidson erik.arfvid...@unisys.com Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- drivers/staging/unisys/visorutil/visorkmodutils.c

[PATCH 01/30] staging: unisys: serverdown variable change bool to int virthba

2015-02-10 Thread Benjamin Romer
From: Erik Arfvidson erik.arfvid...@unisys.com This patch changes serverdown variable to int instead of bool Signed-off-by: Erik Arfvidson erik.arfvid...@unisys.com Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- drivers/staging/unisys/virthba/virthba.c | 8 1 file changed,

[PATCH 00/30] staging: unisys: cleanup patch series

2015-02-10 Thread Benjamin Romer
This set of patches cleans up some more of the code by eliminating unnecessary goto statements, getting rid of the BOOL define and switching over code that used it to use int instead, and switching from using pragma(pack) to __packed. *** BLURB HERE *** Benjamin Romer (3): staging: unisys:

Re: [RFC PATCH 0/5] unisys: kthread cleanup

2015-02-10 Thread Romer, Benjamin M
On Mon, 2015-01-26 at 00:39 -0500, devendra.aaru wrote: Hello, Sorry for the late reply. Would reverting last two patches help ? Thanks, Devendra Devendra, I am really sorry for how long this is taking. I don't think that there's anything wrong with your patches, but rather that

[PATCH 09/30] staging: unisys: changes bool to int in sig related function

2015-02-10 Thread Benjamin Romer
From: Erik Arfvidson erik.arfvid...@unisys.com This patch changes the return value from bool to int to the following sig related function: sig_read_data, sig_write_data, and sig_do_data Signed-off-by: Erik Arfvidson erik.arfvid...@unisys.com Signed-off-by: Benjamin Romer

[PATCH 16/30] staging: unisys: change visorchannel_funcs.c function variable from bool to int

2015-02-10 Thread Benjamin Romer
From: Erik Arfvidson erik.arfvid...@unisys.com This patch changes visorchanne_funcs.c function variable from bool to int. Signed-off-by: Erik Arfvidson erik.arfvid...@unisys.com Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- drivers/staging/unisys/visorchannel/visorchannel_funcs.c

[PATCH 03/30] staging: unisys: change bool to int variable found

2015-02-10 Thread Benjamin Romer
From: Erik Arfvidson erik.arfvid...@unisys.com this patch changes variable 'found' from bool to int Signed-off-by: Erik Arfvidson erik.arfvid...@unisys.com Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com --- drivers/staging/unisys/virtpci/virtpci.c | 8 1 file changed, 4

[PATCH 29/30] staging: unisys: iochannel.h pragma

2015-02-10 Thread Benjamin Romer
From: Jeffrey Brown jeffrey.br...@unisys.com Replaced pragma code surrounding multiple structs at once with __packed statements for each struct Signed-off-by: Jeffrey Brown jeffrey.br...@unisys.com Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com ---

[PATCH v4 2/3] staging: lustre: fix coding style errors

2015-02-10 Thread Tal Shorer
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: Tal Shorer tal.sho...@gmail.com ---

[PATCH v4 3/3] staging: lustre: fix coding style errors fix the

2015-02-10 Thread Tal Shorer
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 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: doubt about sm7xxfb

2015-02-10 Thread Greg Kroah-Hartman
On Wed, Feb 11, 2015 at 11:58:18AM +0530, Sudip Mukherjee wrote: On Wed, Feb 11, 2015 at 12:36:36PM +0800, Greg Kroah-Hartman wrote: On Tue, Feb 10, 2015 at 07:37:47PM +0530, Sudip Mukherjee wrote: Hi Greg, we have this sm7xxfb in drivers/staging now which is supporting SM710, SM712,

[PATCH v2 2/6] staging: rtl8188eu: hal: removed code indent error

2015-02-10 Thread Pushpendra Singh
This patch removed indentation with a tab error ERROR: code indent should use tabs where possible Signed-off-by: Pushpendra Singh pushpendra.si...@smartplayin.com --- drivers/staging/rtl8188eu/hal/rf.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

Re: [PATCH 2/6] staging: rtl8188eu: hal: removed code indent error

2015-02-10 Thread Greg KH
On Tue, Feb 10, 2015 at 06:32:49PM +0530, Pushpendra Singh wrote: Removed checkpatch.pl error ERROR: code indent should use tabs where possible Then fix the indentation with a tab, not using over 8 spaces, don't make this a new coding style issue that I will then get an additional patch for

Re: [PATCH 1/1] staging: android: Remove allocation from declaration line

2015-02-10 Thread Greg Kroah-Hartman
On Tue, Feb 10, 2015 at 12:09:45PM +0100, Quentin Lambert wrote: This patch removes allocation from declaration line because people are known to gloss over declarations. Then those people should not be reviewing kernel code :) Seriously, this is unneeded churn, the code is fine as-is. thanks,

Re: [PATCH 1/1] staging: unisys: Remove allocation from declaration line

2015-02-10 Thread Greg Kroah-Hartman
On Tue, Feb 10, 2015 at 02:02:14PM +0100, Quentin Lambert wrote: This patch removes allocation from declaration line because people are known to gloss over declarations. Again, who are these lazy people, and why are they reading kernel code? Please, it's ok to allocate things in the

[PATCHv2 1/2] ft1000-pcmcia: ft1000_hw.c: fix style issues not requiring code refactoring

2015-02-10 Thread Daniele Alessandrelli
Fix all the trivial style issues (as reported by checkpatch.pl) not requiring code refactoring. A following patch is expected to fix the remaining issues by performing some code refactoring. Signed-off-by: Daniele Alessandrelli daniele.alessandre...@gmail.com ---

[PATCHv2 2/2] ft1000-pcmcia: ft1000_hw.c: code refactoring: add ft1000_read_dsp_timer()

2015-02-10 Thread Daniele Alessandrelli
Add new function ft1000_read_dsp_timer() replacing recurring code block for reading DSP timer. Such code refactoring solves all remaining line over 80 characters warnings reported by checkpatch.pl. Signed-off-by: Daniele Alessandrelli daniele.alessandre...@gmail.com ---

[PATCHv2 0/2] staging: ft1000-pcmcia: ft1000_hw.c: fix checkpatch warnings

2015-02-10 Thread Daniele Alessandrelli
This small patchset fixes all the style problems reported by checkpatch.pl on ft1000-pcmcia/ft1000_hw.c * Patch 1/2 fixes all trivial issues not requiring code refactoring * Patch 2/2 fixes all remaining line over 80 characters warnings by means of some code refactoring. Specifically, the

[PATCH 2/6] staging: rtl8188eu: hal: removed code indent error

2015-02-10 Thread Pushpendra Singh
Removed checkpatch.pl error ERROR: code indent should use tabs where possible Signed-off-by: Pushpendra Singh pushpendra.si...@smartplayin.com --- drivers/staging/rtl8188eu/hal/rf.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/rtl8188eu/hal/rf.c

[PATCH 4/6] staging: rtl8188eu: hal: added blank line after declarations

2015-02-10 Thread Pushpendra Singh
Removed checkpatch.pl below warning WARNING: Missing a blank line after declarations Signed-off-by: Pushpendra Singh pushpendra.si...@smartplayin.com --- .../staging/rtl8188eu/hal/Hal8188ERateAdaptive.c | 1 + drivers/staging/rtl8188eu/hal/hal_intf.c | 2 ++

[PATCH 1/6] staging: rt8188eu: hal: removed space before ','

2015-02-10 Thread Pushpendra Singh
Removed checkpatch.pl error ERROR: space prohibited before that ',' (ctx:WxW) Signed-off-by: Pushpendra Singh pushpendra.si...@smartplayin.com --- drivers/staging/rtl8188eu/hal/Hal8188ERateAdaptive.c | 4 ++-- drivers/staging/rtl8188eu/hal/odm.c | 8

[PATCH 3/6] staging: rtl8188eu: hal: removed unnecessary braces

2015-02-10 Thread Pushpendra Singh
Removed checkpatch.pl below warning WARNING: braces {} are not necessary for single statement blocks Signed-off-by: Pushpendra Singh pushpendra.si...@smartplayin.com --- drivers/staging/rtl8188eu/hal/bb_cfg.c| 15 +++ drivers/staging/rtl8188eu/hal/fw.c| 3

[PATCH 5/6] staging: rtl8188eu: hal: removed spaces at start of line

2015-02-10 Thread Pushpendra Singh
Removed checkpatch.pl below warning WARNING: please, no spaces at the start of a line Signed-off-by: Pushpendra Singh pushpendra.si...@smartplayin.com --- drivers/staging/rtl8188eu/hal/usb_halinit.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 6/6] staging: rtl8188eu: hal: removed unnecessary parentheses

2015-02-10 Thread Pushpendra Singh
Removed checkpatch.pl below warning WARNING: Unnecessary parentheses - maybe == should be = ? Signed-off-by: Pushpendra Singh pushpendra.si...@smartplayin.com --- drivers/staging/rtl8188eu/hal/usb_halinit.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 1/1] staging: unisys: Remove allocation from declaration line

2015-02-10 Thread Quentin Lambert
This patch removes allocation from declaration line because people are known to gloss over declarations. Signed-off-by: Quentin Lambert lambert.quen...@gmail.com --- drivers/staging/unisys/visorchipset/visorchipset_main.c | 6 +++--- drivers/staging/unisys/visorutil/charqueue.c| 3

NOTIS RASMI HADIAH TELEKOM MALAYSIA

2015-02-10 Thread Telekom Malaysia Berhad
Telekom Malaysia Berhad G.03B, Ground Floor, Kompleks Antarabangsa, Jln Sultan Ismail, Off Jalan Ampang 50250 Kuala Lumpur Malaysia. NOTIS RASMI HADIAH TELEKOM MALAYSIA Pihak Telekom Malaysia @Program Kemenangan yang telah diadakan pada 9th Februari 2015 di mana alamat email anda yang

doubt about sm7xxfb

2015-02-10 Thread Sudip Mukherjee
Hi Greg, we have this sm7xxfb in drivers/staging now which is supporting SM710, SM712, SM721 and SM722. I am also working on another new hardware SM750, which will be ready for staging in next 1 -2 weeks. this SM750 is entirely different hardware and these two drivers will have nothing in

[PATCH v2 1/1] staging: unisys: Remove allocation from declaration line

2015-02-10 Thread Quentin Lambert
This patch removes allocation from declaration line because people are known to gloss over declarations. Signed-off-by: Quentin Lambert lambert.quen...@gmail.com --- The second of version of this patch fix the checkpatch warning of line over 80 char by rewriting the size argument of kmalloc as

Re: [PATCH 2/6] staging: rtl8188eu: hal: removed code indent error

2015-02-10 Thread Sudip Mukherjee
On Tue, Feb 10, 2015 at 06:32:49PM +0530, Pushpendra Singh wrote: Removed checkpatch.pl error ERROR: code indent should use tabs where possible Signed-off-by: Pushpendra Singh pushpendra.si...@smartplayin.com --- drivers/staging/rtl8188eu/hal/rf.c | 3 +-- 1 file changed, 1 insertion(+),