Thread (7 messages) 7 messages, 3 authors, 2020-01-02

Re: [PATCH] stmmac: debugfs entry name is not be changed when udev rename device name.

From: Randy Dunlap <hidden>
Date: 2019-12-31 02:11:32
Also in: linux-arm-kernel, lkml

Hi,

On 12/30/19 6:03 PM, Jiping Ma wrote:
quoted hunk ↗ jump to hunk
Add one notifier for udev changes net device name.

Signed-off-by: Jiping Ma <redacted>
---
 .../net/ethernet/stmicro/stmmac/stmmac_main.c | 38 ++++++++++++++++++-
 1 file changed, 37 insertions(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c
index b14f46a57154..c1c877bb4421 100644
--- a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c
+++ b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c
@@ -4038,6 +4038,40 @@ static int stmmac_dma_cap_show(struct seq_file *seq, void *v)
 }
 DEFINE_SHOW_ATTRIBUTE(stmmac_dma_cap);
 
+/**
Just use /* here since this is not a kernel-doc comment.
/** is reserved for kernel-doc comments/notation.
+ * Use network device events to create/remove/rename
+ * debugfs file entries
+ */
+static int stmmac_device_event(struct notifier_block *unused,
+			       unsigned long event, void *ptr)
+{
quoted hunk ↗ jump to hunk
@@ -4050,7 +4084,6 @@ static int stmmac_init_fs(struct net_device *dev)
 
 		return -ENOMEM;
 	}
-
 	/* Entry to report DMA RX/TX rings */
 	priv->dbgfs_rings_status =
 		debugfs_create_file("descriptors_status", 0444,
I don't see a problem with the blank line.  Why remove it?


thanks.
-- 
~Randy
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help