Fixed coding style errors. No errors with checkpatch.pl
Signed-off-by: Prasant Jalan
---
drivers/staging/vt6656/rf.c | 19 ++-
1 file changed, 10 insertions(+), 9 deletions(-)
diff --git a/drivers/staging/vt6656/rf.c b/drivers/staging/vt6656/rf.c
index 068c1c8..3a9d19a 100644
This patch fixes the following:
- Replace spaces with tabs for indentation
- adds spaces around symbol '-'
- uses __func__ macro to print function name
- truncated the line such that it is within 80 char limit
as per kernel coding standards.
Signed-off-by: Prasant Jalan
---
drive
This patch replaces spaces with tabs for indentation as per kernel
coding standards.
Signed-off-by: Prasant Jalan
---
drivers/staging/vt6656/rf.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/vt6656/rf.c b/drivers/staging/vt6656/rf.c
index
This patch replaces spaces with tabs for indentation as per kernel
coding standards.
Signed-off-by: Prasant Jalan
---
drivers/staging/vt6656/rf.c | 11 ++-
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/vt6656/rf.c b/drivers/staging/vt6656/rf.c
index
This patch replaces spaces with tabs for indentation as per kernel
coding standards.
Signed-off-by: Prasant Jalan
---
drivers/staging/vt6656/rf.c | 11 ++-
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/vt6656/rf.c b/drivers/staging/vt6656/rf.c
index
This patch replaces spaces with tabs for indentation as per kernel
coding standards.
Signed-off-by: Prasant Jalan
---
drivers/staging/vt6656/rf.c | 11 ++-
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/vt6656/rf.c b/drivers/staging/vt6656/rf.c
index
The patch replaces spaces with tabs as required by kernel coding
standards.
Signed-off-by: Prasant Jalan
---
drivers/staging/vt6656/rxtx.c | 40 ++--
1 file changed, 22 insertions(+), 18 deletions(-)
diff --git a/drivers/staging/vt6656/rxtx.c b/drivers
On Thu, Mar 23, 2017 at 1:02 PM, Greg KH wrote:
>
> On Wed, Mar 22, 2017 at 09:39:24PM +0530, Prasant Jalan wrote:
> > The patch replaces spaces with tabs as required by kernel coding
> > standards.
> >
> > Signed-off-by: Prasant Jalan
> > ---
>
Replacing spaces with tabs for indetation to fix checkpatch.pl error
"code indent should use tabs where possible".
Signed-off-by: Prasant Jalan
---
v2:
- improved changelog comments
- fixed only one issue
drivers/staging/vt6656/rxtx.c | 35 +++-
On Sat, Mar 25, 2017 at 6:01 PM, Arushi Singhal
wrote:
>
>
> On Sat, Mar 25, 2017 at 5:40 PM, Prasant Jalan
> wrote:
>>
>>
>> Replacing spaces with tabs for indetation to fix checkpatch.pl error
>> "code indent should use tabs where possible".
>
On Sat, Mar 25, 2017 at 6:28 PM, Prasant Jalan wrote:
> On Sat, Mar 25, 2017 at 6:01 PM, Arushi Singhal
> wrote:
>>
>>
>> On Sat, Mar 25, 2017 at 5:40 PM, Prasant Jalan
>> wrote:
>>>
>>>
>>> Replacing spaces with tabs for indetation to
Checkpatch emits WARNING: Avoid multiple line dereference.
Trivial indentation improvement helps fix the checkpatch warning.
Signed-off-by: Prasant Jalan
---
drivers/staging/rtl8712/rtl871x_xmit.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/staging
lynxfb_suspend() & lynxfb_resume() return on errors while holding
console_lock.
Adding 'goto' such that proper cleanups can be done before returning
from function and therefore console_lock can be released before
returning.
Signed-off-by: Prasant Jalan
---
drivers/staging/sm750f
checkpatch gives WARNING: Avoid line continuations in quoted strings.
Trivial fix by removing line continuations and adding another quote at
the start of next line.
Signed-off-by: Prasant Jalan
---
drivers/staging/sm750fb/sm750.c | 18 +-
1 file changed, 9 insertions(+), 9
On Thu, Mar 30, 2017 at 12:03 AM, Prasant Jalan wrote:
> Checkpatch emits WARNING: Avoid multiple line dereference.
>
> Trivial indentation improvement helps fix the checkpatch warning.
>
> Signed-off-by: Prasant Jalan
> ---
> drivers/staging/rtl8712/rtl871x_xmit.c | 12 +
On Mon, Apr 3, 2017 at 10:27 AM, Joe Perches wrote:
> On Mon, 2017-04-03 at 09:43 +0530, Prasant Jalan wrote:
>> On Thu, Mar 30, 2017 at 12:03 AM, Prasant Jalan
>> wrote:
>> > Checkpatch emits WARNING: Avoid multiple line dereference.
>> >
>> > Tri
Checkpatch emits WARNING: Avoid multiple line dereference.
Checkpatch warning is fixed by:
* Trivial indentation improvement and
* Using += instead of + helps shortening the statement.
Signed-off-by: Prasant Jalan
---
v2:
* Used += instead of + (pointed out by Joe Perches)
* Rebased
On Sat, Apr 1, 2017 at 12:40 AM, Prasant Jalan wrote:
> checkpatch gives WARNING: Avoid line continuations in quoted strings.
>
> Trivial fix by removing line continuations and adding another quote at
> the start of next line.
>
> Signed-off-by: Prasant Jalan
> ---
>
18 matches
Mail list logo