All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] can: uapi: update CAN-FD frame description
@ 2021-05-17 11:37 Marc Kleine-Budde
  2021-05-17 16:55 ` Oliver Hartkopp
  0 siblings, 1 reply; 2+ messages in thread
From: Marc Kleine-Budde @ 2021-05-17 11:37 UTC (permalink / raw)
  To: linux-can; +Cc: socketcan, Marc Kleine-Budde, Ayoub Kaanich

Since an early version of the CAN-FD specification the bit that
defines a CAN-FD frame on the wire, has been renamed from Extended
Data Length (EDL) to FD Frame (FDF).

To avoid confusion, update the struct canfd_frame description in the
UAPI headers accordingly.

Suggested-by: Ayoub Kaanich <kayoub5@live.com>
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
---
 include/uapi/linux/can.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/include/uapi/linux/can.h b/include/uapi/linux/can.h
index c7535352fef6..ac5d7a31671f 100644
--- a/include/uapi/linux/can.h
+++ b/include/uapi/linux/can.h
@@ -123,8 +123,8 @@ struct can_frame {
 /*
  * defined bits for canfd_frame.flags
  *
- * The use of struct canfd_frame implies the Extended Data Length (EDL) bit to
- * be set in the CAN frame bitstream on the wire. The EDL bit switch turns
+ * The use of struct canfd_frame implies the FD Frame (FDF) bit to
+ * be set in the CAN frame bitstream on the wire. The FDF bit switch turns
  * the CAN controllers bitstream processor into the CAN FD mode which creates
  * two new options within the CAN FD frame specification:
  *
-- 
2.30.2



^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] can: uapi: update CAN-FD frame description
  2021-05-17 11:37 [PATCH] can: uapi: update CAN-FD frame description Marc Kleine-Budde
@ 2021-05-17 16:55 ` Oliver Hartkopp
  0 siblings, 0 replies; 2+ messages in thread
From: Oliver Hartkopp @ 2021-05-17 16:55 UTC (permalink / raw)
  To: Marc Kleine-Budde, linux-can; +Cc: Ayoub Kaanich



On 17.05.21 13:37, Marc Kleine-Budde wrote:
> Since an early version of the CAN-FD specification the bit that
> defines a CAN-FD frame on the wire, has been renamed from Extended
> Data Length (EDL) to FD Frame (FDF).
> 
> To avoid confusion, update the struct canfd_frame description in the
> UAPI headers accordingly.
> 
> Suggested-by: Ayoub Kaanich <kayoub5@live.com>
> Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>

Thanks Marc!

Acked-by: Oliver Hartkopp <socketcan@hartkopp.net>

> ---
>   include/uapi/linux/can.h | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/include/uapi/linux/can.h b/include/uapi/linux/can.h
> index c7535352fef6..ac5d7a31671f 100644
> --- a/include/uapi/linux/can.h
> +++ b/include/uapi/linux/can.h
> @@ -123,8 +123,8 @@ struct can_frame {
>   /*
>    * defined bits for canfd_frame.flags
>    *
> - * The use of struct canfd_frame implies the Extended Data Length (EDL) bit to
> - * be set in the CAN frame bitstream on the wire. The EDL bit switch turns
> + * The use of struct canfd_frame implies the FD Frame (FDF) bit to
> + * be set in the CAN frame bitstream on the wire. The FDF bit switch turns
>    * the CAN controllers bitstream processor into the CAN FD mode which creates
>    * two new options within the CAN FD frame specification:
>    *
> 

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2021-05-17 16:55 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-05-17 11:37 [PATCH] can: uapi: update CAN-FD frame description Marc Kleine-Budde
2021-05-17 16:55 ` Oliver Hartkopp

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.