On Sun, May 17, 2015 at 05:41:04PM +0200, Hauke Mehrtens wrote:
> This driver registers at the bcma bus and drives the NAND core if it
> was found on this bus. The bcma bus with this NAND core is used on the
> bcm53xx and bcm47xx Northstar SoC with ARM CPU cores. The memory ranges
> are automatically detected by bcma and the irq numbers read from device
> tree by bcma bus driver.
If you're going to use device tree for part of it (IRQs) why not the
whole thing?
> This is based on the iproc driver.
This looks like you could probably get by with just using iproc_nand.c
as-is. The main NAND core is apparently MMIO-accessible on your chips,
so aren't the BCMA bits you're touching also?
> Signed-off-by: Hauke Mehrtens <hauke@xxxxxxxxxx>
> ---
> drivers/mtd/nand/Kconfig | 8 ++
> drivers/mtd/nand/brcmnand/Makefile | 1 +
> drivers/mtd/nand/brcmnand/bcma_nand.c | 153 ++++++++++++++++++++++++++++++++++
> 3 files changed, 162 insertions(+)
> create mode 100644 drivers/mtd/nand/brcmnand/bcma_nand.c
>
> diff --git a/drivers/mtd/nand/Kconfig b/drivers/mtd/nand/Kconfig
> index 376b538..b698212 100644
> --- a/drivers/mtd/nand/Kconfig
> +++ b/drivers/mtd/nand/Kconfig
> @@ -402,6 +402,14 @@ config MTD_NAND_BRCMNAND
> originally designed for Set-Top Box but is used on various BCM7xxx,
> BCM3xxx, BCM63xxx, iProc/Cygnus and more.
>
> +config MTD_NAND_BRCMNAND_BCMA
> + tristate "Broadcom BCMA NAND controller"
> + depends on ARM || MIPS
depends on BCMA ?
> + select MTD_NAND_BRCMNAND
> + help
> + Enables the driver which registers Broadcom NAND controller against
> + bcma.
If you're going to add new Kconfigs for brcmnand, let's add a Kconfig
file under drivers/mtd/nand/brcmnand/.
Brian
> +
> config MTD_NAND_BCM47XXNFLASH
> tristate "Support for NAND flash on BCM4706 BCMA bus"
> depends on BCMA_NFLASH
> diff --git a/drivers/mtd/nand/brcmnand/Makefile b/drivers/mtd/nand/brcmnand/Makefile
> index 3b1fbfd..8483984 100644
> --- a/drivers/mtd/nand/brcmnand/Makefile
> +++ b/drivers/mtd/nand/brcmnand/Makefile
> @@ -1,6 +1,7 @@
> # link order matters; don't link the more generic brcmstb_nand.o before the
> # more specific iproc_nand.o, for instance
> obj-$(CONFIG_MTD_NAND_BRCMNAND) += iproc_nand.o
> +obj-$(CONFIG_MTD_NAND_BRCMNAND_BCMA) += bcma_nand.o
> obj-$(CONFIG_MTD_NAND_BRCMNAND) += bcm63138_nand.o
> obj-$(CONFIG_MTD_NAND_BRCMNAND) += brcmstb_nand.o
> obj-$(CONFIG_MTD_NAND_BRCMNAND) += brcmnand.o
> diff --git a/drivers/mtd/nand/brcmnand/bcma_nand.c b/drivers/mtd/nand/brcmnand/bcma_nand.c
> new file mode 100644
> index 0000000..58380bd
> --- /dev/null
> +++ b/drivers/mtd/nand/brcmnand/bcma_nand.c
> @@ -0,0 +1,153 @@
> +/*
> + * Copyright © 2015 Broadcom Corporation
> + * Copyright 2015 Hauke Mehrtens <hauke@xxxxxxxxxx>
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2 as
> + * published by the Free Software Foundation.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> + * GNU General Public License for more details.
> + */
> +
> +#include <linux/device.h>
> +#include <linux/io.h>
> +#include <linux/ioport.h>
> +#include <linux/module.h>
> +#include <linux/bcma/bcma.h>
> +#include <linux/slab.h>
> +
> +#include "brcmnand.h"
> +
> +struct bcma_nand_soc_priv {
> + struct brcmnand_soc soc;
> + struct bcma_device *core;
> + spinlock_t idm_lock;
> +};
> +
> +#define BCMA_NAND_CTLR_READY_OFFSET 0x0f10
> +#define BCMA_NAND_CTLR_READY BIT(0)
> +
> +#define BCMA_NAND_APB_LE_MODE BIT(24)
> +#define BCMA_NAND_INT_CTRL_READ_ENABLE BIT(6)
> +
> +static bool bcma_nand_intc_ack(struct brcmnand_soc *soc)
> +{
> + struct bcma_nand_soc_priv *priv = soc->priv;
> + struct bcma_device *core = priv->core;
> +
> + u32 val = bcma_read32(core, BCMA_NAND_CTLR_READY_OFFSET);
> +
> + if (val & BCMA_NAND_CTLR_READY) {
> + bcma_write32(core, BCMA_NAND_CTLR_READY_OFFSET,
> + BCMA_NAND_CTLR_READY);
> + return true;
> + }
> +
> + return false;
> +}
> +
> +static void bcma_nand_intc_set(struct brcmnand_soc *soc, bool en)
> +{
> + struct bcma_nand_soc_priv *priv = soc->priv;
> + struct bcma_device *core = priv->core;
> + u32 val;
> + unsigned long flags;
> +
> + spin_lock_irqsave(&priv->idm_lock, flags);
> +
> + val = bcma_aread32(core, BCMA_IOCTL);
> +
> + if (en)
> + val |= BCMA_NAND_INT_CTRL_READ_ENABLE;
> + else
> + val &= ~BCMA_NAND_INT_CTRL_READ_ENABLE;
> +
> + bcma_awrite32(core, BCMA_IOCTL, val);
> +
> + spin_unlock_irqrestore(&priv->idm_lock, flags);
> +}
> +
> +static void bcma_nand_apb_access(struct brcmnand_soc *soc, bool prepare)
> +{
> + struct bcma_nand_soc_priv *priv = soc->priv;
> + struct bcma_device *core = priv->core;
> + u32 val;
> + unsigned long flags;
> +
> + spin_lock_irqsave(&priv->idm_lock, flags);
> +
> + val = bcma_aread32(core, BCMA_IOCTL);
> +
> + if (prepare)
> + val |= BCMA_NAND_APB_LE_MODE;
> + else
> + val &= ~BCMA_NAND_APB_LE_MODE;
> +
> + bcma_awrite32(core, BCMA_IOCTL, val);
> +
> + spin_unlock_irqrestore(&priv->idm_lock, flags);
> +}
> +
> +static int bcma_nand_probe(struct bcma_device *core)
> +{
> + struct device *dev = &core->dev;
> + struct bcma_nand_soc_priv *priv;
> + struct brcmnand_soc *soc;
> +
> + priv = devm_kzalloc(dev, sizeof(*priv), GFP_KERNEL);
> + if (!priv)
> + return -ENOMEM;
> +
> + priv->core = core;
> +
> + soc = &priv->soc;
> + soc->nand_base = core->io_addr;
> + soc->irq = bcma_core_irq(core, 4);
> +
> + soc->dev = dev;
> + soc->priv = priv;
> + soc->ctlrdy_ack = bcma_nand_intc_ack;
> + soc->ctlrdy_set_enabled = bcma_nand_intc_set;
> + soc->prepare_data_bus = bcma_nand_apb_access;
> +
> + bcma_core_enable(core, 0);
> + return brcmnand_probe_dev(dev, soc);
> +}
> +
> +static void bcma_nand_remove(struct bcma_device *core)
> +{
> + brcmnand_remove_dev(&core->dev);
> +}
> +
> +static const struct bcma_device_id bcma_nand_tbl[] = {
> + BCMA_CORE(BCMA_MANUF_BCM, BCMA_CORE_NS_NAND, BCMA_ANY_REV, BCMA_ANY_CLASS),
> + {},
> +};
> +MODULE_DEVICE_TABLE(bcma, bcma_nand_tbl);
> +
> +static struct bcma_driver bcma_nand_driver = {
> + .name = KBUILD_MODNAME,
> + .id_table = bcma_nand_tbl,
> + .probe = bcma_nand_probe,
> + .remove = bcma_nand_remove,
> +};
> +
> +static int __init bcma_nand_init(void)
> +{
> + return bcma_driver_register(&bcma_nand_driver);
> +}
> +
> +static void __exit bcma_nand_exit(void)
> +{
> + bcma_driver_unregister(&bcma_nand_driver);
> +}
> +
> +module_init(bcma_nand_init)
> +module_exit(bcma_nand_exit)
> +
> +MODULE_LICENSE("GPL v2");
> +MODULE_AUTHOR("Hauke Mehrtens");
> +MODULE_DESCRIPTION("NAND driver for Broadcom bcma based SoCs");
> --
> 2.1.4
>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
