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

Commit bc9269b8 authored by Ahmad Masri's avatar Ahmad Masri
Browse files

wil6210: accessing 802.3 addresses via utility functions 



Rearrange the code by having functions to access 802.3 header
members, source and destination addresses.

Signed-off-by: default avatarAhmad Masri <amasri@codeaurora.org>
Signed-off-by: default avatarMaya Erez <merez@codeaurora.org>
Signed-off-by: default avatarKalle Valo <kvalo@codeaurora.org>
Git-commit: 6d1ba32c8070bf9e4088b16d3af1eb2b6f05543c
Git-repo: git://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git


[merez@codeaurora.org: SPDX license conflict]
Change-Id: Ib5417ce5ba77e5f96cb3b4d5d9cd322e78f39806
Signed-off-by: default avatarMaya Erez <merez@codeaurora.org>
parent ff7ca53d

drivers/net/wireless/ath/wil6210/txrx.c

+17 −17
Original line number Diff line number Diff line
@@ -750,11 +750,11 @@ void wil_netif_rx_any(struct sk_buff *skb, struct net_device *ndev) 
	unsigned int len = skb->len;

	int cid;

	int security;

	struct ethhdr *eth = (void *)skb->data;

	u8 *sa, *da = wil_skb_get_da(skb);

	/* here looking for DA, not A1, thus Rxdesc's 'mcast' indication

	 * is not suitable, need to look at data

	 */

	int mcast = is_multicast_ether_addr(eth->h_dest);

	int mcast = is_multicast_ether_addr(da);

	struct wil_net_stats *stats;

	struct sk_buff *xmit_skb = NULL;

	static const char * const gro_res_str[] = {
@@ -798,7 +798,8 @@ void wil_netif_rx_any(struct sk_buff *skb, struct net_device *ndev) 
	}



	if (wdev->iftype == NL80211_IFTYPE_STATION) {

		if (mcast && ether_addr_equal(eth->h_source, ndev->dev_addr)) {

		sa = wil_skb_get_sa(skb);

		if (mcast && ether_addr_equal(sa, ndev->dev_addr)) {

			/* mcast packet looped back to us */

			rc = GRO_DROP;

			dev_kfree_skb(skb);
@@ -813,8 +814,7 @@ void wil_netif_rx_any(struct sk_buff *skb, struct net_device *ndev) 
			 */

			xmit_skb = skb_copy(skb, GFP_ATOMIC);

		} else {

			int xmit_cid = wil_find_cid(wil, vif->mid,

						    eth->h_dest);

			int xmit_cid = wil_find_cid(wil, vif->mid, da);



			if (xmit_cid >= 0) {

				/* The destination station is associated to
@@ -1258,12 +1258,13 @@ static struct wil_ring *wil_find_tx_ucast(struct wil6210_priv *wil, 
					  struct wil6210_vif *vif,

					  struct sk_buff *skb)

{

	int i;

	struct ethhdr *eth = (void *)skb->data;

	int cid = wil_find_cid(wil, vif->mid, eth->h_dest);

	int i, cid;

	const u8 *da = wil_skb_get_da(skb);

	int min_ring_id = wil_get_min_tx_ring_id(wil);



	if (cid < 0)

	cid = wil_find_cid(wil, vif->mid, da);



	if (cid < 0 || cid >= WIL6210_MAX_CID)

		return NULL;



	/* TODO: fix for multiple TID */
@@ -1276,7 +1277,7 @@ static struct wil_ring *wil_find_tx_ucast(struct wil6210_priv *wil, 
			struct wil_ring_tx_data *txdata = &wil->ring_tx_data[i];



			wil_dbg_txrx(wil, "find_tx_ucast: (%pM) -> [%d]\n",

				     eth->h_dest, i);

				     da, i);

			if (v->va && txdata->enabled) {

				return v;

			} else {
@@ -1367,10 +1368,10 @@ static struct wil_ring *wil_find_tx_bcast_1(struct wil6210_priv *wil, 
static void wil_set_da_for_vring(struct wil6210_priv *wil,

				 struct sk_buff *skb, int vring_index)

{

	struct ethhdr *eth = (void *)skb->data;

	u8 *da = wil_skb_get_da(skb);

	int cid = wil->ring2cid_tid[vring_index][0];



	ether_addr_copy(eth->h_dest, wil->sta[cid].addr);

	ether_addr_copy(da, wil->sta[cid].addr);

}



static struct wil_ring *wil_find_tx_bcast_2(struct wil6210_priv *wil,
@@ -1381,8 +1382,7 @@ static struct wil_ring *wil_find_tx_bcast_2(struct wil6210_priv *wil, 
	struct sk_buff *skb2;

	int i;

	u8 cid;

	struct ethhdr *eth = (void *)skb->data;

	char *src = eth->h_source;

	const u8 *src = wil_skb_get_sa(skb);

	struct wil_ring_tx_data *txdata, *txdata2;

	int min_ring_id = wil_get_min_tx_ring_id(wil);
@@ -2136,8 +2136,8 @@ netdev_tx_t wil_start_xmit(struct sk_buff *skb, struct net_device *ndev) 
{

	struct wil6210_vif *vif = ndev_to_vif(ndev);

	struct wil6210_priv *wil = vif_to_wil(vif);

	struct ethhdr *eth = (void *)skb->data;

	bool bcast = is_multicast_ether_addr(eth->h_dest);

	const u8 *da = wil_skb_get_da(skb);

	bool bcast = is_multicast_ether_addr(da);

	struct wil_ring *ring;

	static bool pr_once_fw;

	int rc;
@@ -2184,7 +2184,7 @@ netdev_tx_t wil_start_xmit(struct sk_buff *skb, struct net_device *ndev) 
		ring = wil_find_tx_ucast(wil, vif, skb);

	}

	if (unlikely(!ring)) {

		wil_dbg_txrx(wil, "No Tx RING found for %pM\n", eth->h_dest);

		wil_dbg_txrx(wil, "No Tx RING found for %pM\n", da);

		goto drop;

	}

	/* set up vring entry */

drivers/net/wireless/ath/wil6210/txrx.h

+16 −2
Original line number Diff line number Diff line
@@ -567,11 +567,25 @@ static inline __be16 wil_skb_get_protocol(struct sk_buff *skb) 
	return eth->h_proto;

}



static inline bool wil_need_txstat(struct sk_buff *skb)

static inline u8 *wil_skb_get_da(struct sk_buff *skb)

{

	struct ethhdr *eth = (void *)skb->data;



	return eth->h_dest;

}



static inline u8 *wil_skb_get_sa(struct sk_buff *skb)

{

	struct ethhdr *eth = (void *)skb->data;



	return is_unicast_ether_addr(eth->h_dest) && skb->sk &&

	return eth->h_source;

}



static inline bool wil_need_txstat(struct sk_buff *skb)

{

	const u8 *da = wil_skb_get_da(skb);



	return is_unicast_ether_addr(da) && skb->sk &&

	       (skb_shinfo(skb)->tx_flags & SKBTX_WIFI_STATUS);

}