jwang12340 commented on code in PR #17907:
URL: https://github.com/apache/nuttx/pull/17907#discussion_r2690670855


##########
net/can/can_setsockopt.c:
##########
@@ -133,7 +133,7 @@ int can_setsockopt(FAR struct socket *psock, int level, int 
option,
             return -EINVAL;
           }
 
-        conn->err_mask = *(FAR can_err_mask_t *)value & CAN_ERR_MASK;
+        conn->err_mask = *(FAR can_err_mask_t *)value;

Review Comment:
   LIN and CAN are different on driver level, but above in software protocal 
level the lin frame can be process by socket CAN with standard CAN protocal, 
LIN related flags are defined in "include/nuttx/lin.h".



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to