2021-02-10 21:46:49

by Heiko Stuebner

[permalink] [raw]
Subject: [PATCH 0/2] phy: rockchip: add Innosilicon-based CSI DPHY

Newer Rockchip SoCs use a different DPHY for camera operation
based on an IP block from Innosilicon.

This adds a driver for it for px30/rk3326/rk3368/rk1808.

Heiko Stuebner (2):
dt-bindings: phy: add yaml binding for rockchip-inno-csi-dphy
phy/rockchip: add Innosilicon-based CSI dphy

.../bindings/phy/rockchip-inno-csi-dphy.yaml | 79 +++
drivers/phy/rockchip/Kconfig | 9 +
drivers/phy/rockchip/Makefile | 1 +
.../phy/rockchip/phy-rockchip-inno-csidphy.c | 480 ++++++++++++++++++
4 files changed, 569 insertions(+)
create mode 100644 Documentation/devicetree/bindings/phy/rockchip-inno-csi-dphy.yaml
create mode 100644 drivers/phy/rockchip/phy-rockchip-inno-csidphy.c

--
2.29.2


2021-02-10 21:47:24

by Heiko Stuebner

[permalink] [raw]
Subject: [PATCH 2/2] phy/rockchip: add Innosilicon-based CSI dphy

From: Heiko Stuebner <[email protected]>

The CSI dphy found for example on the rk3326/px30 and rk3368 is based
on an IP design from Innosilicon. Add a driver for it.

Signed-off-by: Heiko Stuebner <[email protected]>
---
drivers/phy/rockchip/Kconfig | 9 +
drivers/phy/rockchip/Makefile | 1 +
.../phy/rockchip/phy-rockchip-inno-csidphy.c | 480 ++++++++++++++++++
3 files changed, 490 insertions(+)
create mode 100644 drivers/phy/rockchip/phy-rockchip-inno-csidphy.c

diff --git a/drivers/phy/rockchip/Kconfig b/drivers/phy/rockchip/Kconfig
index 159285f42e5c..e812adad7242 100644
--- a/drivers/phy/rockchip/Kconfig
+++ b/drivers/phy/rockchip/Kconfig
@@ -48,6 +48,15 @@ config PHY_ROCKCHIP_INNO_USB2
help
Support for Rockchip USB2.0 PHY with Innosilicon IP block.

