From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bartlomiej Zolnierkiewicz Subject: Re: [RESEND PATCH] arm: assabet_defconfig: disable IDE subsystem Date: Thu, 09 Mar 2017 15:57:37 +0100 Message-ID: <3001681.2O13sqAj0p@amdc3058> References: <1467991490-24895-1-git-send-email-b.zolnierkie@samsung.com> <3285453.g1rFzTv5DW@amdc3058> <20170309145147.GL21222@n2100.armlinux.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7Bit Return-path: In-reply-to: <20170309145147.GL21222@n2100.armlinux.org.uk> Sender: linux-kernel-owner@vger.kernel.org To: Russell King - ARM Linux Cc: Sergei Shtylyov , Sekhar Nori , Arnd Bergmann , Dmitry Eremin-Solenikov , Kevin Hilman , linux-kernel@vger.kernel.org, linux-ide@vger.kernel.org, Olof Johansson , linux-arm-kernel@lists.infradead.org List-Id: linux-ide@vger.kernel.org On Thursday, March 09, 2017 02:51:47 PM Russell King - ARM Linux wrote: > On Thu, Mar 09, 2017 at 03:42:49PM +0100, Bartlomiej Zolnierkiewicz wrote: > > > > Hi, > > > > On Thursday, March 09, 2017 01:57:15 PM Russell King - ARM Linux wrote: > > > On Thu, Mar 09, 2017 at 03:24:56PM +0300, Sergei Shtylyov wrote: > > > > On 03/09/2017 03:20 PM, Sekhar Nori wrote: > > > > > > > > >[...] > > > > > > > > >>>diff --git a/arch/arm/mach-davinci/board-dm644x-evm.c > > > > >>>b/arch/arm/mach-davinci/board-dm644x-evm.c > > > > >>>index 023480b75244..60a1f23890cd 100644 > > > > >>>--- a/arch/arm/mach-davinci/board-dm644x-evm.c > > > > >>>+++ b/arch/arm/mach-davinci/board-dm644x-evm.c > > > > >>>@@ -744,7 +744,7 @@ static int davinci_phy_fixup(struct phy_device > > > > >>>*phydev) > > > > >>> return 0; > > > > >>> } > > > > >>> > > > > >>>-#define HAS_ATA IS_ENABLED(CONFIG_BLK_DEV_PALMCHIP_BK3710) > > > > >>>+#define HAS_ATA IS_ENABLED(CONFIG_PATA_BK3710) > > > > >> > > > > >> I think it would be more correct to check for both libata and IDE > > > > >>drivers here... > > > > > > > > > >As I understand, the plan is to remove the IDE driver soon. > > > > > > > > I'm not sure DaveM would support any removals in drivers/ide/. He has > > > > explicitly expressed his will to maintain the IDE driuver forever. > > > > > > As I keep saying, there are ARM machines where the IDE driver works but > > > the PATA driver doesn't. EBSA110's PCMCIA slots for instance. > > > > Please be more specific, I'll be happy to help with migrating this > > machines to libata PATA. > > > > BTW the example you're giving has never worked with the old IDE > > driver in the upstream kernel. > > That's where you are wrong. It worked for 10+ years with no > modifications in the upstream IDE driver or ide-cs driver - only the > patch I quoted was required. ..and the patch you quoted is in which upstream kernel version exactly? Best regards, -- Bartlomiej Zolnierkiewicz Samsung R&D Institute Poland Samsung Electronics