From: Sekhar Nori <nsekhar@ti.com>
To: Kevin Hilman <khilman@baylibre.com>, Arnd Bergmann <arnd@arndb.de>
Cc: linux-arm-kernel@lists.infradead.org,
Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>,
Russell King <linux@arm.linux.org.uk>,
Dmitry Eremin-Solenikov <dbaryshkov@gmail.com>,
linux-kernel@vger.kernel.org, linux-ide@vger.kernel.org,
Olof Johansson <olof@lixom.net>
Subject: Re: [RESEND PATCH] arm: assabet_defconfig: disable IDE subsystem
Date: Mon, 18 Jul 2016 20:15:08 +0530 [thread overview]
Message-ID: <578CEB74.2080002@ti.com> (raw)
In-Reply-To: <m27fcn3pb1.fsf@baylibre.com>
On Friday 15 July 2016 08:45 PM, Kevin Hilman wrote:
> Arnd Bergmann <arnd@arndb.de> writes:
>
>> On Wednesday, July 13, 2016 12:59:23 PM CEST Bartlomiej Zolnierkiewicz wrote:
>>>
>>> On Friday, July 08, 2016 10:23:48 PM Arnd Bergmann wrote:
>>>> On Friday, July 8, 2016 5:24:41 PM CEST Bartlomiej Zolnierkiewicz wrote:
>>>>> This patch disables deprecated IDE subsystem in assabet_defconfig
>>>>> (no IDE host drivers are selected in this config so there is no
>>>>> valid reason to enable IDE subsystem itself).
>>>>>
>>>>> Cc: Dmitry Eremin-Solenikov <dbaryshkov@gmail.com>
>>>>> Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
>>>>
>>>> I think the series makes a lot of sense. I have checked your assertions
>>>> in the changelogs and found no flaws in your logic, so I think we should
>>>> take them all through arm-soc unless there are other concerns.
>>>
>>> Thank you.
>>>
>>> Should I resend everything or just patches that were not reposted yet
>>> (the ones that were marked as RFT initially and got no feedback)?
>>
>> I'd be fine with just getting a pull request with all the patches that
>> had no negative feedback and that were not already applied (if any).
>>
>>>> Do you have a list of ARM defconfigs that keep using CONFIG_IDE and
>>>> how you determined that they need it?
>>>
>>> The only such defconfig is davinci_all_defconfig which uses
>>> palm_bk3710 host driver (CONFIG_BLK_DEV_PALMCHIP_BK3710).
>>>
>>>> I know that ARCH_RPC/ARCH_ACORN has a couple of special drivers that
>>>> have no libata replacement, are there any others like that, or are
>>>> they all platforms that should in theory work with libata but need
>>>> testing?
>>>
>>> All platforms except ARCH_ACORN, ARCH_DAVINCI & ARCH_RPC should work
>>> with libata.
>>
>> Adding Sekhar and Kevin for DaVinci: At first sight, palm_bk3710 looks
>> fairly straightforward (meaning someone has to do a few day's work)
>> to convert into a libata driver.
>>
>> As this is on on-chip controller that is part of a dm644x and dm646x,
>> it should also not be hard to test (as long as someone can find
>> a hard drive to plug in).
>
> I have a hard drive, but don't have any dm64xx hardware anymore to test
> this. My last working dm644x board died last year.
I have a working DM6446 EVM. I was able to connect a hard drive to it
and do some basic tests with v4.6 kernel.
I will look into converting the driver to libata. Might take some time
because this is unfamiliar territory for me.
Thanks,
Sekhar
WARNING: multiple messages have this Message-ID (diff)
From: nsekhar@ti.com (Sekhar Nori)
To: linux-arm-kernel@lists.infradead.org
Subject: [RESEND PATCH] arm: assabet_defconfig: disable IDE subsystem
Date: Mon, 18 Jul 2016 20:15:08 +0530 [thread overview]
Message-ID: <578CEB74.2080002@ti.com> (raw)
In-Reply-To: <m27fcn3pb1.fsf@baylibre.com>
On Friday 15 July 2016 08:45 PM, Kevin Hilman wrote:
> Arnd Bergmann <arnd@arndb.de> writes:
>
>> On Wednesday, July 13, 2016 12:59:23 PM CEST Bartlomiej Zolnierkiewicz wrote:
>>>
>>> On Friday, July 08, 2016 10:23:48 PM Arnd Bergmann wrote:
>>>> On Friday, July 8, 2016 5:24:41 PM CEST Bartlomiej Zolnierkiewicz wrote:
>>>>> This patch disables deprecated IDE subsystem in assabet_defconfig
>>>>> (no IDE host drivers are selected in this config so there is no
>>>>> valid reason to enable IDE subsystem itself).
>>>>>
>>>>> Cc: Dmitry Eremin-Solenikov <dbaryshkov@gmail.com>
>>>>> Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
>>>>
>>>> I think the series makes a lot of sense. I have checked your assertions
>>>> in the changelogs and found no flaws in your logic, so I think we should
>>>> take them all through arm-soc unless there are other concerns.
>>>
>>> Thank you.
>>>
>>> Should I resend everything or just patches that were not reposted yet
>>> (the ones that were marked as RFT initially and got no feedback)?
>>
>> I'd be fine with just getting a pull request with all the patches that
>> had no negative feedback and that were not already applied (if any).
>>
>>>> Do you have a list of ARM defconfigs that keep using CONFIG_IDE and
>>>> how you determined that they need it?
>>>
>>> The only such defconfig is davinci_all_defconfig which uses
>>> palm_bk3710 host driver (CONFIG_BLK_DEV_PALMCHIP_BK3710).
>>>
>>>> I know that ARCH_RPC/ARCH_ACORN has a couple of special drivers that
>>>> have no libata replacement, are there any others like that, or are
>>>> they all platforms that should in theory work with libata but need
>>>> testing?
>>>
>>> All platforms except ARCH_ACORN, ARCH_DAVINCI & ARCH_RPC should work
>>> with libata.
>>
>> Adding Sekhar and Kevin for DaVinci: At first sight, palm_bk3710 looks
>> fairly straightforward (meaning someone has to do a few day's work)
>> to convert into a libata driver.
>>
>> As this is on on-chip controller that is part of a dm644x and dm646x,
>> it should also not be hard to test (as long as someone can find
>> a hard drive to plug in).
>
> I have a hard drive, but don't have any dm64xx hardware anymore to test
> this. My last working dm644x board died last year.
I have a working DM6446 EVM. I was able to connect a hard drive to it
and do some basic tests with v4.6 kernel.
I will look into converting the driver to libata. Might take some time
because this is unfamiliar territory for me.
Thanks,
Sekhar
WARNING: multiple messages have this Message-ID (diff)
From: Sekhar Nori <nsekhar@ti.com>
To: Kevin Hilman <khilman@baylibre.com>, Arnd Bergmann <arnd@arndb.de>
Cc: <linux-arm-kernel@lists.infradead.org>,
Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>,
Russell King <linux@arm.linux.org.uk>,
Dmitry Eremin-Solenikov <dbaryshkov@gmail.com>,
<linux-kernel@vger.kernel.org>, <linux-ide@vger.kernel.org>,
Olof Johansson <olof@lixom.net>
Subject: Re: [RESEND PATCH] arm: assabet_defconfig: disable IDE subsystem
Date: Mon, 18 Jul 2016 20:15:08 +0530 [thread overview]
Message-ID: <578CEB74.2080002@ti.com> (raw)
In-Reply-To: <m27fcn3pb1.fsf@baylibre.com>
On Friday 15 July 2016 08:45 PM, Kevin Hilman wrote:
> Arnd Bergmann <arnd@arndb.de> writes:
>
>> On Wednesday, July 13, 2016 12:59:23 PM CEST Bartlomiej Zolnierkiewicz wrote:
>>>
>>> On Friday, July 08, 2016 10:23:48 PM Arnd Bergmann wrote:
>>>> On Friday, July 8, 2016 5:24:41 PM CEST Bartlomiej Zolnierkiewicz wrote:
>>>>> This patch disables deprecated IDE subsystem in assabet_defconfig
>>>>> (no IDE host drivers are selected in this config so there is no
>>>>> valid reason to enable IDE subsystem itself).
>>>>>
>>>>> Cc: Dmitry Eremin-Solenikov <dbaryshkov@gmail.com>
>>>>> Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
>>>>
>>>> I think the series makes a lot of sense. I have checked your assertions
>>>> in the changelogs and found no flaws in your logic, so I think we should
>>>> take them all through arm-soc unless there are other concerns.
>>>
>>> Thank you.
>>>
>>> Should I resend everything or just patches that were not reposted yet
>>> (the ones that were marked as RFT initially and got no feedback)?
>>
>> I'd be fine with just getting a pull request with all the patches that
>> had no negative feedback and that were not already applied (if any).
>>
>>>> Do you have a list of ARM defconfigs that keep using CONFIG_IDE and
>>>> how you determined that they need it?
>>>
>>> The only such defconfig is davinci_all_defconfig which uses
>>> palm_bk3710 host driver (CONFIG_BLK_DEV_PALMCHIP_BK3710).
>>>
>>>> I know that ARCH_RPC/ARCH_ACORN has a couple of special drivers that
>>>> have no libata replacement, are there any others like that, or are
>>>> they all platforms that should in theory work with libata but need
>>>> testing?
>>>
>>> All platforms except ARCH_ACORN, ARCH_DAVINCI & ARCH_RPC should work
>>> with libata.
>>
>> Adding Sekhar and Kevin for DaVinci: At first sight, palm_bk3710 looks
>> fairly straightforward (meaning someone has to do a few day's work)
>> to convert into a libata driver.
>>
>> As this is on on-chip controller that is part of a dm644x and dm646x,
>> it should also not be hard to test (as long as someone can find
>> a hard drive to plug in).
>
> I have a hard drive, but don't have any dm64xx hardware anymore to test
> this. My last working dm644x board died last year.
I have a working DM6446 EVM. I was able to connect a hard drive to it
and do some basic tests with v4.6 kernel.
I will look into converting the driver to libata. Might take some time
because this is unfamiliar territory for me.
Thanks,
Sekhar
next prev parent reply other threads:[~2016-07-18 14:45 UTC|newest]
Thread overview: 106+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-07-08 15:24 [RESEND PATCH] arm: assabet_defconfig: disable IDE subsystem Bartlomiej Zolnierkiewicz
2016-07-08 15:24 ` Bartlomiej Zolnierkiewicz
2016-07-08 15:24 ` [RESEND PATCH] arm: badge4_defconfig: " Bartlomiej Zolnierkiewicz
2016-07-08 15:24 ` Bartlomiej Zolnierkiewicz
2016-07-08 15:24 ` Bartlomiej Zolnierkiewicz
2016-07-08 15:24 ` [RESEND PATCH] arm: cerfcube_defconfig: " Bartlomiej Zolnierkiewicz
2016-07-08 15:24 ` Bartlomiej Zolnierkiewicz
2016-07-08 15:24 ` [RESEND PATCH] arm: colibri_pxa270_defconfig: " Bartlomiej Zolnierkiewicz
2016-07-08 15:24 ` Bartlomiej Zolnierkiewicz
2016-07-16 9:01 ` Robert Jarzmik
2016-07-16 9:01 ` Robert Jarzmik
2016-07-08 15:24 ` [RESEND PATCH] arm: lart_defconfig: " Bartlomiej Zolnierkiewicz
2016-07-08 15:24 ` Bartlomiej Zolnierkiewicz
2016-07-08 15:24 ` [RESEND PATCH] arm: lpda270_defconfig: " Bartlomiej Zolnierkiewicz
2016-07-08 15:24 ` Bartlomiej Zolnierkiewicz
2016-07-08 15:24 ` [PATCH v2] arm: mainstone_defconfig: " Bartlomiej Zolnierkiewicz
2016-07-08 15:24 ` Bartlomiej Zolnierkiewicz
2016-07-08 15:24 ` Bartlomiej Zolnierkiewicz
2016-07-08 15:24 ` [RESEND PATCH] arm: pxa255-idp_defconfig: " Bartlomiej Zolnierkiewicz
2016-07-08 15:24 ` Bartlomiej Zolnierkiewicz
2016-07-08 15:24 ` [RESEND PATCH] arm: shannon_defconfig: " Bartlomiej Zolnierkiewicz
2016-07-08 15:24 ` Bartlomiej Zolnierkiewicz
2016-07-08 15:24 ` [RESEND PATCH] arm: trizeps4_defconfig: " Bartlomiej Zolnierkiewicz
2016-07-08 15:24 ` Bartlomiej Zolnierkiewicz
2016-07-08 20:23 ` [RESEND PATCH] arm: assabet_defconfig: " Arnd Bergmann
2016-07-08 20:23 ` Arnd Bergmann
2016-07-08 20:23 ` Arnd Bergmann
2016-07-13 10:59 ` Bartlomiej Zolnierkiewicz
2016-07-13 10:59 ` Bartlomiej Zolnierkiewicz
2016-07-13 14:37 ` Arnd Bergmann
2016-07-13 14:37 ` Arnd Bergmann
2016-07-13 14:37 ` Arnd Bergmann
2016-07-13 17:45 ` Russell King - ARM Linux
2016-07-13 17:45 ` Russell King - ARM Linux
2016-07-14 9:59 ` Bartlomiej Zolnierkiewicz
2016-07-14 9:59 ` Bartlomiej Zolnierkiewicz
2016-07-15 15:15 ` Kevin Hilman
2016-07-15 15:15 ` Kevin Hilman
2016-07-15 15:15 ` Kevin Hilman
2016-07-18 14:45 ` Sekhar Nori [this message]
2016-07-18 14:45 ` Sekhar Nori
2016-07-18 14:45 ` Sekhar Nori
2016-12-12 12:45 ` Bartlomiej Zolnierkiewicz
2016-12-12 12:45 ` Bartlomiej Zolnierkiewicz
2016-12-12 12:45 ` Bartlomiej Zolnierkiewicz
2016-12-12 13:54 ` Sekhar Nori
2016-12-12 13:54 ` Sekhar Nori
2016-12-12 13:54 ` Sekhar Nori
2017-03-07 17:51 ` Bartlomiej Zolnierkiewicz
2017-03-07 17:51 ` Bartlomiej Zolnierkiewicz
2017-03-07 17:51 ` Bartlomiej Zolnierkiewicz
2017-03-08 15:00 ` Sekhar Nori
2017-03-08 15:00 ` Sekhar Nori
2017-03-08 15:00 ` Sekhar Nori
2017-03-09 8:39 ` Sekhar Nori
2017-03-09 8:39 ` Sekhar Nori
2017-03-09 8:39 ` Sekhar Nori
2017-03-09 9:25 ` Sergei Shtylyov
2017-03-09 9:25 ` Sergei Shtylyov
2017-03-09 12:20 ` Sekhar Nori
2017-03-09 12:20 ` Sekhar Nori
2017-03-09 12:20 ` Sekhar Nori
2017-03-09 12:24 ` Sergei Shtylyov
2017-03-09 12:24 ` Sergei Shtylyov
2017-03-09 13:57 ` Russell King - ARM Linux
2017-03-09 13:57 ` Russell King - ARM Linux
2017-03-09 13:57 ` Russell King - ARM Linux
2017-03-09 14:42 ` Bartlomiej Zolnierkiewicz
2017-03-09 14:42 ` Bartlomiej Zolnierkiewicz
2017-03-09 14:42 ` Bartlomiej Zolnierkiewicz
2017-03-09 14:51 ` Russell King - ARM Linux
2017-03-09 14:51 ` Russell King - ARM Linux
2017-03-09 14:57 ` Bartlomiej Zolnierkiewicz
2017-03-09 14:57 ` Bartlomiej Zolnierkiewicz
2017-03-09 15:08 ` Russell King - ARM Linux
2017-03-09 15:08 ` Russell King - ARM Linux
2017-03-09 15:41 ` Bartlomiej Zolnierkiewicz
2017-03-09 15:41 ` Bartlomiej Zolnierkiewicz
2017-03-09 15:41 ` Bartlomiej Zolnierkiewicz
2017-03-09 14:52 ` Bartlomiej Zolnierkiewicz
2017-03-09 14:52 ` Bartlomiej Zolnierkiewicz
2017-03-09 16:22 ` Sergei Shtylyov
2017-03-09 16:22 ` Sergei Shtylyov
2017-03-09 16:22 ` Sergei Shtylyov
2017-03-10 17:11 ` Bartlomiej Zolnierkiewicz
2017-03-10 17:11 ` Bartlomiej Zolnierkiewicz
2017-03-09 12:21 ` Bartlomiej Zolnierkiewicz
2017-03-09 12:21 ` Bartlomiej Zolnierkiewicz
2017-03-09 11:47 ` Bartlomiej Zolnierkiewicz
2017-03-09 11:47 ` Bartlomiej Zolnierkiewicz
2017-03-09 11:57 ` Sekhar Nori
2017-03-09 11:57 ` Sekhar Nori
2017-03-09 11:57 ` Sekhar Nori
2017-03-09 12:14 ` Bartlomiej Zolnierkiewicz
2017-03-09 12:14 ` Bartlomiej Zolnierkiewicz
2016-10-26 17:01 ` Bartlomiej Zolnierkiewicz
2016-10-26 17:01 ` Bartlomiej Zolnierkiewicz
2016-10-31 15:46 ` Russell King - ARM Linux
2016-10-31 15:46 ` Russell King - ARM Linux
2016-10-31 18:14 ` Bartlomiej Zolnierkiewicz
2016-10-31 18:14 ` Bartlomiej Zolnierkiewicz
2016-10-31 18:14 ` Bartlomiej Zolnierkiewicz
2016-10-31 18:24 ` Bartlomiej Zolnierkiewicz
2016-10-31 18:24 ` Bartlomiej Zolnierkiewicz
2016-11-25 22:22 ` Arnd Bergmann
2016-11-25 22:22 ` Arnd Bergmann
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=578CEB74.2080002@ti.com \
--to=nsekhar@ti.com \
--cc=arnd@arndb.de \
--cc=b.zolnierkie@samsung.com \
--cc=dbaryshkov@gmail.com \
--cc=khilman@baylibre.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-ide@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@arm.linux.org.uk \
--cc=olof@lixom.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.