+config PHY_ROCKCHIP_INNO_CSIDPHY
+ tristate "Rockchip Innosilicon MIPI CSI PHY driver"
+ depends on (ARCH_ROCKCHIP || COMPILE_TEST) && OF
+ select GENERIC_PHY
+ select GENERIC_PHY_MIPI_DPHY
+ help
+ Enable this to support the Rockchip MIPI CSI PHY with
+ Innosilicon IP block.
+
config PHY_ROCKCHIP_INNO_DSIDPHY
tristate "Rockchip Innosilicon MIPI/LVDS/TTL PHY driver"
depends on (ARCH_ROCKCHIP || COMPILE_TEST) && OF
diff --git a/drivers/phy/rockchip/Makefile b/drivers/phy/rockchip/Makefile
index c3cfc7f0af5c..f0eec212b2aa 100644
--- a/drivers/phy/rockchip/Makefile
+++ b/drivers/phy/rockchip/Makefile
@@ -2,6 +2,7 @@
obj-$(CONFIG_PHY_ROCKCHIP_DP) += phy-rockchip-dp.o
obj-$(CONFIG_PHY_ROCKCHIP_DPHY_RX0) += phy-rockchip-dphy-rx0.o
obj-$(CONFIG_PHY_ROCKCHIP_EMMC) += phy-rockchip-emmc.o
+obj-$(CONFIG_PHY_ROCKCHIP_INNO_CSIDPHY) += phy-rockchip-inno-csidphy.o
obj-$(CONFIG_PHY_ROCKCHIP_INNO_DSIDPHY) += phy-rockchip-inno-dsidphy.o
obj-$(CONFIG_PHY_ROCKCHIP_INNO_HDMI) += phy-rockchip-inno-hdmi.o
obj-$(CONFIG_PHY_ROCKCHIP_INNO_USB2) += phy-rockchip-inno-usb2.o
diff --git a/drivers/phy/rockchip/phy-rockchip-inno-csidphy.c b/drivers/phy/rockchip/phy-rockchip-inno-csidphy.c
new file mode 100644
index 000000000000..b30bb2885029
--- /dev/null
+++ b/drivers/phy/rockchip/phy-rockchip-inno-csidphy.c
@@ -0,0 +1,480 @@
+// SPDX-License-Identifier: GPL-2.0
+/*
+ * Rockchip MIPI RX Innosilicon DPHY driver
+ *
+ * Copyright (C) 2017 Fuzhou Rockchip Electronics Co., Ltd.
+ */
+
+#include <linux/clk.h>
+#include <linux/delay.h>
+#include <linux/io.h>
+#include <linux/mfd/syscon.h>
+#include <linux/module.h>
+#include <linux/of.h>
+#include <linux/of_platform.h>
+#include <linux/phy/phy.h>
+#include <linux/phy/phy-mipi-dphy.h>
+#include <linux/platform_device.h>
+#include <linux/pm_runtime.h>
+#include <linux/regmap.h>
+#include <linux/reset.h>
+
+/* GRF */
+#define RK1808_GRF_PD_VI_CON_OFFSET 0x0430
+
+#define RK3326_GRF_PD_VI_CON_OFFSET 0x0430
+
+#define RK3368_GRF_SOC_CON6_OFFSET 0x0418
+
+/* PHY */
+#define CSIDPHY_CTRL_LANE_ENABLE 0x00
+#define CSIDPHY_CTRL_LANE_ENABLE_CK BIT(6)
+#define CSIDPHY_CTRL_LANE_ENABLE_LANE3 BIT(5)
+#define CSIDPHY_CTRL_LANE_ENABLE_LANE2 BIT(4)
+#define CSIDPHY_CTRL_LANE_ENABLE_LANE1 BIT(3)
+#define CSIDPHY_CTRL_LANE_ENABLE_LANE0 BIT(2)
+#define CSIDPHY_CTRL_LANE_ENABLE_UNDEFINED BIT(0)
+
+#define CSIDPHY_CTRL_LANE_ENABLE_SHIFT 2
+
+/* not present on all variants */
+#define CSIDPHY_CTRL_PWRCTL 0x04
+#define CSIDPHY_CTRL_PWRCTL_UNDEFINED GENMASK(7, 5)
+#define CSIDPHY_CTRL_PWRCTL_SYNCRST BIT(2)
+#define CSIDPHY_CTRL_PWRCTL_LDO_PD BIT(1)
+#define CSIDPHY_CTRL_PWRCTL_PLL_PD BIT(0)
+
+#define CSIDPHY_CTRL_DIG_RST 0x80
+#define CSIDPHY_CTRL_DIG_RST_UNDEFINED 0x1e
+#define CSIDPHY_CTRL_DIG_RST_RESET BIT(0)
+
+/* offset after ths_settle_offset */
+#define CSIDPHY_CLK_THS_SETTLE 0
+#define CSIDPHY_LANE_THS_SETTLE(n) ((n + 1) * 0x80)
+#define CSIDPHY_THS_SETTLE_MASK 0x7f
+
+/* offset after calib_offset */
+#define CSIDPHY_CLK_CALIB_EN 0
+#define CSIDPHY_LANE_CALIB_EN(n) ((n + 1) * 0x80)
+#define CSIDPHY_CALIB_EN BIT(7)
+
+/* Configure the count time of the THS-SETTLE by protocol. */
+#define RK1808_CSIDPHY_CLK_WR_THS_SETTLE 0x160
+#define RK3326_CSIDPHY_CLK_WR_THS_SETTLE 0x100
+#define RK3368_CSIDPHY_CLK_WR_THS_SETTLE 0x100
+
+/* Calibration reception enable */
+#define RK1808_CSIDPHY_CLK_CALIB_EN 0x168
+
+#define HIWORD_UPDATE(val, mask) ((val) | (mask) << 16)
+
+enum dphy_reg_id {
+ GRF_DPHY_RX0_TURNDISABLE = 0,
+ GRF_DPHY_RX0_FORCERXMODE,
+ GRF_DPHY_RX0_FORCETXSTOPMODE,
+ GRF_DPHY_RX0_ENABLE,
+ GRF_DPHY_RX0_TURNREQUEST,
+ GRF_DPHY_TX0_TURNDISABLE,
+ GRF_DPHY_TX0_FORCERXMODE,
+ GRF_DPHY_TX0_FORCETXSTOPMODE,
+ GRF_DPHY_TX0_TURNREQUEST,
+ GRF_DPHY_RX1_SRC_SEL,
+ /* rk1808 & rk3326 */
+ GRF_DPHY_CSIPHY_FORCERXMODE,
+ GRF_DPHY_CSIPHY_CLKLANE_EN,
+ GRF_DPHY_CSIPHY_DATALANE_EN,
+};
+
+struct dphy_reg {
+ u32 offset;
+ u32 mask;
+ u32 shift;
+};
+
+#define PHY_REG(_offset, _width, _shift) \
+ { .offset = _offset, .mask = BIT(_width) - 1, .shift = _shift, }
+
+static const struct dphy_reg rk1808_grf_dphy_regs[] = {
+ [GRF_DPHY_CSIPHY_FORCERXMODE] = PHY_REG(RK1808_GRF_PD_VI_CON_OFFSET, 4, 0),
+ [GRF_DPHY_CSIPHY_CLKLANE_EN] = PHY_REG(RK1808_GRF_PD_VI_CON_OFFSET, 1, 8),
+ [GRF_DPHY_CSIPHY_DATALANE_EN] = PHY_REG(RK1808_GRF_PD_VI_CON_OFFSET, 4, 4),
+};
+
+static const struct dphy_reg rk3326_grf_dphy_regs[] = {
+ [GRF_DPHY_CSIPHY_FORCERXMODE] = PHY_REG(RK3326_GRF_PD_VI_CON_OFFSET, 4, 0),
+ [GRF_DPHY_CSIPHY_CLKLANE_EN] = PHY_REG(RK3326_GRF_PD_VI_CON_OFFSET, 1, 8),
+ [GRF_DPHY_CSIPHY_DATALANE_EN] = PHY_REG(RK3326_GRF_PD_VI_CON_OFFSET, 4, 4),
+};
+
+static const struct dphy_reg rk3368_grf_dphy_regs[] = {
+ [GRF_DPHY_CSIPHY_FORCERXMODE] = PHY_REG(RK3368_GRF_SOC_CON6_OFFSET, 4, 8),
+};
+
+struct hsfreq_range {
+ u32 range_h;
+ u8 cfg_bit;
+};
+
+struct rockchip_inno_csidphy;
+
+struct dphy_drv_data {
+ int pwrctl_offset;
+ int ths_settle_offset;
+ int calib_offset;
+ const struct hsfreq_range *hsfreq_ranges;
+ int num_hsfreq_ranges;
+ const struct dphy_reg *grf_regs;
+};
+
+struct rockchip_inno_csidphy {
+ struct device *dev;
+ void __iomem *phy_base;
+ struct clk *pclk;
+ struct regmap *grf;
+ struct reset_control *rst;
+ const struct dphy_drv_data *drv_data;
+ struct phy_configure_opts_mipi_dphy config;
+ u8 hsfreq;
+};
+
+static inline void write_grf_reg(struct rockchip_inno_csidphy *priv,
+ int index, u8 value)
+{
+ const struct dphy_drv_data *drv_data = priv->drv_data;
+ const struct dphy_reg *reg = &drv_data->grf_regs[index];
+
+ /* Update high word */
+ unsigned int val = (value << reg->shift) |
+ (reg->mask << (reg->shift + 16));
+
+ if (reg->offset)
+ regmap_write(priv->grf, reg->offset, val);
+}
+
+static inline u32 read_grf_reg(struct rockchip_inno_csidphy *priv, int index)
+{
+ const struct dphy_drv_data *drv_data = priv->drv_data;
+ const struct dphy_reg *reg = &drv_data->grf_regs[index];
+ unsigned int val = 0;
+
+ if (reg->offset) {
+ regmap_read(priv->grf, reg->offset, &val);
+ val = (val >> reg->shift) & reg->mask;
+ }
+ return val;
+}
+
+/* These tables must be sorted by .range_h ascending. */
+static const struct hsfreq_range rk1808_mipidphy_hsfreq_ranges[] = {
+ { 109, 0x02}, { 149, 0x03}, { 199, 0x06}, { 249, 0x06},
+ { 299, 0x06}, { 399, 0x08}, { 499, 0x0b}, { 599, 0x0e},
+ { 699, 0x10}, { 799, 0x12}, { 999, 0x16}, {1199, 0x1e},
+ {1399, 0x23}, {1599, 0x2d}, {1799, 0x32}, {1999, 0x37},
+ {2199, 0x3c}, {2399, 0x41}, {2499, 0x46}
+};
+
+static const struct hsfreq_range rk3326_mipidphy_hsfreq_ranges[] = {
+ { 109, 0x00}, { 149, 0x01}, { 199, 0x02}, { 249, 0x03},
+ { 299, 0x04}, { 399, 0x05}, { 499, 0x06}, { 599, 0x07},
+ { 699, 0x08}, { 799, 0x09}, { 899, 0x0a}, {1099, 0x0b},
+ {1249, 0x0c}, {1349, 0x0d}, {1500, 0x0e}
+};
+
+static const struct hsfreq_range rk3368_mipidphy_hsfreq_ranges[] = {
+ { 109, 0x00}, { 149, 0x01}, { 199, 0x02}, { 249, 0x03},
+ { 299, 0x04}, { 399, 0x05}, { 499, 0x06}, { 599, 0x07},
+ { 699, 0x08}, { 799, 0x09}, { 899, 0x0a}, {1099, 0x0b},
+ {1249, 0x0c}, {1349, 0x0d}, {1500, 0x0e}
+};
+
+static void rockchip_inno_csidphy_ths_settle(struct rockchip_inno_csidphy *priv,
+ int hsfreq, int offset)
+{
+ const struct dphy_drv_data *drv_data = priv->drv_data;
+ u32 val;
+
+ val = readl(priv->phy_base + drv_data->ths_settle_offset + offset);
+ val &= ~CSIDPHY_THS_SETTLE_MASK;
+ val |= hsfreq;
+ writel(val, priv->phy_base + drv_data->ths_settle_offset + offset);
+}
+
+static int rockchip_inno_csidphy_configure(struct phy *phy,
+ union phy_configure_opts *opts)
+{
+ struct rockchip_inno_csidphy *priv = phy_get_drvdata(phy);
+ const struct dphy_drv_data *drv_data = priv->drv_data;
+ struct phy_configure_opts_mipi_dphy *config = &opts->mipi_dphy;
+ unsigned int hsfreq = 0;
+ unsigned int i;
+ u64 data_rate_mbps;
+ int ret;
+
+ /* pass with phy_mipi_dphy_get_default_config (with pixel rate?) */
+ ret = phy_mipi_dphy_config_validate(config);
+ if (ret)
+ return ret;
+
+ data_rate_mbps = div_u64(config->hs_clk_rate, 1000 * 1000);
+
+ dev_dbg(priv->dev, "lanes %d - data_rate_mbps %llu\n",
+ config->lanes, data_rate_mbps);
+ for (i = 0; i < drv_data->num_hsfreq_ranges; i++) {
+ if (drv_data->hsfreq_ranges[i].range_h >= data_rate_mbps) {
+ hsfreq = drv_data->hsfreq_ranges[i].cfg_bit;
+ break;
+ }
+ }
+ if (!hsfreq)
+ return -EINVAL;
+
+ priv->hsfreq = hsfreq;
+ priv->config = *config;
+ return 0;
+}
+
+static int rockchip_inno_csidphy_power_on(struct phy *phy)
+{
+ struct rockchip_inno_csidphy *priv = phy_get_drvdata(phy);
+ const struct dphy_drv_data *drv_data = priv->drv_data;
+ u64 data_rate_mbps;
+ u32 val = 0;
+ int ret, i;
+
+ data_rate_mbps = div_u64(priv->config.hs_clk_rate, 1000 * 1000);
+
+ ret = clk_enable(priv->pclk);
+ if (ret < 0)
+ return ret;
+
+ /* phy start */
+ if (drv_data->pwrctl_offset >= 0)
+ writel(CSIDPHY_CTRL_PWRCTL_UNDEFINED |
+ CSIDPHY_CTRL_PWRCTL_SYNCRST,
+ priv->phy_base + drv_data->pwrctl_offset);
+
+ /* set data lane num and enable clock lane */
+ val = GENMASK(priv->config.lanes - 1, 0) << CSIDPHY_CTRL_LANE_ENABLE_SHIFT;
+ writel(val | CSIDPHY_CTRL_LANE_ENABLE_CK |
+ CSIDPHY_CTRL_LANE_ENABLE_UNDEFINED,
+ priv->phy_base + CSIDPHY_CTRL_LANE_ENABLE);
+
+ /* Reset dphy analog part */
+ if (drv_data->pwrctl_offset >= 0)
+ writel(CSIDPHY_CTRL_PWRCTL_UNDEFINED,
+ priv->phy_base + drv_data->pwrctl_offset);
+ usleep_range(500, 1000);
+
+ /* Reset dphy digital part */
+ writel(CSIDPHY_CTRL_DIG_RST_UNDEFINED,
+ priv->phy_base + CSIDPHY_CTRL_DIG_RST);
+ writel(CSIDPHY_CTRL_DIG_RST_UNDEFINED + CSIDPHY_CTRL_DIG_RST_RESET,
+ priv->phy_base + CSIDPHY_CTRL_DIG_RST);
+
+ /* not into receive mode/wait stopstate */
+ write_grf_reg(priv, GRF_DPHY_CSIPHY_FORCERXMODE, 0x0);
+
+ /* enable calibration */
+ if (data_rate_mbps > 1500 && drv_data->calib_offset >= 0) {
+ writel(CSIDPHY_CALIB_EN,
+ priv->phy_base + drv_data->calib_offset +
+ CSIDPHY_CLK_CALIB_EN);
+ for (i = 0; i < priv->config.lanes; i++)
+ writel(CSIDPHY_CALIB_EN,
+ priv->phy_base + drv_data->calib_offset +
+ CSIDPHY_LANE_CALIB_EN(i));
+ }
+
+ rockchip_inno_csidphy_ths_settle(priv, priv->hsfreq,
+ CSIDPHY_CLK_THS_SETTLE);
+ for (i = 0; i < priv->config.lanes; i++)
+ rockchip_inno_csidphy_ths_settle(priv, priv->hsfreq,
+ CSIDPHY_LANE_THS_SETTLE(i));
+
+ write_grf_reg(priv, GRF_DPHY_CSIPHY_CLKLANE_EN, 0x1);
+ write_grf_reg(priv, GRF_DPHY_CSIPHY_DATALANE_EN,
+ GENMASK(priv->config.lanes - 1, 0));
+
+ return 0;
+}
+
+static int rockchip_inno_csidphy_power_off(struct phy *phy)
+{
+ struct rockchip_inno_csidphy *priv = phy_get_drvdata(phy);
+ const struct dphy_drv_data *drv_data = priv->drv_data;
+
+ /* disable all lanes */
+ writel(CSIDPHY_CTRL_LANE_ENABLE_UNDEFINED,
+ priv->phy_base + CSIDPHY_CTRL_LANE_ENABLE);
+
+ /* disable pll and ldo */
+ if (drv_data->pwrctl_offset >= 0)
+ writel(CSIDPHY_CTRL_PWRCTL_UNDEFINED |
+ CSIDPHY_CTRL_PWRCTL_LDO_PD |
+ CSIDPHY_CTRL_PWRCTL_PLL_PD,
+ priv->phy_base + drv_data->pwrctl_offset);
+ usleep_range(500, 1000);
+
+ clk_disable(priv->pclk);
+
+ return 0;
+}
+
+static int rockchip_inno_csidphy_init(struct phy *phy)
+{
+ struct rockchip_inno_csidphy *priv = phy_get_drvdata(phy);
+
+ return clk_prepare(priv->pclk);
+}
+
+static int rockchip_inno_csidphy_exit(struct phy *phy)
+{
+ struct rockchip_inno_csidphy *priv = phy_get_drvdata(phy);
+
+ clk_unprepare(priv->pclk);
+
+ return 0;
+}
+
+static const struct phy_ops rockchip_inno_csidphy_ops = {
+ .power_on = rockchip_inno_csidphy_power_on,
+ .power_off = rockchip_inno_csidphy_power_off,
+ .init = rockchip_inno_csidphy_init,
+ .exit = rockchip_inno_csidphy_exit,
+ .configure = rockchip_inno_csidphy_configure,
+ .owner = THIS_MODULE,
+};
+
+static const struct dphy_drv_data rk1808_mipidphy_drv_data = {
+ .pwrctl_offset = -1,
+ .ths_settle_offset = RK1808_CSIDPHY_CLK_WR_THS_SETTLE,
+ .calib_offset = RK1808_CSIDPHY_CLK_CALIB_EN,
+ .hsfreq_ranges = rk1808_mipidphy_hsfreq_ranges,
+ .num_hsfreq_ranges = ARRAY_SIZE(rk1808_mipidphy_hsfreq_ranges),
+ .grf_regs = rk1808_grf_dphy_regs,
+};
+
+static const struct dphy_drv_data rk3326_mipidphy_drv_data = {
+ .pwrctl_offset = CSIDPHY_CTRL_PWRCTL,
+ .ths_settle_offset = RK3326_CSIDPHY_CLK_WR_THS_SETTLE,
+ .calib_offset = -1,
+ .hsfreq_ranges = rk3326_mipidphy_hsfreq_ranges,
+ .num_hsfreq_ranges = ARRAY_SIZE(rk3326_mipidphy_hsfreq_ranges),
+ .grf_regs = rk3326_grf_dphy_regs,
+};
+
+static const struct dphy_drv_data rk3368_mipidphy_drv_data = {
+ .pwrctl_offset = CSIDPHY_CTRL_PWRCTL,
+ .ths_settle_offset = RK3368_CSIDPHY_CLK_WR_THS_SETTLE,
+ .calib_offset = -1,
+ .hsfreq_ranges = rk3368_mipidphy_hsfreq_ranges,
+ .num_hsfreq_ranges = ARRAY_SIZE(rk3368_mipidphy_hsfreq_ranges),
+ .grf_regs = rk3368_grf_dphy_regs,
+};
+
+static const struct of_device_id rockchip_inno_csidphy_match_id[] = {
+ {
+ .compatible = "rockchip,px30-csi-dphy",
+ .data = &rk3326_mipidphy_drv_data,
+ },
+ {
+ .compatible = "rockchip,rk1808-csi-dphy",
+ .data = &rk1808_mipidphy_drv_data,
+ },
+ {
+ .compatible = "rockchip,rk3326-csi-dphy",
+ .data = &rk3326_mipidphy_drv_data,
+ },
+ {
+ .compatible = "rockchip,rk3368-csi-dphy",
+ .data = &rk3368_mipidphy_drv_data,
+ },
+ {}
+};
+MODULE_DEVICE_TABLE(of, rockchip_inno_csidphy_match_id);
+
+
+static int rockchip_inno_csidphy_probe(struct platform_device *pdev)
+{
+ struct rockchip_inno_csidphy *priv;
+ struct device *dev = &pdev->dev;
+ struct phy_provider *phy_provider;
+ struct phy *phy;
+
+ priv = devm_kzalloc(dev, sizeof(*priv), GFP_KERNEL);
+ if (!priv)
+ return -ENOMEM;
+
+ priv->dev = dev;
+ platform_set_drvdata(pdev, priv);
+
+ priv->drv_data = of_device_get_match_data(dev);
+ if (!priv->drv_data) {
+ dev_err(dev, "Can't find device data\n");
+ return -ENODEV;
+ }
+
+ priv->grf = syscon_regmap_lookup_by_phandle(dev->of_node,
+ "rockchip,grf");
+ if (IS_ERR(priv->grf)) {
+ dev_err(dev, "Can't find GRF syscon\n");
+ return PTR_ERR(priv->grf);
+ }
+
+ priv->phy_base = devm_platform_ioremap_resource(pdev, 0);
+ if (IS_ERR(priv->phy_base))
+ return PTR_ERR(priv->phy_base);
+
+ priv->pclk = devm_clk_get(dev, "pclk");
+ if (IS_ERR(priv->pclk)) {
+ dev_err(dev, "failed to get pclk\n");
+ return PTR_ERR(priv->pclk);
+ }
+
+ priv->rst = devm_reset_control_get(dev, "apb");
+ if (IS_ERR(priv->rst)) {
+ dev_err(dev, "failed to get system reset control\n");
+ return PTR_ERR(priv->rst);
+ }
+
+ phy = devm_phy_create(dev, NULL, &rockchip_inno_csidphy_ops);
+ if (IS_ERR(phy)) {
+ dev_err(dev, "failed to create phy\n");
+ return PTR_ERR(phy);
+ }
+
+ phy_set_drvdata(phy, priv);
+
+ phy_provider = devm_of_phy_provider_register(dev, of_phy_simple_xlate);
+ if (IS_ERR(phy_provider)) {
+ dev_err(dev, "failed to register phy provider\n");
+ return PTR_ERR(phy_provider);
+ }
+
+ pm_runtime_enable(dev);
+
+ return 0;
+}
+
+static int rockchip_inno_csidphy_remove(struct platform_device *pdev)
+{
+ struct rockchip_inno_csidphy *priv = platform_get_drvdata(pdev);
+
+ pm_runtime_disable(priv->dev);
+
+ return 0;
+}
+
+static struct platform_driver rockchip_inno_csidphy_driver = {
+ .driver = {
+ .name = "rockchip-inno-csidphy",
+ .of_match_table = rockchip_inno_csidphy_match_id,
+ },
+ .probe = rockchip_inno_csidphy_probe,
+ .remove = rockchip_inno_csidphy_remove,
+};
+
+module_platform_driver(rockchip_inno_csidphy_driver);
+MODULE_AUTHOR("Heiko Stuebner <[email protected]>");
+MODULE_DESCRIPTION("Rockchip MIPI Innosilicon CSI-DPHY driver");
+MODULE_LICENSE("Dual BSD/GPL");
--
2.29.2

