On Wed, Apr 14, 2021 at 10:55:25AM +0300, Dan Carpenter wrote:
> On Wed, Apr 14, 2021 at 12:26:01PM +0530, Mitali Borkar wrote:
> > Added a generic function of static inline bool in
> > include/linux/etherdevice.h to replace memcmp with
> > ether_oui_equal throughout the exe
On Wed, Apr 14, 2021 at 10:16:59AM +0200, Greg KH wrote:
> On Wed, Apr 14, 2021 at 12:26:01PM +0530, Mitali Borkar wrote:
> > Added a generic function of static inline bool in
> > include/linux/etherdevice.h to replace memcmp with
> > ether_oui_equal throughout the exe
On Wed, Apr 14, 2021 at 12:38:41PM +0200, z...@gentoo.org wrote:
> Am 13.04.21 um 21:50 schrieb Mitali Borkar:
> > Added #include and replaced bit shifts by BIT() macro.
> > This BIT() macro from linux/bitops.h is used to define ZR36057_VFESPFR_*
> > bitmasks.
>
Removed unnecessary (void *) cast and parentheses to meet linux kernel
coding style.
Signed-off-by: Mitali Borkar
---
Changes from v5:- No changes.
Changes from v4:- Removed unnecessary casts and parentheses.
Changes from v3:- No changes.
Changes from v2:- Rectified spelling mistake in subject
Removed unnecessary parentheses around '!xyz' boolean expression as '!'
has higher precedance than '||'
Signed-off-by: Mitali Borkar
---
Changes from v5:- No changes.
Changes from v4:- No changes.
Changes from v3:- No changes.
Changes from v2:- Modified subject description. Changes has been
]:- Removed unnecessary parentheses around boolean expression.
Changes has been made in v2.
[PATCH 2/2]:- No changes.
Mitali Borkar (2):
staging: rtl8192e: remove parentheses around boolean expression
staging: rtl8192e: remove casts and parentheses
drivers/staging/rtl8192e/rtl819x_HTProc.c | 14
Removed multiple blank lines to make code better and neater.
Reported by checkpatch.
Signed-off-by: Mitali Borkar
---
Changes from v1:- No changes.
drivers/staging/rtl8192e/rtl819x_HTProc.c | 7 ---
1 file changed, 7 deletions(-)
diff --git a/drivers/staging/rtl8192e/rtl819x_HTProc.c
b
Added a generic function of static inline bool in
include/linux/etherdevice.h to replace memcmp with
ether_oui_equal throughout the execution.
Corrected the misspelled words in this file.
Signed-off-by: Mitali Borkar
---
Changes from v1:- Rectified spelling mistake and replaced memcmp
Removed an extra blank line after close brace '{' as it was not
necessary.
Reported by checkpatch
Signed-off-by: Mitali Borkar
---
Changes from v1:- No changes.
drivers/staging/rtl8192e/rtl819x_HTProc.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/staging/rtl8192e
Removed an extra blank line before close brace as it was not necessary.
Reported by checkpatch.
Signed-off-by: Mitali Borkar
---
Changes from v1:- No changes.
drivers/staging/rtl8192e/rtl819x_HTProc.c | 2 --
drivers/staging/rtl8192e/rtl819x_TSProc.c | 2 --
2 files changed, 4 deletions
Added spaces around binary operators like '+', '*', '|', '-', '&',
to improve readability and to meet linux kernel coding style.
Signed-off-by: Mitali Borkar
---
Changes from v1:- No changes.
drivers/staging/rtl8192e/rtl819x_HTProc.c | 8
drivers/staging/rtl8192e/rtl819x_TSPr
]:- No changes.
Mitali Borkar (5):
staging: rtl8192e: add spaces around binary operators
staging: rtl8192e: remove unnecessary blank line before brace
staging: rtl8192e: remove unnecessary blank line after close brace
staging: rtl8192e: rectified spelling mistake and replace memcmp
On Tue, Apr 13, 2021 at 09:17:12PM +0300, Dan Carpenter wrote:
> On Tue, Apr 13, 2021 at 08:59:34PM +0530, Mitali Borkar wrote:
> > Removed unnecessary space before tabs to adhere to linux kernel coding
> > style.
> > Reported by checkpatch.
> >
>
everything properly by using tabs to make code neater
and improve readability.
Reported by checkpatch.
Signed-off-by: Mitali Borkar
---
Changes from v3:- No changes.
Changes from v2:- Aligned the code using tabs.
Changes from v1:- Aligned the code using tabs and readjusted the
comments line.
drivers
Added spaces around '<<' operator to improve readability and meet linux
kernel coding style.
Reported by checkpatch
Signed-off-by: Mitali Borkar
---
Changes from v3:- No changes.
Changes from v2:- No changes.
Changes from v1:- No changes.
drivers/staging/media/zoran/zr36057.
Changes from v3:- Rebased these patches and made changes against
mainline code.
Changes from v2:-
[PATCH 1/2]:- No changes.
[PATCH 2/2]:- Aligned the code using tabs.
Changes from v1:-
[PATCH 1/2]:- No changes.
[PATCH 2/2]:- Aligned the code using tabs and readjusted the
comments line.
Mitali
Added '*' before every line inside long(multi-line) comments. Removed
'*/' from end of the comment line and added to next line as per linux
kernel coding style. Aligned '*' accordingly to make code neater.
Signed-off-by: Mitali Borkar
---
Changes from v3:- Rebased this patch and made changes
Reduced length of a line which exceed the 100 columns limit by splitting
the line into two statements and commenting it with '*' to meet linux
kernel coding style for long(multi-line) comments.
Reported by checkpatch.
Signed-off-by: Mitali Borkar
---
Changes from v2:- Rebased this patch
Declared 32 bit unsigned int as static constant inside a function and
replaced u32[] {x,y} as canvas3, canvas4 in codec_h264.c
This indicates the value of canvas indexes will remain constant
throughout execution.
Signed-off-by: Mitali Borkar
---
Changes from v3:- Made commit message under 75
Removed unnecessary space before tabs to adhere to linux kernel coding
style.
Reported by checkpatch.
Signed-off-by: Mitali Borkar
---
Changes from v2:- No changes.
Changes from v1:- No changes.
drivers/staging/media/ipu3/include/intel-ipu3.h | 2 +-
1 file changed, 1 insertion(+), 1
. This was
patch 5/6 previously.
[PATCH 4/4]:- No changes.
Mitali Borkar (4):
staging: media: intel-ipu3: remove unnecessary blank line
staging: media: intel-ipu3: reduce length of line
staging: media: intel-ipu3: line should not end with '['
staging: media: intel-ipu3: remove space before tabs
Fixed the issue of line should not end with '[' by moving argument
from next line to line ending with '[' and made it under 80
characters.
Reported by checkpatch.
Signed-off-by: Mitali Borkar
---
Changes from v2:- No changes
Changes from v1:- Made line length under 80 characters
drivers
Removed an unnecessary blank line to meet linux kernel coding style.
Reported by checkpatch.pl
Signed-off-by: Mitali Borkar
---
Changes from v2:- No changes.
Changes from v1:- NO changes.
drivers/staging/media/ipu3/include/intel-ipu3.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers
Reduced length of line as it was exceeding 100 characters by removing
comments from same line and adding it to previous line. This makes code
neater, and meets linux kernel coding style.
Reported by checkpatch.
Signed-off-by: Mitali Borkar
---
Changes from v2:- No changes.
Changes from v1
Declared 32 bit unsigned int as static constant inside a function and
replaced u32[] {x,y} as canvas3, canvas4 in codec_h264.c
This indicates the value of canvas indexes will remain constant throughout
execution.
Reported-by: kernel test robot
Signed-off-by: Mitali Borkar
---
Changes from v2
On Tue, Apr 13, 2021 at 09:26:01AM +0200, Hans Verkuil wrote:
> On 13/04/2021 08:27, Mitali Borkar wrote:
> > Declared 32 bit unsigned int as static constant inside a function
> > appropriately.
> >
> > Reported-by: kernel test robot
> > Signed-off-by: Mitali
On Fri, Apr 09, 2021 at 07:31:53PM -0700, Joe Perches wrote:
> On Sat, 2021-04-10 at 07:55 +0530, Mitali Borkar wrote:
> > On Fri, Apr 09, 2021 at 07:07:09PM -0700, Joe Perches wrote:
> > > On Sat, 2021-04-10 at 07:05 +0530, Mitali Borkar wrote:
> > > > Matched the a
On Tue, Apr 13, 2021 at 01:44:32PM +0300, Sakari Ailus wrote:
> On Tue, Apr 13, 2021 at 04:13:04PM +0530, Mitali Borkar wrote:
> > On Tue, Apr 13, 2021 at 01:01:34PM +0300, Sakari Ailus wrote:
> > > Hi Mitali,
> > >
> > > Thanks for the update.
> > &
On Tue, Apr 13, 2021 at 09:52:48AM +0200, Greg KH wrote:
> On Tue, Apr 13, 2021 at 08:55:03AM +0530, Mitali Borkar wrote:
> > Changes from v4:-
> > [PATCH v4 1/3]:- No changes.
> > [PATCH v4 2/3]:- No changes.
> > [PATCH V4 3/3]:- Removed casts and parentheses.
>
On Tue, Apr 13, 2021 at 01:01:34PM +0300, Sakari Ailus wrote:
> Hi Mitali,
>
> Thanks for the update.
>
> On Tue, Apr 13, 2021 at 10:46:06AM +0530, Mitali Borkar wrote:
> > Reduced length of the line under 80 characters to meet linux-kernel
> > coding style.
> >
Declared 32 bit unsigned int as static constant inside a function
appropriately.
Reported-by: kernel test robot
Signed-off-by: Mitali Borkar
---
Changes from v1:- Rectified the mistake by declaring u32 as static const
properly.
drivers/staging/media/meson/vdec/codec_h264.c | 4 ++--
1 file
Removed unnecessary space before tabs to adhere to linux kernel coding
style.
Reported by checkpatch.
Signed-off-by: Mitali Borkar
---
Changes from v1:- No changes.
drivers/staging/media/ipu3/include/intel-ipu3.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers
Reduced length of the line under 80 characters to meet linux-kernel
coding style.
Signed-off-by: Mitali Borkar
---
Changes from v1:- Reduced length of the line under 80 characters
drivers/staging/media/ipu3/include/intel-ipu3.h | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff
Reduced length of line as it was exceeding 100 characters by removing
comments from same line and adding it to previous line. This makes code
neater, and meets linux kernel coding style.
Reported by checkpatch.
Signed-off-by: Mitali Borkar
---
Changes from v1:- No changes.
drivers/staging
Removed an unnecessary blank line to meet linux kernel coding style.
Reported by checkpatch.pl
Signed-off-by: Mitali Borkar
---
Changes from v1:- No changes.
drivers/staging/media/ipu3/include/intel-ipu3.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/media/ipu3/include
Changes from v1:-
Dropped patches 1/6 and 2/6 and compiled this as a patchset of 4
patches.
[PATCH 1/4]:- No changes.
[PATCH 2/4]:- No changes.
[PATCH 3/4]:- Reduced length of a line under 80 characters. This was
patch 5/6 previously.
[PATCH 4/4]:- No changes.
Mitali Borkar (4):
staging: media
Removed unnecessary (void *) cast and parentheses to meet linux kernel
coding style.
Signed-off-by: Mitali Borkar
---
Changes from v4:- Removed unnecessary casts and parentheses.
Changes from v3:- No changes.
Changes from v2:- Rectified spelling mistake in subject description.
Changes has been
Removed unncessary ftrace-like logging by simply deleting that statement
as we have other modes of logging like ftrace.
Reported by checkpatch.
Signed-off-by: Mitali Borkar
---
Changes from v4:- No changes.
Changes from v3:- No changes.
Changes from v2:- No changes.
Changes from v1
Removed unnecessary parentheses around '!xyz' boolean expression as '!'
has higher precedance than '||'
Signed-off-by: Mitali Borkar
---
Changes from v4:- No changes.
Changes from v3:- No changes.
Changes from v2:- Modified subject description. Changes has been made in
v3.
Changes from v1
.
Changes has been made in v2.
[PATCH 2/3]:- No changes.
[PATCH 3/3]:- No changes.
Mitali Borkar (3):
staging: rtl8192e: remove parentheses around boolean expression
staging: rtl8192e: remove unnecessary ftrace-like logging
staging: rtl8192e: remove casts and parentheses
drivers/staging
On Mon, Apr 12, 2021 at 01:46:54PM -0700, Joe Perches wrote:
> On Mon, 2021-04-12 at 16:52 +0530, Mitali Borkar wrote:
> > Removed unnecessary parentheses because they must be used only when it
> > is necessary or they improve readability.
> > Reported by checkpatch.
>
>
On Mon, Apr 12, 2021 at 12:43:15PM +0300, Sakari Ailus wrote:
> Hi Mitali,
>
> On Mon, Apr 12, 2021 at 04:38:59AM +0530, Mitali Borkar wrote:
> > This patch fixes the warning identified by checkpatch.pl by replacing
> > __attribute__aligned(size) with __aligned(siz
Declared 32 bit unsigned int as static constant inside a function and
replaced u32[] {x,y} as canvas3, canvas4 in codec_h264.c
This indicates the value of canvas indexes will remain constant throughout
execution.
Signed-off-by: Mitali Borkar
---
drivers/staging/media/meson/vdec/codec_h264.c
On Mon, Apr 12, 2021 at 11:17:22AM +0200, Hans Verkuil wrote:
> On 10/04/2021 21:59, Mitali Borkar wrote:
> > Declared 32 bit unsigned int as static constant inside a function and
> > replaced u32[] {x,y} as canvas1, canvas2 in codec_mpeg12.c
> > This indicates the va
Removed unnecessary parentheses because they must be used only when it
is necessary or they improve readability.
Reported by checkpatch.
Signed-off-by: Mitali Borkar
---
Changes from v3:- No changes.
Changes from v2:- Rectified spelling mistake in subject description.
Changes has been made
Removed unncessary ftrace-like logging by simply deleting that statement
as we have other modes of logging like ftrace.
Reported by checkpatch.
Signed-off-by: Mitali Borkar
---
Changes from v3:- No changes.
Changes from v2:- No changes.
Changes from v1:- No changes.
drivers/staging/rtl8192e
Removed unnecessary parentheses around '!xyz' boolean expression as '!'
has higher precedance than '||'
Signed-off-by: Mitali Borkar
---
Changes from v3:- No changes.
Changes from v2:- Modified subject description. Changes has been made in
v3.
Changes from v1:- Removed unnecessary parentheses
]:- Rectified spelling mistake in subject description.
Changes has been made in v3.
Changes from v1:-
[PATCH 1/3]:- Removed unnecessary parentheses around boolean expression.
Changes has been made in v2.
[PATCH 2/3]:- No changes.
[PATCH 3/3]:- No changes.
Mitali Borkar (6):
staging: rtl8192e
Corrected the misspelled word.
Reported by checkpatch.
Signed-off-by: Mitali Borkar
---
Changes from v1:- No changes.
drivers/staging/vc04_services/interface/TODO | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/vc04_services/interface/TODO
b/drivers
Removed trailing whitespace to adhere to linux kernel coding style.
Reported by checkpatch.
Signed-off-by: Mitali Borkar
---
Changes from v1:- No changes.
drivers/staging/vc04_services/interface/TODO | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging
Changes from v1:- Threaded mails properly and changed the subject line
to match prefix on patches.
[PATCH 1/2]:- No changes.
[PATCH 2/2]:- No changes.
Mitali Borkar (2):
staging: vc04_services: remove trailing whitespace
staging: vc04_services: rectify spelling mistake
drivers/staging
On Mon, Apr 12, 2021 at 08:19:02AM +0200, Greg KH wrote:
> On Mon, Apr 12, 2021 at 11:40:53AM +0530, Mitali Borkar wrote:
> > Fixed the warning:-msleep < 20ms can sleep for up to 20ms by replacing
> > msleep(unsigned long msecs) by usleep_range(unsigned long min, unsi
Fixed the warning:-msleep < 20ms can sleep for up to 20ms by replacing
msleep(unsigned long msecs) by usleep_range(unsigned long min, unsigned long
max)
in usecs as msleep(1ms~20ms) can sleep for upto 20 ms.
Signed-off-by: Mitali Borkar
---
drivers/staging/rtl8188eu/core/rtw_mlme_ext.c
Removed unnecessary space before tabs to adhere to linux kernel coding
style.
Reported by checkpatch.
Signed-off-by: Mitali Borkar
---
drivers/staging/media/ipu3/include/intel-ipu3.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/media/ipu3/include/intel
Fixed the issue of line should not end with '[' by moving arguments
from next line to line ending with '['
Reported by checkpatch.
Signed-off-by: Mitali Borkar
---
drivers/staging/media/ipu3/include/intel-ipu3.h | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers
Reduced length of line as it was exceeding 100 characters by removing
comments from same line and adding it to previous line. This makes code
neater, and meets linux kernel coding style.
Reported by checkpatch.
Signed-off-by: Mitali Borkar
---
drivers/staging/media/ipu3/include/intel-ipu3.h | 6
Removed an unnecessary blank line to meet linux kernel coding style.
Reported by checkpatch.pl
Signed-off-by: Mitali Borkar
---
drivers/staging/media/ipu3/include/intel-ipu3.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/media/ipu3/include/intel-ipu3.h
b/drivers/staging
Added #include and replaced bit shifts by BIT() macro.
This BIT() macro from linux/bitops.h is used to define IPU3_UAPI_GRID_Y_START_EN
and IPU3_UAPI_AWB_RGBS_THR_B_* bitmask.
Use of macro is better and neater. It maintains consistency.
Reported by checkpatch.
Signed-off-by: Mitali Borkar
This patch fixes the warning identified by checkpatch.pl by replacing
__attribute__aligned(size) with __aligned(size)
Signed-off-by: Mitali Borkar
---
.../staging/media/ipu3/include/intel-ipu3.h | 74 +--
1 file changed, 37 insertions(+), 37 deletions(-)
diff --git a/drivers
:
This pacthset fixes the style issues reported by checkpatch.pl
Mitali Borkar (6):
staging: media: intel-ipu3: replace bit shifts with BIT() macro
staging: media: intel-ipu3: preferred __aligned(size) over
__attribute__aligned(size)
staging: media: intel-ipu3: remove unnecessary blank line
Corrected the misspelled word.
Reported by checkpatch.
Signed-off-by: Mitali Borkar
---
drivers/staging/vc04_services/interface/TODO | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/vc04_services/interface/TODO
b/drivers/staging/vc04_services/interface/TODO
Removed trailing whitespace to adhere to linux kernel coding style.
Reported by checkpatch.
Signed-off-by: Mitali Borkar
---
drivers/staging/vc04_services/interface/TODO | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/vc04_services/interface/TODO
b
This patch fixes whitespace and spelling mistake issue.
Mitali Borkar (2):
staging: vc04_services: remove trailing whitespace
staging: vc04_services: rectify spelling mistake
drivers/staging/vc04_services/interface/TODO | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
--
2.30.2
Removed unnecessary ftrace-like logging by simply deleting that statement
as we have other modes of logging like ftrace.
Reported by checkpatch.
Signed-off-by: Mitali Borkar
---
drivers/staging/rts5208/xd.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/staging/rts5208/xd.c b
On Fri, Apr 09, 2021 at 09:26:48AM +0200, Hans Verkuil wrote:
> Hi Mitali,
>
> Something to improve:
>
> On 08/04/2021 23:21, Mitali Borkar wrote:
> > Reduced length of a line which exceed the 100 columns limit by splitting
> > the line into two statements and commenti
On Fri, Apr 09, 2021 at 12:53:35PM +0200, Hans Verkuil wrote:
> On 09/04/2021 12:49, Mitali Borkar wrote:
> > Added '*' before every line inside long(multi-line) comments. Removed
> > '*/' from end of the comment line and added to next line as per linux
> > kernel
Removed unnecessary parentheses because they must be used only when it
is necessary or they improve readability.
Reported by checkpatch.
Signed-off-by: Mitali Borkar
---
Changes from v3:- No changes.
Changes from v2:- Rectified spelling mistake in subject description.
Changes from v1
Removed unncessary ftrace-like logging by simply deleting that statement
as we have other modes of logging like ftrace.
Reported by checkpatch.
Signed-off-by: Mitali Borkar
---
Changes from v3:- No changes.
Changes from v2:- No changes.
Changes from v1:- NO changes.
drivers/staging/rtl8192e
Removed unnecessary parentheses around '!xyz' boolean expression as '!'
has higher precedance than '||'
Signed-off-by: Mitali Borkar
---
Changes from v3:- Modified log message.
Changes from v2:- Modified subject description.
Changes from v1:- Removed unnecessary parentheses around boolean
description.
Changes from v1:-
[PATCH 1/3]:- Removed unnecessary parentheses around boolean expression.
Changes has been made in v2.
[PATCH 2/3]:- No changes.
[PATCH 3/3]:- No changes.
Mitali Borkar (3):
staging: rtl8192e: remove parentheses around boolean expression
staging: rtl8192e: remove
Removed unnecessary parentheses because they must be used only when it
is necessary or they improve readability.
Reported by checkpatch.
Signed-off-by: Mitali Borkar
---
Changes from v2:- Rectified spelling mistake in subject description.
Changes from v1:- No changes.
drivers/staging
Removed unncessary ftrace-like logging by simply deleting that statement
as we have other modes of logging like ftrace.
Reported by checkpatch.
Signed-off-by: Mitali Borkar
---
Changes from v2:- No changes.
Changes from v1:- NO changes.
drivers/staging/rtl8192e/rtl819x_HTProc.c | 4
1
Removed unnecessary parentheses around '!xyz' boolean expression as '!' has
higher
precedance than '||'
Signed-off-by: Mitali Borkar
---
Changes from v2:- Modified subject description.
Changes from v1:- Removed unnecessary parentheses around boolean expression.
Changes has been made in v2
been made in v2.
[PATCH 2/3]:- No changes.
[PATCH 3/3]:- No changes.
Mitali Borkar (3):
staging: rtl8192e: remove parentheses around boolean expression
staging: rtl8192e: remove unnecessary ftrace-like logging
staging: rtl8192e: remove unnecessary parentheses
drivers/staging/rtl8192e
Removed unnecessary parentheses because they must be used only when it
is necessary or they improve readability.
Reported by checkpatch.
Signed-off-by: Mitali Borkar
---
Changes from v1:- No changes.
drivers/staging/rtl8192e/rtl819x_HTProc.c | 12 ++--
1 file changed, 6 insertions
Removed unncessary ftrace-like logging by simply deleting that statement
as we have other modes of logging like ftrace.
Reported by checkpatch.
Signed-off-by: Mitali Borkar
---
Changes from v1:- No changes made.
drivers/staging/rtl8192e/rtl819x_HTProc.c | 4
1 file changed, 4 deletions
Removed unnecessary parentheses around '!xyz' as '!' has higher
precedance than '||'
Signed-off-by: Mitali Borkar
---
Changes from v1:- removed unnecessary parentheses around boolean
expression
drivers/staging/rtl8192e/rtl819x_HTProc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion
This patch fixes style issues
Changes from v1:-
[Patch 1/3]:- Removed unnecessary parentheses around boolean expressions
[Patch 2/3]:- No changes
[Patch 3/3]:- No changes
Mitali Borkar (3):
staging: rtl8192e: remove unnecessary parentheses
staging: rtl8192e: remove unnecessary ftrace-like
reg_name as it will contain data/registry bases to write static
const indexes declared above and will keep track of of contiguos
registers after each reg_base.
This makes code look better, neater. It improves readability.
Signed-off-by: Mitali Borkar
---
drivers/staging/media/meson/vdec/codec_mpeg12.c
Aligned the code properly by using tabs to make code neater and improve
readability.
Signed-off-by: Mitali Borkar
---
drivers/staging/media/zoran/zr36057.h | 108 +-
1 file changed, 54 insertions(+), 54 deletions(-)
diff --git a/drivers/staging/media/zoran/zr36057.h
b
Signed-off-by: Mitali Borkar
---
Changes from v2:- No changes.
Changes from v1:- NO changes.
drivers/staging/media/zoran/zr36057.h | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/staging/media/zoran/zr36057.h
b/drivers/staging/media/zoran/zr36057.h
This patch fix the alignment of code in proper manner
Changes from v2:- Aligned the code using tabs.
Changes from v1:- Aligned the code using tabs and readjusted the
comments line.
Mitali Borkar (2):
media: zoran: add spaces around '<<'
staging: media: zoran: align code appropr
Fixed pointer error by adding '*' to the function.
Signed-off-by: Mitali Borkar
Reported-by: Julia Lawall
---
Changes from v3:- added reported by tag in proper manner.
Changes from v2:- changed the patch nody to describe changes.
Changes from v1:- added pointer to the function.
drivers
Fixed pointer error by adding '*' to the function.
Reported by Julia.
Signed-off-by: Mitali Borkar
---
Changes from v2:- modified patch body but writing commit message
clearly.
Changes from v1:- added pointer to the function.
drivers/staging/rtl8192e/rtl819x_TSProc.c | 2 +-
1 file changed, 1
On Sat, Apr 10, 2021 at 03:14:24PM +0200, Greg KH wrote:
> On Sat, Apr 10, 2021 at 06:30:38PM +0530, Mitali Borkar wrote:
> > Fixed Comparison to NULL can be written as '!...' by replacing it with
> > simpler form i.e. boolean expression. This makes code more readable
> > alte
On Sat, Apr 10, 2021 at 01:56:24PM +0200, Julia Lawall wrote:
>
>
> On Sat, 10 Apr 2021, Mitali Borkar wrote:
>
> > On Fri, Apr 09, 2021 at 10:12:12PM +0200, Julia Lawall wrote:
> > >
> > >
> > > On Sat, 10 Apr 2021, Mitali Borkar wrote:
>
Fixed Comparison to NULL can be written as '!...' by replacing it with
simpler form i.e. boolean expression. This makes code more readable
alternative.
Reported by checkpatch.
Signed-off-by: Mitali Borkar
---
Changes from v1:- added pointer to the function, which was missed during
fixing v1
On Sat, Apr 10, 2021 at 02:10:38PM +0200, Greg KH wrote:
> On Sat, Apr 10, 2021 at 02:08:30PM +0200, Julia Lawall wrote:
> >
> >
> > On Sat, 10 Apr 2021, Mitali Borkar wrote:
> >
> > > Fixed Comparison to NULL can be written as '!...' by replacing it
On Fri, Apr 09, 2021 at 07:07:09PM -0700, Joe Perches wrote:
> On Sat, 2021-04-10 at 07:05 +0530, Mitali Borkar wrote:
> > Matched the alignment with open parenthesis to meet linux kernel coding
> > style.
> > Reported by checkpatch.
> []
> > diff --git a/drivers/sta
Removed unnecessary parentheses because they must be used only when it
is necessary or they improve readability.
Reported by checkpatch.
Signed-off-by: Mitali Borkar
---
drivers/staging/rtl8192e/rtl819x_HTProc.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git
These patches fix the cleanup style issues identified by checkpatch
Mitali Borkar (3):
staging: rtl8192e: replace comparison to NULL by boolean expression
staging: rtl8192e: remove unnecessary ftrace-like logging
staging: rtl8192e: remove unncessary parentheses
drivers/staging/rtl8192e
Removed unncessary ftrace-like logging by simply deleting that statement
as we have other modes of logging like ftrace.
Reported by checkpatch.
Signed-off-by: Mitali Borkar
---
drivers/staging/rtl8192e/rtl819x_HTProc.c | 4
1 file changed, 4 deletions(-)
diff --git a/drivers/staging
Replaced comparison to NULL by boolean expressions
(here used boolean negations). This improves readability of code.
Reported by checkpatch.
Signed-off-by: Mitali Borkar
---
drivers/staging/rtl8192e/rtl819x_HTProc.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git
Aligned the statements properly in one line to make code neater and to
improve readability.
Signed-off-by: Mitali Borkar
---
drivers/staging/rtl8192e/rtl819x_HTProc.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/rtl8192e/rtl819x_HTProc.c
b/drivers
Removed multiple blank lines to make code better and neater.
Reported by checkpatch.
Signed-off-by: Mitali Borkar
---
drivers/staging/rtl8192e/rtl819x_HTProc.c | 7 ---
1 file changed, 7 deletions(-)
diff --git a/drivers/staging/rtl8192e/rtl819x_HTProc.c
b/drivers/staging/rtl8192e
Matched the alignment with open parenthesis to meet linux kernel coding
style.
Reported by checkpatch.
Signed-off-by: Mitali Borkar
---
drivers/staging/rtl8192e/rtl819x_HTProc.c | 20 ++--
drivers/staging/rtl8192e/rtl819x_TSProc.c | 18 +-
2 files changed, 19
Removed an exra blank line after close brace '{' as it was not
necessary.
Reported by checkpatch
Signed-off-by: Mitali Borkar
---
drivers/staging/rtl8192e/rtl819x_HTProc.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/staging/rtl8192e/rtl819x_HTProc.c
b/drivers/staging/rtl8192e
Removed an extra blank line before close brace as it was not necessary.
Reported by checkpatch.
Signed-off-by: Mitali Borkar
---
drivers/staging/rtl8192e/rtl819x_HTProc.c | 2 --
drivers/staging/rtl8192e/rtl819x_TSProc.c | 2 --
2 files changed, 4 deletions(-)
diff --git a/drivers/staging
Added spaces around binary operators like '+', '*', '|', '-', '&',
to improve readability and to meet linux kernel coding style.
Signed-off-by: Mitali Borkar
---
drivers/staging/rtl8192e/rtl819x_HTProc.c | 8
drivers/staging/rtl8192e/rtl819x_TSProc.c | 8
2 files change
This patches fix the cleanup style issues.
Mitali Borkar (6):
staging: rtl8192e: add spaces around binary operators
staging: rtl8192e: remove unnecessary blank line before brace
staging: rtl8192e: remove unncessary blank line after brace
staging: rtl8192e: matched alignment with open
1 - 100 of 130 matches
Mail list logo