* [PATCH] Staging:comedi:cb_pcidas64=>Fixed check patch warnings
@ 2016-12-18 10:13 devendra sharma
2016-12-18 10:30 ` Greg KH
2016-12-18 17:30 ` Joe Perches
0 siblings, 2 replies; 3+ messages in thread
From: devendra sharma @ 2016-12-18 10:13 UTC (permalink / raw)
To: abbotti
Cc: hsweeten, gregkh, ansonjacob.aj, amitoj1606, devel, linux-kernel,
devendra sharma
Fixed check patch warnings about multiple line dereferencing
Signed-off-by: Devendra Sharma <devendra.sharma9091@gmail.com>
---
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 cb9c269..946c580 100644
--- 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)
if (ao_cmd_is_supported(board)) {
devpriv->ao_buffer[i] =
dma_alloc_coherent(&pcidev->dev,
- DMA_BUFFER_SIZE,
- &devpriv->
- ao_buffer_bus_addr[i],
- GFP_KERNEL);
+ DMA_BUFFER_SIZE,
+ &devpriv->ao_buffer_bus_addr[i],
+ GFP_KERNEL);
if (!devpriv->ao_buffer[i])
return -ENOMEM;
}
@@ -2476,18 +2475,15 @@ static int setup_channel_queue(struct comedi_device *dev,
for (i = 0; i < cmd->chanlist_len; i++) {
bits = 0;
/* set channel */
- bits |= adc_chan_bits(CR_CHAN(cmd->
- chanlist[i]));
+ bits |= adc_chan_bits(
+ CR_CHAN(cmd->chanlist[i]));
/* set gain */
bits |= ai_range_bits_6xxx(dev,
- CR_RANGE(cmd->
- chanlist
- [i]));
+ CR_RANGE(cmd->chanlist[i]));
/* set single-ended / differential */
bits |= se_diff_bit_6xxx(dev,
- CR_AREF(cmd->
- chanlist[i]) ==
- AREF_DIFF);
+ CR_AREF(cmd->chanlist[i]) ==
+ AREF_DIFF);
if (CR_AREF(cmd->chanlist[i]) == AREF_COMMON)
bits |= ADC_COMMON_BIT;
/* mark end of queue */
--
2.7.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] Staging:comedi:cb_pcidas64=>Fixed check patch warnings
2016-12-18 10:13 [PATCH] Staging:comedi:cb_pcidas64=>Fixed check patch warnings devendra sharma
@ 2016-12-18 10:30 ` Greg KH
2016-12-18 17:30 ` Joe Perches
1 sibling, 0 replies; 3+ messages in thread
From: Greg KH @ 2016-12-18 10:30 UTC (permalink / raw)
To: devendra sharma
Cc: abbotti, hsweeten, ansonjacob.aj, amitoj1606, devel, linux-kernel
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 <devendra.sharma9091@gmail.com>
> ---
> 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 cb9c269..946c580 100644
> --- 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)
> if (ao_cmd_is_supported(board)) {
> devpriv->ao_buffer[i] =
> dma_alloc_coherent(&pcidev->dev,
> - DMA_BUFFER_SIZE,
> - &devpriv->
> - ao_buffer_bus_addr[i],
> - GFP_KERNEL);
> + DMA_BUFFER_SIZE,
> + &devpriv->ao_buffer_bus_addr[i],
> + GFP_KERNEL);
> if (!devpriv->ao_buffer[i])
> return -ENOMEM;
> }
> @@ -2476,18 +2475,15 @@ static int setup_channel_queue(struct comedi_device *dev,
> for (i = 0; i < cmd->chanlist_len; i++) {
> bits = 0;
> /* set channel */
> - bits |= adc_chan_bits(CR_CHAN(cmd->
> - chanlist[i]));
> + bits |= adc_chan_bits(
> + CR_CHAN(cmd->chanlist[i]));
That doesn't look better to me, does it to you?
checkpatch is a guide, don't let it make you write code that isn't easy
to understand.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] Staging:comedi:cb_pcidas64=>Fixed check patch warnings
2016-12-18 10:13 [PATCH] Staging:comedi:cb_pcidas64=>Fixed check patch warnings devendra sharma
2016-12-18 10:30 ` Greg KH
@ 2016-12-18 17:30 ` Joe Perches
1 sibling, 0 replies; 3+ messages in thread
From: Joe Perches @ 2016-12-18 17:30 UTC (permalink / raw)
To: devendra sharma, abbotti
Cc: hsweeten, gregkh, ansonjacob.aj, amitoj1606, devel, linux-kernel
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(struct comedi_device *dev,
> for (i = 0; i < cmd->chanlist_len; i++) {
> bits = 0;
> /* set channel */
> - bits |= adc_chan_bits(CR_CHAN(cmd->
> - chanlist[i]));
> + bits |= adc_chan_bits(
> + CR_CHAN(cmd->chanlist[i]));
> /* set gain */
> bits |= ai_range_bits_6xxx(dev,
> - CR_RANGE(cmd->
> - chanlist
> - [i]));
> + CR_RANGE(cmd->chanlist[i]));
> /* set single-ended / differential */
> bits |= se_diff_bit_6xxx(dev,
> - CR_AREF(cmd->
> - chanlist[i]) ==
> - AREF_DIFF);
> + CR_AREF(cmd->chanlist[i]) ==
> + AREF_DIFF);
> if (CR_AREF(cmd->chanlist[i]) == AREF_COMMON)
> bits |= ADC_COMMON_BIT;
> /* mark end of queue */
This would be better handled by using a temporary like:
for (i = 0; i < cmd->chanlist_len; i++) {
<type> list = cmd->chanlist[i];
/* set channel */
bits |= adc_chan_bits(CR_CHAN(list));
/* set gain */
bits |= ai_range_bmts_6xxx(dev, CR_RANGE(list));
/* set single-ended / differential */
bits |= se_diff_bit_6xxx(dev,
CR_AREF(list) == AREF_DIFF);
if (CR_AREF(list) == AREF_COMMON)
bits |= ADC_COMMON_BIT;
Or moving the indented block into a separate function.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-12-18 17:30 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-12-18 10:13 [PATCH] Staging:comedi:cb_pcidas64=>Fixed check patch warnings devendra sharma
2016-12-18 10:30 ` Greg KH
2016-12-18 17:30 ` Joe Perches
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).