Re: [PATCH v2] fix code alignment with open parenthesis

2016-12-18 Thread Greg KH
On Sun, Dec 18, 2016 at 11:47:30AM -0600, Scott Matheina wrote: > These changes where identified by checkpatch.pl as needed changes to > align the code with the linux development coding style. The several > lines of text where aligned with the precending parenthesis. > > Signed-off-by: Scott Mathe

RE: [upstream-release] [PATCH v5 5/8] bus: fsl-mc: dpio: add QBMan portal APIs for DPAA2

2016-12-18 Thread Bharat Bhushan
> -Original Message- > From: upstream-release-boun...@linux.freescale.net [mailto:upstream- > release-boun...@linux.freescale.net] On Behalf Of Stuart Yoder > Sent: Friday, December 16, 2016 10:01 PM > To: gre...@linuxfoundation.org > Cc: de...@driverdev.osuosl.org; ag...@suse.de; Ruxandr

[PATCH v2] staging: emxx_udc: Fix styling issues

2016-12-18 Thread Afonso Bordado
Fix CamelCase styling issues with EP0_out_PIO Remove unecessary temporary variables Signed-off-by: Afonso Bordado --- drivers/staging/emxx_udc/emxx_udc.c | 29 - 1 file changed, 12 insertions(+), 17 deletions(-) diff --git a/drivers/staging/emxx_udc/emxx_udc.c b/dri

Re: [PATCH] fix code alignment with open parenthesis in drivers/staging/fbtft/fb_agm1264k-fl.c

2016-12-18 Thread Scott Matheina
I just read an email from Linus saying keep the line length shorter than 100, so my patch isn't necessary. I sent the v2 out today, but wasn't aware of 100 being the new limit checkpatch.pl still has 80 as the limit. Sorry to bother. Please disregard my patch. On 12/18/2016 11:43 AM, Scott Mat

Re: [PATCH] staging: emxx_udc: Fix CamelCase styling issue

2016-12-18 Thread Afonso Bordado
> Instead of merely converting Hungarian CamelCase to lowercase > with underscores where word transitions occurred, try reading > the code and making sense of what it does to perhaps find a > better variable name instead. Maybe eliminate the variable > altogether. > > iWordLength could be reads or

Re: [PATCH] Staging:comedi:cb_pcidas64=>Fixed check patch warnings

2016-12-18 Thread Joe Perches
On Sun, 2016-12-18 at 15:43 +0530, devendra sharma wrote: > Fixed check patch warnings about multiple line dereferencing []. > diff --git a/drivers/staging/comedi/drivers/cb_pcidas64.c > b/drivers/staging/comedi/drivers/cb_pcidas64.c [] > @@ -2476,18 +2475,15 @@ static int setup_channel_queue(stru

Re: [PATCH] staging: emxx_udc: Fix CamelCase styling issue

2016-12-18 Thread Joe Perches
On Sun, 2016-12-18 at 16:46 +, Afonso Bordado wrote: > Converts from CamelCase to the recommended style. > > Signed-off-by: Afonso Bordado > --- > drivers/staging/emxx_udc/emxx_udc.c | 18 +- > 1 file changed, 9 insertions(+), 9 deletions(-) > > diff --git a/drivers/staging/

[PATCH v2] fix code alignment with open parenthesis

2016-12-18 Thread Scott Matheina
These changes where identified by checkpatch.pl as needed changes to align the code with the linux development coding style. The several lines of text where aligned with the precending parenthesis. Signed-off-by: Scott Matheina Changes to be committed: modified: drivers/staging/fbtft/

Re: [PATCH] fix code alignment with open parenthesis in drivers/staging/fbtft/fb_agm1264k-fl.c

2016-12-18 Thread Scott Matheina
On 12/18/2016 01:46 AM, Greg KH wrote: On Sat, Dec 17, 2016 at 08:42:53PM -0600, Scott Matheina wrote: Signed-off-by: Scott Matheina I can't take patches without any changelog text :( Yah. Forgot to add the description. Added for v2. --- drivers/staging/fbtft/fb_agm1264k-fl.c | 17 ++

[PATCH] staging: emxx_udc: Fix CamelCase styling issue

2016-12-18 Thread Afonso Bordado
Converts from CamelCase to the recommended style. Signed-off-by: Afonso Bordado --- drivers/staging/emxx_udc/emxx_udc.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/staging/emxx_udc/emxx_udc.c b/drivers/staging/emxx_udc/emxx_udc.c index 3f42fa8..

Re: [PATCH] Staging:comedi:cb_pcidas64=>Fixed check patch warnings

2016-12-18 Thread Greg KH
On Sun, Dec 18, 2016 at 03:43:15PM +0530, devendra sharma wrote: > Fixed check patch warnings about multiple line dereferencing What does that mean? > > Signed-off-by: Devendra Sharma > --- > drivers/staging/comedi/drivers/cb_pcidas64.c | 20 > 1 file changed, 8 insertions

[PATCH] Staging:comedi:cb_pcidas64=>Fixed check patch warnings

2016-12-18 Thread devendra sharma
Fixed check patch warnings about multiple line dereferencing Signed-off-by: Devendra Sharma --- drivers/staging/comedi/drivers/cb_pcidas64.c | 20 1 file changed, 8 insertions(+), 12 deletions(-) diff --git a/drivers/staging/comedi/drivers/cb_pcidas64.c b/drivers/staging/c

Re: [PATCH] Staging:comedi:cb_pcidas64=>Fixed checkpatch warnings

2016-12-18 Thread Greg KH
On Sun, Dec 18, 2016 at 03:06:18PM +0530, Devendra Sharma wrote: > From: devendra sharma > > Signed-off-by Devendra Sharma I can't take patches without any changelog text :( ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdr

Re: [PATCH] Staging:comedi:cb_pcidas64=>Fixed checkpatch warnings

2016-12-18 Thread Joe Perches
On Sun, 2016-12-18 at 15:06 +0530, Devendra Sharma wrote: > From: devendra sharma [] > diff --git a/drivers/staging/comedi/drivers/cb_pcidas64.c > b/drivers/staging/comedi/drivers/cb_pcidas64.c [] > @@ -1500,10 +1500,9 @@ static int alloc_and_init_dma_members(struct > comedi_device *dev) >

[PATCH] Staging:comedi:cb_pcidas64=>Fixed checkpatch warnings

2016-12-18 Thread Devendra Sharma
From: devendra sharma Signed-off-by Devendra Sharma --- drivers/staging/comedi/drivers/cb_pcidas64.c | 20 1 file changed, 8 insertions(+), 12 deletions(-) diff --git a/drivers/staging/comedi/drivers/cb_pcidas64.c b/drivers/staging/comedi/drivers/cb_pcidas64.c index cb9c2

Re: [PATCH] vmbus: expose numa_node via sysfs

2016-12-18 Thread Greg KH
On Fri, Dec 16, 2016 at 02:54:48PM -0800, Stephen Hemminger wrote: > From: Stephen Hemminger > > Provide numa node information via sysfs for devices on Hyper-V > (same as PCI). > This is for some NUMA aware applications (like DPDK). > > Signed-off-by: Stephen Hemminger > --- > drivers/hv/vmbus