Re: [RFT PATCH] ia64: generic_defconfig: convert to use libata PATA drivers

[Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>]

Hi,

On Monday, August 17, 2015 07:19:06 PM Sergei Shtylyov wrote:
> Hello.
> 
> On 08/17/2015 06:46 PM, Bartlomiej Zolnierkiewicz wrote:
> 
> > IDE subsystem has been deprecated since 2009 and the majority
> > (if not all) of Linux distributions have switched to use
> > libata for ATA support exclusively.  However there are still
> > some users (mostly old or/and embedded non-x86 systems) that
> > have not converted from using IDE subsystem to libata PATA
> > drivers.  This doesn't seem to be good thing in the long-term
> > for Linux as while there is less and less PATA systems left
> > in use:
> >
> > * testing efforts are divided between two subsystems
> >
> > * having duplicate drivers for same hardware confuses users
> >
> > This patch converts generic_defconfig to use libata PATA
> > drivers (sgiioc4 IDE host driver has no corresponding libata
> > driver yet so it is not converted).
> >
> > Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> > ---
> > Build tested only.
> > If you have affected hardware please test.  Thank you.
> >
> >   arch/ia64/configs/generic_defconfig | 8 ++++----
> >   1 file changed, 4 insertions(+), 4 deletions(-)
> >
> > diff --git a/arch/ia64/configs/generic_defconfig b/arch/ia64/configs/generic_defconfig
> > index 81f686d..f55be82 100644
> > --- a/arch/ia64/configs/generic_defconfig
> > +++ b/arch/ia64/configs/generic_defconfig
> [...]
> > @@ -62,6 +59,9 @@ CONFIG_SCSI_QLOGIC_1280=y
> >   CONFIG_ATA=y
> >   CONFIG_ATA_PIIX=y
> >   CONFIG_SATA_VITESSE=y
> > +CONFIG_PATA_CMD64X=y
> > +CONFIG_PATA_MPIIX=y
> 
>     If you're enabling the MPIIX support (which I doubt is necessary), why not 

I also doubt that it is necessary but IDE's piix host driver
includes support for it (so after conversion MPIIX is also
enabled).

> enable the "old" PIIX support?

Oversight on my side, will fix.

Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R&D Institute Poland
Samsung Electronics