Hi Paulo, On Thu, Jun 14, 2012, Paulo Alcantara wrote: > + uint8_t bdaddr_type; > > idev = g_new0(struct input_device, 1); > adapter_get_address(adapter, &idev->src); > - device_get_address(device, &idev->dst, NULL); > + device_get_address(device, &idev->dst, &bdaddr_type); I think it'd be clearer to call this dst_type (since it matches dst_addr and not src_addr). Could you also change this in all other places the variable naming occurs in your patches. Johan -- 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