Re: [RESEND PATCH] arm: assabet_defconfig: disable IDE subsystem

Bartlomiej Zolnierkiewicz
2017-03-09T12:14:35Z

On Thursday, March 09, 2017 05:27:12 PM Sekhar Nori wrote:
> On Thursday 09 March 2017 05:17 PM, Bartlomiej Zolnierkiewicz wrote:
> >
> > Hi Sekhar,
> >
> > On Thursday, March 09, 2017 02:09:30 PM Sekhar Nori wrote:
> >> Hi Bartlomiej,
> >>
> >> On Wednesday 08 March 2017 08:30 PM, 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 <nsekhar@ti.com>
> >
> > Thanks for testing!
> >
> >> Here are the additional changes I did. These changes do not clash with
> >
> > I'll split the draft patch on ATA & platform changes and make a proper
> > submission. I'll also include your changes (can I have your Signed-by
> > for them?).
>
> Signed-of-by: Sekhar Nori <nsekhar@ti.com>

Thanks.

> >> 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.
> >
> > I think that it would be the best to ask Tejun to make a separate branch
> > with a pata_bk3710 driver so it can be pulled into your tree to preserve
> > bisectability.
>
> Sounds good. What about removal of IDE driver? Do you want to have both
> drivers existing in v4.12 and get rid of the IDE driver in v4.13 ?

Sounds good to me.

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