]> git.baikalelectronics.ru Git - kernel.git/commit
can: netlink: allow configuring of fixed bit rates without need for do_set_bittiming...
authorMarc Kleine-Budde <mkl@pengutronix.de>
Sat, 11 Jun 2022 14:20:58 +0000 (16:20 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 17 Aug 2022 12:23:20 +0000 (14:23 +0200)
commitd8d3a22ef00bef721c7ca2486d10fee0a8e85d3a
tree3d0dc80f4ee5bbd3176f8c77c4e664af5a127f65
parent80d3c6a7a8a37f7f1c225ede3760aaf88d6f0d94
can: netlink: allow configuring of fixed bit rates without need for do_set_bittiming callback

[ Upstream commit 3b204db75b63618b913e67acdacb7c0c49b43379 ]

Usually CAN devices support configurable bit rates. The limits are
defined by struct can_priv::bittiming_const. Another way is to
implement the struct can_priv::do_set_bittiming callback.

If the bit rate is configured via netlink, the can_changelink()
function checks that either can_priv::bittiming_const or struct
can_priv::do_set_bittiming is implemented.

In commit 405abb0e0c7b ("can: dev: add CAN interface API for fixed
bitrates") an API for configuring bit rates on CAN interfaces that
only support fixed bit rates was added. The supported bit rates are
defined by struct can_priv::bitrate_const.

However the above mentioned commit forgot to add the struct
can_priv::bitrate_const to the check in can_changelink().

In order to avoid to implement a no-op can_priv::do_set_bittiming
callback on devices with fixed bit rates, extend the check in
can_changelink() accordingly.

Link: https://lore.kernel.org/all/20220611144248.3924903-1-mkl@pengutronix.de
Fixes: 405abb0e0c7b ("can: dev: add CAN interface API for fixed bitrates")
Reported-by: Max Staudt <max@enpas.org>
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/net/can/dev/netlink.c