Fixed a brace coding style issue, found via checkpatch.
Signed-off-by: Riccardo Marotti
---
drivers/staging/rtl8192u/ieee80211/ieee80211_module.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_module.c
b/drivers/staging
On Sun, May 7, 2017 at 5:14 AM, Greg KH wrote:
> On Thu, May 04, 2017 at 10:06:51PM +0200, Riccardo wrote:
>> Fixed a brace coding style issue, found via checkpatch.
>>
>> Signed-off-by: Riccardo Marotti
>
> This name doesn't match the name on the From: line of y
From: Riccardo
Fixed a brace coding style issue, found via checkpatch.
Signed-off-by: Riccardo Marotti
---
drivers/staging/rtl8192u/ieee80211/ieee80211_module.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_module.c
b
On 05/07/2017 08:47 AM, Riccardo Marotti wrote:
> From: Riccardo
>
> Fixed a brace coding style issue, found via checkpatch.
>
> Signed-off-by: Riccardo Marotti
> ---
> drivers/staging/rtl8192u/ieee80211/ieee80211_module.c | 3 +--
> 1 file changed, 1 insertion(+),
Fixed a brace coding style issue, found via checkpatch.
Signed-off-by: Riccardo Marotti
---
drivers/staging/rtl8192u/ieee80211/ieee80211_module.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_module.c
b/drivers/staging
Fixed a brace coding style issue, found via checkpatch.
Signed-off-by: Riccardo Marotti
---
Changes in v2:
- Fix mismatch between "Signed-off-by:" and "From:" names.
Changes in v3:
- Fix missing summary of changes in version 2.
drivers/staging/rtl8192u/ieee80211/ie
On 05/12/2017 11:07 AM, Greg KH wrote:
> On Sun, May 07, 2017 at 11:01:07AM +0200, Riccardo Marotti wrote:
>> Fixed a brace coding style issue, found via checkpatch.
>>
>> Signed-off-by: Riccardo Marotti
>> ---
>> drivers/staging/rtl8192u/ieee80211/ieee80211_mo
Hello,
This patch series fixes two coding style issues and removes an unneeded
'if' statement in lov_request.c.
Thanks,
riccardo
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo
Fix the following checkpatch.pl issue in lov_request.c:
WARNING: Missing a blank line after declarations
Signed-off-by: Riccardo Lucchese
---
drivers/staging/lustre/lustre/lov/lov_request.c | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/lustre/lustre
It is silly to go through an if statement to set a single boolean
value in function of a single boolean expression. In the function
lov_check_set, assign the return value directly.
Signed-off-by: Riccardo Lucchese
---
drivers/staging/lustre/lustre/lov/lov_request.c | 11 +--
1 file
Fix the following checkpatch.pl issue in lov_request.c:
ERROR: space required before the open brace '{'
Signed-off-by: Riccardo Lucchese
---
drivers/staging/lustre/lustre/lov/lov_request.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/lus
On Sat, Jul 19, 2014 at 12:59:22PM -0700, Joe Perches wrote:
> On Sat, 2014-07-19 at 21:34 +0200, Riccardo Lucchese wrote:
> > It is silly to go through an if statement to set a single boolean
> > value in function of a single boolean expression. In the function
> > lov_
Dan,
On Sun, Jul 20, 2014 at 07:52:53AM +0300, Dan Carpenter wrote:
> On Sat, Jul 19, 2014 at 09:34:56PM +0200, Riccardo Lucchese wrote:
> > It is silly to go through an if statement to set a single boolean
> > value in function of a single boolean expression. In the function
&g
On Sun, Jul 20, 2014 at 02:37:47PM +0300, Dan Carpenter wrote:
> On Sun, Jul 20, 2014 at 01:08:36PM +0200, Riccardo Lucchese wrote:
> > Dan,
> >
> > On Sun, Jul 20, 2014 at 07:52:53AM +0300, Dan Carpenter wrote:
> > > On Sat, Jul 19, 2014 at 09:34:56PM
Fix the following checkpatch.pl issue in lov_request.c:
WARNING: Missing a blank line after declarations
Signed-off-by: Riccardo Lucchese
Acked-by: Julia Lawall
---
Changes in v2:
- Changed the subject line to follow the same convention used by
previous commits to the same file.
- Added
Hello,
This patch series contains one cleanup and two coding style fixes for
lov_request.c.
This iteration addresses the review comments received by v1.
Changes in v2:
- Improved the commit messages.
- Introduced a local variable in lov_check_set() to make the code clearer.
Thanks,
riccardo
Make the code clearer by introducing a local variable and removing the
unnecessary 'if' statement.
Signed-off-by: Riccardo Lucchese
Acked-by: Julia Lawall
---
Changes in v2:
- Improved the commit message:
- changed the subject line to follow the same convention used by
Fix the following checkpatch.pl issue in lov_request.c:
ERROR: space required before the open brace '{'
Signed-off-by: Riccardo Lucchese
Acked-by: Julia Lawall
---
Changes in v2:
- Changed the subject line to follow the same convention used by
previous commits to the same file
On Sun, Jul 20, 2014 at 08:30:23AM -0700, Joe Perches wrote:
> On Sun, 2014-07-20 at 15:22 +0200, Riccardo Lucchese wrote:
> > Make the code clearer by introducing a local variable and removing the
> > unnecessary 'if' statement.
> >
> > Signed-off-by: R
Make the code clearer by introducing a local variable and removing the
unnecessary 'if' statement.
Signed-off-by: Riccardo Lucchese
Acked-by: Julia Lawall
---
Changes in v3:
- Renamed the local variable 'desc' to 'tgt' to be consistent with
other definitions
:
- Improved the commit messages.
- Introduced a local variable in lov_check_set() to make the code
clearer.
Thanks,
riccardo
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Fix the following checkpatch.pl issue in lov_request.c:
WARNING: Missing a blank line after declarations
Signed-off-by: Riccardo Lucchese
Acked-by: Julia Lawall
---
No changes in v3.
Changes in v2:
- Changed the subject line to follow the same convention used by
previous commits to the
Fix the following checkpatch.pl issue in lov_request.c:
ERROR: space required before the open brace '{'
Signed-off-by: Riccardo Lucchese
Acked-by: Julia Lawall
---
No changes in v3.
Changes in v2:
- Changed the subject line to follow the same convention used by
previous commits t
>Da: Jérôme Pouiller
>Inviato: venerdì 18 febbraio 2022 15:04
>A: Greg Kroah-Hartman; Riccardo Ferrazzo
>Cc: linux-wirel...@vger.kernel.org; net...@vger.kernel.org; Kalle Valo;
>de...@driverdev.osuosl.org; linux-ker...@vger.kernel.org; David S . Miller
>Oggetto: Re: [PATCH]
24 matches
Mail list logo
