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

== NULL) Signed-off-by: Yannis Damigos <giannis.dami...@gmail.com> --- 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 ---

[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 <giannis.dami

[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 <giannis.dami...@gmail.com> --- d

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

On Mon, Sep 26, 2016 at 1:24 PM, Greg KH <gre...@linuxfoundation.org> 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 <gre...@linuxfoundation.org> wrote: >>

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

5PM +0300, Yannis Damigos wrote: >> This patch removes the empty function rmem_ion_device_release >> >> Signed-off-by: Yannis Damigos <giannis.dami...@gmail.com> >> --- >> drivers/staging/android/ion/ion_of.c | 7 --- >> 1 file changed, 7 deletion

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

On Fri, Sep 23, 2016 at 3:17 PM, Greg KH <gre...@linuxfoundation.org> wrote: > On Fri, Sep 23, 2016 at 11:05:15AM +0300, giannis.dami...@gmail.com wrote: >> From: Yannis Damigos <giannis.dami...@gmail.com> >> >> This patch fixes the following blank line issu

[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 <giannis.dami...@gmail.com> --- 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/io

[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 <giannis.dami...@gmail.com> --- drivers/staging/android/ion/ion_of.c | 49 ++-- 1 file changed, 24 insertions(+), 25 deletions(-) diff --git a/drivers/s

[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 *noident Signed-off-by: Yannis Damigos giannis.dami...@gmail.com --- drivers/staging/rtl8723au/hal/rtl8723au_xmit.c | 4

[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 giannis.dami...@gmail.com --- drivers/staging

[PATCH 1/1] Fix some code style issues

Signed-off-by: Yannis Damigos giannis.dami...@gmail.com --- 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

[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 giannis.dami...@gmail.com --- drivers/staging/imx-drm/ipuv3-crtc.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff

[no subject]

I updated the patch to keep the alignment of the x/y/width/height parameters for readability. regards, Yannis ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[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 giannis.dami...@gmail.com --- drivers/staging/imx-drm/ipuv3-crtc.c

[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 giannis.dami...@gmail.com --- drivers/staging/imx-drm/ipuv3-crtc.c