From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757690AbcBDOfl (ORCPT ); Thu, 4 Feb 2016 09:35:41 -0500 Received: from mailout1.samsung.com ([203.254.224.24]:49418 "EHLO mailout1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750727AbcBDOfh (ORCPT ); Thu, 4 Feb 2016 09:35:37 -0500 X-AuditID: cbfee61a-f79266d000003652-08-56b361b744c3 From: Bartlomiej Zolnierkiewicz To: Gerhard Pircher Cc: linuxppc-dev@lists.ozlabs.org, Benjamin Herrenschmidt , Paul Mackerras , Michael Ellerman , linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [RFT PATCH] powerpc: convert amigaone_defconfig to use libata PATA drivers Date: Thu, 04 Feb 2016 15:35:08 +0100 Message-id: <2550250.nXgBuiRPBT@amdc1976> User-Agent: KMail/4.13.3 (Linux/3.13.0-74-generic; KDE/4.13.3; x86_64; ; ) In-reply-to: <56B26E7F.70307@gmx.net> References: <1454514630-29269-1-git-send-email-b.zolnierkie@samsung.com> <1454514630-29269-12-git-send-email-b.zolnierkie@samsung.com> <56B26E7F.70307@gmx.net> MIME-version: 1.0 Content-transfer-encoding: 7Bit Content-type: text/plain; charset=us-ascii X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFnrCLMWRmVeSWpSXmKPExsVy+t9jAd0diZvDDFZZWnxoamW2WPPvGLvF sR2PmCwu75rDZvH7+z9Wiy0Nh9gs1hxZzO7A7rHoZQOLx+JN+9k8et60sHqcn7GQ0WPurj5G j8+b5ALYorhsUlJzMstSi/TtErgyZizqYC/YJl7RcGMKUwNjp3AXIyeHhICJxO/dz1khbDGJ C/fWs3UxcnEICSxllFjbtJgRwvnKKPGyeQ9YFZuAlcTE9lWMILaIgK7E7/brTCBFzAJ3GSWO Lv7MBJIQFoiUOPfqFxuIzSKgKtHetBesgVdAU+Lx8kVgg0QFvCR23NkNVM/BwQlU836JAsSy xYwSX5+vZYGoF5T4MfkemM0sIC+xb/9UVghbS2L9zuNMExgFZiEpm4WkbBaSsgWMzKsYJVIL kguKk9JzDfNSy/WKE3OLS/PS9ZLzczcxgsP+mdQOxoO73A8xCnAwKvHw3vDeFCbEmlhWXJl7 iFGCg1lJhHdZxOYwId6UxMqq1KL8+KLSnNTiQ4zSHCxK4ryP/68LExJITyxJzU5NLUgtgsky cXBKNTC6q6vOtV1w2rR7lr2Kp9+lKqbHJeevbDByrhVueO2W1vDT48nJvacTXl48KFsQFn77 wPHA7geveKI6kwLevgk7oq8YXFFyJcxTzPLq9OAnM1XZxTgXuLupX5So2JmcXnj1XSbb0g2l Kc9arBMCpfc7xxuYNYefVhP/sFm79mDUu3+LlzlP/arEUpyRaKjFXFScCABquu12dwIAAA== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, On Wednesday, February 03, 2016 10:17:51 PM Gerhard Pircher wrote: > Am 2016-02-03 um 16:50 schrieb Bartlomiej Zolnierkiewicz: > > 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 amigaone_defconfig to use libata PATA > > drivers. > > > > Signed-off-by: Bartlomiej Zolnierkiewicz > > --- > > Build tested only. > > If you have affected hardware please test. Thank you. > > > > arch/powerpc/configs/amigaone_defconfig | 10 ++++------ > > 1 file changed, 4 insertions(+), 6 deletions(-) > > > > diff --git a/arch/powerpc/configs/amigaone_defconfig b/arch/powerpc/configs/amigaone_defconfig > > index 84f1b41..55a4929 100644 > > --- a/arch/powerpc/configs/amigaone_defconfig > > +++ b/arch/powerpc/configs/amigaone_defconfig > > @@ -46,12 +46,6 @@ CONFIG_PARPORT_PC_FIFO=y > > CONFIG_BLK_DEV_FD=y > > CONFIG_BLK_DEV_LOOP=y > > CONFIG_BLK_DEV_RAM=y > > -CONFIG_IDE=y > > -CONFIG_BLK_DEV_IDECD=y > > -# CONFIG_IDEPCI_PCIBUS_ORDER is not set > > -CONFIG_BLK_DEV_GENERIC=y > > -CONFIG_BLK_DEV_SIIMAGE=y > > -CONFIG_BLK_DEV_VIA82CXXX=y > > CONFIG_SCSI=y > > CONFIG_BLK_DEV_SD=y > > CONFIG_CHR_DEV_ST=y > > @@ -62,6 +56,10 @@ CONFIG_SCSI_CONSTANTS=y > > CONFIG_SCSI_SYM53C8XX_2=y > > CONFIG_SCSI_SYM53C8XX_DMA_ADDRESSING_MODE=0 > > # CONFIG_SCSI_SYM53C8XX_MMIO is not set > > +CONFIG_ATA=y > > +CONFIG_PATA_SIL680=y > > +CONFIG_PATA_VIA=y > > +CONFIG_ATA_GENERIC=y > > CONFIG_NETDEVICES=y > > CONFIG_VORTEX=y > > CONFIG_8139CP=y > > > Thanks for cleaning up the defconfig file! > > libata drivers work fine on the amigaone platform (tested on all three > first-gen AmigaOne machines). BTW: could it be that CONFIG_ATA_SFF=y > and CONFIG_ATA_BMDMA=y are missing in the patch? Thank you for testing! When it comes to CONFIG_ATA_SFF and CONFIG_ATA_BMDMA there is no need to explicitly enable them because once CONFIG_ATA is enabled they both are also enabled by default (they both have 'default y' in Kconfig). [ defconfig changes in the patch were obtained by: - doing 'make amigaone_defconfig' - changing IDE options to libata ones using 'make menuconfig' - doing 'make savedefconfig' - doing 'diff -u arch/powerpc/configs/amigaone_defconfig defconfig' so there should be no missing options etc. ] Best regards, -- Bartlomiej Zolnierkiewicz Samsung R&D Institute Poland Samsung Electronics