Skip to content

[3.8] bpo-37085: Expose SocketCAN bcm_msg_head flags (GH-13646)#15049

Merged
miss-islington merged 1 commit into
python:3.8from
miss-islington:backport-31c4fd2-3.8
Jul 31, 2019
Merged

[3.8] bpo-37085: Expose SocketCAN bcm_msg_head flags (GH-13646)#15049
miss-islington merged 1 commit into
python:3.8from
miss-islington:backport-31c4fd2-3.8

Conversation

@miss-islington

@miss-islington miss-islington commented Jul 31, 2019

Copy link
Copy Markdown
Contributor

Expose the CAN_BCM SocketCAN constants used in the bcm_msg_head struct
flags (provided by <linux/can/bcm.h>) under the socket library.

This adds the following constants with a CAN_BCM prefix:

  • SETTIMER
  • STARTTIMER
  • TX_COUNTEVT
  • TX_ANNOUNCE
  • TX_CP_CAN_ID
  • RX_FILTER_ID
  • RX_CHECK_DLC
  • RX_NO_AUTOTIMER
  • RX_ANNOUNCE_RESUME
  • TX_RESET_MULTI_IDX
  • RX_RTR_FRAME
  • CAN_FD_FRAME

The CAN_FD_FRAME flag was introduced in the 4.8 kernel, while the other
ones were present since SocketCAN drivers were mainlined in 2.6.25. As
such, it is probably unnecessary to guard against these constants being
missing.
(cherry picked from commit 31c4fd2)

Co-authored-by: karl ding karlding@users.noreply.github.com

https://bugs.python.org/issue37085

Expose the CAN_BCM SocketCAN constants used in the bcm_msg_head struct
flags (provided by <linux/can/bcm.h>) under the socket library.

This adds the following constants with a CAN_BCM prefix:

  * SETTIMER
  * STARTTIMER
  * TX_COUNTEVT
  * TX_ANNOUNCE
  * TX_CP_CAN_ID
  * RX_FILTER_ID
  * RX_CHECK_DLC
  * RX_NO_AUTOTIMER
  * RX_ANNOUNCE_RESUME
  * TX_RESET_MULTI_IDX
  * RX_RTR_FRAME
  * CAN_FD_FRAME

The CAN_FD_FRAME flag was introduced in the 4.8 kernel, while the other
ones were present since SocketCAN drivers were mainlined in 2.6.25. As
such, it is probably unnecessary to guard against these constants being
missing.
(cherry picked from commit 31c4fd2)

Co-authored-by: karl ding <karlding@users.noreply.github.com>
@miss-islington

Copy link
Copy Markdown
Contributor Author

@karlding and @ambv: Status check is done, and it's a success ✅ .

@miss-islington miss-islington merged commit d8b914a into python:3.8 Jul 31, 2019
@miss-islington miss-islington deleted the backport-31c4fd2-3.8 branch July 31, 2019 09:10
@miss-islington

Copy link
Copy Markdown
Contributor Author

@karlding and @ambv: Status check is done, and it's a success ✅ .

1 similar comment
@miss-islington

Copy link
Copy Markdown
Contributor Author

@karlding and @ambv: Status check is done, and it's a success ✅ .

@ambv

ambv commented Jul 31, 2019

Copy link
Copy Markdown
Contributor

But is it done, and is it a success?

@miss-islington

Copy link
Copy Markdown
Contributor Author

@karlding and @ambv: Status check is done, and it's a success ✅ .

@ambv

ambv commented Jul 31, 2019

Copy link
Copy Markdown
Contributor

Good, that's what I wanted to know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type-feature A feature request or enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants