[PATCH] staging: rtl8188eu: core: find and remove code valid only for 5 HGz.

2015-07-15 Thread Sreenath Madasu
This one of the TODO tasks for staging rtl8188eu driver. I have removed the code referring to channel > 14 for rtw_ap.c, rtw_ieee80211.c and rtw_mlme.c files. Please review. Signed-off-by: Sreenath Madasu --- drivers/staging/rtl8188eu/core/rtw_ap.c| 31 +++--- drivers

RE: [PATCH] Drivers: hv: vmbus: prevent new subchannel creation on device shutdown

2015-07-15 Thread KY Srinivasan
> -Original Message- > From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] > Sent: Tuesday, July 14, 2015 9:03 AM > To: Dexuan Cui > Cc: de...@linuxdriverproject.org; KY Srinivasan; Haiyang Zhang; linux- > ker...@vger.kernel.org > Subject: Re: [PATCH] Drivers: hv: vmbus: prevent new subch

Re: [PATCH] staging: unisys: Add s-Par visorhba

2015-07-15 Thread Jes Sorensen
"Kershner, David A" writes: >> -Original Message- >> From: Dan Carpenter [mailto:dan.carpen...@oracle.com] >> Sent: Wednesday, July 15, 2015 8:16 AM >> To: Romer, Benjamin M >> Cc: gre...@linuxfoundation.org; driverdev-devel@linuxdriverproject.org; >> jes.soren...@redhat.com; *S-Par-Mainta

[PATCH] sm750fb: coding style fixes lines over 80 chars

2015-07-15 Thread Vinay Simha BN
scripts/checkpatch.pl kernel coding style fixes of WARNING WARNING: line over 80 characters Signed-off-by: Vinay Simha BN --- drivers/staging/sm750fb/ddk750_display.h | 10 +- drivers/staging/sm750fb/ddk750_help.h| 6 -- drivers/staging/sm750fb/ddk750_hwi2c.h | 6 -- dri

Re: [PATCH] staging: unisys: Add s-Par visorhba

2015-07-15 Thread Dan Carpenter
On Wed, Jul 15, 2015 at 04:53:24PM +, Kershner, David A wrote: > ERROR: Macros with complex values should be enclosed in parentheses > #31: FILE: drivers/staging/unisys/visorhba/visorhba_main.c:157: > +#define for_each_vdisk_match(iter, list, match) \ > + for (ite

RE: [PATCH] staging: unisys: Add s-Par visorhba

2015-07-15 Thread Kershner, David A
> -Original Message- > From: Dan Carpenter [mailto:dan.carpen...@oracle.com] > Sent: Wednesday, July 15, 2015 8:16 AM > To: Romer, Benjamin M > Cc: gre...@linuxfoundation.org; driverdev-devel@linuxdriverproject.org; > jes.soren...@redhat.com; *S-Par-Maintainer > Subject: Re: [PATCH] stagi

[PATCH] staging/lustre/ldlm: Unregister ldlm namespace from sysfs on free

2015-07-15 Thread green
From: Oleg Drokin ldlm_namespace_sysfs_unregister needs to be called ldlm_namespace_free_post so that we don't have this dangling object there after the namespace has disappeared. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/ldlm/ldlm_resource.c | 1 + 1 file changed, 1 inserti

Re: [PATCH] staging: unisys: Add s-Par visorhba

2015-07-15 Thread Dan Carpenter
Since you are redoing this anyway... On Mon, Jul 13, 2015 at 02:38:23PM -0400, Benjamin Romer wrote: > + for (vdisk = &devdata->head; vdisk->next; vdisk = vdisk->next) { > + if ((scsidev->channel == vdisk->channel) && > + (scsidev->id == vdisk->id) && > +

Re: [staging:staging-testing 134/169] drivers/staging/fbtft/fb_uc1611.c:105:24-28: duplicated argument to & or |

2015-07-15 Thread Henri Chain
Yes they are all intentional, it was a way to make the logic more obvious (argument on the right is bitmask). Henri On 07/15/2015 01:08 PM, Julia Lawall wrote: > TThese look intentional, but maybe double check the first one, with the > code shown in this message. > > julia > > On Wed, 15 Jul 20

Re: [staging:staging-testing 134/169] drivers/staging/fbtft/fb_uc1611.c:105:24-28: duplicated argument to & or |

2015-07-15 Thread Julia Lawall
TThese look intentional, but maybe double check the first one, with the code shown in this message. julia On Wed, 15 Jul 2015, kbuild test robot wrote: > TO: Henri Chain > CC: "Greg Kroah-Hartman" > CC: de...@driverdev.osuosl.org > > tree: git://git.kernel.org/pub/scm/linux/kernel/git/gregkh

Re: [PATCH v2] Staging: fbtft: Add support for the Ultrachip UC1611 LCD controller

2015-07-15 Thread Paul Bolle
On di, 2015-07-14 at 14:59 +0200, Henri Chain wrote: > --- /dev/null > +++ b/drivers/staging/fbtft/fb_uc1611.c > +#define DRVNAME "fb_uc1611" > +MODULE_ALIAS("spi:" DRVNAME); > +MODULE_ALIAS("platform:" DRVNAME); > +MODULE_ALIAS("spi:uc1611"); > +MODULE_ALIAS("platform:uc1611"); Man

[PATCH v2 2/2] staging: sm7xxfb: usr fb_read and fb_write

2015-07-15 Thread Sudip Mukherjee
Now since the Big-Endian and Little-Endian based calculations are moved into a macro we can make fb_read() and fb_write() common for both Little-Endian and Big-Endian. Signed-off-by: Sudip Mukherjee --- drivers/staging/sm7xxfb/sm7xxfb.c | 4 1 file changed, 4 deletions(-) diff --git a/driv

[PATCH v2 1/2] staging: sm7xxfb: define new macros

2015-07-15 Thread Sudip Mukherjee
Define and use some new macros to work with different situations based on little-endian and big-endian. Signed-off-by: Sudip Mukherjee --- v2: fixed empty macro drivers/staging/sm7xxfb/sm7xx.h | 19 drivers/staging/sm7xxfb/sm7xxfb.c | 48 -

[PATCH] staging: ft1000: ft1000-usb: remove unnecessary out of memory warning

2015-07-15 Thread CHANG FU CHIAO
fixes "Possible unnecessary 'out of memory' message" Signed-off-by: CHANG FU CHIAO --- drivers/staging/ft1000/ft1000-usb/ft1000_hw.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/ft1000/ft1000-usb/ft1000_hw.c b/drivers/staging/ft1000/ft1000-usb/ft1000_hw.c index e6b5976..96

Re: [PATCH v2 04/23] staging: rtl8192e: Remove unused enums

2015-07-15 Thread Mateusz Kulikowski
On July 15, 2015 8:34:02 AM GMT+02:00, Jakub Sitnicki wrote: >On Tue, Jul 14, 2015 at 10:04 PM CEST, Mateusz Kulikowski > wrote: >> Remove ack_policy enum and some unused RTL_DEBUG enums. >> >> Signed-off-by: Mateusz Kulikowski >> --- > >[snip] > >> diff --git a/drivers/staging/rtl8192e/rtllib