diff options
| author | Artem Bityutskiy <artem.bityutskiy@linux.intel.com> | 2012-07-10 13:35:50 +0000 |
|---|---|---|
| committer | Rob Herring <rob.herring@calxeda.com> | 2012-07-10 10:32:06 -0500 |
| commit | e95d8aafa5d911bf523bc47fe89f3336eb8a1b51 (patch) | |
| tree | f9c0f35017fb68d36dbb9357682d24e000a49950 /drivers/of | |
| parent | 74a7f08448adea6cb47cd9b260c98ff168117e92 (diff) | |
| download | op-kernel-dev-e95d8aafa5d911bf523bc47fe89f3336eb8a1b51.zip op-kernel-dev-e95d8aafa5d911bf523bc47fe89f3336eb8a1b51.tar.gz | |
of: mtd: nuke useless const qualifier
This patch does the following:
-const int of_get_nand_ecc_mode(struct device_node *np)
+int of_get_nand_ecc_mode(struct device_node *np)
because:
1. it is probably just a typo?
2. it causes warnings like this when people assing the returned
value to an 'int' variable:
include/linux/of_mtd.h:14:18: warning: type qualifiers ignored on functi=
on return type [-Wignored-qualifiers]
Remove also the unnecessary "extern" qualifier to be consistent with other
declarations in this file.
Signed-off-by: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
Signed-off-by: Rob Herring <rob.herring@calxeda.com>
Diffstat (limited to 'drivers/of')
| -rw-r--r-- | drivers/of/of_mtd.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/of/of_mtd.c b/drivers/of/of_mtd.c index e7cad62..a27ec94 100644 --- a/drivers/of/of_mtd.c +++ b/drivers/of/of_mtd.c @@ -32,7 +32,7 @@ static const char *nand_ecc_modes[] = { * The function gets ecc mode string from property 'nand-ecc-mode', * and return its index in nand_ecc_modes table, or errno in error case. */ -const int of_get_nand_ecc_mode(struct device_node *np) +int of_get_nand_ecc_mode(struct device_node *np) { const char *pm; int err, i; |
