Re: [PATCH] staging: android: ion: fix coding style issue

2017-04-26 Thread adheer chandravanshi
On Wed, Apr 26, 2017 at 12:26 AM, Greg KH wrote: > On Sun, Apr 23, 2017 at 03:37:29PM +0530, Adheer Chandravanshi wrote: >> Use tab instead of spaces for indentation, as reported by checkpatch.pl >> >> Signed-off-by: Adheer Chandravanshi >> --- >> drivers/staging/android/ion/ion_cma_heap.c | 6 +

Re: [PATCH] staging: android: ion: fix coding style issue

2017-04-25 Thread Greg KH
On Sun, Apr 23, 2017 at 03:37:29PM +0530, Adheer Chandravanshi wrote: > Use tab instead of spaces for indentation, as reported by checkpatch.pl > > Signed-off-by: Adheer Chandravanshi > --- > drivers/staging/android/ion/ion_cma_heap.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-)

Re: [PATCH] staging: android/ion: Fix coding style issues

2017-01-11 Thread Greg KH
On Wed, Jan 11, 2017 at 11:20:39PM +0200, Bogdan Purcareata wrote: > Fix following checkpatch warnings: > - Lines over 80 characters > - void function with return statement > - Unaligned comment mark > - Header function prototypes missing variable names That's a lot of different things to be doing

Re: [PATCH] staging: android/ion: Fix coding style issues

2017-01-10 Thread Joe Perches
On Tue, 2017-01-10 at 23:40 +0200, Bogdan Purcareata wrote: > Fix following checkpatch warnings: > - Lines over 80 characters > - void function with return statement > - Unaligned comment mark > - Header function prototypes missing variable names Please read Documentation/SubmittingPatches (umm, m

Re: [PATCH] staging: android/ion: Fix coding style issues

2017-01-10 Thread Greg KH
On Tue, Jan 10, 2017 at 11:40:30PM +0200, Bogdan Purcareata wrote: > Fix following checkpatch warnings: > - Lines over 80 characters > - void function with return statement > - Unaligned comment mark > - Header function prototypes missing variable names > > Sent against staging-next. > > Signed-o

Re: [PATCH] Staging: android: ion: fix coding style

2014-09-14 Thread Greg KH
First off, thanks for the patches. But, they need some tweaks before I can accept them. You sent a bunch of patches, all with the same exact subject, with no hint of which order they should be applied in, or what exactly they do. On Sat, Sep 13, 2014 at 04:46:10PM -0700, Sorin Facaoaru wrote: >