In the patch below I see lines being removed and replaced by the same
line. That does not hurt, but you may want to take a look at the tool
chain you use to create patches.
Acked-by: Roland Vossen
On 05/07/2011 04:34 PM, Guilherme Salazar wrote:
---
drivers/staging/brcm80211/brcmfmac/wl_cf
On 05/07/2011 04:34 PM, Guilherme Salazar wrote:
---
drivers/staging/brcm80211/brcmfmac/dhd_sdio.c |3 +--
1 files changed, 1 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/brcm80211/brcmfmac/dhd_sdio.c
b/drivers/staging/brcm80211/brcmfmac/dhd_sdio.c
index 464f52a..16bb618 10
Thanks for the cleanup Guilherme.
On 05/07/2011 04:34 PM, Guilherme Salazar wrote:
---
drivers/staging/brcm80211/brcmfmac/dhd_cdc.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/brcm80211/brcmfmac/dhd_cdc.c
b/drivers/staging/brcm80211/brcmfmac/dh
On Mon, May 09, 2011 at 02:56:52PM +, KY Srinivasan wrote:
> I will address this. Greg had a concern about module reference counting
> and looking at the current code, it did not appear to be an issue. The
> change you are suggesting will not affect the vmbus core which is what I want
> to focu
Dear sirs,
I upgraded my ubuntu from 10.10 into 11.04 version.
And after that I have issue with wireless connection:
10 minutes network connection is working and 10 minutes
network connection is down. And again 10 min - up, 10 min down.
I am using r8712u driver.
Could you, please, help me and/or
From: Kalle Valo
Signed-off-by: Kalle Valo
---
drivers/staging/ath6kl/os/linux/ar6000_drv.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/drivers/staging/ath6kl/os/linux/ar6000_drv.c
b/drivers/staging/ath6kl/os/linux/ar6000_drv.c
index 513f205..48dd9e3 100644
--- a/d
From: Naveen Singh
implementing the cfg ops that gets called when iw dev wlan0
link is issued by user. The ops that needs to be implemented
is get_station.
kvalo: check the mac address and fix style issues
Signed-off-by: Naveen Singh
Signed-off-by: Kalle Valo
---
drivers/staging/ath6kl/os/li
Sorry for the split message, I mistyped and accidentally sent my
mostly incomplete message.
On Tue, May 10, 2011 at 14:18, Julian Calaby wrote:
> obj-$(CONFIG_BRCMWLUTIL)
I meant:
obj-$(CONFIG_BRCMWLUTIL) += util/
> The LIB80211 symbol used by the Intel 2x00 (old non-mac80211) wireless
> drive
Arend, Henry,
One problem I noticed with your patch:
On Tue, May 10, 2011 at 00:33, Arend van Spriel wrote:
> From: Henry Ptasinski
>
> Move utility functions shared by the brcmsmac and brcmfmac drivers into a new
> module, brcmutil.ko. This eliminates the need to compile and link the shared
>
You are receiving this email because we wish you to use our digital photo
editing services.
Digital Photo Editing Services :
We are a China based Imaging Professionals. We offer basic and advanced digital
photo Editing services and solutions like photo Cutout, morphological photo
Editing, phot
On Mon, May 09, 2011 at 11:53:35PM +, KY Srinivasan wrote:
> > Note, I wouldn't recommend sending that many patches to lkml, it's not
> > needed, just stick to the devel@linuxdriverproject.org list as I really
> > doubt anyone else cares about this code at the moment.
>
> Agreed; I will do tha
This patch fixes brace coding issues. There are
also lots of line over 80 characters issues, but
this patch doesn't fixes it up.
Signed-off-by: Guilherme Salazar
---
drivers/staging/brcm80211/brcmsmac/wlc_bmac.c | 51 ++--
1 files changed, 21 insertions(+), 30 deletions(-)
This patch fixes up some brace coding and spaces at the
start of a line issues. In this source file, there are
lots of warnings related to line over 80 characters, but
this patch doesn't fixes it up.
Signed-off-by: Guilherme Salazar
---
drivers/staging/brcm80211/brcmsmac/wlc_ampdu.c | 33 +
> -Original Message-
> From: Greg KH [mailto:g...@kroah.com]
> Sent: Monday, May 09, 2011 6:53 PM
> To: KY Srinivasan
> Cc: gre...@suse.de; linux-ker...@vger.kernel.org;
> devel@linuxdriverproject.org; virtualizat...@lists.osdl.org; Haiyang Zhang;
> Abhishek Kane (Mindtree Consulting PVT
On Mon, May 09, 2011 at 02:55:59PM -0700, K. Y. Srinivasan wrote:
> cleanup() is an empty function; get rid of it.
>
> Signed-off-by: K. Y. Srinivasan
> Signed-off-by: Haiyang Zhang
> Signed-off-by: Abhishek Kane
> Signed-off-by: Hank Janssen
> ---
> drivers/staging/hv/netvsc_drv.c |4 ---
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/channel.c | 71 --
1 files changed, 0 insertions(+), 71 deletions(-)
diff --git
Access to this variable is already serialized via a spin lock. Get
rid of the unnecessary volatile qualifier.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/ring_buffer.c |2 +-
1 files changed,
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/storvsc_drv.c |8
1 files changed, 0 insertions(+), 8 deletions(-)
diff --git a/drivers/staging/hv/storvsc_d
Now, get rid of the empty function.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/blkvsc_drv.c | 15 ---
1 files changed, 0 insertions(+), 15 deletions(-)
diff --git a/drivers/staging
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/hv.c | 18 +-
1 files changed, 9 insertions(+), 9 deletions(-)
diff --git a/drivers/staging/hv/hv.c b/d
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/ring_buffer.c | 88 +++---
1 files changed, 44 insertions(+), 44 deletions(-)
diff --gi
This is unused state; get rid of it.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/storvsc_api.h |3 ---
1 files changed, 0 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/hv/storvsc
Use the completion abstraction in struct rndis_request to synchronize.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/rndis_filter.c | 33 ++---
1 files changed, 14 inse
In preperation to eliminating blk_vsc_initialize(), move the assignment
of driver name.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/blkvsc_drv.c |2 +-
1 files changed, 1 insertions(+), 1 dele
Get rid some commented code and dated comments.T
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/connection.c | 10 --
1 files changed, 4 insertions(+), 6 deletions(-)
diff --git a/drivers/
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/channel_mgmt.c |6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/hv/channel_mg
Now that we had gotten rid of the indirection; get rid of some
unnecessary code.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/netvsc.c |3 ---
1 files changed, 0 insertions(+), 3 deletions(-)
Get rid of unnecessary code/comments.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/blkvsc_drv.c | 15 ---
1 files changed, 0 insertions(+), 15 deletions(-)
diff --git a/drivers/stagi
Now that we have gotten rid of the indirection for invoking dev_rm,
get rid of this.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/vmbus_api.h |1 -
1 files changed, 0 insertions(+), 1 deletions
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/connection.c | 20 ++--
1 files changed, 10 insertions(+), 10 deletions(-)
diff --git a/drivers/staging
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/channel.c |4 ++--
drivers/staging/hv/ring_buffer.c |4 ++--
drivers/staging/hv/ring_buffer.h |2 +-
3 fil
In preperation to eliminating blk_vsc_initialize(), move the copying of
dev_type field.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/blkvsc_drv.c |3 +--
1 files changed, 1 insertions(+), 2 del
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/channel_mgmt.c |5 +++--
1 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/hv/channel_mgm
Use completion abstraction in struct netvsc_device instead of
struct wait_queue_head_t to synchronize.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/netvsc.c | 32 +---
Get rid unnecessary DPRINT.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/blkvsc_drv.c |4
1 files changed, 0 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/hv/blkvsc_drv.c b/d
cleanup() is an empty function; get rid of it.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/netvsc_drv.c |4
1 files changed, 0 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/
Compute max_outstanding_req_per_channel where it is needed.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/storvsc_drv.c | 30 +++---
1 files changed, 15 insertions(+), 15 d
Get rid of some commented code.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/hv.c | 31 ---
1 files changed, 0 insertions(+), 31 deletions(-)
diff --git a/drivers/sta
Cleanup comments in ring_buffer.c
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/ring_buffer.c | 323 +++---
1 files changed, 129 insertions(+), 194 deletions(-)
dif
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/channel_mgmt.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/staging/hv/channel_mgmt.c
Assign the driver name directly.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/blkvsc_drv.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/staging/hv/blkvsc_drv.c
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/netvsc_drv.c |6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/hv/netvsc_drv.c
Now, get rid of the redundant function and make
blkvsc_drv_init the module init function.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/blkvsc_drv.c | 11 +--
1 files changed, 1 insertion
mousevsc_on_cleanup() is an empty function; get rid of it.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/hv_mouse.c |4
1 files changed, 0 insertions(+), 4 deletions(-)
diff --git a/drive
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/hv_mouse.c |8
1 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/hv/hv_mouse.c b
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/channel.c |4 ++--
drivers/staging/hv/ring_buffer.c |4 ++--
drivers/staging/hv/ring_buffer.h |2 +-
3 fil
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/netvsc_api.h |3 +++
drivers/staging/hv/rndis_filter.c |5 +
2 files changed, 4 insertions(+), 4 deletions(-
Get rid of the forward declaration by moving the code around.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/storvsc_drv.c | 338 +++---
1 files changed, 169 insertio
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/rndis_filter.c |3 +--
1 files changed, 1 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/hv/rndis_filter.
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/hv_mouse.c |5 +
1 files changed, 1 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/hv/hv_mouse.c b/dr
Get rid of the forward declaration by moving the code around.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/storvsc_drv.c | 49 ++---
1 files changed, 24 insertions
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/storvsc_drv.c |6 ++
1 files changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/hv/storvsc_drv
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/storvsc_drv.c |6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/hv/storvsc_drv
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/storvsc.c | 12 +---
1 files changed, 1 insertions(+), 11 deletions(-)
diff --git a/drivers/staging/hv/storvsc.
Get rid of the forward declaration by moving the code around.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/ring_buffer.c | 66 +++--
1 files changed, 27 insertions
This state is not used anywhere; get rid of it.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/vmbus_private.h |4
1 files changed, 0 insertions(+), 4 deletions(-)
diff --git a/drivers/stag
Rename the function ringbuffer_init to not pollute the
namespace.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/channel.c |7 +--
drivers/staging/hv/ring_buffer.c |4 ++--
drivers/st
Now that we have gotten rid of the indirection, get rid of the
unnecessary state struct hv_driver.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/vmbus_api.h |1 -
1 files changed, 0 insertions(+
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/storvsc_drv.c | 15 +++
1 files changed, 7 insertions(+), 8 deletions(-)
diff --git a/drivers/staging/hv/st
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/channel.c |6 +++---
drivers/staging/hv/ring_buffer.c |4 ++--
drivers/staging/hv/ring_buffer.h |2 +-
3 f
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/netvsc_drv.c | 11 ++-
1 files changed, 2 insertions(+), 9 deletions(-)
diff --git a/drivers/staging/hv/netvsc_
Get rid of the forward declaration by moving the code around.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/storvsc_drv.c | 55 ++---
1 files changed, 27 insertions
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/storvsc_drv.c | 99 +++---
1 files changed, 50 insertions(+), 49 deletions(-)
diff --gi
rndis_filter_cleanup() is an empty function; get rid of it.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/rndis_filter.c |6 --
1 files changed, 0 insertions(+), 6 deletions(-)
diff --git a
Use completion abstraction to synchronize in vmbus_channel_msginfo.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/channel.c | 31 ---
drivers/staging/hv/channel_mg
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/storvsc_drv.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/staging/hv/storvsc_drv.c b
Get rid of the forward declaration by moving the code around.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/storvsc_drv.c | 24 ++--
1 files changed, 10 insertions(+), 14 delet
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/channel.c |4 ++--
drivers/staging/hv/ring_buffer.c |4 ++--
drivers/staging/hv/ring_buffer.h |2 +-
3 fil
Get rid of the forward declaration by moving code around.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/ring_buffer.c | 83 +-
1 files changed, 37 insertions(+
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/storvsc_drv.c |7 +--
1 files changed, 1 insertions(+), 6 deletions(-)
diff --git a/drivers/staging/hv/storvsc_dr
Get rid of the forward declaration by moving the code around.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/storvsc_drv.c | 143 +++---
1 files changed, 71 insertion
Get rid of unnecessary comments/dead code.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/storvsc_drv.c | 17 -
1 files changed, 0 insertions(+), 17 deletions(-)
diff --git a/drive
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/blkvsc_drv.c |2 +-
drivers/staging/hv/storvsc.c |2 +-
drivers/staging/hv/storvsc_api.h |2 +-
3 files c
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/netvsc.c |2 +-
drivers/staging/hv/netvsc_api.h |1 +
2 files changed, 2 insertions(+), 1 deletions(-)
diff -
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/blkvsc_drv.c |7 +--
1 files changed, 1 insertions(+), 6 deletions(-)
diff --git a/drivers/staging/hv/blkvsc_drv.
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/storvsc_drv.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/hv/storvsc_drv.c
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/storvsc.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/hv/storvsc.c b/drive
Get rid of the indirection in invoking storvsc_dev_remove()
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/storvsc_drv.c |6 ++
1 files changed, 2 insertions(+), 4 deletions(-)
diff --git a/
Statically initialize probe/remove.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/hv_mouse.c |8
1 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/hv/hv_mou
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/netvsc_api.h |1 +
drivers/staging/hv/rndis_filter.c |4 +---
2 files changed, 2 insertions(+), 3 deletions(-)
This spin lock is potentially acquired from interrupt context.
Ensure that the interrupts are blocked whenever the lock is held.
The current code was not consistent with regards to blocking
interrupts - the same lock would be acquired without blocking
interrupts in some instance while the interrupt
netvsc_cleanup() is an empty function; get rid of it.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/netvsc.c |8
1 files changed, 0 insertions(+), 8 deletions(-)
diff --git a/drivers/s
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/netvsc_drv.c |5 +
1 files changed, 1 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/hv/netvsc_drv.c
Get rid of the forward declaration by moving the code around.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/storvsc_drv.c | 32 +++-
1 files changed, 15 insertions(+),
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/netvsc_drv.c |8
1 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/hv/netvsc_drv
The storvsc_cleanup() is an empty function; get rid of it.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/blkvsc_drv.c |3 ---
drivers/staging/hv/storvsc.c |7 ---
drivers/staging/hv
Now that we have gotten rid of the indirection, cleanup
mouse_vsc_initialize().
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/hv_mouse.c | 10 +-
1 files changed, 1 insertions(+), 9 delet
Rename the variable g_blkvsc_drv.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/blkvsc_drv.c | 10 +-
1 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/hv/blkv
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/netvsc.c |2 +-
drivers/staging/hv/netvsc_api.h |1 +
2 files changed, 2 insertions(+), 1 deletions(-)
diff -
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/storvsc_drv.c |7 ---
1 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/hv/storvsc_dr
Get rid of the forward declaration by moving the code around.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/storvsc_drv.c | 132 +++---
1 files changed, 66 insertion
Now that we have removed calls to this empty function, get rid of the
unnecessary state in struct hv_driver.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/vmbus_api.h |2 --
1 files changed, 0
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/channel_mgmt.c |6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/hv/channel_mg
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/storvsc_drv.c |6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/hv/storvsc_drv
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/blkvsc_drv.c |4 +---
1 files changed, 1 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/hv/blkvsc_drv.c b
We are making up the drive parameters; simplify the code.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/storvsc_drv.c | 84 +++---
1 files changed, 15 insertions(+)
The function storvsc_dev_remove() already deals with draining of
the outstanding I/O. Cleanup blkvsc_remove() keeping this in mind.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/blkvsc_drv.c | 18
Get rid of the forward declaration by moving the code around.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/storvsc_drv.c | 61 ++
1 files changed, 29 insertion
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/rndis_filter.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/staging/hv/rndis_filter.c
The subject line says it all.
Signed-off-by: K. Y. Srinivasan
Signed-off-by: Haiyang Zhang
Signed-off-by: Abhishek Kane
Signed-off-by: Hank Janssen
---
drivers/staging/hv/storvsc.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/hv/storvsc.c b/drive
1 - 100 of 184 matches
Mail list logo