Messages in this thread | | | Subject | Re: [2.6 patch] no longer mark MTD_OBSOLETE_CHIPS as BROKEN and remove broken MTD_OBSOLETE_CHIPS drivers | From | David Woodhouse <> | Date | Sun, 08 Jan 2006 00:38:54 +0000 |
| |
On Sun, 2006-01-08 at 01:24 +0100, Adrian Bunk wrote: > > 1. Precisely when were these chip drivers marked obsolete? > > Since kernel 2.4.11-pre4, released Thu, 4 Oct 2001 20:47:23 -0700.
Good. That's when they were first marked 'theoretically obsolete', at least. Now, when did they _actually_ become obsolete, and why?
> > 2. What was the reason for marking them obsolete? > > The changelog says: > - David Woodhouse: large MTD and JFFS[2] update
I didn't ask who; I knew that. I asked you _why_. Admittedly, I happen to know that too - but I want to know if _you_ know it.
Since you've taken it upon yourself to decide the timescale of the removal, surely it's reasonable to expect that you do actually know what you're removing and why it's obsolescent?
> > 3. What are the factors which led you to conclude that _now_ is the time > > to actually remove them? > > http://lkml.org/lkml/2005/12/12/43
That's also an insufficient answer. Why now? What are the factors you weighed up in favour of removal, and against it? I assume you gave it some serious consideration?
> > 4. What are the factors which led you to _remove_ the map drivers which > > currently use the obsolete chip drivers, rather than taking the obvious > > alternative solution for those map drivers? > > It seems that for one and a half years noone considered it a problem > that they were no longer available...
Even if the chip drivers in question really been unavailable for that period of time, rather than merely hidden behind an extra config option -- people with embedded systems are often slow to update. That isn't really sufficient reason for removing those map drivers rather than taking the obvious alternative solution. You do know enough about the code you're changing to know what that alternative solution is, right?
-- dwmw2
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |