From: Fernando Apesteguia fernando.apesteg...@gmail.com
Add static to the definition of the pci device table.
Signed-off-by: Fernando Apesteguia fernando.apesteg...@gmail.com
---
drivers/staging/vt6655/device_main.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers
From: Fernando Apesteguia fernando.apesteg...@gmail.com
Add static to variable.
Signed-off-by: Fernando Apesteguia fernando.apesteg...@gmail.com
---
drivers/staging/vt6655/ioctl.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/vt6655/ioctl.c b/drivers
From: Fernando Apesteguia fernando.apesteg...@gmail.com
Add static to debug method.
Signed-off-by: Fernando Apesteguia fernando.apesteg...@gmail.com
---
drivers/staging/rtl8192u/r8192U_core.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8192u
From: Fernando Apesteguia fernando.apesteg...@gmail.com
Remove ComputeTxTime since it is not used.
Signed-off-by: Fernando Apesteguia fernando.apesteg...@gmail.com
---
drivers/staging/rtl8192u/r8192U_core.c | 22 --
1 file changed, 22 deletions(-)
diff --git a/drivers
Fix alignment in multi line comment block.
Remove extra '*' to use the preferred comment style as in
Documentation/CodingStyle
Signed-off-by: Fernando Apesteguia <fernando.apesteg...@gmail.com>
---
drivers/staging/dgnc/dgnc_driver.c | 50 +++---
1 file c
On Mon, Sep 05, 2016 at 11:45:30PM +0300, Andrey Utkin wrote:
> On Mon, Sep 05, 2016 at 08:28:32PM +0200, Fernando Apesteguia wrote:
> > Fix alignment in multi line comment block.
> >
> > Remove extra '*' to use the preferred comment style as in
> > Documentation/Codi
All the chunks of the patch apply to comments save the first one.
Signed-off-by: Fernando Apesteguia <fernando.apesteg...@gmail.com>
---
drivers/staging/dgnc/dgnc_neo.c | 67 -
1 file changed, 46 insertions(+), 21 deletions(-)
diff --git a/drivers/s
As reported by checkpatch.
Signed-off-by: Fernando Apesteguia <fernando.apesteg...@gmail.com>
---
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
This fixes a checkpatch warning.
Also, change the line above so it is aligned to the others in the
same block.
Signed-off-by: Fernando Apesteguia <fernando.apesteg...@gmail.com>
---
drivers/staging/dgnc/digi.h | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff
Done by either unindenting some comments or converting them to
multi line comments.
This fixes some checkpatch warnings.
Signed-off-by: Fernando Apesteguia <fernando.apesteg...@gmail.com>
---
drivers/staging/dgnc/dgnc_neo.h | 32 ++--
1 file changed, 22 inse
The patch replaces the macro with a function (dgnc_get_board) and
substitutes the macro statement with a call to that function and a
comparison on the returned value.
This removes a checkpatch warning.
Signed-off-by: Fernando Apesteguia <fernando.apesteg...@gmail.com>
---
drivers/stagin
Remove blank lines between open brace and comment.
Remove blank lines after comment in line with the rest of the comments
of the file.
Signed-off-by: Fernando Apesteguia <fernando.apesteg...@gmail.com>
---
drivers/staging/dgnc/dgnc_tty.c | 6 --
1 file changed, 6 deletions(-)
diff
For two cases (beginning and end of the patch) I opted to create small
functions instead of breaking the the lines in a weird way.
The other changes are simple ones: either by breaking the line when
appropriate or by turning a comment into a multi-line one.
Signed-off-by: Fernando Apesteguia
Remove reference to checkpatch warnings since its output is now clean.
---
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
+++
On Sat, Dec 03, 2016 at 09:51:13AM +0100, Greg KH wrote:
> On Fri, Dec 02, 2016 at 08:13:49PM +0100, Fernando Apesteguia wrote:
> > For the first lines of the patch, I opted to create a small function
> > instead of breaking the the line in a weird way.
> >
> > The ot
For the first lines of the patch, I opted to create a small function
instead of breaking the the line in a weird way.
This is v2 of the patch with the name of the function changed from
v1
The other changes are simple ones.
Signed-off-by: Fernando Apesteguia <fernando.apesteg...@gmail.
On Sat, Dec 03, 2016 at 11:11:23AM +0100, Greg KH wrote:
> On Sat, Dec 03, 2016 at 10:56:54AM +0100, Fernando Apesteguia wrote:
> > On Sat, Dec 03, 2016 at 09:51:13AM +0100, Greg KH wrote:
> > > On Fri, Dec 02, 2016 at 08:13:49PM +0100, Fernando Apesteguia wrote:
> >
This patch fixes a checkpatch warning.
Signed-off-by: Fernando Apesteguia <fernando.apesteg...@gmail.com>
---
drivers/staging/dgnc/dgnc_tty.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/dgnc/dgnc_tty.h b/drivers/staging/dgnc/dgnc_tty.h
index 8
For the first lines of the patch, I opted to create a small function
instead of breaking the the line in a weird way.
The other changes are simple ones.
Signed-off-by: Fernando Apesteguia <fernando.apesteg...@gmail.com>
---
drivers/staging/dgnc/dgnc_tty.
On Tue, Dec 06, 2016 at 10:12:56AM +0100, Greg KH wrote:
> On Sun, Dec 04, 2016 at 08:41:04PM +0100, Fernando Apesteguia wrote:
> > For the first lines of the patch, I opted to create a small function
> > instead of breaking the the line in a weird way.
> >
> &
From: Fernando Apesteguia
Add static to the definition of the pci device table.
Signed-off-by: Fernando Apesteguia
---
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
From: Fernando Apesteguia
Add static to variable.
Signed-off-by: Fernando Apesteguia
---
drivers/staging/vt6655/ioctl.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/vt6655/ioctl.c b/drivers/staging/vt6655/ioctl.c
index 65e5933..cc6e47b 100644
From: Fernando Apesteguia
Add static to debug method.
Signed-off-by: Fernando Apesteguia
---
drivers/staging/rtl8192u/r8192U_core.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8192u/r8192U_core.c
b/drivers/staging/rtl8192u/r8192U_core.c
index
From: Fernando Apesteguia
Remove ComputeTxTime since it is not used.
Signed-off-by: Fernando Apesteguia
---
drivers/staging/rtl8192u/r8192U_core.c | 22 --
1 file changed, 22 deletions(-)
diff --git a/drivers/staging/rtl8192u/r8192U_core.c
b/drivers/staging/rtl8192u
On Tue, Dec 06, 2016 at 10:12:56AM +0100, Greg KH wrote:
> On Sun, Dec 04, 2016 at 08:41:04PM +0100, Fernando Apesteguia wrote:
> > For the first lines of the patch, I opted to create a small function
> > instead of breaking the the line in a weird way.
> >
> &
For two cases (beginning and end of the patch) I opted to create small
functions instead of breaking the the lines in a weird way.
The other changes are simple ones: either by breaking the line when
appropriate or by turning a comment into a multi-line one.
Signed-off-by: Fernando Apesteguia
Remove blank lines between open brace and comment.
Remove blank lines after comment in line with the rest of the comments
of the file.
Signed-off-by: Fernando Apesteguia
---
drivers/staging/dgnc/dgnc_tty.c | 6 --
1 file changed, 6 deletions(-)
diff --git a/drivers/staging/dgnc/dgnc_tty.c
Done by either unindenting some comments or converting them to
multi line comments.
This fixes some checkpatch warnings.
Signed-off-by: Fernando Apesteguia
---
drivers/staging/dgnc/dgnc_neo.h | 32 ++--
1 file changed, 22 insertions(+), 10 deletions(-)
diff --git
This fixes a checkpatch warning.
Also, change the line above so it is aligned to the others in the
same block.
Signed-off-by: Fernando Apesteguia
---
drivers/staging/dgnc/digi.h | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/dgnc/digi.h b/drivers
The patch replaces the macro with a function (dgnc_get_board) and
substitutes the macro statement with a call to that function and a
comparison on the returned value.
This removes a checkpatch warning.
Signed-off-by: Fernando Apesteguia
---
drivers/staging/dgnc/dgnc_sysfs.c | 74
Fix alignment in multi line comment block.
Remove extra '*' to use the preferred comment style as in
Documentation/CodingStyle
Signed-off-by: Fernando Apesteguia
---
drivers/staging/dgnc/dgnc_driver.c | 50 +++---
1 file changed, 25 insertions(+), 25 deletions
On Mon, Sep 05, 2016 at 11:45:30PM +0300, Andrey Utkin wrote:
> On Mon, Sep 05, 2016 at 08:28:32PM +0200, Fernando Apesteguia wrote:
> > Fix alignment in multi line comment block.
> >
> > Remove extra '*' to use the preferred comment style as in
> > Documentation/Codi
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
All the chunks of the patch apply to comments save the first one.
Signed-off-by: Fernando Apesteguia
---
drivers/staging/dgnc/dgnc_neo.c | 67 -
1 file changed, 46 insertions(+), 21 deletions(-)
diff --git a/drivers/staging/dgnc/dgnc_neo.c b/drivers
For the first lines of the patch, I opted to create a small function
instead of breaking the the line in a weird way.
The other changes are simple ones.
Signed-off-by: Fernando Apesteguia
---
drivers/staging/dgnc/dgnc_tty.c | 42 +
1 file changed, 26
This patch fixes a checkpatch warning.
Signed-off-by: Fernando Apesteguia
---
drivers/staging/dgnc/dgnc_tty.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/dgnc/dgnc_tty.h b/drivers/staging/dgnc/dgnc_tty.h
index 85a1310..1ee0eee 100644
--- a/drivers
On Sat, Dec 03, 2016 at 09:51:13AM +0100, Greg KH wrote:
> On Fri, Dec 02, 2016 at 08:13:49PM +0100, Fernando Apesteguia wrote:
> > For the first lines of the patch, I opted to create a small function
> > instead of breaking the the line in a weird way.
> >
> > The ot
On Sat, Dec 03, 2016 at 11:11:23AM +0100, Greg KH wrote:
> On Sat, Dec 03, 2016 at 10:56:54AM +0100, Fernando Apesteguia wrote:
> > On Sat, Dec 03, 2016 at 09:51:13AM +0100, Greg KH wrote:
> > > On Fri, Dec 02, 2016 at 08:13:49PM +0100, Fernando Apesteguia wrote:
> >
For the first lines of the patch, I opted to create a small function
instead of breaking the the line in a weird way.
This is v2 of the patch with the name of the function changed from
v1
The other changes are simple ones.
Signed-off-by: Fernando Apesteguia
---
drivers/staging/dgnc/dgnc_tty.c
Remove reference to checkpatch warnings since its output is now clean.
---
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
+++
40 matches
Mail list logo
