On Fri, Jun 29, 2018 at 07:10:05PM +0100, John Whitmore wrote: > Chnaged logging statements to use %s and __func__ instead of hard coding the > function name in a string. > > Signed-off-by: John Whitmore <johnfwhitmore@xxxxxxxxx> > --- > drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c | 12 ++++++------ > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c b/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c > index 1dd4c6ae7319..a4fbc0435de5 100644 > --- a/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c > +++ b/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c > @@ -534,7 +534,7 @@ void HTConstructCapabilityElement(struct ieee80211_device *ieee, u8 *posHTCap, u > //u8 bIsDeclareMCS13; > > if (!posHTCap || !pHT) { > - IEEE80211_DEBUG(IEEE80211_DL_ERR, "posHTCap or pHTInfo can't be null in HTConstructCapabilityElement()\n"); > + IEEE80211_DEBUG(IEEE80211_DL_ERR, "posHTCap or pHTInfo can't be null in %s\n", __func__); You should properly line-wrap thse lines, otherwise you will have to go back and fix them after this patch. thanks, greg k-h _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel