Re: [PATCH] I2C: add support for the PCF8575 chip

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Jean Delvare
Date: Wednesday, October 24, 2007 - 1:34 am

Hi Andrew,

On Tue, 23 Oct 2007 09:18:45 -0700, Andrew Morton wrote:

Again you are unfair. I never suggested that people _remove_ LKML from
the Cc. I asked that they do not include it to start with, which is
different.

That being said, chances are higher that a list gets dropped in the
course of the discussions if many lists were included in the first
place. Which is just one more reason to not flood too many lists in the
first place.


Anyone picking a random patch on any mailing list instead of waiting
for it to go upstream or at least in some developer tree, should be
aware that he/she is running experimental code, and should search for
possible updates. This includes google, marc.info and getting in touch
with the author of the patch if the patch is getting old. If nothing
else, letting the author know that you tested his/her patch is the bare
minimum you can do as a tester if you care about the patch going
upstream.

This really doesn't have anything to do with what list(s) the patch has
been sent to in the first place.


That's unfortunate, but I'd guess this happens all the time and this is
inherently bound to the way you work. Picking random patches after 3
weeks and assuming that nothing happened to them just because LKML says
so, is quite a risky bet, methinks.

My opinion on the matter is that it's up to submitters to make sure
that non-bugfix patches they send aren't lost, not maintainers. If
someone sends a patch adding support for a new device, and nobody
replies, I say it's up to the sender to resend the patch, or even
better to actively look for someone to review the patch, instead of
passively waiting for something to happen. The kernel is growing fast,
and the only way for us maintainers to scale is to teach contributors
how to help us keep up with the workload.

Bugfix patches are of course different, it's up to the maintainers to
review them quickly and make sure they aren't lost, no question about
that.

You have a different opinion, you insist on picking _all_ patches and
taking care of everything. This is your own right, you are free to work
the way you want and I respect that. But please don't expect me to
change the way _I_ want to work to accommodate the way _you_ want work.

-- 
Jean Delvare
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH] I2C: add support for the PCF8575 chip, Bart Van Assche, (Fri Oct 5, 2:32 am)
Re: [PATCH] I2C: add support for the PCF8575 chip, Jean Delvare, (Fri Oct 5, 1:11 pm)
Re: [PATCH] I2C: add support for the PCF8575 chip, Andrew Morton, (Mon Oct 22, 2:58 pm)
Re: [i2c] [PATCH] I2C: add support for the PCF8575 chip, Trent Piepho, (Mon Oct 22, 3:32 pm)
Re: [PATCH] I2C: add support for the PCF8575 chip, Jean Delvare, (Tue Oct 23, 4:50 am)
Re: [PATCH] I2C: add support for the PCF8575 chip, Andrew Morton, (Tue Oct 23, 9:18 am)
Re: [PATCH] I2C: add support for the PCF8575 chip, Jean Delvare, (Wed Oct 24, 1:34 am)