[PATCH] nand: brcmnand: add iproc support
Michael Nazzareno Trimarchi
michael at amarulasolutions.com
Wed Mar 8 21:13:02 CET 2023
Hi
On Sun, Jan 22, 2023 at 12:45 AM Linus Walleij <linus.walleij at linaro.org> wrote:
>
> Add support for the iproc Broadcom NAND controller,
> used in Northstar SoCs for example. Based on the Linux
> driver.
>
> Cc: Philippe Reynes <philippe.reynes at softathome.com>
> Signed-off-by: Linus Walleij <linus.walleij at linaro.org>
> ---
> drivers/mtd/nand/raw/Kconfig | 7 +
> drivers/mtd/nand/raw/brcmnand/Makefile | 1 +
> drivers/mtd/nand/raw/brcmnand/iproc_nand.c | 141 +++++++++++++++++++++
> 3 files changed, 149 insertions(+)
> create mode 100644 drivers/mtd/nand/raw/brcmnand/iproc_nand.c
>
> diff --git a/drivers/mtd/nand/raw/Kconfig b/drivers/mtd/nand/raw/Kconfig
> index 5b35da45f584..6a13bc1e228a 100644
> --- a/drivers/mtd/nand/raw/Kconfig
> +++ b/drivers/mtd/nand/raw/Kconfig
> @@ -156,6 +156,13 @@ config NAND_BRCMNAND_63158
> help
> Enable support for broadcom nand driver on bcm63158.
>
> +config NAND_BRCMNAND_IPROC
> + bool "Support Broadcom NAND controller on the iproc family"
> + depends on NAND_BRCMNAND
> + help
> + Enable support for broadcom nand driver on the Broadcom
> + iproc family such as Northstar (BCM5301x, BCM4708...)
> +
> config NAND_DAVINCI
> bool "Support TI Davinci NAND controller"
> select SYS_NAND_SELF_INIT if TARGET_DA850EVM
> diff --git a/drivers/mtd/nand/raw/brcmnand/Makefile b/drivers/mtd/nand/raw/brcmnand/Makefile
> index f46a7edae321..0c6325aaa618 100644
> --- a/drivers/mtd/nand/raw/brcmnand/Makefile
> +++ b/drivers/mtd/nand/raw/brcmnand/Makefile
> @@ -6,5 +6,6 @@ obj-$(CONFIG_NAND_BRCMNAND_6753) += bcm6753_nand.o
> obj-$(CONFIG_NAND_BRCMNAND_68360) += bcm68360_nand.o
> obj-$(CONFIG_NAND_BRCMNAND_6838) += bcm6838_nand.o
> obj-$(CONFIG_NAND_BRCMNAND_6858) += bcm6858_nand.o
> +obj-$(CONFIG_NAND_BRCMNAND_IPROC) += iproc_nand.o
> obj-$(CONFIG_NAND_BRCMNAND) += brcmnand.o
> obj-$(CONFIG_NAND_BRCMNAND) += brcmnand_compat.o
> diff --git a/drivers/mtd/nand/raw/brcmnand/iproc_nand.c b/drivers/mtd/nand/raw/brcmnand/iproc_nand.c
> new file mode 100644
> index 000000000000..c61a120b6d27
> --- /dev/null
> +++ b/drivers/mtd/nand/raw/brcmnand/iproc_nand.c
> @@ -0,0 +1,141 @@
> +// SPDX-License-Identifier: GPL-2.0
> +/*
> + * Code borrowed from the Linux driver
> + * Copyright (C) 2015 Broadcom Corporation
> + */
> +
> +#include <common.h>
> +#include <asm/io.h>
> +#include <memalign.h>
> +#include <nand.h>
> +#include <linux/bitops.h>
> +#include <linux/err.h>
> +#include <linux/errno.h>
> +#include <linux/io.h>
> +#include <linux/ioport.h>
> +#include <dm.h>
> +
> +#include "brcmnand.h"
> +
> +struct iproc_nand_soc {
> + struct brcmnand_soc soc;
> + void __iomem *idm_base;
> + void __iomem *ext_base;
> +};
> +
> +#define IPROC_NAND_CTLR_READY_OFFSET 0x10
> +#define IPROC_NAND_CTLR_READY BIT(0)
> +
> +#define IPROC_NAND_IO_CTRL_OFFSET 0x00
> +#define IPROC_NAND_APB_LE_MODE BIT(24)
> +#define IPROC_NAND_INT_CTRL_READ_ENABLE BIT(6)
> +
> +static bool iproc_nand_intc_ack(struct brcmnand_soc *soc)
> +{
> + struct iproc_nand_soc *priv =
> + container_of(soc, struct iproc_nand_soc, soc);
> + void __iomem *mmio = priv->ext_base + IPROC_NAND_CTLR_READY_OFFSET;
> + u32 val = brcmnand_readl(mmio);
> +
> + if (val & IPROC_NAND_CTLR_READY) {
> + brcmnand_writel(IPROC_NAND_CTLR_READY, mmio);
> + return true;
> + }
> +
> + return false;
> +}
> +
> +static void iproc_nand_intc_set(struct brcmnand_soc *soc, bool en)
> +{
> + struct iproc_nand_soc *priv =
> + container_of(soc, struct iproc_nand_soc, soc);
> + void __iomem *mmio = priv->idm_base + IPROC_NAND_IO_CTRL_OFFSET;
> + u32 val = brcmnand_readl(mmio);
> +
> + if (en)
> + val |= IPROC_NAND_INT_CTRL_READ_ENABLE;
> + else
> + val &= ~IPROC_NAND_INT_CTRL_READ_ENABLE;
> +
> + brcmnand_writel(val, mmio);
> +}
> +
> +static void iproc_nand_apb_access(struct brcmnand_soc *soc, bool prepare,
> + bool is_param)
> +{
> + struct iproc_nand_soc *priv =
> + container_of(soc, struct iproc_nand_soc, soc);
> + void __iomem *mmio = priv->idm_base + IPROC_NAND_IO_CTRL_OFFSET;
> + u32 val;
> +
> + val = brcmnand_readl(mmio);
> +
> + /*
> + * In the case of BE or when dealing with NAND data, always configure
> + * the APB bus to LE mode before accessing the FIFO and back to BE mode
> + * after the access is done
> + */
> + if (IS_ENABLED(CONFIG_SYS_BIG_ENDIAN) || !is_param) {
> + if (prepare)
> + val |= IPROC_NAND_APB_LE_MODE;
> + else
> + val &= ~IPROC_NAND_APB_LE_MODE;
> + } else { /* when in LE accessing the parameter page, keep APB in BE */
> + val &= ~IPROC_NAND_APB_LE_MODE;
> + }
> +
> + brcmnand_writel(val, mmio);
> +}
> +
> +static int iproc_nand_probe(struct udevice *dev)
> +{
> + struct udevice *pdev = dev;
> + struct iproc_nand_soc *priv = dev_get_priv(dev);
> + struct brcmnand_soc *soc;
> + struct resource res;
> +
> + soc = &priv->soc;
> +
> + dev_read_resource_byname(pdev, "iproc-idm", &res);
Check the return value of dev_read_resource
>
> + priv->idm_base = ioremap(res.start, resource_size(&res));
To be more close to linux use devm_ioremap.
> + if (IS_ERR(priv->idm_base))
> + return PTR_ERR(priv->idm_base);
> +
> + dev_read_resource_byname(pdev, "iproc-ext", &res);
Check the return value of dev_read_resource
>
> + priv->ext_base = ioremap(res.start, resource_size(&res));
To be more close to linux use devm_ioremap.
> + if (IS_ERR(priv->ext_base))
> + return PTR_ERR(priv->ext_base);
> +
> + soc->ctlrdy_ack = iproc_nand_intc_ack;
> + soc->ctlrdy_set_enabled = iproc_nand_intc_set;
> + soc->prepare_data_bus = iproc_nand_apb_access;
> +
> + return brcmnand_probe(pdev, soc);
> +}
> +
> +static const struct udevice_id iproc_nand_dt_ids[] = {
> + {
> + .compatible = "brcm,nand-iproc",
> + },
> + { /* sentinel */ }
> +};
> +
> +U_BOOT_DRIVER(iproc_nand) = {
> + .name = "iproc-nand",
> + .id = UCLASS_MTD,
> + .of_match = iproc_nand_dt_ids,
> + .probe = iproc_nand_probe,
> + .priv_auto = sizeof(struct iproc_nand_soc),
> +};
> +
> +void board_nand_init(void)
> +{
> + struct udevice *dev;
> + int ret;
> +
> + ret = uclass_get_device_by_driver(UCLASS_MTD,
> + DM_DRIVER_GET(iproc_nand), &dev);
> + if (ret && ret != -ENODEV)
> + pr_err("Failed to initialize %s. (error %d)\n", dev->name,
> + ret);
> +}
> --
> 2.39.0
>
Another then that:
Reviewed-by: Michael Trimarchi <michael at amarulasolutions.com>
More information about the U-Boot
mailing list