2021-03-17 10:23:37

by Vinod Koul

[permalink] [raw]
Subject: Re: [PATCH 2/2] phy/rockchip: add Innosilicon-based CSI dphy

On 10-02-21, 22:42, Heiko Stuebner wrote:
> From: Heiko Stuebner <[email protected]>
>
> The CSI dphy found for example on the rk3326/px30 and rk3368 is based
> on an IP design from Innosilicon. Add a driver for it.
>
> Signed-off-by: Heiko Stuebner <[email protected]>
> ---
> drivers/phy/rockchip/Kconfig | 9 +
> drivers/phy/rockchip/Makefile | 1 +
> .../phy/rockchip/phy-rockchip-inno-csidphy.c | 480 ++++++++++++++++++
> 3 files changed, 490 insertions(+)
> create mode 100644 drivers/phy/rockchip/phy-rockchip-inno-csidphy.c
>
> diff --git a/drivers/phy/rockchip/Kconfig b/drivers/phy/rockchip/Kconfig
> index 159285f42e5c..e812adad7242 100644
> --- a/drivers/phy/rockchip/Kconfig
> +++ b/drivers/phy/rockchip/Kconfig
> @@ -48,6 +48,15 @@ config PHY_ROCKCHIP_INNO_USB2
> help
> Support for Rockchip USB2.0 PHY with Innosilicon IP block.
>
> +config PHY_ROCKCHIP_INNO_CSIDPHY
> + tristate "Rockchip Innosilicon MIPI CSI PHY driver"
> + depends on (ARCH_ROCKCHIP || COMPILE_TEST) && OF
> + select GENERIC_PHY
> + select GENERIC_PHY_MIPI_DPHY
> + help
> + Enable this to support the Rockchip MIPI CSI PHY with
> + Innosilicon IP block.
> +
> config PHY_ROCKCHIP_INNO_DSIDPHY
> tristate "Rockchip Innosilicon MIPI/LVDS/TTL PHY driver"
> depends on (ARCH_ROCKCHIP || COMPILE_TEST) && OF
> diff --git a/drivers/phy/rockchip/Makefile b/drivers/phy/rockchip/Makefile
> index c3cfc7f0af5c..f0eec212b2aa 100644
> --- a/drivers/phy/rockchip/Makefile
> +++ b/drivers/phy/rockchip/Makefile
> @@ -2,6 +2,7 @@
> obj-$(CONFIG_PHY_ROCKCHIP_DP) += phy-rockchip-dp.o
> obj-$(CONFIG_PHY_ROCKCHIP_DPHY_RX0) += phy-rockchip-dphy-rx0.o
> obj-$(CONFIG_PHY_ROCKCHIP_EMMC) += phy-rockchip-emmc.o
> +obj-$(CONFIG_PHY_ROCKCHIP_INNO_CSIDPHY) += phy-rockchip-inno-csidphy.o
> obj-$(CONFIG_PHY_ROCKCHIP_INNO_DSIDPHY) += phy-rockchip-inno-dsidphy.o
> obj-$(CONFIG_PHY_ROCKCHIP_INNO_HDMI) += phy-rockchip-inno-hdmi.o
> obj-$(CONFIG_PHY_ROCKCHIP_INNO_USB2) += phy-rockchip-inno-usb2.o
> diff --git a/drivers/phy/rockchip/phy-rockchip-inno-csidphy.c b/drivers/phy/rockchip/phy-rockchip-inno-csidphy.c
> new file mode 100644
> index 000000000000..b30bb2885029
> --- /dev/null
> +++ b/drivers/phy/rockchip/phy-rockchip-inno-csidphy.c
> @@ -0,0 +1,480 @@
> +// SPDX-License-Identifier: GPL-2.0
> +/*
> + * Rockchip MIPI RX Innosilicon DPHY driver
> + *
> + * Copyright (C) 2017 Fuzhou Rockchip Electronics Co., Ltd.

We are in 2021 now :)

> + */
> +
> +#include <linux/clk.h>
> +#include <linux/delay.h>
> +#include <linux/io.h>
> +#include <linux/mfd/syscon.h>
> +#include <linux/module.h>
> +#include <linux/of.h>
> +#include <linux/of_platform.h>
> +#include <linux/phy/phy.h>
> +#include <linux/phy/phy-mipi-dphy.h>
> +#include <linux/platform_device.h>
> +#include <linux/pm_runtime.h>
> +#include <linux/regmap.h>
> +#include <linux/reset.h>
> +
> +/* GRF */
> +#define RK1808_GRF_PD_VI_CON_OFFSET 0x0430
> +
> +#define RK3326_GRF_PD_VI_CON_OFFSET 0x0430
> +
> +#define RK3368_GRF_SOC_CON6_OFFSET 0x0418
> +
> +/* PHY */
> +#define CSIDPHY_CTRL_LANE_ENABLE 0x00
> +#define CSIDPHY_CTRL_LANE_ENABLE_CK BIT(6)
> +#define CSIDPHY_CTRL_LANE_ENABLE_LANE3 BIT(5)
> +#define CSIDPHY_CTRL_LANE_ENABLE_LANE2 BIT(4)
> +#define CSIDPHY_CTRL_LANE_ENABLE_LANE1 BIT(3)
> +#define CSIDPHY_CTRL_LANE_ENABLE_LANE0 BIT(2)
> +#define CSIDPHY_CTRL_LANE_ENABLE_UNDEFINED BIT(0)
> +
> +#define CSIDPHY_CTRL_LANE_ENABLE_SHIFT 2
> +
> +/* not present on all variants */
> +#define CSIDPHY_CTRL_PWRCTL 0x04
> +#define CSIDPHY_CTRL_PWRCTL_UNDEFINED GENMASK(7, 5)
> +#define CSIDPHY_CTRL_PWRCTL_SYNCRST BIT(2)
> +#define CSIDPHY_CTRL_PWRCTL_LDO_PD BIT(1)
> +#define CSIDPHY_CTRL_PWRCTL_PLL_PD BIT(0)
> +
> +#define CSIDPHY_CTRL_DIG_RST 0x80
> +#define CSIDPHY_CTRL_DIG_RST_UNDEFINED 0x1e
> +#define CSIDPHY_CTRL_DIG_RST_RESET BIT(0)
> +
> +/* offset after ths_settle_offset */
> +#define CSIDPHY_CLK_THS_SETTLE 0
> +#define CSIDPHY_LANE_THS_SETTLE(n) ((n + 1) * 0x80)
> +#define CSIDPHY_THS_SETTLE_MASK 0x7f

GENMASK for this?

> +
> +/* offset after calib_offset */
> +#define CSIDPHY_CLK_CALIB_EN 0
> +#define CSIDPHY_LANE_CALIB_EN(n) ((n + 1) * 0x80)
> +#define CSIDPHY_CALIB_EN BIT(7)
> +
> +/* Configure the count time of the THS-SETTLE by protocol. */
> +#define RK1808_CSIDPHY_CLK_WR_THS_SETTLE 0x160
> +#define RK3326_CSIDPHY_CLK_WR_THS_SETTLE 0x100
> +#define RK3368_CSIDPHY_CLK_WR_THS_SETTLE 0x100
> +
> +/* Calibration reception enable */
> +#define RK1808_CSIDPHY_CLK_CALIB_EN 0x168
> +
> +#define HIWORD_UPDATE(val, mask) ((val) | (mask) << 16)
> +
> +enum dphy_reg_id {
> + GRF_DPHY_RX0_TURNDISABLE = 0,
> + GRF_DPHY_RX0_FORCERXMODE,
> + GRF_DPHY_RX0_FORCETXSTOPMODE,
> + GRF_DPHY_RX0_ENABLE,
> + GRF_DPHY_RX0_TURNREQUEST,
> + GRF_DPHY_TX0_TURNDISABLE,
> + GRF_DPHY_TX0_FORCERXMODE,
> + GRF_DPHY_TX0_FORCETXSTOPMODE,
> + GRF_DPHY_TX0_TURNREQUEST,
> + GRF_DPHY_RX1_SRC_SEL,
> + /* rk1808 & rk3326 */
> + GRF_DPHY_CSIPHY_FORCERXMODE,
> + GRF_DPHY_CSIPHY_CLKLANE_EN,
> + GRF_DPHY_CSIPHY_DATALANE_EN,
> +};
> +
> +struct dphy_reg {
> + u32 offset;
> + u32 mask;
> + u32 shift;
> +};
> +
> +#define PHY_REG(_offset, _width, _shift) \
> + { .offset = _offset, .mask = BIT(_width) - 1, .shift = _shift, }
> +
> +static const struct dphy_reg rk1808_grf_dphy_regs[] = {
> + [GRF_DPHY_CSIPHY_FORCERXMODE] = PHY_REG(RK1808_GRF_PD_VI_CON_OFFSET, 4, 0),
> + [GRF_DPHY_CSIPHY_CLKLANE_EN] = PHY_REG(RK1808_GRF_PD_VI_CON_OFFSET, 1, 8),
> + [GRF_DPHY_CSIPHY_DATALANE_EN] = PHY_REG(RK1808_GRF_PD_VI_CON_OFFSET, 4, 4),
> +};
> +
> +static const struct dphy_reg rk3326_grf_dphy_regs[] = {
> + [GRF_DPHY_CSIPHY_FORCERXMODE] = PHY_REG(RK3326_GRF_PD_VI_CON_OFFSET, 4, 0),
> + [GRF_DPHY_CSIPHY_CLKLANE_EN] = PHY_REG(RK3326_GRF_PD_VI_CON_OFFSET, 1, 8),
> + [GRF_DPHY_CSIPHY_DATALANE_EN] = PHY_REG(RK3326_GRF_PD_VI_CON_OFFSET, 4, 4),
> +};
> +
> +static const struct dphy_reg rk3368_grf_dphy_regs[] = {
> + [GRF_DPHY_CSIPHY_FORCERXMODE] = PHY_REG(RK3368_GRF_SOC_CON6_OFFSET, 4, 8),
> +};
> +
> +struct hsfreq_range {
> + u32 range_h;
> + u8 cfg_bit;
> +};
> +
> +struct rockchip_inno_csidphy;

why this forward declaration here, it is not used before defining few
lines below?

> +
> +struct dphy_drv_data {
> + int pwrctl_offset;
> + int ths_settle_offset;
> + int calib_offset;
> + const struct hsfreq_range *hsfreq_ranges;
> + int num_hsfreq_ranges;
> + const struct dphy_reg *grf_regs;
> +};
> +
> +struct rockchip_inno_csidphy {
> + struct device *dev;
> + void __iomem *phy_base;
> + struct clk *pclk;
> + struct regmap *grf;
> + struct reset_control *rst;
> + const struct dphy_drv_data *drv_data;
> + struct phy_configure_opts_mipi_dphy config;
> + u8 hsfreq;
> +};
> +
> +static inline void write_grf_reg(struct rockchip_inno_csidphy *priv,
> + int index, u8 value)

align with precceding line opening brace please (checkpatch --strict
would warn about such alignment)

> +{
> + const struct dphy_drv_data *drv_data = priv->drv_data;
> + const struct dphy_reg *reg = &drv_data->grf_regs[index];
> +
> + /* Update high word */
> + unsigned int val = (value << reg->shift) |
> + (reg->mask << (reg->shift + 16));

please use macros in bitfield to avoid shifting..

why calculate val, if reg->offset is not valid, why not do both under
the if condition?

> +
> + if (reg->offset)
> + regmap_write(priv->grf, reg->offset, val);
> +}
> +
> +static inline u32 read_grf_reg(struct rockchip_inno_csidphy *priv, int index)
> +{
> + const struct dphy_drv_data *drv_data = priv->drv_data;
> + const struct dphy_reg *reg = &drv_data->grf_regs[index];
> + unsigned int val = 0;
> +
> + if (reg->offset) {
> + regmap_read(priv->grf, reg->offset, &val);
> + val = (val >> reg->shift) & reg->mask;
> + }
> + return val;

so is 0 a valid value to be read?

> +}
> +
> +/* These tables must be sorted by .range_h ascending. */
> +static const struct hsfreq_range rk1808_mipidphy_hsfreq_ranges[] = {
> + { 109, 0x02}, { 149, 0x03}, { 199, 0x06}, { 249, 0x06},
> + { 299, 0x06}, { 399, 0x08}, { 499, 0x0b}, { 599, 0x0e},
> + { 699, 0x10}, { 799, 0x12}, { 999, 0x16}, {1199, 0x1e},
> + {1399, 0x23}, {1599, 0x2d}, {1799, 0x32}, {1999, 0x37},
> + {2199, 0x3c}, {2399, 0x41}, {2499, 0x46}
> +};
> +
> +static const struct hsfreq_range rk3326_mipidphy_hsfreq_ranges[] = {
> + { 109, 0x00}, { 149, 0x01}, { 199, 0x02}, { 249, 0x03},
> + { 299, 0x04}, { 399, 0x05}, { 499, 0x06}, { 599, 0x07},
> + { 699, 0x08}, { 799, 0x09}, { 899, 0x0a}, {1099, 0x0b},
> + {1249, 0x0c}, {1349, 0x0d}, {1500, 0x0e}
> +};
> +
> +static const struct hsfreq_range rk3368_mipidphy_hsfreq_ranges[] = {
> + { 109, 0x00}, { 149, 0x01}, { 199, 0x02}, { 249, 0x03},
> + { 299, 0x04}, { 399, 0x05}, { 499, 0x06}, { 599, 0x07},
> + { 699, 0x08}, { 799, 0x09}, { 899, 0x0a}, {1099, 0x0b},
> + {1249, 0x0c}, {1349, 0x0d}, {1500, 0x0e}
> +};
> +
> +static void rockchip_inno_csidphy_ths_settle(struct rockchip_inno_csidphy *priv,
> + int hsfreq, int offset)
> +{
> + const struct dphy_drv_data *drv_data = priv->drv_data;
> + u32 val;
> +
> + val = readl(priv->phy_base + drv_data->ths_settle_offset + offset);
> + val &= ~CSIDPHY_THS_SETTLE_MASK;
> + val |= hsfreq;
> + writel(val, priv->phy_base + drv_data->ths_settle_offset + offset);
> +}
> +
> +static int rockchip_inno_csidphy_configure(struct phy *phy,
> + union phy_configure_opts *opts)
> +{
> + struct rockchip_inno_csidphy *priv = phy_get_drvdata(phy);
> + const struct dphy_drv_data *drv_data = priv->drv_data;
> + struct phy_configure_opts_mipi_dphy *config = &opts->mipi_dphy;
> + unsigned int hsfreq = 0;
> + unsigned int i;
> + u64 data_rate_mbps;
> + int ret;
> +
> + /* pass with phy_mipi_dphy_get_default_config (with pixel rate?) */
> + ret = phy_mipi_dphy_config_validate(config);
> + if (ret)
> + return ret;
> +
> + data_rate_mbps = div_u64(config->hs_clk_rate, 1000 * 1000);
> +
> + dev_dbg(priv->dev, "lanes %d - data_rate_mbps %llu\n",
> + config->lanes, data_rate_mbps);
> + for (i = 0; i < drv_data->num_hsfreq_ranges; i++) {
> + if (drv_data->hsfreq_ranges[i].range_h >= data_rate_mbps) {
> + hsfreq = drv_data->hsfreq_ranges[i].cfg_bit;
> + break;
> + }
> + }
> + if (!hsfreq)
> + return -EINVAL;
> +
> + priv->hsfreq = hsfreq;
> + priv->config = *config;
> + return 0;
> +}
> +
> +static int rockchip_inno_csidphy_power_on(struct phy *phy)
> +{
> + struct rockchip_inno_csidphy *priv = phy_get_drvdata(phy);
> + const struct dphy_drv_data *drv_data = priv->drv_data;
> + u64 data_rate_mbps;
> + u32 val = 0;

superfluous init?

> + int ret, i;
> +
> + data_rate_mbps = div_u64(priv->config.hs_clk_rate, 1000 * 1000);

helper for this?

> +
> + ret = clk_enable(priv->pclk);
> + if (ret < 0)
> + return ret;
> +
> + /* phy start */
> + if (drv_data->pwrctl_offset >= 0)
> + writel(CSIDPHY_CTRL_PWRCTL_UNDEFINED |
> + CSIDPHY_CTRL_PWRCTL_SYNCRST,
> + priv->phy_base + drv_data->pwrctl_offset);
> +
> + /* set data lane num and enable clock lane */
> + val = GENMASK(priv->config.lanes - 1, 0) << CSIDPHY_CTRL_LANE_ENABLE_SHIFT;

Consider using helpers in bitfield.h instead of using shifts

> + writel(val | CSIDPHY_CTRL_LANE_ENABLE_CK |
> + CSIDPHY_CTRL_LANE_ENABLE_UNDEFINED,
> + priv->phy_base + CSIDPHY_CTRL_LANE_ENABLE);
> +
> + /* Reset dphy analog part */
> + if (drv_data->pwrctl_offset >= 0)
> + writel(CSIDPHY_CTRL_PWRCTL_UNDEFINED,
> + priv->phy_base + drv_data->pwrctl_offset);
> + usleep_range(500, 1000);
> +
> + /* Reset dphy digital part */
> + writel(CSIDPHY_CTRL_DIG_RST_UNDEFINED,
> + priv->phy_base + CSIDPHY_CTRL_DIG_RST);
> + writel(CSIDPHY_CTRL_DIG_RST_UNDEFINED + CSIDPHY_CTRL_DIG_RST_RESET,
> + priv->phy_base + CSIDPHY_CTRL_DIG_RST);
> +
> + /* not into receive mode/wait stopstate */
> + write_grf_reg(priv, GRF_DPHY_CSIPHY_FORCERXMODE, 0x0);
> +
> + /* enable calibration */
> + if (data_rate_mbps > 1500 && drv_data->calib_offset >= 0) {
> + writel(CSIDPHY_CALIB_EN,
> + priv->phy_base + drv_data->calib_offset +
> + CSIDPHY_CLK_CALIB_EN);
> + for (i = 0; i < priv->config.lanes; i++)
> + writel(CSIDPHY_CALIB_EN,
> + priv->phy_base + drv_data->calib_offset +
> + CSIDPHY_LANE_CALIB_EN(i));
> + }
> +
> + rockchip_inno_csidphy_ths_settle(priv, priv->hsfreq,
> + CSIDPHY_CLK_THS_SETTLE);
> + for (i = 0; i < priv->config.lanes; i++)
> + rockchip_inno_csidphy_ths_settle(priv, priv->hsfreq,
> + CSIDPHY_LANE_THS_SETTLE(i));
> +
> + write_grf_reg(priv, GRF_DPHY_CSIPHY_CLKLANE_EN, 0x1);
> + write_grf_reg(priv, GRF_DPHY_CSIPHY_DATALANE_EN,
> + GENMASK(priv->config.lanes - 1, 0));
> +
> + return 0;
> +}
> +
> +static int rockchip_inno_csidphy_power_off(struct phy *phy)
> +{
> + struct rockchip_inno_csidphy *priv = phy_get_drvdata(phy);
> + const struct dphy_drv_data *drv_data = priv->drv_data;
> +
> + /* disable all lanes */
> + writel(CSIDPHY_CTRL_LANE_ENABLE_UNDEFINED,
> + priv->phy_base + CSIDPHY_CTRL_LANE_ENABLE);
> +
> + /* disable pll and ldo */
> + if (drv_data->pwrctl_offset >= 0)
> + writel(CSIDPHY_CTRL_PWRCTL_UNDEFINED |
> + CSIDPHY_CTRL_PWRCTL_LDO_PD |
> + CSIDPHY_CTRL_PWRCTL_PLL_PD,
> + priv->phy_base + drv_data->pwrctl_offset);
> + usleep_range(500, 1000);
> +
> + clk_disable(priv->pclk);
> +
> + return 0;
> +}
> +
> +static int rockchip_inno_csidphy_init(struct phy *phy)
> +{
> + struct rockchip_inno_csidphy *priv = phy_get_drvdata(phy);
> +
> + return clk_prepare(priv->pclk);
> +}
> +
> +static int rockchip_inno_csidphy_exit(struct phy *phy)
> +{
> + struct rockchip_inno_csidphy *priv = phy_get_drvdata(phy);
> +
> + clk_unprepare(priv->pclk);
> +
> + return 0;
> +}
> +
> +static const struct phy_ops rockchip_inno_csidphy_ops = {
> + .power_on = rockchip_inno_csidphy_power_on,
> + .power_off = rockchip_inno_csidphy_power_off,
> + .init = rockchip_inno_csidphy_init,
> + .exit = rockchip_inno_csidphy_exit,
> + .configure = rockchip_inno_csidphy_configure,
> + .owner = THIS_MODULE,
> +};
> +
> +static const struct dphy_drv_data rk1808_mipidphy_drv_data = {
> + .pwrctl_offset = -1,
> + .ths_settle_offset = RK1808_CSIDPHY_CLK_WR_THS_SETTLE,
> + .calib_offset = RK1808_CSIDPHY_CLK_CALIB_EN,
> + .hsfreq_ranges = rk1808_mipidphy_hsfreq_ranges,
> + .num_hsfreq_ranges = ARRAY_SIZE(rk1808_mipidphy_hsfreq_ranges),
> + .grf_regs = rk1808_grf_dphy_regs,
> +};
> +
> +static const struct dphy_drv_data rk3326_mipidphy_drv_data = {
> + .pwrctl_offset = CSIDPHY_CTRL_PWRCTL,
> + .ths_settle_offset = RK3326_CSIDPHY_CLK_WR_THS_SETTLE,
> + .calib_offset = -1,
> + .hsfreq_ranges = rk3326_mipidphy_hsfreq_ranges,
> + .num_hsfreq_ranges = ARRAY_SIZE(rk3326_mipidphy_hsfreq_ranges),
> + .grf_regs = rk3326_grf_dphy_regs,
> +};
> +
> +static const struct dphy_drv_data rk3368_mipidphy_drv_data = {
> + .pwrctl_offset = CSIDPHY_CTRL_PWRCTL,
> + .ths_settle_offset = RK3368_CSIDPHY_CLK_WR_THS_SETTLE,
> + .calib_offset = -1,
> + .hsfreq_ranges = rk3368_mipidphy_hsfreq_ranges,
> + .num_hsfreq_ranges = ARRAY_SIZE(rk3368_mipidphy_hsfreq_ranges),
> + .grf_regs = rk3368_grf_dphy_regs,
> +};
> +
> +static const struct of_device_id rockchip_inno_csidphy_match_id[] = {
> + {
> + .compatible = "rockchip,px30-csi-dphy",
> + .data = &rk3326_mipidphy_drv_data,
> + },
> + {
> + .compatible = "rockchip,rk1808-csi-dphy",
> + .data = &rk1808_mipidphy_drv_data,
> + },
> + {
> + .compatible = "rockchip,rk3326-csi-dphy",
> + .data = &rk3326_mipidphy_drv_data,
> + },
> + {
> + .compatible = "rockchip,rk3368-csi-dphy",
> + .data = &rk3368_mipidphy_drv_data,
> + },
> + {}
> +};
> +MODULE_DEVICE_TABLE(of, rockchip_inno_csidphy_match_id);
> +
> +
> +static int rockchip_inno_csidphy_probe(struct platform_device *pdev)
> +{
> + struct rockchip_inno_csidphy *priv;
> + struct device *dev = &pdev->dev;
> + struct phy_provider *phy_provider;
> + struct phy *phy;
> +
> + priv = devm_kzalloc(dev, sizeof(*priv), GFP_KERNEL);
> + if (!priv)
> + return -ENOMEM;
> +
> + priv->dev = dev;
> + platform_set_drvdata(pdev, priv);
> +
> + priv->drv_data = of_device_get_match_data(dev);
> + if (!priv->drv_data) {
> + dev_err(dev, "Can't find device data\n");
> + return -ENODEV;
> + }
> +
> + priv->grf = syscon_regmap_lookup_by_phandle(dev->of_node,
> + "rockchip,grf");
> + if (IS_ERR(priv->grf)) {
> + dev_err(dev, "Can't find GRF syscon\n");
> + return PTR_ERR(priv->grf);
> + }
> +
> + priv->phy_base = devm_platform_ioremap_resource(pdev, 0);
> + if (IS_ERR(priv->phy_base))
> + return PTR_ERR(priv->phy_base);
> +
> + priv->pclk = devm_clk_get(dev, "pclk");
> + if (IS_ERR(priv->pclk)) {
> + dev_err(dev, "failed to get pclk\n");
> + return PTR_ERR(priv->pclk);
> + }
> +
> + priv->rst = devm_reset_control_get(dev, "apb");
> + if (IS_ERR(priv->rst)) {
> + dev_err(dev, "failed to get system reset control\n");
> + return PTR_ERR(priv->rst);
> + }
> +
> + phy = devm_phy_create(dev, NULL, &rockchip_inno_csidphy_ops);
> + if (IS_ERR(phy)) {
> + dev_err(dev, "failed to create phy\n");
> + return PTR_ERR(phy);
> + }
> +
> + phy_set_drvdata(phy, priv);
> +
> + phy_provider = devm_of_phy_provider_register(dev, of_phy_simple_xlate);
> + if (IS_ERR(phy_provider)) {
> + dev_err(dev, "failed to register phy provider\n");
> + return PTR_ERR(phy_provider);
> + }
> +
> + pm_runtime_enable(dev);
> +
> + return 0;
> +}
> +
> +static int rockchip_inno_csidphy_remove(struct platform_device *pdev)
> +{
> + struct rockchip_inno_csidphy *priv = platform_get_drvdata(pdev);
> +
> + pm_runtime_disable(priv->dev);
> +
> + return 0;
> +}
> +
> +static struct platform_driver rockchip_inno_csidphy_driver = {
> + .driver = {
> + .name = "rockchip-inno-csidphy",
> + .of_match_table = rockchip_inno_csidphy_match_id,

double indent here?

> + },
> + .probe = rockchip_inno_csidphy_probe,
> + .remove = rockchip_inno_csidphy_remove,
> +};
> +
> +module_platform_driver(rockchip_inno_csidphy_driver);
> +MODULE_AUTHOR("Heiko Stuebner <[email protected]>");
> +MODULE_DESCRIPTION("Rockchip MIPI Innosilicon CSI-DPHY driver");
> +MODULE_LICENSE("Dual BSD/GPL");

That is not what the SPDX tag for file says?

--
~Vinod