Re: [PATCH v2] can: length: can_fd_len2dlc(): make legnth calculation readable again

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Thanks Marc!

[PATCH v2] can: length: can_fd_len2dlc(): make legnth calculation readable again

legnth -> length?

;-)

I've sent a V3 which also adds a BUILD_BUG_ON() to make sure that the table has the correct size at build time.

Would that make sense to you?

Regards,
Oliver


On 17.01.21 23:40, Marc Kleine-Budde wrote:
In commit 652562e5ff06 ("can: length: can_fd_len2dlc(): simplify length
calculcation") the readability of the code degraded and became more error
prone. To counteract this, partially convert that patch and replace open coded
values (of the original code) with proper defines.

Cc: Oliver Hartkopp <socketcan@xxxxxxxxxxxx>
Cc: Vincent MAILHOL <mailhol.vincent@xxxxxxxxxx>
Fixes: 652562e5ff06 ("can: length: can_fd_len2dlc(): simplify length calculcation")
Signed-off-by: Marc Kleine-Budde <mkl@xxxxxxxxxxxxxx>
---
changes since v1:
- fix macro name (CANFD_MAX_LEN -> CANFD_MAX_DLEN)

  drivers/net/can/dev/length.c | 4 +++-
  1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/drivers/net/can/dev/length.c b/drivers/net/can/dev/length.c
index d35c4e82314d..f0d7cc969c1f 100644
--- a/drivers/net/can/dev/length.c
+++ b/drivers/net/can/dev/length.c
@@ -27,12 +27,14 @@ static const u8 len2dlc[] = {
  	13, 13, 13, 13, 13, 13, 13, 13,	/* 25 - 32 */
  	14, 14, 14, 14, 14, 14, 14, 14,	/* 33 - 40 */
  	14, 14, 14, 14, 14, 14, 14, 14,	/* 41 - 48 */
+	15, 15, 15, 15, 15, 15, 15, 15,	/* 49 - 56 */
+	15, 15, 15, 15, 15, 15, 15, 15	/* 57 - 64 */
  };
/* map the sanitized data length to an appropriate data length code */
  u8 can_fd_len2dlc(u8 len)
  {
-	if (len >= ARRAY_SIZE(len2dlc))
+	if (unlikely(len > CANFD_MAX_DLEN))
  		return CANFD_MAX_DLC;
return len2dlc[len];




[Index of Archives]     [Automotive Discussions]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]     [CAN Bus]

  Powered by Linux