Re: [PATCH] 2.4.19pre9 in pdc202xx.c bug

["Hank Yang" <hanky@promise.com.tw>]

Dears.

    There is something wrong in drivers/ide/pdc202xx.c ide driver.
Andre Hedrick has merged ide stuff to 2.4.18 kernel that released for
RedHat 7.3, SuSE 8.0 and Mandrake 8.2. That has a bug inside to
harmful our company.

First, in pdc202xx_dmaproc() function.

Our source code is:
unsigned long atapi_port =high_16+ 0x20 + (hwif->channel ? 0x04 : 0x00);

2.4.19pre8(RedHat 7.3, SuSE 8.0 and Mandrake 8.2) is:
unsigned long atapi_reg = high_16 + (hwif->channel ? 0x24 : 0x00);

The Primary channel get wrong address, So this cause our PDC20265/67
couldn't
work on Primary channel with LBA48 drives.
Andre, I think this is your mistake, Could you please kind to fix it?

Second, our PDC20262 controller do not support LBA48 with hardware48hack
function. Please fix it in pdc202xx_dmaproc() function also.

Third, I told Alan before.
In pdc202xx_new_tune_chipset() function.
We need to set timing for only ATA133 drives exist on both channel.
If not ATA133 drives exists, we do not need to set timing here.
Please update this function also.

Fourth.
I hope can add quirk drives list to pdc202xx.c below
"QUANTUM FIREBALLP KA9.1"
"QUANTUM FIREBALLP KX13.6"
please append these if you are in available.

The last.
I hope you guys could help us to notice RedHat, SuSE, Mandrake and
etc to fix this bug as soon as possible.

Thanks in advance and sincerely
Hank Yang