]> git.baikalelectronics.ru Git - kernel.git/commit
can: netlink: allow configuring of fixed data bit rates without need for do_set_data_...
authorMarc Kleine-Budde <mkl@pengutronix.de>
Sat, 11 Jun 2022 14:20:58 +0000 (16:20 +0200)
committerMarc Kleine-Budde <mkl@pengutronix.de>
Mon, 13 Jun 2022 16:25:29 +0000 (18:25 +0200)
commit130dff5abdc9cea1442a048ef063aa00ce4fa07a
tree3aabf9e35fa9bca7d79839cb7f54ad2703be189f
parentb6c6c1f80d78bf0b97ab76d911e2ea372153177d
can: netlink: allow configuring of fixed data bit rates without need for do_set_data_bittiming callback

This patch is similar to 3b204db75b63 ("can: netlink: allow
configuring of fixed bit rates without need for do_set_bittiming
callback") but for data bit rates instead of bit rates.

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

If the bit rate is configured via netlink, the can_changelink()
function checks that either can_priv::data_bittiming_const or struct
can_priv::do_set_data_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::data_bitrate_const to the check in can_changelink().

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

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