Hi Hemant, On Thu, Dec 15, 2011, Hemant Gupta wrote: > @@ -218,7 +218,7 @@ struct mgmt_rp_user_confirm_reply { > #define MGMT_OP_USER_PASSKEY_REPLY 0x001C > struct mgmt_cp_user_passkey_reply { > bdaddr_t bdaddr; > - uint32_t passkey; > + __le32 passkey; > } __packed; Otherwise the patch looks ok but this change shouldn't be in it. __le32 is only for the kernel side. 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