Re: [patch] staging: comedi: dt2811: fix a precedence bug

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




Am 21.06.2016 15:56, schrieb Ian Abbott:
> On 21/06/16 12:46, Dan Carpenter wrote:
>> Bitwise | has higher precedence than ?: so we need to add some
>> parenthesis for this to work as intended.
>>
>> Fixes: 7c9574090d30 ('staging: comedi: dt2811: simplify A/D reference
>> configuration')
>> Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
>>
>> diff --git a/drivers/staging/comedi/drivers/dt2811.c
>> b/drivers/staging/comedi/drivers/dt2811.c
>> index 904f6377..8bbd938 100644
>> --- a/drivers/staging/comedi/drivers/dt2811.c
>> +++ b/drivers/staging/comedi/drivers/dt2811.c
>> @@ -588,8 +588,8 @@ static int dt2811_attach(struct comedi_device
>> *dev, struct comedi_devconfig *it)
>>      s = &dev->subdevices[0];
>>      s->type        = COMEDI_SUBD_AI;
>>      s->subdev_flags    = SDF_READABLE |
>> -              (it->options[2] == 1) ? SDF_DIFF :
>> -              (it->options[2] == 2) ? SDF_COMMON : SDF_GROUND;
>> +              ((it->options[2] == 1) ? SDF_DIFF :
>> +               (it->options[2] == 2) ? SDF_COMMON : SDF_GROUND);
>>      s->n_chan    = (it->options[2] == 1) ? 8 : 16;
>>      s->maxdata    = 0x0fff;
>>      s->range_table    = board->is_pgh ? &dt2811_pgh_ai_ranges
>>
> 

perhaps we can improve readability with something like that (untested):

switch(it->options[2]) {
case 1:
     s->subdev_flags= SDF_READABLE|SDF_DIFF;
     break;
case 2:
     s->subdev_flags= SDF_READABLE|SDF_COMMON;
     break;
default:
     s->subdev_flags=SDF_READABLE|SDF_GROUND;
}

so everyone has a chance to see what is going on.

just my 2 cents,

re,
 wh


> Nice catch, thanks!
> 
> Reviewed-by: Ian Abbott <abbotti@xxxxxxxxx>
> 
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux