Commit dd2dcc00 authored by Rafał Miłecki's avatar Rafał Miłecki Committed by Boris Brezillon

of: mtd: prepare helper reading NAND ECC algo from DT

NAND subsystem is being slightly reworked to store ECC details in
separated fields. In future we'll want to add support for more DT
properties as specifying every possible setup with a single
"nand-ecc-mode" is a pretty bad idea.
To allow this let's add a helper that will support something like
"nand-ecc-algo" in future. Right now we use it for keeping backward
compatibility.
Signed-off-by: default avatarRafał Miłecki <zajec5@gmail.com>
Signed-off-by: default avatarBoris Brezillon <boris.brezillon@free-electrons.com>
parent b0fcd8ab
...@@ -49,6 +49,42 @@ int of_get_nand_ecc_mode(struct device_node *np) ...@@ -49,6 +49,42 @@ int of_get_nand_ecc_mode(struct device_node *np)
} }
EXPORT_SYMBOL_GPL(of_get_nand_ecc_mode); EXPORT_SYMBOL_GPL(of_get_nand_ecc_mode);
/**
* of_get_nand_ecc_algo - Get nand ecc algorithm for given device_node
* @np: Pointer to the given device_node
*
* The function gets ecc algorithm and returns its enum value, or errno in error
* case.
*/
int of_get_nand_ecc_algo(struct device_node *np)
{
const char *pm;
int err;
/*
* TODO: Read ECC algo OF property and map it to enum nand_ecc_algo.
* It's not implemented yet as currently NAND subsystem ignores
* algorithm explicitly set this way. Once it's handled we should
* document & support new property.
*/
/*
* For backward compatibility we also read "nand-ecc-mode" checking
* for some obsoleted values that were specifying ECC algorithm.
*/
err = of_property_read_string(np, "nand-ecc-mode", &pm);
if (err < 0)
return err;
if (!strcasecmp(pm, "soft"))
return NAND_ECC_HAMMING;
else if (!strcasecmp(pm, "soft_bch"))
return NAND_ECC_BCH;
return -ENODEV;
}
EXPORT_SYMBOL_GPL(of_get_nand_ecc_algo);
/** /**
* of_get_nand_ecc_step_size - Get ECC step size associated to * of_get_nand_ecc_step_size - Get ECC step size associated to
* the required ECC strength (see below). * the required ECC strength (see below).
......
...@@ -13,6 +13,7 @@ ...@@ -13,6 +13,7 @@
#include <linux/of.h> #include <linux/of.h>
int of_get_nand_ecc_mode(struct device_node *np); int of_get_nand_ecc_mode(struct device_node *np);
int of_get_nand_ecc_algo(struct device_node *np);
int of_get_nand_ecc_step_size(struct device_node *np); int of_get_nand_ecc_step_size(struct device_node *np);
int of_get_nand_ecc_strength(struct device_node *np); int of_get_nand_ecc_strength(struct device_node *np);
int of_get_nand_bus_width(struct device_node *np); int of_get_nand_bus_width(struct device_node *np);
...@@ -25,6 +26,11 @@ static inline int of_get_nand_ecc_mode(struct device_node *np) ...@@ -25,6 +26,11 @@ static inline int of_get_nand_ecc_mode(struct device_node *np)
return -ENOSYS; return -ENOSYS;
} }
static inline int of_get_nand_ecc_algo(struct device_node *np)
{
return -ENOSYS;
}
static inline int of_get_nand_ecc_step_size(struct device_node *np) static inline int of_get_nand_ecc_step_size(struct device_node *np)
{ {
return -ENOSYS; return -ENOSYS;
......
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