Re: disable_ide_dma gcc-3.0 warn

[Ingo Oeser]

On Mon, Apr 23, 2001 at 11:07:53AM +0200, J . A . Magallon wrote:
> In dmi_scan.c there is the func:
> static __init int disable_ide_dma(struct dmi_blacklist *d)
> 
> But now it is unused (intentionally ?):
> 
> static __initdata struct dmi_blacklist dmi_blacklist[]={
> #if 0    <==================
>     { disable_ide_dma, "KT7", { /* Overbroad right now - kill DMA on problem KT7
> boards */
>             MATCH(DMI_PRODUCT_NAME, "KT7-RAID"),
>             NO_MATCH, NO_MATCH, NO_MATCH
>             } },
> #endif  

I guess this is a leftover from the VIA buggy southbridge workaround hunt.

Code marked with "#if 0" and "#if 1" is usally under
development and subject to changes. 

Or left intentionally in "#if 0" to show the reader that we had
an wrong idea once, which seemed to be obviously correct (may be
from the docs) and we solved it with a different method, which is
not obvious or even not stated in the docs, but is the right one.

So we avoid stupid patches by leaving such things for reference.

In short: Don't care about new dead code too much, if it will be
   needed by a "#if 0" marked code section.

BTW: Which revision of gcc 3.0 do you use? I had no luck compiling
   it yet. Please answer in private to gcc issues.

Regards

Ingo Oeser
-- 
10.+11.03.2001 - 3. Chemnitzer LinuxTag <http://www.tu-chemnitz.de/linux/tag>
         <<<<<<<<<<<<     been there and had much fun   >>>>>>>>>>>>