Commit 11ab03e1 authored by Thomas Petazzoni's avatar Thomas Petazzoni Committed by Ben Hutchings

mtd: nand: fsmc: fix NAND width handling

commit ee56874f upstream.

In commit eea62819 ("mtd: Add device-tree support to fsmc_nand"),
Device Tree support was added to the fmsc_nand driver. However, this
code has a bug in how it handles the bank-width DT property to set the
bus width.

Indeed, in the function fsmc_nand_probe_config_dt() that parses the
Device Tree, it sets pdata->width to either 8 or 16 depending on the
value of the bank-width DT property.

Then, the ->probe() function will test if pdata->width is equal to
FSMC_NAND_BW16 (which is 2) to set NAND_BUSWIDTH_16 in
nand->options. Therefore, with the DT probing, this condition will never
match.

This commit fixes that by removing the "width" field from
fsmc_nand_platform_data and instead have the fsmc_nand_probe_config_dt()
function directly set the appropriate nand->options value.

It is worth mentioning that if this commit gets backported to older
kernels, prior to the drop of non-DT probing, then non-DT probing will
be broken because nand->options will no longer be set to
NAND_BUSWIDTH_16.

Fixes: eea62819 ("mtd: Add device-tree support to fsmc_nand")
Signed-off-by: default avatarThomas Petazzoni <thomas.petazzoni@free-electrons.com>
Reviewed-by: default avatarLinus Walleij <linus.walleij@linaro.org>
Signed-off-by: default avatarBoris Brezillon <boris.brezillon@free-electrons.com>
[bwh: Backported to 3.16: keep fsmc_and_platform_data::width and the
 test in fsmc_nand_probe()]
Signed-off-by: default avatarBen Hutchings <ben@decadent.org.uk>
parent c45304b8
...@@ -874,18 +874,19 @@ static int fsmc_nand_probe_config_dt(struct platform_device *pdev, ...@@ -874,18 +874,19 @@ static int fsmc_nand_probe_config_dt(struct platform_device *pdev,
struct fsmc_nand_platform_data *pdata = dev_get_platdata(&pdev->dev); struct fsmc_nand_platform_data *pdata = dev_get_platdata(&pdev->dev);
u32 val; u32 val;
/* Set default NAND width to 8 bits */ pdata->options = 0;
pdata->width = 8;
if (!of_property_read_u32(np, "bank-width", &val)) { if (!of_property_read_u32(np, "bank-width", &val)) {
if (val == 2) { if (val == 2) {
pdata->width = 16; pdata->options |= NAND_BUSWIDTH_16;
} else if (val != 1) { } else if (val != 1) {
dev_err(&pdev->dev, "invalid bank-width %u\n", val); dev_err(&pdev->dev, "invalid bank-width %u\n", val);
return -EINVAL; return -EINVAL;
} }
} }
if (of_get_property(np, "nand-skip-bbtscan", NULL)) if (of_get_property(np, "nand-skip-bbtscan", NULL))
pdata->options = NAND_SKIP_BBTSCAN; pdata->options |= NAND_SKIP_BBTSCAN;
pdata->nand_timings = devm_kzalloc(&pdev->dev, pdata->nand_timings = devm_kzalloc(&pdev->dev,
sizeof(*pdata->nand_timings), GFP_KERNEL); sizeof(*pdata->nand_timings), GFP_KERNEL);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment