diff options
author | Thomas Haller <thaller@redhat.com> | 2022-03-15 08:16:16 (GMT) |
---|---|---|
committer | Thomas Haller <thaller@redhat.com> | 2022-03-15 08:24:39 (GMT) |
commit | 841553b1c4fc2452b881df19266302b764fc02a8 (patch) | |
tree | a989261ff9d8f758680dbc2529a7c8340ecba72e /include | |
parent | 37998f7651e8eb3cb94db23ce8e807db7245cf24 (diff) | |
download | libnl-841553b1c4fc2452b881df19266302b764fc02a8.zip libnl-841553b1c4fc2452b881df19266302b764fc02a8.tar.gz libnl-841553b1c4fc2452b881df19266302b764fc02a8.tar.bz2 |
route: drop bitrate,sample-point getters/setters from can link
These accessor functions just get/set a field of ci_data_bittiming,
which can already be get/set as a whole. It seems redundant, so drop it.
It's really important that things are principle at all (like, setting
ci_data_bittiming). It's not clear that we need such convenience
functions, if you can achieve the same result otherwise.
Yes, we have similar accessors already. I think that is not a good
approach though.
If they turn out very useful, we can still revert this patch.
Diffstat (limited to 'include')
-rw-r--r-- | include/netlink/route/link/can.h | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/include/netlink/route/link/can.h b/include/netlink/route/link/can.h index c57898e..6c0c0b8 100644 --- a/include/netlink/route/link/can.h +++ b/include/netlink/route/link/can.h @@ -58,12 +58,6 @@ extern int rtnl_link_can_get_data_bittiming(struct rtnl_link *, extern int rtnl_link_can_set_data_bittiming(struct rtnl_link *, struct can_bittiming *); -extern int rtnl_link_can_get_data_bitrate(struct rtnl_link *, uint32_t *); -extern int rtnl_link_can_set_data_bitrate(struct rtnl_link *, uint32_t); - -extern int rtnl_link_can_get_data_sample_point(struct rtnl_link *, uint32_t *); -extern int rtnl_link_can_set_data_sample_point(struct rtnl_link *, uint32_t); - #ifdef __cplusplus } #endif |