Commit 69ebb370 authored by Vladimir Oltean's avatar Vladimir Oltean Committed by David S. Miller

net: dsa: tag_8021q: use symbolic error names

Use %pe to give the user a string holding the error code instead of just
a number.
Signed-off-by: default avatarVladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent a81a4574
...@@ -214,8 +214,8 @@ static int dsa_8021q_setup_port(struct dsa_8021q_context *ctx, int port, ...@@ -214,8 +214,8 @@ static int dsa_8021q_setup_port(struct dsa_8021q_context *ctx, int port,
err = dsa_8021q_vid_apply(ctx, i, rx_vid, flags, enabled); err = dsa_8021q_vid_apply(ctx, i, rx_vid, flags, enabled);
if (err) { if (err) {
dev_err(ctx->ds->dev, dev_err(ctx->ds->dev,
"Failed to apply RX VID %d to port %d: %d\n", "Failed to apply RX VID %d to port %d: %pe\n",
rx_vid, port, err); rx_vid, port, ERR_PTR(err));
return err; return err;
} }
} }
...@@ -226,8 +226,8 @@ static int dsa_8021q_setup_port(struct dsa_8021q_context *ctx, int port, ...@@ -226,8 +226,8 @@ static int dsa_8021q_setup_port(struct dsa_8021q_context *ctx, int port,
err = dsa_8021q_vid_apply(ctx, upstream, rx_vid, 0, enabled); err = dsa_8021q_vid_apply(ctx, upstream, rx_vid, 0, enabled);
if (err) { if (err) {
dev_err(ctx->ds->dev, dev_err(ctx->ds->dev,
"Failed to apply RX VID %d to port %d: %d\n", "Failed to apply RX VID %d to port %d: %pe\n",
rx_vid, port, err); rx_vid, port, ERR_PTR(err));
return err; return err;
} }
...@@ -242,15 +242,15 @@ static int dsa_8021q_setup_port(struct dsa_8021q_context *ctx, int port, ...@@ -242,15 +242,15 @@ static int dsa_8021q_setup_port(struct dsa_8021q_context *ctx, int port,
enabled); enabled);
if (err) { if (err) {
dev_err(ctx->ds->dev, dev_err(ctx->ds->dev,
"Failed to apply TX VID %d on port %d: %d\n", "Failed to apply TX VID %d on port %d: %pe\n",
tx_vid, port, err); tx_vid, port, ERR_PTR(err));
return err; return err;
} }
err = dsa_8021q_vid_apply(ctx, upstream, tx_vid, 0, enabled); err = dsa_8021q_vid_apply(ctx, upstream, tx_vid, 0, enabled);
if (err) { if (err) {
dev_err(ctx->ds->dev, dev_err(ctx->ds->dev,
"Failed to apply TX VID %d on port %d: %d\n", "Failed to apply TX VID %d on port %d: %pe\n",
tx_vid, upstream, err); tx_vid, upstream, ERR_PTR(err));
return err; return err;
} }
...@@ -267,8 +267,8 @@ int dsa_8021q_setup(struct dsa_8021q_context *ctx, bool enabled) ...@@ -267,8 +267,8 @@ int dsa_8021q_setup(struct dsa_8021q_context *ctx, bool enabled)
err = dsa_8021q_setup_port(ctx, port, enabled); err = dsa_8021q_setup_port(ctx, port, enabled);
if (err < 0) { if (err < 0) {
dev_err(ctx->ds->dev, dev_err(ctx->ds->dev,
"Failed to setup VLAN tagging for port %d: %d\n", "Failed to setup VLAN tagging for port %d: %pe\n",
port, err); port, ERR_PTR(err));
return err; return err;
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment