Hi Johan,
On Thu, Dec 15, 2011 at 02:08:51PM +0200, Johan Hedberg wrote:
> 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.
Minor comment related to debugging in general. We always allocate space
for bluetooth address string even when we do not use debug.
Best regards
Andrei Emeltchenko
>
> 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
--
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
