Hi Marcel,
> I would have put an extra comment here as well to remind us that conn is for indicating to terminate the connection or not. See my review of v4 of your patch.
>
>> if (!conn) {
>> err = cmd_complete(sk, hdev->id, MGMT_OP_UNPAIR_DEVICE, 0,
>> &rp, sizeof(rp));
Ops, I missed that one. v6 includes the comment.
Thanks,
Fons
--
Alfonso Acosta
Embedded Systems Engineer at Spotify
Birger Jarlsgatan 61, Stockholm, Sweden
http://www.spotify.com
--
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
