]> git.baikalelectronics.ru Git - kernel.git/commit
net: stmmac: replace all pr_xxx by their netdev_xxx counterpart
authorLABBE Corentin <clabbe.montjoie@gmail.com>
Wed, 16 Nov 2016 19:09:39 +0000 (20:09 +0100)
committerDavid S. Miller <davem@davemloft.net>
Thu, 17 Nov 2016 18:30:30 +0000 (13:30 -0500)
commitaa15aea1467d03721cffea118804064c232ce242
treeeb5f591aeb9e5e19e551531fb0aeef910406a8ff
parentb4e5413f9c73a557bc7e35b49aad16ce1ffff4ac
net: stmmac: replace all pr_xxx by their netdev_xxx counterpart

The stmmac driver use lots of pr_xxx functions to print information.
This is bad since we cannot know which device logs the information.
(moreover if two stmmac device are present)

Furthermore, it seems that it assumes wrongly that all logs will always
be subsequent by using a dev_xxx then some indented pr_xxx like this:
kernel: sun7i-dwmac 1c50000.ethernet: no reset control found
kernel:  Ring mode enabled
kernel:  No HW DMA feature register supported
kernel:  Normal descriptors
kernel:  TX Checksum insertion supported

So this patch replace all pr_xxx by their netdev_xxx counterpart.
Excepts for some printing where netdev "cause" unpretty output like:
sun7i-dwmac 1c50000.ethernet (unnamed net_device) (uninitialized): no reset control found
In those case, I keep dev_xxx.

In the same time I remove some "stmmac:" print since
this will be a duplicate with that dev_xxx displays.

Signed-off-by: Corentin Labbe <clabbe.montjoie@gmail.com>
Acked-by: Giuseppe Cavallaro <peppe.cavallaro@st.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/stmicro/stmmac/stmmac_main.c
drivers/net/ethernet/stmicro/stmmac/stmmac_mdio.c