Re: [PATCH v2 09/10] rtlwifi: use debugfs to debug.

[Joe Perches <joe@perches.com>]

On Mon, 2017-08-14 at 16:17 -0500, Larry Finger wrote:
> On 08/14/2017 01:56 PM, Joe Perches wrote:
> > On Sun, 2017-06-25 at 10:43 -0500, Larry Finger wrote:
> > > From: Ping-Ke Shih <pkshih@realtek.com>
> > > 
> > > Use debugfs to dump register and btcoex status.
> > 
> > []
> > > diff --git a/drivers/net/wireless/realtek/rtlwifi/debug.c b/drivers/net/wireless/realtek/rtlwifi/debug.c
> > 
> > []
> > > +void rtl_debug_add_one(struct ieee80211_hw *hw)
> > > +{
> > > +	struct rtl_priv *rtlpriv = rtl_priv(hw);
> > > +	struct rtl_efuse *rtlefuse = rtl_efuse(rtl_priv(hw));
> > > +	struct dentry *parent;
> > > +
> > > +	snprintf(rtlpriv->dbg.debugfs_name, 18, "%02x-%02x-%02x-%02x-%02x-%02x",
> > > +		 rtlefuse->dev_addr[0], rtlefuse->dev_addr[1],
> > > +		 rtlefuse->dev_addr[2], rtlefuse->dev_addr[3],
> > > +		 rtlefuse->dev_addr[4], rtlefuse->dev_addr[5]);
> > 
> > Why uses dashes and why not just use %pM
> 
> This snprintf() supplies the MAC address in a file name of the type
> /sys/kernel/debug/rtlwifi/00-11-22-33-44-55-66/mac_0
> Using %pM would put colons there, and I have had trouble with colons in a file 
> name. We could use %#pM and drop the separators.

%pm emits without colons