Commit f5619f37 authored by Florian Fainelli's avatar Florian Fainelli Committed by Miquel Raynal
Browse files

mtd: rawnand: brcmnand: Allow working without interrupts 



The BCMA devices include the brcmnand controller but they do not wire up
any interrupt line, allow the main interrupt to be optional and update
the completion path to also check for the lack of an interrupt line.

Signed-off-by: default avatarFlorian Fainelli <f.fainelli@gmail.com>
Signed-off-by: default avatarMiquel Raynal <miquel.raynal@bootlin.com>
Link: https://lore.kernel.org/linux-mtd/20220107184614.2670254-6-f.fainelli@gmail.com
parent 75ac9447

drivers/mtd/nand/raw/brcmnand/brcmnand.c

+24 −28
Original line number Diff line number Diff line
@@ -216,7 +216,7 @@ struct brcmnand_controller { 
	void __iomem		*nand_base;

	void __iomem		*nand_fc; /* flash cache */

	void __iomem		*flash_dma_base;

	unsigned int		irq;

	int			irq;

	unsigned int		dma_irq;

	int			nand_version;
@@ -1610,7 +1610,7 @@ static bool brcmstb_nand_wait_for_completion(struct nand_chip *chip) 
	bool err = false;

	int sts;



	if (mtd->oops_panic_write) {

	if (mtd->oops_panic_write || ctrl->irq < 0) {

		/* switch to interrupt polling and PIO mode */

		disable_ctrl_irqs(ctrl);

		sts = bcmnand_ctrl_poll_status(ctrl, NAND_CTRL_RDY,
@@ -3144,13 +3144,8 @@ int brcmnand_probe(struct platform_device *pdev, struct brcmnand_soc *soc) 
	}



	/* IRQ */

	ctrl->irq = platform_get_irq(pdev, 0);

	if ((int)ctrl->irq < 0) {

		dev_err(dev, "no IRQ defined\n");

		ret = -ENODEV;

		goto err;

	}



	ctrl->irq = platform_get_irq_optional(pdev, 0);

	if (ctrl->irq > 0) {

		/*

		 * Some SoCs integrate this controller (e.g., its interrupt bits) in

		 * interesting ways
@@ -3172,6 +3167,7 @@ int brcmnand_probe(struct platform_device *pdev, struct brcmnand_soc *soc) 
				ctrl->irq, ret);

			goto err;

		}

	}



	for_each_available_child_of_node(dn, child) {

		if (of_device_is_compatible(child, "brcm,nandcs")) {