Hi Frédéric, On Wed, Jan 30, 2013, Frédéric Dalleau wrote: > + switch (mode) { > + case 0: > + cp.max_latency = __constant_cpu_to_le16(0xffff); > + cp.content_format = cpu_to_le16(hdev->voice_setting); > + cp.retrans_effort = 0xff; > + break; > + case 1: > + if (conn->pkt_type & ESCO_2EV3) > + cp.max_latency = __constant_cpu_to_le16(0x0008); > + else > + cp.max_latency = __constant_cpu_to_le16(0x000D); > + cp.content_format = __constant_cpu_to_le16(0x0003); > + cp.retrans_effort = 0x02; > + break; > + } Why are you using 0 and 1 here instead of your nice defines? 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