From mboxrd@z Thu Jan 1 00:00:00 1970 From: sergei.shtylyov@cogentembedded.com (Sergei Shtylyov) Date: Thu, 9 Mar 2017 12:25:47 +0300 Subject: [RESEND PATCH] arm: assabet_defconfig: disable IDE subsystem In-Reply-To: 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> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hello! On 3/9/2017 11:39 AM, Sekhar Nori wrote: >>> It took a while to get to it but here is the draft driver patch >>> against v4.11-rc1. Please test. >> >> I tested this on DM6446 EVM. I was able to mount existing partitions on >> the hard disk and see that the directory listing looks good[1]. I will do >> more tests (including comparing performance with old driver) tomorrow. I > > I completed the tests I wanted to, including some read/write/data > integrity tests. Performance is same as before too. > > Tested-by: Sekhar Nori > > Here are the additional changes I did. These changes do not clash with > what I have already queued for v4.12. That said, its probably better if > I carry the platform pieces through my tree. Let me know how you want > to proceed. > > Thanks, > Sekhar > > ---8<--- > diff --git a/arch/arm/configs/davinci_all_defconfig b/arch/arm/configs/davinci_all_defconfig > index b9a7cb98ffda..67db82999c06 100644 > --- a/arch/arm/configs/davinci_all_defconfig > +++ b/arch/arm/configs/davinci_all_defconfig [...] > 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... > #define HAS_NOR IS_ENABLED(CONFIG_MTD_PHYSMAP) > > diff --git a/arch/arm/mach-davinci/board-dm646x-evm.c b/arch/arm/mach-davinci/board-dm646x-evm.c > index f702d4fc8eb8..589f3c33c4a0 100644 > --- a/arch/arm/mach-davinci/board-dm646x-evm.c > +++ b/arch/arm/mach-davinci/board-dm646x-evm.c > @@ -119,7 +119,7 @@ static struct platform_device davinci_nand_device = { > }, > }; > > -#define HAS_ATA IS_ENABLED(CONFIG_BLK_DEV_PALMCHIP_BK3710) > +#define HAS_ATA IS_ENABLED(CONFIG_PATA_BK3710) And here... > > #ifdef CONFIG_I2C > /* CPLD Register 0 bits to control ATA */ > diff --git a/arch/arm/mach-davinci/board-neuros-osd2.c b/arch/arm/mach-davinci/board-neuros-osd2.c > index 0a7838852649..075e304ce7be 100644 > --- a/arch/arm/mach-davinci/board-neuros-osd2.c > +++ b/arch/arm/mach-davinci/board-neuros-osd2.c > @@ -163,7 +163,7 @@ static struct davinci_mmc_config davinci_ntosd2_mmc_config = { > .wires = 4, > }; > > -#define HAS_ATA IS_ENABLED(CONFIG_BLK_DEV_PALMCHIP_BK3710) > +#define HAS_ATA IS_ENABLED(CONFIG_PATA_BK3710) And here... [...] MBR, Sergei