Date:   Fri, 22 Mar 2019 12:02:26 +0100
From:   Lorenzo Bianconi <lorenzo.bianconi@redhat.com>
To:     Stanislaw Gruszka <sgruszka@redhat.com>
Cc:     Felix Fietkau <nbd@nbd.name>, linux-wireless@vger.kernel.org
Subject: Re: [PATCH 00/12] mt76usb: some cleanups and optimizations
Message-ID: <20190322110226.GB22889@localhost.localdomain>
References: <20190321152537.19105-1-sgruszka@redhat.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
        protocol="application/pgp-signature"; boundary="/NkBOFFp2J2Af1nK"
Content-Disposition: inline
In-Reply-To: <20190321152537.19105-1-sgruszka@redhat.com>
User-Agent: Mutt/1.11.3 (2019-02-01)
Sender: linux-wireless-owner@vger.kernel.org
Precedence: bulk


--/NkBOFFp2J2Af1nK
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

> Significant change is remove mt76u_buf and use urb directly
> and allocate urb and sg as linear data for better cache usage.
> Other that that, set consist only of some minor cleanups.
>=20
> RFC -> v1:
> - introduce mt76u_tx_setup_buffers
> - calculate data_len differently
> - fix *usb->sg bug on intermediate patches
> - use (struct scatterlist *)(e->urb + 1) trick
>=20
> Rebased on latest tree and=20
> [PATCH v2 00/12] mt76x02: AP support for USB with PS
> https://lore.kernel.org/linux-wireless/1552991867-5087-1-git-send-email-s=
gruszka@redhat.com/
>  =20
> Not rebased on
> [PATCH 1/6] mt76: use mac80211 txq scheduling
> https://lore.kernel.org/linux-wireless/20190316204242.73560-1-nbd@nbd.nam=
e/
> It easer to rebase '[PATCH 5/6] mt76: move tx tasklet to struct mt76_dev'
> from that set of top of my 2 pending sets.=20

LGTM

Acked-by: Lorenzo Bianconi <lorenzo@kernel.org>

>=20
> Stanislaw Gruszka (12):
>   mt76usb: change mt76u_submit_buf
>   mt76: remove rx_page_lock
>   mt76usb: change mt76u_fill_rx_sg arguments
>   mt76usb: use usb_dev private data
>   mt76usb: remove mt76u_buf redundant fileds
>   mt76usb: move mt76u_buf->done to queue entry
>   mt76usb: remove mt76u_buf and use urb directly
>   mt76usb: remove MT_RXQ_MAIN queue from mt76u_urb_alloc
>   mt76usb: resue mt76u_urb_alloc for tx
>   mt76usb: remove unneded sg_init_table
>   mt76usb: allocate urb and sg as linear data
>   mt76usb: remove queue variable from rx_tasklet
>=20
>  drivers/net/wireless/mediatek/mt76/mt76.h     |  15 +-
>  .../net/wireless/mediatek/mt76/mt76x0/usb.c   |   2 +-
>  .../net/wireless/mediatek/mt76/mt76x2/usb.c   |   4 +-
>  drivers/net/wireless/mediatek/mt76/usb.c      | 243 ++++++++----------
>  4 files changed, 118 insertions(+), 146 deletions(-)
>=20
> --=20
> 2.20.1
>=20

--/NkBOFFp2J2Af1nK
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iHUEABYIAB0WIQTquNwa3Txd3rGGn7Y6cBh0uS2trAUCXJTAvgAKCRA6cBh0uS2t
rKT0AQD+G40GFyJ0qyDcO6m27Yl/CeM6kLhgTyghuDaKxV7exwEAo5GeELz/jNB0
VEz6FPwvOV+oIvZ3XNye+syp5SCEbAg=
=QzUz
-----END PGP SIGNATURE-----

--/NkBOFFp2J2Af1nK--