Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 1e8d13b0 authored by Vasanthakumar Thiagarajan's avatar Vasanthakumar Thiagarajan Committed by Kalle Valo
Browse files

ath6kl: Fix target assert in p2p bringup with multi vif 



Using interface 0 for p2p causes target assert. This is because
interface 0 is always initialized to non-p2p operations. Fix this
issue by initializing all the interfaces for p2p when fw is capable
of dynamic interface switching. When fw is not capable of dynamic
switching, make sure p2p is not brought up on interface which is
not initialized for this purpose.

Reported-by: default avatarNaveen Singh <navesing@qca.qualcomm.com>
Signed-off-by: default avatarVasanthakumar Thiagarajan <vthiagar@qca.qualcomm.com>
Signed-off-by: default avatarKalle Valo <kvalo@qca.qualcomm.com>
parent b514fab5

drivers/net/wireless/ath/ath6kl/cfg80211.c

+29 −0
Original line number Diff line number Diff line
@@ -1451,9 +1451,38 @@ static int ath6kl_cfg80211_change_iface(struct wiphy *wiphy, 
					struct vif_params *params)

{

	struct ath6kl_vif *vif = netdev_priv(ndev);

	int i;



	ath6kl_dbg(ATH6KL_DBG_WLAN_CFG, "%s: type %u\n", __func__, type);



	/*

	 * Don't bring up p2p on an interface which is not initialized

	 * for p2p operation where fw does not have capability to switch

	 * dynamically between non-p2p and p2p type interface.

	 */

	if (!test_bit(ATH6KL_FW_CAPABILITY_STA_P2PDEV_DUPLEX,

		      vif->ar->fw_capabilities) &&

	    (type == NL80211_IFTYPE_P2P_CLIENT ||

	     type == NL80211_IFTYPE_P2P_GO)) {

		if (vif->ar->vif_max == 1) {

			if (vif->fw_vif_idx != 0)

				return -EINVAL;

			else

				goto set_iface_type;

		}



		for (i = vif->ar->max_norm_iface; i < vif->ar->vif_max; i++) {

			if (i == vif->fw_vif_idx)

				break;

		}



		if (i == vif->ar->vif_max) {

			ath6kl_err("Invalid interface to bring up P2P\n");

			return -EINVAL;

		}

	}



set_iface_type:

	switch (type) {

	case NL80211_IFTYPE_STATION:

		vif->next_mode = INFRA_NETWORK;

drivers/net/wireless/ath/ath6kl/init.c

+18 −9
Original line number Diff line number Diff line
@@ -488,12 +488,20 @@ int ath6kl_configure_target(struct ath6kl *ar) 
		fw_mode |= fw_iftype << (i * HI_OPTION_FW_MODE_BITS);



	/*

	 * By default, submodes :

	 * Submodes when fw does not support dynamic interface

	 * switching:

	 *		vif[0] - AP/STA/IBSS

	 *		vif[1] - "P2P dev"/"P2P GO"/"P2P Client"

	 *		vif[2] - "P2P dev"/"P2P GO"/"P2P Client"

	 * Otherwise, All the interface are initialized to p2p dev.

	 */



	if (test_bit(ATH6KL_FW_CAPABILITY_STA_P2PDEV_DUPLEX,

		     ar->fw_capabilities)) {

		for (i = 0; i < ar->vif_max; i++)

			fw_submode |= HI_OPTION_FW_SUBMODE_P2PDEV <<

				(i * HI_OPTION_FW_SUBMODE_BITS);

	} else {

		for (i = 0; i < ar->max_norm_iface; i++)

			fw_submode |= HI_OPTION_FW_SUBMODE_NONE <<

				(i * HI_OPTION_FW_SUBMODE_BITS);
@@ -504,6 +512,7 @@ int ath6kl_configure_target(struct ath6kl *ar) 


		if (ar->p2p && ar->vif_max == 1)

			fw_submode = HI_OPTION_FW_SUBMODE_P2PDEV;

	}



	if (ath6kl_bmi_write_hi32(ar, hi_app_host_interest,

				  HTC_PROTOCOL_VERSION) != 0) {