[PATCH] Staging:rtl8188eu: Fix multiple blank lines

This patch fixes multiple black lines issue found by checkpatch.pl Signed-off-by: Yannis Damigos --- drivers/staging/rtl8188eu/core/rtw_rf.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/rtl8188eu/core/rtw_rf.c b/drivers/staging/rtl8188eu/core/rtw_rf.c index 3fc1a8f

[PATCH] staging:android:io: Fix multiple styling issues

This patch fixes 1 error, 1 warning and 14 checks found by checkpatch. Signed-off-by: Yannis Damigos --- drivers/staging/android/ion/ion_of.c | 49 ++-- 1 file changed, 24 insertions(+), 25 deletions(-) diff --git a/drivers/staging/android/ion/ion_of.c b

[PATCHv3 5/5] staging:android:ion: Remove empty release function

This patch removes the empty function rmem_ion_device_release Signed-off-by: Yannis Damigos --- drivers/staging/android/ion/ion_of.c | 7 --- 1 file changed, 7 deletions(-) diff --git a/drivers/staging/android/ion/ion_of.c b/drivers/staging/android/ion/ion_of.c index 49f052c..91c0642

Re: [PATCHv2 3/5] staging:android:ion: Fix blank line issues

On Fri, Sep 23, 2016 at 3:17 PM, Greg KH wrote: > On Fri, Sep 23, 2016 at 11:05:15AM +0300, giannis.dami...@gmail.com wrote: >> From: Yannis Damigos >> >> This patch fixes the following blank line issues: >> >> CHECK: Please don't use multiple blank lines

Re: [PATCHv3 5/5] staging:android:ion: Remove empty release function

I believed I tested it but I found out that the CONFIG_OF_RESERVED_MEM was not enabled in my config. So it was not properly tested. I will try to test it again. Sorry, Yannis On Fri, Sep 23, 2016 at 3:06 PM, Greg KH wrote: > On Fri, Sep 23, 2016 at 12:50:15PM +0300, Yannis Damigos wr

Re: [PATCHv3 5/5] staging:android:ion: Remove empty release function

Hi Greg, On Fri, Sep 23, 2016 at 3:06 PM, Greg KH wrote: > > Have you tested that this works? > In order to test the change I need a board with Hi6220 SoC. If I do not own the hardware to test the change, how should I proceed? thanks, Yannis

Re: [PATCHv3 5/5] staging:android:ion: Remove empty release function

On Mon, Sep 26, 2016 at 1:24 PM, Greg KH wrote: > On Mon, Sep 26, 2016 at 01:11:50PM +0300, Yannis Damigos wrote: >> Hi Greg, >> >> On Fri, Sep 23, 2016 at 3:06 PM, Greg KH wrote: >> > >> > Have you tested that this works? >> > >> >>

[PATCHv4 3/4] staging:android:ion: Fix blank line issues

This patch fixes the following blank line issues: CHECK: Please don't use multiple blank lines CHECK: Please use a blank line after function/struct/union/enum declarations +} +RESERVEDMEM_OF_DECLARE(ion, "ion-region", rmem_ion_setup); Signed-off-by: Yannis Damigos --- drivers/

[PATCHv4 1/4] staging:android:ion: Fix comparison with NULL

== NULL) Signed-off-by: Yannis Damigos --- drivers/staging/android/ion/ion_of.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/android/ion/ion_of.c b/drivers/staging/android/ion/ion_of.c index 826e756..a8050c6 100644 --- a/drivers/staging/android/ion/io

[PATCHv4 4/4] staging:android:ion: Fix whitespace issue

This patch fixes the following whitespace issue: CHECK: spaces preferred around that '*' (ctx:VxV) FILE: drivers/staging/android/ion/ion_of.c:91: +sizeof(struct ion_platform_heap)*num_heaps, Signed-off-by: Yannis Damigos --- drivers/staging/android/ion/io

[PATCHv4 2/4] staging:android:ion: Fix alignment issues

ze, dev); CHECK: Alignment should match open parenthesis +static void rmem_ion_device_release(struct reserved_mem *rmem, + struct device *dev) Signed-off-by: Yannis Damigos --- drivers/staging/android/ion/ion_of.c | 16 1 file changed, 8 i

[PATCH 1/1] Fix some code style issues

Signed-off-by: Yannis Damigos --- drivers/staging/imx-drm/ipuv3-crtc.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/drivers/staging/imx-drm/ipuv3-crtc.c b/drivers/staging/imx-drm/ipuv3-crtc.c index 720868b..c4c6455 100644 --- a/drivers/staging/imx-drm/ipuv3

[PATCH 1/1] staging: imx-drm: Lines over 80 characters fixed.

This is a patch to the ipuv3-crtc.c file that fixes up two "lines over 80 characters" warnings found by the checkpatch.pl tool. Signed-off-by: Yannis Damigos --- drivers/staging/imx-drm/ipuv3-crtc.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/drivers/s

[no subject]

I updated the patch to keep the alignment of the x/y/width/height parameters for readability. regards, Yannis -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-in

[PATCH 1/1] staging: imx-drm: Lines over 80 characters fixed.

This is a patch to the ipuv3-crtc.c file that fixes up two "lines over 80 characters" warnings found by the checkpatch.pl tool, keeping the alignment of the x/y/width/height parameters for readability. Signed-off-by: Yannis Damigos --- drivers/staging/imx-drm/ipuv3-crtc.c | 10 +++

[PATCH 1/1] staging: imx-drm: Lines over 80 characters fixed.

This is a patch to the ipuv3-crtc.c file that fixes up two "lines over 80 characters" warnings found by the checkpatch.pl tool, keeping the alignment of the x/y/width/height parameters for readability. Signed-off-by: Yannis Damigos --- drivers/staging/imx-drm/ipuv3-crtc.c | 9 ++-

[PATCHv2 1/1] staging: rtl8723au: Fix sparse invalid assignment warnings

/staging/rtl8723au//hal/rtl8723au_xmit.c:185:51:expected unsigned int [usertype] *pdw drivers/staging/rtl8723au//hal/rtl8723au_xmit.c:185:51:got restricted __le32 * Signed-off-by: Yannis Damigos --- drivers/staging/rtl8723au/hal/rtl8723au_xmit.c | 4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH 1/1] staging: rtl8723au: Fix sparse warnings invalid assignment |=

This is a patch to the hal/rtl8723au_xmit.c file that fixes various warnings: "invalid warning: invalid assignment: |= left side has type unsigned int right side has type restricted __le32" found by sparse tool. Signed-off-by: Yannis Damigos --- drivers/staging/rtl8723au/hal/rtl872