[coreboot] flashrom: Remove dead M-Systems Disk on Chip code

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Sun Jun 22 16:40:03 CEST 2008

On 22.06.2008 12:24, Stefan Reinauer wrote:
> Peter Stuge wrote:
>> Patch attached.
>> //Peter
>> flashrom: Remove dead M-Systems Disk on Chip code
>> DOC support has been disabled by default for many years. The write function
>> does nothing but print text. It has a call to write_page_md2802() commented
>> out, but that function does not exist. This is dead code with ugly #ifdefs.
>> Updates README to reflect that there was a time when there was code, but it
>> didn't work. Removes M-Systems #defines and also includes svn rm msys_doc.*
>> Signed-off-by: Peter Stuge <peter at stuge.se>
> Absolutely NACK.
> The reason the code is not compiled in per default is that the probing
> takes forever. Still, no reason to drop support.

No, the flashrom DoC code in subversion never worked. The write
functions were commented out already at the time flashrom was checked in
into the v2 tree at the end of the year 2003. The code in flashrom has
been useless since forever. By the way, I checked the DoC code in
another repository (v1 flash_and_burn), and that code also never worked.

> I don't think we should drop features because we don't like a few ifdefs.

Agreed. Dropping code that _never_ worked is very reasonable, though.



More information about the coreboot mailing list