Re: [PATCH 1/1] cris: fix %d confusingly prefixed with 0x in format string

2014-08-02 Thread Joe Perches
On Sat, 2014-08-02 at 18:19 -0700, Hans Wennborg wrote: > Yes, I have a patch for a bunch of these, but I figured it would be > easier to get it merged if I split it up. (Complete kernel newbie here.) You as a kernel newbie did good, thanks. A small tip would be to do all of them in a single ser

Re: [PATCH 1/1] cris: fix %d confusingly prefixed with 0x in format string

2014-08-02 Thread Steve French
Thanks for pointing this out. I fixed the instances of this that you spotted in the cifs.ko driver (pushed to cifs-2.6.git for-next) and noted that you pointed it out in the commit message. http://git.samba.org/?p=sfrench/cifs-2.6.git;a=commit;h=59b04c5df75bd715002bb535930ae1982a739269 On Sat,

[PATCH 2/7] staging: unisys: remove MEMORYBARRIER AND VolatileBarrier from commontypes

2014-08-02 Thread Benjamin Romer
Remove the MEMORYBARRIER and VolatileBarrier() defines from commontypes.h and convert the spots that used this to mb(). Add comments to each use to indicate that the barrier is used to ensure channel synchronization between guests. Signed-off-by: Benjamin Romer --- drivers/staging/unisys/channel

[PATCH 7/7] staging: unisys: remove commontypes.h

2014-08-02 Thread Benjamin Romer
Delete commontypes.h, and replace all of the places that #included it with correct #includes for the types used in that file. Signed-off-by: Benjamin Romer --- .../unisys/common-spar/include/channels/channel.h | 4 +-- .../include/channels/controlframework.h| 2 +- .../include/cha

[PATCH 1/7] staging: unisys: remove MEMCPY functions from commontypes.h

2014-08-02 Thread Benjamin Romer
This patch removes MEMCPY, MEMCPY_FROMIO, and MEMCPY_TOIO from commontypes.h, and switches all use of these macros to the appropriate built in definition. Signed-off-by: Benjamin Romer --- drivers/staging/unisys/channels/channel.c | 6 +++--- drivers/staging/unisys/common-s

[PATCH 3/7] staging: unisys: remove INLINE define from commontypes

2014-08-02 Thread Benjamin Romer
There was a macro for INLINE. Delete it and fix the one occurrance of it to just use 'inline' instead, though that function looks awfully big for an inline... Signed-off-by: Benjamin Romer --- drivers/staging/unisys/common-spar/include/channels/iochannel.h | 2 +- drivers/staging/unisys/include/

[PATCH 6/7] staging: unisys: move remaining macros from commontypes.h to channel.h

2014-08-02 Thread Benjamin Romer
All of the remaining macros in commontypes.h - the CHANNEL_*_MISMATCH macros and UltraLogEvent() - are used only in channel.h. Move the entire set of them to their new home. Signed-off-by: Benjamin Romer --- .../unisys/common-spar/include/channels/channel.h | 31 ++ drivers/

[PATCH 5/7] staging: unisys: remove OFFSETOF from commontypes

2014-08-02 Thread Benjamin Romer
The only place OFFSETOF was being used was in iochannel.h. Remove the macro from commontypes.h and replace all uses of it in iochannel.h with offsetof() instead. Signed-off-by: Benjamin Romer --- .../common-spar/include/channels/iochannel.h | 36 +++--- drivers/staging/unis

[PATCH 0/7] staging: unisys: cleanup and removal of commontypes.h

2014-08-02 Thread Benjamin Romer
This patch series completes the cleanup of redundantly-defined types and macros from commontypes.h, and then removes the file entirely. Benjamin Romer (7): staging: unisys: remove MEMCPY functions from commontypes.h staging: unisys: remove MEMORYBARRIER AND VolatileBarrier from commontypes

[PATCH 4/7] staging: unisys: remove GUEST_PHYSICAL_ADDRESS from commontypes.h

2014-08-02 Thread Benjamin Romer
The typedef for GUEST_PHYSICAL address is only used in two files, so remove it from commontypes.h and add it to visorchipset/file.c and controlvmchannel.h. Signed-off-by: Benjamin Romer --- drivers/staging/unisys/common-spar/include/channels/controlvmchannel.h | 3 +++ drivers/staging/unisys/inc

Re: [PATCH 1/1] cris: fix %d confusingly prefixed with 0x in format string

2014-08-02 Thread Hans Wennborg
On 08/02/2014 05:46 PM, Joe Perches wrote: On Sat, 2014-08-02 at 15:20 -0700, Hans Wennborg wrote: Signed-off-by: Hans Wennborg --- arch/cris/arch-v32/drivers/cryptocop.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/cris/arch-v32/drivers/cryptocop.c b/arch/cris

Re: [PATCH 1/1] cris: fix %d confusingly prefixed with 0x in format string

2014-08-02 Thread Joe Perches
On Sat, 2014-08-02 at 15:20 -0700, Hans Wennborg wrote: > Signed-off-by: Hans Wennborg > --- > arch/cris/arch-v32/drivers/cryptocop.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/cris/arch-v32/drivers/cryptocop.c > b/arch/cris/arch-v32/drivers/cryptocop.c [] > @

[PATCH 1/1 V2] Drivers: net-next: hyperv: Increase the size of the sendbuf region

2014-08-02 Thread K. Y. Srinivasan
Intel did some benchmarking on our network throughput when Linux on Hyper-V is as used as a gateway. This fix gave us almost a 1 Gbps additional throughput on about 5Gbps base throughput we hadi, prior to increasing the sendbuf size. The sendbuf mechanism is a copy based transport that we have whic

RE: [PATCH 1/1] Drivers: net-next: hyperv: Increase the size of the sendbuf region

2014-08-02 Thread KY Srinivasan
> -Original Message- > From: David Miller [mailto:da...@davemloft.net] > Sent: Friday, August 1, 2014 11:14 PM > To: KY Srinivasan > Cc: net...@vger.kernel.org; linux-ker...@vger.kernel.org; > de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; > jasow...@redhat.com > Subjec

[PATCH] staging: dgnc: remove some unused code

2014-08-02 Thread Seunghun Lee
Remove some unused code. CC: Lidza Louina CC: Mark Hounschell Signed-off-by: Seunghun Lee --- drivers/staging/dgnc/dgnc_cls.c| 72 +- drivers/staging/dgnc/dgnc_driver.c | 39 +- drivers/staging/dgnc/dgnc_driver.h | 70 +- drivers/staging/dgnc/dgnc_mgmt.c | 23

Re: [PATCH] staging: rtl8723au: Fix static symbol sparse warning

2014-08-02 Thread Jes Sorensen
Larry Finger writes: > On 08/01/2014 08:06 AM, Miguel Oliveira wrote: >> Fix sparse warning: >> drivers/staging/rtl8723au/core/rtw_efuse.c:579:5: warning: symbol >> efuse_GetCurrentSize23a' was not declared. Should it be static? >> >> Signed-off-by: Miguel Oliveira >> --- >> drivers/staging/rtl