Bartlomiej Zolnierkiewicz wrote:
Bartlomiej Zolnierkiewicz wrote:
There's no reason to have the speedproc() method wrapper for the two quite
different chip families, so just get rid of it.
Signed-off-by: Sergei Shtylyov <[EMAIL PROTECTED]>
applied
I forgot to notice/mention the side effect: there would be no speedproc()
method installed if hwif->dma_base happens to be 0.
It doesn't sound too nice since ->autotune is always set and ->tuneproc
uses ->speedproc unconditionally (=> OOPS).
Yeah, I have overlooked that -- will need to be reworked.
Looks like we really need an extra
if (atp850)
choose atp850 speedproc
else
choose atp86x speedproc
Yeah, I'll add this to the initial if statement dealing with serialization.
before hwif->dma_base check... or maybe even separate ->init_hwif
methods for atp850 and atp86x.
Probably doesn't worth it...
Thanks,
Bart
WBR, Sergei
-
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
