Hi Andre, On Wed, Nov 16, 2011 at 05:36:38PM -0300, Andre Guedes wrote: ... > >> > >> if (cp->enable == 0x01) { > > > > Can we fix also magic number here and below? > > I see your point here, but this is not a magic number, it is > just a true/false value. If this true/false no need to compare with hex numbers why not if(cp->enable) ... Best regards Andrei Emeltchenko -- 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