Re: [PATCH] Staging: rtl8188eu: fix coding style errors

On Fri, Feb 27, 2015 at 05:44:09PM +0100, Matteo Semenzato wrote: > diff --git a/drivers/staging/rtl8188eu/core/rtw_efuse.c > b/drivers/staging/rtl8188eu/core/rtw_efuse.c > index 8816d11..c31fdbc 100644 > --- a/drivers/staging/rtl8188eu/core/rtw_efuse.c > +++ b/drivers/staging/rtl8188eu/core/rtw_e

Re: [PATCH] Staging: rtl8188eu: fix coding style errors

On Fri, Feb 27, 2015 at 10:58:28AM -0600, Larry Finger wrote: > On 02/27/2015 10:44 AM, Matteo Semenzato wrote: > >From: Matteo Semenzato > > > >This patch fixes the following errors: > >ERROR: space prohibited before that ',' (ctx:WxW) > >ERROR: space prohibited before that ',' (ctx:WxW) > >ERROR

Re: [PATCH V2] Staging: fbtft: fix coding style errors

On Sat, Feb 28, 2015 at 06:37:30PM +0100, Matteo Semenzato wrote: > static int hy28b_init_sequence[] = { > - -1,0x00e7,0x0010,-1,0x,0x0001,-1,0x0001,0x0100,-1,0x0002,0x0700, > - -1,0x0003,0x1030,-1,0x0004,0x,-1,0x0008,0x0207,-1,0x0009,0x, > - -1,0x000a,0x,-1,0x000c,0x00

Re: [PATCH] Staging: fbtft: fix space errors

On Sat, Feb 28, 2015 at 06:59:19AM -0800, Joe Perches wrote: > If you're really going to change these, please > remove the unnecessary \ line continuations > indent the blocks properly and group the blocks > more intelligibly. Maybe something like; > > static const int st7735r_init[] = { >

Re: [PATCH char-misc] mei: bus: () can be static

On Mon, Mar 02, 2015 at 11:49:23AM +0800, kbuild test robot wrote: > 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/v

RE: [PATCH char-misc] mei: bus: () can be static

> -Original Message- > From: Dan Carpenter [mailto:dan.carpen...@oracle.com] > Sent: Monday, March 02, 2015 11:41 > To: Wu, Fengguang > Cc: Winkler, Tomas; Greg Kroah-Hartman; Haiyang Zhang; linux- > ker...@vger.kernel.org; Nick Meier; kbuild-...@01.org; > de...@linuxdriverproject.org > S

[PATCH 03/27 V2] staging: lustre: Convert uses of "int rc = seq_printf(...)"

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: seq_printf(seq, ...); return 0; Done via cocci sc

Re: [PATCH] Drivers: hv: hv_balloon: keep locks balanced on add_memory() failure

Vitaly Kuznetsov writes: > When add_memory() fails the following BUG is observed: > [ 743.646107] hv_balloon: hot_add memory failed error is -17 > [ 743.679973] > [ 743.680930] = > [ 743.680930] [ BUG: bad unlock balance detected! ] > [ 743.680930] 3.19.0-

[RFC 4/7] staging: fbtft: lcdctrl: add ssd1306 support

Add support for Solomon SSD1306 Monochrome OLED controller. The controller supports up to 128x64 pixles and supports interfaces: Parallel 6800, parallel 8080, SPI (3/4), I2C. Signed-off-by: Noralf Trønnes --- drivers/staging/fbtft/lcdctrl/Kconfig | 5 + drivers/staging/fbtft/lcdctrl/Makefile

[RFC 5/7] staging: fbtft: don't require platform data

Add dummy platform data when it's not present. Signed-off-by: Noralf Trønnes --- drivers/staging/fbtft/fbtft-core.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/staging/fbtft/fbtft-core.c b/drivers/staging/fbtft/fbtft-core.c index ac4287f..59c17c1 100644 ---

[RFC 2/7] staging: fbtft: lcdreg: add i2c support

Add I2C bus support to lcdreg. Signed-off-by: Noralf Trønnes --- drivers/staging/fbtft/lcdreg/Kconfig | 6 ++ drivers/staging/fbtft/lcdreg/Makefile | 2 + drivers/staging/fbtft/lcdreg/lcdreg-i2c.c | 129 ++ drivers/staging/fbtft/lcdreg/lcdreg.h | 3

[RFC 0/7] staging: fbtft: minimize coupling to the fbdev subsystem

Hi, This patchset introduces a new API to minimize the coupling to the fbdev graphics subsystem. There have been calls to deprecate fbdev and new fbdev drivers are discouraged. Currently the FBTFT drivers are tightly coupled to fbdev through the fbtft module and the fbtft_par structure. What is F

[RFC 7/7] staging: fbtft: add driver for Adafruit ssd1306 based displays

The Adafuit SSD1306 based displays comes in either 128x32 or 128x64 resolutions and uses I2C or SPI interface. This driver has support for the I2C interface. Support for SPI can be added when lcdreg has SPI support. Signed-off-by: Noralf Trønnes --- drivers/staging/fbtft/Kconfig | 9 ++

[RFC 6/7] staging: fbtft: extend core to use lcdctrl

Add lcdctrl support in core fbtft module. Provide new API for drivers to use. Signed-off-by: Noralf Trønnes --- drivers/staging/fbtft/Makefile| 1 + drivers/staging/fbtft/fbtft-lcdctrl.c | 311 ++ drivers/staging/fbtft/fbtft.h | 13 ++ 3 files c

[RFC 1/7] staging: fbtft: add lcd register abstraction

Add LCD register abstraction for MIPI DCS like controllers. This hides LCD controller interface implementation details. When built with debugfs, the register is available to userspace. Signed-off-by: Noralf Trønnes --- drivers/staging/fbtft/Kconfig | 2 + drivers/staging/fbtft/

[RFC 3/7] staging: fbtft: add lcd controller abstraction

Add abstraction for MIPI DCS/DBI like LCD controllers. Signed-off-by: Noralf Trønnes --- drivers/staging/fbtft/Kconfig | 1 + drivers/staging/fbtft/Makefile | 1 + drivers/staging/fbtft/lcdctrl/Kconfig | 3 + drivers/staging/fbtft/lcdctrl/Makefile | 1 + drivers/sta

Re: [RFC 5/7] staging: fbtft: don't require platform data

On Mon, Mar 02, 2015 at 11:54:27AM +0100, Noralf Trønnes wrote: > Add dummy platform data when it's not present. > > Signed-off-by: Noralf Trønnes > --- > drivers/staging/fbtft/fbtft-core.c | 6 ++ > 1 file changed, 2 insertions(+), 4 deletions(-) > > diff --git a/drivers/staging/fbtft/fbtf

Re: [RFC 3/7] staging: fbtft: add lcd controller abstraction

On Mon, Mar 02, 2015 at 11:54:25AM +0100, Noralf Trønnes wrote: > +int lcdctrl_enable(struct lcdctrl *ctrl, void *videomem) > +{ > + struct lcdctrl_update update = { > + .base = videomem, > + .ys = 0, > + .ye = lcdctrl_yres(ctrl) - 1, > + }; > + int r

Re: [RFC 1/7] staging: fbtft: add lcd register abstraction

On Mon, Mar 02, 2015 at 11:54:23AM +0100, Noralf Trønnes wrote: > diff --git a/drivers/staging/fbtft/lcdreg/lcdreg-debugfs.c > b/drivers/staging/fbtft/lcdreg/lcdreg-debugfs.c > new file mode 100644 > index 000..1cba4c2 > --- /dev/null > +++ b/drivers/staging/fbtft/lcdreg/lcdreg-debugfs.c > @@

Re: [PATCH] Staging: rtl8188eu: fix coding style errors

On Mon, Mar 02, 2015 at 11:46:57AM +0300, Dan Carpenter wrote: > On Fri, Feb 27, 2015 at 10:58:28AM -0600, Larry Finger wrote: > > On 02/27/2015 10:44 AM, Matteo Semenzato wrote: > > >From: Matteo Semenzato > > > > > >This patch fixes the following errors: > > >ERROR: space prohibited before that

[PATCH V2] Staging: fbtft: fix space prohibition

From: Matteo Semenzato This patch fixes the following warnings: space prohibited before that close parenthesis ')' space prohibited after that open parenthesis '(' V2: The patch should apply now. Signed-off-by: Matteo Semenzato --- drivers/staging/fbtft/fb_bd663474.c | 100 ++-

Re: [PATCH RFC 0/3] Drivers: hv: utils: re-implement the kernel/userspace communication layer

Radim Krčmář writes: > 2015-02-27 17:14+0100, Vitaly Kuznetsov: >> This series converts kvp/vss daemons to use misc char devices instead of >> netlink for userspace/kernel communication and then updates fcopy to be >> consistent with kvp/vss. >> >> Userspace/kernel communication via netlink has

[PATCH V2] Staging: fbtft: fix coding style errors

From: Matteo Semenzato This patch fixes the following error: space required after that ';' V2: Indented and organized the init sequences. Signed-off-by: Matteo Semenzato --- drivers/staging/fbtft/fbtft_device.c | 145 +++ 1 file changed, 111 insertions(+), 34 d

Re: [PATCH RFC 0/3] Drivers: hv: utils: re-implement the kernel/userspace communication layer

KY Srinivasan writes: >> -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-Hart

[PATCH V2] Staging: fbtft: fix whitespace errors

From: Matteo Semenzato This patch fixes the following errors: ERROR: space required after that ',' ERROR: trailing whitespace ERROR: code indent should use tabs where possible V2: The patch should apply now Signed-off-by: Matteo Semenzato --- drivers/staging/fbtft/fb_st7735r.c | 24 ++

Re: [PATCH] Staging: fbtft: fix space errors

On Mon, 2015-03-02 at 12:37 +0300, Dan Carpenter wrote: > On Sat, Feb 28, 2015 at 06:59:19AM -0800, Joe Perches wrote: > > If you're really going to change these, please > > remove the unnecessary \ line continuations > > indent the blocks properly and group the blocks > > more intelligibly. Maybe

Re: [PATCH] Staging: fbtft: fix space errors

On 02 Mar, Joe Perches wrote: > On Mon, 2015-03-02 at 12:37 +0300, Dan Carpenter wrote: > > On Sat, Feb 28, 2015 at 06:59:19AM -0800, Joe Perches wrote: > > > If you're really going to change these, please > > > remove the unnecessary \ line continuations > > > indent the blocks properly and group

Re: [PATCH] Staging: fbtft: fix space errors

Den 02.03.2015 20:21, skrev Andrey Skvortsov: On 02 Mar, Joe Perches wrote: On Mon, 2015-03-02 at 12:37 +0300, Dan Carpenter wrote: On Sat, Feb 28, 2015 at 06:59:19AM -0800, Joe Perches wrote: If you're really going to change these, please remove the unnecessary \ line continuations indent th

[PATCH 04/28] staging: rtl8723au: ODM_BB_RA_MASK is always set

From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/hal/odm.c | 17 ++--- drivers/staging/rtl8723au/hal/rtl8723a_dm.c | 1 - drivers/staging/rtl8723au/include/odm.h | 1 - 3 files changed, 2 insertions(+), 17 deletions(-) diff --git a/drivers/

[PATCH 02/28] staging: rtl8723au: Avoid zero initializing variables unnecessarily

From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/hal/odm.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/rtl8723au/hal/odm.c b/drivers/staging/rtl8723au/hal/odm.c index 2c604da..a77d151 100644 --- a/drivers/staging/rtl8723au

[PATCH 13/28] staging: rtl8723au: Remove unused ODM ability flags

From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/include/odm.h | 11 --- 1 file changed, 11 deletions(-) diff --git a/drivers/staging/rtl8723au/include/odm.h b/drivers/staging/rtl8723au/include/odm.h index 7d505cf..78c17f3 100644 --- a/drivers/staging/rtl87

[PATCH 24/28] staging: rtl8723au: Clean up odm_RSSIMonitorCheck()

From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/hal/odm.c | 41 +++-- 1 file changed, 21 insertions(+), 20 deletions(-) diff --git a/drivers/staging/rtl8723au/hal/odm.c b/drivers/staging/rtl8723au/hal/odm.c index b9f09d9..bfbb1a4 10

[PATCH 06/28] staging: rtl8723au: ODM_BB_DYNAMIC_TXPWR isn't used for anything

From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/hal/rtl8723a_dm.c | 3 +-- drivers/staging/rtl8723au/include/odm.h | 1 - 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/staging/rtl8723au/hal/rtl8723a_dm.c b/drivers/staging/rtl8723au/hal/r

[PATCH 23/28] staging: rtl8723au: ODM_TXPowerTrackingCheck23a(): Remove no-op function

From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/hal/odm.c | 20 1 file changed, 20 deletions(-) diff --git a/drivers/staging/rtl8723au/hal/odm.c b/drivers/staging/rtl8723au/hal/odm.c index d3d4efc..b9f09d9 100644 --- a/drivers/staging/rtl8723a

[PATCH 26/28] staging: rtl8723au: Clean up odm_RefreshRateAdaptiveMask()

From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/hal/odm.c | 27 +++ 1 file changed, 15 insertions(+), 12 deletions(-) diff --git a/drivers/staging/rtl8723au/hal/odm.c b/drivers/staging/rtl8723au/hal/odm.c index 5269c39..e4500e9 100644 --- a

[PATCH 14/28] staging: rtl8723au: Make odm_PHY_SaveAFERegisters() readable

From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/hal/odm.c | 9 ++--- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/drivers/staging/rtl8723au/hal/odm.c b/drivers/staging/rtl8723au/hal/odm.c index 3077735..c50b8a5 100644 --- a/drivers/staging/rtl8

[PATCH 18/28] staging: rtl8723au: Remove write-only variable ControlChannel

From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/hal/odm.c | 9 - drivers/staging/rtl8723au/include/odm.h | 1 - 2 files changed, 10 deletions(-) diff --git a/drivers/staging/rtl8723au/hal/odm.c b/drivers/staging/rtl8723au/hal/odm.c index 11c9a36..cb2b7

[PATCH 17/28] staging: rtl8723au: SupportInterface is always set to USB

From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/hal/HalHWImg8723A_BB.c | 9 ++ drivers/staging/rtl8723au/hal/HalHWImg8723A_MAC.c | 3 +- drivers/staging/rtl8723au/hal/HalHWImg8723A_RF.c | 3 +- drivers/staging/rtl8723au/hal/odm.c | 4 --- dri

[PATCH 21/28] staging: rtl8723au: Remove unused struct pri_cca

From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/include/odm.h | 9 - 1 file changed, 9 deletions(-) diff --git a/drivers/staging/rtl8723au/include/odm.h b/drivers/staging/rtl8723au/include/odm.h index 3f93b3d..3a68774 100644 --- a/drivers/staging/rtl8723au

[PATCH 28/28] staging: rtl8723au: odm.c: Break some lines down to 80 characters

From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/hal/odm.c | 184 +++- 1 file changed, 117 insertions(+), 67 deletions(-) diff --git a/drivers/staging/rtl8723au/hal/odm.c b/drivers/staging/rtl8723au/hal/odm.c index 69072c0..8d42661 1

[PATCH 27/28] staging: rtl8723au: ODM_Write_DIG23A(): Cosmetic cleanups

From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/hal/odm.c | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/drivers/staging/rtl8723au/hal/odm.c b/drivers/staging/rtl8723au/hal/odm.c index e4500e9..69072c0 100644 --- a/drivers/stagi

[PATCH 00/28] staging: rtl8723au: Another round of cleanups

From: Jes Sorensen Hi, Another round of cleanups getting rid of unnecessary cruft in the rtl8723au driver. Cheers, Jes Jes Sorensen (28): staging: rtl8723au: Variable bbtchange is always false staging: rtl8723au: Avoid zero initializing variables unnecessarily staging: rtl8723au: ODM_MAC

[PATCH 15/28] staging: rtl8723au: odm_dtc(): Remove no-op function

From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/hal/odm.c | 7 --- drivers/staging/rtl8723au/include/odm.h | 2 -- 2 files changed, 9 deletions(-) diff --git a/drivers/staging/rtl8723au/hal/odm.c b/drivers/staging/rtl8723au/hal/odm.c index c50b8a5..5a0568b

[PATCH 03/28] staging: rtl8723au: ODM_MAC_EDCA_TURBO is always set

From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/hal/odm.c | 3 --- drivers/staging/rtl8723au/hal/rtl8723a_dm.c | 1 - drivers/staging/rtl8723au/include/odm.h | 1 - 3 files changed, 5 deletions(-) diff --git a/drivers/staging/rtl8723au/hal/odm.c b/driv

[PATCH 09/28] staging: rtl8723au: ODM_BB_CCK_PD is always set

From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/hal/odm.c | 3 --- drivers/staging/rtl8723au/hal/rtl8723a_dm.c | 3 +-- drivers/staging/rtl8723au/include/odm.h | 1 - 3 files changed, 1 insertion(+), 6 deletions(-) diff --git a/drivers/staging/rtl8723au

[PATCH 19/28] staging: rtl8723au: Remove unused Funai TV hack

From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/hal/odm.c | 8 drivers/staging/rtl8723au/hal/rtl8723a_dm.c | 1 - drivers/staging/rtl8723au/include/odm.h | 2 -- 3 files changed, 11 deletions(-) diff --git a/drivers/staging/rtl8723au/hal/odm.c

[PATCH 07/28] staging: rtl8723au: ODM_BB_FA_CNT is always set

From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/hal/odm.c | 11 +-- drivers/staging/rtl8723au/hal/rtl8723a_dm.c | 3 +-- drivers/staging/rtl8723au/include/odm.h | 1 - 3 files changed, 2 insertions(+), 13 deletions(-) diff --git a/drivers/stag

[PATCH 10/28] staging: rtl8723au: ODM_BB_PWR_SAVE is unused

From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/hal/rtl8723a_dm.c | 3 +-- drivers/staging/rtl8723au/include/odm.h | 1 - 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/staging/rtl8723au/hal/rtl8723a_dm.c b/drivers/staging/rtl8723au/hal/r

[PATCH 16/28] staging: rtl8723au: Remove no-op ODM_CMNINFO_PLATFORM

From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/hal/odm.c | 2 -- drivers/staging/rtl8723au/hal/rtl8723a_dm.c | 1 - drivers/staging/rtl8723au/include/odm.h | 15 +++ 3 files changed, 7 insertions(+), 11 deletions(-) diff --git a/drivers/s

[PATCH 05/28] staging: rtl8723au: ODM_BB_DIG is always set

From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/hal/odm.c | 5 ++--- drivers/staging/rtl8723au/hal/rtl8723a_dm.c | 3 +-- drivers/staging/rtl8723au/include/odm.h | 1 - 3 files changed, 3 insertions(+), 6 deletions(-) diff --git a/drivers/staging/rtl872

[PATCH 08/28] staging: rtl8723au: ODM_BB_RSSI_MONITOR is always set

From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/hal/odm.c | 17 + drivers/staging/rtl8723au/hal/rtl8723a_dm.c | 3 +-- drivers/staging/rtl8723au/include/odm.h | 1 - 3 files changed, 2 insertions(+), 19 deletions(-) diff --git a/driver

[PATCH 22/28] staging: rtl8723au: Remove write only bIsCurRDLState

From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/hal/odm.c | 1 - drivers/staging/rtl8723au/include/odm.h | 1 - 2 files changed, 2 deletions(-) diff --git a/drivers/staging/rtl8723au/hal/odm.c b/drivers/staging/rtl8723au/hal/odm.c index d8f2095..d3d4efc 100644

[PATCH 12/28] staging: rtl8723au: ODM_RF_CALIBRATION is never set

From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/hal/rtl8723a_dm.c | 2 +- drivers/staging/rtl8723au/include/odm.h | 2 -- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/staging/rtl8723au/hal/rtl8723a_dm.c b/drivers/staging/rtl8723au/hal/r

[PATCH 20/28] staging: rtl8723au: Remove unused struct odm_fat_t

From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/include/odm.h | 28 1 file changed, 28 deletions(-) diff --git a/drivers/staging/rtl8723au/include/odm.h b/drivers/staging/rtl8723au/include/odm.h index 3a65195..3f93b3d 100644 --- a/driv

[PATCH 25/28] staging: rtl8723au: Clean up FindMinimumRSSI()

From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/hal/odm.c | 10 -- 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/drivers/staging/rtl8723au/hal/odm.c b/drivers/staging/rtl8723au/hal/odm.c index bfbb1a4..5269c39 100644 --- a/drivers/staging/rt

[PATCH 01/28] staging: rtl8723au: Variable bbtchange is always false

From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/hal/odm.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/rtl8723au/hal/odm.c b/drivers/staging/rtl8723au/hal/odm.c index 1b2baae..2c604da 100644 --- a/drivers/staging/rtl8723au/h

[PATCH 11/28] staging: rtl8723au: ODM_RF_TX_PWR_TRACK is always set

From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/hal/HalDMOutSrc8723A_CE.c | 11 +-- drivers/staging/rtl8723au/hal/rtl8723a_dm.c | 3 +-- drivers/staging/rtl8723au/include/odm.h | 1 - 3 files changed, 2 insertions(+), 13 deletions(-) d

RE: [char-misc:char-misc-testing 25/45] drivers/hv/vmbus_drv.c:67:9: sparse: constant 0x8000000000000000 is so big it is unsigned long

I will correct this constant definition and resubmit. -Nick Meier -Original Message- From: kbuild test robot [mailto:fengguang...@intel.com] Sent: Sunday, March 1, 2015 8:49 PM To: Nick Meier Cc: kbuild-...@01.org; Greg Kroah-Hartman; KY Srinivasan; Haiyang Zhang; de...@linuxdriverproje

RE: [PATCH char-misc] mei: bus: () can be static

Yes - the function should have been static. This change correctly corrects the oversight. thanks, -Nick -Original Message- From: kbuild test robot [mailto:fengguang...@intel.com] Sent: Sunday, March 1, 2015 8:49 PM To: Tomas Winkler Cc: kbuild-...@01.org; Greg Kroah-Hartman; Nick Meie

Re: [PATCH char-misc] mei: bus: () can be static

On Mon, Mar 02, 2015 at 12:41:05PM +0300, Dan Carpenter wrote: > On Mon, Mar 02, 2015 at 11:49:23AM +0800, kbuild test robot wrote: > > 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 'hyper

[PATCH v2] staging: dgnc: Remove sysfs file

sysfs file is not useful for anything. Signed-off-by: Salah Triki --- drivers/staging/dgnc/Makefile | 3 +- drivers/staging/dgnc/dgnc_driver.c | 10 +- drivers/staging/dgnc/dgnc_driver.h | 2 - drivers/staging/dgnc/dgnc_sysfs.c | 713 - drivers/stag

[PATCH] Staging: fbtft: fb_pcd8544: Fixed coding style errors and warnings

The following errors were fixed: ERROR: code indent should use tabs where possible WARNING: line over 80 characters Signed-off-by: Cláudio Maia --- drivers/staging/fbtft/fb_pcd8544.c | 74 +++- 1 file changed, 39 insertions(+), 35 deletions(-)

[PATCH] Staging: dgnc: use goto for error handling in dgnc_start()

The error handling for register_chrdev is taken care by goto statement. The failure code from register_chrdev is returned. Signed-off-by: Salah Triki --- drivers/staging/dgnc/dgnc_driver.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/dgnc/dgnc_driver.c b

Re: [PATCH] Staging: fbtft: fb_pcd8544: Fixed coding style errors and warnings

On Tue, Mar 03, 2015 at 03:36:17AM +, Cláudio Maia wrote: > The following errors were fixed: > > ERROR: code indent should use tabs where possible > WARNING: line over 80 characters > > Signed-off-by: Cláudio Maia Don't indent this. > @@ -52,60 +52,63 @@ static int init_dis

Re: [PATCH] Staging: dgnc: use goto for error handling in dgnc_start()

On Tue, Mar 03, 2015 at 04:55:31AM +0100, Salah Triki wrote: > The error handling for register_chrdev is taken care by goto statement. No. Direct returns are more readable. Don't add extra bunny hops for no reason. > The failure code from register_chrdev is returned. > The test is "if (rc <=

[PATCH] staging/lustre/llite: Fix obd name after c&p error

From: Oleg Drokin commit 95745e9b1de2 ("staging: lustre: Use kasprintf.") introduced a copy and paste error causing two different obd types to be assigned same content causing lustre to fail on mount with a warning from procfs followed by a bizzare error about OST not having enough MDS capabiliti