Hi Anderson,
> Hi Szymon,
>
> On Wed, Dec 11, 2013 at 7:31 AM, Szymon Janc <szymon.janc@xxxxxxxxx> wrote:
> \> @@ -2306,11 +2309,13 @@ static uint8_t
> get_device_friendly_name(struct device *dev)
> >
> > static uint8_t get_device_rssi(struct device *dev)
> > {
> > - DBG("Not implemented");
> > + if (!dev->rssi)
> > + return HAL_STATUS_FAILED;
>
> 0 dBm is a valid RSSI value, so I suppose this code needs some other
> way to identify an uninitialized value (e.g. a second boolean field.)
Kernel sends rssi==0 if rssi info was not present on inquiry so I used this
value, but maybe it should use 127 for that.. ? (yet, this might cause trouble
as linux bluetoothd is checking 0 as well)
--
BR
Szymon Janc
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
