Re: [PATCH 10/12] mtd: nand: davinci: don't set timings if AEMIF is used

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 11/18/2013 04:08 PM, Santosh Shilimkar wrote:
On Monday 18 November 2013 06:35 AM, Grygorii Strashko wrote:
On 11/12/2013 06:13 PM, Santosh Shilimkar wrote:
On Monday 11 November 2013 12:10 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 <ivan.khoronzhuk@xxxxxx>
---
   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)

Instead above #if, just use a variable.
     bool aemif = IS_ENABLED(CONFIG_TI_DAVINCI_AEMIF) and then skip
the below code. #if block in the middle of the code looks ugly.

Yes, this is the hack.
The problem is that this part of code contains the call of Davinci platform function davinci_aemif_setup_timing() which is not accessible if Kernel is built for Keystone only.
That's why "#if" has been used.

This part of code has to be removed together with Davinci aemif platform code (aemif.c), once Davinci will be converted to DT and use
new driver.

The corresponding comment can be added here. Is it ok?

Thats more or less what I proposed in another reply. Lets clearly
document it on why such a check was added.

Regards,
Santosh


Ok, I'll add more explanation on it.

--
Regards,
Ivan Khoronzhuk
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux