* [PATCH v4 0/3] drm/rockchip: Add driver for the new DSI2 controller
@ 2024-12-09 23:10 Heiko Stuebner
2024-12-09 23:10 ` [PATCH v4 1/3] drm/bridge/synopsys: Add MIPI DSI2 host controller bridge Heiko Stuebner
` (3 more replies)
0 siblings, 4 replies; 10+ messages in thread
From: Heiko Stuebner @ 2024-12-09 23:10 UTC (permalink / raw)
To: heiko
Cc: andy.yan, maarten.lankhorst, mripard, tzimmermann, robh, krzk+dt,
conor+dt, andrzej.hajda, neil.armstrong, rfoss, Laurent.pinchart,
jonas, jernej.skrabec, dri-devel, devicetree, linux-arm-kernel,
linux-rockchip, linux-kernel, quentin.schulz
This series adds a bridge and glue driver for the DSI2 controller found
in the rk3588 soc from Rockchip, that is based on a Synopsis IP block.
As the manual states:
The Display Serial Interface 2 (DSI-2) is part of a group of communication
protocols defined by the MIPI Alliance. The MIPI DSI-2 Host Controller is
a digital core that implements all protocol functions defined in the
MIPI DSI-2 Specification.
While the driver structure is very similar to the previous DSI controller,
the programming model of the core is quite different, with a completely
new register set.
Another notable difference is that the phy interface is variable now too
in its width and some other settings.
changes in v4:
- Update binding example to use phy-type argument
- Update commit description to use correctly spelled Synopsys
- use devm_pm_runtime_enable()
- drop pclk+sys_clk handling from the Rockchip glue-driver
(the bridge does that for us already)
changes in v3:
- remove double-empty line in bridge part (Neil)
- add bitfield.h to both bridge and glue for FIELD_PROP
(reported by kernel test robot)
- add Neil's Reviewed-by to bridge part
- add Rob's Reviewed-by to binding
changes in v2:
- clean up includes (Diederik)
- fix Kconfig description (Diederik)
- constant naming (Diederik)
- binding fixes (paths, sorting, labels) (Rob)
- move to use regmap
- drop custom UPDATE macro and use FIELD_PREP instead
- use dev_err instead of DRM_DEV_ERROR
Heiko Stuebner (3):
drm/bridge/synopsys: Add MIPI DSI2 host controller bridge
dt-bindings: display: rockchip: Add schema for RK3588 DW DSI2
controller
drm/rockchip: Add MIPI DSI2 glue driver for RK3588
.../rockchip/rockchip,rk3588-mipi-dsi2.yaml | 120 ++
drivers/gpu/drm/bridge/synopsys/Kconfig | 6 +
drivers/gpu/drm/bridge/synopsys/Makefile | 1 +
.../gpu/drm/bridge/synopsys/dw-mipi-dsi2.c | 1030 +++++++++++++++++
drivers/gpu/drm/rockchip/Kconfig | 10 +
drivers/gpu/drm/rockchip/Makefile | 1 +
.../gpu/drm/rockchip/dw-mipi-dsi2-rockchip.c | 487 ++++++++
drivers/gpu/drm/rockchip/rockchip_drm_drv.c | 2 +
drivers/gpu/drm/rockchip/rockchip_drm_drv.h | 1 +
include/drm/bridge/dw_mipi_dsi2.h | 95 ++
10 files changed, 1753 insertions(+)
create mode 100644 Documentation/devicetree/bindings/display/rockchip/rockchip,rk3588-mipi-dsi2.yaml
create mode 100644 drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi2.c
create mode 100644 drivers/gpu/drm/rockchip/dw-mipi-dsi2-rockchip.c
create mode 100644 include/drm/bridge/dw_mipi_dsi2.h
--
2.45.2
^ permalink raw reply [flat|nested] 10+ messages in thread* [PATCH v4 1/3] drm/bridge/synopsys: Add MIPI DSI2 host controller bridge 2024-12-09 23:10 [PATCH v4 0/3] drm/rockchip: Add driver for the new DSI2 controller Heiko Stuebner @ 2024-12-09 23:10 ` Heiko Stuebner 2024-12-10 8:50 ` Andy Yan ` (2 more replies) 2024-12-09 23:10 ` [PATCH v4 2/3] dt-bindings: display: rockchip: Add schema for RK3588 DW DSI2 controller Heiko Stuebner ` (2 subsequent siblings) 3 siblings, 3 replies; 10+ messages in thread From: Heiko Stuebner @ 2024-12-09 23:10 UTC (permalink / raw) To: heiko Cc: andy.yan, maarten.lankhorst, mripard, tzimmermann, robh, krzk+dt, conor+dt, andrzej.hajda, neil.armstrong, rfoss, Laurent.pinchart, jonas, jernej.skrabec, dri-devel, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel, quentin.schulz, Heiko Stuebner, Daniel Semkowicz, Dmitry Yashin From: Heiko Stuebner <heiko.stuebner@cherry.de> Add a Synopsys Designware MIPI DSI host DRM bridge driver for their DSI2 host controller, based on the Rockchip version from the driver rockchip/dw-mipi-dsi2.c in their vendor-kernel with phy & bridge APIs. While the driver is heavily modelled after the previous IP, the register set of this DSI2 controller is completely different and there are also additional properties like the variable-width phy interface. Tested-by: Daniel Semkowicz <dse@thaumatec.com> Tested-by: Dmitry Yashin <dmt.yashin@gmail.com> Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org> Signed-off-by: Heiko Stuebner <heiko.stuebner@cherry.de> --- drivers/gpu/drm/bridge/synopsys/Kconfig | 6 + drivers/gpu/drm/bridge/synopsys/Makefile | 1 + .../gpu/drm/bridge/synopsys/dw-mipi-dsi2.c | 1030 +++++++++++++++++ include/drm/bridge/dw_mipi_dsi2.h | 95 ++ 4 files changed, 1132 insertions(+) create mode 100644 drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi2.c create mode 100644 include/drm/bridge/dw_mipi_dsi2.h diff --git a/drivers/gpu/drm/bridge/synopsys/Kconfig b/drivers/gpu/drm/bridge/synopsys/Kconfig index ca416dab156d..f3ab2f985f8c 100644 --- a/drivers/gpu/drm/bridge/synopsys/Kconfig +++ b/drivers/gpu/drm/bridge/synopsys/Kconfig @@ -59,3 +59,9 @@ config DRM_DW_MIPI_DSI select DRM_KMS_HELPER select DRM_MIPI_DSI select DRM_PANEL_BRIDGE + +config DRM_DW_MIPI_DSI2 + tristate + select DRM_KMS_HELPER + select DRM_MIPI_DSI + select DRM_PANEL_BRIDGE diff --git a/drivers/gpu/drm/bridge/synopsys/Makefile b/drivers/gpu/drm/bridge/synopsys/Makefile index 9869d9651ed1..9dc376d220ad 100644 --- a/drivers/gpu/drm/bridge/synopsys/Makefile +++ b/drivers/gpu/drm/bridge/synopsys/Makefile @@ -8,3 +8,4 @@ obj-$(CONFIG_DRM_DW_HDMI_CEC) += dw-hdmi-cec.o obj-$(CONFIG_DRM_DW_HDMI_QP) += dw-hdmi-qp.o obj-$(CONFIG_DRM_DW_MIPI_DSI) += dw-mipi-dsi.o +obj-$(CONFIG_DRM_DW_MIPI_DSI2) += dw-mipi-dsi2.o diff --git a/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi2.c b/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi2.c new file mode 100644 index 000000000000..054ad39c451b --- /dev/null +++ b/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi2.c @@ -0,0 +1,1030 @@ +// SPDX-License-Identifier: GPL-2.0+ +/* + * Copyright (c) 2024, Fuzhou Rockchip Electronics Co., Ltd + * + * Modified by Heiko Stuebner <heiko.stuebner@cherry.de> + * This generic Synopsys DesignWare MIPI DSI2 host driver is based on the + * Rockchip version from rockchip/dw-mipi-dsi2.c converted to use bridge APIs. + */ + +#include <linux/bitfield.h> +#include <linux/clk.h> +#include <linux/iopoll.h> +#include <linux/media-bus-format.h> +#include <linux/module.h> +#include <linux/platform_device.h> +#include <linux/pm_runtime.h> +#include <linux/reset.h> + +#include <video/mipi_display.h> + +#include <drm/bridge/dw_mipi_dsi2.h> +#include <drm/drm_atomic_helper.h> +#include <drm/drm_bridge.h> +#include <drm/drm_mipi_dsi.h> +#include <drm/drm_of.h> +#include <drm/drm_print.h> + +#define DSI2_PWR_UP 0x000c +#define RESET 0 +#define POWER_UP BIT(0) +#define CMD_TX_MODE(x) FIELD_PREP(BIT(24), x) +#define DSI2_SOFT_RESET 0x0010 +#define SYS_RSTN BIT(2) +#define PHY_RSTN BIT(1) +#define IPI_RSTN BIT(0) +#define INT_ST_MAIN 0x0014 +#define DSI2_MODE_CTRL 0x0018 +#define DSI2_MODE_STATUS 0x001c +#define DSI2_CORE_STATUS 0x0020 +#define PRI_RD_DATA_AVAIL BIT(26) +#define PRI_FIFOS_NOT_EMPTY BIT(25) +#define PRI_BUSY BIT(24) +#define CRI_RD_DATA_AVAIL BIT(18) +#define CRT_FIFOS_NOT_EMPTY BIT(17) +#define CRI_BUSY BIT(16) +#define IPI_FIFOS_NOT_EMPTY BIT(9) +#define IPI_BUSY BIT(8) +#define CORE_FIFOS_NOT_EMPTY BIT(1) +#define CORE_BUSY BIT(0) +#define MANUAL_MODE_CFG 0x0024 +#define MANUAL_MODE_EN BIT(0) +#define DSI2_TIMEOUT_HSTX_CFG 0x0048 +#define TO_HSTX(x) FIELD_PREP(GENMASK(15, 0), x) +#define DSI2_TIMEOUT_HSTXRDY_CFG 0x004c +#define TO_HSTXRDY(x) FIELD_PREP(GENMASK(15, 0), x) +#define DSI2_TIMEOUT_LPRX_CFG 0x0050 +#define TO_LPRXRDY(x) FIELD_PREP(GENMASK(15, 0), x) +#define DSI2_TIMEOUT_LPTXRDY_CFG 0x0054 +#define TO_LPTXRDY(x) FIELD_PREP(GENMASK(15, 0), x) +#define DSI2_TIMEOUT_LPTXTRIG_CFG 0x0058 +#define TO_LPTXTRIG(x) FIELD_PREP(GENMASK(15, 0), x) +#define DSI2_TIMEOUT_LPTXULPS_CFG 0x005c +#define TO_LPTXULPS(x) FIELD_PREP(GENMASK(15, 0), x) +#define DSI2_TIMEOUT_BTA_CFG 0x60 +#define TO_BTA(x) FIELD_PREP(GENMASK(15, 0), x) + +#define DSI2_PHY_MODE_CFG 0x0100 +#define PPI_WIDTH(x) FIELD_PREP(GENMASK(9, 8), x) +#define PHY_LANES(x) FIELD_PREP(GENMASK(5, 4), (x) - 1) +#define PHY_TYPE(x) FIELD_PREP(BIT(0), x) +#define DSI2_PHY_CLK_CFG 0X0104 +#define PHY_LPTX_CLK_DIV(x) FIELD_PREP(GENMASK(12, 8), x) +#define CLK_TYPE_MASK BIT(0) +#define NON_CONTINUOUS_CLK BIT(0) +#define CONTINUOUS_CLK 0 +#define DSI2_PHY_LP2HS_MAN_CFG 0x010c +#define PHY_LP2HS_TIME(x) FIELD_PREP(GENMASK(28, 0), x) +#define DSI2_PHY_HS2LP_MAN_CFG 0x0114 +#define PHY_HS2LP_TIME(x) FIELD_PREP(GENMASK(28, 0), x) +#define DSI2_PHY_MAX_RD_T_MAN_CFG 0x011c +#define PHY_MAX_RD_TIME(x) FIELD_PREP(GENMASK(26, 0), x) +#define DSI2_PHY_ESC_CMD_T_MAN_CFG 0x0124 +#define PHY_ESC_CMD_TIME(x) FIELD_PREP(GENMASK(28, 0), x) +#define DSI2_PHY_ESC_BYTE_T_MAN_CFG 0x012c +#define PHY_ESC_BYTE_TIME(x) FIELD_PREP(GENMASK(28, 0), x) + +#define DSI2_PHY_IPI_RATIO_MAN_CFG 0x0134 +#define PHY_IPI_RATIO(x) FIELD_PREP(GENMASK(21, 0), x) +#define DSI2_PHY_SYS_RATIO_MAN_CFG 0x013C +#define PHY_SYS_RATIO(x) FIELD_PREP(GENMASK(16, 0), x) + +#define DSI2_DSI_GENERAL_CFG 0x0200 +#define BTA_EN BIT(1) +#define EOTP_TX_EN BIT(0) +#define DSI2_DSI_VCID_CFG 0x0204 +#define TX_VCID(x) FIELD_PREP(GENMASK(1, 0), x) +#define DSI2_DSI_SCRAMBLING_CFG 0x0208 +#define SCRAMBLING_SEED(x) FIELD_PREP(GENMASK(31, 16), x) +#define SCRAMBLING_EN BIT(0) +#define DSI2_DSI_VID_TX_CFG 0x020c +#define LPDT_DISPLAY_CMD_EN BIT(20) +#define BLK_VFP_HS_EN BIT(14) +#define BLK_VBP_HS_EN BIT(13) +#define BLK_VSA_HS_EN BIT(12) +#define BLK_HFP_HS_EN BIT(6) +#define BLK_HBP_HS_EN BIT(5) +#define BLK_HSA_HS_EN BIT(4) +#define VID_MODE_TYPE(x) FIELD_PREP(GENMASK(1, 0), x) +#define DSI2_CRI_TX_HDR 0x02c0 +#define CMD_TX_MODE(x) FIELD_PREP(BIT(24), x) +#define DSI2_CRI_TX_PLD 0x02c4 +#define DSI2_CRI_RX_HDR 0x02c8 +#define DSI2_CRI_RX_PLD 0x02cc + +#define DSI2_IPI_COLOR_MAN_CFG 0x0300 +#define IPI_DEPTH(x) FIELD_PREP(GENMASK(7, 4), x) +#define IPI_DEPTH_5_6_5_BITS 0x02 +#define IPI_DEPTH_6_BITS 0x03 +#define IPI_DEPTH_8_BITS 0x05 +#define IPI_DEPTH_10_BITS 0x06 +#define IPI_FORMAT(x) FIELD_PREP(GENMASK(3, 0), x) +#define IPI_FORMAT_RGB 0x0 +#define IPI_FORMAT_DSC 0x0b +#define DSI2_IPI_VID_HSA_MAN_CFG 0x0304 +#define VID_HSA_TIME(x) FIELD_PREP(GENMASK(29, 0), x) +#define DSI2_IPI_VID_HBP_MAN_CFG 0x030c +#define VID_HBP_TIME(x) FIELD_PREP(GENMASK(29, 0), x) +#define DSI2_IPI_VID_HACT_MAN_CFG 0x0314 +#define VID_HACT_TIME(x) FIELD_PREP(GENMASK(29, 0), x) +#define DSI2_IPI_VID_HLINE_MAN_CFG 0x031c +#define VID_HLINE_TIME(x) FIELD_PREP(GENMASK(29, 0), x) +#define DSI2_IPI_VID_VSA_MAN_CFG 0x0324 +#define VID_VSA_LINES(x) FIELD_PREP(GENMASK(9, 0), x) +#define DSI2_IPI_VID_VBP_MAN_CFG 0X032C +#define VID_VBP_LINES(x) FIELD_PREP(GENMASK(9, 0), x) +#define DSI2_IPI_VID_VACT_MAN_CFG 0X0334 +#define VID_VACT_LINES(x) FIELD_PREP(GENMASK(13, 0), x) +#define DSI2_IPI_VID_VFP_MAN_CFG 0X033C +#define VID_VFP_LINES(x) FIELD_PREP(GENMASK(9, 0), x) +#define DSI2_IPI_PIX_PKT_CFG 0x0344 +#define MAX_PIX_PKT(x) FIELD_PREP(GENMASK(15, 0), x) + +#define DSI2_INT_ST_PHY 0x0400 +#define DSI2_INT_MASK_PHY 0x0404 +#define DSI2_INT_ST_TO 0x0410 +#define DSI2_INT_MASK_TO 0x0414 +#define DSI2_INT_ST_ACK 0x0420 +#define DSI2_INT_MASK_ACK 0x0424 +#define DSI2_INT_ST_IPI 0x0430 +#define DSI2_INT_MASK_IPI 0x0434 +#define DSI2_INT_ST_FIFO 0x0440 +#define DSI2_INT_MASK_FIFO 0x0444 +#define DSI2_INT_ST_PRI 0x0450 +#define DSI2_INT_MASK_PRI 0x0454 +#define DSI2_INT_ST_CRI 0x0460 +#define DSI2_INT_MASK_CRI 0x0464 +#define DSI2_INT_FORCE_CRI 0x0468 +#define DSI2_MAX_REGISGER DSI2_INT_FORCE_CRI + +#define MODE_STATUS_TIMEOUT_US 10000 +#define CMD_PKT_STATUS_TIMEOUT_US 20000 + +enum vid_mode_type { + VID_MODE_TYPE_NON_BURST_SYNC_PULSES, + VID_MODE_TYPE_NON_BURST_SYNC_EVENTS, + VID_MODE_TYPE_BURST, +}; + +enum mode_ctrl { + IDLE_MODE, + AUTOCALC_MODE, + COMMAND_MODE, + VIDEO_MODE, + DATA_STREAM_MODE, + VIDEO_TEST_MODE, + DATA_STREAM_TEST_MODE, +}; + +enum ppi_width { + PPI_WIDTH_8_BITS, + PPI_WIDTH_16_BITS, + PPI_WIDTH_32_BITS, +}; + +struct cmd_header { + u8 cmd_type; + u8 delay; + u8 payload_length; +}; + +struct dw_mipi_dsi2 { + struct drm_bridge bridge; + struct mipi_dsi_host dsi_host; + struct drm_bridge *panel_bridge; + struct device *dev; + struct regmap *regmap; + struct clk *pclk; + struct clk *sys_clk; + + unsigned int lane_mbps; /* per lane */ + u32 channel; + u32 lanes; + u32 format; + unsigned long mode_flags; + + struct drm_display_mode mode; + const struct dw_mipi_dsi2_plat_data *plat_data; +}; + +static inline struct dw_mipi_dsi2 *host_to_dsi2(struct mipi_dsi_host *host) +{ + return container_of(host, struct dw_mipi_dsi2, dsi_host); +} + +static inline struct dw_mipi_dsi2 *bridge_to_dsi2(struct drm_bridge *bridge) +{ + return container_of(bridge, struct dw_mipi_dsi2, bridge); +} + +static int cri_fifos_wait_avail(struct dw_mipi_dsi2 *dsi2) +{ + u32 sts, mask; + int ret; + + mask = CRI_BUSY | CRT_FIFOS_NOT_EMPTY; + ret = regmap_read_poll_timeout(dsi2->regmap, DSI2_CORE_STATUS, sts, + !(sts & mask), 0, CMD_PKT_STATUS_TIMEOUT_US); + if (ret < 0) { + dev_err(dsi2->dev, "command interface is busy\n"); + return ret; + } + + return 0; +} + +static void dw_mipi_dsi2_set_vid_mode(struct dw_mipi_dsi2 *dsi2) +{ + u32 val = 0, mode; + int ret; + + if (dsi2->mode_flags & MIPI_DSI_MODE_VIDEO_NO_HFP) + val |= BLK_HFP_HS_EN; + + if (dsi2->mode_flags & MIPI_DSI_MODE_VIDEO_NO_HBP) + val |= BLK_HBP_HS_EN; + + if (dsi2->mode_flags & MIPI_DSI_MODE_VIDEO_NO_HSA) + val |= BLK_HSA_HS_EN; + + if (dsi2->mode_flags & MIPI_DSI_MODE_VIDEO_BURST) + val |= VID_MODE_TYPE_BURST; + else if (dsi2->mode_flags & MIPI_DSI_MODE_VIDEO_SYNC_PULSE) + val |= VID_MODE_TYPE_NON_BURST_SYNC_PULSES; + else + val |= VID_MODE_TYPE_NON_BURST_SYNC_EVENTS; + + regmap_write(dsi2->regmap, DSI2_DSI_VID_TX_CFG, val); + + regmap_write(dsi2->regmap, DSI2_MODE_CTRL, VIDEO_MODE); + ret = regmap_read_poll_timeout(dsi2->regmap, DSI2_MODE_STATUS, + mode, mode & VIDEO_MODE, + 1000, MODE_STATUS_TIMEOUT_US); + if (ret < 0) + dev_err(dsi2->dev, "failed to enter video mode\n"); +} + +static void dw_mipi_dsi2_set_data_stream_mode(struct dw_mipi_dsi2 *dsi2) +{ + u32 mode; + int ret; + + regmap_write(dsi2->regmap, DSI2_MODE_CTRL, DATA_STREAM_MODE); + ret = regmap_read_poll_timeout(dsi2->regmap, DSI2_MODE_STATUS, + mode, mode & DATA_STREAM_MODE, + 1000, MODE_STATUS_TIMEOUT_US); + if (ret < 0) + dev_err(dsi2->dev, "failed to enter data stream mode\n"); +} + +static void dw_mipi_dsi2_set_cmd_mode(struct dw_mipi_dsi2 *dsi2) +{ + u32 mode; + int ret; + + regmap_write(dsi2->regmap, DSI2_MODE_CTRL, COMMAND_MODE); + ret = regmap_read_poll_timeout(dsi2->regmap, DSI2_MODE_STATUS, + mode, mode & COMMAND_MODE, + 1000, MODE_STATUS_TIMEOUT_US); + if (ret < 0) + dev_err(dsi2->dev, "failed to enter data stream mode\n"); +} + +static void dw_mipi_dsi2_host_softrst(struct dw_mipi_dsi2 *dsi2) +{ + regmap_write(dsi2->regmap, DSI2_SOFT_RESET, 0x0); + usleep_range(50, 100); + regmap_write(dsi2->regmap, DSI2_SOFT_RESET, + SYS_RSTN | PHY_RSTN | IPI_RSTN); +} + +static void dw_mipi_dsi2_phy_clk_mode_cfg(struct dw_mipi_dsi2 *dsi2) +{ + u32 sys_clk, esc_clk_div; + u32 val = 0; + + /* + * clk_type should be NON_CONTINUOUS_CLK before + * initial deskew calibration be sent. + */ + val |= NON_CONTINUOUS_CLK; + + /* The maximum value of the escape clock frequency is 20MHz */ + sys_clk = clk_get_rate(dsi2->sys_clk) / USEC_PER_SEC; + esc_clk_div = DIV_ROUND_UP(sys_clk, 20 * 2); + val |= PHY_LPTX_CLK_DIV(esc_clk_div); + + regmap_write(dsi2->regmap, DSI2_PHY_CLK_CFG, val); +} + +static void dw_mipi_dsi2_phy_ratio_cfg(struct dw_mipi_dsi2 *dsi2) +{ + struct drm_display_mode *mode = &dsi2->mode; + u64 sys_clk = clk_get_rate(dsi2->sys_clk); + u64 pixel_clk, ipi_clk, phy_hsclk; + u64 tmp; + + /* + * in DPHY mode, the phy_hstx_clk is exactly 1/16 the Lane high-speed + * data rate; In CPHY mode, the phy_hstx_clk is exactly 1/7 the trio + * high speed symbol rate. + */ + phy_hsclk = DIV_ROUND_CLOSEST_ULL(dsi2->lane_mbps * USEC_PER_SEC, 16); + + /* IPI_RATIO_MAN_CFG = PHY_HSTX_CLK / IPI_CLK */ + pixel_clk = mode->crtc_clock * MSEC_PER_SEC; + ipi_clk = pixel_clk / 4; + + tmp = DIV_ROUND_CLOSEST_ULL(phy_hsclk << 16, ipi_clk); + regmap_write(dsi2->regmap, DSI2_PHY_IPI_RATIO_MAN_CFG, + PHY_IPI_RATIO(tmp)); + + /* + * SYS_RATIO_MAN_CFG = MIPI_DCPHY_HSCLK_Freq / MIPI_DCPHY_HSCLK_Freq + */ + tmp = DIV_ROUND_CLOSEST_ULL(phy_hsclk << 16, sys_clk); + regmap_write(dsi2->regmap, DSI2_PHY_SYS_RATIO_MAN_CFG, + PHY_SYS_RATIO(tmp)); +} + +static void dw_mipi_dsi2_lp2hs_or_hs2lp_cfg(struct dw_mipi_dsi2 *dsi2) +{ + const struct dw_mipi_dsi2_phy_ops *phy_ops = dsi2->plat_data->phy_ops; + struct dw_mipi_dsi2_phy_timing timing; + int ret; + + ret = phy_ops->get_timing(dsi2->plat_data->priv_data, + dsi2->lane_mbps, &timing); + if (ret) + dev_err(dsi2->dev, "Retrieving phy timings failed\n"); + + regmap_write(dsi2->regmap, DSI2_PHY_LP2HS_MAN_CFG, PHY_LP2HS_TIME(timing.data_lp2hs)); + regmap_write(dsi2->regmap, DSI2_PHY_HS2LP_MAN_CFG, PHY_HS2LP_TIME(timing.data_hs2lp)); +} + +static void dw_mipi_dsi2_phy_init(struct dw_mipi_dsi2 *dsi2) +{ + const struct dw_mipi_dsi2_phy_ops *phy_ops = dsi2->plat_data->phy_ops; + struct dw_mipi_dsi2_phy_iface iface; + u32 val = 0; + + phy_ops->get_interface(dsi2->plat_data->priv_data, &iface); + + switch (iface.ppi_width) { + case 8: + val |= PPI_WIDTH(PPI_WIDTH_8_BITS); + break; + case 16: + val |= PPI_WIDTH(PPI_WIDTH_16_BITS); + break; + case 32: + val |= PPI_WIDTH(PPI_WIDTH_32_BITS); + break; + default: + /* Caught in probe */ + break; + } + + val |= PHY_LANES(dsi2->lanes); + val |= PHY_TYPE(DW_MIPI_DSI2_DPHY); + regmap_write(dsi2->regmap, DSI2_PHY_MODE_CFG, val); + + dw_mipi_dsi2_phy_clk_mode_cfg(dsi2); + dw_mipi_dsi2_phy_ratio_cfg(dsi2); + dw_mipi_dsi2_lp2hs_or_hs2lp_cfg(dsi2); + + /* phy configuration 8 - 10 */ +} + +static void dw_mipi_dsi2_tx_option_set(struct dw_mipi_dsi2 *dsi2) +{ + u32 val; + + val = BTA_EN | EOTP_TX_EN; + + if (dsi2->mode_flags & MIPI_DSI_MODE_NO_EOT_PACKET) + val &= ~EOTP_TX_EN; + + regmap_write(dsi2->regmap, DSI2_DSI_GENERAL_CFG, val); + regmap_write(dsi2->regmap, DSI2_DSI_VCID_CFG, TX_VCID(dsi2->channel)); +} + +static void dw_mipi_dsi2_ipi_color_coding_cfg(struct dw_mipi_dsi2 *dsi2) +{ + u32 val, color_depth; + + switch (dsi2->format) { + case MIPI_DSI_FMT_RGB666: + case MIPI_DSI_FMT_RGB666_PACKED: + color_depth = IPI_DEPTH_6_BITS; + break; + case MIPI_DSI_FMT_RGB565: + color_depth = IPI_DEPTH_5_6_5_BITS; + break; + case MIPI_DSI_FMT_RGB888: + default: + color_depth = IPI_DEPTH_8_BITS; + break; + } + + val = IPI_DEPTH(color_depth) | + IPI_FORMAT(IPI_FORMAT_RGB); + regmap_write(dsi2->regmap, DSI2_IPI_COLOR_MAN_CFG, val); +} + +static void dw_mipi_dsi2_vertical_timing_config(struct dw_mipi_dsi2 *dsi2, + const struct drm_display_mode *mode) +{ + u32 vactive, vsa, vfp, vbp; + + vactive = mode->vdisplay; + vsa = mode->vsync_end - mode->vsync_start; + vfp = mode->vsync_start - mode->vdisplay; + vbp = mode->vtotal - mode->vsync_end; + + regmap_write(dsi2->regmap, DSI2_IPI_VID_VSA_MAN_CFG, VID_VSA_LINES(vsa)); + regmap_write(dsi2->regmap, DSI2_IPI_VID_VBP_MAN_CFG, VID_VBP_LINES(vbp)); + regmap_write(dsi2->regmap, DSI2_IPI_VID_VACT_MAN_CFG, VID_VACT_LINES(vactive)); + regmap_write(dsi2->regmap, DSI2_IPI_VID_VFP_MAN_CFG, VID_VFP_LINES(vfp)); +} + +static void dw_mipi_dsi2_ipi_set(struct dw_mipi_dsi2 *dsi2) +{ + struct drm_display_mode *mode = &dsi2->mode; + u32 hline, hsa, hbp, hact; + u64 hline_time, hsa_time, hbp_time, hact_time, tmp; + u64 pixel_clk, phy_hs_clk; + u16 val; + + val = mode->hdisplay; + + regmap_write(dsi2->regmap, DSI2_IPI_PIX_PKT_CFG, MAX_PIX_PKT(val)); + + dw_mipi_dsi2_ipi_color_coding_cfg(dsi2); + + /* + * if the controller is intended to operate in data stream mode, + * no more steps are required. + */ + if (!(dsi2->mode_flags & MIPI_DSI_MODE_VIDEO)) + return; + + hact = mode->hdisplay; + hsa = mode->hsync_end - mode->hsync_start; + hbp = mode->htotal - mode->hsync_end; + hline = mode->htotal; + + pixel_clk = mode->crtc_clock * MSEC_PER_SEC; + + phy_hs_clk = DIV_ROUND_CLOSEST_ULL(dsi2->lane_mbps * USEC_PER_SEC, 16); + + tmp = hsa * phy_hs_clk; + hsa_time = DIV_ROUND_CLOSEST_ULL(tmp << 16, pixel_clk); + regmap_write(dsi2->regmap, DSI2_IPI_VID_HSA_MAN_CFG, VID_HSA_TIME(hsa_time)); + + tmp = hbp * phy_hs_clk; + hbp_time = DIV_ROUND_CLOSEST_ULL(tmp << 16, pixel_clk); + regmap_write(dsi2->regmap, DSI2_IPI_VID_HBP_MAN_CFG, VID_HBP_TIME(hbp_time)); + + tmp = hact * phy_hs_clk; + hact_time = DIV_ROUND_CLOSEST_ULL(tmp << 16, pixel_clk); + regmap_write(dsi2->regmap, DSI2_IPI_VID_HACT_MAN_CFG, VID_HACT_TIME(hact_time)); + + tmp = hline * phy_hs_clk; + hline_time = DIV_ROUND_CLOSEST_ULL(tmp << 16, pixel_clk); + regmap_write(dsi2->regmap, DSI2_IPI_VID_HLINE_MAN_CFG, VID_HLINE_TIME(hline_time)); + + dw_mipi_dsi2_vertical_timing_config(dsi2, mode); +} + +static void +dw_mipi_dsi2_work_mode(struct dw_mipi_dsi2 *dsi2, u32 mode) +{ + /* + * select controller work in Manual mode + * Manual: MANUAL_MODE_EN + * Automatic: 0 + */ + regmap_write(dsi2->regmap, MANUAL_MODE_CFG, mode); +} + +static int dw_mipi_dsi2_host_attach(struct mipi_dsi_host *host, + struct mipi_dsi_device *device) +{ + struct dw_mipi_dsi2 *dsi2 = host_to_dsi2(host); + const struct dw_mipi_dsi2_plat_data *pdata = dsi2->plat_data; + struct drm_bridge *bridge; + int ret; + + if (device->lanes > dsi2->plat_data->max_data_lanes) { + dev_err(dsi2->dev, "the number of data lanes(%u) is too many\n", + device->lanes); + return -EINVAL; + } + + dsi2->lanes = device->lanes; + dsi2->channel = device->channel; + dsi2->format = device->format; + dsi2->mode_flags = device->mode_flags; + + bridge = devm_drm_of_get_bridge(dsi2->dev, dsi2->dev->of_node, 1, 0); + if (IS_ERR(bridge)) + return PTR_ERR(bridge); + + bridge->pre_enable_prev_first = true; + dsi2->panel_bridge = bridge; + + drm_bridge_add(&dsi2->bridge); + + if (pdata->host_ops && pdata->host_ops->attach) { + ret = pdata->host_ops->attach(pdata->priv_data, device); + if (ret < 0) + return ret; + } + + return 0; +} + +static int dw_mipi_dsi2_host_detach(struct mipi_dsi_host *host, + struct mipi_dsi_device *device) +{ + struct dw_mipi_dsi2 *dsi2 = host_to_dsi2(host); + const struct dw_mipi_dsi2_plat_data *pdata = dsi2->plat_data; + int ret; + + if (pdata->host_ops && pdata->host_ops->detach) { + ret = pdata->host_ops->detach(pdata->priv_data, device); + if (ret < 0) + return ret; + } + + drm_bridge_remove(&dsi2->bridge); + + drm_of_panel_bridge_remove(host->dev->of_node, 1, 0); + + return 0; +} + +static int dw_mipi_dsi2_gen_pkt_hdr_write(struct dw_mipi_dsi2 *dsi2, + u32 hdr_val, bool lpm) +{ + int ret; + + regmap_write(dsi2->regmap, DSI2_CRI_TX_HDR, hdr_val | CMD_TX_MODE(lpm)); + + ret = cri_fifos_wait_avail(dsi2); + if (ret) { + dev_err(dsi2->dev, "failed to write command header\n"); + return ret; + } + + return 0; +} + +static int dw_mipi_dsi2_write(struct dw_mipi_dsi2 *dsi2, + const struct mipi_dsi_packet *packet, bool lpm) +{ + const u8 *tx_buf = packet->payload; + int len = packet->payload_length, pld_data_bytes = sizeof(u32); + __le32 word; + + /* Send payload */ + while (len) { + if (len < pld_data_bytes) { + word = 0; + memcpy(&word, tx_buf, len); + regmap_write(dsi2->regmap, DSI2_CRI_TX_PLD, le32_to_cpu(word)); + len = 0; + } else { + memcpy(&word, tx_buf, pld_data_bytes); + regmap_write(dsi2->regmap, DSI2_CRI_TX_PLD, le32_to_cpu(word)); + tx_buf += pld_data_bytes; + len -= pld_data_bytes; + } + } + + word = 0; + memcpy(&word, packet->header, sizeof(packet->header)); + return dw_mipi_dsi2_gen_pkt_hdr_write(dsi2, le32_to_cpu(word), lpm); +} + +static int dw_mipi_dsi2_read(struct dw_mipi_dsi2 *dsi2, + const struct mipi_dsi_msg *msg) +{ + u8 *payload = msg->rx_buf; + int i, j, ret, len = msg->rx_len; + u8 data_type; + u16 wc; + u32 val; + + ret = regmap_read_poll_timeout(dsi2->regmap, DSI2_CORE_STATUS, + val, val & CRI_RD_DATA_AVAIL, + 100, CMD_PKT_STATUS_TIMEOUT_US); + if (ret) { + dev_err(dsi2->dev, "CRI has no available read data\n"); + return ret; + } + + regmap_read(dsi2->regmap, DSI2_CRI_RX_HDR, &val); + data_type = val & 0x3f; + + if (mipi_dsi_packet_format_is_short(data_type)) { + for (i = 0; i < len && i < 2; i++) + payload[i] = (val >> (8 * (i + 1))) & 0xff; + + return 0; + } + + wc = (val >> 8) & 0xffff; + /* Receive payload */ + for (i = 0; i < len && i < wc; i += 4) { + regmap_read(dsi2->regmap, DSI2_CRI_RX_PLD, &val); + for (j = 0; j < 4 && j + i < len && j + i < wc; j++) + payload[i + j] = val >> (8 * j); + } + + return 0; +} + +static ssize_t dw_mipi_dsi2_host_transfer(struct mipi_dsi_host *host, + const struct mipi_dsi_msg *msg) +{ + struct dw_mipi_dsi2 *dsi2 = host_to_dsi2(host); + bool lpm = msg->flags & MIPI_DSI_MSG_USE_LPM; + struct mipi_dsi_packet packet; + int ret, nb_bytes; + + regmap_update_bits(dsi2->regmap, DSI2_DSI_VID_TX_CFG, + LPDT_DISPLAY_CMD_EN, + lpm ? LPDT_DISPLAY_CMD_EN : 0); + + /* create a packet to the DSI protocol */ + ret = mipi_dsi_create_packet(&packet, msg); + if (ret) { + dev_err(dsi2->dev, "failed to create packet: %d\n", ret); + return ret; + } + + ret = cri_fifos_wait_avail(dsi2); + if (ret) + return ret; + + ret = dw_mipi_dsi2_write(dsi2, &packet, lpm); + if (ret) + return ret; + + if (msg->rx_buf && msg->rx_len) { + ret = dw_mipi_dsi2_read(dsi2, msg); + if (ret < 0) + return ret; + nb_bytes = msg->rx_len; + } else { + nb_bytes = packet.size; + } + + return nb_bytes; +} + +static const struct mipi_dsi_host_ops dw_mipi_dsi2_host_ops = { + .attach = dw_mipi_dsi2_host_attach, + .detach = dw_mipi_dsi2_host_detach, + .transfer = dw_mipi_dsi2_host_transfer, +}; + +static u32 * +dw_mipi_dsi2_bridge_atomic_get_input_bus_fmts(struct drm_bridge *bridge, + struct drm_bridge_state *bridge_state, + struct drm_crtc_state *crtc_state, + struct drm_connector_state *conn_state, + u32 output_fmt, + unsigned int *num_input_fmts) +{ + struct dw_mipi_dsi2 *dsi2 = bridge_to_dsi2(bridge); + const struct dw_mipi_dsi2_plat_data *pdata = dsi2->plat_data; + u32 *input_fmts; + + if (pdata->get_input_bus_fmts) + return pdata->get_input_bus_fmts(pdata->priv_data, + bridge, bridge_state, + crtc_state, conn_state, + output_fmt, num_input_fmts); + + /* Fall back to MEDIA_BUS_FMT_FIXED as the only input format. */ + input_fmts = kmalloc(sizeof(*input_fmts), GFP_KERNEL); + if (!input_fmts) + return NULL; + input_fmts[0] = MEDIA_BUS_FMT_FIXED; + *num_input_fmts = 1; + + return input_fmts; +} + +static int dw_mipi_dsi2_bridge_atomic_check(struct drm_bridge *bridge, + struct drm_bridge_state *bridge_state, + struct drm_crtc_state *crtc_state, + struct drm_connector_state *conn_state) +{ + struct dw_mipi_dsi2 *dsi2 = bridge_to_dsi2(bridge); + const struct dw_mipi_dsi2_plat_data *pdata = dsi2->plat_data; + bool ret; + + bridge_state->input_bus_cfg.flags = + DRM_BUS_FLAG_DE_HIGH | DRM_BUS_FLAG_PIXDATA_SAMPLE_NEGEDGE; + + if (pdata->mode_fixup) { + ret = pdata->mode_fixup(pdata->priv_data, &crtc_state->mode, + &crtc_state->adjusted_mode); + if (!ret) { + DRM_DEBUG_DRIVER("failed to fixup mode " DRM_MODE_FMT "\n", + DRM_MODE_ARG(&crtc_state->mode)); + return -EINVAL; + } + } + + return 0; +} + +static void dw_mipi_dsi2_bridge_post_atomic_disable(struct drm_bridge *bridge, + struct drm_bridge_state *old_bridge_state) +{ + struct dw_mipi_dsi2 *dsi2 = bridge_to_dsi2(bridge); + const struct dw_mipi_dsi2_phy_ops *phy_ops = dsi2->plat_data->phy_ops; + + regmap_write(dsi2->regmap, DSI2_IPI_PIX_PKT_CFG, 0); + + /* + * Switch to command mode before panel-bridge post_disable & + * panel unprepare. + * Note: panel-bridge disable & panel disable has been called + * before by the drm framework. + */ + dw_mipi_dsi2_set_cmd_mode(dsi2); + + regmap_write(dsi2->regmap, DSI2_PWR_UP, RESET); + + if (phy_ops->power_off) + phy_ops->power_off(dsi2->plat_data->priv_data); + + clk_disable_unprepare(dsi2->sys_clk); + clk_disable_unprepare(dsi2->pclk); + pm_runtime_put(dsi2->dev); +} + +static unsigned int dw_mipi_dsi2_get_lanes(struct dw_mipi_dsi2 *dsi2) +{ + /* single-dsi, so no other instance to consider */ + return dsi2->lanes; +} + +static void dw_mipi_dsi2_mode_set(struct dw_mipi_dsi2 *dsi2, + const struct drm_display_mode *adjusted_mode) +{ + const struct dw_mipi_dsi2_phy_ops *phy_ops = dsi2->plat_data->phy_ops; + void *priv_data = dsi2->plat_data->priv_data; + u32 lanes = dw_mipi_dsi2_get_lanes(dsi2); + int ret; + + clk_prepare_enable(dsi2->pclk); + clk_prepare_enable(dsi2->sys_clk); + + ret = phy_ops->get_lane_mbps(priv_data, adjusted_mode, dsi2->mode_flags, + lanes, dsi2->format, &dsi2->lane_mbps); + if (ret) + DRM_DEBUG_DRIVER("Phy get_lane_mbps() failed\n"); + + pm_runtime_get_sync(dsi2->dev); + + dw_mipi_dsi2_host_softrst(dsi2); + regmap_write(dsi2->regmap, DSI2_PWR_UP, RESET); + + dw_mipi_dsi2_work_mode(dsi2, MANUAL_MODE_EN); + dw_mipi_dsi2_phy_init(dsi2); + + if (phy_ops->power_on) + phy_ops->power_on(dsi2->plat_data->priv_data); + + dw_mipi_dsi2_tx_option_set(dsi2); + + /* + * initial deskew calibration is send after phy_power_on, + * then we can configure clk_type. + */ + + regmap_update_bits(dsi2->regmap, DSI2_PHY_CLK_CFG, CLK_TYPE_MASK, + dsi2->mode_flags & MIPI_DSI_CLOCK_NON_CONTINUOUS ? NON_CONTINUOUS_CLK : + CONTINUOUS_CLK); + + regmap_write(dsi2->regmap, DSI2_PWR_UP, POWER_UP); + dw_mipi_dsi2_set_cmd_mode(dsi2); + + dw_mipi_dsi2_ipi_set(dsi2); +} + +static void dw_mipi_dsi2_bridge_atomic_pre_enable(struct drm_bridge *bridge, + struct drm_bridge_state *old_bridge_state) +{ + struct dw_mipi_dsi2 *dsi2 = bridge_to_dsi2(bridge); + + /* Power up the dsi ctl into a command mode */ + dw_mipi_dsi2_mode_set(dsi2, &dsi2->mode); +} + +static void dw_mipi_dsi2_bridge_mode_set(struct drm_bridge *bridge, + const struct drm_display_mode *mode, + const struct drm_display_mode *adjusted_mode) +{ + struct dw_mipi_dsi2 *dsi2 = bridge_to_dsi2(bridge); + + /* Store the display mode for later use in pre_enable callback */ + drm_mode_copy(&dsi2->mode, adjusted_mode); +} + +static void dw_mipi_dsi2_bridge_atomic_enable(struct drm_bridge *bridge, + struct drm_bridge_state *old_bridge_state) +{ + struct dw_mipi_dsi2 *dsi2 = bridge_to_dsi2(bridge); + + /* Switch to video mode for panel-bridge enable & panel enable */ + if (dsi2->mode_flags & MIPI_DSI_MODE_VIDEO) + dw_mipi_dsi2_set_vid_mode(dsi2); + else + dw_mipi_dsi2_set_data_stream_mode(dsi2); +} + +static enum drm_mode_status +dw_mipi_dsi2_bridge_mode_valid(struct drm_bridge *bridge, + const struct drm_display_info *info, + const struct drm_display_mode *mode) +{ + struct dw_mipi_dsi2 *dsi2 = bridge_to_dsi2(bridge); + const struct dw_mipi_dsi2_plat_data *pdata = dsi2->plat_data; + enum drm_mode_status mode_status = MODE_OK; + + if (pdata->mode_valid) + mode_status = pdata->mode_valid(pdata->priv_data, mode, + dsi2->mode_flags, + dw_mipi_dsi2_get_lanes(dsi2), + dsi2->format); + + return mode_status; +} + +static int dw_mipi_dsi2_bridge_attach(struct drm_bridge *bridge, + enum drm_bridge_attach_flags flags) +{ + struct dw_mipi_dsi2 *dsi2 = bridge_to_dsi2(bridge); + + /* Set the encoder type as caller does not know it */ + bridge->encoder->encoder_type = DRM_MODE_ENCODER_DSI; + + /* Attach the panel-bridge to the dsi bridge */ + return drm_bridge_attach(bridge->encoder, dsi2->panel_bridge, bridge, + flags); +} + +static const struct drm_bridge_funcs dw_mipi_dsi2_bridge_funcs = { + .atomic_duplicate_state = drm_atomic_helper_bridge_duplicate_state, + .atomic_destroy_state = drm_atomic_helper_bridge_destroy_state, + .atomic_get_input_bus_fmts = dw_mipi_dsi2_bridge_atomic_get_input_bus_fmts, + .atomic_check = dw_mipi_dsi2_bridge_atomic_check, + .atomic_reset = drm_atomic_helper_bridge_reset, + .atomic_pre_enable = dw_mipi_dsi2_bridge_atomic_pre_enable, + .atomic_enable = dw_mipi_dsi2_bridge_atomic_enable, + .atomic_post_disable = dw_mipi_dsi2_bridge_post_atomic_disable, + .mode_set = dw_mipi_dsi2_bridge_mode_set, + .mode_valid = dw_mipi_dsi2_bridge_mode_valid, + .attach = dw_mipi_dsi2_bridge_attach, +}; + +static const struct regmap_config dw_mipi_dsi2_regmap_config = { + .name = "dsi2-host", + .reg_bits = 32, + .val_bits = 32, + .reg_stride = 4, + .fast_io = true, +}; + +static struct dw_mipi_dsi2 * +__dw_mipi_dsi2_probe(struct platform_device *pdev, + const struct dw_mipi_dsi2_plat_data *plat_data) +{ + struct device *dev = &pdev->dev; + struct reset_control *apb_rst; + struct dw_mipi_dsi2 *dsi2; + int ret; + + dsi2 = devm_kzalloc(dev, sizeof(*dsi2), GFP_KERNEL); + if (!dsi2) + return ERR_PTR(-ENOMEM); + + dsi2->dev = dev; + dsi2->plat_data = plat_data; + + if (!plat_data->phy_ops->init || !plat_data->phy_ops->get_lane_mbps || + !plat_data->phy_ops->get_timing) + return dev_err_ptr_probe(dev, -ENODEV, "Phy not properly configured\n"); + + if (!plat_data->regmap) { + void __iomem *base = devm_platform_ioremap_resource(pdev, 0); + + if (IS_ERR(base)) + return dev_err_cast_probe(dev, base, "failed to registers\n"); + + dsi2->regmap = devm_regmap_init_mmio(dev, base, + &dw_mipi_dsi2_regmap_config); + if (IS_ERR(dsi2->regmap)) + return dev_err_cast_probe(dev, dsi2->regmap, "failed to init regmap\n"); + } else { + dsi2->regmap = plat_data->regmap; + } + + dsi2->pclk = devm_clk_get(dev, "pclk"); + if (IS_ERR(dsi2->pclk)) + return dev_err_cast_probe(dev, dsi2->pclk, "Unable to get pclk\n"); + + dsi2->sys_clk = devm_clk_get(dev, "sys"); + if (IS_ERR(dsi2->sys_clk)) + return dev_err_cast_probe(dev, dsi2->sys_clk, "Unable to get sys_clk\n"); + + /* + * Note that the reset was not defined in the initial device tree, so + * we have to be prepared for it not being found. + */ + apb_rst = devm_reset_control_get_optional_exclusive(dev, "apb"); + if (IS_ERR(apb_rst)) + return dev_err_cast_probe(dev, apb_rst, "Unable to get reset control\n"); + + if (apb_rst) { + ret = clk_prepare_enable(dsi2->pclk); + if (ret) { + dev_err(dev, "%s: Failed to enable pclk\n", __func__); + return ERR_PTR(ret); + } + + reset_control_assert(apb_rst); + usleep_range(10, 20); + reset_control_deassert(apb_rst); + + clk_disable_unprepare(dsi2->pclk); + } + + devm_pm_runtime_enable(dev); + + dsi2->dsi_host.ops = &dw_mipi_dsi2_host_ops; + dsi2->dsi_host.dev = dev; + ret = mipi_dsi_host_register(&dsi2->dsi_host); + if (ret) { + dev_err(dev, "Failed to register MIPI host: %d\n", ret); + pm_runtime_disable(dev); + return ERR_PTR(ret); + } + + dsi2->bridge.driver_private = dsi2; + dsi2->bridge.funcs = &dw_mipi_dsi2_bridge_funcs; + dsi2->bridge.of_node = pdev->dev.of_node; + + return dsi2; +} + +static void __dw_mipi_dsi2_remove(struct dw_mipi_dsi2 *dsi2) +{ + mipi_dsi_host_unregister(&dsi2->dsi_host); +} + +/* + * Probe/remove API, used to create the bridge instance. + */ +struct dw_mipi_dsi2 * +dw_mipi_dsi2_probe(struct platform_device *pdev, + const struct dw_mipi_dsi2_plat_data *plat_data) +{ + return __dw_mipi_dsi2_probe(pdev, plat_data); +} +EXPORT_SYMBOL_GPL(dw_mipi_dsi2_probe); + +void dw_mipi_dsi2_remove(struct dw_mipi_dsi2 *dsi2) +{ + __dw_mipi_dsi2_remove(dsi2); +} +EXPORT_SYMBOL_GPL(dw_mipi_dsi2_remove); + +/* + * Bind/unbind API, used from platforms based on the component framework + * to attach the bridge to an encoder. + */ +int dw_mipi_dsi2_bind(struct dw_mipi_dsi2 *dsi2, struct drm_encoder *encoder) +{ + return drm_bridge_attach(encoder, &dsi2->bridge, NULL, 0); +} +EXPORT_SYMBOL_GPL(dw_mipi_dsi2_bind); + +void dw_mipi_dsi2_unbind(struct dw_mipi_dsi2 *dsi2) +{ +} +EXPORT_SYMBOL_GPL(dw_mipi_dsi2_unbind); + +MODULE_AUTHOR("Guochun Huang <hero.huang@rock-chips.com>"); +MODULE_AUTHOR("Heiko Stuebner <heiko.stuebner@cherry.de>"); +MODULE_DESCRIPTION("DW MIPI DSI2 host controller driver"); +MODULE_LICENSE("GPL"); +MODULE_ALIAS("platform:dw-mipi-dsi2"); diff --git a/include/drm/bridge/dw_mipi_dsi2.h b/include/drm/bridge/dw_mipi_dsi2.h new file mode 100644 index 000000000000..c18c49379247 --- /dev/null +++ b/include/drm/bridge/dw_mipi_dsi2.h @@ -0,0 +1,95 @@ +/* SPDX-License-Identifier: GPL-2.0-only */ +/* + * Copyright (c) 2024, Fuzhou Rockchip Electronics Co., Ltd + * + * Authors: Guochun Huang <hero.huang@rock-chips.com> + * Heiko Stuebner <heiko.stuebner@cherry.de> + */ + +#ifndef __DW_MIPI_DSI2__ +#define __DW_MIPI_DSI2__ + +#include <linux/regmap.h> +#include <linux/types.h> + +#include <drm/drm_atomic.h> +#include <drm/drm_bridge.h> +#include <drm/drm_connector.h> +#include <drm/drm_crtc.h> +#include <drm/drm_modes.h> + +struct drm_display_mode; +struct drm_encoder; +struct dw_mipi_dsi2; +struct mipi_dsi_device; +struct platform_device; + +enum dw_mipi_dsi2_phy_type { + DW_MIPI_DSI2_DPHY, + DW_MIPI_DSI2_CPHY, +}; + +struct dw_mipi_dsi2_phy_iface { + int ppi_width; + enum dw_mipi_dsi2_phy_type phy_type; +}; + +struct dw_mipi_dsi2_phy_timing { + u32 data_hs2lp; + u32 data_lp2hs; +}; + +struct dw_mipi_dsi2_phy_ops { + int (*init)(void *priv_data); + void (*power_on)(void *priv_data); + void (*power_off)(void *priv_data); + void (*get_interface)(void *priv_data, struct dw_mipi_dsi2_phy_iface *iface); + int (*get_lane_mbps)(void *priv_data, + const struct drm_display_mode *mode, + unsigned long mode_flags, u32 lanes, u32 format, + unsigned int *lane_mbps); + int (*get_timing)(void *priv_data, unsigned int lane_mbps, + struct dw_mipi_dsi2_phy_timing *timing); + int (*get_esc_clk_rate)(void *priv_data, unsigned int *esc_clk_rate); +}; + +struct dw_mipi_dsi2_host_ops { + int (*attach)(void *priv_data, + struct mipi_dsi_device *dsi); + int (*detach)(void *priv_data, + struct mipi_dsi_device *dsi); +}; + +struct dw_mipi_dsi2_plat_data { + struct regmap *regmap; + unsigned int max_data_lanes; + + enum drm_mode_status (*mode_valid)(void *priv_data, + const struct drm_display_mode *mode, + unsigned long mode_flags, + u32 lanes, u32 format); + + bool (*mode_fixup)(void *priv_data, const struct drm_display_mode *mode, + struct drm_display_mode *adjusted_mode); + + u32 *(*get_input_bus_fmts)(void *priv_data, + struct drm_bridge *bridge, + struct drm_bridge_state *bridge_state, + struct drm_crtc_state *crtc_state, + struct drm_connector_state *conn_state, + u32 output_fmt, + unsigned int *num_input_fmts); + + const struct dw_mipi_dsi2_phy_ops *phy_ops; + const struct dw_mipi_dsi2_host_ops *host_ops; + + void *priv_data; +}; + +struct dw_mipi_dsi2 *dw_mipi_dsi2_probe(struct platform_device *pdev, + const struct dw_mipi_dsi2_plat_data *plat_data); +void dw_mipi_dsi2_remove(struct dw_mipi_dsi2 *dsi2); +int dw_mipi_dsi2_bind(struct dw_mipi_dsi2 *dsi2, struct drm_encoder *encoder); +void dw_mipi_dsi2_unbind(struct dw_mipi_dsi2 *dsi2); + +#endif /* __DW_MIPI_DSI2__ */ -- 2.45.2 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH v4 1/3] drm/bridge/synopsys: Add MIPI DSI2 host controller bridge 2024-12-09 23:10 ` [PATCH v4 1/3] drm/bridge/synopsys: Add MIPI DSI2 host controller bridge Heiko Stuebner @ 2024-12-10 8:50 ` Andy Yan 2024-12-30 2:20 ` Chris Hofstaedtler 2025-02-02 20:34 ` Pavel Golikov 2 siblings, 0 replies; 10+ messages in thread From: Andy Yan @ 2024-12-10 8:50 UTC (permalink / raw) To: Heiko Stuebner Cc: maarten.lankhorst, mripard, tzimmermann, robh, krzk+dt, conor+dt, andrzej.hajda, neil.armstrong, rfoss, Laurent.pinchart, jonas, jernej.skrabec, dri-devel, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel, quentin.schulz, Heiko Stuebner, Daniel Semkowicz, Dmitry Yashin Hello Heiko, On 12/10/24 07:10, Heiko Stuebner wrote: > From: Heiko Stuebner <heiko.stuebner@cherry.de> > > Add a Synopsys Designware MIPI DSI host DRM bridge driver for their > DSI2 host controller, based on the Rockchip version from the driver > rockchip/dw-mipi-dsi2.c in their vendor-kernel with phy & bridge APIs. > > While the driver is heavily modelled after the previous IP, the register > set of this DSI2 controller is completely different and there are also > additional properties like the variable-width phy interface. > > Tested-by: Daniel Semkowicz <dse@thaumatec.com> > Tested-by: Dmitry Yashin <dmt.yashin@gmail.com> > Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org> > Signed-off-by: Heiko Stuebner <heiko.stuebner@cherry.de> Reviewed-by: Andy Yan <andy.yan@rock-chips.com> > --- > drivers/gpu/drm/bridge/synopsys/Kconfig | 6 + > drivers/gpu/drm/bridge/synopsys/Makefile | 1 + > .../gpu/drm/bridge/synopsys/dw-mipi-dsi2.c | 1030 +++++++++++++++++ > include/drm/bridge/dw_mipi_dsi2.h | 95 ++ > 4 files changed, 1132 insertions(+) > create mode 100644 drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi2.c > create mode 100644 include/drm/bridge/dw_mipi_dsi2.h > > diff --git a/drivers/gpu/drm/bridge/synopsys/Kconfig b/drivers/gpu/drm/bridge/synopsys/Kconfig > index ca416dab156d..f3ab2f985f8c 100644 > --- a/drivers/gpu/drm/bridge/synopsys/Kconfig > +++ b/drivers/gpu/drm/bridge/synopsys/Kconfig > @@ -59,3 +59,9 @@ config DRM_DW_MIPI_DSI > select DRM_KMS_HELPER > select DRM_MIPI_DSI > select DRM_PANEL_BRIDGE > + > +config DRM_DW_MIPI_DSI2 > + tristate > + select DRM_KMS_HELPER > + select DRM_MIPI_DSI > + select DRM_PANEL_BRIDGE > diff --git a/drivers/gpu/drm/bridge/synopsys/Makefile b/drivers/gpu/drm/bridge/synopsys/Makefile > index 9869d9651ed1..9dc376d220ad 100644 > --- a/drivers/gpu/drm/bridge/synopsys/Makefile > +++ b/drivers/gpu/drm/bridge/synopsys/Makefile > @@ -8,3 +8,4 @@ obj-$(CONFIG_DRM_DW_HDMI_CEC) += dw-hdmi-cec.o > obj-$(CONFIG_DRM_DW_HDMI_QP) += dw-hdmi-qp.o > > obj-$(CONFIG_DRM_DW_MIPI_DSI) += dw-mipi-dsi.o > +obj-$(CONFIG_DRM_DW_MIPI_DSI2) += dw-mipi-dsi2.o > diff --git a/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi2.c b/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi2.c > new file mode 100644 > index 000000000000..054ad39c451b > --- /dev/null > +++ b/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi2.c > @@ -0,0 +1,1030 @@ > +// SPDX-License-Identifier: GPL-2.0+ > +/* > + * Copyright (c) 2024, Fuzhou Rockchip Electronics Co., Ltd > + * > + * Modified by Heiko Stuebner <heiko.stuebner@cherry.de> > + * This generic Synopsys DesignWare MIPI DSI2 host driver is based on the > + * Rockchip version from rockchip/dw-mipi-dsi2.c converted to use bridge APIs. > + */ > + > +#include <linux/bitfield.h> > +#include <linux/clk.h> > +#include <linux/iopoll.h> > +#include <linux/media-bus-format.h> > +#include <linux/module.h> > +#include <linux/platform_device.h> > +#include <linux/pm_runtime.h> > +#include <linux/reset.h> > + > +#include <video/mipi_display.h> > + > +#include <drm/bridge/dw_mipi_dsi2.h> > +#include <drm/drm_atomic_helper.h> > +#include <drm/drm_bridge.h> > +#include <drm/drm_mipi_dsi.h> > +#include <drm/drm_of.h> > +#include <drm/drm_print.h> > + > +#define DSI2_PWR_UP 0x000c > +#define RESET 0 > +#define POWER_UP BIT(0) > +#define CMD_TX_MODE(x) FIELD_PREP(BIT(24), x) > +#define DSI2_SOFT_RESET 0x0010 > +#define SYS_RSTN BIT(2) > +#define PHY_RSTN BIT(1) > +#define IPI_RSTN BIT(0) > +#define INT_ST_MAIN 0x0014 > +#define DSI2_MODE_CTRL 0x0018 > +#define DSI2_MODE_STATUS 0x001c > +#define DSI2_CORE_STATUS 0x0020 > +#define PRI_RD_DATA_AVAIL BIT(26) > +#define PRI_FIFOS_NOT_EMPTY BIT(25) > +#define PRI_BUSY BIT(24) > +#define CRI_RD_DATA_AVAIL BIT(18) > +#define CRT_FIFOS_NOT_EMPTY BIT(17) > +#define CRI_BUSY BIT(16) > +#define IPI_FIFOS_NOT_EMPTY BIT(9) > +#define IPI_BUSY BIT(8) > +#define CORE_FIFOS_NOT_EMPTY BIT(1) > +#define CORE_BUSY BIT(0) > +#define MANUAL_MODE_CFG 0x0024 > +#define MANUAL_MODE_EN BIT(0) > +#define DSI2_TIMEOUT_HSTX_CFG 0x0048 > +#define TO_HSTX(x) FIELD_PREP(GENMASK(15, 0), x) > +#define DSI2_TIMEOUT_HSTXRDY_CFG 0x004c > +#define TO_HSTXRDY(x) FIELD_PREP(GENMASK(15, 0), x) > +#define DSI2_TIMEOUT_LPRX_CFG 0x0050 > +#define TO_LPRXRDY(x) FIELD_PREP(GENMASK(15, 0), x) > +#define DSI2_TIMEOUT_LPTXRDY_CFG 0x0054 > +#define TO_LPTXRDY(x) FIELD_PREP(GENMASK(15, 0), x) > +#define DSI2_TIMEOUT_LPTXTRIG_CFG 0x0058 > +#define TO_LPTXTRIG(x) FIELD_PREP(GENMASK(15, 0), x) > +#define DSI2_TIMEOUT_LPTXULPS_CFG 0x005c > +#define TO_LPTXULPS(x) FIELD_PREP(GENMASK(15, 0), x) > +#define DSI2_TIMEOUT_BTA_CFG 0x60 > +#define TO_BTA(x) FIELD_PREP(GENMASK(15, 0), x) > + > +#define DSI2_PHY_MODE_CFG 0x0100 > +#define PPI_WIDTH(x) FIELD_PREP(GENMASK(9, 8), x) > +#define PHY_LANES(x) FIELD_PREP(GENMASK(5, 4), (x) - 1) > +#define PHY_TYPE(x) FIELD_PREP(BIT(0), x) > +#define DSI2_PHY_CLK_CFG 0X0104 > +#define PHY_LPTX_CLK_DIV(x) FIELD_PREP(GENMASK(12, 8), x) > +#define CLK_TYPE_MASK BIT(0) > +#define NON_CONTINUOUS_CLK BIT(0) > +#define CONTINUOUS_CLK 0 > +#define DSI2_PHY_LP2HS_MAN_CFG 0x010c > +#define PHY_LP2HS_TIME(x) FIELD_PREP(GENMASK(28, 0), x) > +#define DSI2_PHY_HS2LP_MAN_CFG 0x0114 > +#define PHY_HS2LP_TIME(x) FIELD_PREP(GENMASK(28, 0), x) > +#define DSI2_PHY_MAX_RD_T_MAN_CFG 0x011c > +#define PHY_MAX_RD_TIME(x) FIELD_PREP(GENMASK(26, 0), x) > +#define DSI2_PHY_ESC_CMD_T_MAN_CFG 0x0124 > +#define PHY_ESC_CMD_TIME(x) FIELD_PREP(GENMASK(28, 0), x) > +#define DSI2_PHY_ESC_BYTE_T_MAN_CFG 0x012c > +#define PHY_ESC_BYTE_TIME(x) FIELD_PREP(GENMASK(28, 0), x) > + > +#define DSI2_PHY_IPI_RATIO_MAN_CFG 0x0134 > +#define PHY_IPI_RATIO(x) FIELD_PREP(GENMASK(21, 0), x) > +#define DSI2_PHY_SYS_RATIO_MAN_CFG 0x013C > +#define PHY_SYS_RATIO(x) FIELD_PREP(GENMASK(16, 0), x) > + > +#define DSI2_DSI_GENERAL_CFG 0x0200 > +#define BTA_EN BIT(1) > +#define EOTP_TX_EN BIT(0) > +#define DSI2_DSI_VCID_CFG 0x0204 > +#define TX_VCID(x) FIELD_PREP(GENMASK(1, 0), x) > +#define DSI2_DSI_SCRAMBLING_CFG 0x0208 > +#define SCRAMBLING_SEED(x) FIELD_PREP(GENMASK(31, 16), x) > +#define SCRAMBLING_EN BIT(0) > +#define DSI2_DSI_VID_TX_CFG 0x020c > +#define LPDT_DISPLAY_CMD_EN BIT(20) > +#define BLK_VFP_HS_EN BIT(14) > +#define BLK_VBP_HS_EN BIT(13) > +#define BLK_VSA_HS_EN BIT(12) > +#define BLK_HFP_HS_EN BIT(6) > +#define BLK_HBP_HS_EN BIT(5) > +#define BLK_HSA_HS_EN BIT(4) > +#define VID_MODE_TYPE(x) FIELD_PREP(GENMASK(1, 0), x) > +#define DSI2_CRI_TX_HDR 0x02c0 > +#define CMD_TX_MODE(x) FIELD_PREP(BIT(24), x) > +#define DSI2_CRI_TX_PLD 0x02c4 > +#define DSI2_CRI_RX_HDR 0x02c8 > +#define DSI2_CRI_RX_PLD 0x02cc > + > +#define DSI2_IPI_COLOR_MAN_CFG 0x0300 > +#define IPI_DEPTH(x) FIELD_PREP(GENMASK(7, 4), x) > +#define IPI_DEPTH_5_6_5_BITS 0x02 > +#define IPI_DEPTH_6_BITS 0x03 > +#define IPI_DEPTH_8_BITS 0x05 > +#define IPI_DEPTH_10_BITS 0x06 > +#define IPI_FORMAT(x) FIELD_PREP(GENMASK(3, 0), x) > +#define IPI_FORMAT_RGB 0x0 > +#define IPI_FORMAT_DSC 0x0b > +#define DSI2_IPI_VID_HSA_MAN_CFG 0x0304 > +#define VID_HSA_TIME(x) FIELD_PREP(GENMASK(29, 0), x) > +#define DSI2_IPI_VID_HBP_MAN_CFG 0x030c > +#define VID_HBP_TIME(x) FIELD_PREP(GENMASK(29, 0), x) > +#define DSI2_IPI_VID_HACT_MAN_CFG 0x0314 > +#define VID_HACT_TIME(x) FIELD_PREP(GENMASK(29, 0), x) > +#define DSI2_IPI_VID_HLINE_MAN_CFG 0x031c > +#define VID_HLINE_TIME(x) FIELD_PREP(GENMASK(29, 0), x) > +#define DSI2_IPI_VID_VSA_MAN_CFG 0x0324 > +#define VID_VSA_LINES(x) FIELD_PREP(GENMASK(9, 0), x) > +#define DSI2_IPI_VID_VBP_MAN_CFG 0X032C > +#define VID_VBP_LINES(x) FIELD_PREP(GENMASK(9, 0), x) > +#define DSI2_IPI_VID_VACT_MAN_CFG 0X0334 > +#define VID_VACT_LINES(x) FIELD_PREP(GENMASK(13, 0), x) > +#define DSI2_IPI_VID_VFP_MAN_CFG 0X033C > +#define VID_VFP_LINES(x) FIELD_PREP(GENMASK(9, 0), x) > +#define DSI2_IPI_PIX_PKT_CFG 0x0344 > +#define MAX_PIX_PKT(x) FIELD_PREP(GENMASK(15, 0), x) > + > +#define DSI2_INT_ST_PHY 0x0400 > +#define DSI2_INT_MASK_PHY 0x0404 > +#define DSI2_INT_ST_TO 0x0410 > +#define DSI2_INT_MASK_TO 0x0414 > +#define DSI2_INT_ST_ACK 0x0420 > +#define DSI2_INT_MASK_ACK 0x0424 > +#define DSI2_INT_ST_IPI 0x0430 > +#define DSI2_INT_MASK_IPI 0x0434 > +#define DSI2_INT_ST_FIFO 0x0440 > +#define DSI2_INT_MASK_FIFO 0x0444 > +#define DSI2_INT_ST_PRI 0x0450 > +#define DSI2_INT_MASK_PRI 0x0454 > +#define DSI2_INT_ST_CRI 0x0460 > +#define DSI2_INT_MASK_CRI 0x0464 > +#define DSI2_INT_FORCE_CRI 0x0468 > +#define DSI2_MAX_REGISGER DSI2_INT_FORCE_CRI > + > +#define MODE_STATUS_TIMEOUT_US 10000 > +#define CMD_PKT_STATUS_TIMEOUT_US 20000 > + > +enum vid_mode_type { > + VID_MODE_TYPE_NON_BURST_SYNC_PULSES, > + VID_MODE_TYPE_NON_BURST_SYNC_EVENTS, > + VID_MODE_TYPE_BURST, > +}; > + > +enum mode_ctrl { > + IDLE_MODE, > + AUTOCALC_MODE, > + COMMAND_MODE, > + VIDEO_MODE, > + DATA_STREAM_MODE, > + VIDEO_TEST_MODE, > + DATA_STREAM_TEST_MODE, > +}; > + > +enum ppi_width { > + PPI_WIDTH_8_BITS, > + PPI_WIDTH_16_BITS, > + PPI_WIDTH_32_BITS, > +}; > + > +struct cmd_header { > + u8 cmd_type; > + u8 delay; > + u8 payload_length; > +}; > + > +struct dw_mipi_dsi2 { > + struct drm_bridge bridge; > + struct mipi_dsi_host dsi_host; > + struct drm_bridge *panel_bridge; > + struct device *dev; > + struct regmap *regmap; > + struct clk *pclk; > + struct clk *sys_clk; > + > + unsigned int lane_mbps; /* per lane */ > + u32 channel; > + u32 lanes; > + u32 format; > + unsigned long mode_flags; > + > + struct drm_display_mode mode; > + const struct dw_mipi_dsi2_plat_data *plat_data; > +}; > + > +static inline struct dw_mipi_dsi2 *host_to_dsi2(struct mipi_dsi_host *host) > +{ > + return container_of(host, struct dw_mipi_dsi2, dsi_host); > +} > + > +static inline struct dw_mipi_dsi2 *bridge_to_dsi2(struct drm_bridge *bridge) > +{ > + return container_of(bridge, struct dw_mipi_dsi2, bridge); > +} > + > +static int cri_fifos_wait_avail(struct dw_mipi_dsi2 *dsi2) > +{ > + u32 sts, mask; > + int ret; > + > + mask = CRI_BUSY | CRT_FIFOS_NOT_EMPTY; > + ret = regmap_read_poll_timeout(dsi2->regmap, DSI2_CORE_STATUS, sts, > + !(sts & mask), 0, CMD_PKT_STATUS_TIMEOUT_US); > + if (ret < 0) { > + dev_err(dsi2->dev, "command interface is busy\n"); > + return ret; > + } > + > + return 0; > +} > + > +static void dw_mipi_dsi2_set_vid_mode(struct dw_mipi_dsi2 *dsi2) > +{ > + u32 val = 0, mode; > + int ret; > + > + if (dsi2->mode_flags & MIPI_DSI_MODE_VIDEO_NO_HFP) > + val |= BLK_HFP_HS_EN; > + > + if (dsi2->mode_flags & MIPI_DSI_MODE_VIDEO_NO_HBP) > + val |= BLK_HBP_HS_EN; > + > + if (dsi2->mode_flags & MIPI_DSI_MODE_VIDEO_NO_HSA) > + val |= BLK_HSA_HS_EN; > + > + if (dsi2->mode_flags & MIPI_DSI_MODE_VIDEO_BURST) > + val |= VID_MODE_TYPE_BURST; > + else if (dsi2->mode_flags & MIPI_DSI_MODE_VIDEO_SYNC_PULSE) > + val |= VID_MODE_TYPE_NON_BURST_SYNC_PULSES; > + else > + val |= VID_MODE_TYPE_NON_BURST_SYNC_EVENTS; > + > + regmap_write(dsi2->regmap, DSI2_DSI_VID_TX_CFG, val); > + > + regmap_write(dsi2->regmap, DSI2_MODE_CTRL, VIDEO_MODE); > + ret = regmap_read_poll_timeout(dsi2->regmap, DSI2_MODE_STATUS, > + mode, mode & VIDEO_MODE, > + 1000, MODE_STATUS_TIMEOUT_US); > + if (ret < 0) > + dev_err(dsi2->dev, "failed to enter video mode\n"); > +} > + > +static void dw_mipi_dsi2_set_data_stream_mode(struct dw_mipi_dsi2 *dsi2) > +{ > + u32 mode; > + int ret; > + > + regmap_write(dsi2->regmap, DSI2_MODE_CTRL, DATA_STREAM_MODE); > + ret = regmap_read_poll_timeout(dsi2->regmap, DSI2_MODE_STATUS, > + mode, mode & DATA_STREAM_MODE, > + 1000, MODE_STATUS_TIMEOUT_US); > + if (ret < 0) > + dev_err(dsi2->dev, "failed to enter data stream mode\n"); > +} > + > +static void dw_mipi_dsi2_set_cmd_mode(struct dw_mipi_dsi2 *dsi2) > +{ > + u32 mode; > + int ret; > + > + regmap_write(dsi2->regmap, DSI2_MODE_CTRL, COMMAND_MODE); > + ret = regmap_read_poll_timeout(dsi2->regmap, DSI2_MODE_STATUS, > + mode, mode & COMMAND_MODE, > + 1000, MODE_STATUS_TIMEOUT_US); > + if (ret < 0) > + dev_err(dsi2->dev, "failed to enter data stream mode\n"); > +} > + > +static void dw_mipi_dsi2_host_softrst(struct dw_mipi_dsi2 *dsi2) > +{ > + regmap_write(dsi2->regmap, DSI2_SOFT_RESET, 0x0); > + usleep_range(50, 100); > + regmap_write(dsi2->regmap, DSI2_SOFT_RESET, > + SYS_RSTN | PHY_RSTN | IPI_RSTN); > +} > + > +static void dw_mipi_dsi2_phy_clk_mode_cfg(struct dw_mipi_dsi2 *dsi2) > +{ > + u32 sys_clk, esc_clk_div; > + u32 val = 0; > + > + /* > + * clk_type should be NON_CONTINUOUS_CLK before > + * initial deskew calibration be sent. > + */ > + val |= NON_CONTINUOUS_CLK; > + > + /* The maximum value of the escape clock frequency is 20MHz */ > + sys_clk = clk_get_rate(dsi2->sys_clk) / USEC_PER_SEC; > + esc_clk_div = DIV_ROUND_UP(sys_clk, 20 * 2); > + val |= PHY_LPTX_CLK_DIV(esc_clk_div); > + > + regmap_write(dsi2->regmap, DSI2_PHY_CLK_CFG, val); > +} > + > +static void dw_mipi_dsi2_phy_ratio_cfg(struct dw_mipi_dsi2 *dsi2) > +{ > + struct drm_display_mode *mode = &dsi2->mode; > + u64 sys_clk = clk_get_rate(dsi2->sys_clk); > + u64 pixel_clk, ipi_clk, phy_hsclk; > + u64 tmp; > + > + /* > + * in DPHY mode, the phy_hstx_clk is exactly 1/16 the Lane high-speed > + * data rate; In CPHY mode, the phy_hstx_clk is exactly 1/7 the trio > + * high speed symbol rate. > + */ > + phy_hsclk = DIV_ROUND_CLOSEST_ULL(dsi2->lane_mbps * USEC_PER_SEC, 16); > + > + /* IPI_RATIO_MAN_CFG = PHY_HSTX_CLK / IPI_CLK */ > + pixel_clk = mode->crtc_clock * MSEC_PER_SEC; > + ipi_clk = pixel_clk / 4; > + > + tmp = DIV_ROUND_CLOSEST_ULL(phy_hsclk << 16, ipi_clk); > + regmap_write(dsi2->regmap, DSI2_PHY_IPI_RATIO_MAN_CFG, > + PHY_IPI_RATIO(tmp)); > + > + /* > + * SYS_RATIO_MAN_CFG = MIPI_DCPHY_HSCLK_Freq / MIPI_DCPHY_HSCLK_Freq > + */ > + tmp = DIV_ROUND_CLOSEST_ULL(phy_hsclk << 16, sys_clk); > + regmap_write(dsi2->regmap, DSI2_PHY_SYS_RATIO_MAN_CFG, > + PHY_SYS_RATIO(tmp)); > +} > + > +static void dw_mipi_dsi2_lp2hs_or_hs2lp_cfg(struct dw_mipi_dsi2 *dsi2) > +{ > + const struct dw_mipi_dsi2_phy_ops *phy_ops = dsi2->plat_data->phy_ops; > + struct dw_mipi_dsi2_phy_timing timing; > + int ret; > + > + ret = phy_ops->get_timing(dsi2->plat_data->priv_data, > + dsi2->lane_mbps, &timing); > + if (ret) > + dev_err(dsi2->dev, "Retrieving phy timings failed\n"); > + > + regmap_write(dsi2->regmap, DSI2_PHY_LP2HS_MAN_CFG, PHY_LP2HS_TIME(timing.data_lp2hs)); > + regmap_write(dsi2->regmap, DSI2_PHY_HS2LP_MAN_CFG, PHY_HS2LP_TIME(timing.data_hs2lp)); > +} > + > +static void dw_mipi_dsi2_phy_init(struct dw_mipi_dsi2 *dsi2) > +{ > + const struct dw_mipi_dsi2_phy_ops *phy_ops = dsi2->plat_data->phy_ops; > + struct dw_mipi_dsi2_phy_iface iface; > + u32 val = 0; > + > + phy_ops->get_interface(dsi2->plat_data->priv_data, &iface); > + > + switch (iface.ppi_width) { > + case 8: > + val |= PPI_WIDTH(PPI_WIDTH_8_BITS); > + break; > + case 16: > + val |= PPI_WIDTH(PPI_WIDTH_16_BITS); > + break; > + case 32: > + val |= PPI_WIDTH(PPI_WIDTH_32_BITS); > + break; > + default: > + /* Caught in probe */ > + break; > + } > + > + val |= PHY_LANES(dsi2->lanes); > + val |= PHY_TYPE(DW_MIPI_DSI2_DPHY); > + regmap_write(dsi2->regmap, DSI2_PHY_MODE_CFG, val); > + > + dw_mipi_dsi2_phy_clk_mode_cfg(dsi2); > + dw_mipi_dsi2_phy_ratio_cfg(dsi2); > + dw_mipi_dsi2_lp2hs_or_hs2lp_cfg(dsi2); > + > + /* phy configuration 8 - 10 */ > +} > + > +static void dw_mipi_dsi2_tx_option_set(struct dw_mipi_dsi2 *dsi2) > +{ > + u32 val; > + > + val = BTA_EN | EOTP_TX_EN; > + > + if (dsi2->mode_flags & MIPI_DSI_MODE_NO_EOT_PACKET) > + val &= ~EOTP_TX_EN; > + > + regmap_write(dsi2->regmap, DSI2_DSI_GENERAL_CFG, val); > + regmap_write(dsi2->regmap, DSI2_DSI_VCID_CFG, TX_VCID(dsi2->channel)); > +} > + > +static void dw_mipi_dsi2_ipi_color_coding_cfg(struct dw_mipi_dsi2 *dsi2) > +{ > + u32 val, color_depth; > + > + switch (dsi2->format) { > + case MIPI_DSI_FMT_RGB666: > + case MIPI_DSI_FMT_RGB666_PACKED: > + color_depth = IPI_DEPTH_6_BITS; > + break; > + case MIPI_DSI_FMT_RGB565: > + color_depth = IPI_DEPTH_5_6_5_BITS; > + break; > + case MIPI_DSI_FMT_RGB888: > + default: > + color_depth = IPI_DEPTH_8_BITS; > + break; > + } > + > + val = IPI_DEPTH(color_depth) | > + IPI_FORMAT(IPI_FORMAT_RGB); > + regmap_write(dsi2->regmap, DSI2_IPI_COLOR_MAN_CFG, val); > +} > + > +static void dw_mipi_dsi2_vertical_timing_config(struct dw_mipi_dsi2 *dsi2, > + const struct drm_display_mode *mode) > +{ > + u32 vactive, vsa, vfp, vbp; > + > + vactive = mode->vdisplay; > + vsa = mode->vsync_end - mode->vsync_start; > + vfp = mode->vsync_start - mode->vdisplay; > + vbp = mode->vtotal - mode->vsync_end; > + > + regmap_write(dsi2->regmap, DSI2_IPI_VID_VSA_MAN_CFG, VID_VSA_LINES(vsa)); > + regmap_write(dsi2->regmap, DSI2_IPI_VID_VBP_MAN_CFG, VID_VBP_LINES(vbp)); > + regmap_write(dsi2->regmap, DSI2_IPI_VID_VACT_MAN_CFG, VID_VACT_LINES(vactive)); > + regmap_write(dsi2->regmap, DSI2_IPI_VID_VFP_MAN_CFG, VID_VFP_LINES(vfp)); > +} > + > +static void dw_mipi_dsi2_ipi_set(struct dw_mipi_dsi2 *dsi2) > +{ > + struct drm_display_mode *mode = &dsi2->mode; > + u32 hline, hsa, hbp, hact; > + u64 hline_time, hsa_time, hbp_time, hact_time, tmp; > + u64 pixel_clk, phy_hs_clk; > + u16 val; > + > + val = mode->hdisplay; > + > + regmap_write(dsi2->regmap, DSI2_IPI_PIX_PKT_CFG, MAX_PIX_PKT(val)); > + > + dw_mipi_dsi2_ipi_color_coding_cfg(dsi2); > + > + /* > + * if the controller is intended to operate in data stream mode, > + * no more steps are required. > + */ > + if (!(dsi2->mode_flags & MIPI_DSI_MODE_VIDEO)) > + return; > + > + hact = mode->hdisplay; > + hsa = mode->hsync_end - mode->hsync_start; > + hbp = mode->htotal - mode->hsync_end; > + hline = mode->htotal; > + > + pixel_clk = mode->crtc_clock * MSEC_PER_SEC; > + > + phy_hs_clk = DIV_ROUND_CLOSEST_ULL(dsi2->lane_mbps * USEC_PER_SEC, 16); > + > + tmp = hsa * phy_hs_clk; > + hsa_time = DIV_ROUND_CLOSEST_ULL(tmp << 16, pixel_clk); > + regmap_write(dsi2->regmap, DSI2_IPI_VID_HSA_MAN_CFG, VID_HSA_TIME(hsa_time)); > + > + tmp = hbp * phy_hs_clk; > + hbp_time = DIV_ROUND_CLOSEST_ULL(tmp << 16, pixel_clk); > + regmap_write(dsi2->regmap, DSI2_IPI_VID_HBP_MAN_CFG, VID_HBP_TIME(hbp_time)); > + > + tmp = hact * phy_hs_clk; > + hact_time = DIV_ROUND_CLOSEST_ULL(tmp << 16, pixel_clk); > + regmap_write(dsi2->regmap, DSI2_IPI_VID_HACT_MAN_CFG, VID_HACT_TIME(hact_time)); > + > + tmp = hline * phy_hs_clk; > + hline_time = DIV_ROUND_CLOSEST_ULL(tmp << 16, pixel_clk); > + regmap_write(dsi2->regmap, DSI2_IPI_VID_HLINE_MAN_CFG, VID_HLINE_TIME(hline_time)); > + > + dw_mipi_dsi2_vertical_timing_config(dsi2, mode); > +} > + > +static void > +dw_mipi_dsi2_work_mode(struct dw_mipi_dsi2 *dsi2, u32 mode) > +{ > + /* > + * select controller work in Manual mode > + * Manual: MANUAL_MODE_EN > + * Automatic: 0 > + */ > + regmap_write(dsi2->regmap, MANUAL_MODE_CFG, mode); > +} > + > +static int dw_mipi_dsi2_host_attach(struct mipi_dsi_host *host, > + struct mipi_dsi_device *device) > +{ > + struct dw_mipi_dsi2 *dsi2 = host_to_dsi2(host); > + const struct dw_mipi_dsi2_plat_data *pdata = dsi2->plat_data; > + struct drm_bridge *bridge; > + int ret; > + > + if (device->lanes > dsi2->plat_data->max_data_lanes) { > + dev_err(dsi2->dev, "the number of data lanes(%u) is too many\n", > + device->lanes); > + return -EINVAL; > + } > + > + dsi2->lanes = device->lanes; > + dsi2->channel = device->channel; > + dsi2->format = device->format; > + dsi2->mode_flags = device->mode_flags; > + > + bridge = devm_drm_of_get_bridge(dsi2->dev, dsi2->dev->of_node, 1, 0); > + if (IS_ERR(bridge)) > + return PTR_ERR(bridge); > + > + bridge->pre_enable_prev_first = true; > + dsi2->panel_bridge = bridge; > + > + drm_bridge_add(&dsi2->bridge); > + > + if (pdata->host_ops && pdata->host_ops->attach) { > + ret = pdata->host_ops->attach(pdata->priv_data, device); > + if (ret < 0) > + return ret; > + } > + > + return 0; > +} > + > +static int dw_mipi_dsi2_host_detach(struct mipi_dsi_host *host, > + struct mipi_dsi_device *device) > +{ > + struct dw_mipi_dsi2 *dsi2 = host_to_dsi2(host); > + const struct dw_mipi_dsi2_plat_data *pdata = dsi2->plat_data; > + int ret; > + > + if (pdata->host_ops && pdata->host_ops->detach) { > + ret = pdata->host_ops->detach(pdata->priv_data, device); > + if (ret < 0) > + return ret; > + } > + > + drm_bridge_remove(&dsi2->bridge); > + > + drm_of_panel_bridge_remove(host->dev->of_node, 1, 0); > + > + return 0; > +} > + > +static int dw_mipi_dsi2_gen_pkt_hdr_write(struct dw_mipi_dsi2 *dsi2, > + u32 hdr_val, bool lpm) > +{ > + int ret; > + > + regmap_write(dsi2->regmap, DSI2_CRI_TX_HDR, hdr_val | CMD_TX_MODE(lpm)); > + > + ret = cri_fifos_wait_avail(dsi2); > + if (ret) { > + dev_err(dsi2->dev, "failed to write command header\n"); > + return ret; > + } > + > + return 0; > +} > + > +static int dw_mipi_dsi2_write(struct dw_mipi_dsi2 *dsi2, > + const struct mipi_dsi_packet *packet, bool lpm) > +{ > + const u8 *tx_buf = packet->payload; > + int len = packet->payload_length, pld_data_bytes = sizeof(u32); > + __le32 word; > + > + /* Send payload */ > + while (len) { > + if (len < pld_data_bytes) { > + word = 0; > + memcpy(&word, tx_buf, len); > + regmap_write(dsi2->regmap, DSI2_CRI_TX_PLD, le32_to_cpu(word)); > + len = 0; > + } else { > + memcpy(&word, tx_buf, pld_data_bytes); > + regmap_write(dsi2->regmap, DSI2_CRI_TX_PLD, le32_to_cpu(word)); > + tx_buf += pld_data_bytes; > + len -= pld_data_bytes; > + } > + } > + > + word = 0; > + memcpy(&word, packet->header, sizeof(packet->header)); > + return dw_mipi_dsi2_gen_pkt_hdr_write(dsi2, le32_to_cpu(word), lpm); > +} > + > +static int dw_mipi_dsi2_read(struct dw_mipi_dsi2 *dsi2, > + const struct mipi_dsi_msg *msg) > +{ > + u8 *payload = msg->rx_buf; > + int i, j, ret, len = msg->rx_len; > + u8 data_type; > + u16 wc; > + u32 val; > + > + ret = regmap_read_poll_timeout(dsi2->regmap, DSI2_CORE_STATUS, > + val, val & CRI_RD_DATA_AVAIL, > + 100, CMD_PKT_STATUS_TIMEOUT_US); > + if (ret) { > + dev_err(dsi2->dev, "CRI has no available read data\n"); > + return ret; > + } > + > + regmap_read(dsi2->regmap, DSI2_CRI_RX_HDR, &val); > + data_type = val & 0x3f; > + > + if (mipi_dsi_packet_format_is_short(data_type)) { > + for (i = 0; i < len && i < 2; i++) > + payload[i] = (val >> (8 * (i + 1))) & 0xff; > + > + return 0; > + } > + > + wc = (val >> 8) & 0xffff; > + /* Receive payload */ > + for (i = 0; i < len && i < wc; i += 4) { > + regmap_read(dsi2->regmap, DSI2_CRI_RX_PLD, &val); > + for (j = 0; j < 4 && j + i < len && j + i < wc; j++) > + payload[i + j] = val >> (8 * j); > + } > + > + return 0; > +} > + > +static ssize_t dw_mipi_dsi2_host_transfer(struct mipi_dsi_host *host, > + const struct mipi_dsi_msg *msg) > +{ > + struct dw_mipi_dsi2 *dsi2 = host_to_dsi2(host); > + bool lpm = msg->flags & MIPI_DSI_MSG_USE_LPM; > + struct mipi_dsi_packet packet; > + int ret, nb_bytes; > + > + regmap_update_bits(dsi2->regmap, DSI2_DSI_VID_TX_CFG, > + LPDT_DISPLAY_CMD_EN, > + lpm ? LPDT_DISPLAY_CMD_EN : 0); > + > + /* create a packet to the DSI protocol */ > + ret = mipi_dsi_create_packet(&packet, msg); > + if (ret) { > + dev_err(dsi2->dev, "failed to create packet: %d\n", ret); > + return ret; > + } > + > + ret = cri_fifos_wait_avail(dsi2); > + if (ret) > + return ret; > + > + ret = dw_mipi_dsi2_write(dsi2, &packet, lpm); > + if (ret) > + return ret; > + > + if (msg->rx_buf && msg->rx_len) { > + ret = dw_mipi_dsi2_read(dsi2, msg); > + if (ret < 0) > + return ret; > + nb_bytes = msg->rx_len; > + } else { > + nb_bytes = packet.size; > + } > + > + return nb_bytes; > +} > + > +static const struct mipi_dsi_host_ops dw_mipi_dsi2_host_ops = { > + .attach = dw_mipi_dsi2_host_attach, > + .detach = dw_mipi_dsi2_host_detach, > + .transfer = dw_mipi_dsi2_host_transfer, > +}; > + > +static u32 * > +dw_mipi_dsi2_bridge_atomic_get_input_bus_fmts(struct drm_bridge *bridge, > + struct drm_bridge_state *bridge_state, > + struct drm_crtc_state *crtc_state, > + struct drm_connector_state *conn_state, > + u32 output_fmt, > + unsigned int *num_input_fmts) > +{ > + struct dw_mipi_dsi2 *dsi2 = bridge_to_dsi2(bridge); > + const struct dw_mipi_dsi2_plat_data *pdata = dsi2->plat_data; > + u32 *input_fmts; > + > + if (pdata->get_input_bus_fmts) > + return pdata->get_input_bus_fmts(pdata->priv_data, > + bridge, bridge_state, > + crtc_state, conn_state, > + output_fmt, num_input_fmts); > + > + /* Fall back to MEDIA_BUS_FMT_FIXED as the only input format. */ > + input_fmts = kmalloc(sizeof(*input_fmts), GFP_KERNEL); > + if (!input_fmts) > + return NULL; > + input_fmts[0] = MEDIA_BUS_FMT_FIXED; > + *num_input_fmts = 1; > + > + return input_fmts; > +} > + > +static int dw_mipi_dsi2_bridge_atomic_check(struct drm_bridge *bridge, > + struct drm_bridge_state *bridge_state, > + struct drm_crtc_state *crtc_state, > + struct drm_connector_state *conn_state) > +{ > + struct dw_mipi_dsi2 *dsi2 = bridge_to_dsi2(bridge); > + const struct dw_mipi_dsi2_plat_data *pdata = dsi2->plat_data; > + bool ret; > + > + bridge_state->input_bus_cfg.flags = > + DRM_BUS_FLAG_DE_HIGH | DRM_BUS_FLAG_PIXDATA_SAMPLE_NEGEDGE; > + > + if (pdata->mode_fixup) { > + ret = pdata->mode_fixup(pdata->priv_data, &crtc_state->mode, > + &crtc_state->adjusted_mode); > + if (!ret) { > + DRM_DEBUG_DRIVER("failed to fixup mode " DRM_MODE_FMT "\n", > + DRM_MODE_ARG(&crtc_state->mode)); > + return -EINVAL; > + } > + } > + > + return 0; > +} > + > +static void dw_mipi_dsi2_bridge_post_atomic_disable(struct drm_bridge *bridge, > + struct drm_bridge_state *old_bridge_state) > +{ > + struct dw_mipi_dsi2 *dsi2 = bridge_to_dsi2(bridge); > + const struct dw_mipi_dsi2_phy_ops *phy_ops = dsi2->plat_data->phy_ops; > + > + regmap_write(dsi2->regmap, DSI2_IPI_PIX_PKT_CFG, 0); > + > + /* > + * Switch to command mode before panel-bridge post_disable & > + * panel unprepare. > + * Note: panel-bridge disable & panel disable has been called > + * before by the drm framework. > + */ > + dw_mipi_dsi2_set_cmd_mode(dsi2); > + > + regmap_write(dsi2->regmap, DSI2_PWR_UP, RESET); > + > + if (phy_ops->power_off) > + phy_ops->power_off(dsi2->plat_data->priv_data); > + > + clk_disable_unprepare(dsi2->sys_clk); > + clk_disable_unprepare(dsi2->pclk); > + pm_runtime_put(dsi2->dev); > +} > + > +static unsigned int dw_mipi_dsi2_get_lanes(struct dw_mipi_dsi2 *dsi2) > +{ > + /* single-dsi, so no other instance to consider */ > + return dsi2->lanes; > +} > + > +static void dw_mipi_dsi2_mode_set(struct dw_mipi_dsi2 *dsi2, > + const struct drm_display_mode *adjusted_mode) > +{ > + const struct dw_mipi_dsi2_phy_ops *phy_ops = dsi2->plat_data->phy_ops; > + void *priv_data = dsi2->plat_data->priv_data; > + u32 lanes = dw_mipi_dsi2_get_lanes(dsi2); > + int ret; > + > + clk_prepare_enable(dsi2->pclk); > + clk_prepare_enable(dsi2->sys_clk); > + > + ret = phy_ops->get_lane_mbps(priv_data, adjusted_mode, dsi2->mode_flags, > + lanes, dsi2->format, &dsi2->lane_mbps); > + if (ret) > + DRM_DEBUG_DRIVER("Phy get_lane_mbps() failed\n"); > + > + pm_runtime_get_sync(dsi2->dev); > + > + dw_mipi_dsi2_host_softrst(dsi2); > + regmap_write(dsi2->regmap, DSI2_PWR_UP, RESET); > + > + dw_mipi_dsi2_work_mode(dsi2, MANUAL_MODE_EN); > + dw_mipi_dsi2_phy_init(dsi2); > + > + if (phy_ops->power_on) > + phy_ops->power_on(dsi2->plat_data->priv_data); > + > + dw_mipi_dsi2_tx_option_set(dsi2); > + > + /* > + * initial deskew calibration is send after phy_power_on, > + * then we can configure clk_type. > + */ > + > + regmap_update_bits(dsi2->regmap, DSI2_PHY_CLK_CFG, CLK_TYPE_MASK, > + dsi2->mode_flags & MIPI_DSI_CLOCK_NON_CONTINUOUS ? NON_CONTINUOUS_CLK : > + CONTINUOUS_CLK); > + > + regmap_write(dsi2->regmap, DSI2_PWR_UP, POWER_UP); > + dw_mipi_dsi2_set_cmd_mode(dsi2); > + > + dw_mipi_dsi2_ipi_set(dsi2); > +} > + > +static void dw_mipi_dsi2_bridge_atomic_pre_enable(struct drm_bridge *bridge, > + struct drm_bridge_state *old_bridge_state) > +{ > + struct dw_mipi_dsi2 *dsi2 = bridge_to_dsi2(bridge); > + > + /* Power up the dsi ctl into a command mode */ > + dw_mipi_dsi2_mode_set(dsi2, &dsi2->mode); > +} > + > +static void dw_mipi_dsi2_bridge_mode_set(struct drm_bridge *bridge, > + const struct drm_display_mode *mode, > + const struct drm_display_mode *adjusted_mode) > +{ > + struct dw_mipi_dsi2 *dsi2 = bridge_to_dsi2(bridge); > + > + /* Store the display mode for later use in pre_enable callback */ > + drm_mode_copy(&dsi2->mode, adjusted_mode); > +} > + > +static void dw_mipi_dsi2_bridge_atomic_enable(struct drm_bridge *bridge, > + struct drm_bridge_state *old_bridge_state) > +{ > + struct dw_mipi_dsi2 *dsi2 = bridge_to_dsi2(bridge); > + > + /* Switch to video mode for panel-bridge enable & panel enable */ > + if (dsi2->mode_flags & MIPI_DSI_MODE_VIDEO) > + dw_mipi_dsi2_set_vid_mode(dsi2); > + else > + dw_mipi_dsi2_set_data_stream_mode(dsi2); > +} > + > +static enum drm_mode_status > +dw_mipi_dsi2_bridge_mode_valid(struct drm_bridge *bridge, > + const struct drm_display_info *info, > + const struct drm_display_mode *mode) > +{ > + struct dw_mipi_dsi2 *dsi2 = bridge_to_dsi2(bridge); > + const struct dw_mipi_dsi2_plat_data *pdata = dsi2->plat_data; > + enum drm_mode_status mode_status = MODE_OK; > + > + if (pdata->mode_valid) > + mode_status = pdata->mode_valid(pdata->priv_data, mode, > + dsi2->mode_flags, > + dw_mipi_dsi2_get_lanes(dsi2), > + dsi2->format); > + > + return mode_status; > +} > + > +static int dw_mipi_dsi2_bridge_attach(struct drm_bridge *bridge, > + enum drm_bridge_attach_flags flags) > +{ > + struct dw_mipi_dsi2 *dsi2 = bridge_to_dsi2(bridge); > + > + /* Set the encoder type as caller does not know it */ > + bridge->encoder->encoder_type = DRM_MODE_ENCODER_DSI; > + > + /* Attach the panel-bridge to the dsi bridge */ > + return drm_bridge_attach(bridge->encoder, dsi2->panel_bridge, bridge, > + flags); > +} > + > +static const struct drm_bridge_funcs dw_mipi_dsi2_bridge_funcs = { > + .atomic_duplicate_state = drm_atomic_helper_bridge_duplicate_state, > + .atomic_destroy_state = drm_atomic_helper_bridge_destroy_state, > + .atomic_get_input_bus_fmts = dw_mipi_dsi2_bridge_atomic_get_input_bus_fmts, > + .atomic_check = dw_mipi_dsi2_bridge_atomic_check, > + .atomic_reset = drm_atomic_helper_bridge_reset, > + .atomic_pre_enable = dw_mipi_dsi2_bridge_atomic_pre_enable, > + .atomic_enable = dw_mipi_dsi2_bridge_atomic_enable, > + .atomic_post_disable = dw_mipi_dsi2_bridge_post_atomic_disable, > + .mode_set = dw_mipi_dsi2_bridge_mode_set, > + .mode_valid = dw_mipi_dsi2_bridge_mode_valid, > + .attach = dw_mipi_dsi2_bridge_attach, > +}; > + > +static const struct regmap_config dw_mipi_dsi2_regmap_config = { > + .name = "dsi2-host", > + .reg_bits = 32, > + .val_bits = 32, > + .reg_stride = 4, > + .fast_io = true, > +}; > + > +static struct dw_mipi_dsi2 * > +__dw_mipi_dsi2_probe(struct platform_device *pdev, > + const struct dw_mipi_dsi2_plat_data *plat_data) > +{ > + struct device *dev = &pdev->dev; > + struct reset_control *apb_rst; > + struct dw_mipi_dsi2 *dsi2; > + int ret; > + > + dsi2 = devm_kzalloc(dev, sizeof(*dsi2), GFP_KERNEL); > + if (!dsi2) > + return ERR_PTR(-ENOMEM); > + > + dsi2->dev = dev; > + dsi2->plat_data = plat_data; > + > + if (!plat_data->phy_ops->init || !plat_data->phy_ops->get_lane_mbps || > + !plat_data->phy_ops->get_timing) > + return dev_err_ptr_probe(dev, -ENODEV, "Phy not properly configured\n"); > + > + if (!plat_data->regmap) { > + void __iomem *base = devm_platform_ioremap_resource(pdev, 0); > + > + if (IS_ERR(base)) > + return dev_err_cast_probe(dev, base, "failed to registers\n"); > + > + dsi2->regmap = devm_regmap_init_mmio(dev, base, > + &dw_mipi_dsi2_regmap_config); > + if (IS_ERR(dsi2->regmap)) > + return dev_err_cast_probe(dev, dsi2->regmap, "failed to init regmap\n"); > + } else { > + dsi2->regmap = plat_data->regmap; > + } > + > + dsi2->pclk = devm_clk_get(dev, "pclk"); > + if (IS_ERR(dsi2->pclk)) > + return dev_err_cast_probe(dev, dsi2->pclk, "Unable to get pclk\n"); > + > + dsi2->sys_clk = devm_clk_get(dev, "sys"); > + if (IS_ERR(dsi2->sys_clk)) > + return dev_err_cast_probe(dev, dsi2->sys_clk, "Unable to get sys_clk\n"); > + > + /* > + * Note that the reset was not defined in the initial device tree, so > + * we have to be prepared for it not being found. > + */ > + apb_rst = devm_reset_control_get_optional_exclusive(dev, "apb"); > + if (IS_ERR(apb_rst)) > + return dev_err_cast_probe(dev, apb_rst, "Unable to get reset control\n"); > + > + if (apb_rst) { > + ret = clk_prepare_enable(dsi2->pclk); > + if (ret) { > + dev_err(dev, "%s: Failed to enable pclk\n", __func__); > + return ERR_PTR(ret); > + } > + > + reset_control_assert(apb_rst); > + usleep_range(10, 20); > + reset_control_deassert(apb_rst); > + > + clk_disable_unprepare(dsi2->pclk); > + } > + > + devm_pm_runtime_enable(dev); > + > + dsi2->dsi_host.ops = &dw_mipi_dsi2_host_ops; > + dsi2->dsi_host.dev = dev; > + ret = mipi_dsi_host_register(&dsi2->dsi_host); > + if (ret) { > + dev_err(dev, "Failed to register MIPI host: %d\n", ret); > + pm_runtime_disable(dev); > + return ERR_PTR(ret); > + } > + > + dsi2->bridge.driver_private = dsi2; > + dsi2->bridge.funcs = &dw_mipi_dsi2_bridge_funcs; > + dsi2->bridge.of_node = pdev->dev.of_node; > + > + return dsi2; > +} > + > +static void __dw_mipi_dsi2_remove(struct dw_mipi_dsi2 *dsi2) > +{ > + mipi_dsi_host_unregister(&dsi2->dsi_host); > +} > + > +/* > + * Probe/remove API, used to create the bridge instance. > + */ > +struct dw_mipi_dsi2 * > +dw_mipi_dsi2_probe(struct platform_device *pdev, > + const struct dw_mipi_dsi2_plat_data *plat_data) > +{ > + return __dw_mipi_dsi2_probe(pdev, plat_data); > +} > +EXPORT_SYMBOL_GPL(dw_mipi_dsi2_probe); > + > +void dw_mipi_dsi2_remove(struct dw_mipi_dsi2 *dsi2) > +{ > + __dw_mipi_dsi2_remove(dsi2); > +} > +EXPORT_SYMBOL_GPL(dw_mipi_dsi2_remove); > + > +/* > + * Bind/unbind API, used from platforms based on the component framework > + * to attach the bridge to an encoder. > + */ > +int dw_mipi_dsi2_bind(struct dw_mipi_dsi2 *dsi2, struct drm_encoder *encoder) > +{ > + return drm_bridge_attach(encoder, &dsi2->bridge, NULL, 0); > +} > +EXPORT_SYMBOL_GPL(dw_mipi_dsi2_bind); > + > +void dw_mipi_dsi2_unbind(struct dw_mipi_dsi2 *dsi2) > +{ > +} > +EXPORT_SYMBOL_GPL(dw_mipi_dsi2_unbind); > + > +MODULE_AUTHOR("Guochun Huang <hero.huang@rock-chips.com>"); > +MODULE_AUTHOR("Heiko Stuebner <heiko.stuebner@cherry.de>"); > +MODULE_DESCRIPTION("DW MIPI DSI2 host controller driver"); > +MODULE_LICENSE("GPL"); > +MODULE_ALIAS("platform:dw-mipi-dsi2"); > diff --git a/include/drm/bridge/dw_mipi_dsi2.h b/include/drm/bridge/dw_mipi_dsi2.h > new file mode 100644 > index 000000000000..c18c49379247 > --- /dev/null > +++ b/include/drm/bridge/dw_mipi_dsi2.h > @@ -0,0 +1,95 @@ > +/* SPDX-License-Identifier: GPL-2.0-only */ > +/* > + * Copyright (c) 2024, Fuzhou Rockchip Electronics Co., Ltd > + * > + * Authors: Guochun Huang <hero.huang@rock-chips.com> > + * Heiko Stuebner <heiko.stuebner@cherry.de> > + */ > + > +#ifndef __DW_MIPI_DSI2__ > +#define __DW_MIPI_DSI2__ > + > +#include <linux/regmap.h> > +#include <linux/types.h> > + > +#include <drm/drm_atomic.h> > +#include <drm/drm_bridge.h> > +#include <drm/drm_connector.h> > +#include <drm/drm_crtc.h> > +#include <drm/drm_modes.h> > + > +struct drm_display_mode; > +struct drm_encoder; > +struct dw_mipi_dsi2; > +struct mipi_dsi_device; > +struct platform_device; > + > +enum dw_mipi_dsi2_phy_type { > + DW_MIPI_DSI2_DPHY, > + DW_MIPI_DSI2_CPHY, > +}; > + > +struct dw_mipi_dsi2_phy_iface { > + int ppi_width; > + enum dw_mipi_dsi2_phy_type phy_type; > +}; > + > +struct dw_mipi_dsi2_phy_timing { > + u32 data_hs2lp; > + u32 data_lp2hs; > +}; > + > +struct dw_mipi_dsi2_phy_ops { > + int (*init)(void *priv_data); > + void (*power_on)(void *priv_data); > + void (*power_off)(void *priv_data); > + void (*get_interface)(void *priv_data, struct dw_mipi_dsi2_phy_iface *iface); > + int (*get_lane_mbps)(void *priv_data, > + const struct drm_display_mode *mode, > + unsigned long mode_flags, u32 lanes, u32 format, > + unsigned int *lane_mbps); > + int (*get_timing)(void *priv_data, unsigned int lane_mbps, > + struct dw_mipi_dsi2_phy_timing *timing); > + int (*get_esc_clk_rate)(void *priv_data, unsigned int *esc_clk_rate); > +}; > + > +struct dw_mipi_dsi2_host_ops { > + int (*attach)(void *priv_data, > + struct mipi_dsi_device *dsi); > + int (*detach)(void *priv_data, > + struct mipi_dsi_device *dsi); > +}; > + > +struct dw_mipi_dsi2_plat_data { > + struct regmap *regmap; > + unsigned int max_data_lanes; > + > + enum drm_mode_status (*mode_valid)(void *priv_data, > + const struct drm_display_mode *mode, > + unsigned long mode_flags, > + u32 lanes, u32 format); > + > + bool (*mode_fixup)(void *priv_data, const struct drm_display_mode *mode, > + struct drm_display_mode *adjusted_mode); > + > + u32 *(*get_input_bus_fmts)(void *priv_data, > + struct drm_bridge *bridge, > + struct drm_bridge_state *bridge_state, > + struct drm_crtc_state *crtc_state, > + struct drm_connector_state *conn_state, > + u32 output_fmt, > + unsigned int *num_input_fmts); > + > + const struct dw_mipi_dsi2_phy_ops *phy_ops; > + const struct dw_mipi_dsi2_host_ops *host_ops; > + > + void *priv_data; > +}; > + > +struct dw_mipi_dsi2 *dw_mipi_dsi2_probe(struct platform_device *pdev, > + const struct dw_mipi_dsi2_plat_data *plat_data); > +void dw_mipi_dsi2_remove(struct dw_mipi_dsi2 *dsi2); > +int dw_mipi_dsi2_bind(struct dw_mipi_dsi2 *dsi2, struct drm_encoder *encoder); > +void dw_mipi_dsi2_unbind(struct dw_mipi_dsi2 *dsi2); > + > +#endif /* __DW_MIPI_DSI2__ */ ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v4 1/3] drm/bridge/synopsys: Add MIPI DSI2 host controller bridge 2024-12-09 23:10 ` [PATCH v4 1/3] drm/bridge/synopsys: Add MIPI DSI2 host controller bridge Heiko Stuebner 2024-12-10 8:50 ` Andy Yan @ 2024-12-30 2:20 ` Chris Hofstaedtler 2025-01-14 17:30 ` Heiko Stübner 2025-02-02 20:34 ` Pavel Golikov 2 siblings, 1 reply; 10+ messages in thread From: Chris Hofstaedtler @ 2024-12-30 2:20 UTC (permalink / raw) To: Heiko Stuebner Cc: andy.yan, maarten.lankhorst, mripard, tzimmermann, robh, krzk+dt, conor+dt, andrzej.hajda, neil.armstrong, rfoss, Laurent.pinchart, jonas, jernej.skrabec, dri-devel, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel, quentin.schulz, Heiko Stuebner, Daniel Semkowicz, Dmitry Yashin Hi, On Tue, Dec 10, 2024 at 12:10:19AM +0100, Heiko Stuebner wrote: > From: Heiko Stuebner <heiko.stuebner@cherry.de> > > Add a Synopsys Designware MIPI DSI host DRM bridge driver for their > DSI2 host controller, based on the Rockchip version from the driver > rockchip/dw-mipi-dsi2.c in their vendor-kernel with phy & bridge APIs. > > While the driver is heavily modelled after the previous IP, the register > set of this DSI2 controller is completely different and there are also > additional properties like the variable-width phy interface. > > Tested-by: Daniel Semkowicz <dse@thaumatec.com> > Tested-by: Dmitry Yashin <dmt.yashin@gmail.com> > Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org> > Signed-off-by: Heiko Stuebner <heiko.stuebner@cherry.de> [..] > +static void dw_mipi_dsi2_set_vid_mode(struct dw_mipi_dsi2 *dsi2) > +{ > + u32 val = 0, mode; > + int ret; > + > + if (dsi2->mode_flags & MIPI_DSI_MODE_VIDEO_NO_HFP) > + val |= BLK_HFP_HS_EN; > + > + if (dsi2->mode_flags & MIPI_DSI_MODE_VIDEO_NO_HBP) > + val |= BLK_HBP_HS_EN; > + > + if (dsi2->mode_flags & MIPI_DSI_MODE_VIDEO_NO_HSA) > + val |= BLK_HSA_HS_EN; For all three of these: is setting an ENable bit the right thing to turn features *off*? > + if (dsi2->mode_flags & MIPI_DSI_MODE_VIDEO_BURST) > + val |= VID_MODE_TYPE_BURST; > + else if (dsi2->mode_flags & MIPI_DSI_MODE_VIDEO_SYNC_PULSE) > + val |= VID_MODE_TYPE_NON_BURST_SYNC_PULSES; > + else > + val |= VID_MODE_TYPE_NON_BURST_SYNC_EVENTS; > + > + regmap_write(dsi2->regmap, DSI2_DSI_VID_TX_CFG, val); > + > + regmap_write(dsi2->regmap, DSI2_MODE_CTRL, VIDEO_MODE); > + ret = regmap_read_poll_timeout(dsi2->regmap, DSI2_MODE_STATUS, > + mode, mode & VIDEO_MODE, > + 1000, MODE_STATUS_TIMEOUT_US); > + if (ret < 0) > + dev_err(dsi2->dev, "failed to enter video mode\n"); > +} ... Chris ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v4 1/3] drm/bridge/synopsys: Add MIPI DSI2 host controller bridge 2024-12-30 2:20 ` Chris Hofstaedtler @ 2025-01-14 17:30 ` Heiko Stübner 0 siblings, 0 replies; 10+ messages in thread From: Heiko Stübner @ 2025-01-14 17:30 UTC (permalink / raw) To: Chris Hofstaedtler, andy.yan, Guochun Huang Cc: maarten.lankhorst, mripard, tzimmermann, robh, krzk+dt, conor+dt, andrzej.hajda, neil.armstrong, rfoss, Laurent.pinchart, jonas, jernej.skrabec, dri-devel, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel, quentin.schulz, Heiko Stuebner, Daniel Semkowicz, Dmitry Yashin Hi Chris, Am Montag, 30. Dezember 2024, 03:20:55 CET schrieb Chris Hofstaedtler: > On Tue, Dec 10, 2024 at 12:10:19AM +0100, Heiko Stuebner wrote: > > From: Heiko Stuebner <heiko.stuebner@cherry.de> > > > > Add a Synopsys Designware MIPI DSI host DRM bridge driver for their > > DSI2 host controller, based on the Rockchip version from the driver > > rockchip/dw-mipi-dsi2.c in their vendor-kernel with phy & bridge APIs. > > > > While the driver is heavily modelled after the previous IP, the register > > set of this DSI2 controller is completely different and there are also > > additional properties like the variable-width phy interface. > > > > Tested-by: Daniel Semkowicz <dse@thaumatec.com> > > Tested-by: Dmitry Yashin <dmt.yashin@gmail.com> > > Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org> > > Signed-off-by: Heiko Stuebner <heiko.stuebner@cherry.de> > [..] > > +static void dw_mipi_dsi2_set_vid_mode(struct dw_mipi_dsi2 *dsi2) > > +{ > > + u32 val = 0, mode; > > + int ret; > > + > > + if (dsi2->mode_flags & MIPI_DSI_MODE_VIDEO_NO_HFP) > > + val |= BLK_HFP_HS_EN; > > + > > + if (dsi2->mode_flags & MIPI_DSI_MODE_VIDEO_NO_HBP) > > + val |= BLK_HBP_HS_EN; > > + > > + if (dsi2->mode_flags & MIPI_DSI_MODE_VIDEO_NO_HSA) > > + val |= BLK_HSA_HS_EN; > > For all three of these: is setting an ENable bit the right thing to > turn features *off*? first of all, thanks a lot for noticing this discrepancy :-) . Looking at the documentation, all 3 of those hw-bits are described as "Enables filling the H.. period with blanking packets. ..." where the MIPI_DSI_VIDEO_MODE_NO_* flags are described as "disable hfront-porch/... area" So yes, I _think_ "disable front-porch" would _should_ result in "don't fill the period with blanking packets", but am not fully sure. I've run the two boards I have with inverting the checks as sounds sensible right now, aka doing: if (!(dsi2->mode_flags & MIPI_DSI_MODE_VIDEO_NO_HFP)) etc and both displays I have ran just fine. As the driver was originally part of a vendor-kernel based on 5.10, which I think was before the _NO addition from [0] it could be caused by a misread of the flags that were named differently back then. So yes, switching things around does sound like the right thing to do. Heiko [0] https://lore.kernel.org/all/20210629074703.v2.1.I629b2366a6591410359c7fcf6d385b474b705ca2@changeid/ ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v4 1/3] drm/bridge/synopsys: Add MIPI DSI2 host controller bridge 2024-12-09 23:10 ` [PATCH v4 1/3] drm/bridge/synopsys: Add MIPI DSI2 host controller bridge Heiko Stuebner 2024-12-10 8:50 ` Andy Yan 2024-12-30 2:20 ` Chris Hofstaedtler @ 2025-02-02 20:34 ` Pavel Golikov 2 siblings, 0 replies; 10+ messages in thread From: Pavel Golikov @ 2025-02-02 20:34 UTC (permalink / raw) To: Heiko Stuebner Cc: andy.yan, maarten.lankhorst, mripard, tzimmermann, robh, krzk+dt, conor+dt, andrzej.hajda, neil.armstrong, rfoss, Laurent.pinchart, jonas, jernej.skrabec, dri-devel, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel, quentin.schulz, Heiko Stuebner, Daniel Semkowicz, Dmitry Yashin, Pavel Golikov Hi, On Tue, Dec 10, 2024 at 12:10:19AM +0100, Heiko Stuebner wrote: > From: Heiko Stuebner <heiko.stuebner@cherry.de> > > Add a Synopsys Designware MIPI DSI host DRM bridge driver for their > DSI2 host controller, based on the Rockchip version from the driver > rockchip/dw-mipi-dsi2.c in their vendor-kernel with phy & bridge APIs. > > While the driver is heavily modelled after the previous IP, the register > set of this DSI2 controller is completely different and there are also > additional properties like the variable-width phy interface. > > Tested-by: Daniel Semkowicz <dse@thaumatec.com> > Tested-by: Dmitry Yashin <dmt.yashin@gmail.com> > Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org> > Signed-off-by: Heiko Stuebner <heiko.stuebner@cherry.de> [..] > +static void dw_mipi_dsi2_set_data_stream_mode(struct dw_mipi_dsi2 *dsi2) > +{ > + u32 mode; > + int ret; > + > + regmap_write(dsi2->regmap, DSI2_MODE_CTRL, DATA_STREAM_MODE); > + ret = regmap_read_poll_timeout(dsi2->regmap, DSI2_MODE_STATUS, > + mode, mode & DATA_STREAM_MODE, > + 1000, MODE_STATUS_TIMEOUT_US); > + if (ret < 0) > + dev_err(dsi2->dev, "failed to enter data stream mode\n"); > +} > + > +static void dw_mipi_dsi2_set_cmd_mode(struct dw_mipi_dsi2 *dsi2) > +{ > + u32 mode; > + int ret; > + > + regmap_write(dsi2->regmap, DSI2_MODE_CTRL, COMMAND_MODE); > + ret = regmap_read_poll_timeout(dsi2->regmap, DSI2_MODE_STATUS, > + mode, mode & COMMAND_MODE, > + 1000, MODE_STATUS_TIMEOUT_US); > + if (ret < 0) > + dev_err(dsi2->dev, "failed to enter data stream mode\n"); We've failed to enter command mode, not stream mode. Wrong error message caused by copy and paste? > +} > + > +static void dw_mipi_dsi2_host_softrst(struct dw_mipi_dsi2 *dsi2) > +{ Looks like it is necessary to also do apb reset here, like vendor kernel does. Without apb reset, panel sometimes does not properly come up after DPMS switch. Tested with modetest -M rockchip -w 83:DPMS:3 I'm not sure whether it is Debian specific, but setting connector's DPMS property to OFF causes immediate panel reset. Something is restoring its value to ON immediately. > + regmap_write(dsi2->regmap, DSI2_SOFT_RESET, 0x0); > + usleep_range(50, 100); > + regmap_write(dsi2->regmap, DSI2_SOFT_RESET, > + SYS_RSTN | PHY_RSTN | IPI_RSTN); > +} ... > +static void dw_mipi_dsi2_phy_ratio_cfg(struct dw_mipi_dsi2 *dsi2) > +{ > + struct drm_display_mode *mode = &dsi2->mode; > + u64 sys_clk = clk_get_rate(dsi2->sys_clk); > + u64 pixel_clk, ipi_clk, phy_hsclk; > + u64 tmp; > + > + /* > + * in DPHY mode, the phy_hstx_clk is exactly 1/16 the Lane high-speed > + * data rate; In CPHY mode, the phy_hstx_clk is exactly 1/7 the trio > + * high speed symbol rate. > + */ > + phy_hsclk = DIV_ROUND_CLOSEST_ULL(dsi2->lane_mbps * USEC_PER_SEC, 16); > + > + /* IPI_RATIO_MAN_CFG = PHY_HSTX_CLK / IPI_CLK */ > + pixel_clk = mode->crtc_clock * MSEC_PER_SEC; > + ipi_clk = pixel_clk / 4; > + > + tmp = DIV_ROUND_CLOSEST_ULL(phy_hsclk << 16, ipi_clk); > + regmap_write(dsi2->regmap, DSI2_PHY_IPI_RATIO_MAN_CFG, > + PHY_IPI_RATIO(tmp)); > + > + /* > + * SYS_RATIO_MAN_CFG = MIPI_DCPHY_HSCLK_Freq / MIPI_DCPHY_HSCLK_Freq > + */ According to TRM SYS_RATIO_MAN_CFG = MIPI_DCPHY_HSCLK_Freq / SYS_CLK_Freq So, this comment is simply wrong. Looks like a typo. Also, earlier comment does not mention _Freq part in frequency names. Maybe it would be better to make it consistent? > + tmp = DIV_ROUND_CLOSEST_ULL(phy_hsclk << 16, sys_clk); > + regmap_write(dsi2->regmap, DSI2_PHY_SYS_RATIO_MAN_CFG, > + PHY_SYS_RATIO(tmp)); > +} > + > +static void dw_mipi_dsi2_lp2hs_or_hs2lp_cfg(struct dw_mipi_dsi2 *dsi2) > +{ > + const struct dw_mipi_dsi2_phy_ops *phy_ops = dsi2->plat_data->phy_ops; > + struct dw_mipi_dsi2_phy_timing timing; > + int ret; > + > + ret = phy_ops->get_timing(dsi2->plat_data->priv_data, > + dsi2->lane_mbps, &timing); > + if (ret) > + dev_err(dsi2->dev, "Retrieving phy timings failed\n"); > + > + regmap_write(dsi2->regmap, DSI2_PHY_LP2HS_MAN_CFG, PHY_LP2HS_TIME(timing.data_lp2hs)); > + regmap_write(dsi2->regmap, DSI2_PHY_HS2LP_MAN_CFG, PHY_HS2LP_TIME(timing.data_hs2lp)); I also had to set DSI2_PHY_MAX_RD_T_MAN_CFG to 10000 as it is done in dw-mipi-dsi (v1) driver to make my panel work properly in LPM mode. > +} ... > +static const struct regmap_config dw_mipi_dsi2_regmap_config = { > + .name = "dsi2-host", > + .reg_bits = 32, > + .val_bits = 32, > + .reg_stride = 4, > + .fast_io = true, Maybe it would be good to also set max_register here to make proper regmap available through debugfs? Or max_register_is_0 otherwise? Without this, only first register is available for dumping. Not quite relevant here, as this bridge is not currently used alone, but relevant for Rockchip specific glue driver. > +}; ... I've managed to successfully enable my DSI panel [0]. It has Lincoln Technology Solutions ZV070WUV-L50 board [1], which, in turn, is based on Focal Tech FT7250 display controller. This panel works flawlessly with rk3568 (but at most at ~55 Hz refresh rate, because rk3568 does not support modes higher than 1920x1080@60, so, I had to lower refresh rate to make it work). But has some troubles with rk3588. It works fine with vendor kernel with LPM disabled and "auto-calculation-mode" device tree property set. As far as I can understand, auto calculation mode is not going to be supported by mainline, and disabling LPM is not an option to me. So, to make it work with this series I had to increase PHY_SYS_RATIO twice. Maybe Andy could shade the light on what automatic mode calculation algorithm does? Why in automatic mode PHY_SYS_RATIO value is twice larger than value computed by formula in DSI controller application note? Panel configuration: static const struct drm_display_mode default_mode_lincoln = { .hdisplay = 1200, .hsync_start = 1200 + 20, .hsync_end = 1200 + 20 + 8, .htotal = 1200 + 20 + 8 + 22, .vdisplay = 1920, .vsync_start = 1920 + 178, .vsync_end = 1920 + 178 + 8, .vtotal = 1920 + 178 + 8 + 32, .clock = 148500, .width_mm = 94, .height_mm = 151, }; dsi->lanes = 4; dsi->format = MIPI_DSI_FMT_RGB888; dsi->mode_flags = MIPI_DSI_MODE_LPM | MIPI_DSI_MODE_VIDEO | MIPI_DSI_MODE_NO_EOT_PACKET | MIPI_DSI_MODE_VIDEO_BURST; Regmap diff between manual mode and automatic mode: diff --git a/root/regs_manual b/root/regs_auto index 117cb5b..5dce301 100644 --- a/root/regs_manual +++ b/root/regs_auto @@ -7,9 +7,9 @@ 018: 00000000 01c: 00000003 020: 00000303 -024: 00000001 +024: 00000000 028: 00000000 -02c: 0161082c +02c: 04b70728 030: 00000000 034: 00000000 038: 00000000 @@ -65,20 +65,20 @@ 100: 00000130 104: 00000900 108: 001f0000 -10c: 000cb0b5 -110: 00000000 -114: 000ae64f -118: 00000000 +10c: 00000000 +110: 0013be38 +114: 00000000 +118: 00114671 11c: 00000000 -120: 00000000 +120: 000003d3 124: 00000000 -128: 00000000 +128: 00952167 12c: 00000000 -130: 00000000 -134: 0001aaab -138: 00000000 -13c: 00002d21 -140: 00000000 +130: 00658920 +134: 00000000 +138: 0001aaad +13c: 00000000 +140: 00005a41 144: 00000000 148: 00000000 14c: 00000000 @@ -129,7 +129,7 @@ 200: 00000002 204: 00000000 208: ffff0000 -20c: 00100072 +20c: 00000072 210: 00000000 214: 00000000 218: 00000000 @@ -191,20 +191,20 @@ 2f8: 00000000 2fc: 00000000 300: 00000050 -304: 00035555 -308: 00000000 -30c: 00092aab -310: 00000000 -314: 01f40000 -318: 00000000 -31c: 0208d555 -320: 00000000 -324: 00000008 -328: 00000000 -32c: 00000020 -330: 00000000 -334: 00000780 -338: 00000000 -33c: 000000b2 -340: 00000000 +304: 00000000 +308: 0003555a +30c: 00000000 +310: 00092ab7 +314: 00000000 +318: 01f402bc +31c: 00000000 +320: 0208d82e +324: 00000000 +328: 00000008 +32c: 00000000 +330: 00000020 +334: 00000000 +338: 00000780 +33c: 00000000 +340: 000000b2 344: 000004b0 The only irrelevant difference here is LPM mode enable bit. Calculated hsclk is 61875000, sys clk is 351000000, MIPI lane speed is 990 mbps. VOP pixel clock is 37125000 if it is relevant in any way. Thanks, Pavel [0] https://www.hello-lighting.com/product/7-0-ips-1200x1920/ [1] http://www.microtech-lcd.cn/upfile/202109/2021092649634025.jpg ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH v4 2/3] dt-bindings: display: rockchip: Add schema for RK3588 DW DSI2 controller 2024-12-09 23:10 [PATCH v4 0/3] drm/rockchip: Add driver for the new DSI2 controller Heiko Stuebner 2024-12-09 23:10 ` [PATCH v4 1/3] drm/bridge/synopsys: Add MIPI DSI2 host controller bridge Heiko Stuebner @ 2024-12-09 23:10 ` Heiko Stuebner 2024-12-09 23:10 ` [PATCH v4 3/3] drm/rockchip: Add MIPI DSI2 glue driver for RK3588 Heiko Stuebner 2024-12-10 23:12 ` [PATCH v4 0/3] drm/rockchip: Add driver for the new DSI2 controller Heiko Stuebner 3 siblings, 0 replies; 10+ messages in thread From: Heiko Stuebner @ 2024-12-09 23:10 UTC (permalink / raw) To: heiko Cc: andy.yan, maarten.lankhorst, mripard, tzimmermann, robh, krzk+dt, conor+dt, andrzej.hajda, neil.armstrong, rfoss, Laurent.pinchart, jonas, jernej.skrabec, dri-devel, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel, quentin.schulz, Heiko Stuebner, Dmitry Yashin From: Heiko Stuebner <heiko.stuebner@cherry.de> The Display Serial Interface 2 (DSI-2) is part of a group of communication protocols defined by the MIPI Alliance. The RK3588 implements this specification in its two MIPI DSI-2 Host Controllers that are based on a new Synopsys IP. Tested-by: Dmitry Yashin <dmt.yashin@gmail.com> Reviewed-by: Rob Herring (Arm) <robh@kernel.org> Signed-off-by: Heiko Stuebner <heiko.stuebner@cherry.de> --- .../rockchip/rockchip,rk3588-mipi-dsi2.yaml | 120 ++++++++++++++++++ 1 file changed, 120 insertions(+) create mode 100644 Documentation/devicetree/bindings/display/rockchip/rockchip,rk3588-mipi-dsi2.yaml diff --git a/Documentation/devicetree/bindings/display/rockchip/rockchip,rk3588-mipi-dsi2.yaml b/Documentation/devicetree/bindings/display/rockchip/rockchip,rk3588-mipi-dsi2.yaml new file mode 100644 index 000000000000..53384e47b507 --- /dev/null +++ b/Documentation/devicetree/bindings/display/rockchip/rockchip,rk3588-mipi-dsi2.yaml @@ -0,0 +1,120 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/display/rockchip/rockchip,rk3588-mipi-dsi2.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Rockchip specific extensions to the Synopsys Designware MIPI DSI2 + +maintainers: + - Heiko Stuebner <heiko@sntech.de> + +properties: + compatible: + enum: + - rockchip,rk3588-mipi-dsi2 + + reg: + maxItems: 1 + + interrupts: + maxItems: 1 + + clocks: + maxItems: 2 + + clock-names: + items: + - const: pclk + - const: sys + + rockchip,grf: + $ref: /schemas/types.yaml#/definitions/phandle + description: + This SoC uses GRF regs to switch between vopl/vopb. + + phys: + maxItems: 1 + + phy-names: + const: dcphy + + power-domains: + maxItems: 1 + + resets: + maxItems: 1 + + reset-names: + const: apb + + ports: + $ref: /schemas/graph.yaml#/properties/ports + + properties: + port@0: + $ref: /schemas/graph.yaml#/properties/port + description: Input node to receive pixel data. + + port@1: + $ref: /schemas/graph.yaml#/properties/port + description: DSI output node to panel. + + required: + - port@0 + - port@1 + +required: + - compatible + - clocks + - clock-names + - rockchip,grf + - phys + - phy-names + - ports + - reg + +allOf: + - $ref: /schemas/display/dsi-controller.yaml# + +unevaluatedProperties: false + +examples: + - | + #include <dt-bindings/clock/rockchip,rk3588-cru.h> + #include <dt-bindings/interrupt-controller/arm-gic.h> + #include <dt-bindings/interrupt-controller/irq.h> + #include <dt-bindings/phy/phy.h> + #include <dt-bindings/power/rk3588-power.h> + #include <dt-bindings/reset/rockchip,rk3588-cru.h> + + soc { + #address-cells = <2>; + #size-cells = <2>; + + dsi@fde20000 { + compatible = "rockchip,rk3588-mipi-dsi2"; + reg = <0x0 0xfde20000 0x0 0x10000>; + interrupts = <GIC_SPI 167 IRQ_TYPE_LEVEL_HIGH 0>; + clocks = <&cru PCLK_DSIHOST0>, <&cru CLK_DSIHOST0>; + clock-names = "pclk", "sys"; + resets = <&cru SRST_P_DSIHOST0>; + reset-names = "apb"; + power-domains = <&power RK3588_PD_VOP>; + phys = <&mipidcphy0 PHY_TYPE_DPHY>; + phy-names = "dcphy"; + rockchip,grf = <&vop_grf>; + + ports { + #address-cells = <1>; + #size-cells = <0>; + dsi0_in: port@0 { + reg = <0>; + }; + + dsi0_out: port@1 { + reg = <1>; + }; + }; + }; + }; -- 2.45.2 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH v4 3/3] drm/rockchip: Add MIPI DSI2 glue driver for RK3588 2024-12-09 23:10 [PATCH v4 0/3] drm/rockchip: Add driver for the new DSI2 controller Heiko Stuebner 2024-12-09 23:10 ` [PATCH v4 1/3] drm/bridge/synopsys: Add MIPI DSI2 host controller bridge Heiko Stuebner 2024-12-09 23:10 ` [PATCH v4 2/3] dt-bindings: display: rockchip: Add schema for RK3588 DW DSI2 controller Heiko Stuebner @ 2024-12-09 23:10 ` Heiko Stuebner 2024-12-10 8:51 ` Andy Yan 2024-12-10 23:12 ` [PATCH v4 0/3] drm/rockchip: Add driver for the new DSI2 controller Heiko Stuebner 3 siblings, 1 reply; 10+ messages in thread From: Heiko Stuebner @ 2024-12-09 23:10 UTC (permalink / raw) To: heiko Cc: andy.yan, maarten.lankhorst, mripard, tzimmermann, robh, krzk+dt, conor+dt, andrzej.hajda, neil.armstrong, rfoss, Laurent.pinchart, jonas, jernej.skrabec, dri-devel, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel, quentin.schulz, Heiko Stuebner, Daniel Semkowicz, Dmitry Yashin From: Heiko Stuebner <heiko.stuebner@cherry.de> This adds the glue code for the MIPI DSI2 bridge on Rockchip SoCs and enables its use on the RK3588. Right now the DSI2 controller is always paired with a DC-phy based on a Samsung IP, so the interface values are set statically for now. This stays true for the upcoming RK3576 as well. Tested-by: Daniel Semkowicz <dse@thaumatec.com> Tested-by: Dmitry Yashin <dmt.yashin@gmail.com> Signed-off-by: Heiko Stuebner <heiko.stuebner@cherry.de> --- drivers/gpu/drm/rockchip/Kconfig | 10 + drivers/gpu/drm/rockchip/Makefile | 1 + .../gpu/drm/rockchip/dw-mipi-dsi2-rockchip.c | 487 ++++++++++++++++++ drivers/gpu/drm/rockchip/rockchip_drm_drv.c | 2 + drivers/gpu/drm/rockchip/rockchip_drm_drv.h | 1 + 5 files changed, 501 insertions(+) create mode 100644 drivers/gpu/drm/rockchip/dw-mipi-dsi2-rockchip.c diff --git a/drivers/gpu/drm/rockchip/Kconfig b/drivers/gpu/drm/rockchip/Kconfig index 448fadd4ba15..adee24b4e7bc 100644 --- a/drivers/gpu/drm/rockchip/Kconfig +++ b/drivers/gpu/drm/rockchip/Kconfig @@ -10,6 +10,7 @@ config DRM_ROCKCHIP select DRM_DW_HDMI if ROCKCHIP_DW_HDMI select DRM_DW_HDMI_QP if ROCKCHIP_DW_HDMI_QP select DRM_DW_MIPI_DSI if ROCKCHIP_DW_MIPI_DSI + select DRM_DW_MIPI_DSI2 if ROCKCHIP_DW_MIPI_DSI2 select GENERIC_PHY if ROCKCHIP_DW_MIPI_DSI select GENERIC_PHY_MIPI_DPHY if ROCKCHIP_DW_MIPI_DSI select SND_SOC_HDMI_CODEC if ROCKCHIP_CDN_DP && SND_SOC @@ -81,6 +82,15 @@ config ROCKCHIP_DW_MIPI_DSI enable MIPI DSI on RK3288 or RK3399 based SoC, you should select this option. +config ROCKCHIP_DW_MIPI_DSI2 + bool "Rockchip specific extensions for Synopsys DW MIPI DSI2" + select GENERIC_PHY_MIPI_DPHY + help + This selects support for Rockchip SoC specific extensions + for the Synopsys DesignWare DSI2 driver. If you want to + enable MIPI DSI on RK3576 or RK3588 based SoC, you should + select this option. + config ROCKCHIP_INNO_HDMI bool "Rockchip specific extensions for Innosilicon HDMI" select DRM_DISPLAY_HDMI_HELPER diff --git a/drivers/gpu/drm/rockchip/Makefile b/drivers/gpu/drm/rockchip/Makefile index 3eab662a5a1d..2b867cebbc12 100644 --- a/drivers/gpu/drm/rockchip/Makefile +++ b/drivers/gpu/drm/rockchip/Makefile @@ -13,6 +13,7 @@ rockchipdrm-$(CONFIG_ROCKCHIP_CDN_DP) += cdn-dp-core.o cdn-dp-reg.o rockchipdrm-$(CONFIG_ROCKCHIP_DW_HDMI) += dw_hdmi-rockchip.o rockchipdrm-$(CONFIG_ROCKCHIP_DW_HDMI_QP) += dw_hdmi_qp-rockchip.o rockchipdrm-$(CONFIG_ROCKCHIP_DW_MIPI_DSI) += dw-mipi-dsi-rockchip.o +rockchipdrm-$(CONFIG_ROCKCHIP_DW_MIPI_DSI2) += dw-mipi-dsi2-rockchip.o rockchipdrm-$(CONFIG_ROCKCHIP_INNO_HDMI) += inno_hdmi.o rockchipdrm-$(CONFIG_ROCKCHIP_LVDS) += rockchip_lvds.o rockchipdrm-$(CONFIG_ROCKCHIP_RGB) += rockchip_rgb.o diff --git a/drivers/gpu/drm/rockchip/dw-mipi-dsi2-rockchip.c b/drivers/gpu/drm/rockchip/dw-mipi-dsi2-rockchip.c new file mode 100644 index 000000000000..cdd490778756 --- /dev/null +++ b/drivers/gpu/drm/rockchip/dw-mipi-dsi2-rockchip.c @@ -0,0 +1,487 @@ +// SPDX-License-Identifier: GPL-2.0+ +/* + * Copyright (C) 2024 Rockchip Electronics Co., Ltd. + * Author: + * Guochun Huang <hero.huang@rock-chips.com> + * Heiko Stuebner <heiko.stuebner@cherry.de> + */ + +#include <linux/bitfield.h> +#include <linux/clk.h> +#include <linux/component.h> +#include <linux/media-bus-format.h> +#include <linux/mod_devicetable.h> +#include <linux/module.h> +#include <linux/of.h> +#include <linux/pm_runtime.h> +#include <linux/platform_device.h> +#include <linux/regmap.h> +#include <linux/reset.h> +#include <linux/mfd/syscon.h> +#include <linux/phy/phy.h> + +#include <drm/bridge/dw_mipi_dsi2.h> +#include <drm/drm_mipi_dsi.h> +#include <drm/drm_of.h> +#include <drm/drm_simple_kms_helper.h> + +#include <uapi/linux/videodev2.h> + +#include "rockchip_drm_drv.h" + +#define PSEC_PER_SEC 1000000000000LL + +struct dsigrf_reg { + u16 offset; + u16 lsb; + u16 msb; +}; + +enum grf_reg_fields { + TXREQCLKHS_EN, + GATING_EN, + IPI_SHUTDN, + IPI_COLORM, + IPI_COLOR_DEPTH, + IPI_FORMAT, + MAX_FIELDS, +}; + +#define IPI_DEPTH_5_6_5_BITS 0x02 +#define IPI_DEPTH_6_BITS 0x03 +#define IPI_DEPTH_8_BITS 0x05 +#define IPI_DEPTH_10_BITS 0x06 + +struct rockchip_dw_dsi2_chip_data { + u32 reg; + const struct dsigrf_reg *grf_regs; + unsigned long long max_bit_rate_per_lane; +}; + +struct dw_mipi_dsi2_rockchip { + struct device *dev; + struct rockchip_encoder encoder; + struct regmap *regmap; + + unsigned int lane_mbps; /* per lane */ + u32 format; + + struct regmap *grf_regmap; + struct phy *phy; + union phy_configure_opts phy_opts; + + struct dw_mipi_dsi2 *dmd; + struct dw_mipi_dsi2_plat_data pdata; + const struct rockchip_dw_dsi2_chip_data *cdata; +}; + +static inline struct dw_mipi_dsi2_rockchip *to_dsi2(struct drm_encoder *encoder) +{ + struct rockchip_encoder *rkencoder = to_rockchip_encoder(encoder); + + return container_of(rkencoder, struct dw_mipi_dsi2_rockchip, encoder); +} + +static void grf_field_write(struct dw_mipi_dsi2_rockchip *dsi2, enum grf_reg_fields index, + unsigned int val) +{ + const struct dsigrf_reg *field = &dsi2->cdata->grf_regs[index]; + + if (!field) + return; + + regmap_write(dsi2->grf_regmap, field->offset, + (val << field->lsb) | (GENMASK(field->msb, field->lsb) << 16)); +} + +static int dw_mipi_dsi2_phy_init(void *priv_data) +{ + return 0; +} + +static void dw_mipi_dsi2_phy_power_on(void *priv_data) +{ + struct dw_mipi_dsi2_rockchip *dsi2 = priv_data; + int ret; + + ret = phy_set_mode(dsi2->phy, PHY_MODE_MIPI_DPHY); + if (ret) { + dev_err(dsi2->dev, "Failed to set phy mode: %d\n", ret); + return; + } + + phy_configure(dsi2->phy, &dsi2->phy_opts); + phy_power_on(dsi2->phy); +} + +static void dw_mipi_dsi2_phy_power_off(void *priv_data) +{ + struct dw_mipi_dsi2_rockchip *dsi2 = priv_data; + + phy_power_off(dsi2->phy); +} + +static int +dw_mipi_dsi2_get_lane_mbps(void *priv_data, const struct drm_display_mode *mode, + unsigned long mode_flags, u32 lanes, u32 format, + unsigned int *lane_mbps) +{ + struct dw_mipi_dsi2_rockchip *dsi2 = priv_data; + u64 max_lane_rate, target_phyclk; + unsigned int lane_rate_kbps; + int bpp; + + max_lane_rate = dsi2->cdata->max_bit_rate_per_lane; + + dsi2->format = format; + bpp = mipi_dsi_pixel_format_to_bpp(format); + if (bpp < 0) { + dev_err(dsi2->dev, "failed to get bpp for pixel format %d\n", format); + return bpp; + } + + lane_rate_kbps = mode->clock * bpp / lanes; + + /* + * Set BW a little larger only in video burst mode in + * consideration of the protocol overhead and HS mode + * switching to BLLP mode, take 1 / 0.9, since Mbps must + * big than bandwidth of RGB + */ + if (mode_flags & MIPI_DSI_MODE_VIDEO_BURST) + lane_rate_kbps = (lane_rate_kbps * 10) / 9; + + if (lane_rate_kbps > max_lane_rate) { + dev_err(dsi2->dev, "DPHY clock frequency is out of range\n"); + return -ERANGE; + } + + dsi2->lane_mbps = lane_rate_kbps / 1000; + *lane_mbps = dsi2->lane_mbps; + + if (dsi2->phy) { + target_phyclk = DIV_ROUND_CLOSEST_ULL(lane_rate_kbps * lanes * 1000, bpp); + phy_mipi_dphy_get_default_config(target_phyclk, bpp, lanes, + &dsi2->phy_opts.mipi_dphy); + } + + return 0; +} + +static void dw_mipi_dsi2_phy_get_iface(void *priv_data, struct dw_mipi_dsi2_phy_iface *iface) +{ + /* PPI width is fixed to 16 bits in DCPHY */ + iface->ppi_width = 16; + iface->phy_type = DW_MIPI_DSI2_DPHY; +} + +static int +dw_mipi_dsi2_phy_get_timing(void *priv_data, unsigned int lane_mbps, + struct dw_mipi_dsi2_phy_timing *timing) +{ + struct dw_mipi_dsi2_rockchip *dsi2 = priv_data; + struct phy_configure_opts_mipi_dphy *cfg = &dsi2->phy_opts.mipi_dphy; + unsigned long long tmp, ui; + unsigned long long hstx_clk; + + hstx_clk = DIV_ROUND_CLOSEST_ULL(dsi2->lane_mbps * USEC_PER_SEC, 16); + + ui = ALIGN(PSEC_PER_SEC, hstx_clk); + do_div(ui, hstx_clk); + + /* PHY_LP2HS_TIME = (TLPX + THS-PREPARE + THS-ZERO) / Tphy_hstx_clk */ + tmp = cfg->lpx + cfg->hs_prepare + cfg->hs_zero; + tmp = DIV_ROUND_CLOSEST_ULL(tmp << 16, ui); + timing->data_lp2hs = tmp; + + /* PHY_HS2LP_TIME = (THS-TRAIL + THS-EXIT) / Tphy_hstx_clk */ + tmp = cfg->hs_trail + cfg->hs_exit; + tmp = DIV_ROUND_CLOSEST_ULL(tmp << 16, ui); + timing->data_hs2lp = tmp; + + return 0; +} + +static const struct dw_mipi_dsi2_phy_ops dw_mipi_dsi2_rockchip_phy_ops = { + .init = dw_mipi_dsi2_phy_init, + .power_on = dw_mipi_dsi2_phy_power_on, + .power_off = dw_mipi_dsi2_phy_power_off, + .get_interface = dw_mipi_dsi2_phy_get_iface, + .get_lane_mbps = dw_mipi_dsi2_get_lane_mbps, + .get_timing = dw_mipi_dsi2_phy_get_timing, +}; + +static void dw_mipi_dsi2_encoder_atomic_enable(struct drm_encoder *encoder, + struct drm_atomic_state *state) +{ + struct dw_mipi_dsi2_rockchip *dsi2 = to_dsi2(encoder); + u32 color_depth; + + switch (dsi2->format) { + case MIPI_DSI_FMT_RGB666: + case MIPI_DSI_FMT_RGB666_PACKED: + color_depth = IPI_DEPTH_6_BITS; + break; + case MIPI_DSI_FMT_RGB565: + color_depth = IPI_DEPTH_5_6_5_BITS; + break; + case MIPI_DSI_FMT_RGB888: + color_depth = IPI_DEPTH_8_BITS; + break; + default: + /* Should've been caught by atomic_check */ + WARN_ON(1); + return; + } + + grf_field_write(dsi2, IPI_COLOR_DEPTH, color_depth); +} + +static int +dw_mipi_dsi2_encoder_atomic_check(struct drm_encoder *encoder, + struct drm_crtc_state *crtc_state, + struct drm_connector_state *conn_state) +{ + struct rockchip_crtc_state *s = to_rockchip_crtc_state(crtc_state); + struct dw_mipi_dsi2_rockchip *dsi2 = to_dsi2(encoder); + struct drm_connector *connector = conn_state->connector; + struct drm_display_info *info = &connector->display_info; + + switch (dsi2->format) { + case MIPI_DSI_FMT_RGB666: + case MIPI_DSI_FMT_RGB666_PACKED: + s->output_mode = ROCKCHIP_OUT_MODE_P666; + break; + case MIPI_DSI_FMT_RGB565: + s->output_mode = ROCKCHIP_OUT_MODE_P565; + break; + case MIPI_DSI_FMT_RGB888: + s->output_mode = ROCKCHIP_OUT_MODE_P888; + break; + default: + WARN_ON(1); + return -EINVAL; + } + + if (info->num_bus_formats) + s->bus_format = info->bus_formats[0]; + else + s->bus_format = MEDIA_BUS_FMT_RGB888_1X24; + + s->output_type = DRM_MODE_CONNECTOR_DSI; + s->bus_flags = info->bus_flags; + s->color_space = V4L2_COLORSPACE_DEFAULT; + + return 0; +} + +static const struct drm_encoder_helper_funcs +dw_mipi_dsi2_encoder_helper_funcs = { + .atomic_enable = dw_mipi_dsi2_encoder_atomic_enable, + .atomic_check = dw_mipi_dsi2_encoder_atomic_check, +}; + +static int rockchip_dsi2_drm_create_encoder(struct dw_mipi_dsi2_rockchip *dsi2, + struct drm_device *drm_dev) +{ + struct drm_encoder *encoder = &dsi2->encoder.encoder; + int ret; + + encoder->possible_crtcs = drm_of_find_possible_crtcs(drm_dev, + dsi2->dev->of_node); + + ret = drm_simple_encoder_init(drm_dev, encoder, DRM_MODE_ENCODER_DSI); + if (ret) { + dev_err(dsi2->dev, "Failed to initialize encoder with drm\n"); + return ret; + } + + drm_encoder_helper_add(encoder, &dw_mipi_dsi2_encoder_helper_funcs); + + return 0; +} + +static int dw_mipi_dsi2_rockchip_bind(struct device *dev, struct device *master, + void *data) +{ + struct dw_mipi_dsi2_rockchip *dsi2 = dev_get_drvdata(dev); + struct drm_device *drm_dev = data; + int ret; + + ret = rockchip_dsi2_drm_create_encoder(dsi2, drm_dev); + if (ret) + return dev_err_probe(dev, ret, "Failed to create drm encoder\n"); + + rockchip_drm_encoder_set_crtc_endpoint_id(&dsi2->encoder, + dev->of_node, 0, 0); + + ret = dw_mipi_dsi2_bind(dsi2->dmd, &dsi2->encoder.encoder); + if (ret) + return dev_err_probe(dev, ret, "Failed to bind\n"); + + return 0; +} + +static void dw_mipi_dsi2_rockchip_unbind(struct device *dev, struct device *master, + void *data) +{ + struct dw_mipi_dsi2_rockchip *dsi2 = dev_get_drvdata(dev); + + dw_mipi_dsi2_unbind(dsi2->dmd); +} + +static const struct component_ops dw_mipi_dsi2_rockchip_ops = { + .bind = dw_mipi_dsi2_rockchip_bind, + .unbind = dw_mipi_dsi2_rockchip_unbind, +}; + +static int dw_mipi_dsi2_rockchip_host_attach(void *priv_data, + struct mipi_dsi_device *device) +{ + struct dw_mipi_dsi2_rockchip *dsi2 = priv_data; + int ret; + + ret = component_add(dsi2->dev, &dw_mipi_dsi2_rockchip_ops); + if (ret) + return dev_err_probe(dsi2->dev, ret, "Failed to register component\n"); + + return 0; +} + +static int dw_mipi_dsi2_rockchip_host_detach(void *priv_data, + struct mipi_dsi_device *device) +{ + struct dw_mipi_dsi2_rockchip *dsi2 = priv_data; + + component_del(dsi2->dev, &dw_mipi_dsi2_rockchip_ops); + + return 0; +} + +static const struct dw_mipi_dsi2_host_ops dw_mipi_dsi2_rockchip_host_ops = { + .attach = dw_mipi_dsi2_rockchip_host_attach, + .detach = dw_mipi_dsi2_rockchip_host_detach, +}; + +static const struct regmap_config dw_mipi_dsi2_rockchip_regmap_config = { + .name = "dsi2-host", + .reg_bits = 32, + .val_bits = 32, + .reg_stride = 4, + .fast_io = true, +}; + +static int dw_mipi_dsi2_rockchip_probe(struct platform_device *pdev) +{ + struct device *dev = &pdev->dev; + struct device_node *np = dev->of_node; + const struct rockchip_dw_dsi2_chip_data *cdata = + of_device_get_match_data(dev); + struct dw_mipi_dsi2_rockchip *dsi2; + struct resource *res; + void __iomem *base; + int i; + + dsi2 = devm_kzalloc(dev, sizeof(*dsi2), GFP_KERNEL); + if (!dsi2) + return -ENOMEM; + + base = devm_platform_get_and_ioremap_resource(pdev, 0, &res); + if (IS_ERR(base)) + return dev_err_probe(dev, PTR_ERR(base), "Unable to get dsi registers\n"); + + dsi2->regmap = devm_regmap_init_mmio(dev, base, &dw_mipi_dsi2_rockchip_regmap_config); + if (IS_ERR(dsi2->regmap)) + return dev_err_probe(dev, PTR_ERR(dsi2->regmap), "failed to init register map\n"); + + i = 0; + while (cdata[i].reg) { + if (cdata[i].reg == res->start) { + dsi2->cdata = &cdata[i]; + break; + } + + i++; + } + + if (!dsi2->cdata) + return dev_err_probe(dev, -EINVAL, "No dsi-config for %s node\n", np->name); + + dsi2->grf_regmap = syscon_regmap_lookup_by_phandle(dev->of_node, "rockchip,grf"); + if (IS_ERR(dsi2->grf_regmap)) + return dev_err_probe(dsi2->dev, PTR_ERR(dsi2->grf_regmap), "Unable to get grf\n"); + + dsi2->phy = devm_phy_optional_get(dev, "dcphy"); + if (IS_ERR(dsi2->phy)) + return dev_err_probe(dev, PTR_ERR(dsi2->phy), "failed to get mipi phy\n"); + + dsi2->dev = dev; + dsi2->pdata.regmap = dsi2->regmap; + dsi2->pdata.max_data_lanes = 4; + dsi2->pdata.phy_ops = &dw_mipi_dsi2_rockchip_phy_ops; + dsi2->pdata.host_ops = &dw_mipi_dsi2_rockchip_host_ops; + dsi2->pdata.priv_data = dsi2; + platform_set_drvdata(pdev, dsi2); + + dsi2->dmd = dw_mipi_dsi2_probe(pdev, &dsi2->pdata); + if (IS_ERR(dsi2->dmd)) + return dev_err_probe(dev, PTR_ERR(dsi2->dmd), "Failed to probe dw_mipi_dsi2\n"); + + return 0; +} + +static void dw_mipi_dsi2_rockchip_remove(struct platform_device *pdev) +{ + struct dw_mipi_dsi2_rockchip *dsi2 = platform_get_drvdata(pdev); + + dw_mipi_dsi2_remove(dsi2->dmd); +} + +static const struct dsigrf_reg rk3588_dsi0_grf_reg_fields[MAX_FIELDS] = { + [TXREQCLKHS_EN] = { 0x0000, 11, 11 }, + [GATING_EN] = { 0x0000, 10, 10 }, + [IPI_SHUTDN] = { 0x0000, 9, 9 }, + [IPI_COLORM] = { 0x0000, 8, 8 }, + [IPI_COLOR_DEPTH] = { 0x0000, 4, 7 }, + [IPI_FORMAT] = { 0x0000, 0, 3 }, +}; + +static const struct dsigrf_reg rk3588_dsi1_grf_reg_fields[MAX_FIELDS] = { + [TXREQCLKHS_EN] = { 0x0004, 11, 11 }, + [GATING_EN] = { 0x0004, 10, 10 }, + [IPI_SHUTDN] = { 0x0004, 9, 9 }, + [IPI_COLORM] = { 0x0004, 8, 8 }, + [IPI_COLOR_DEPTH] = { 0x0004, 4, 7 }, + [IPI_FORMAT] = { 0x0004, 0, 3 }, +}; + +static const struct rockchip_dw_dsi2_chip_data rk3588_chip_data[] = { + { + .reg = 0xfde20000, + .grf_regs = rk3588_dsi0_grf_reg_fields, + .max_bit_rate_per_lane = 4500000ULL, + }, + { + .reg = 0xfde30000, + .grf_regs = rk3588_dsi1_grf_reg_fields, + .max_bit_rate_per_lane = 4500000ULL, + } +}; + +static const struct of_device_id dw_mipi_dsi2_rockchip_dt_ids[] = { + { + .compatible = "rockchip,rk3588-mipi-dsi2", + .data = &rk3588_chip_data, + }, + {} +}; +MODULE_DEVICE_TABLE(of, dw_mipi_dsi2_rockchip_dt_ids); + +struct platform_driver dw_mipi_dsi2_rockchip_driver = { + .probe = dw_mipi_dsi2_rockchip_probe, + .remove = dw_mipi_dsi2_rockchip_remove, + .driver = { + .of_match_table = dw_mipi_dsi2_rockchip_dt_ids, + .name = "dw-mipi-dsi2", + }, +}; diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_drv.c b/drivers/gpu/drm/rockchip/rockchip_drm_drv.c index ddf0be331c0a..5327ce035003 100644 --- a/drivers/gpu/drm/rockchip/rockchip_drm_drv.c +++ b/drivers/gpu/drm/rockchip/rockchip_drm_drv.c @@ -511,6 +511,8 @@ static int __init rockchip_drm_init(void) CONFIG_ROCKCHIP_DW_HDMI_QP); ADD_ROCKCHIP_SUB_DRIVER(dw_mipi_dsi_rockchip_driver, CONFIG_ROCKCHIP_DW_MIPI_DSI); + ADD_ROCKCHIP_SUB_DRIVER(dw_mipi_dsi2_rockchip_driver, + CONFIG_ROCKCHIP_DW_MIPI_DSI2); ADD_ROCKCHIP_SUB_DRIVER(inno_hdmi_driver, CONFIG_ROCKCHIP_INNO_HDMI); ADD_ROCKCHIP_SUB_DRIVER(rk3066_hdmi_driver, CONFIG_ROCKCHIP_RK3066_HDMI); diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_drv.h b/drivers/gpu/drm/rockchip/rockchip_drm_drv.h index 24b4ce5ceaf1..9c9d38a06cdf 100644 --- a/drivers/gpu/drm/rockchip/rockchip_drm_drv.h +++ b/drivers/gpu/drm/rockchip/rockchip_drm_drv.h @@ -90,6 +90,7 @@ extern struct platform_driver cdn_dp_driver; extern struct platform_driver dw_hdmi_rockchip_pltfm_driver; extern struct platform_driver dw_hdmi_qp_rockchip_pltfm_driver; extern struct platform_driver dw_mipi_dsi_rockchip_driver; +extern struct platform_driver dw_mipi_dsi2_rockchip_driver; extern struct platform_driver inno_hdmi_driver; extern struct platform_driver rockchip_dp_driver; extern struct platform_driver rockchip_lvds_driver; -- 2.45.2 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH v4 3/3] drm/rockchip: Add MIPI DSI2 glue driver for RK3588 2024-12-09 23:10 ` [PATCH v4 3/3] drm/rockchip: Add MIPI DSI2 glue driver for RK3588 Heiko Stuebner @ 2024-12-10 8:51 ` Andy Yan 0 siblings, 0 replies; 10+ messages in thread From: Andy Yan @ 2024-12-10 8:51 UTC (permalink / raw) To: Heiko Stuebner Cc: maarten.lankhorst, mripard, tzimmermann, robh, krzk+dt, conor+dt, andrzej.hajda, neil.armstrong, rfoss, Laurent.pinchart, jonas, jernej.skrabec, dri-devel, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel, quentin.schulz, Heiko Stuebner, Daniel Semkowicz, Dmitry Yashin Hello Heiko, On 12/10/24 07:10, Heiko Stuebner wrote: > From: Heiko Stuebner <heiko.stuebner@cherry.de> > > This adds the glue code for the MIPI DSI2 bridge on Rockchip SoCs and > enables its use on the RK3588. > > Right now the DSI2 controller is always paired with a DC-phy based on a > Samsung IP, so the interface values are set statically for now. > This stays true for the upcoming RK3576 as well. > > Tested-by: Daniel Semkowicz <dse@thaumatec.com> > Tested-by: Dmitry Yashin <dmt.yashin@gmail.com> > Signed-off-by: Heiko Stuebner <heiko.stuebner@cherry.de> Reviewed-by: Andy Yan <andy.yan@rock-chips.com> > --- > drivers/gpu/drm/rockchip/Kconfig | 10 + > drivers/gpu/drm/rockchip/Makefile | 1 + > .../gpu/drm/rockchip/dw-mipi-dsi2-rockchip.c | 487 ++++++++++++++++++ > drivers/gpu/drm/rockchip/rockchip_drm_drv.c | 2 + > drivers/gpu/drm/rockchip/rockchip_drm_drv.h | 1 + > 5 files changed, 501 insertions(+) > create mode 100644 drivers/gpu/drm/rockchip/dw-mipi-dsi2-rockchip.c > > diff --git a/drivers/gpu/drm/rockchip/Kconfig b/drivers/gpu/drm/rockchip/Kconfig > index 448fadd4ba15..adee24b4e7bc 100644 > --- a/drivers/gpu/drm/rockchip/Kconfig > +++ b/drivers/gpu/drm/rockchip/Kconfig > @@ -10,6 +10,7 @@ config DRM_ROCKCHIP > select DRM_DW_HDMI if ROCKCHIP_DW_HDMI > select DRM_DW_HDMI_QP if ROCKCHIP_DW_HDMI_QP > select DRM_DW_MIPI_DSI if ROCKCHIP_DW_MIPI_DSI > + select DRM_DW_MIPI_DSI2 if ROCKCHIP_DW_MIPI_DSI2 > select GENERIC_PHY if ROCKCHIP_DW_MIPI_DSI > select GENERIC_PHY_MIPI_DPHY if ROCKCHIP_DW_MIPI_DSI > select SND_SOC_HDMI_CODEC if ROCKCHIP_CDN_DP && SND_SOC > @@ -81,6 +82,15 @@ config ROCKCHIP_DW_MIPI_DSI > enable MIPI DSI on RK3288 or RK3399 based SoC, you should > select this option. > > +config ROCKCHIP_DW_MIPI_DSI2 > + bool "Rockchip specific extensions for Synopsys DW MIPI DSI2" > + select GENERIC_PHY_MIPI_DPHY > + help > + This selects support for Rockchip SoC specific extensions > + for the Synopsys DesignWare DSI2 driver. If you want to > + enable MIPI DSI on RK3576 or RK3588 based SoC, you should > + select this option. > + > config ROCKCHIP_INNO_HDMI > bool "Rockchip specific extensions for Innosilicon HDMI" > select DRM_DISPLAY_HDMI_HELPER > diff --git a/drivers/gpu/drm/rockchip/Makefile b/drivers/gpu/drm/rockchip/Makefile > index 3eab662a5a1d..2b867cebbc12 100644 > --- a/drivers/gpu/drm/rockchip/Makefile > +++ b/drivers/gpu/drm/rockchip/Makefile > @@ -13,6 +13,7 @@ rockchipdrm-$(CONFIG_ROCKCHIP_CDN_DP) += cdn-dp-core.o cdn-dp-reg.o > rockchipdrm-$(CONFIG_ROCKCHIP_DW_HDMI) += dw_hdmi-rockchip.o > rockchipdrm-$(CONFIG_ROCKCHIP_DW_HDMI_QP) += dw_hdmi_qp-rockchip.o > rockchipdrm-$(CONFIG_ROCKCHIP_DW_MIPI_DSI) += dw-mipi-dsi-rockchip.o > +rockchipdrm-$(CONFIG_ROCKCHIP_DW_MIPI_DSI2) += dw-mipi-dsi2-rockchip.o > rockchipdrm-$(CONFIG_ROCKCHIP_INNO_HDMI) += inno_hdmi.o > rockchipdrm-$(CONFIG_ROCKCHIP_LVDS) += rockchip_lvds.o > rockchipdrm-$(CONFIG_ROCKCHIP_RGB) += rockchip_rgb.o > diff --git a/drivers/gpu/drm/rockchip/dw-mipi-dsi2-rockchip.c b/drivers/gpu/drm/rockchip/dw-mipi-dsi2-rockchip.c > new file mode 100644 > index 000000000000..cdd490778756 > --- /dev/null > +++ b/drivers/gpu/drm/rockchip/dw-mipi-dsi2-rockchip.c > @@ -0,0 +1,487 @@ > +// SPDX-License-Identifier: GPL-2.0+ > +/* > + * Copyright (C) 2024 Rockchip Electronics Co., Ltd. > + * Author: > + * Guochun Huang <hero.huang@rock-chips.com> > + * Heiko Stuebner <heiko.stuebner@cherry.de> > + */ > + > +#include <linux/bitfield.h> > +#include <linux/clk.h> > +#include <linux/component.h> > +#include <linux/media-bus-format.h> > +#include <linux/mod_devicetable.h> > +#include <linux/module.h> > +#include <linux/of.h> > +#include <linux/pm_runtime.h> > +#include <linux/platform_device.h> > +#include <linux/regmap.h> > +#include <linux/reset.h> > +#include <linux/mfd/syscon.h> > +#include <linux/phy/phy.h> > + > +#include <drm/bridge/dw_mipi_dsi2.h> > +#include <drm/drm_mipi_dsi.h> > +#include <drm/drm_of.h> > +#include <drm/drm_simple_kms_helper.h> > + > +#include <uapi/linux/videodev2.h> > + > +#include "rockchip_drm_drv.h" > + > +#define PSEC_PER_SEC 1000000000000LL > + > +struct dsigrf_reg { > + u16 offset; > + u16 lsb; > + u16 msb; > +}; > + > +enum grf_reg_fields { > + TXREQCLKHS_EN, > + GATING_EN, > + IPI_SHUTDN, > + IPI_COLORM, > + IPI_COLOR_DEPTH, > + IPI_FORMAT, > + MAX_FIELDS, > +}; > + > +#define IPI_DEPTH_5_6_5_BITS 0x02 > +#define IPI_DEPTH_6_BITS 0x03 > +#define IPI_DEPTH_8_BITS 0x05 > +#define IPI_DEPTH_10_BITS 0x06 > + > +struct rockchip_dw_dsi2_chip_data { > + u32 reg; > + const struct dsigrf_reg *grf_regs; > + unsigned long long max_bit_rate_per_lane; > +}; > + > +struct dw_mipi_dsi2_rockchip { > + struct device *dev; > + struct rockchip_encoder encoder; > + struct regmap *regmap; > + > + unsigned int lane_mbps; /* per lane */ > + u32 format; > + > + struct regmap *grf_regmap; > + struct phy *phy; > + union phy_configure_opts phy_opts; > + > + struct dw_mipi_dsi2 *dmd; > + struct dw_mipi_dsi2_plat_data pdata; > + const struct rockchip_dw_dsi2_chip_data *cdata; > +}; > + > +static inline struct dw_mipi_dsi2_rockchip *to_dsi2(struct drm_encoder *encoder) > +{ > + struct rockchip_encoder *rkencoder = to_rockchip_encoder(encoder); > + > + return container_of(rkencoder, struct dw_mipi_dsi2_rockchip, encoder); > +} > + > +static void grf_field_write(struct dw_mipi_dsi2_rockchip *dsi2, enum grf_reg_fields index, > + unsigned int val) > +{ > + const struct dsigrf_reg *field = &dsi2->cdata->grf_regs[index]; > + > + if (!field) > + return; > + > + regmap_write(dsi2->grf_regmap, field->offset, > + (val << field->lsb) | (GENMASK(field->msb, field->lsb) << 16)); > +} > + > +static int dw_mipi_dsi2_phy_init(void *priv_data) > +{ > + return 0; > +} > + > +static void dw_mipi_dsi2_phy_power_on(void *priv_data) > +{ > + struct dw_mipi_dsi2_rockchip *dsi2 = priv_data; > + int ret; > + > + ret = phy_set_mode(dsi2->phy, PHY_MODE_MIPI_DPHY); > + if (ret) { > + dev_err(dsi2->dev, "Failed to set phy mode: %d\n", ret); > + return; > + } > + > + phy_configure(dsi2->phy, &dsi2->phy_opts); > + phy_power_on(dsi2->phy); > +} > + > +static void dw_mipi_dsi2_phy_power_off(void *priv_data) > +{ > + struct dw_mipi_dsi2_rockchip *dsi2 = priv_data; > + > + phy_power_off(dsi2->phy); > +} > + > +static int > +dw_mipi_dsi2_get_lane_mbps(void *priv_data, const struct drm_display_mode *mode, > + unsigned long mode_flags, u32 lanes, u32 format, > + unsigned int *lane_mbps) > +{ > + struct dw_mipi_dsi2_rockchip *dsi2 = priv_data; > + u64 max_lane_rate, target_phyclk; > + unsigned int lane_rate_kbps; > + int bpp; > + > + max_lane_rate = dsi2->cdata->max_bit_rate_per_lane; > + > + dsi2->format = format; > + bpp = mipi_dsi_pixel_format_to_bpp(format); > + if (bpp < 0) { > + dev_err(dsi2->dev, "failed to get bpp for pixel format %d\n", format); > + return bpp; > + } > + > + lane_rate_kbps = mode->clock * bpp / lanes; > + > + /* > + * Set BW a little larger only in video burst mode in > + * consideration of the protocol overhead and HS mode > + * switching to BLLP mode, take 1 / 0.9, since Mbps must > + * big than bandwidth of RGB > + */ > + if (mode_flags & MIPI_DSI_MODE_VIDEO_BURST) > + lane_rate_kbps = (lane_rate_kbps * 10) / 9; > + > + if (lane_rate_kbps > max_lane_rate) { > + dev_err(dsi2->dev, "DPHY clock frequency is out of range\n"); > + return -ERANGE; > + } > + > + dsi2->lane_mbps = lane_rate_kbps / 1000; > + *lane_mbps = dsi2->lane_mbps; > + > + if (dsi2->phy) { > + target_phyclk = DIV_ROUND_CLOSEST_ULL(lane_rate_kbps * lanes * 1000, bpp); > + phy_mipi_dphy_get_default_config(target_phyclk, bpp, lanes, > + &dsi2->phy_opts.mipi_dphy); > + } > + > + return 0; > +} > + > +static void dw_mipi_dsi2_phy_get_iface(void *priv_data, struct dw_mipi_dsi2_phy_iface *iface) > +{ > + /* PPI width is fixed to 16 bits in DCPHY */ > + iface->ppi_width = 16; > + iface->phy_type = DW_MIPI_DSI2_DPHY; > +} > + > +static int > +dw_mipi_dsi2_phy_get_timing(void *priv_data, unsigned int lane_mbps, > + struct dw_mipi_dsi2_phy_timing *timing) > +{ > + struct dw_mipi_dsi2_rockchip *dsi2 = priv_data; > + struct phy_configure_opts_mipi_dphy *cfg = &dsi2->phy_opts.mipi_dphy; > + unsigned long long tmp, ui; > + unsigned long long hstx_clk; > + > + hstx_clk = DIV_ROUND_CLOSEST_ULL(dsi2->lane_mbps * USEC_PER_SEC, 16); > + > + ui = ALIGN(PSEC_PER_SEC, hstx_clk); > + do_div(ui, hstx_clk); > + > + /* PHY_LP2HS_TIME = (TLPX + THS-PREPARE + THS-ZERO) / Tphy_hstx_clk */ > + tmp = cfg->lpx + cfg->hs_prepare + cfg->hs_zero; > + tmp = DIV_ROUND_CLOSEST_ULL(tmp << 16, ui); > + timing->data_lp2hs = tmp; > + > + /* PHY_HS2LP_TIME = (THS-TRAIL + THS-EXIT) / Tphy_hstx_clk */ > + tmp = cfg->hs_trail + cfg->hs_exit; > + tmp = DIV_ROUND_CLOSEST_ULL(tmp << 16, ui); > + timing->data_hs2lp = tmp; > + > + return 0; > +} > + > +static const struct dw_mipi_dsi2_phy_ops dw_mipi_dsi2_rockchip_phy_ops = { > + .init = dw_mipi_dsi2_phy_init, > + .power_on = dw_mipi_dsi2_phy_power_on, > + .power_off = dw_mipi_dsi2_phy_power_off, > + .get_interface = dw_mipi_dsi2_phy_get_iface, > + .get_lane_mbps = dw_mipi_dsi2_get_lane_mbps, > + .get_timing = dw_mipi_dsi2_phy_get_timing, > +}; > + > +static void dw_mipi_dsi2_encoder_atomic_enable(struct drm_encoder *encoder, > + struct drm_atomic_state *state) > +{ > + struct dw_mipi_dsi2_rockchip *dsi2 = to_dsi2(encoder); > + u32 color_depth; > + > + switch (dsi2->format) { > + case MIPI_DSI_FMT_RGB666: > + case MIPI_DSI_FMT_RGB666_PACKED: > + color_depth = IPI_DEPTH_6_BITS; > + break; > + case MIPI_DSI_FMT_RGB565: > + color_depth = IPI_DEPTH_5_6_5_BITS; > + break; > + case MIPI_DSI_FMT_RGB888: > + color_depth = IPI_DEPTH_8_BITS; > + break; > + default: > + /* Should've been caught by atomic_check */ > + WARN_ON(1); > + return; > + } > + > + grf_field_write(dsi2, IPI_COLOR_DEPTH, color_depth); > +} > + > +static int > +dw_mipi_dsi2_encoder_atomic_check(struct drm_encoder *encoder, > + struct drm_crtc_state *crtc_state, > + struct drm_connector_state *conn_state) > +{ > + struct rockchip_crtc_state *s = to_rockchip_crtc_state(crtc_state); > + struct dw_mipi_dsi2_rockchip *dsi2 = to_dsi2(encoder); > + struct drm_connector *connector = conn_state->connector; > + struct drm_display_info *info = &connector->display_info; > + > + switch (dsi2->format) { > + case MIPI_DSI_FMT_RGB666: > + case MIPI_DSI_FMT_RGB666_PACKED: > + s->output_mode = ROCKCHIP_OUT_MODE_P666; > + break; > + case MIPI_DSI_FMT_RGB565: > + s->output_mode = ROCKCHIP_OUT_MODE_P565; > + break; > + case MIPI_DSI_FMT_RGB888: > + s->output_mode = ROCKCHIP_OUT_MODE_P888; > + break; > + default: > + WARN_ON(1); > + return -EINVAL; > + } > + > + if (info->num_bus_formats) > + s->bus_format = info->bus_formats[0]; > + else > + s->bus_format = MEDIA_BUS_FMT_RGB888_1X24; > + > + s->output_type = DRM_MODE_CONNECTOR_DSI; > + s->bus_flags = info->bus_flags; > + s->color_space = V4L2_COLORSPACE_DEFAULT; > + > + return 0; > +} > + > +static const struct drm_encoder_helper_funcs > +dw_mipi_dsi2_encoder_helper_funcs = { > + .atomic_enable = dw_mipi_dsi2_encoder_atomic_enable, > + .atomic_check = dw_mipi_dsi2_encoder_atomic_check, > +}; > + > +static int rockchip_dsi2_drm_create_encoder(struct dw_mipi_dsi2_rockchip *dsi2, > + struct drm_device *drm_dev) > +{ > + struct drm_encoder *encoder = &dsi2->encoder.encoder; > + int ret; > + > + encoder->possible_crtcs = drm_of_find_possible_crtcs(drm_dev, > + dsi2->dev->of_node); > + > + ret = drm_simple_encoder_init(drm_dev, encoder, DRM_MODE_ENCODER_DSI); > + if (ret) { > + dev_err(dsi2->dev, "Failed to initialize encoder with drm\n"); > + return ret; > + } > + > + drm_encoder_helper_add(encoder, &dw_mipi_dsi2_encoder_helper_funcs); > + > + return 0; > +} > + > +static int dw_mipi_dsi2_rockchip_bind(struct device *dev, struct device *master, > + void *data) > +{ > + struct dw_mipi_dsi2_rockchip *dsi2 = dev_get_drvdata(dev); > + struct drm_device *drm_dev = data; > + int ret; > + > + ret = rockchip_dsi2_drm_create_encoder(dsi2, drm_dev); > + if (ret) > + return dev_err_probe(dev, ret, "Failed to create drm encoder\n"); > + > + rockchip_drm_encoder_set_crtc_endpoint_id(&dsi2->encoder, > + dev->of_node, 0, 0); > + > + ret = dw_mipi_dsi2_bind(dsi2->dmd, &dsi2->encoder.encoder); > + if (ret) > + return dev_err_probe(dev, ret, "Failed to bind\n"); > + > + return 0; > +} > + > +static void dw_mipi_dsi2_rockchip_unbind(struct device *dev, struct device *master, > + void *data) > +{ > + struct dw_mipi_dsi2_rockchip *dsi2 = dev_get_drvdata(dev); > + > + dw_mipi_dsi2_unbind(dsi2->dmd); > +} > + > +static const struct component_ops dw_mipi_dsi2_rockchip_ops = { > + .bind = dw_mipi_dsi2_rockchip_bind, > + .unbind = dw_mipi_dsi2_rockchip_unbind, > +}; > + > +static int dw_mipi_dsi2_rockchip_host_attach(void *priv_data, > + struct mipi_dsi_device *device) > +{ > + struct dw_mipi_dsi2_rockchip *dsi2 = priv_data; > + int ret; > + > + ret = component_add(dsi2->dev, &dw_mipi_dsi2_rockchip_ops); > + if (ret) > + return dev_err_probe(dsi2->dev, ret, "Failed to register component\n"); > + > + return 0; > +} > + > +static int dw_mipi_dsi2_rockchip_host_detach(void *priv_data, > + struct mipi_dsi_device *device) > +{ > + struct dw_mipi_dsi2_rockchip *dsi2 = priv_data; > + > + component_del(dsi2->dev, &dw_mipi_dsi2_rockchip_ops); > + > + return 0; > +} > + > +static const struct dw_mipi_dsi2_host_ops dw_mipi_dsi2_rockchip_host_ops = { > + .attach = dw_mipi_dsi2_rockchip_host_attach, > + .detach = dw_mipi_dsi2_rockchip_host_detach, > +}; > + > +static const struct regmap_config dw_mipi_dsi2_rockchip_regmap_config = { > + .name = "dsi2-host", > + .reg_bits = 32, > + .val_bits = 32, > + .reg_stride = 4, > + .fast_io = true, > +}; > + > +static int dw_mipi_dsi2_rockchip_probe(struct platform_device *pdev) > +{ > + struct device *dev = &pdev->dev; > + struct device_node *np = dev->of_node; > + const struct rockchip_dw_dsi2_chip_data *cdata = > + of_device_get_match_data(dev); > + struct dw_mipi_dsi2_rockchip *dsi2; > + struct resource *res; > + void __iomem *base; > + int i; > + > + dsi2 = devm_kzalloc(dev, sizeof(*dsi2), GFP_KERNEL); > + if (!dsi2) > + return -ENOMEM; > + > + base = devm_platform_get_and_ioremap_resource(pdev, 0, &res); > + if (IS_ERR(base)) > + return dev_err_probe(dev, PTR_ERR(base), "Unable to get dsi registers\n"); > + > + dsi2->regmap = devm_regmap_init_mmio(dev, base, &dw_mipi_dsi2_rockchip_regmap_config); > + if (IS_ERR(dsi2->regmap)) > + return dev_err_probe(dev, PTR_ERR(dsi2->regmap), "failed to init register map\n"); > + > + i = 0; > + while (cdata[i].reg) { > + if (cdata[i].reg == res->start) { > + dsi2->cdata = &cdata[i]; > + break; > + } > + > + i++; > + } > + > + if (!dsi2->cdata) > + return dev_err_probe(dev, -EINVAL, "No dsi-config for %s node\n", np->name); > + > + dsi2->grf_regmap = syscon_regmap_lookup_by_phandle(dev->of_node, "rockchip,grf"); > + if (IS_ERR(dsi2->grf_regmap)) > + return dev_err_probe(dsi2->dev, PTR_ERR(dsi2->grf_regmap), "Unable to get grf\n"); > + > + dsi2->phy = devm_phy_optional_get(dev, "dcphy"); > + if (IS_ERR(dsi2->phy)) > + return dev_err_probe(dev, PTR_ERR(dsi2->phy), "failed to get mipi phy\n"); > + > + dsi2->dev = dev; > + dsi2->pdata.regmap = dsi2->regmap; > + dsi2->pdata.max_data_lanes = 4; > + dsi2->pdata.phy_ops = &dw_mipi_dsi2_rockchip_phy_ops; > + dsi2->pdata.host_ops = &dw_mipi_dsi2_rockchip_host_ops; > + dsi2->pdata.priv_data = dsi2; > + platform_set_drvdata(pdev, dsi2); > + > + dsi2->dmd = dw_mipi_dsi2_probe(pdev, &dsi2->pdata); > + if (IS_ERR(dsi2->dmd)) > + return dev_err_probe(dev, PTR_ERR(dsi2->dmd), "Failed to probe dw_mipi_dsi2\n"); > + > + return 0; > +} > + > +static void dw_mipi_dsi2_rockchip_remove(struct platform_device *pdev) > +{ > + struct dw_mipi_dsi2_rockchip *dsi2 = platform_get_drvdata(pdev); > + > + dw_mipi_dsi2_remove(dsi2->dmd); > +} > + > +static const struct dsigrf_reg rk3588_dsi0_grf_reg_fields[MAX_FIELDS] = { > + [TXREQCLKHS_EN] = { 0x0000, 11, 11 }, > + [GATING_EN] = { 0x0000, 10, 10 }, > + [IPI_SHUTDN] = { 0x0000, 9, 9 }, > + [IPI_COLORM] = { 0x0000, 8, 8 }, > + [IPI_COLOR_DEPTH] = { 0x0000, 4, 7 }, > + [IPI_FORMAT] = { 0x0000, 0, 3 }, > +}; > + > +static const struct dsigrf_reg rk3588_dsi1_grf_reg_fields[MAX_FIELDS] = { > + [TXREQCLKHS_EN] = { 0x0004, 11, 11 }, > + [GATING_EN] = { 0x0004, 10, 10 }, > + [IPI_SHUTDN] = { 0x0004, 9, 9 }, > + [IPI_COLORM] = { 0x0004, 8, 8 }, > + [IPI_COLOR_DEPTH] = { 0x0004, 4, 7 }, > + [IPI_FORMAT] = { 0x0004, 0, 3 }, > +}; > + > +static const struct rockchip_dw_dsi2_chip_data rk3588_chip_data[] = { > + { > + .reg = 0xfde20000, > + .grf_regs = rk3588_dsi0_grf_reg_fields, > + .max_bit_rate_per_lane = 4500000ULL, > + }, > + { > + .reg = 0xfde30000, > + .grf_regs = rk3588_dsi1_grf_reg_fields, > + .max_bit_rate_per_lane = 4500000ULL, > + } > +}; > + > +static const struct of_device_id dw_mipi_dsi2_rockchip_dt_ids[] = { > + { > + .compatible = "rockchip,rk3588-mipi-dsi2", > + .data = &rk3588_chip_data, > + }, > + {} > +}; > +MODULE_DEVICE_TABLE(of, dw_mipi_dsi2_rockchip_dt_ids); > + > +struct platform_driver dw_mipi_dsi2_rockchip_driver = { > + .probe = dw_mipi_dsi2_rockchip_probe, > + .remove = dw_mipi_dsi2_rockchip_remove, > + .driver = { > + .of_match_table = dw_mipi_dsi2_rockchip_dt_ids, > + .name = "dw-mipi-dsi2", > + }, > +}; > diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_drv.c b/drivers/gpu/drm/rockchip/rockchip_drm_drv.c > index ddf0be331c0a..5327ce035003 100644 > --- a/drivers/gpu/drm/rockchip/rockchip_drm_drv.c > +++ b/drivers/gpu/drm/rockchip/rockchip_drm_drv.c > @@ -511,6 +511,8 @@ static int __init rockchip_drm_init(void) > CONFIG_ROCKCHIP_DW_HDMI_QP); > ADD_ROCKCHIP_SUB_DRIVER(dw_mipi_dsi_rockchip_driver, > CONFIG_ROCKCHIP_DW_MIPI_DSI); > + ADD_ROCKCHIP_SUB_DRIVER(dw_mipi_dsi2_rockchip_driver, > + CONFIG_ROCKCHIP_DW_MIPI_DSI2); > ADD_ROCKCHIP_SUB_DRIVER(inno_hdmi_driver, CONFIG_ROCKCHIP_INNO_HDMI); > ADD_ROCKCHIP_SUB_DRIVER(rk3066_hdmi_driver, > CONFIG_ROCKCHIP_RK3066_HDMI); > diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_drv.h b/drivers/gpu/drm/rockchip/rockchip_drm_drv.h > index 24b4ce5ceaf1..9c9d38a06cdf 100644 > --- a/drivers/gpu/drm/rockchip/rockchip_drm_drv.h > +++ b/drivers/gpu/drm/rockchip/rockchip_drm_drv.h > @@ -90,6 +90,7 @@ extern struct platform_driver cdn_dp_driver; > extern struct platform_driver dw_hdmi_rockchip_pltfm_driver; > extern struct platform_driver dw_hdmi_qp_rockchip_pltfm_driver; > extern struct platform_driver dw_mipi_dsi_rockchip_driver; > +extern struct platform_driver dw_mipi_dsi2_rockchip_driver; > extern struct platform_driver inno_hdmi_driver; > extern struct platform_driver rockchip_dp_driver; > extern struct platform_driver rockchip_lvds_driver; ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v4 0/3] drm/rockchip: Add driver for the new DSI2 controller 2024-12-09 23:10 [PATCH v4 0/3] drm/rockchip: Add driver for the new DSI2 controller Heiko Stuebner ` (2 preceding siblings ...) 2024-12-09 23:10 ` [PATCH v4 3/3] drm/rockchip: Add MIPI DSI2 glue driver for RK3588 Heiko Stuebner @ 2024-12-10 23:12 ` Heiko Stuebner 3 siblings, 0 replies; 10+ messages in thread From: Heiko Stuebner @ 2024-12-10 23:12 UTC (permalink / raw) To: Heiko Stuebner Cc: andy.yan, maarten.lankhorst, mripard, tzimmermann, robh, krzk+dt, conor+dt, andrzej.hajda, neil.armstrong, rfoss, Laurent.pinchart, jonas, jernej.skrabec, dri-devel, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel, quentin.schulz On Tue, 10 Dec 2024 00:10:18 +0100, Heiko Stuebner wrote: > This series adds a bridge and glue driver for the DSI2 controller found > in the rk3588 soc from Rockchip, that is based on a Synopsis IP block. > > As the manual states: > The Display Serial Interface 2 (DSI-2) is part of a group of communication > protocols defined by the MIPI Alliance. The MIPI DSI-2 Host Controller is > a digital core that implements all protocol functions defined in the > MIPI DSI-2 Specification. > > [...] Applied, thanks! [1/3] drm/bridge/synopsys: Add MIPI DSI2 host controller bridge commit: 0d6d86253fef1e6b1e38a54db14bcbea9d0d9ca4 [2/3] dt-bindings: display: rockchip: Add schema for RK3588 DW DSI2 controller commit: 77889f2baadc856a26eef4ed601e5e277d0518b5 [3/3] drm/rockchip: Add MIPI DSI2 glue driver for RK3588 commit: 9f1e1e14f59de8e5a62226840abecbcdbd50221a Best regards, -- Heiko Stuebner <heiko@sntech.de> ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2025-02-02 20:36 UTC | newest] Thread overview: 10+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2024-12-09 23:10 [PATCH v4 0/3] drm/rockchip: Add driver for the new DSI2 controller Heiko Stuebner 2024-12-09 23:10 ` [PATCH v4 1/3] drm/bridge/synopsys: Add MIPI DSI2 host controller bridge Heiko Stuebner 2024-12-10 8:50 ` Andy Yan 2024-12-30 2:20 ` Chris Hofstaedtler 2025-01-14 17:30 ` Heiko Stübner 2025-02-02 20:34 ` Pavel Golikov 2024-12-09 23:10 ` [PATCH v4 2/3] dt-bindings: display: rockchip: Add schema for RK3588 DW DSI2 controller Heiko Stuebner 2024-12-09 23:10 ` [PATCH v4 3/3] drm/rockchip: Add MIPI DSI2 glue driver for RK3588 Heiko Stuebner 2024-12-10 8:51 ` Andy Yan 2024-12-10 23:12 ` [PATCH v4 0/3] drm/rockchip: Add driver for the new DSI2 controller Heiko Stuebner
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).
