Hi Marcel, On Thu, Mar 1, 2012 at 6:32 PM, Marcel Holtmann <marcel@xxxxxxxxxxxx> wrote: > The MGMT and SMP timeout constants are always used in form of jiffies. So > just include the conversion from msecs in the define itself. This has the > advantage of making the code where the timeout is used more readable. > > Signed-off-by: Marcel Holtmann <marcel@xxxxxxxxxxxx> > --- > net/bluetooth/mgmt.c | 5 ++--- > net/bluetooth/smp.c | 5 ++--- > 2 files changed, 4 insertions(+), 6 deletions(-) > > diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c > index 7bd7d57..40b3da3 100644 > --- a/net/bluetooth/mgmt.c > +++ b/net/bluetooth/mgmt.c > @@ -116,7 +116,7 @@ static const u16 mgmt_events[] = { > #define INQUIRY_LEN_BREDR 0x08 /* TGAP(100) */ > #define INQUIRY_LEN_BREDR_LE 0x04 /* TGAP(100)/2 */ > > -#define SERVICE_CACHE_TIMEOUT (5 * 1000) > +#define CACHE_TIMEOUT msecs_to_jiffies(2 * 1000) Was it intentional to change timeout from 5 seconds to 2 seconds on this commit? Regards, -- Anderson Lizardo Instituto Nokia de Tecnologia - INdT Manaus - Brazil -- 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