On 1/11/21 11:41 AM, Vincent Mailhol wrote: > Add a frame_len argument to can_put_echo_skb() which is used to save > length of the CAN frame into field frame_len of struct can_skb_priv so > that it can be later used after transmission completion. Convert all > users of this function, too. > > Drivers which implement BQL call can_put_echo_skb() with the output of > can_skb_get_frame_len(skb) and drivers which do not simply pass zero > as an input (in the same way that NULL would be given to > can_get_echo_skb()). This way, we have a nice symmetry between the two > echo functions. > > Signed-off-by: Vincent Mailhol <mailhol.vincent@xxxxxxxxxx> > -- > v2: make the assignemend unconditional. > > Hi Marc, > > If you like the idea to modify can_put_echo_skb(), you can use this > patch. Just put it between patches 12 and 13 of your series. I think I've done the same changes, why applying. The series I just send is based on your v1 with my changes. regards, Marc -- Pengutronix e.K. | Marc Kleine-Budde | Embedded Linux | https://www.pengutronix.de | Vertretung West/Dortmund | Phone: +49-231-2826-924 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
Attachment:
signature.asc
Description: OpenPGP digital signature