mbox series

[0/6] mt76: move connac txp code in shared module

Message ID cover.1655649421.git.lorenzo@kernel.org
Headers show
Series mt76: move connac txp code in shared module | expand

Message

Lorenzo Bianconi June 19, 2022, 8:42 p.m. UTC
Move fw/hw txp code in mt76-connac module since it is shared between mt7915e,
mt7921e and mt7615 drivers. This is a preliminary series to add mt7990e chipset
support.

Lorenzo Bianconi (6):
  mt76: connac: move mt76_connac_fw_txp in common module
  mt76: move mt7615_txp_ptr in mt76_connac module
  mt76: connac: move mt76_connac_tx_free in shared code
  mt76: connac: move mt76_connac_tx_complete_skb in shared code
  mt76: connac: move mt76_connac_write_hw_txp in shared code
  mt76: connac: move mt7615_txp_skb_unmap in common code

 .../net/wireless/mediatek/mt76/mt7615/mac.c   |  63 +--------
 .../net/wireless/mediatek/mt76/mt7615/mac.h   |  69 ----------
 .../net/wireless/mediatek/mt76/mt7615/mmio.c  |   4 +-
 .../wireless/mediatek/mt76/mt7615/mt7615.h    |   3 -
 .../wireless/mediatek/mt76/mt7615/pci_mac.c   |  78 +----------
 .../net/wireless/mediatek/mt76/mt76_connac.h  |  91 ++++++++++++
 .../wireless/mediatek/mt76/mt76_connac2_mac.h |  13 +-
 .../wireless/mediatek/mt76/mt76_connac_mac.c  | 130 ++++++++++++++++++
 .../net/wireless/mediatek/mt76/mt7915/mac.c   |  41 +-----
 .../net/wireless/mediatek/mt76/mt7915/mac.h   |  32 -----
 .../net/wireless/mediatek/mt76/mt7915/mmio.c  |   4 +-
 .../wireless/mediatek/mt76/mt7915/mt7915.h    |   1 -
 .../net/wireless/mediatek/mt76/mt7921/mac.h   |  68 ---------
 .../wireless/mediatek/mt76/mt7921/mt7921.h    |   1 -
 .../net/wireless/mediatek/mt76/mt7921/pci.c   |   4 +-
 .../wireless/mediatek/mt76/mt7921/pci_mac.c   | 102 ++------------
 16 files changed, 257 insertions(+), 447 deletions(-)

Comments

Lorenzo Bianconi June 22, 2022, 12:11 p.m. UTC | #1
[...]

> diff --git a/drivers/net/wireless/mediatek/mt76/mt7921/pci.c b/drivers/net/wireless/mediatek/mt76/mt7921/pci.c
> index 624eb75c15cd..dde7709edc6c 100644
> --- a/drivers/net/wireless/mediatek/mt76/mt7921/pci.c
> +++ b/drivers/net/wireless/mediatek/mt76/mt7921/pci.c
> @@ -230,7 +230,7 @@ static int mt7921_pci_probe(struct pci_dev *pdev,
>  {
>  	static const struct mt76_driver_ops drv_ops = {
>  		/* txwi_size = txd size + txp size */
> -		.txwi_size = MT_TXD_SIZE + sizeof(struct mt7921_txp_common),
> +		.txwi_size = MT_TXD_SIZE + sizeof(struct mt76_connac_txp_common),

I spotted a minor issue in this patch, we should use mt76_connac_hw_txp here.
@Felix: please drop this patch, I will post v2 soon.

Regards,
Lorenzo

>  		.drv_flags = MT_DRV_TXWI_NO_FREE | MT_DRV_HW_MGMT_TXQ,
>  		.survey_flags = SURVEY_INFO_TIME_TX |
>  				SURVEY_INFO_TIME_RX |
> diff --git a/drivers/net/wireless/mediatek/mt76/mt7921/pci_mac.c b/drivers/net/wireless/mediatek/mt76/mt7921/pci_mac.c
> index 9608b5ae8820..8e051bdae673 100644
> --- a/drivers/net/wireless/mediatek/mt76/mt7921/pci_mac.c
> +++ b/drivers/net/wireless/mediatek/mt76/mt7921/pci_mac.c
> @@ -9,8 +9,8 @@ static void
>  mt7921_write_hw_txp(struct mt7921_dev *dev, struct mt76_tx_info *tx_info,
>  		    void *txp_ptr, u32 id)
>  {
> -	struct mt7921_hw_txp *txp = txp_ptr;
> -	struct mt7921_txp_ptr *ptr = &txp->ptr[0];
> +	struct mt76_connac_hw_txp *txp = txp_ptr;
> +	struct mt76_connac_txp_ptr *ptr = &txp->ptr[0];
>  	int i, nbuf = tx_info->nbuf - 1;
>  
>  	tx_info->buf[0].len = MT_TXD_SIZE + sizeof(*txp);
> @@ -44,8 +44,8 @@ int mt7921e_tx_prepare_skb(struct mt76_dev *mdev, void *txwi_ptr,
>  	struct mt7921_dev *dev = container_of(mdev, struct mt7921_dev, mt76);
>  	struct ieee80211_tx_info *info = IEEE80211_SKB_CB(tx_info->skb);
>  	struct ieee80211_key_conf *key = info->control.hw_key;
> +	struct mt76_connac_txp_common *txp;
>  	struct mt76_txwi_cache *t;
> -	struct mt7921_txp_common *txp;
>  	int id, pid;
>  	u8 *txwi = (u8 *)txwi_ptr;
>  
> @@ -75,8 +75,8 @@ int mt7921e_tx_prepare_skb(struct mt76_dev *mdev, void *txwi_ptr,
>  	mt76_connac2_mac_write_txwi(mdev, txwi_ptr, tx_info->skb, wcid, key,
>  				    pid, 0);
>  
> -	txp = (struct mt7921_txp_common *)(txwi + MT_TXD_SIZE);
> -	memset(txp, 0, sizeof(struct mt7921_txp_common));
> +	txp = (struct mt76_connac_txp_common *)(txwi + MT_TXD_SIZE);
> +	memset(txp, 0, sizeof(struct mt76_connac_txp_common));
>  	mt7921_write_hw_txp(dev, tx_info, txp, id);
>  
>  	tx_info->skb = DMA_DUMMY_DATA;
> @@ -87,13 +87,13 @@ int mt7921e_tx_prepare_skb(struct mt76_dev *mdev, void *txwi_ptr,
>  static void
>  mt7921_txp_skb_unmap(struct mt76_dev *dev, struct mt76_txwi_cache *t)
>  {
> -	struct mt7921_txp_common *txp;
> +	struct mt76_connac_txp_common *txp;
>  	int i;
>  
>  	txp = mt76_connac_txwi_to_txp(dev, t);
>  
>  	for (i = 0; i < ARRAY_SIZE(txp->hw.ptr); i++) {
> -		struct mt7921_txp_ptr *ptr = &txp->hw.ptr[i];
> +		struct mt76_connac_txp_ptr *ptr = &txp->hw.ptr[i];
>  		bool last;
>  		u16 len;
>  
> @@ -271,8 +271,8 @@ void mt7921e_tx_complete_skb(struct mt76_dev *mdev, struct mt76_queue_entry *e)
>  
>  	/* error path */
>  	if (e->skb == DMA_DUMMY_DATA) {
> +		struct mt76_connac_txp_common *txp;
>  		struct mt76_txwi_cache *t;
> -		struct mt7921_txp_common *txp;
>  		u16 token;
>  
>  		txp = mt76_connac_txwi_to_txp(mdev, e->txwi);
> -- 
> 2.36.1
>