From mboxrd@z Thu Jan 1 00:00:00 1970 Message-ID: <5284DFE7.70308@ti.com> Date: Thu, 14 Nov 2013 09:36:23 -0500 From: Santosh Shilimkar MIME-Version: 1.0 To: "Nori, Sekhar" Subject: Re: [PATCH 10/12] mtd: nand: davinci: don't set timings if AEMIF is used References: <1384187188-5776-1-git-send-email-ivan.khoronzhuk@ti.com>, <1384187188-5776-11-git-send-email-ivan.khoronzhuk@ti.com> <4F5844B3A985794BA902E12C070812375F8D53@DNCE04.ent.ti.com> <528307F0.6000805@ti.com> <52838941.4060207@ti.com> <5284ABB9.70608@ti.com> In-Reply-To: <5284ABB9.70608@ti.com> Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Cc: Mark Rutland , "devicetree@vger.kernel.org" , "Strashko, Grygorii" , Russell King , Pawel Moll , Stephen Warren , Ian Campbell , Kumar Gala , Rob Herring , "linux-kernel@vger.kernel.org" , "linux-mtd@lists.infradead.org" , Rob Landley , "Khoronzhuk, Ivan" , "linux-arm-kernel@lists.infradead.org" List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Thursday 14 November 2013 05:53 AM, Nori, Sekhar wrote: > On Wednesday 13 November 2013 07:44 PM, Santosh Shilimkar wrote: >> On Wednesday 13 November 2013 12:02 AM, Sekhar Nori wrote: >>> On Monday 11 November 2013 10:40 PM, Khoronzhuk, Ivan wrote: >>>> If Davinci AEMIF is used we don't need to set timings and bus width. >>>> It is done by AEMIF driver (drivers/memory/davinci-aemfi.c). >>>> >>>> Signed-off-by: Ivan Khoronzhuk >>>> --- >>>> drivers/mtd/nand/davinci_nand.c | 22 +++++++++++++++------- >>>> 1 file changed, 15 insertions(+), 7 deletions(-) >>>> >>>> diff --git a/drivers/mtd/nand/davinci_nand.c b/drivers/mtd/nand/davinci_nand.c >>>> index 4705214..879e915 100644 >>>> --- a/drivers/mtd/nand/davinci_nand.c >>>> +++ b/drivers/mtd/nand/davinci_nand.c >>>> @@ -742,27 +742,35 @@ static int __init nand_davinci_probe(struct platform_device *pdev) >>>> goto err_clk_enable; >>>> } >>>> >>>> +#if !IS_ENABLED(CONFIG_TI_DAVINCI_AEMIF) >>> >>> This is a hack! Just because AEMIF driver is enabled, it does not >>> guarantee that the timings have been setup by it. Instead of configuring >>> timings in two drivers, why not just convert everyone over to use the >>> new driver. Dont worry about breaking old platforms, I will help test >>> and ack them. >>> >> How about you take a stab at and convert the DaVinci code over to make >> use of new driver. We are trying to re-use as much as possible from the >> common drivers and also making an option so that the DaVinci arch can >> move over to these drivers if they want to. > > Sure I could. > Thanks > Ivan, > > The AEMIF driver does not use platfrom_data currently. Is that something > you can add or you want me to take a stab at that as well? > The AEMIF new driver is device tree only and thats the direction. So the better thing would be to convert Davinci to DT and then make use of this new driver. Thats how most of the new drivers used on arm socs are moving. Adding platform data to new driver is a step in backward direction so I would want to avoid that. Once you start using new driver the $subject patch code skipping is trivial. Till then I would like to proceed with current approach which allows Nand support for Keystone. Regards, Santosh From mboxrd@z Thu Jan 1 00:00:00 1970 From: santosh.shilimkar@ti.com (Santosh Shilimkar) Date: Thu, 14 Nov 2013 09:36:23 -0500 Subject: [PATCH 10/12] mtd: nand: davinci: don't set timings if AEMIF is used In-Reply-To: <5284ABB9.70608@ti.com> References: <1384187188-5776-1-git-send-email-ivan.khoronzhuk@ti.com>, <1384187188-5776-11-git-send-email-ivan.khoronzhuk@ti.com> <4F5844B3A985794BA902E12C070812375F8D53@DNCE04.ent.ti.com> <528307F0.6000805@ti.com> <52838941.4060207@ti.com> <5284ABB9.70608@ti.com> Message-ID: <5284DFE7.70308@ti.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Thursday 14 November 2013 05:53 AM, Nori, Sekhar wrote: > On Wednesday 13 November 2013 07:44 PM, Santosh Shilimkar wrote: >> On Wednesday 13 November 2013 12:02 AM, Sekhar Nori wrote: >>> On Monday 11 November 2013 10:40 PM, Khoronzhuk, Ivan wrote: >>>> If Davinci AEMIF is used we don't need to set timings and bus width. >>>> It is done by AEMIF driver (drivers/memory/davinci-aemfi.c). >>>> >>>> Signed-off-by: Ivan Khoronzhuk >>>> --- >>>> drivers/mtd/nand/davinci_nand.c | 22 +++++++++++++++------- >>>> 1 file changed, 15 insertions(+), 7 deletions(-) >>>> >>>> diff --git a/drivers/mtd/nand/davinci_nand.c b/drivers/mtd/nand/davinci_nand.c >>>> index 4705214..879e915 100644 >>>> --- a/drivers/mtd/nand/davinci_nand.c >>>> +++ b/drivers/mtd/nand/davinci_nand.c >>>> @@ -742,27 +742,35 @@ static int __init nand_davinci_probe(struct platform_device *pdev) >>>> goto err_clk_enable; >>>> } >>>> >>>> +#if !IS_ENABLED(CONFIG_TI_DAVINCI_AEMIF) >>> >>> This is a hack! Just because AEMIF driver is enabled, it does not >>> guarantee that the timings have been setup by it. Instead of configuring >>> timings in two drivers, why not just convert everyone over to use the >>> new driver. Dont worry about breaking old platforms, I will help test >>> and ack them. >>> >> How about you take a stab at and convert the DaVinci code over to make >> use of new driver. We are trying to re-use as much as possible from the >> common drivers and also making an option so that the DaVinci arch can >> move over to these drivers if they want to. > > Sure I could. > Thanks > Ivan, > > The AEMIF driver does not use platfrom_data currently. Is that something > you can add or you want me to take a stab at that as well? > The AEMIF new driver is device tree only and thats the direction. So the better thing would be to convert Davinci to DT and then make use of this new driver. Thats how most of the new drivers used on arm socs are moving. Adding platform data to new driver is a step in backward direction so I would want to avoid that. Once you start using new driver the $subject patch code skipping is trivial. Till then I would like to proceed with current approach which allows Nand support for Keystone. Regards, Santosh From mboxrd@z Thu Jan 1 00:00:00 1970 From: Santosh Shilimkar Subject: Re: [PATCH 10/12] mtd: nand: davinci: don't set timings if AEMIF is used Date: Thu, 14 Nov 2013 09:36:23 -0500 Message-ID: <5284DFE7.70308@ti.com> References: <1384187188-5776-1-git-send-email-ivan.khoronzhuk@ti.com>, <1384187188-5776-11-git-send-email-ivan.khoronzhuk@ti.com> <4F5844B3A985794BA902E12C070812375F8D53@DNCE04.ent.ti.com> <528307F0.6000805@ti.com> <52838941.4060207@ti.com> <5284ABB9.70608@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <5284ABB9.70608-l0cyMroinI0@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: "Nori, Sekhar" Cc: "Khoronzhuk, Ivan" , Rob Landley , Russell King , Mark Rutland , "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "Strashko, Grygorii" , Pawel Moll , Stephen Warren , Ian Campbell , Kumar Gala , Rob Herring , "linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "linux-mtd-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" , "linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" List-Id: devicetree@vger.kernel.org On Thursday 14 November 2013 05:53 AM, Nori, Sekhar wrote: > On Wednesday 13 November 2013 07:44 PM, Santosh Shilimkar wrote: >> On Wednesday 13 November 2013 12:02 AM, Sekhar Nori wrote: >>> On Monday 11 November 2013 10:40 PM, Khoronzhuk, Ivan wrote: >>>> If Davinci AEMIF is used we don't need to set timings and bus width. >>>> It is done by AEMIF driver (drivers/memory/davinci-aemfi.c). >>>> >>>> Signed-off-by: Ivan Khoronzhuk >>>> --- >>>> drivers/mtd/nand/davinci_nand.c | 22 +++++++++++++++------- >>>> 1 file changed, 15 insertions(+), 7 deletions(-) >>>> >>>> diff --git a/drivers/mtd/nand/davinci_nand.c b/drivers/mtd/nand/davinci_nand.c >>>> index 4705214..879e915 100644 >>>> --- a/drivers/mtd/nand/davinci_nand.c >>>> +++ b/drivers/mtd/nand/davinci_nand.c >>>> @@ -742,27 +742,35 @@ static int __init nand_davinci_probe(struct platform_device *pdev) >>>> goto err_clk_enable; >>>> } >>>> >>>> +#if !IS_ENABLED(CONFIG_TI_DAVINCI_AEMIF) >>> >>> This is a hack! Just because AEMIF driver is enabled, it does not >>> guarantee that the timings have been setup by it. Instead of configuring >>> timings in two drivers, why not just convert everyone over to use the >>> new driver. Dont worry about breaking old platforms, I will help test >>> and ack them. >>> >> How about you take a stab at and convert the DaVinci code over to make >> use of new driver. We are trying to re-use as much as possible from the >> common drivers and also making an option so that the DaVinci arch can >> move over to these drivers if they want to. > > Sure I could. > Thanks > Ivan, > > The AEMIF driver does not use platfrom_data currently. Is that something > you can add or you want me to take a stab at that as well? > The AEMIF new driver is device tree only and thats the direction. So the better thing would be to convert Davinci to DT and then make use of this new driver. Thats how most of the new drivers used on arm socs are moving. Adding platform data to new driver is a step in backward direction so I would want to avoid that. Once you start using new driver the $subject patch code skipping is trivial. Till then I would like to proceed with current approach which allows Nand support for Keystone. Regards, Santosh -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754077Ab3KNOiN (ORCPT ); Thu, 14 Nov 2013 09:38:13 -0500 Received: from comal.ext.ti.com ([198.47.26.152]:48315 "EHLO comal.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753361Ab3KNOiI (ORCPT ); Thu, 14 Nov 2013 09:38:08 -0500 Message-ID: <5284DFE7.70308@ti.com> Date: Thu, 14 Nov 2013 09:36:23 -0500 From: Santosh Shilimkar User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/17.0 Thunderbird/17.0 MIME-Version: 1.0 To: "Nori, Sekhar" CC: "Khoronzhuk, Ivan" , Rob Landley , Russell King , Mark Rutland , "devicetree@vger.kernel.org" , "Strashko, Grygorii" , Pawel Moll , Stephen Warren , Ian Campbell , Kumar Gala , Rob Herring , "linux-kernel@vger.kernel.org" , "linux-mtd@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 References: <1384187188-5776-1-git-send-email-ivan.khoronzhuk@ti.com>, <1384187188-5776-11-git-send-email-ivan.khoronzhuk@ti.com> <4F5844B3A985794BA902E12C070812375F8D53@DNCE04.ent.ti.com> <528307F0.6000805@ti.com> <52838941.4060207@ti.com> <5284ABB9.70608@ti.com> In-Reply-To: <5284ABB9.70608@ti.com> Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thursday 14 November 2013 05:53 AM, Nori, Sekhar wrote: > On Wednesday 13 November 2013 07:44 PM, Santosh Shilimkar wrote: >> On Wednesday 13 November 2013 12:02 AM, Sekhar Nori wrote: >>> On Monday 11 November 2013 10:40 PM, Khoronzhuk, Ivan wrote: >>>> If Davinci AEMIF is used we don't need to set timings and bus width. >>>> It is done by AEMIF driver (drivers/memory/davinci-aemfi.c). >>>> >>>> Signed-off-by: Ivan Khoronzhuk >>>> --- >>>> drivers/mtd/nand/davinci_nand.c | 22 +++++++++++++++------- >>>> 1 file changed, 15 insertions(+), 7 deletions(-) >>>> >>>> diff --git a/drivers/mtd/nand/davinci_nand.c b/drivers/mtd/nand/davinci_nand.c >>>> index 4705214..879e915 100644 >>>> --- a/drivers/mtd/nand/davinci_nand.c >>>> +++ b/drivers/mtd/nand/davinci_nand.c >>>> @@ -742,27 +742,35 @@ static int __init nand_davinci_probe(struct platform_device *pdev) >>>> goto err_clk_enable; >>>> } >>>> >>>> +#if !IS_ENABLED(CONFIG_TI_DAVINCI_AEMIF) >>> >>> This is a hack! Just because AEMIF driver is enabled, it does not >>> guarantee that the timings have been setup by it. Instead of configuring >>> timings in two drivers, why not just convert everyone over to use the >>> new driver. Dont worry about breaking old platforms, I will help test >>> and ack them. >>> >> How about you take a stab at and convert the DaVinci code over to make >> use of new driver. We are trying to re-use as much as possible from the >> common drivers and also making an option so that the DaVinci arch can >> move over to these drivers if they want to. > > Sure I could. > Thanks > Ivan, > > The AEMIF driver does not use platfrom_data currently. Is that something > you can add or you want me to take a stab at that as well? > The AEMIF new driver is device tree only and thats the direction. So the better thing would be to convert Davinci to DT and then make use of this new driver. Thats how most of the new drivers used on arm socs are moving. Adding platform data to new driver is a step in backward direction so I would want to avoid that. Once you start using new driver the $subject patch code skipping is trivial. Till then I would like to proceed with current approach which allows Nand support for Keystone. Regards, Santosh