[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <cover.1456845896.git.geliangtang@163.com>
Date: Tue, 1 Mar 2016 23:35:30 +0800
From: Geliang Tang <geliangtang@....com>
To: Larry Finger <Larry.Finger@...inger.net>,
Jes Sorensen <Jes.Sorensen@...hat.com>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
Julian Calaby <julian.calaby@...il.com>
Cc: Geliang Tang <geliangtang@....com>, linux-wireless@...r.kernel.org,
devel@...verdev.osuosl.org, linux-kernel@...r.kernel.org
Subject: [PATCH v2 0/3] staging: rtl8723au: use list_first_entry*
On Mon, Feb 22, 2016 at 09:39:15PM -0500, Jes Sorensen wrote:
> Geliang Tang <geliangtang@....com> writes:
> > Use list_first_entry*() instead of container_of() to simplify the code.
> >
> > Signed-off-by: Geliang Tang <geliangtang@....com>
> > ---
> > drivers/staging/rtl8723au/core/rtw_recv.c | 49 +++++++++----------------------
> > drivers/staging/rtl8723au/core/rtw_xmit.c | 26 +++++-----------
> > 2 files changed, 22 insertions(+), 53 deletions(-)
>
> This looks fine to me. When these changes gets large, it may be better
> to break them down into multiple patches as it's easier to debug if
> there is a bug somewhere.
>
Changes in v2:
- split it into three patches.
Geliang Tang (3):
staging: rtl8723au: core: rtw_recv: use list_first_entry_or_null()
staging: rtl8723au: core: rtw_xmit: use list_first_entry_or_null()
staging: rtl8723au: core: rtw_recv: use list_first_entry()
drivers/staging/rtl8723au/core/rtw_recv.c | 47 +++++++++----------------------
drivers/staging/rtl8723au/core/rtw_xmit.c | 26 ++++++-----------
2 files changed, 21 insertions(+), 52 deletions(-)
--
2.5.0
Powered by blists - more mailing lists