arduino / arduino/ArduinoCore-API
Feature Request - add "remote" member to the CanMsg class to denote Remote Transfer Request (RTR) frames
- Ngôn ngữ chính
- C++
- Star
- 306
- Fork
- 150
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Mô tả
The CAN specification reserves a bit in the arbitration header to signal if the frame is a Remote Transfer Request (RTR) frame or a normal Data frame. This is the bit that follows the 11 identifier bits in the standard header, or the 18 bits of the second part of the identifier in an extended frame.
This is useful for system that use RTR (yes I know this is frowned on by some in the CAN community, see linked doc). However systems exist that require this support so we should enable the Uno R4 to detect if a frame has been sent with the RTR bit.
Given that the flag is already detected by the underlying FSP and the type appears in the "can_frame_t" CAN data frame struct, there is a relatively simple enhancement which I will float here.
The first part is to add it as a data member to the CanMsg class (in CanMsg.h)
```
uint32_t id;
uint8_t data_length;
uint8_t data[MAX_DATA_LENGTH];
bool remote; // == new member ==
```
and then add it to the constructors as an optional argument
```
class CanMsg : public Printable
// snip
CanMsg(uint32_t const can_id, uint8_t const can_data_len, uint8_t const * can_data_ptr, bool const remote_frame_type = false)
: id{can_id}
, data_length{min(can_data_len, MAX_DATA_LENGTH)}
, data{0}
, remote{remote_frame_type}
{
//snip
```
(It would also need to be copied across in the copy constructor and probably useful to emit it in the printTo function.)
This now makes it available, however it is always set to false so the msg will default as a data frame unless the flag is set when the object is created.
For incoming messages in the R4 Uno this occurs in the R7FA4M1_CAN.cpp file
```
void R7FA4M1_CAN::onCanCallback(can_callback_args_t * p_args)
{
switch (p_args->event)
{
case CAN_EVENT_TX_COMPLETE: break;
case CAN_EVENT_RX_COMPLETE: // Currently driver don't support this. This is unreachable code for now.
{
/* Extract the received CAN message. */
CanMsg const msg
(
(p_args->frame.id_mode == CAN_ID_MODE_STANDARD) ? CanStandardId(p_args->frame.id) : CanExtendedId(p_args->frame.id),
p_args->frame.data_length_code,
p_args->frame.data,
(p_args->frame.type == CAN_FRAME_TYPE_REMOTE) // == new argument ==
);
/* Store the received CAN message in the receive buffer. */
_can_rx_buf.enqueue(msg);
}
//snip
```
I have not considered whether this would also require a change to files related to other 32 bit Renases chips (most likely yes).
However it appears to be largely backwards compatible as it does not affect the id (the other option was to use bit 30 if the id).
So, looking for some feedback before I do any further work and potentially provide a PR.
/-------------------
https://copperhilltech.com/content/CiA%20802%20AN%20V1.0%20CANopen%20CAN%20remote%20frames%20%E2%80%93%20Avoiding%20of%20usage.pdf
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
Hướng nghiên cứu
Bắt đầu với CanMsg.h để theo dõi các constructor, hành vi sao chép và đầu ra của printTo, sau đó kiểm tra R7FA4M1_CAN.cpp cùng cách onCanCallback xử lý can_frame_t. Kiểm tra xem các tệp CAN Renesas 32-bit khác có cần được xử lý tương tự hay không. Hoàn thành có nghĩa là trạng thái remote được giữ nguyên cho các message được tạo và nhận mà không làm hỏng các đường dẫn khởi tạo hiện có.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- cpp
- Lĩnh vực
- embedded-iot
- Loại issue
- Tính năng
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 35/100