Hi Vincent, On 03.12.24 10:45, Vincent Mailhol wrote:
https://lore.kernel.org/linux-can/20241201112333.6950-1-socketcan@xxxxxxxxxxxx/T/#u https://lore.kernel.org/linux-can/20241201112230.6917-1-socketcan@xxxxxxxxxxxx/T/#t
Thanks for all the testing and the fixes. Because of the lack of testing of this RFC on my side, I was expecting such issues. But I really appreciate that you took time to investigate and debug, really helpful! I will make sure to incorporate these fixes in the next version.
I'll send out an extended RFC V2 which is my current test base for you in some minutes.
Either for the kernel and iproute2 there are two new patches that add new controlmode flags.
The next series I send will add the pwm and drop the RFC patch.
Excellent! I assume it will follow the TDC pattern with CAN_CTRLMODE_XL_PWM_AUTO and CAN_CTRLMODE_XL_PWM_MANUAL or something similar?
My goal is to have this CAN XL series ready for inclusion in linux 6.14. I don't want to overcommit, but hopefully, I would like to send the v1 either this weekend or next weekend.
No hurry so far ;-)
I will also rethink whether or not is it worth doing some NLA nesting as suggested by Marc here: https://lore.kernel.org/linux-can/20241112-flashy-straight-poodle-9a796d-mkl@xxxxxxxxxxxxxx/ (I am still divided on this subject).
I'll take a look into this too. Best regards, Oliver
