mediatek: fix bootup problem on mt7629 EVB
authorJohn Crispin <john@phrozen.org>
Fri, 15 Nov 2019 06:51:32 +0000 (07:51 +0100)
committerJohn Crispin <john@phrozen.org>
Fri, 15 Nov 2019 06:51:32 +0000 (07:51 +0100)
Signed-off-by: John Crispin <john@phrozen.org>
target/linux/mediatek/mt7629/config-4.19
target/linux/mediatek/patches-4.1 [new file with mode: 0644]
target/linux/mediatek/patches-4.19/0307-spi-mem-Mediatek-Add-SPI-Nand-support-for-MT7629.patch [new file with mode: 0644]

index e43f758cdf347ae92382bff5a8264f3b24a32279..f32d9d1919231004580b6e7e1954307538b72471 100644 (file)
@@ -232,8 +232,10 @@ CONFIG_MODULES_USE_ELF_REL=y
 CONFIG_MT753X_GSW=y
 CONFIG_MTD_MT81xx_NOR=y
 CONFIG_MTD_NAND=y
+CONFIG_MTD_NAND_CORE=y
 CONFIG_MTD_NAND_ECC=y
 CONFIG_MTD_NAND_MTK=y
++CONFIG_MTD_SPI_NAND=y
 CONFIG_MTD_SPI_NOR=y
 CONFIG_MTD_SPLIT_FIRMWARE=y
 CONFIG_MTD_SPLIT_FIT_FW=y
@@ -330,6 +332,11 @@ CONFIG_SGL_ALLOC=y
 CONFIG_SG_POOL=y
 CONFIG_SMP=y
 CONFIG_SMP_ON_UP=y
+CONFIG_SPI=y
+CONFIG_SPI_MASTER=y
+CONFIG_SPI_MEM=y
+# CONFIG_SPI_MT65XX is not set
+CONFIG_SPI_MTK_SNFI=y
 CONFIG_SPARSE_IRQ=y
 CONFIG_SRCU=y
 CONFIG_STACKTRACE=y
diff --git a/target/linux/mediatek/patches-4.1 b/target/linux/mediatek/patches-4.1
new file mode 100644 (file)
index 0000000..bf7c1ea
--- /dev/null
@@ -0,0 +1,534 @@
+diff -urN a/drivers/clk/clk-devres.c b/drivers/clk/clk-devres.c
+--- a/drivers/clk/clk-devres.c 2019-08-29 16:59:26.540010395 +0800
++++ b/drivers/clk/clk-devres.c 2019-08-29 17:02:09.215924786 +0800
+@@ -34,6 +34,17 @@
+ }
+ EXPORT_SYMBOL(devm_clk_get);
++struct clk *devm_clk_get_optional(struct device *dev, const char *id)
++{
++        struct clk *clk = devm_clk_get(dev, id);
++
++        if (clk == ERR_PTR(-ENOENT))
++                return NULL;
++
++        return clk;
++}
++EXPORT_SYMBOL(devm_clk_get_optional);
++
+ struct clk_bulk_devres {
+       struct clk_bulk_data *clks;
+       int num_clks;
+diff -urN a/drivers/pci/controller/pcie-mediatek.c b/drivers/pci/controller/pcie-mediatek.c
+--- a/drivers/pci/controller/pcie-mediatek.c   2019-08-29 16:59:10.520410188 +0800
++++ b/drivers/pci/controller/pcie-mediatek.c   2019-08-29 17:01:58.340199243 +0800
+@@ -15,6 +15,7 @@
+ #include <linux/irqdomain.h>
+ #include <linux/kernel.h>
+ #include <linux/msi.h>
++#include <linux/module.h>
+ #include <linux/of_address.h>
+ #include <linux/of_pci.h>
+ #include <linux/of_platform.h>
+@@ -162,6 +163,7 @@
+  * @phy: pointer to PHY control block
+  * @lane: lane count
+  * @slot: port slot
++ * @irq: GIC irq
+  * @irq_domain: legacy INTx IRQ domain
+  * @inner_domain: inner IRQ domain
+  * @msi_domain: MSI IRQ domain
+@@ -182,6 +184,7 @@
+       struct phy *phy;
+       u32 lane;
+       u32 slot;
++      int irq;
+       struct irq_domain *irq_domain;
+       struct irq_domain *inner_domain;
+       struct irq_domain *msi_domain;
+@@ -225,10 +228,8 @@
+       clk_disable_unprepare(pcie->free_ck);
+-      if (dev->pm_domain) {
+-              pm_runtime_put_sync(dev);
+-              pm_runtime_disable(dev);
+-      }
++      pm_runtime_put_sync(dev);
++      pm_runtime_disable(dev);
+ }
+ static void mtk_pcie_port_free(struct mtk_pcie_port *port)
+@@ -394,75 +395,6 @@
+       .write = mtk_pcie_config_write,
+ };
+-static int mtk_pcie_startup_port_v2(struct mtk_pcie_port *port)
+-{
+-      struct mtk_pcie *pcie = port->pcie;
+-      struct resource *mem = &pcie->mem;
+-      const struct mtk_pcie_soc *soc = port->pcie->soc;
+-      u32 val;
+-      size_t size;
+-      int err;
+-
+-      /* MT7622 platforms need to enable LTSSM and ASPM from PCIe subsys */
+-      if (pcie->base) {
+-              val = readl(pcie->base + PCIE_SYS_CFG_V2);
+-              val |= PCIE_CSR_LTSSM_EN(port->slot) |
+-                     PCIE_CSR_ASPM_L1_EN(port->slot);
+-              writel(val, pcie->base + PCIE_SYS_CFG_V2);
+-      }
+-
+-      /* Assert all reset signals */
+-      writel(0, port->base + PCIE_RST_CTRL);
+-
+-      /*
+-       * Enable PCIe link down reset, if link status changed from link up to
+-       * link down, this will reset MAC control registers and configuration
+-       * space.
+-       */
+-      writel(PCIE_LINKDOWN_RST_EN, port->base + PCIE_RST_CTRL);
+-
+-      /* De-assert PHY, PE, PIPE, MAC and configuration reset */
+-      val = readl(port->base + PCIE_RST_CTRL);
+-      val |= PCIE_PHY_RSTB | PCIE_PERSTB | PCIE_PIPE_SRSTB |
+-             PCIE_MAC_SRSTB | PCIE_CRSTB;
+-      writel(val, port->base + PCIE_RST_CTRL);
+-
+-      /* Set up vendor ID and class code */
+-      if (soc->need_fix_class_id) {
+-              val = PCI_VENDOR_ID_MEDIATEK;
+-              writew(val, port->base + PCIE_CONF_VEND_ID);
+-
+-              val = PCI_CLASS_BRIDGE_HOST;
+-              writew(val, port->base + PCIE_CONF_CLASS_ID);
+-      }
+-
+-      /* 100ms timeout value should be enough for Gen1/2 training */
+-      err = readl_poll_timeout(port->base + PCIE_LINK_STATUS_V2, val,
+-                               !!(val & PCIE_PORT_LINKUP_V2), 20,
+-                               100 * USEC_PER_MSEC);
+-      if (err)
+-              return -ETIMEDOUT;
+-
+-      /* Set INTx mask */
+-      val = readl(port->base + PCIE_INT_MASK);
+-      val &= ~INTX_MASK;
+-      writel(val, port->base + PCIE_INT_MASK);
+-
+-      /* Set AHB to PCIe translation windows */
+-      size = mem->end - mem->start;
+-      val = lower_32_bits(mem->start) | AHB2PCIE_SIZE(fls(size));
+-      writel(val, port->base + PCIE_AHB_TRANS_BASE0_L);
+-
+-      val = upper_32_bits(mem->start);
+-      writel(val, port->base + PCIE_AHB_TRANS_BASE0_H);
+-
+-      /* Set PCIe to AXI translation memory space.*/
+-      val = fls(0xffffffff) | WIN_ENABLE;
+-      writel(val, port->base + PCIE_AXI_WINDOW0);
+-
+-      return 0;
+-}
+-
+ static void mtk_compose_msi_msg(struct irq_data *data, struct msi_msg *msg)
+ {
+       struct mtk_pcie_port *port = irq_data_get_irq_chip_data(data);
+@@ -601,6 +533,27 @@
+       writel(val, port->base + PCIE_INT_MASK);
+ }
++static void mtk_pcie_irq_teardown(struct mtk_pcie *pcie)
++{
++      struct mtk_pcie_port *port, *tmp;
++
++      list_for_each_entry_safe(port, tmp, &pcie->ports, list) {
++              irq_set_chained_handler_and_data(port->irq, NULL, NULL);
++
++              if (port->irq_domain)
++                      irq_domain_remove(port->irq_domain);
++
++              if (IS_ENABLED(CONFIG_PCI_MSI)) {
++                      if (port->msi_domain)
++                              irq_domain_remove(port->msi_domain);
++                      if (port->inner_domain)
++                              irq_domain_remove(port->inner_domain);
++              }
++
++              irq_dispose_mapping(port->irq);
++      }
++}
++
+ static int mtk_pcie_intx_map(struct irq_domain *domain, unsigned int irq,
+                            irq_hw_number_t hwirq)
+ {
+@@ -630,6 +583,7 @@
+       port->irq_domain = irq_domain_add_linear(pcie_intc_node, PCI_NUM_INTX,
+                                                &intx_domain_ops, port);
++      of_node_put(pcie_intc_node);
+       if (!port->irq_domain) {
+               dev_err(dev, "failed to get INTx IRQ domain\n");
+               return -ENODEV;
+@@ -639,8 +593,6 @@
+               ret = mtk_pcie_allocate_msi_domains(port);
+               if (ret)
+                       return ret;
+-
+-              mtk_pcie_enable_msi(port);
+       }
+       return 0;
+@@ -693,7 +645,7 @@
+       struct mtk_pcie *pcie = port->pcie;
+       struct device *dev = pcie->dev;
+       struct platform_device *pdev = to_platform_device(dev);
+-      int err, irq;
++      int err;
+       err = mtk_pcie_init_irq_domain(port, node);
+       if (err) {
+@@ -701,8 +653,81 @@
+               return err;
+       }
+-      irq = platform_get_irq(pdev, port->slot);
+-      irq_set_chained_handler_and_data(irq, mtk_pcie_intr_handler, port);
++      port->irq = platform_get_irq(pdev, port->slot);
++      irq_set_chained_handler_and_data(port->irq,
++                                       mtk_pcie_intr_handler, port);
++
++      return 0;
++}
++
++static int mtk_pcie_startup_port_v2(struct mtk_pcie_port *port)
++{
++      struct mtk_pcie *pcie = port->pcie;
++      struct resource *mem = &pcie->mem;
++      const struct mtk_pcie_soc *soc = port->pcie->soc;
++      u32 val;
++      size_t size;
++      int err;
++
++      /* MT7622 platforms need to enable LTSSM and ASPM from PCIe subsys */
++      if (pcie->base) {
++              val = readl(pcie->base + PCIE_SYS_CFG_V2);
++              val |= PCIE_CSR_LTSSM_EN(port->slot) |
++                     PCIE_CSR_ASPM_L1_EN(port->slot);
++              writel(val, pcie->base + PCIE_SYS_CFG_V2);
++      }
++
++      /* Assert all reset signals */
++      writel(0, port->base + PCIE_RST_CTRL);
++
++      /*
++       * Enable PCIe link down reset, if link status changed from link up to
++       * link down, this will reset MAC control registers and configuration
++       * space.
++       */
++      writel(PCIE_LINKDOWN_RST_EN, port->base + PCIE_RST_CTRL);
++
++      /* De-assert PHY, PE, PIPE, MAC and configuration reset */
++      val = readl(port->base + PCIE_RST_CTRL);
++      val |= PCIE_PHY_RSTB | PCIE_PERSTB | PCIE_PIPE_SRSTB |
++             PCIE_MAC_SRSTB | PCIE_CRSTB;
++      writel(val, port->base + PCIE_RST_CTRL);
++
++      /* Set up vendor ID and class code */
++      if (soc->need_fix_class_id) {
++              val = PCI_VENDOR_ID_MEDIATEK;
++              writew(val, port->base + PCIE_CONF_VEND_ID);
++
++              val = PCI_CLASS_BRIDGE_PCI;
++              writew(val, port->base + PCIE_CONF_CLASS_ID);
++      }
++
++      /* 100ms timeout value should be enough for Gen1/2 training */
++      err = readl_poll_timeout(port->base + PCIE_LINK_STATUS_V2, val,
++                               !!(val & PCIE_PORT_LINKUP_V2), 20,
++                               100 * USEC_PER_MSEC);
++      if (err)
++              return -ETIMEDOUT;
++
++      /* Set INTx mask */
++      val = readl(port->base + PCIE_INT_MASK);
++      val &= ~INTX_MASK;
++      writel(val, port->base + PCIE_INT_MASK);
++
++      if (IS_ENABLED(CONFIG_PCI_MSI))
++              mtk_pcie_enable_msi(port);
++
++      /* Set AHB to PCIe translation windows */
++      size = mem->end - mem->start;
++      val = lower_32_bits(mem->start) | AHB2PCIE_SIZE(fls(size));
++      writel(val, port->base + PCIE_AHB_TRANS_BASE0_L);
++
++      val = upper_32_bits(mem->start);
++      writel(val, port->base + PCIE_AHB_TRANS_BASE0_H);
++
++      /* Set PCIe to AXI translation memory space.*/
++      val = fls(0xffffffff) | WIN_ENABLE;
++      writel(val, port->base + PCIE_AXI_WINDOW0);
+       return 0;
+ }
+@@ -903,49 +928,29 @@
+       /* sys_ck might be divided into the following parts in some chips */
+       snprintf(name, sizeof(name), "ahb_ck%d", slot);
+-      port->ahb_ck = devm_clk_get(dev, name);
+-      if (IS_ERR(port->ahb_ck)) {
+-              if (PTR_ERR(port->ahb_ck) == -EPROBE_DEFER)
+-                      return -EPROBE_DEFER;
+-
+-              port->ahb_ck = NULL;
+-      }
++      port->ahb_ck = devm_clk_get_optional(dev, name);
++      if (IS_ERR(port->ahb_ck))
++              return PTR_ERR(port->ahb_ck);
+       snprintf(name, sizeof(name), "axi_ck%d", slot);
+-      port->axi_ck = devm_clk_get(dev, name);
+-      if (IS_ERR(port->axi_ck)) {
+-              if (PTR_ERR(port->axi_ck) == -EPROBE_DEFER)
+-                      return -EPROBE_DEFER;
+-
+-              port->axi_ck = NULL;
+-      }
++      port->axi_ck = devm_clk_get_optional(dev, name);
++      if (IS_ERR(port->axi_ck))
++              return PTR_ERR(port->axi_ck);
+       snprintf(name, sizeof(name), "aux_ck%d", slot);
+-      port->aux_ck = devm_clk_get(dev, name);
+-      if (IS_ERR(port->aux_ck)) {
+-              if (PTR_ERR(port->aux_ck) == -EPROBE_DEFER)
+-                      return -EPROBE_DEFER;
+-
+-              port->aux_ck = NULL;
+-      }
++      port->aux_ck = devm_clk_get_optional(dev, name);
++      if (IS_ERR(port->aux_ck))
++              return PTR_ERR(port->aux_ck);
+       snprintf(name, sizeof(name), "obff_ck%d", slot);
+-      port->obff_ck = devm_clk_get(dev, name);
+-      if (IS_ERR(port->obff_ck)) {
+-              if (PTR_ERR(port->obff_ck) == -EPROBE_DEFER)
+-                      return -EPROBE_DEFER;
+-
+-              port->obff_ck = NULL;
+-      }
++      port->obff_ck = devm_clk_get_optional(dev, name);
++      if (IS_ERR(port->obff_ck))
++              return PTR_ERR(port->obff_ck);
+       snprintf(name, sizeof(name), "pipe_ck%d", slot);
+-      port->pipe_ck = devm_clk_get(dev, name);
+-      if (IS_ERR(port->pipe_ck)) {
+-              if (PTR_ERR(port->pipe_ck) == -EPROBE_DEFER)
+-                      return -EPROBE_DEFER;
+-
+-              port->pipe_ck = NULL;
+-      }
++      port->pipe_ck = devm_clk_get_optional(dev, name);
++      if (IS_ERR(port->pipe_ck))
++              return PTR_ERR(port->pipe_ck);
+       snprintf(name, sizeof(name), "pcie-rst%d", slot);
+       port->reset = devm_reset_control_get_optional_exclusive(dev, name);
+@@ -998,10 +1003,8 @@
+               pcie->free_ck = NULL;
+       }
+-      if (dev->pm_domain) {
+-              pm_runtime_enable(dev);
+-              pm_runtime_get_sync(dev);
+-      }
++      pm_runtime_enable(dev);
++      pm_runtime_get_sync(dev);
+       /* enable top level clock */
+       err = clk_prepare_enable(pcie->free_ck);
+@@ -1013,10 +1016,8 @@
+       return 0;
+ err_free_ck:
+-      if (dev->pm_domain) {
+-              pm_runtime_put_sync(dev);
+-              pm_runtime_disable(dev);
+-      }
++      pm_runtime_put_sync(dev);
++      pm_runtime_disable(dev);
+       return err;
+ }
+@@ -1125,34 +1126,6 @@
+       return 0;
+ }
+-static int mtk_pcie_register_host(struct pci_host_bridge *host)
+-{
+-      struct mtk_pcie *pcie = pci_host_bridge_priv(host);
+-      struct pci_bus *child;
+-      int err;
+-
+-      host->busnr = pcie->busn.start;
+-      host->dev.parent = pcie->dev;
+-      host->ops = pcie->soc->ops;
+-      host->map_irq = of_irq_parse_and_map_pci;
+-      host->swizzle_irq = pci_common_swizzle;
+-      host->sysdata = pcie;
+-
+-      err = pci_scan_root_bus_bridge(host);
+-      if (err < 0)
+-              return err;
+-
+-      pci_bus_size_bridges(host->bus);
+-      pci_bus_assign_resources(host->bus);
+-
+-      list_for_each_entry(child, &host->bus->children, node)
+-              pcie_bus_configure_settings(child);
+-
+-      pci_bus_add_devices(host->bus);
+-
+-      return 0;
+-}
+-
+ static int mtk_pcie_probe(struct platform_device *pdev)
+ {
+       struct device *dev = &pdev->dev;
+@@ -1179,7 +1152,14 @@
+       if (err)
+               goto put_resources;
+-      err = mtk_pcie_register_host(host);
++      host->busnr = pcie->busn.start;
++      host->dev.parent = pcie->dev;
++      host->ops = pcie->soc->ops;
++      host->map_irq = of_irq_parse_and_map_pci;
++      host->swizzle_irq = pci_common_swizzle;
++      host->sysdata = pcie;
++
++      err = pci_host_probe(host);
+       if (err)
+               goto put_resources;
+@@ -1192,6 +1172,80 @@
+       return err;
+ }
++
++static void mtk_pcie_free_resources(struct mtk_pcie *pcie)
++{
++      struct pci_host_bridge *host = pci_host_bridge_from_priv(pcie);
++      struct list_head *windows = &host->windows;
++
++      pci_free_resource_list(windows);
++}
++
++static int mtk_pcie_remove(struct platform_device *pdev)
++{
++      struct mtk_pcie *pcie = platform_get_drvdata(pdev);
++      struct pci_host_bridge *host = pci_host_bridge_from_priv(pcie);
++
++      pci_stop_root_bus(host->bus);
++      pci_remove_root_bus(host->bus);
++      mtk_pcie_free_resources(pcie);
++
++      mtk_pcie_irq_teardown(pcie);
++
++      mtk_pcie_put_resources(pcie);
++
++      return 0;
++}
++
++static int __maybe_unused mtk_pcie_suspend_noirq(struct device *dev)
++{
++      struct mtk_pcie *pcie = dev_get_drvdata(dev);
++      struct mtk_pcie_port *port;
++
++      if (list_empty(&pcie->ports))
++              return 0;
++
++      list_for_each_entry(port, &pcie->ports, list) {
++              clk_disable_unprepare(port->pipe_ck);
++              clk_disable_unprepare(port->obff_ck);
++              clk_disable_unprepare(port->axi_ck);
++              clk_disable_unprepare(port->aux_ck);
++              clk_disable_unprepare(port->ahb_ck);
++              clk_disable_unprepare(port->sys_ck);
++              phy_power_off(port->phy);
++              phy_exit(port->phy);
++      }
++
++      clk_disable_unprepare(pcie->free_ck);
++
++      return 0;
++}
++
++static int __maybe_unused mtk_pcie_resume_noirq(struct device *dev)
++{
++      struct mtk_pcie *pcie = dev_get_drvdata(dev);
++      struct mtk_pcie_port *port, *tmp;
++
++      if (list_empty(&pcie->ports))
++              return 0;
++
++      clk_prepare_enable(pcie->free_ck);
++
++      list_for_each_entry_safe(port, tmp, &pcie->ports, list)
++              mtk_pcie_enable_port(port);
++
++      /* In case of EP was removed while system suspend. */
++      if (list_empty(&pcie->ports))
++              clk_disable_unprepare(pcie->free_ck);
++
++      return 0;
++}
++
++static const struct dev_pm_ops mtk_pcie_pm_ops = {
++      SET_NOIRQ_SYSTEM_SLEEP_PM_OPS(mtk_pcie_suspend_noirq,
++                                    mtk_pcie_resume_noirq)
++};
++
+ static const struct mtk_pcie_soc mtk_pcie_soc_v1 = {
+       .ops = &mtk_pcie_ops,
+       .startup = mtk_pcie_startup_port,
+@@ -1220,10 +1274,13 @@
+ static struct platform_driver mtk_pcie_driver = {
+       .probe = mtk_pcie_probe,
++      .remove = mtk_pcie_remove,
+       .driver = {
+               .name = "mtk-pcie",
+               .of_match_table = mtk_pcie_ids,
+               .suppress_bind_attrs = true,
++              .pm = &mtk_pcie_pm_ops,
+       },
+ };
+-builtin_platform_driver(mtk_pcie_driver);
++module_platform_driver(mtk_pcie_driver);
++MODULE_LICENSE("GPL v2");
+diff -urN a/include/linux/clk.h b/include/linux/clk.h
+--- a/include/linux/clk.h      2019-08-29 16:59:52.335365591 +0800
++++ b/include/linux/clk.h      2019-08-29 17:02:17.107725525 +0800
+@@ -349,6 +349,17 @@
+ struct clk *devm_clk_get(struct device *dev, const char *id);
+ /**
++ * devm_clk_get_optional - lookup and obtain a managed reference to an optional
++ *                         clock producer.
++ * @dev: device for clock "consumer"
++ * @id: clock consumer ID
++ *
++ * Behaves the same as devm_clk_get() except where there is no clock producer.
++ * In this case, instead of returning -ENOENT, the function returns NULL.
++ */
++struct clk *devm_clk_get_optional(struct device *dev, const char *id);
++
++/**
+  * devm_get_clk_from_child - lookup and obtain a managed reference to a
+  *                         clock producer from child node.
+  * @dev: device for clock "consumer"
diff --git a/target/linux/mediatek/patches-4.19/0307-spi-mem-Mediatek-Add-SPI-Nand-support-for-MT7629.patch b/target/linux/mediatek/patches-4.19/0307-spi-mem-Mediatek-Add-SPI-Nand-support-for-MT7629.patch
new file mode 100644 (file)
index 0000000..72e5694
--- /dev/null
@@ -0,0 +1,187 @@
+From c813fbe806257c574240770ef716fbee19f7dbfa Mon Sep 17 00:00:00 2001
+From: Xiangsheng Hou <xiangsheng.hou@mediatek.com>
+Date: Thu, 6 Jun 2019 16:29:04 +0800
+Subject: [PATCH] spi: spi-mem: Mediatek: Add SPI Nand support for MT7629
+
+Signed-off-by: Xiangsheng Hou <xiangsheng.hou@mediatek.com>
+---
+ arch/arm/boot/dts/mt7629-rfb.dts | 45 ++++++++++++++++++++++++++++++++
+ arch/arm/boot/dts/mt7629-lynx-rfb.dts | 45 ++++++++++++++++++++++++++++++++
+ arch/arm/boot/dts/mt7629.dtsi    | 22 ++++++++++++++++
+ drivers/spi/spi-mtk-snfi.c       | 12 +++++++++
+ 3 files changed, 79 insertions(+)
+
+diff --git a/arch/arm/boot/dts/mt7629-rfb.dts b/arch/arm/boot/dts/mt7629-rfb.dts
+index 8043238fe..ef140623e 100755
+--- a/arch/arm/boot/dts/mt7629-rfb.dts
++++ b/arch/arm/boot/dts/mt7629-rfb.dts
+@@ -258,6 +258,51 @@
+       };
+ };
++&bch {
++      status = "okay";
++};
++
++&snfi {
++      pinctrl-names = "default";
++      pinctrl-0 = <&serial_nand_pins>;
++      status = "okay";
++
++      spi_nand@0 {
++              #address-cells = <1>;
++              #size-cells = <1>;
++              compatible = "spi-nand";
++              spi-max-frequency = <104000000>;
++              reg = <0>;
++
++              partitions {
++                compatible = "fixed-partitions";
++                #address-cells = <1>;
++                #size-cells = <1>;
++
++                partition@0 {
++                        label = "Bootloader";
++                        reg = <0x00000 0x0100000>;
++                        read-only;
++                };
++
++                partition@100000 {
++                        label = "Config";
++                        reg = <0x100000 0x0040000>;
++                };
++
++                partition@140000 {
++                        label = "factory";
++                        reg = <0x140000 0x0080000>;
++                };
++
++                partition@1c0000 {
++                        label = "firmware";
++                        reg = <0x1c0000 0x1000000>;
++                };
++        };
++      };
++};
++
+ &spi {
+       pinctrl-names = "default";
+       pinctrl-0 = <&spi_pins>;
+diff --git a/arch/arm/boot/dts/mt7629.dtsi b/arch/arm/boot/dts/mt7629.dtsi
+index 53f47796b..c7bd5fc9b 100755
+--- a/arch/arm/boot/dts/mt7629.dtsi
++++ b/arch/arm/boot/dts/mt7629.dtsi
+@@ -259,6 +259,28 @@
+                       status = "disabled";
+               };
++              bch: ecc@1100e000 {
++                      compatible = "mediatek,mt7622-ecc";
++                      reg = <0x1100e000 0x1000>;
++                      interrupts = <GIC_SPI 95 IRQ_TYPE_LEVEL_LOW>;
++                      clocks = <&pericfg CLK_PERI_NFIECC_PD>;
++                      clock-names = "nfiecc_clk";
++                      status = "disabled";
++              };
++
++              snfi: spi@1100d000 {
++                      compatible = "mediatek,mt7629-snfi";
++                      reg = <0x1100d000 0x1000>;
++                      interrupts = <GIC_SPI 96 IRQ_TYPE_LEVEL_LOW>;
++                      clocks = <&pericfg CLK_PERI_NFI_PD>,
++                               <&pericfg CLK_PERI_SNFI_PD>;
++                      clock-names = "nfi_clk", "spi_clk";
++                      ecc-engine = <&bch>;
++                      #address-cells = <1>;
++                      #size-cells = <0>;
++                      status = "disabled";
++              };
++
+               spi: spi@1100a000 {
+                       compatible = "mediatek,mt7629-spi",
+                                    "mediatek,mt7622-spi";
+diff --git a/drivers/spi/spi-mtk-snfi.c b/drivers/spi/spi-mtk-snfi.c
+index cd600d9fb..113f541e6 100644
+--- a/drivers/spi/spi-mtk-snfi.c
++++ b/drivers/spi/spi-mtk-snfi.c
+@@ -1029,8 +1029,20 @@ static const struct mtk_snfi_caps snfi_mt7622 = {
+       .bad_mark_swap = 0,
+ };
++static const struct mtk_snfi_caps snfi_mt7629 = {
++      .spare_size = spare_size_mt7622,
++      .num_spare_size = 4,
++      .nand_sec_size = 512,
++      .nand_fdm_size = 8,
++      .nand_fdm_ecc_size = 1,
++      .ecc_parity_bits = 13,
++      .pageformat_spare_shift = 4,
++      .bad_mark_swap = 1,
++};
++
+ static const struct of_device_id mtk_snfi_id_table[] = {
+       { .compatible = "mediatek,mt7622-snfi", .data = &snfi_mt7622, },
++      { .compatible = "mediatek,mt7629-snfi", .data = &snfi_mt7629, },
+       {  /* sentinel */ }
+ };
+-- 
+2.21.0
+
+diff --git a/arch/arm/boot/dts/mt7629-lynx-rfb.dts b/arch/arm/boot/dts/mt7629-lynx-rfb.dts
+index f3fadd3..895d6c4 100755
+--- a/arch/arm/boot/dts/mt7629-lynx-rfb.dts
++++ b/arch/arm/boot/dts/mt7629-lynx-rfb.dts
+@@ -278,6 +278,52 @@
+       };
+ };
++&bch {
++      status = "okay";
++};
++
++&snfi {
++      pinctrl-names = "default";
++      pinctrl-0 = <&serial_nand_pins>;
++      status = "okay";
++
++      spi_nand@0 {
++              #address-cells = <1>;
++              #size-cells = <1>;
++              compatible = "spi-nand";
++              spi-max-frequency = <104000000>;
++              reg = <0>;
++
++              partitions {
++              compatible = "fixed-partitions";
++              #address-cells = <1>;
++              #size-cells = <1>;
++
++              partition@0 {
++                      label = "Bootloader";
++                      reg = <0x00000 0x0100000>;
++                      read-only;
++              };
++
++              partition@100000 {
++                      label = "Config";
++                      reg = <0x100000 0x0040000>;
++              };
++
++              partition@140000 {
++                      label = "factory";
++                      reg = <0x140000 0x0080000>;
++              };
++
++              partition@1c0000 {
++                      label = "firmware";
++                      reg = <0x1c0000 0x1000000>;
++              };
++
++              };
++      };
++};
++
+ &spi {
+       pinctrl-names = "default";
+       pinctrl-0 = <&spi_pins>;