login
Header Space

 
 

Re: [PATCH] 2/3 mtd: add support for flash on the SEGA Dreamcast Visual Memory Unit

Score:
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Adrian McMenamin <adrian@...>
Cc: dwmw2 <dwmw2@...>, Andrew Morton <akpm@...>, linux-sh <linux-sh@...>, Greg KH <greg@...>, Paul Mundt <lethal@...>, LKML <linux-kernel@...>, MTD <linux-mtd@...>
Date: Saturday, March 22, 2008 - 2:32 pm

On Sat, 22 March 2008 18:03:55 +0000, Adrian McMenamin wrote:

Without a doubt, buffering is useful.  However I question how useful it
is to implement this in individual device drivers instead of once in
mtd_core.c.


Given that you have ignored most of my previous comments, NAK.  I don't
mind merging code that isn't up to our standards yet.  But I have a bad
feeling about a maintainer that does not understand review comments.
Since you had similar problems understanding Andrew, part of the blame
may sit on your side.

Jörn

-- 
The key to performance is elegance, not battalions of special cases.
-- Jon Bentley and Doug McIlroy
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Re: [PATCH] 3/3 maple: update bus driver to support Dreamcas..., Adrian McMenamin, (Mon Mar 24, 11:06 am)
Re: [PATCH] 3/3 maple: update bus driver to support Dreamcas..., Adrian McMenamin, (Mon Mar 24, 11:51 am)
Re: [PATCH] 2/3 mtd: add support for flash on the SEGA Dream..., Jörn, (Sat Mar 22, 2:32 pm)
Re: [PATCH] 2/3 mtd: add support for flash on the SEGA Dream..., Adrian McMenamin, (Mon Mar 24, 10:43 am)
speck-geostationary