Re: drivers/bluetooth/dtl1_cs.c:559: bad if statement ?

[Marcel Holtmann]

Hi David,

> drivers/bluetooth/dtl1_cs.c:559:32: warning: logical ‘or’ of collectively 
> exhaustive tests is always true [-Wlogical-op]
> 
> Source code is
> 
>    if ((p_dev->resource[1]->end) || (p_dev->resource[1]->end < 8))
>        return -ENODEV;
> 
> Maybe better code
> 
>    if ((p_dev->resource[1]->end < 0) || (p_dev->resource[1]->end >= 8))
>        return -ENODEV;

mind sending a patch?

Regards

Marcel