From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754818AbdCIOKZ (ORCPT ); Thu, 9 Mar 2017 09:10:25 -0500 Received: from pandora.armlinux.org.uk ([78.32.30.218]:48590 "EHLO pandora.armlinux.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753690AbdCIOKY (ORCPT ); Thu, 9 Mar 2017 09:10:24 -0500 Date: Thu, 9 Mar 2017 13:57:15 +0000 From: Russell King - ARM Linux To: Sergei Shtylyov Cc: Sekhar Nori , Bartlomiej Zolnierkiewicz , 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 Subject: Re: [RESEND PATCH] arm: assabet_defconfig: disable IDE subsystem Message-ID: <20170309135714.GJ21222@n2100.armlinux.org.uk> References: <1467991490-24895-1-git-send-email-b.zolnierkie@samsung.com> <8ea5971a-338d-e8b9-6be8-8fe2658fc04d@ti.com> <1709119.SQyio4Ct6n@amdc3058> <1fd840c5-b0bb-726b-6357-669977c4ae5f@ti.com> <98031160-f7b4-3cff-043a-1f2ee62c8a01@ti.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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. Yes, it's very obscure - but what it comes down to is the silly way the "ISA" IO is handled on the platform (which depends whether it's an 8-bit or 16-bit ISA-style peripheral.) The old IDE CS driver _could_ be bent to work via a patch like this: ARM: EBSA110: Enable 16-bit IO type for ide-iops diff --git a/arch/arm/Makefile b/arch/arm/Makefile index 047a20780fc1..2eb501d6b5fe 100644 --- a/arch/arm/Makefile +++ b/arch/arm/Makefile @@ -217,7 +217,8 @@ ifeq ($(CONFIG_ARCH_EBSA110),y) # This is what happens if you forget the IOCS16 line. # PCMCIA cards stop working. CFLAGS_3c589_cs.o :=-DISA_SIXTEEN_BIT_PERIPHERAL -export CFLAGS_3c589_cs.o +CFLAGS_ide-io-std.o :=-DISA_SIXTEEN_BIT_PERIPHERAL +export CFLAGS_3c589_cs.o CFLAGS_ide-io-std.o endif # The byte offset of the kernel image in RAM from the start of RAM. but the PATA driver is much harder. That said, the EBSA110 is now a prime museum piece, and towards the end of the time I was using it, the kernel was becoming more and more inefficient on the hardware - as in, it appeared it could route network packets _or_ run userspace, but not both, even with drivers converted to NAPI. -- RMK's Patch system: http://www.armlinux.org.uk/developer/patches/ FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up according to speedtest.net.