On 09/06/2011 09:58 AM, Kalle Valo wrote:
> On 09/05/2011 05:38 PM, Jouni Malinen wrote:
>> + if (ar->p2p) {
>> + ret = ath6kl_wmi_info_req_cmd(ar->wmi,
>> + P2P_FLAG_CAPABILITIES_REQ |
>> + P2P_FLAG_MACADDR_REQ |
>> + P2P_FLAG_HMODEL_REQ);
>> + if (ret) {
>> + ath6kl_dbg(ATH6KL_DBG_TRC, "failed to request P2P "
>> + "capabilities (%d) - assuming P2P not "
>> + "supported\n", ret);
>> + ar->p2p = 0;
>> + }
>> + }
>> +
>> + if (ar->p2p) {
>
> Can we combine this if block with the first one?
Forget this comment. I was blind and didn't see 'ar->p2p = 0'.
Kalle
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
