Re: [PATCH 2.4] decrypt/update ide help entries

[Bartlomiej Zolnierkiewicz <B.Zolnierkiewicz@elka.pw.edu.pl>]

On Tuesday 18 of May 2004 15:42, O.Sezer wrote:
> Bartlomiej Zolnierkiewicz wrote:
> > This patch was disccussed long time ago and nobody cared to correct it.
>
> Hmm.. too many errors, careless duplicate entries.. ...
> too many trust in trusted trees.. Me embarassed ;)

hehe

> > +  "Override-Enable UDMA for Promise Contr." (or "Special UDMA Feature")
> > +  to force UDMA mode for connected UDMA capable disk drives.
> >
> >  It is about forcing burst UDMA transfers not UDMA mode.
>
> Fixed

OK

> > +PROMISE PDC202{68|69|70|71|75|76|77} support
> > +CONFIG_BLK_DEV_PDC202XX_NEW
>
> [...]
>
> > This is just copied from CONFIG_BLK_DEV_PDC202XX_OLD
> > ('Ultra33') and probably is incorrect for newer Promise controllers.
>
> Removed the old copied one wrote something generic

OK

> > +  You need to say Y here if you have a PDC20276 IDE interface but either
> > +  you do not have a RAID disk array, or you wish to use the Linux
> > +  internal RAID software (/dev/mdX).
> >
> > This is needed not only for PDC20276.
>
> I think this time I took the correct chipset names upon reading
> pdc202XX_old.h and pdc202XX_new.h. Please check.

OK

> > +  You need to say N here if you wish to use your Promise controller to
> > +  control a FastTrak RAID disk array, and you you must also say Y to
> > +  CONFIG_BLK_DEV_ATARAID_PDC.
> >
> > This is incorrect.
> >
> > You must say Y to this option and to CONFIG_BLK_DEV_ATARAID_PDC.
>
> Whoops, sorry. Fixed.

+  Setting this option causes the kernel to use your Promise IDE disk
+  controller as an ordinary IDE controller, rather than as a FastTrak
+  RAID controller (RAID is a system for using multiple physical disks
+  as one virtual disk).
+
+  You need to say Y here if you have one of the above mentioned IDE
+  interfaces,  but either you do not have a RAID disk array,  or you
+  wish to use the Linux internal RAID software (/dev/mdX).
+
+  If you wish to use your Promise controller to control a FastTrak
+  RAID disk array, you need to say Y here AND you you must also say Y
+  to CONFIG_BLK_DEV_ATARAID_PDC.

This sounds awful. ;-)

Use controller even if marked as disabled by BIOS
CONFIG_PDC202XX_FORCE

Say Y unless you want to use Promise proprietary RAID driver.

Makes a lot more sense IMHO.
BTW it needs to be split-up for old and new driver (hint, hint!).

> > If you want to correct Promise IDE help entries, do it for 2.6 first.
>
> Don't know much about 2.6; if you can review this one, I can make
> similar changes for 2.6 (in case options didn't change much).

No, they are the same (see drivers/ide/Kconfig).

Thanks,
Bartlomiej