Thread (19 messages) 19 messages, 2 authors, 2021-03-29

Re: [PATCH v13 00/11] Introducing ETAS ES58X CAN USB interfaces

From: Vincent MAILHOL <hidden>
Date: 2021-03-19 13:33:53

On Fri 19 mars 2021 at 21:41, Marc Kleine-Budde [off-list ref] wrote:
Hello,

this my attempt to split the driver into 3 patches, so that it can be
send via mailing lists:
- core support
- es581_4
- es58x_fd

For now I've removed the setting of dql.min_limit, that can be added
back in a later patch, once discussed with upstream netdev. While
working on the code, I've added some typo and kernel doc fixes.

I plan to squash the patches, I've kept them separate for better
illustration.
Thanks a lot for your support! You could have asked me to split the patches :)

I was also myself working on a new version:
  - addressing the comments from Jimmy
  - following my exchange with Oliver yesterday, I realised that
    I totally misunderstood the meaning of CAN_CTRLMODE_LOOPBACK,
    I will remove it.

If fine with you, I can directly squash your suggestions while
preparing my v14 and just keep the 3 patches split as suggested.


Yours sincerely,
Vincent
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help