Re: Re: [PATCH v2] staging: gasket: core: Fix a coding style issue in gasket_core.c

2020-07-15 Thread Dan Carpenter
On Wed, Jul 15, 2020 at 06:45:21PM +0800, Zhao wrote: > At 2020-07-15 17:04:06, "Joe Perches" wrote: > >On Wed, 2020-07-15 at 11:37 +0300, Dan Carpenter wrote: > >> On Wed, Jul 15, 2020 at 09:57:55AM +0200, Greg KH wrote: > >> > On Wed, Jul 15, 2020 at 12:24:22AM -0700, Joe Perches wrote: > >> >

Re:Re: [PATCH v2] staging: gasket: core: Fix a coding style issue in gasket_core.c

2020-07-15 Thread Zhao
At 2020-07-15 17:04:06, "Joe Perches" wrote: >On Wed, 2020-07-15 at 11:37 +0300, Dan Carpenter wrote: >> On Wed, Jul 15, 2020 at 09:57:55AM +0200, Greg KH wrote: >> > On Wed, Jul 15, 2020 at 12:24:22AM -0700, Joe Perches wrote: >> > > On Wed, 2020-07-15 at 09:17 +0200, Greg KH wrote: >> > > > On

Re: [PATCH v2] staging: gasket: core: Fix a coding style issue in gasket_core.c

2020-07-15 Thread Zhao
At 2020-06-18 03:10:02, "Dan Carpenter" wrote: >On Thu, Jun 18, 2020 at 12:11:27AM +0800, Zhixu Zhao wrote: >> diff --git a/drivers/staging/gasket/gasket_core.c >> b/drivers/staging/gasket/gasket_core.c >> index 67325fbaf760..28dab302183b 100644 >> --- a/drivers/staging/gasket/gasket_core.c >>

Re: [PATCH v2] staging: gasket: core: Fix a coding style issue in gasket_core.c

2020-07-15 Thread Joe Perches
On Wed, 2020-07-15 at 11:37 +0300, Dan Carpenter wrote: > On Wed, Jul 15, 2020 at 09:57:55AM +0200, Greg KH wrote: > > On Wed, Jul 15, 2020 at 12:24:22AM -0700, Joe Perches wrote: > > > On Wed, 2020-07-15 at 09:17 +0200, Greg KH wrote: > > > > On Wed, Jul 15, 2020 at 07:44:40AM +0800, Zhixu Zhao

Re: [PATCH v2] staging: gasket: core: Fix a coding style issue in gasket_core.c

2020-07-15 Thread Dan Carpenter
On Wed, Jul 15, 2020 at 09:57:55AM +0200, Greg KH wrote: > On Wed, Jul 15, 2020 at 12:24:22AM -0700, Joe Perches wrote: > > On Wed, 2020-07-15 at 09:17 +0200, Greg KH wrote: > > > On Wed, Jul 15, 2020 at 07:44:40AM +0800, Zhixu Zhao wrote: > > > > On Thu, Jun 18, 2020 at 12:11:27AM +0800, Zhixu

Re: [PATCH v2] staging: gasket: core: Fix a coding style issue in gasket_core.c

2020-07-15 Thread Greg KH
On Wed, Jul 15, 2020 at 12:24:22AM -0700, Joe Perches wrote: > On Wed, 2020-07-15 at 09:17 +0200, Greg KH wrote: > > On Wed, Jul 15, 2020 at 07:44:40AM +0800, Zhixu Zhao wrote: > > > On Thu, Jun 18, 2020 at 12:11:27AM +0800, Zhixu Zhao wrote: > > > > A coding alignment issue is found by

Re: [PATCH v2] staging: gasket: core: Fix a coding style issue in gasket_core.c

2020-07-15 Thread Joe Perches
On Wed, 2020-07-15 at 09:17 +0200, Greg KH wrote: > On Wed, Jul 15, 2020 at 07:44:40AM +0800, Zhixu Zhao wrote: > > On Thu, Jun 18, 2020 at 12:11:27AM +0800, Zhixu Zhao wrote: > > > A coding alignment issue is found by checkpatch.pl. > > > Fix it by using a temporary for

Re: [PATCH v2] staging: gasket: core: Fix a coding style issue in gasket_core.c

2020-07-15 Thread Greg KH
On Wed, Jul 15, 2020 at 07:44:40AM +0800, Zhixu Zhao wrote: > On Thu, Jun 18, 2020 at 12:11:27AM +0800, Zhixu Zhao wrote: > > A coding alignment issue is found by checkpatch.pl. > > Fix it by using a temporary for gasket_dev->bar_data[bar_num]. > > > > Signed-off-by: Zhixu Zhao > > Hi, there~ >

Re: [PATCH v2] staging: gasket: core: Fix a coding style issue in gasket_core.c

2020-07-14 Thread Zhixu Zhao
On Thu, Jun 18, 2020 at 12:11:27AM +0800, Zhixu Zhao wrote: > A coding alignment issue is found by checkpatch.pl. > Fix it by using a temporary for gasket_dev->bar_data[bar_num]. > > Signed-off-by: Zhixu Zhao Hi, there~ Does anybody have any further comments on this? Can it be merged? Zhixu

Re: [PATCH v2] staging: gasket: core: Fix a coding style issue in gasket_core.c

2020-06-18 Thread Dan Carpenter
On Thu, Jun 18, 2020 at 09:44:22PM +0800, Zhixu Zhao wrote: > At 2020-06-18 03:10:02, "Dan Carpenter" wrote: > >It would be better to do this in the declaration block so you can change > >the earlier two uses in this function: > > > >+struct gasket_bar_data *data = _dev->bar_data[bar_num]; >

Re: [PATCH v2] staging: gasket: core: Fix a coding style issue in gasket_core.c

2020-06-18 Thread Zhixu Zhao
At 2020-06-18 03:10:02, "Dan Carpenter" wrote: >It would be better to do this in the declaration block so you can change >the earlier two uses in this function: > >+ struct gasket_bar_data *data = _dev->bar_data[bar_num]; >- ulong desc_bytes =

Re: [PATCH v2] staging: gasket: core: Fix a coding style issue in gasket_core.c

2020-06-17 Thread Dan Carpenter
On Thu, Jun 18, 2020 at 12:11:27AM +0800, Zhixu Zhao wrote: > diff --git a/drivers/staging/gasket/gasket_core.c > b/drivers/staging/gasket/gasket_core.c > index 67325fbaf760..28dab302183b 100644 > --- a/drivers/staging/gasket/gasket_core.c > +++ b/drivers/staging/gasket/gasket_core.c > @@ -261,6

[PATCH v2] staging: gasket: core: Fix a coding style issue in gasket_core.c

2020-06-17 Thread Zhixu Zhao
A coding alignment issue is found by checkpatch.pl. Fix it by using a temporary for gasket_dev->bar_data[bar_num]. Signed-off-by: Zhixu Zhao --- Please ignore the last email (if you received it) because I forgot to Cc the mailing list. Sorry for the noise... Changes in v2: -