ERROR: Macros with complex values should be enclosed in parentheses.
Signed-off-by: Vikas Kumar Sharma
---
drivers/staging/ks7010/ks_hostif.h | 24
1 file changed, 12 insertions(+), 12 deletions(-)
diff --git a/drivers/staging/ks7010/ks_hostif.h
b/drivers/staging/ks701
On Tue, Mar 02, 2021 at 06:12:54PM +0530, Vikas Kumar Sharma wrote:
> Disclaimer:This message is intended only for the designated recipient(s).
> It may contain confidential or proprietary information and may be subject
> to other confidentiality protections. If you are not a designated
> recipi
---
Staging: android: fix parenthesis coding style issue in alarm-dev.c
This is a patch to the alarm-dev.c file that fixes up a
parenthesis warning found by the checkpatch.pl tool.
Signed-off-by: Evan Hosseini
---
drivers/staging/android/alarm-dev.c | 4 ++--
1 file changed, 2 insertions(+), 2
>From 043428d63637a6dd8e52449b73dbb8341885d7e4 Mon Sep 17 00:00:00 2001
From: Armin Schoenlieb
Date: Mon, 10 Jul 2017 01:12:52 +0200
Subject: [PATCH] Staging: media: atomisp2: fixed trailing whitespace error in
atomisp_v4l2.c This is a patch to the atomisp_v4l2.c file that fixes up a
trail
>From bb1aac6ae6b21b903d8743712e21aeb1a6b22163 Mon Sep 17 00:00:00 2001
From: Armin Schoenlieb
Date: Mon, 10 Jul 2017 01:52:41 +0200
Subject: [PATCH] staging: unisys: visorbus: fix brace coding style issue in
visorbus_main.c This is a patch to the visorbus_main.c file that fixes up six
br
Fixed check patch warnings about multiple line dereferencing
Signed-off-by: Devendra Sharma
---
drivers/staging/comedi/drivers/cb_pcidas64.c | 20
1 file changed, 8 insertions(+), 12 deletions(-)
diff --git a/drivers/staging/comedi/drivers/cb_pcidas64.c
b/drivers/staging
>From a1635ea5e75cb2f10728ae4ddf3a21567958e98f Mon Sep 17 00:00:00 2001
From: Sanidhya Solanki
Date: Sat, 12 Dec 2015 02:20:03 -0500
Subject: [PATCH] [PATCH 1/3] staging: dgnc: Patch includes the checkpatch
fixes
Patch contains the spacing fixes that checkpatch prompted for,
as asked by
On Sat, Dec 12, 2015 at 02:58:50AM -0500, Sanidhya Solanki wrote:
> >From a1635ea5e75cb2f10728ae4ddf3a21567958e98f Mon Sep 17 00:00:00 2001
> From: Sanidhya Solanki
> Date: Sat, 12 Dec 2015 02:20:03 -0500
> Subject: [PATCH] [PATCH 1/3] staging: dgnc: Patch includes the chec
Patch contains the spacing fixes that checkpatch prompted for,
as asked by the TODO.
Signed-off-by: Sanidhya Solanki
---
drivers/staging/dgnc/dgnc_cls.c| 4 +-
drivers/staging/dgnc/dgnc_driver.h | 8 --
drivers/staging/dgnc/dgnc_neo.c| 235
Patch contains the CamelCase fixes & Macro fixes that
checkpatch prompted for, as asked by the TODO.
Signed-off-by: Sanidhya Solanki
---
drivers/staging/dgnc/dgnc_driver.c | 56 -
drivers/staging/dgnc/dgnc_driver.h | 25 ++--
drivers/staging/dgnc/dgnc_mgmt.c | 28 ++---
dri
Patch updates the TODO file.
Signed-off-by: Sanidhya Solanki
---
drivers/staging/dgnc/TODO | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/dgnc/TODO b/drivers/staging/dgnc/TODO
index 0e0825b..0bdfd26 100644
--- a/drivers/staging/dgnc/TODO
+++ b/drivers/staging/dgnc/TODO
On Mon, Dec 14, 2015 at 07:03:31PM -0500, Sanidhya Solanki wrote:
> Patch updates the TODO file.
>
> Signed-off-by: Sanidhya Solanki
> ---
> drivers/staging/dgnc/TODO | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/staging/dgnc/TODO b/drivers/staging/d
Can you tell me what was incorrect about them?
I thought you requested the dgnc patches to be resent without the
headers.
There are 3 of them because Dan Carpenter asked my patch to be broken
up so each one does only one thing. I also resent you the skein
subsystem patches, as I did not hear
On Mon, Dec 14, 2015 at 07:45:25PM -0500, Sanidhya Solanki wrote:
> Can you tell me what was incorrect about them?
The subjects are all wrong, go look at them... PATCH twice? 1/3 for
every message? Same string for every message?
> I thought you requested the dgnc patches to be resent w
Replace formatting as suggested by checkpatch.
Signed-off-by: Michiel Schuurmans
---
drivers/staging/rtl8192e/rtllib_crypt_ccmp.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/rtl8192e/rtllib_crypt_ccmp.c
b/drivers/staging/rtl8192e/rtllib_crypt_ccmp.c
---
drivers/staging/iio/light/tsl2x7x_core.c | 20 ++--
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git a/drivers/staging/iio/light/tsl2x7x_core.c
b/drivers/staging/iio/light/tsl2x7x_core.c
index af3910b..c63fe6a 100644
--- a/drivers/staging/iio/light/tsl2x7x_core.c
From: Vaibhav Kothari
- Removed white-space before comma in memset()
- Added blank line between declaration and defination
at various places
Signed-off-by: Vaibhav Kothari
---
drivers/staging/media/atomisp/i2c/gc2235.c | 16 +++-
1 file changed, 11 insertions(+), 5 deletions(-)
On Sun, Dec 18, 2016 at 03:43:15PM +0530, devendra sharma wrote:
> Fixed check patch warnings about multiple line dereferencing
What does that mean?
>
> Signed-off-by: Devendra Sharma
> ---
> drivers/staging/comedi/drivers/cb_pcidas64.c | 20
>
On Sun, 2016-12-18 at 15:43 +0530, devendra sharma wrote:
> Fixed check patch warnings about multiple line dereferencing
[].
> diff --git a/drivers/staging/comedi/drivers/cb_pcidas64.c
> b/drivers/staging/comedi/drivers/cb_pcidas64.c
[]
> @@ -2476,18 +2475,15 @@ static int setup_c
WARNING: line has more than 80 characters
Signed-off-by: Linus Liu
---
drivers/staging/octeon-usb/octeon-hcd.c |9 +++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/octeon-usb/octeon-hcd.c
b/drivers/staging/octeon-usb/octeon-hcd.c
index 9a7858a..5390655
As reported by checkpatch.
Signed-off-by: Fernando Apesteguia
---
drivers/staging/dgnc/dgnc_cls.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/dgnc/dgnc_cls.c b/drivers/staging/dgnc/dgnc_cls.c
index 4e1e0dc6..aedca66 100644
--- a/drivers/staging/dgnc/dgnc_c
This series of patch includes new design for 64 bits. The driver uses the
redundant typecasting to communicate with the chipset, which causes several
compile warnings.
However, this patch uses the real data type and removes unnecessary typecasting.
Also, the driver allocates the ID value to the
On Sun, Nov 30, 2014 at 08:54:08PM +, Parth Sane wrote:
> Hi,
> Please find the attached patch file. I hope it's OK.
> -Parth
>
> From 3a429077981db3a7cf96d1e503c0bbbc932cab32 Mon Sep 17 00:00:00 2001
> From: Parth Sane
> Date: Mon, 1 Dec 2014 01:55:
s are required or not? This is nearly impossible for me to
guess this, since I haven't written the driver.
Please help me out with this.
-Parth
On 1 December 2014 at 08:19, Greg Kroah-Hartman
wrote:
> On Sun, Nov 30, 2014 at 08:54:08PM +, Parth Sane wrote:
>> Hi,
>> Please fin
On Mon, Dec 01, 2014 at 11:05:10AM +0530, Parth Sane wrote:
> Hi Greg,
> I have looked at the patches and documentation on how to proceed
> forward. But I have a doubt. Was hoping you could make it clear for
> me. You have said that you don't want unnecessary binary firmware
> files in the source w
On Tue, Dec 24, 2013 at 09:45:03AM -0600, Evan Hosseini wrote:
> ---
> Staging: android: fix parenthesis coding style issue in alarm-dev.c
What happened to your Subject:?
Also, why put the '---' line here? That means that git will delete
everything else.
> This is a patch
Fix Warning from checkpatch.pl
Block comments use * on subsequent lines
Signed-off-by: Harold Gomez
---
drivers/staging/media/atomisp/i2c/ap1302.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/media/atomisp/i2c/ap1302.c
b/drivers/staging/media/atomisp
> -Original Message-
> From: armetallica [mailto:armetall...@gmail.com]
> Sent: Sunday, July 9, 2017 7:58 PM
> To: gre...@linuxfoundation.org
> Cc: *S-Par-Maintainer ;
> de...@driverdev.osuosl.org; linux-ker...@vger.kernel.org
> Subje
Em Mon, 10 Jul 2017 01:35:11 +0200
armetallica escreveu:
> From 043428d63637a6dd8e52449b73dbb8341885d7e4 Mon Sep 17 00:00:00 2001
> From: Armin Schoenlieb
> Date: Mon, 10 Jul 2017 01:12:52 +0200
> Subject: [PATCH] Staging: media: atomisp2: fixed trailing whitespace error in
>
Here is how I am going about making the patches. It is basically
what I have picked up from kernel newbies among other sites
and videos on making patches. I would be greatful for any
pointers on what seems to be the problem(s) with why it does
not produce a proper patch.
I will use drivers
From: Adithya Krishnamurthy
Fixed sparse warning fix.
Signed-off-by: Adithya Krishnamurthy
---
---
drivers/staging/vt6655/device_main.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/vt6655/device_main.c
b/drivers/staging/vt6655/device_main.c
index 83e4162
From: Adithya Krishnamurthy
Fixed sparse warning(symbol was not declared. Should it be static) fix.
Signed-off-by: Adithya Krishnamurthy
---
---
drivers/staging/vt6655/device_main.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/vt6655/device_main.c
b/driv
is is the friendly patch-bot of Greg Kroah-Hartman. You have sent him
a patch that has triggered this response. He used to manually respond
to these common problems, but in order to save his sanity (he kept
writing the same thing over and over, yet to different people), I was
created. Hopefully
On Sat, Mar 31, 2018 at 07:16:21PM +0530, Wasim Nazir wrote:
> This message contains confidential information and is intended only for the
> individual(s) named. If you are not the intended recipient, you are
> notified that disclosing, copying, distributing or taking any action in
> reliance on
Completely remove symlink codes and definitions.
In the previous patch, it was not completely removed.
Signed-off-by: Tetsuhiro Kohada
---
drivers/staging/exfat/exfat.h | 3 ---
drivers/staging/exfat/exfat_core.c | 3 ---
drivers/staging/exfat/exfat_super.c | 27
This patch fixes multiple line dereference warning reported
by checkpatch script.
Signed-off-by: Janani S
---
drivers/staging/wlan-ng/hfa384x_usb.c | 12
1 file changed, 4 insertions(+), 8 deletions(-)
diff --git a/drivers/staging/wlan-ng/hfa384x_usb.c
b/drivers/staging/wlan-ng
--
> 2.10.2
>
> ___
> devel mailing list
> de...@linuxdriverproject.org
> http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Hi,
This is the friendly patch-bot of Greg Kroah-Hartman. You have sent him
a patch that
"WARNING: Avoid multiple line dereference - prefer
'poid_par_priv->information_buf_len'"
"WARNING: Avoid multiple line dereference - prefer
'padapter->registrypriv.dev_network.Configuration'"
"WARNING: Avoid multiple line dereference - prefer
'Adapter->mppriv.workparam.io_value'"
Signed-off-by:
No. The original was more clear to read.
regards,
dan carpenter
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
The patch is included below.
I would like to apologize for the past few emails. It turns out the
documentation included in the eMail clients file is woefully out of
date. I have checked the patches for errors using the scripts provided.
However, it turns out that the scripts are slightly incorrect
>From 1dbe78ce98037da5467d817a9db838d678b338ae Mon Sep 17 00:00:00 2001
From: Sanidhya Solanki
Date: Fri, 11 Dec 2015 03:08:53 -0500
Subject: [PATCH] staging: dgnc: Patch includes the checkpatch fixes
Patch contains the checkpatch fixes as asked by the TODO.
TODO file is updated as well.
Sig
This set of patches cleans up some more of the code by eliminating unnecessary
goto statements, getting rid of the BOOL define and switching over code that
used it to use int instead, and switching from using pragma(pack) to __packed.
*** BLURB HERE ***
Benjamin Romer (3):
staging: unisys: remo
This series of patch includes new design for 64 bits. The driver uses the
redundant typecasting to communicate with the chipset, which causes several
compile warnings.
However, this patch uses the real data type and removes unnecessary typecasting.
Also, the driver allocates the ID value to the
From: Niranjan Dighe
Remove IOC_LIBCFS_MEMHOG ioctl functionality as it is no longer needed thereby
making functions like - kportal_memhog_alloc(), kportal_memhog_free() and type -
struct libcfs_device_userstate unused.
Signed-off-by: Niranjan Dighe
---
.../lustre/include/linux/libcfs/libcfs_p
Declared cfg80211_infrastructure_mode method static to remove sparse warning
Signed-off-by: Rahul Garg
---
drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c
b/drivers/staging/rt
Now the Hyper-V code has it's own monkey on a tree!
Make it easier to manage patch flow to upper level maintainers.
Acked-by: Haiyang Zhang
Acked-by: K. Y. Srinivasan
Signed-off-by: Sasha Levin
---
MAINTAINERS | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/MAINTA
On Mon, Dec 08, 2014 at 11:33:25PM +0530, Adithya K wrote:
> From: Adithya Krishnamurthy
>
> Fixed sparse warning fix.
you should mention what type of warning you have fixed.
sudip
>
> Signed-off-by: Adithya Krishnamurthy
> ---
> ---
> drivers/staging/vt6655/device_main.c | 2 +-
> 1 file ch
On Tue, Dec 09, 2014 at 08:58:31PM +0530, Adithya K wrote:
> From: Adithya Krishnamurthy
>
> Fixed sparse warning(symbol was not declared. Should it be static) fix.
>
> Signed-off-by: Adithya Krishnamurthy
> ---
> ---
> drivers/staging/vt6655/device_main.c | 2 +-
> 1 file changed, 1 insertion
Fix checkpatch warning "CHECK: Please don't use multiple blank lines"
on upc.h
Signed-off-by: Cesar Santos
---
drivers/staging/vt6655/upc.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/staging/vt6655/upc.h b/drivers/staging/vt6655/upc.h
index d028ad2ba0d4..e086ec6e77f7 100644
---
Make common function for detaching internals of device
during changes to MTU and RSS. Make sure no more packets
are transmitted and all packets have been received before
doing device teardown.
Change the wait logic to be common and use usleep_range().
Changes transmit enabling logic so that trans
On Wed, 26 Feb 2020 15:37:46 +0900, Tetsuhiro Kohada said:
> Completely remove symlink codes and definitions.
> In the previous patch, it was not completely removed.
Then this should have been [PATCH v2], and the fixed version [PATCH v3]
> Signed-off-by: Tetsuhiro Kohada
> ---
Thanks for comment.
> Then this should have been [PATCH v2], and the fixed version [PATCH
> v3]
2nd patch(Additional patch) not include 1st patch(RFC PATCH).
And the 1st patch has been merged into 'staging-next'.
Now I can't decide.
a) Add only version information to the 2n
On Thu, 27 Feb 2020 02:14:02 +,
"kohada.tetsuh...@dc.mitsubishielectric.co.jp" said:
> Thanks for comment.
>
> > Then this should have been [PATCH v2], and the fixed version [PATCH
> > v3]
>
> 2nd patch(Additional patch) not include 1st patch(RFC PATCH).
&
WARNING: Statements should start on a tabstop
Signed-off-by: Prit Raj
---
drivers/staging/fbtft/fb_agm1264k-fl.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/fbtft/fb_agm1264k-fl.c
b/drivers/staging/fbtft/fb_agm1264k-fl.c
index 82b46cd..7561385 100644
Since the netvsc_channel_cb is already called in interrupt
context from vmbus, there is no need to do irqsave/restore.
Signed-off-by: Stephen Hemminger
---
drivers/net/hyperv/netvsc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/hyperv/netvsc.c b/drivers/net/hy
These are improvements to netvsc driver. They aren't functionality
changes so not targeting net-next; and they are not show stopper
bugs that need to go to stable either.
v2
- drop the irq flags patch, defer it to net-next
- split the multicast filter flag patch out
- change propoga
In this patch:
1. the if condition is modified to remove
the unnecessary equality check.
2. The else if condition is replaced by an else condition
since the variable checked in the if statement is a bool
making the else if unnecessary.
3. Fixed indentation problem on line 1369.
These changes were
On Sat, Dec 05, 2015 at 06:44:10AM -0500, Sanidhya Solanki wrote:
> The patch is included below.
> I would like to apologize for the past few emails. It turns out the
> documentation included in the eMail clients file is woefully out of
> date. I have checked the patches for erro
The patch is still corrupt. Read Documentation/email-clients.txt and
test it as described.
regards,
dan carpenter
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
On Fri, Dec 11, 2015 at 03:21:49AM -0500, Sanidhya Solanki wrote:
> >From 1dbe78ce98037da5467d817a9db838d678b338ae Mon Sep 17 00:00:00 2001
> From: Sanidhya Solanki
> Date: Fri, 11 Dec 2015 03:08:53 -0500
> Subject: [PATCH] staging: dgnc: Patch includes the checkpatch fixes
Don
if (bd->state != BOARD_READY)
> > + return 0;
> > + } while (0);
>
> Google about why do while(0) loops are used in macros and then redo
> this. Mostly the patch isn't bad, but I suspect I'm going to complain
> about how you split up so
);
> > > + if (!bd || bd->magic != DGNC_BOARD_MAGIC)
> > > + return 0;
> > > + if (bd->state != BOARD_READY)
> > > + return 0;
> > > + } while (0);
> >
> > Google about why do while(0) l
This patch V2 includes only patches rejected. The rejected patch for the
following, #ifdef conditionals cover entire functions, introduced new error on
required space before the open parenthesis '('. This is my mistake while
rewriting codes.
The original 13th patch didn't
On Fri, 11 Dec 2015 16:02:33 +0300
Dan Carpenter wrote:
> On Fri, Dec 11, 2015 at 03:21:49AM -0500, Sanidhya Solanki wrote:
> > >From 1dbe78ce98037da5467d817a9db838d678b338ae Mon Sep 17 00:00:00 2001
> > From: Sanidhya Solanki
> > Date: Fri, 11 Dec 2015 03:08:53 -
0 2001
> > > From: Sanidhya Solanki
> > > Date: Fri, 11 Dec 2015 03:08:53 -0500
> > > Subject: [PATCH] staging: dgnc: Patch includes the checkpatch fixes
> >
> >
> > Don't include this stuff.
> Whenever I remove that part, the "git am"
a Solanki wrote:
> > > > >From 1dbe78ce98037da5467d817a9db838d678b338ae Mon Sep 17 00:00:00 2001
> > > > From: Sanidhya Solanki
> > > > Date: Fri, 11 Dec 2015 03:08:53 -0500
> > > > Subject: [PATCH] staging: dgnc: Patch includes the checkpatch fixes
> >
Thank you everyone for your concern regarding my email client.
Unfortunately, there is a bug in "git send-email" that leads to the gmail
SMTP server rejecting certain patches sent by the git command. If I can
replicate and fix the issue for " git am" (which I do apply after emailing
to myself) I
Is it? I am using git send-email from my beginning, never faced any
problem in sending patch through gmail smtp server. Can you please send
me the patch (offlist), so that i can test with my setup.
regards
sudip
___
devel mailing list
de...@linux
Sorry about Patch 1/3 having the header information. The rest should be fine.
I broke up the patches as you requested.
I can also confirm that git am gives me an error if the patch to be applied
does not contain the header information. Specifically, "Patch format detection
failed.".
On Wed, Dec 23, 2015 at 02:33:29PM +0530, Niranjan Dighe wrote:
> From: Niranjan Dighe
>
> Remove IOC_LIBCFS_MEMHOG ioctl functionality as it is no longer needed thereby
> making functions like - kportal_memhog_alloc(), kportal_memhog_free() and
> type -
> struct libcfs_device_userstate unused.
I think patch subject lines with "checkpatch" in
them are almost never really useful.
Maybe a new checkpatch test to see if a subject line
is perhaps less than informational should be added.
Something like:
---
scripts/checkpatch.pl | 7 +++
1 file changed, 7 insertions(+)
di
Fix Warning Block comments use * on subsequent lines
Signed-off-by: Harold Gomez
---
drivers/staging/media/atomisp/i2c/ap1302.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/media/atomisp/i2c/ap1302.c
b/drivers/staging/media/atomisp/i2c/ap1302.c
index
Hi Harold,
Please use a better subject line.
On Thu, Aug 10, 2017 at 03:08:26PM +0530, Harold Gomez wrote:
> Fix Warning from checkpatch.pl
> Block comments use * on subsequent lines
>
> Signed-off-by: Harold Gomez
> ---
> drivers/staging/media/atomisp/i2c/ap1302.c | 5 +++--
> 1 file changed
Hi,
I'm submitting this patch as part of Eudyptula challenge to fix a
coding style problem. Thanks for taking time on this trivial patch.
sl424
Chewie Lin (1):
fixed a checkpatch warning
drivers/staging/vt6656/main_usb.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
--
2
Hi greg k-h and forest:
Sorry about all the spam I've been sending earlier. One more try.
I'm submitting this patch as part of Eudyptula challenge to fix a
coding style problem. Thanks for taking time on this trivial patch.
sl424
Chewie Lin (1):
fixed a checkpatch warning
drive
Hi greg and forest:
I hate to bother but one more, with better changelog.
I'm submitting this patch as part of Eudyptula challenge to fix a
coding style problem. Thanks for taking time on this trivial patch.
sl424
Chewie Lin (1):
drivers/staging/vt6656/main_usb.c: checkpatch wa
Hi all:
Better and improved changelog version 3.
I'm submitting this patch as part of Eudyptula challenge to fix a
coding style problem. Thanks for taking time on this trivial patch.
sl424
Chewie Lin (1):
drivers/staging/vt6656/main_usb.c: checkpatch warning
drivers/staging/v
When not associated with an AP, wifi device drivers should respond to the
SIOCGIWESSID ioctl with a zero-length string for the SSID, which is the
behavior expected by dhcpcd.
Currently, this driver returns an error code (-1) from the ioctl call,
which causes dhcpcd to assume that the device is not
From: Kangmin Park
enum RT_RF_TYPE_DEFINITION on rtl8723bs/include/rtw_rf.h
is different from enum tag_HAL_RF_Type_Definition on
rtl8723bs/include/HalVerDef.h
So, update them to be the same reference from
enum rf_type on rtlwifi/wifi.h which recent version.
As a result, ODM_RF_TYPE_E needs to b
(s) with why it does
> not produce a proper patch.
Honestly, it's not our job to review someone's patch creation
proceedures and notes as everyone does it differently. We will be glad
to review your patches that you create.
I think the review that I previously provided should be
> -Original Message-
> From: linux-kernel-ow...@vger.kernel.org [mailto:linux-kernel-
> ow...@vger.kernel.org] On Behalf Of feas
> Sent: Monday, November 14, 2016 7:16 PM
> To: de...@driverdev.osuosl.org; gre...@linuxfoundation.org; linux-
> ker...@vger.kernel.or
: de...@driverdev.osuosl.org; gre...@linuxfoundation.org; linux-
> > ker...@vger.kernel.org
> > Subject: Patch procedure
> >
>
>
>
> I know you mentioned the kernel newbies page but just in case you missed this
> toturial, here is a link https://kernelnewbies.org/Firs
The main goal of this series is to get rid of a needless and ugly typedef
in the rtl8712 wlan driver.
In the course of fixing that, I found a bug thati will can might (at least in
theory)
lead to a overrun during a memcpy, as well as a duplicate struct.
Finally after cleaning up the typedef, I co
I just got mutt working, the last message was an error as it was a blank.
Dan, I will avoid Gmail WebUI like the plague now when I am talking to the
kernel mailing list.
Joe, thanks for the advice, I will work on it and hopefully have something to
send by Monday.
___
>From 5b6fbb6496ddc4e428f5dd0950b0617faed6596b Mon Sep 17 00:00:00 2001
From: Paul McQuade
Date: Wed, 7 Aug 2013 16:21:16 +0100
Subject: [PATCH] Staging:BCM:DDRInit.c:Replacing __FUNCTION__
__func__ replaces __Function__.
Signed-off-by: Paul McQuade
---
drivers/staging/bcm/DDRInit.c |
>From 31cc3c00d2699a846a8bb54d0831ab6480c8bd12 Mon Sep 17 00:00:00 2001
From: Paul McQuade
Date: Wed, 7 Aug 2013 20:29:13 +0100
Subject: [PATCH] Subject: [PATCH] Staging:BCM:DDRInit.c:Replacing
__FUNCTION__
__func__ replaces __Function__.
Signed-off-by: Paul McQuade
---
drivers/staging/
__Function__ gets replaced with __func__
Signed-Off-By: Paul McQuade
---
drivers/staging/bcm/DDRInit.c | 54
-
1 file changed, 27 insertions(+), 27 deletions(-)
diff --git a/drivers/staging/bcm/DDRInit.c b/drivers/staging/bcm/DDRInit.c
index 8c696b6..
>From c21d0da84c92d351f37b70c0d9c01a66fcb84e88 Mon Sep 17 00:00:00 2001
From: Paul McQuade
Date: Thu, 15 Aug 2013 20:00:50 +0100
Subject: [PATCH] Staging:BCM:DDRInit.c:Replacing __FUNCTION__
__Function__ gets replaced with __func__
Signed-Off-By: Paul McQuade
---
drivers/staging/
>From c21d0da84c92d351f37b70c0d9c01a66fcb84e88 Mon Sep 17 00:00:00 2001
From: Paul McQuade
Date: Thu, 15 Aug 2013 20:00:50 +0100
Subject: [PATCH] Staging:BCM:DDRInit.c:Renaming __FUNCTION__
__Function__ gets renamed with __func__
Signed-Off-By: Paul McQuade
---
drivers/staging/bcm/DDRIni
Add new entry for bcm wimax
Kevin McKinney (1):
Staging: bcm: Add entry for bcm wimax driver support
MAINTAINERS |7 +++
1 file changed, 7 insertions(+)
--
1.7.9.5
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxd
>From f548b09a0fc387b94fee3372c0f7c4de1f281f80 Mon Sep 17 00:00:00 2001
From: Paul McQuade
Date: Thu, 3 Apr 2014 17:12:20 +0100
Subject: [PATCH] Staging:ft1000_debug.c:Renaming __FUNCTION__
__Function__ gets renamed with __func__
Signed-off-by: Paul McQuade
---
drivers/staging/ft1000/ft1
fbtft-bus.c:
fixing ERROR: space prohibited before that close parenthesis ')' by removing
that space and ',' in line 65 and 67.
Signed-off-by: B K Karthik
---
drivers/staging/fbtft/fbtft-bus.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/fbtft/fbtft-b
On Tue, Dec 11, 2018 at 12:09:49PM -0500, Sasha Levin wrote:
> Now the Hyper-V code has it's own monkey on a tree!
>
> Make it easier to manage patch flow to upper level maintainers.
>
> Acked-by: Haiyang Zhang
> Acked-by: K. Y. Srinivasan
> Signed-off-by: Sasha Levin
On Tue, Dec 11, 2018 at 07:09:59PM +0100, Greg KH wrote:
On Tue, Dec 11, 2018 at 12:09:49PM -0500, Sasha Levin wrote:
Now the Hyper-V code has it's own monkey on a tree!
Make it easier to manage patch flow to upper level maintainers.
Acked-by: Haiyang Zhang
Acked-by: K. Y. Srinivasan
S
Hello!
This patch series cleans up some checkpatch fixes in erofs. The patches
include balancing conditional braces and fixing some indentation. No testing
done, all patches build and checkpath cleanly.
Caitlyn (2):
staging/erofs/xattr.h: Fixed misaligned function arguments.
staging/erofs
From: Jérôme Pouiller
Hi,
The following PR contains now usual maintenance for the wfx driver. I have
more-or-less sorted the patches by importance:
- the first ones are fixes for a few corner-cases reported by users
- the patches 9 and 10 add support for CSA and TDLS
- then the end o
kernel panic is here:
Unable to handle kernel paging request at virtual address b038
pgd = d9d94000
[b038] *pgd=
Internal error: Oops: 2805 [#1] PREEMPT SMP ARM
...
task: daa2dd00 task.stack: da194000
PC is at v7_dma_clean_range+0x1c/0x34
LR is at arm_dma_sync_single_for_device+0x4
In the original commit 4d99b2581effe115376402e710fbcb1c3c073769
was missed one hunk. Added it now to avoid issue with use after free.
Signed-off-by: Dmitry Eremin
---
drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers
gt;
>
> 3329
> &kiblnd_data.kib_reconn_wait);
After attached patch this code will use struct kib_conn only when it was not
freed.
Signed-off-by: Dm
The netvsc driver was always enabling all multicast and broadcast
even if netdevice flag had not enabled it.
Signed-off-by: Stephen Hemminger
---
drivers/net/hyperv/rndis_filter.c | 20
1 file changed, 12 insertions(+), 8 deletions(-)
diff --git a/drivers/net/hyperv/rndis_f
1 - 100 of 58646 matches
Mail list logo
