Skip to contentSkip to conversation

Thanks for reviewing can327!

DISCUSSION Vincent MAILHOL 2 participants 2 messages Jun 26, 2022 1 message not in the archive CAN network drivers

lore
Share & tools
lore

A message that is not in the archive (sent off-list or not yet received): [email protected]

#1 lore raw

+CC: linux-can

Quoted 11 lines · Max Staudt

Hi Vincent,

Thank you so much for the thorough review of can327!

(the word "relentless" comes to mind, even if it doesn't really fit...)

You went above and beyond anything I could ever have dreamed of for something that has started as a hobby curiosity, and now likely powers several projects more curious than itself. No idea why you did, but I'm grateful.

I can answer you. Several reasons but the strongest one is that less than two years ago, I was in your shoes.
I posted my first message on the linux-can mailing list in September 2020 with the patches for the etas_es58x driver.

I did mistakes similar to what you did, for example I also had a memory leak in my code:
https://lore.kernel.org/linux-can/[email protected]/
(thanks again to Jimmy for spotting that).

My patch reached v15 and took half a year before being finally accepted. All the comments and mentoring I received back then from Marc, Oliver and the others turned out to be my best learning experience in the last few years.
In some way, my review is a way to give back to the community what I received.

Quoted 2 lines

Please remind me to buy you a beer or other drink the nex^H^H^Hfirst time we meet.

I live in Tokyo, if you have plans to visit, it would be my pleasure to share a beer.

Yours sincerely,
Vincent Mailhol

#2 lore raw
Quoted 4 lines · Vincent MAILHOL
Quoted 3 lines · Max Staudt

Hi Vincent,

Thank you so much for the thorough review of can327!

Thank you for staying on the ball and not being deterred. Hopefully you'll receive such a thankful message from some contributor in the future!

Quoted 18 lines
Quoted 7 lines

(the word "relentless" comes to mind, even if it doesn't really fit...)

You went above and beyond anything I could ever have dreamed of for something that has started as a hobby curiosity, and now likely powers several projects more curious than itself. No idea why you did, but I'm grateful.

I can answer you. Several reasons but the strongest one is that less than two years ago, I was in your shoes.
I posted my first message on the linux-can mailing list in September 2020 with the patches for the etas_es58x driver.

I did mistakes similar to what you did, for example I also had a memory leak in my code:
https://lore.kernel.org/linux-can/[email protected]/
(thanks again to Jimmy for spotting that).

Quoted 5 lines

My patch reached v15 and took half a year before being finally accepted. All the comments and mentoring I received back then from Marc, Oliver and the others turned out to be my best learning experience in the last few years.
In some way, my review is a way to give back to the community what I received.

...and Vincent has given plenty! A proper TDC framework, thousands of patch reviews, and most recently the can-dev/Kconfig cleanups.

Quoted 5 lines
Quoted 2 lines

Please remind me to buy you a beer or other drink the nex^H^H^Hfirst time we meet.

I live in Tokyo, if you have plans to visit, it would be my pleasure to share a beer.

regards,
Marc

Signature · 4 lines
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 |