There are some fixes for typos we discovered in rtl8188eu staging driver
as a part of our Open Source Summit North America 2017 tutorial session.
Changes in v1:
- Merge patches into one
- Better commit log message
Valentine Sinitsyn (1):
staging: rtl8188eu: Fix spelling
drivers/staging/rt
rtl8188eu contains some spelling errors in comment lines as well as in
constants. Harmless as they are, they still make the code feel a bit
unclean, which is not something we want in the kernel.
Improve this by fixing typos so they won't catch eyes of future driver
developers anymore.
Signed-off-
On Thu, Sep 14, 2017 at 12:05:17PM -0700, Valentine Sinitsyn wrote:
> There are some fixes for typos we discovered in rtl8188eu staging driver
> as a part of our Open Source Summit North America 2017 tutorial session.
>
> Valentine Sinitsyn (3):
> staging: rtl8188eu: Fix a typo ("deflaut")
> s
Good work getting this set submitted.
On Thu, Sep 14, 2017 at 12:05:18PM -0700, Valentine Sinitsyn wrote:
> This improves spelling in the comment line to make things easier to get
> for future rtl8188eu developers.
>
> Fixes: ee5f8a431ea (staging: rtl8188eu: Move all efuse related code to
> rtw_e
On Thu, 14 Sep 2017 14:22:25 -0400 Sherry Yang wrote:
> Drop the global lru lock in isolate callback
> before calling zap_page_range which calls
> cond_resched, and re-acquire the global lru
> lock before returning. Also change return
> code to LRU_REMOVED_RETRY.
>
> Use mmput_async when fail to
This improves spelling in the comment line to make things easier to get
for future rtl8188eu developers.
Fixes: 6c984c81fe27 (staging: r8188eu: Add files for new driver - part
12)
Signed-off-by: Wolfgang Hartmann
Signed-off-by: Manish Shrestha
Signed-off-by: Valentine Sinitsyn
---
drivers/stag
This improves spelling in the comment line to make things easier to get
for future rtl8188eu developers.
Fixes: ee5f8a431ea (staging: rtl8188eu: Move all efuse related code to
rtw_efuse.c)
Signed-off-by: Wolfgang Hartmann
Signed-off-by: Manish Shrestha
Signed-off-by: Valentine Sinitsyn
---
dri
This improves spelling in the comment line to make things easier to get
for future rtl8188eu developers.
Fixes: 7b464c9fa5cc (staging: r8188eu: Add files for new driver - part
4)
Signed-off-by: Wolfgang Hartmann
Signed-off-by: Manish Shrestha
Signed-off-by: Valentine Sinitsyn
---
drivers/stagi
There are some fixes for typos we discovered in rtl8188eu staging driver
as a part of our Open Source Summit North America 2017 tutorial session.
Valentine Sinitsyn (3):
staging: rtl8188eu: Fix a typo ("deflaut")
staging: rtl8188eu: Fix a typo ("faliure")
taging: rtl8188eu: Fix a typo ("cacl
Drop the global lru lock in isolate callback
before calling zap_page_range which calls
cond_resched, and re-acquire the global lru
lock before returning. Also change return
code to LRU_REMOVED_RETRY.
Use mmput_async when fail to acquire mmap sem
in an atomic context.
Fix "BUG: sleeping function c
On Thu, 14 Sep 2017 10:02:03 -0700 (PDT)
David Miller wrote:
> From: Stephen Hemminger
> Date: Thu, 14 Sep 2017 09:31:07 -0700
>
> > The default receive buffer size was reduced by recent change
> > to a value which was appropriate for 10G and Windows Server 2016.
> > But the value is too small
SPI host drivers can use DMA to transfer data, so the buffer should be properly
allocated.
Keeping it on the stack could cause an undefined behavior.
The dedicated reset function solves this issue.
Signed-off-by: Stefan Popa
---
drivers/staging/iio/adc/ad7192.c | 4 +---
1 file changed, 1 inse
From: Stephen Hemminger
Date: Thu, 14 Sep 2017 09:31:07 -0700
> The default receive buffer size was reduced by recent change
> to a value which was appropriate for 10G and Windows Server 2016.
> But the value is too small for full performance with 40G on Azure.
> Increase the default back to maxi
The default receive buffer size was reduced by recent change
to a value which was appropriate for 10G and Windows Server 2016.
But the value is too small for full performance with 40G on Azure.
Increase the default back to maximum supported by host.
Fixes: 8b5327975ae1 ("netvsc: allow controlling
On Thu, 14 Sep 2017, Harsha Sharma wrote:
> Remove unused variable ret as it is not used anywhere.
> Remove multiple blank lines.
> Done using following coccinelle semantic patch
>
> @@
> type T;
> identifier i;
> constant C;
> @@
>
> (
> extern T i;
> |
> - T i;
> <+... when != i
> - i = C;
>
Remove unused variable ret as it is not used anywhere.
Remove multiple blank lines.
Done using following coccinelle semantic patch
@@
type T;
identifier i;
constant C;
@@
(
extern T i;
|
- T i;
<+... when != i
- i = C;
...+>
)
Signed-off-by: Harsha Sharma
---
drivers/staging/rtl8723bs/os_d
On 14.09.2017 15:50, Stefan Popa wrote:
SPI host drivers can use DMA to transfer data, so the buffer should be properly
allocated.
Keeping it on the stack could cause an undefined behavior.
The dedicated reset function solves this issue.
Signed-off-by: Stefan Popa
Acked-by: Michael Henneric
On Wed, Sep 13, 2017 at 08:58:26AM -0700, Tobin C. Harding wrote:
> On Wed, Sep 13, 2017 at 04:14:29PM +0100, Liam Ryan wrote:
> > On Wed, Sep 13, 2017 at 08:47:39AM +0200, Frans Klaver wrote:
> > > On Tue, Sep 12, 2017 at 2:40 AM, Liam Ryan wrote:
> > > > Fix checkpath-reported unbalanced braces
On 09/14/2017 03:50 PM, Stefan Popa wrote:
> SPI host drivers can use DMA to transfer data, so the buffer should be
> properly allocated.
> Keeping it on the stack could cause an undefined behavior.
>
> The dedicated reset function solves this issue.
>
> Signed-off-by: Stefan Popa
Acked-by: La
On Thu, 14 Sep 2017, Srishti Sharma wrote:
> Use kcalloc instead of kzalloc to check for overflow before
> multiplication. Done using the following semantic patch by
> coccinelle.
>
> http://coccinelle.lip6.fr/rules/kzalloc.cocci
>
> Signed-off-by: Srishti Sharma
Acked-by: Julia Lawall
> ---
Use kcalloc instead of kzalloc to check for overflow before
multiplication. Done using the following semantic patch by
coccinelle.
http://coccinelle.lip6.fr/rules/kzalloc.cocci
Signed-off-by: Srishti Sharma
---
Changes in v2:
- eliminate parentheses around the first argument
drivers/staging/c
On Thu, 14 Sep 2017, Srishti Sharma wrote:
> Use kcalloc instead of kzalloc to check for overflow before
> multiplication. Done using the following semantic patch by
> coccinelle.
>
> http://coccinelle.lip6.fr/rules/kzalloc.cocci
>
> Signed-off-by: Srishti Sharma
> ---
> drivers/staging/ccree/
On Thu, 14 Sep 2017, Srishti Sharma wrote:
> Use kcalloc instead of kzalloc to check for overflow before
> multiplication. Done using the following semantic patch by
> coccinelle.
>
> http://coccinelle.lip6.fr/rules/kzalloc.cocci
>
> Signed-off-by: Srishti Sharma
Acked-by: Julia Lawall
> ---
On Thu, 14 Sep 2017, Harsha Sharma wrote:
> Merge assignment with return statement to directly return the value.
> Done using following coccinelle semantic patch
>
> @@
> local idexpression ret;
> expression e;
> @@
>
> -ret =
> +return
> e;
> -return ret;
Acked-by: Julia Lawall
Many of
Use kcalloc instead of kzalloc to check for overflow before
multiplication. Done using the following semantic patch by
coccinelle.
http://coccinelle.lip6.fr/rules/kzalloc.cocci
Signed-off-by: Srishti Sharma
---
drivers/staging/ccree/ssi_sysfs.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion
Use kcalloc instead of kzalloc to check for overflow before
multiplication. Done using the following semantic patch by
coccinelle.
http://coccinelle.lip6.fr/rules/kzalloc.cocci
Signed-off-by: Srishti Sharma
---
drivers/staging/lustre/lustre/obdclass/cl_object.c | 2 +-
1 file changed, 1 inserti
Merge assignment with return statement to directly return the value.
Done using following coccinelle semantic patch
@@
local idexpression ret;
expression e;
@@
-ret =
+return
e;
-return ret;
Signed-off-by: Harsha Sharma
---
drivers/staging/rtl8723bs/os_dep/ioctl_linux.c | 20 -
Use kcalloc instead of kzalloc to check for an overflow before
multiplication. Done using the following semantic patch by
coccinelle.
http://coccinelle.lip6.fr/rules/kzalloc.cocci
Signed-off-by: Srishti Sharma
---
drivers/staging/media/atomisp/pci/atomisp2/css2400/sh_css_firmware.c | 4 +++-
1
28 matches
Mail list logo