Return-Path: From: "Duda, Lukasz" To: Alexander Aring , "linux-wpan@vger.kernel.org" CC: "linux-bluetooth@vger.kernel.org" , "netdev@vger.kernel.org" , "kernel@pengutronix.de" Subject: RE: [PATCH bluetooth-next] 6lowpan: fix debugfs interface entry name Date: Tue, 15 Dec 2015 11:47:27 +0000 Message-ID: References: <1450178735-1572-1-git-send-email-alex.aring@gmail.com> In-Reply-To: <1450178735-1572-1-git-send-email-alex.aring@gmail.com> Content-Type: text/plain; charset="iso-8859-2" MIME-Version: 1.0 Sender: netdev-owner@vger.kernel.org List-ID: Hi Alex, > -----Original Message----- > From: Alexander Aring [mailto:alex.aring@gmail.com] > Sent: Tuesday, December 15, 2015 12:26 > To: linux-wpan@vger.kernel.org > Cc: linux-bluetooth@vger.kernel.org; netdev@vger.kernel.org; > kernel@pengutronix.de; Duda, Lukasz; Alexander Aring > Subject: [PATCH bluetooth-next] 6lowpan: fix debugfs interface entry name > > This patches moves the debugfs interface related register after netdevice > register. The function lowpan_dev_debugfs_init will use "dev->name" which > can be before register_netdevice a format string. > The function register_netdevice will evaluate the format string if necessary > and replace "dev->name" to the real interface name. > > Reported-by: Lukasz Duda > Signed-off-by: Alexander Aring > --- > net/6lowpan/core.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/net/6lowpan/core.c b/net/6lowpan/core.c index c7f06f5..faf65ba > 100644 > --- a/net/6lowpan/core.c > +++ b/net/6lowpan/core.c > @@ -29,13 +29,13 @@ int lowpan_register_netdevice(struct net_device > *dev, > > lowpan_priv(dev)->lltype = lltype; > > - ret = lowpan_dev_debugfs_init(dev); > + ret = register_netdevice(dev); > if (ret < 0) > return ret; > > - ret = register_netdevice(dev); > + ret = lowpan_dev_debugfs_init(dev); > if (ret < 0) > - lowpan_dev_debugfs_exit(dev); > + unregister_netdevice(dev); > > return ret; > } > -- > 2.6.1 Thanks, after this change I do have correct interface name under /sys/kernel/debug/6lowpan for Bluetooth Low Energy (bt0). Acked-by: Lukasz Duda Best regards, ?ukasz