Messages in this thread | | | Date | Mon, 23 Jun 2008 00:06:01 +0100 | From | Ben Dooks <> | Subject | Re: [patch 2/3] WATCHDOG: Clean out header of s3c2410_wdt driver. |
| |
On Sun, Jun 22, 2008 at 11:35:16PM +0100, Alan Cox wrote: > On Sun, 22 Jun 2008 22:36:50 +0100 > Ben Dooks <ben-linux@fluff.org> wrote: > > > Remove the changelog from the top of the driver, which > > is redundant as this information is more accurately > > represented from the revision control holding the > > file. > > > > Signed-off-by: Ben Dooks <ben-linux@fluff.org> > > NAK. This is going to clash with the big pile of changes stuck in the > maintainers in queue.
Then why, as the original driver authour and current maintainer do I not get told about this? This isn't the first time I've come to apply this sort of thing and find that someone else has slipped in an 'trivial' change such as whitespace that breaks an entire patch series.
Do people not bother to check who wrote the driver or who is the current maintainer before doing these things?
-- Ben (ben@fluff.org, http://www.fluff.org/)
'a smiley only costs 4 bytes'
| |