Re: [PATCH 1/4] staging: android/ion: Fix line over 80 characters

2017-01-12 Thread Bogdan Purcareata 
On Thu, Jan 12, 2017 at 11:56 AM, Greg KH  wrote:
> On Thu, Jan 12, 2017 at 12:32:26PM +0300, Dan Carpenter wrote:
>> On Thu, Jan 12, 2017 at 09:54:16AM +0200, Bogdan Purcareata wrote:
>> > Signed-off-by: Bogdan Purcareata 
>>
>> All staging patches need a changelog, no matter how trivial.
>
> s/staging//g  :)

Will send a v2 with the changes.

Thank you!
Bogdan
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Re: [PATCH 1/4] staging: android/ion: Fix line over 80 characters

2017-01-12 Thread Greg KH 
On Thu, Jan 12, 2017 at 12:32:26PM +0300, Dan Carpenter wrote:
> On Thu, Jan 12, 2017 at 09:54:16AM +0200, Bogdan Purcareata wrote:
> > Signed-off-by: Bogdan Purcareata 
> 
> All staging patches need a changelog, no matter how trivial.

s/staging//g  :)
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Re: [PATCH 1/4] staging: android/ion: Fix line over 80 characters

2017-01-12 Thread Dan Carpenter 
On Thu, Jan 12, 2017 at 09:54:16AM +0200, Bogdan Purcareata wrote:
> Signed-off-by: Bogdan Purcareata 

All staging patches need a changelog, no matter how trivial.

> ---
>  drivers/staging/android/ion/ion-ioctl.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/staging/android/ion/ion-ioctl.c 
> b/drivers/staging/android/ion/ion-ioctl.c
> index 7e7431d..e28fffb 100644
> --- a/drivers/staging/android/ion/ion-ioctl.c
> +++ b/drivers/staging/android/ion/ion-ioctl.c
> @@ -111,7 +111,8 @@ long ion_ioctl(struct file *filp, unsigned int cmd, 
> unsigned long arg)
>   struct ion_handle *handle;
>  
>   mutex_lock(>lock);
> - handle = ion_handle_get_by_id_nolock(client, 
> data.handle.handle);
> + handle = ion_handle_get_by_id_nolock(client,
> + data.handle.handle);

Please run checkpatch.pl over your patches before sending them.  :P

regards,
dan carpenter
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[PATCH 1/4] staging: android/ion: Fix line over 80 characters

2017-01-12 Thread Bogdan Purcareata 
Signed-off-by: Bogdan Purcareata 
---
 drivers/staging/android/ion/ion-ioctl.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/staging/android/ion/ion-ioctl.c 
b/drivers/staging/android/ion/ion-ioctl.c
index 7e7431d..e28fffb 100644
--- a/drivers/staging/android/ion/ion-ioctl.c
+++ b/drivers/staging/android/ion/ion-ioctl.c
@@ -111,7 +111,8 @@ long ion_ioctl(struct file *filp, unsigned int cmd, 
unsigned long arg)
struct ion_handle *handle;
 
mutex_lock(>lock);
-   handle = ion_handle_get_by_id_nolock(client, 
data.handle.handle);
+   handle = ion_handle_get_by_id_nolock(client,
+   data.handle.handle);
if (IS_ERR(handle)) {
mutex_unlock(>lock);
return PTR_ERR(handle);
-- 
2.7.4

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel