Re: [PATCH v1] misc: nuvoton: Add host interface configuartion driver

2022-06-22 Thread Tom Rini
On Tue, Jun 07, 2022 at 04:34:30PM +0800, Jim Liu wrote:

> add nuvoton BMC npcm750 host configuartion driver
> 
> Signed-off-by: Jim Liu 
> ---
>  drivers/misc/Makefile |   1 +
>  drivers/misc/npcm_host_intf.c | 110 ++
>  2 files changed, 111 insertions(+)
>  create mode 100644 drivers/misc/npcm_host_intf.c

This needs to add a Kconfig option and that in turn needs to select or
depends on the infrastructure it needs:
arm-linux-gnueabi-ld.bfd: drivers/misc/npcm_host_intf.o: in function 
`npcm_host_intf_bind':
drivers/misc/npcm_host_intf.c:56: undefined reference to 
`syscon_regmap_lookup_by_phandle'
arm-linux-gnueabi-ld.bfd: drivers/misc/npcm_host_intf.c:64: undefined reference 
to `regmap_write'
arm-linux-gnueabi-ld.bfd: drivers/misc/npcm_host_intf.c:80: undefined reference 
to `regmap_update_bits'
arm-linux-gnueabi-ld.bfd: drivers/misc/npcm_host_intf.c:81: undefined reference 
to `regmap_update_bits'
arm-linux-gnueabi-ld.bfd: drivers/misc/npcm_host_intf.c:92: undefined reference 
to `regmap_update_bits'
arm-linux-gnueabi-ld.bfd: drivers/misc/npcm_host_intf.c:93: undefined reference 
to `regmap_update_bits'
make[1]: *** [Makefile:1813: u-boot] Error 1

-- 
Tom


signature.asc
Description: PGP signature


[PATCH v1] misc: nuvoton: Add host interface configuartion driver

2022-06-07 Thread Jim Liu
add nuvoton BMC npcm750 host configuartion driver

Signed-off-by: Jim Liu 
---
 drivers/misc/Makefile |   1 +
 drivers/misc/npcm_host_intf.c | 110 ++
 2 files changed, 111 insertions(+)
 create mode 100644 drivers/misc/npcm_host_intf.c

diff --git a/drivers/misc/Makefile b/drivers/misc/Makefile
index 29fbf5640d..5f9cb1f053 100644
--- a/drivers/misc/Makefile
+++ b/drivers/misc/Makefile
@@ -54,6 +54,7 @@ obj-$(CONFIG_MPC83XX_SERDES) += mpc83xx_serdes.o
 obj-$(CONFIG_$(SPL_)MXC_OCOTP) += mxc_ocotp.o
 obj-$(CONFIG_MXS_OCOTP) += mxs_ocotp.o
 obj-$(CONFIG_NPCM_OTP) += npcm_otp.o
+obj-$(CONFIG_ARCH_NPCM) += npcm_host_intf.o
 obj-$(CONFIG_NUVOTON_NCT6102D) += nuvoton_nct6102d.o
 obj-$(CONFIG_P2SB) += p2sb-uclass.o
 obj-$(CONFIG_PCA9551_LED) += pca9551_led.o
diff --git a/drivers/misc/npcm_host_intf.c b/drivers/misc/npcm_host_intf.c
new file mode 100644
index 00..0244e40457
--- /dev/null
+++ b/drivers/misc/npcm_host_intf.c
@@ -0,0 +1,110 @@
+// SPDX-License-Identifier: GPL-2.0+
+/*
+ * Host interface (LPC or eSPI) configuration on Nuvoton BMC
+ * Copyright (c) 2022 Nuvoton Technology Corp.
+ */
+
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+
+#define SMC_CTL_REG_ADDR   0xc0001001
+#define SMC_CTL_HOSTWAIT   0x80
+
+/* GCR Register Offsets */
+#define HIFCR  0x50
+#define MFSEL1 0x260
+#define MFSEL4 0x26c
+
+/* ESPI Register offsets */
+#define ESPICFG0x4
+#define ESPIHINDP  0x80
+
+/* MFSEL bit fileds */
+#define MFSEL1_LPCSEL  BIT(26)
+#define MFSEL4_ESPISEL BIT(8)
+
+/* ESPICFG bit fileds */
+#define CHSUPP_MASKGENMASK(27, 24)
+#define IOMODE_MASKGENMASK(9, 8)
+#define IOMODE_SDQ FIELD_PREP(IOMODE_MASK, 3)
+#define MAXFREQ_MASK   GENMASK(12, 10)
+#define MAXFREQ_33MHZ  FIELD_PREP(MAXFREQ_MASK, 2)
+
+/* ESPIHINDP bit fileds */
+#define AUTO_SBLD  BIT(4)
+#define AUTO_HS1   BIT(8)
+#define AUTO_HS2   BIT(12)
+#define AUTO_HS3   BIT(16)
+
+static int npcm_host_intf_bind(struct udevice *dev)
+{
+   struct regmap *syscon;
+   void __iomem *base;
+   u32 ch_supp, val;
+   u32 ioaddr;
+   const char *type;
+   int ret;
+
+   /* Release host wait */
+   setbits_8(SMC_CTL_REG_ADDR, SMC_CTL_HOSTWAIT);
+
+   syscon = syscon_regmap_lookup_by_phandle(dev, "syscon");
+   if (IS_ERR(syscon)) {
+   dev_err(dev, "%s: unable to get syscon, dev %s\n", __func__, 
dev->name);
+   return PTR_ERR(syscon);
+   }
+
+   ioaddr  = dev_read_u32_default(dev, "ioaddr", 0);
+   if (ioaddr)
+   regmap_write(syscon, HIFCR, ioaddr);
+
+   type = dev_read_string(dev, "type");
+   if (!type)
+   return -EINVAL;
+
+   if (!strcmp(type, "espi")) {
+   base = dev_read_addr_ptr(dev);
+   if (!base)
+   return -EINVAL;
+
+   ret = dev_read_u32(dev, "channel-support", _supp);
+   if (ret)
+   return ret;
+
+   /* Select eSPI pins function */
+   regmap_update_bits(syscon, MFSEL1, MFSEL1_LPCSEL, 0);
+   regmap_update_bits(syscon, MFSEL4, MFSEL4_ESPISEL, 
MFSEL4_ESPISEL);
+
+   val = AUTO_SBLD | AUTO_HS1 | AUTO_HS2 | AUTO_HS3 | ch_supp;
+   writel(val, base + ESPIHINDP);
+
+   val = readl(base + ESPICFG);
+   val &= ~(CHSUPP_MASK | IOMODE_MASK | MAXFREQ_MASK);
+   val |= IOMODE_SDQ | MAXFREQ_33MHZ | FIELD_PREP(CHSUPP_MASK, 
ch_supp);
+   writel(val, base + ESPICFG);
+   } else if (!strcmp(type, "lpc")) {
+   /* Select LPC pin function */
+   regmap_update_bits(syscon, MFSEL4, MFSEL4_ESPISEL, 0);
+   regmap_update_bits(syscon, MFSEL1, MFSEL1_LPCSEL, 
MFSEL1_LPCSEL);
+   }
+
+   return 0;
+}
+
+static const struct udevice_id npcm_hostintf_ids[] = {
+   { .compatible = "nuvoton,npcm750-host-intf" },
+   { .compatible = "nuvoton,npcm845-host-intf" },
+   { }
+};
+
+U_BOOT_DRIVER(npcm_host_intf) = {
+   .name   = "npcm_host_intf",
+   .id = UCLASS_MISC,
+   .of_match = npcm_hostintf_ids,
+   .bind = npcm_host_intf_bind,
+};
-- 
2.17.1