This patch fixes the 80 char length warnings and an indent suspect warning
identified by the checkpath.pl script for the entire ms.c file leaving
with no more warnings or errors left to be fixed for it.
Signed-off-by: Ragavendra Nagraj
---
drivers/staging/rts5208/ms.c | 43
On Mon, Mar 23, 2015 at 10:04:40PM +0100, Greg Kroah-Hartman wrote:
> On Tue, Mar 17, 2015 at 08:31:24PM +0530, Sudip Mukherjee wrote:
>
> This doesn't apply anymore, due to other changes recently to this
> driver.
>
> But even if it did, I don't think it is correct. I really don't
> understand w
This patch fixes the C99-style "// ..." comments warnings identified by the
checkpath.pl script for the entire ddk750_chip.c file by using the
appropriate C89 "/* ... */" style comments accordingly.
Signed-off-by: Ragavendra Nagraj
---
drivers/staging/sm750fb/ddk750_chip.c | 12 ++--
1
add module parameters so that we can configure X and Y resolutions
and bpp when using this driver as a module.
Signed-off-by: Sudip Mukherjee
---
drivers/staging/sm7xxfb/sm7xxfb.c | 18 ++
1 file changed, 18 insertions(+)
diff --git a/drivers/staging/sm7xxfb/sm7xxfb.c
b/drivers
add MODULE_DEVICE_TABLE to support hot-plugging.
Signed-off-by: Sudip Mukherjee
---
drivers/staging/sm7xxfb/sm7xxfb.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/staging/sm7xxfb/sm7xxfb.c
b/drivers/staging/sm7xxfb/sm7xxfb.c
index 6e9b2aa..15509a6 100644
--- a/drivers/staging/s
the lcd type as defined in the Kconfig is not matching in the code.
as a result the rs, rw and en pins were getting interchanged.
Kconfig defines the value of PANEL_LCD to be 1 if we select custom
configuration but in the code LCD_TYPE_CUSTOM is defined as 5.
my hardware is LCD_TYPE_CUSTOM, but th
> -Original Message-
> From: Venkatesh Srinivas [mailto:venkate...@google.com]
> Sent: Monday, March 23, 2015 5:23 PM
> To: KY Srinivasan
> Cc: gre...@linuxfoundation.org; Linux Kernel Developers List;
> de...@linuxdriverproject.org; oher...@suse.com; James E.J. Bottomley;
> Christoph Hel
进冒险家大统助学代胜算、破布戏装虚灵、�P垃迈过小王子奄
有（ 谱 ・ 3wl 桶 ・djm��）
【 曾 ・7hl �� 17% 】
呼叫 137 TEL 1514 TEL 5103
Q Q: 534・022・008 李 惠 茹
保�^观瞻喜看潜到地摊惑乱,���b干吼;一日三餐眼看着�G酒精,带着子孙后代，公�A测验,��腺体艰涩，平开职称。过客大钞切开无力形同虚设水清无鱼圣芭蕉确是面相一家,平江除弊一往无前��，雯芟荣膺，替换一语道破,现金彩球司万贯。隐衷、大年酒阑人散赘肉死水一把把蝰晨有职有权出众无道瑞士、
On Tue, Mar 17, 2015 at 08:31:24PM +0530, Sudip Mukherjee wrote:
> fixed the handling of dev_t and the major number.
> now the major and minor number is passed to the init function.
> similarly in the cleanup function dev_t is passed to unregister it.
>
> Signed-off-by: Sudip Mukherjee
> ---
> d
On Mon, Mar 16, 2015 at 01:58:36PM -0400, Benjamin Romer wrote:
> Take out extra braces from if statements with only a single line of code
> inside.
>
> Signed-off-by: Benjamin Romer
> ---
> drivers/staging/unisys/uislib/uislib.c | 18 ++
> 1 file changed, 6 insertions(+), 12 del
On March 23, 2015 9:06:26 AM EDT, Dan Carpenter
wrote:
>On Mon, Mar 23, 2015 at 08:53:19AM -0400, Nicholas Krause wrote:
>>
>>
>> On March 23, 2015 6:40:06 AM EDT, Dan Carpenter
> wrote:
>> >On Sun, Mar 22, 2015 at 08:04:35PM -0400, Nicholas Krause wrote:
>> >> Fixes the checkpath.pl error wh
From: Jes Sorensen
In 7c3a8f2a5ec95e2b13704562f36321e20ddfc190 I made a mistake and moved
a break below a bracket, causing a situation where the for loop would
always exit when it shouldn't.
This patch corrects that mistake.
Signed-off-by: Jes Sorensen
---
drivers/staging/rtl8723au/core/rtw_m
> -Original Message-
> From: devel [mailto:driverdev-devel-boun...@linuxdriverproject.org] On
> Behalf Of K. Y. Srinivasan
> Sent: Monday, March 23, 2015 2:07 PM
> To: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org;
> de...@linuxdriverproject.org; oher...@suse.com; jbottom...@pa
> -Original Message-
> From: devel [mailto:driverdev-devel-boun...@linuxdriverproject.org] On
> Behalf Of K. Y. Srinivasan
> Sent: Monday, March 23, 2015 2:07 PM
> To: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org;
> de...@linuxdriverproject.org; oher...@suse.com; jbottom...@pa
Set the tablesize based on the information given by the host.
Signed-off-by: K. Y. Srinivasan
Reviewed-by: Long Li
---
v2: Rebase the code and enable clustering
drivers/scsi/storvsc_drv.c | 89 +++
1 files changed, 64 insertions(+), 25 deletion
The storage protocol informs the guest of the I/O capabilities of the storage
stack. Retrieve this information and use it in the guest.
Signed-off-by: K. Y. Srinivasan
Reviewed-by: Long Li
---
drivers/scsi/storvsc_drv.c |6 ++
1 files changed, 6 insertions(+), 0 deletions(-)
diff --git
Increase the default ring buffer size as this can significantly
improve performance especially on high latency storage back-ends.
Signed-off-by: K. Y. Srinivasan
Reviewed-by: Long Li
---
drivers/scsi/storvsc_drv.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers
The current code always sent packets without data on the primary channel.
Properly distribute sending of packets with no data amongst all available
channels. I would like to thank Long Li for noticing this problem.
Signed-off-by: K. Y. Srinivasan
Reviewed-by: Long Li
---
drivers/scsi/storvsc_dr
We may exit this function without properly freeing up the maapings
we may have acquired. Fix the bug.
Signed-off-by: K. Y. Srinivasan
Cc:
---
drivers/scsi/storvsc_drv.c | 15 ---
1 files changed, 8 insertions(+), 7 deletions(-)
diff --git a/drivers/scsi/storvsc_drv.c b/drivers/sc
The current code assumes that the scatterlists presented are not chained.
Fix the code to not make this assumption.
Signed-off-by: K. Y. Srinivasan
---
drivers/scsi/storvsc_drv.c | 98 +--
1 files changed, 57 insertions(+), 41 deletions(-)
diff --git a/
This patch-set addresses perf issues discovered on the Azure storage stack.
These patches also fix a couple of bugs.
K. Y. Srinivasan (7):
scsi: storvsc: Increase the ring buffer size
scsi: storvsc: Size the queue depth based on the ringbuffer size
scsi: storvsc: Always send on the selected
Size the queue depth based on the ringbuffer size. Also accomodate for the
fact that we could have multiple channels (ringbuffers) per adaptor.
Signed-off-by: K. Y. Srinivasan
Reviewed-by: Long Li
---
drivers/scsi/storvsc_drv.c | 27 ---
1 files changed, 16 insertions(
From: Jes Sorensen
Signed-off-by: Jes Sorensen
---
drivers/staging/rtl8723au/hal/rtl8723au_xmit.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/rtl8723au/hal/rtl8723au_xmit.c
b/drivers/staging/rtl8723au/hal/rtl8723au_xmit.c
index 1759487..6bc1297 100
From: Jes Sorensen
Signed-off-by: Jes Sorensen
---
drivers/staging/rtl8723au/hal/odm.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8723au/hal/odm.c
b/drivers/staging/rtl8723au/hal/odm.c
index 428b2b3..ec543cf 100644
--- a/drivers/staging/rtl8723au/hal
From: Jes Sorensen
Signed-off-by: Jes Sorensen
---
drivers/staging/rtl8723au/core/rtw_mlme_ext.c | 7 ++-
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/rtl8723au/core/rtw_mlme_ext.c
b/drivers/staging/rtl8723au/core/rtw_mlme_ext.c
index d01ca1d..c98920e 10064
From: Jes Sorensen
Hi Greg,
3 fixes for the rtl8723au staging driver - one fixes a whitespace mess
I created. One is an old one I forgot to post a while ago, and the
least one fixes an issue where a timeout value would get byteswapped
twice which is somewhat serious.
Cheers,
Jes
Jes Sorensen (
On Fri, Mar 20, 2015 at 03:22:11PM +, Lorenzo Stoakes wrote:
> This patch takes into account that cursor->vstart, crtc->vScreen and
> share->pvMem are pointers to memory-mapped I/O and thus we should use
> memset_io
> to make this explicit. In addition, some architectures require special
> tr
On Mon, Mar 23, 2015 at 01:25:37PM +, Lorenzo Stoakes wrote:
> On 23 March 2015 at 13:21, Lorenzo Stoakes wrote:
> > Sigh. Checking the emails I actually sent, I seem to *somehow* have
> > sent old files in this resend :S I really don't know how this
> > happened. My copies of the patches all
On 23 March 2015 at 13:21, Lorenzo Stoakes wrote:
> Sigh. Checking the emails I actually sent, I seem to *somehow* have
> sent old files in this resend :S I really don't know how this
> happened. My copies of the patches all apply perfectly correctly, but
> these are not the ones I sent.
And just
On 23 March 2015 at 13:16, Lorenzo Stoakes wrote:
> I've tried applying resend 2 patches to both linux-next and
> staging-testing in Greg's staging.git tree, they apply in both places.
Sigh. Checking the emails I actually sent, I seem to *somehow* have
sent old files in this resend :S I really do
On 23 March 2015 at 13:14, Sudip Mukherjee wrote:
> On Mon, Mar 23, 2015 at 01:02:52PM +, Lorenzo Stoakes wrote:
>> On 23 March 2015 at 12:55, Sudip Mukherjee
>> wrote:
>>
>> > and you need to send v3 now :(
>> > your series is not applying. Please refresh it against staging-testing
>>
>> Ap
On Mon, Mar 23, 2015 at 01:02:52PM +, Lorenzo Stoakes wrote:
> On 23 March 2015 at 12:55, Sudip Mukherjee wrote:
>
> > and you need to send v3 now :(
> > your series is not applying. Please refresh it against staging-testing
>
> Applies to staging-testing for me. Are you sure you're applying
On Mon, Mar 23, 2015 at 08:53:19AM -0400, Nicholas Krause wrote:
>
>
> On March 23, 2015 6:40:06 AM EDT, Dan Carpenter
> wrote:
> >On Sun, Mar 22, 2015 at 08:04:35PM -0400, Nicholas Krause wrote:
> >> Fixes the checkpath.pl error where the opening bracket has a unneeded
> >space between
> >> it
On 23 March 2015 at 12:55, Sudip Mukherjee wrote:
> and you need to send v3 now :(
> your series is not applying. Please refresh it against staging-testing
Applies to staging-testing for me. Are you sure you're applying the
correct 'RESEND 2' patches?
--
Lorenzo Stoakes
https:/ljs.io
_
On Sat, Mar 21, 2015 at 12:11:05AM +0530, Amitoj Kaur Chawla wrote:
> Fix the spacing problems with correct indentation and correct use of
> braces and spacing in switch-case statements.
same problem with this patch also, it is not applying.
please refresh it against staging-testing.
regards
sudip
On Mon, Mar 23, 2015 at 12:43:28PM +, Lorenzo Stoakes wrote:
> On 23 March 2015 at 11:51, Dan Carpenter wrote:
> > Oh hm... This is an actual RESEND patch.
> >
> > RESEND is a flag to show a mess up in the system. I like to take a look
> > at those to see what went wrong. The process failur
On March 23, 2015 6:40:06 AM EDT, Dan Carpenter
wrote:
>On Sun, Mar 22, 2015 at 08:04:35PM -0400, Nicholas Krause wrote:
>> Fixes the checkpath.pl error where the opening bracket has a unneeded
>space between
>> it and the function name for a if statement in the
>marco,VMX_WAIT_FOR_FIELD32.
>>
On Fri, Mar 20, 2015 at 07:54:56PM +0530, Amitoj Kaur Chawla wrote:
> The edits have been made to fix C99 Comment and properly indent
> the if-else statements with the braces.
i think you are doing multiple types of changes in one patch.
better to send it in a series of two, one for the C99 and the
On 23 March 2015 at 11:51, Dan Carpenter wrote:
> Oh hm... This is an actual RESEND patch.
>
> RESEND is a flag to show a mess up in the system. I like to take a look
> at those to see what went wrong. The process failure was that you
> messed up the threading earlier. Take a look at the --in-
On Fri, Mar 20, 2015 at 08:41:45PM +0100, Matteo Semenzato wrote:
> From: Matteo Semenzato
>
> The condition pxmitpriv->hwxmit_entry == 5 is always false because
> HWXMIT_ENTRY is always 4.
>
> Signed-off-by: Matteo Semenzato
Reviewed-by: Dan Carpenter
regards,
dan carpenter
___
Oh hm... This is an actual RESEND patch.
RESEND is a flag to show a mess up in the system. I like to take a look
at those to see what went wrong. The process failure was that you
messed up the threading earlier. Take a look at the --in-reply-to flag.
regards,
dan carpenter
__
On 23 March 2015 at 10:53, Dan Carpenter wrote:
> Sorry I wasn't clear on this before. Please, stop putting RESEND in the
> subject. That's only for if you think we are screwing up or ignoring
> you. Just put v2, and put what changed under the --- cut off.
> ---
> v2: changed the commit message
Sorry I wasn't clear on this before. Please, stop putting RESEND in the
subject. That's only for if you think we are screwing up or ignoring
you. Just put v2, and put what changed under the --- cut off.
---
v2: changed the commit message
https://www.google.com/#q=how+to+send+a+v2+patch
Otherwi
On Sun, Mar 22, 2015 at 08:04:35PM -0400, Nicholas Krause wrote:
> Fixes the checkpath.pl error where the opening bracket has a unneeded space
> between
> it and the function name for a if statement in the marco,VMX_WAIT_FOR_FIELD32.
>
> Signed-off-by: Nicholas Krause
> ---
> drivers/staging/oc
44 matches
Mail list logo
