iv/linux
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

mtd: nand: qcom: support for NAND controller properties

Browse Source 
Currently driver data is being assigned directly with ECC modes.
Now, the plan is to add more NAND controller versions which will
have different properties. This patch reorganizes the current driver
data assignment by creating NAND controller properties structure
which will contain all properties specific to NAND controller.

Reviewed-by: Archit Taneja <architt@codeaurora.org>
Signed-off-by: Abhishek Sahu <absahu@codeaurora.org>
Signed-off-by: Boris Brezillon <boris.brezillon@free-electrons.com>
This commit is contained in:
Abhishek Sahu 2017-08-11 17:09:17 +05:30 committed by Boris Brezillon
parent d8a9b320a2
commit 58f1f22ac3
1 changed files with 19 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
drivers/mtd/nand/qcom_nandc.c
View File

@ -240,7 +240,7 @@ struct nandc_regs {
* writes. contains the register values to be * writes. contains the register values to be
* written to controller * written to controller
* @cmd1/vld: some fixed controller register values * @cmd1/vld: some fixed controller register values
* @ecc_modes: supported ECC modes by the current controller, * @props: properties of current NAND controller,
* initialized via DT match data * initialized via DT match data
*/ */
struct qcom_nand_controller { struct qcom_nand_controller {
@ -271,7 +271,7 @@ struct qcom_nand_controller {
struct nandc_regs *regs; struct nandc_regs *regs;
u32 cmd1, vld; u32 cmd1, vld;
u32 ecc_modes; const struct qcom_nandc_props *props;
}; };
/* /*
@ -324,6 +324,15 @@ struct qcom_nand_host {
u32 clrreadstatus; u32 clrreadstatus;
}; };
/*
* This data type corresponds to the NAND controller properties which varies
* among different NAND controllers.
* @ecc_modes - ecc mode for NAND
*/
struct qcom_nandc_props {
u32 ecc_modes;
};
static inline struct qcom_nand_host *to_qcom_nand_host(struct nand_chip *chip) static inline struct qcom_nand_host *to_qcom_nand_host(struct nand_chip *chip)
{ {
return container_of(chip, struct qcom_nand_host, chip); return container_of(chip, struct qcom_nand_host, chip);
@ -1824,7 +1833,7 @@ static int qcom_nand_host_setup(struct qcom_nand_host *host)
* uses lesser bytes for ECC. If RS is used, the ECC bytes is * uses lesser bytes for ECC. If RS is used, the ECC bytes is
* always 10 bytes * always 10 bytes
*/ */
if (nandc->ecc_modes & ECC_BCH_4BIT) { if (nandc->props->ecc_modes & ECC_BCH_4BIT) {
/* BCH */ /* BCH */
host->bch_enabled = true; host->bch_enabled = true;
ecc_mode = 0; ecc_mode = 0;
@ -2171,7 +2180,7 @@ static int qcom_nandc_probe(struct platform_device *pdev)
return -ENODEV; return -ENODEV;
} }
nandc->ecc_modes = (unsigned long)dev_data; nandc->props = dev_data;
res = platform_get_resource(pdev, IORESOURCE_MEM, 0); res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
nandc->base = devm_ioremap_resource(dev, res); nandc->base = devm_ioremap_resource(dev, res);
@ -2240,15 +2249,18 @@ static int qcom_nandc_remove(struct platform_device *pdev)
return 0; return 0;
} }
#define EBI2_NANDC_ECC_MODES (ECC_RS_4BIT | ECC_BCH_8BIT) static const struct qcom_nandc_props ipq806x_nandc_props = {
.ecc_modes = (ECC_RS_4BIT | ECC_BCH_8BIT),
};
/* /*
* data will hold a struct pointer containing more differences once we support * data will hold a struct pointer containing more differences once we support
* more controller variants * more controller variants
*/ */
static const struct of_device_id qcom_nandc_of_match[] = { static const struct of_device_id qcom_nandc_of_match[] = {
{ .compatible = "qcom,ipq806x-nand", {
.data = (void *)EBI2_NANDC_ECC_MODES, .compatible = "qcom,ipq806x-nand",
.data = &ipq806x_nandc_props,
}, },
{} {}
}; };