diff options
author | Vladimir Oltean <vladimir.oltean@nxp.com> | 2020-10-05 12:09:12 +0300 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2020-10-06 06:05:47 -0700 |
commit | 013264936654fd3052fabe700b7a4933d3925395 (patch) | |
tree | f6e5f625fd3afe6ee66e9cf9aa4ce0089b1b0fb7 /drivers/net/dsa/ocelot/seville_vsc9953.c | |
parent | net: mscc: ocelot: divide watermark value by 60 when writing to SYS_ATOP (diff) | |
download | linux-dev-013264936654fd3052fabe700b7a4933d3925395.tar.xz linux-dev-013264936654fd3052fabe700b7a4933d3925395.zip |
net: mscc: ocelot: warn when encoding an out-of-bounds watermark value
There is an upper bound to the value that a watermark may hold. That
upper bound is not immediately obvious during configuration, and it
might be possible to have accidental truncation.
Actually this has happened already, add a warning to prevent it from
happening again.
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/dsa/ocelot/seville_vsc9953.c')
-rw-r--r-- | drivers/net/dsa/ocelot/seville_vsc9953.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/net/dsa/ocelot/seville_vsc9953.c b/drivers/net/dsa/ocelot/seville_vsc9953.c index 110070a60041..9e9fd19e1d00 100644 --- a/drivers/net/dsa/ocelot/seville_vsc9953.c +++ b/drivers/net/dsa/ocelot/seville_vsc9953.c @@ -911,6 +911,8 @@ static int vsc9953_prevalidate_phy_mode(struct ocelot *ocelot, int port, */ static u16 vsc9953_wm_enc(u16 value) { + WARN_ON(value >= 16 * BIT(9)); + if (value >= BIT(9)) return BIT(9) | (value / 16); |