From: Santosh Shilimkar <santosh.shilimkar@ti.com>
To: Sekhar Nori <nsekhar@ti.com>
Cc: Mark Rutland <mark.rutland@arm.com>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
"Strashko, Grygorii" <grygorii.strashko@ti.com>,
Russell King <linux@arm.linux.org.uk>,
Pawel Moll <pawel.moll@arm.com>,
Stephen Warren <swarren@wwwdotorg.org>,
Ian Campbell <ijc+devicetree@hellion.org.uk>,
Kumar Gala <galak@kernel.crashing.org>,
Rob Herring <rob.herring@calxeda.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"linux-mtd@lists.infradead.org" <linux-mtd@lists.infradead.org>,
Rob Landley <rob@landley.net>,
"Khoronzhuk, Ivan" <ivan.khoronzhuk@ti.com>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH 10/12] mtd: nand: davinci: don't set timings if AEMIF is used
Date: Mon, 25 Nov 2013 19:55:57 -0500 [thread overview]
Message-ID: <5293F19D.70901@ti.com> (raw)
In-Reply-To: <5291CB01.1010506@ti.com>
On Sunday 24 November 2013 04:46 AM, Sekhar Nori wrote:
> On Friday 22 November 2013 11:08 PM, Santosh Shilimkar wrote:
>> On Thursday 21 November 2013 12:07 PM, Nori, Sekhar wrote:
>>> On 11/14/2013 8:06 PM, Santosh Shilimkar wrote:
[..]
>>> Yes, that would be the ideal way. But the reality is that there is close
>>> to zero chance of all the DaVinci platforms using AEMIF ever getting
>>> converted to DT. This means that we will never be able to get rid of
>>> this piece of code.
>>>
>> Well that means some of those platforms support will get deprecated
>> soon considering other basic frameworks like irq, timer wheel etc
>> is assuming the DT direction. But its your call because if you
>> want those platforms to live, you need to convert them to DT.
>>
>>> I do agree platform data is not the direction you want to take on ARM
>>> but at the same time its not really a deprecated mechanism as far as the
>>> broader kernel is concerned.
>>>
>>> So, can we consider adding platform data mechanism to the AEMIF driver?
>>>
>> Sorry but answer is "NO". If the above hack is really bothering you,
>> we can actually rip out the above code in question and move it
>> to mach-davinci. Infact thats the best direction to make the
>> mtd nandn driver independent of any platform quirks(dt, pdata etc)
>>
>> Ivan already has a patch to do that and he is going to post
>> that patch. With that patch, we won't need $subject patch.
>>
>
> Okay I will wait for the patch.
>
Ivan has posted a patch already and you are copied on it.
Regards,
Santosh
WARNING: multiple messages have this Message-ID (diff)
From: santosh.shilimkar@ti.com (Santosh Shilimkar)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 10/12] mtd: nand: davinci: don't set timings if AEMIF is used
Date: Mon, 25 Nov 2013 19:55:57 -0500 [thread overview]
Message-ID: <5293F19D.70901@ti.com> (raw)
In-Reply-To: <5291CB01.1010506@ti.com>
On Sunday 24 November 2013 04:46 AM, Sekhar Nori wrote:
> On Friday 22 November 2013 11:08 PM, Santosh Shilimkar wrote:
>> On Thursday 21 November 2013 12:07 PM, Nori, Sekhar wrote:
>>> On 11/14/2013 8:06 PM, Santosh Shilimkar wrote:
[..]
>>> Yes, that would be the ideal way. But the reality is that there is close
>>> to zero chance of all the DaVinci platforms using AEMIF ever getting
>>> converted to DT. This means that we will never be able to get rid of
>>> this piece of code.
>>>
>> Well that means some of those platforms support will get deprecated
>> soon considering other basic frameworks like irq, timer wheel etc
>> is assuming the DT direction. But its your call because if you
>> want those platforms to live, you need to convert them to DT.
>>
>>> I do agree platform data is not the direction you want to take on ARM
>>> but at the same time its not really a deprecated mechanism as far as the
>>> broader kernel is concerned.
>>>
>>> So, can we consider adding platform data mechanism to the AEMIF driver?
>>>
>> Sorry but answer is "NO". If the above hack is really bothering you,
>> we can actually rip out the above code in question and move it
>> to mach-davinci. Infact thats the best direction to make the
>> mtd nandn driver independent of any platform quirks(dt, pdata etc)
>>
>> Ivan already has a patch to do that and he is going to post
>> that patch. With that patch, we won't need $subject patch.
>>
>
> Okay I will wait for the patch.
>
Ivan has posted a patch already and you are copied on it.
Regards,
Santosh
WARNING: multiple messages have this Message-ID (diff)
From: Santosh Shilimkar <santosh.shilimkar@ti.com>
To: Sekhar Nori <nsekhar@ti.com>
Cc: "Khoronzhuk, Ivan" <ivan.khoronzhuk@ti.com>,
Rob Landley <rob@landley.net>,
Russell King <linux@arm.linux.org.uk>,
Mark Rutland <mark.rutland@arm.com>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
"Strashko, Grygorii" <grygorii.strashko@ti.com>,
Pawel Moll <pawel.moll@arm.com>,
Stephen Warren <swarren@wwwdotorg.org>,
Ian Campbell <ijc+devicetree@hellion.org.uk>,
Kumar Gala <galak@kernel.crashing.org>,
Rob Herring <rob.herring@calxeda.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"linux-mtd@lists.infradead.org" <linux-mtd@lists.infradead.org>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH 10/12] mtd: nand: davinci: don't set timings if AEMIF is used
Date: Mon, 25 Nov 2013 19:55:57 -0500 [thread overview]
Message-ID: <5293F19D.70901@ti.com> (raw)
In-Reply-To: <5291CB01.1010506@ti.com>
On Sunday 24 November 2013 04:46 AM, Sekhar Nori wrote:
> On Friday 22 November 2013 11:08 PM, Santosh Shilimkar wrote:
>> On Thursday 21 November 2013 12:07 PM, Nori, Sekhar wrote:
>>> On 11/14/2013 8:06 PM, Santosh Shilimkar wrote:
[..]
>>> Yes, that would be the ideal way. But the reality is that there is close
>>> to zero chance of all the DaVinci platforms using AEMIF ever getting
>>> converted to DT. This means that we will never be able to get rid of
>>> this piece of code.
>>>
>> Well that means some of those platforms support will get deprecated
>> soon considering other basic frameworks like irq, timer wheel etc
>> is assuming the DT direction. But its your call because if you
>> want those platforms to live, you need to convert them to DT.
>>
>>> I do agree platform data is not the direction you want to take on ARM
>>> but at the same time its not really a deprecated mechanism as far as the
>>> broader kernel is concerned.
>>>
>>> So, can we consider adding platform data mechanism to the AEMIF driver?
>>>
>> Sorry but answer is "NO". If the above hack is really bothering you,
>> we can actually rip out the above code in question and move it
>> to mach-davinci. Infact thats the best direction to make the
>> mtd nandn driver independent of any platform quirks(dt, pdata etc)
>>
>> Ivan already has a patch to do that and he is going to post
>> that patch. With that patch, we won't need $subject patch.
>>
>
> Okay I will wait for the patch.
>
Ivan has posted a patch already and you are copied on it.
Regards,
Santosh
next prev parent reply other threads:[~2013-11-26 0:55 UTC|newest]
Thread overview: 200+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <1384187188-5776-1-git-send-email-ivan.khoronzhuk@ti.com>
2013-11-11 16:47 ` [PATCH 00/12] Introduce davinci AEMIF driver Khoronzhuk, Ivan
2013-11-11 16:47 ` Khoronzhuk, Ivan
2013-11-11 16:47 ` Khoronzhuk, Ivan
[not found] ` <1384187188-5776-2-git-send-email-ivan.khoronzhuk@ti.com>
2013-11-11 16:52 ` [PATCH 01/12] mtd: nand: davinci: fix driver registration Khoronzhuk, Ivan
2013-11-11 16:52 ` Khoronzhuk, Ivan
2013-11-11 16:52 ` Khoronzhuk, Ivan
2013-11-12 15:45 ` Santosh Shilimkar
2013-11-12 15:45 ` Santosh Shilimkar
2013-11-12 15:45 ` Santosh Shilimkar
[not found] ` <1384187188-5776-3-git-send-email-ivan.khoronzhuk@ti.com>
2013-11-11 16:53 ` [PATCH 02/12] mtd: nand: davinci: check required ti,davinci-chipselect property Khoronzhuk, Ivan
2013-11-11 16:53 ` Khoronzhuk, Ivan
2013-11-11 16:53 ` Khoronzhuk, Ivan
2013-11-12 15:50 ` [PATCH 02/12] mtd: nand: davinci: check required ti, davinci-chipselect property Santosh Shilimkar
2013-11-12 15:50 ` [PATCH 02/12] mtd: nand: davinci: check required ti,davinci-chipselect property Santosh Shilimkar
2013-11-12 15:50 ` [PATCH 02/12] mtd: nand: davinci: check required ti, davinci-chipselect property Santosh Shilimkar
2013-11-18 18:25 ` ivan.khoronzhuk
2013-11-18 18:25 ` [PATCH 02/12] mtd: nand: davinci: check required ti,davinci-chipselect property ivan.khoronzhuk
2013-11-18 18:25 ` [PATCH 02/12] mtd: nand: davinci: check required ti, davinci-chipselect property ivan.khoronzhuk
2013-11-18 18:25 ` ivan.khoronzhuk
[not found] ` <1384187188-5776-4-git-send-email-ivan.khoronzhuk@ti.com>
2013-11-11 16:54 ` [PATCH 03/12] mtd: nand: davinci: simplify error handling Khoronzhuk, Ivan
2013-11-11 16:54 ` Khoronzhuk, Ivan
2013-11-11 16:54 ` Khoronzhuk, Ivan
2013-11-12 15:52 ` Santosh Shilimkar
2013-11-12 15:52 ` Santosh Shilimkar
2013-11-12 15:52 ` Santosh Shilimkar
2013-11-12 15:52 ` Santosh Shilimkar
[not found] ` <1384187188-5776-5-git-send-email-ivan.khoronzhuk@ti.com>
2013-11-11 16:55 ` [PATCH 04/12] mtd: nand: davinci: move bindings under mtd Khoronzhuk, Ivan
2013-11-11 16:55 ` Khoronzhuk, Ivan
2013-11-11 16:55 ` Khoronzhuk, Ivan
2013-11-11 16:55 ` Khoronzhuk, Ivan
2013-11-12 15:53 ` Santosh Shilimkar
2013-11-12 15:53 ` Santosh Shilimkar
2013-11-12 15:53 ` Santosh Shilimkar
2013-11-12 15:53 ` Santosh Shilimkar
[not found] ` <1384187188-5776-6-git-send-email-ivan.khoronzhuk@ti.com>
2013-11-11 16:58 ` [PATCH 05/12] mtd: nand: davinci: extend description of bindings Khoronzhuk, Ivan
2013-11-11 16:58 ` Khoronzhuk, Ivan
2013-11-11 16:58 ` Khoronzhuk, Ivan
2013-11-11 16:58 ` Khoronzhuk, Ivan
2013-11-12 15:55 ` Santosh Shilimkar
2013-11-12 15:55 ` Santosh Shilimkar
2013-11-12 15:55 ` Santosh Shilimkar
2013-11-12 15:55 ` Santosh Shilimkar
[not found] ` <1384187188-5776-7-git-send-email-ivan.khoronzhuk@ti.com>
2013-11-11 17:01 ` [PATCH 06/12] mtd: nand: davinci: adjust DT properties to MTD generic Khoronzhuk, Ivan
2013-11-11 17:01 ` Khoronzhuk, Ivan
2013-11-11 17:01 ` Khoronzhuk, Ivan
2013-11-26 7:03 ` Sekhar Nori
2013-11-26 7:03 ` Sekhar Nori
2013-11-26 7:03 ` Sekhar Nori
2013-11-26 7:03 ` Sekhar Nori
2013-11-26 10:30 ` Grygorii Strashko
2013-11-26 10:30 ` Grygorii Strashko
2013-11-26 10:30 ` Grygorii Strashko
2013-11-26 10:49 ` ivan.khoronzhuk
2013-11-26 10:49 ` ivan.khoronzhuk
2013-11-26 10:49 ` ivan.khoronzhuk
2013-11-26 10:49 ` ivan.khoronzhuk
[not found] ` <1384187188-5776-8-git-send-email-ivan.khoronzhuk@ti.com>
2013-11-11 17:06 ` [PATCH 07/12] memory: davinci-aemif: introduce AEMIF driver Khoronzhuk, Ivan
2013-11-11 17:06 ` Khoronzhuk, Ivan
2013-11-11 17:06 ` Khoronzhuk, Ivan
2013-11-11 17:06 ` Khoronzhuk, Ivan
2013-11-12 16:08 ` Santosh Shilimkar
2013-11-12 16:08 ` Santosh Shilimkar
2013-11-12 16:08 ` Santosh Shilimkar
2013-11-12 16:08 ` Santosh Shilimkar
2013-11-18 19:15 ` ivan.khoronzhuk
2013-11-18 19:15 ` ivan.khoronzhuk
2013-11-18 19:15 ` ivan.khoronzhuk
2013-11-18 19:15 ` ivan.khoronzhuk
2013-11-26 7:20 ` Sekhar Nori
2013-11-26 7:20 ` Sekhar Nori
2013-11-26 7:20 ` Sekhar Nori
2013-11-26 7:20 ` Sekhar Nori
2013-11-26 15:05 ` Santosh Shilimkar
2013-11-26 15:05 ` Santosh Shilimkar
2013-11-26 15:05 ` Santosh Shilimkar
2013-11-26 15:05 ` Santosh Shilimkar
2013-11-26 17:21 ` Sekhar Nori
2013-11-26 17:21 ` Sekhar Nori
2013-11-26 17:21 ` Sekhar Nori
2013-11-26 17:21 ` Sekhar Nori
2013-11-26 18:26 ` Santosh Shilimkar
2013-11-26 18:26 ` Santosh Shilimkar
2013-11-26 18:26 ` Santosh Shilimkar
2013-11-26 18:26 ` Santosh Shilimkar
2013-11-26 18:29 ` ivan.khoronzhuk
2013-11-26 18:29 ` ivan.khoronzhuk
2013-11-26 18:29 ` ivan.khoronzhuk
2013-11-27 0:37 ` Brian Norris
2013-11-27 0:37 ` Brian Norris
2013-11-27 0:37 ` Brian Norris
2013-11-27 0:37 ` Brian Norris
2013-11-27 10:22 ` ivan.khoronzhuk
2013-11-27 10:22 ` ivan.khoronzhuk
2013-11-27 10:22 ` ivan.khoronzhuk
2013-11-26 17:44 ` ivan.khoronzhuk
2013-11-26 17:44 ` ivan.khoronzhuk
2013-11-26 17:44 ` ivan.khoronzhuk
2013-11-26 18:30 ` Santosh Shilimkar
2013-11-26 18:30 ` Santosh Shilimkar
2013-11-26 18:30 ` Santosh Shilimkar
2013-11-26 18:30 ` Santosh Shilimkar
2013-11-26 18:30 ` ivan.khoronzhuk
2013-11-26 18:30 ` ivan.khoronzhuk
2013-11-26 18:30 ` ivan.khoronzhuk
2013-11-26 18:30 ` ivan.khoronzhuk
2013-11-27 4:05 ` Sekhar Nori
2013-11-27 4:05 ` Sekhar Nori
2013-11-27 4:05 ` Sekhar Nori
2013-11-27 4:05 ` Sekhar Nori
[not found] ` <1384187188-5776-9-git-send-email-ivan.khoronzhuk@ti.com>
2013-11-11 17:09 ` [PATCH 08/12] memory: davinci-aemif: add bindings for " Khoronzhuk, Ivan
2013-11-11 17:09 ` Khoronzhuk, Ivan
2013-11-11 17:09 ` Khoronzhuk, Ivan
2013-11-11 17:09 ` Khoronzhuk, Ivan
[not found] ` <1384187188-5776-10-git-send-email-ivan.khoronzhuk@ti.com>
2013-11-11 17:09 ` [PATCH 09/12] mtd: nand: davinci: reuse driver for Keystone arch Khoronzhuk, Ivan
2013-11-11 17:09 ` Khoronzhuk, Ivan
2013-11-11 17:09 ` Khoronzhuk, Ivan
2013-11-11 17:09 ` Khoronzhuk, Ivan
2013-11-12 16:09 ` Santosh Shilimkar
2013-11-12 16:09 ` Santosh Shilimkar
2013-11-12 16:09 ` Santosh Shilimkar
[not found] ` <1384187188-5776-11-git-send-email-ivan.khoronzhuk@ti.com>
2013-11-11 17:10 ` [PATCH 10/12] mtd: nand: davinci: don't set timings if AEMIF is used Khoronzhuk, Ivan
2013-11-11 17:10 ` Khoronzhuk, Ivan
2013-11-11 17:10 ` Khoronzhuk, Ivan
2013-11-11 17:10 ` Khoronzhuk, Ivan
2013-11-12 16:13 ` Santosh Shilimkar
2013-11-12 16:13 ` Santosh Shilimkar
2013-11-12 16:13 ` Santosh Shilimkar
2013-11-12 16:13 ` Santosh Shilimkar
2013-11-18 11:35 ` Grygorii Strashko
2013-11-18 11:35 ` Grygorii Strashko
2013-11-18 11:35 ` Grygorii Strashko
2013-11-18 11:35 ` Grygorii Strashko
2013-11-18 14:08 ` Santosh Shilimkar
2013-11-18 14:08 ` Santosh Shilimkar
2013-11-18 14:08 ` Santosh Shilimkar
2013-11-18 19:32 ` ivan.khoronzhuk
2013-11-18 19:32 ` ivan.khoronzhuk
2013-11-18 19:32 ` ivan.khoronzhuk
2013-11-13 5:02 ` Sekhar Nori
2013-11-13 5:02 ` Sekhar Nori
2013-11-13 5:02 ` Sekhar Nori
2013-11-13 5:02 ` Sekhar Nori
2013-11-13 14:14 ` Santosh Shilimkar
2013-11-13 14:14 ` Santosh Shilimkar
2013-11-13 14:14 ` Santosh Shilimkar
2013-11-14 10:53 ` Sekhar Nori
2013-11-14 10:53 ` Sekhar Nori
2013-11-14 10:53 ` Sekhar Nori
2013-11-14 10:53 ` Sekhar Nori
2013-11-14 14:36 ` Santosh Shilimkar
2013-11-14 14:36 ` Santosh Shilimkar
2013-11-14 14:36 ` Santosh Shilimkar
2013-11-14 14:36 ` Santosh Shilimkar
2013-11-18 19:35 ` ivan.khoronzhuk
2013-11-18 19:35 ` ivan.khoronzhuk
2013-11-18 19:35 ` ivan.khoronzhuk
2013-11-21 17:07 ` Sekhar Nori
2013-11-21 17:07 ` Sekhar Nori
2013-11-21 17:07 ` Sekhar Nori
2013-11-21 17:07 ` Sekhar Nori
2013-11-22 17:38 ` Santosh Shilimkar
2013-11-22 17:38 ` Santosh Shilimkar
2013-11-22 17:38 ` Santosh Shilimkar
2013-11-22 17:38 ` Santosh Shilimkar
2013-11-24 9:46 ` Sekhar Nori
2013-11-24 9:46 ` Sekhar Nori
2013-11-24 9:46 ` Sekhar Nori
2013-11-24 9:46 ` Sekhar Nori
2013-11-25 20:00 ` [PATCH] ARM: davinci: aemif: get rid of davinci-nand driver dependency on aemif Ivan Khoronzhuk
2013-11-25 20:00 ` Ivan Khoronzhuk
2013-11-25 20:00 ` Ivan Khoronzhuk
2013-11-25 20:00 ` Ivan Khoronzhuk
2013-11-27 8:35 ` Sekhar Nori
2013-11-27 8:35 ` Sekhar Nori
2013-11-27 8:35 ` Sekhar Nori
2013-11-27 11:01 ` ivan.khoronzhuk
2013-11-27 11:01 ` ivan.khoronzhuk
2013-11-27 11:01 ` ivan.khoronzhuk
2013-11-27 13:07 ` Sekhar Nori
2013-11-27 13:07 ` Sekhar Nori
2013-11-27 13:07 ` Sekhar Nori
2013-11-27 13:21 ` ivan.khoronzhuk
2013-11-27 13:21 ` ivan.khoronzhuk
2013-11-27 13:21 ` ivan.khoronzhuk
2013-11-26 0:55 ` Santosh Shilimkar [this message]
2013-11-26 0:55 ` [PATCH 10/12] mtd: nand: davinci: don't set timings if AEMIF is used Santosh Shilimkar
2013-11-26 0:55 ` Santosh Shilimkar
[not found] ` <1384187188-5776-12-git-send-email-ivan.khoronzhuk@ti.com>
2013-11-11 17:12 ` [PATCH 11/12] mtd: nand: davinci: don't request AEMIF address range Khoronzhuk, Ivan
2013-11-11 17:12 ` Khoronzhuk, Ivan
2013-11-11 17:12 ` Khoronzhuk, Ivan
2013-11-12 16:15 ` Santosh Shilimkar
2013-11-12 16:15 ` Santosh Shilimkar
2013-11-12 16:15 ` Santosh Shilimkar
[not found] ` <1384187188-5776-13-git-send-email-ivan.khoronzhuk@ti.com>
2013-11-11 17:13 ` [PATCH 12/12] arm: dts: keystone: add AEMIF/NAND device entry Khoronzhuk, Ivan
2013-11-11 17:13 ` Khoronzhuk, Ivan
2013-11-11 17:13 ` Khoronzhuk, Ivan
2013-11-11 17:13 ` Khoronzhuk, Ivan
2013-11-12 16:19 ` Santosh Shilimkar
2013-11-12 16:19 ` Santosh Shilimkar
2013-11-12 16:19 ` Santosh Shilimkar
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=5293F19D.70901@ti.com \
--to=santosh.shilimkar@ti.com \
--cc=devicetree@vger.kernel.org \
--cc=galak@kernel.crashing.org \
--cc=grygorii.strashko@ti.com \
--cc=ijc+devicetree@hellion.org.uk \
--cc=ivan.khoronzhuk@ti.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mtd@lists.infradead.org \
--cc=linux@arm.linux.org.uk \
--cc=mark.rutland@arm.com \
--cc=nsekhar@ti.com \
--cc=pawel.moll@arm.com \
--cc=rob.herring@calxeda.com \
--cc=rob@landley.net \
--cc=swarren@wwwdotorg.org \
/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.