Fix for warnings reported by checkpatch.pl:
- Multiline comment style
- Bare "unsigned"
- Missing blank line after declarations
- Un-needed braces around single-statement branch
Signed-off-by: Avraham Shukron
---
.../platform/intel-mid/atomisp_gmin_platform.
Fix for error (not warnings) reported by checkpatch.pl
Specifically:
- missing whitespace around "=" and after ","
- indentation with spaces instead of tabs
- lines starting with a whitespace
This patch does not affect the compiled code in any way.
Signed-off
Added "static" storage class to 4 not-declared functions
Signed-off-by: Avraham Shukron
---
.../media/atomisp/platform/intel-mid/atomisp_gmin_platform.c | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git
a/drivers/staging/media/atomisp/platform
Broke argument list so that it won't exceed 80 characters
Signed-off-by: Avraham Shukron
---
drivers/staging/media/omap4iss/iss_video.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/staging/media/omap4iss/iss_video.c
b/drivers/staging/media/omap4iss/iss_vi
Fixed multi-line comments to their preferred style (First line empty)
Signed-off-by: Avraham Shukron
---
drivers/staging/media/omap4iss/iss_video.c | 38 --
1 file changed, 25 insertions(+), 13 deletions(-)
diff --git a/drivers/staging/media/omap4iss/iss_video.c
b
Hi!
I submitted a patch which is now at v3 already.
In patchwork they appear separated / unrelated.
1. Is there a way to tell patchwork that they are all actually iterations
of the same patch?
2. The current status of v1, v2 is "Change Requested". v3 adds the necessary
changes.
Should I upd
Aligned asterisks in multi-line comment.
Signed-off-by: Avraham Shukron
---
drivers/staging/media/davinci_vpfe/dm365_resizer.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/media/davinci_vpfe/dm365_resizer.c
b/drivers/staging/media/davinci_vpfe
Signed-off-by: Avraham Shukron
---
drivers/staging/media/davinci_vpfe/dm365_resizer.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/media/davinci_vpfe/dm365_resizer.c
b/drivers/staging/media/davinci_vpfe/dm365_resizer.c
index 5fbc2d4..857b0e8 100644
Signed-off-by: Avraham Shukron
---
drivers/staging/media/omap4iss/iss_video.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/staging/media/omap4iss/iss_video.c
b/drivers/staging/media/omap4iss/iss_video.c
index e21811a..0bac582 100644
--- a/drivers/staging/media
Signed-off-by: Avraham Shukron
---
drivers/staging/media/omap4iss/iss_video.c | 38 --
1 file changed, 25 insertions(+), 13 deletions(-)
diff --git a/drivers/staging/media/omap4iss/iss_video.c
b/drivers/staging/media/omap4iss/iss_video.c
index bb0e3b4..e21811a
Fixes line-over-80-characters issues as well as multiline comments style.
Signed-off-by: Avraham Shukron
---
drivers/staging/media/omap4iss/iss_video.c | 41 --
1 file changed, 27 insertions(+), 14 deletions(-)
diff --git a/drivers/staging/media/omap4iss
On 29/01/2017 9:31, Ozgur Karatas wrote:
>
>
> 28.01.2017, 20:11, "Avraham Shukron" :
>> This is a patch that fixes issues in omap4iss/iss_video.c
>> Specifically, it fixes "line over 80 characters" issues
>
> Hello,
>
> are you have a sent
This is a patch that fixes checkpatch.pl issues in omap4iss/iss_video.c
Specifically, it fixes "line over 80 characters" issues
Signed-off-by: Avraham Shukron
---
drivers/staging/media/omap4iss/iss_video.c | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git
---
drivers/staging/media/omap4iss/iss_video.c | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/media/omap4iss/iss_video.c
b/drivers/staging/media/omap4iss/iss_video.c
index c16927a..cdab053 100644
--- a/drivers/staging/media/omap4iss/iss_video.c
+++ b/dr
14 matches
Mail list logo