Commit 8afabd27 authored by Vladimir Oltean's avatar Vladimir Oltean Committed by Jakub Kicinski
Browse files

net: dsa: use kernel data types for ethtool ops on conduit 



Suppress some checkpatch 'CHECK' messages about u8 being preferable over
uint8_t, etc. No functional change.

Signed-off-by: default avatarVladimir Oltean <vladimir.oltean@nxp.com>
Reviewed-by: default avatarAndrew Lunn <andrew@lunn.ch>
Link: https://patch.msgid.link/20251122112311.138784-3-vladimir.oltean@nxp.com


Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent eba81b0a

net/dsa/conduit.c

+5 −6
Original line number Diff line number Diff line
@@ -89,7 +89,7 @@ static void dsa_conduit_get_regs(struct net_device *dev, 


static void dsa_conduit_get_ethtool_stats(struct net_device *dev,

					  struct ethtool_stats *stats,

					  uint64_t *data)

					  u64 *data)

{

	struct dsa_port *cpu_dp = dev->dsa_ptr;

	const struct ethtool_ops *ops = cpu_dp->orig_ethtool_ops;
@@ -110,7 +110,7 @@ static void dsa_conduit_get_ethtool_stats(struct net_device *dev, 


static void dsa_conduit_get_ethtool_phy_stats(struct net_device *dev,

					      struct ethtool_stats *stats,

					      uint64_t *data)

					      u64 *data)

{

	struct dsa_port *cpu_dp = dev->dsa_ptr;

	const struct ethtool_ops *ops = cpu_dp->orig_ethtool_ops;
@@ -160,8 +160,8 @@ static int dsa_conduit_get_sset_count(struct net_device *dev, int sset) 
	return count;

}



static void dsa_conduit_get_strings(struct net_device *dev, uint32_t stringset,

				    uint8_t *data)

static void dsa_conduit_get_strings(struct net_device *dev, u32 stringset,

				    u8 *data)

{

	struct dsa_port *cpu_dp = dev->dsa_ptr;

	const struct ethtool_ops *ops = cpu_dp->orig_ethtool_ops;
@@ -169,8 +169,7 @@ static void dsa_conduit_get_strings(struct net_device *dev, uint32_t stringset, 
	int port = cpu_dp->index;

	int len = ETH_GSTRING_LEN;

	int mcount = 0, count, i;

	uint8_t pfx[4];

	uint8_t *ndata;

	u8 pfx[4], *ndata;



	snprintf(pfx, sizeof(pfx), "p%.2d", port);

	/* We do not want to be NULL-terminated, since this is a prefix */