Re: [PATCH 00/49] DRM driver for Hikey 970

From: Sam Ravnborg
Date: Tue Aug 25 2020 - 14:12:21 EST

Next message: kernel test robot: "drivers/gpu/drm/exynos/exynos_drm_dma.c:146:19: warning: variable 'mapping' is uninitialized when used here"
Previous message: Vineet Gupta: "[PATCH v2] irqchip/eznps: Fix build error for !ARC700 builds"
In reply to: Sam Ravnborg: "Re: [PATCH 00/49] DRM driver for Hikey 970"
Next in thread: Sam Ravnborg: "Re: [PATCH 00/49] DRM driver for Hikey 970"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

Hi Mauro

> Before posting the big patch series again, let me send the new
> version folded into a single patch.
>
> If you'd like to see the entire thing, I'm placing it here:
>
> https://gitlab.freedesktop.org/mchehab_kernel/hikey-970/-/commits/hikey970_v2/

Review 3/3

For next submission then to ease review it would be nice that the patch
is spilt up a little. Maybe something like:
- HW specific stuff in two patches (header fiels with register + setup
code)
- Display driver files
- DSI driver files
- build stuff (Makefile, Kconfig fragments)
So all splits on file level - which should be easy to do.

This will make it easier for more people to give feedback. It is a bit
much to walk through 4k loc or what the full size is.
And then we can also provide a-b or r-b for fragments so the reviewed
parts can be ignored for later reviews.

For the DSI parts I may be hit by the old "when you have a hammer then
everything looks like a nail syndrome".
In my case the hammer is the bridge interface.

Suggestions:
- Move encoder to display driver
- Move connector creation to display driver (using
drm_bridge_connector_init())
- Use drm_bridge interface for the DSI driver parts
- Maybe use the HDMI bridge driver as a chained driver - I did not look
to close on this
- Use the standard bridge interface to find the bridge and drop use of
the component framework

I hope that some other drm people chime in here.
Either to tell this is non-sense or confirm this is the way to go.

The above does not give any hint how to use the gpio_mux to shift
between the panel and HDMI. This logic needs to be in the display driver
as the bridge driver will only be used for HDMI - as I understand the
code. And I do not know how to do it.

Some details in the following that are not related to the above.

Sam

> diff --git a/drivers/staging/hikey9xx/gpu/kirin9xx_dw_drm_dsi.c b/drivers/staging/hikey9xx/gpu/kirin9xx_dw_drm_dsi.c
> new file mode 100644
> index 000000000000..a2eed961b7d5
> --- /dev/null
> +++ b/drivers/staging/hikey9xx/gpu/kirin9xx_dw_drm_dsi.c
> @@ -0,0 +1,2126 @@
> +// SPDX-License-Identifier: GPL-2.0
> +/*
> + * DesignWare MIPI DSI Host Controller v1.02 driver
> + *
> + * Copyright (c) 2016 Linaro Limited.
> + * Copyright (c) 2014-2016 Hisilicon Limited.
> + * Copyright (c) 2014-2020, Huawei Technologies Co., Ltd
> + *
> + * Author:
> + * <shizongxuan@xxxxxxxxxx>
> + * <zhangxiubin@xxxxxxxxxx>
> + */
> +
> +#include <linux/clk.h>
> +#include <linux/component.h>
> +#include <linux/of_device.h>
> +#include <linux/of_graph.h>
> +#include <linux/iopoll.h>
> +#include <video/mipi_display.h>
> +#include <linux/gpio/consumer.h>
> +#include <linux/of_address.h>
Sort

> +
> +#include <drm/drm_of.h>
> +#include <drm/drm_crtc_helper.h>
> +#include <drm/drm_mipi_dsi.h>
> +#include <drm/drm_encoder.h>
> +#include <drm/drm_device.h>
> +#include <drm/drm_atomic_helper.h>
> +#include <drm/drm_panel.h>
> +#include <drm/drm_probe_helper.h>
Sort

> +
> +#include "kirin9xx_dw_dsi_reg.h"
> +#include "kirin9xx_dpe.h"
> +#include "kirin9xx_drm_drv.h"
> +#include "kirin9xx_drm_dpe_utils.h"
Sort

> +
> +#define PHY_REF_CLK_RATE 19200000
> +#define PHY_REF_CLK_PERIOD_PS (1000000000 / (PHY_REF_CLK_RATE / 1000))
> +
> +#define encoder_to_dsi(encoder) \
> + container_of(encoder, struct dw_dsi, encoder)
> +#define host_to_dsi(host) \
> + container_of(host, struct dw_dsi, host)
> +#define connector_to_dsi(connector) \
> + container_of(connector, struct dw_dsi, connector)
Move the upcast next to the struct definition.

> +#define DSS_REDUCE(x) ((x) > 0 ? ((x) - 1) : (x))
> +
> +enum dsi_output_client {
> + OUT_HDMI = 0,
> + OUT_PANEL,
> + OUT_MAX
> +};
> +
> +struct mipi_phy_params {
I did not check all fiels but I managed to find at least one unused
field. Cheack and delete what is unused.
> + u64 lane_byte_clk;
> + u32 clk_division;
> +
> + u32 clk_lane_lp2hs_time;
> + u32 clk_lane_hs2lp_time;
> + u32 data_lane_lp2hs_time;
> + u32 data_lane_hs2lp_time;
> + u32 clk2data_delay;
> + u32 data2clk_delay;
> +
> + u32 clk_pre_delay;
> + u32 clk_post_delay;
> + u32 clk_t_lpx;
> + u32 clk_t_hs_prepare;
> + u32 clk_t_hs_zero;
> + u32 clk_t_hs_trial;
> + u32 clk_t_wakeup;
> + u32 data_pre_delay;
> + u32 data_post_delay;
> + u32 data_t_lpx;
> + u32 data_t_hs_prepare;
> + u32 data_t_hs_zero;
> + u32 data_t_hs_trial;
> + u32 data_t_ta_go;
> + u32 data_t_ta_get;
> + u32 data_t_wakeup;
> +
> + u32 phy_stop_wait_time;
> +
> + u32 rg_vrefsel_vcm;
> + u32 rg_hstx_ckg_sel;
> + u32 rg_pll_fbd_div5f;
> + u32 rg_pll_fbd_div1f;
> + u32 rg_pll_fbd_2p;
> + u32 rg_pll_enbwt;
> + u32 rg_pll_fbd_p;
> + u32 rg_pll_fbd_s;
> + u32 rg_pll_pre_div1p;
> + u32 rg_pll_pre_p;
> + u32 rg_pll_vco_750m;
> + u32 rg_pll_lpf_rs;
> + u32 rg_pll_lpf_cs;
> + u32 rg_pll_enswc;
> + u32 rg_pll_chp;
> +
Some indent gone wrong here?
> + u32 pll_register_override; /* 0x1E[0] */
> + u32 pll_power_down; /* 0x1E[1] */
> + u32 rg_band_sel; /* 0x1E[2] */
> + u32 rg_phase_gen_en; /* 0x1E[3] */
> + u32 reload_sel; /* 0x1E[4] */
> + u32 rg_pll_cp_p; /* 0x1E[7:5] */
> + u32 rg_pll_refsel; /* 0x16[1:0] */
> + u32 rg_pll_cp; /* 0x16[7:5] */
> + u32 load_command;
> +
> + /* for CDPHY */
> + u32 rg_cphy_div; /* Q */
> + u32 rg_div; /* M 0x4A[7:0] */
> + u32 rg_pre_div; /* N 0x49[0] */
> + u32 rg_320m; /* 0x48[2] */
> + u32 rg_2p5g; /* 0x48[1] */
> + u32 rg_0p8v; /* 0x48[0] */
> + u32 rg_lpf_r; /* 0x46[5:4] */
> + u32 rg_cp; /* 0x46[3:0] */
> + u32 t_prepare;
> + u32 t_lpx;
> + u32 t_prebegin;
> + u32 t_post;
> +};
> +
> +struct dsi_hw_ctx {
> + void __iomem *base;
> + char __iomem *peri_crg_base;
> + void __iomem *pctrl_base;
> +
> + u32 g_dss_version_tag;
> +
> + struct clk *dss_dphy0_ref_clk;
> + struct clk *dss_dphy1_ref_clk;
Not used, delete.
> + struct clk *dss_dphy0_cfg_clk;
> + struct clk *dss_dphy1_cfg_clk;
Not used, delete.

> + struct clk *dss_pclk_dsi0_clk;
> + struct clk *dss_pclk_dsi1_clk;
Not sued, delete.
> +};
> +
> +struct dw_dsi_client {
> + u32 lanes;
> + u32 phy_clock; /* in kHz */
> + enum mipi_dsi_pixel_format format;
> + unsigned long mode_flags;
> +};
> +
> +struct mipi_panel_info {
Some filed in the following are either not used or in some cases only
read. Clean it up.

Example: clk_t_hs_exit_adjust is assigned 0 and only read.
Looks like stuff to drop.

> + u8 dsi_version;
> + u8 vc;
> + u8 lane_nums;
> + u8 lane_nums_select_support;
> + u8 color_mode;
> + u32 dsi_bit_clk; /* clock lane(p/n) */
> + u32 burst_mode;
> + u32 max_tx_esc_clk;
> + u8 non_continue_en;
> +
> + u32 dsi_bit_clk_val1;
> + u32 dsi_bit_clk_val2;
> + u32 dsi_bit_clk_val3;
> + u32 dsi_bit_clk_val4;
> + u32 dsi_bit_clk_val5;
> + u32 dsi_bit_clk_upt;
> + /* uint32_t dsi_pclk_rate; */
> +
> + u32 hs_wr_to_time;
> +
> + /* dphy config parameter adjust */
> + u32 clk_post_adjust;
> + u32 clk_pre_adjust;
> + u32 clk_pre_delay_adjust;
> + u32 clk_t_hs_exit_adjust;
> + u32 clk_t_hs_trial_adjust;
> + u32 clk_t_hs_prepare_adjust;
> + int clk_t_lpx_adjust;
> + u32 clk_t_hs_zero_adjust;
> + u32 data_post_delay_adjust;
> + int data_t_lpx_adjust;
> + u32 data_t_hs_prepare_adjust;
> + u32 data_t_hs_zero_adjust;
> + u32 data_t_hs_trial_adjust;
> + u32 rg_vrefsel_vcm_adjust;
> +
> + /* only for Chicago<3660> use */
> + u32 rg_vrefsel_vcm_clk_adjust;
> + u32 rg_vrefsel_vcm_data_adjust;
> +
> + u32 phy_mode; /* 0: DPHY, 1:CPHY */
> + u32 lp11_flag;
> +};
> +
> +struct ldi_panel_info {
Fileds are only partially used.

> + u32 h_back_porch;
> + u32 h_front_porch;
> + u32 h_pulse_width;
> +
> + /*
> + * note: vbp > 8 if used overlay compose,
> + * also lcd vbp > 8 in lcd power on sequence
> + */
> + u32 v_back_porch;
> + u32 v_front_porch;
> + u32 v_pulse_width;
> +
> + u8 hsync_plr;
> + u8 vsync_plr;
> + u8 pixelclk_plr;
> + u8 data_en_plr;
> +
> + /* for cabc */
> + u8 dpi0_overlap_size;
> + u8 dpi1_overlap_size;
> +};
> +
> +struct dw_dsi {
> + struct drm_encoder encoder;
> + struct drm_bridge *bridge;
> + struct drm_panel *panel;
> + struct mipi_dsi_host host;
> + struct drm_connector connector; /* connector for panel */
> + struct drm_display_mode cur_mode;
> + struct dsi_hw_ctx *ctx;
Would it be possible to embed dsi_hw_ctx?
So everything is allocated in one go.

> + struct mipi_phy_params phy;
> + struct mipi_panel_info mipi;
> + struct ldi_panel_info ldi;
> + u32 lanes;
> + enum mipi_dsi_pixel_format format;
> + unsigned long mode_flags;
> + struct gpio_desc *gpio_mux;
> + struct dw_dsi_client client[OUT_MAX];
> + enum dsi_output_client cur_client, attached_client;
> + bool enable;
> +};
> +
> +struct dsi_data {
> + struct dw_dsi dsi;
> + struct dsi_hw_ctx ctx;
> +};
If dsi_hw_ctx is embedded in dw_dsi then this struct can be dropped.
> +
> +struct dsi_phy_range {
> + u32 min_range_kHz;
> + u32 max_range_kHz;
> + u32 pll_vco_750M;
> + u32 hstx_ckg_sel;
> +};
Not used, delete.
> +
> +static const struct dsi_phy_range dphy_range_info[] = {
> + { 46875, 62500, 1, 7 },
> + { 62500, 93750, 0, 7 },
> + { 93750, 125000, 1, 6 },
> + { 125000, 187500, 0, 6 },
> + { 187500, 250000, 1, 5 },
> + { 250000, 375000, 0, 5 },
> + { 375000, 500000, 1, 4 },
> + { 500000, 750000, 0, 4 },
> + { 750000, 1000000, 1, 0 },
> + { 1000000, 1500000, 0, 0 }
> +};
Not used, delete.

> +
> +/*
> + * Except for debug, this is identical to the one defined at
> + * kirin9xx_drm_dpe_utils.h.
> + */
> +
> +void dsi_set_output_client(struct drm_device *dev)
> +{
> + struct drm_connector_list_iter conn_iter;
> + struct drm_connector *connector;
> + enum dsi_output_client client;
> + struct drm_encoder *encoder;
> + struct dw_dsi *dsi;
> +
> + mutex_lock(&dev->mode_config.mutex);
> +
> + /* find dsi encoder */
> + drm_for_each_encoder(encoder, dev)
> + if (encoder->encoder_type == DRM_MODE_ENCODER_DSI)
> + break;
> + dsi = encoder_to_dsi(encoder);
> +
> + /* find HDMI connector */
> + drm_connector_list_iter_begin(dev, &conn_iter);
> + drm_for_each_connector_iter(connector, &conn_iter)
> + if (connector->connector_type == DRM_MODE_CONNECTOR_HDMIA)
> + break;
> +
> + /*
> + * set the proper dsi output client
> + */
> + client = connector->status == connector_status_connected ?
> + OUT_HDMI : OUT_PANEL;
> +
> + if (client != dsi->cur_client) {
> + gpiod_set_value_cansleep(dsi->gpio_mux, client);
> + dsi->cur_client = client;
> +
> + msleep(20);
> +
> + drm_info(dev, "client change to %s\n",
> + client == OUT_HDMI ? "HDMI" : "panel");
> + }
> +
> + mutex_unlock(&dev->mode_config.mutex);
> +}
> +EXPORT_SYMBOL(dsi_set_output_client);
> +
> +static void kirin970_get_dsi_dphy_ctrl(struct dw_dsi *dsi,
> + struct mipi_phy_params *phy_ctrl, u32 id)
> +{
> + struct mipi_panel_info *mipi = NULL;
> + struct drm_display_mode *mode = NULL;
> + u32 dphy_req_kHz;
> + int bpp;
> + u32 ui = 0;
> + u32 m_pll = 0;
> + u32 n_pll = 0;
> + u64 lane_clock = 0;
> + u64 vco_div = 1;
> + u32 m_n_fract = 0;
> + u32 m_n_int = 0;
> +
> + u32 accuracy = 0;
> + u32 unit_tx_byte_clk_hs = 0;
> + u32 clk_post = 0;
> + u32 clk_pre = 0;
> + u32 clk_t_hs_exit = 0;
> + u32 clk_pre_delay = 0;
> + u32 clk_t_hs_prepare = 0;
> + u32 clk_t_lpx = 0;
> + u32 clk_t_hs_zero = 0;
> + u32 clk_t_hs_trial = 0;
> + u32 data_post_delay = 0;
> + u32 data_t_hs_prepare = 0;
> + u32 data_t_hs_zero = 0;
> + u32 data_t_hs_trial = 0;
> + u32 data_t_lpx = 0;
> +
> + WARN_ON(!phy_ctrl);
> + WARN_ON(!dsi);
> +
> + mode = &dsi->cur_mode;
> + mipi = &dsi->mipi;
> +
> + /*
> + * count phy params
> + */
> + bpp = mipi_dsi_pixel_format_to_bpp(dsi->client[id].format);
> + if (bpp < 0)
> + return;
> +
> + if (mode->clock > 80000)
> + dsi->client[id].lanes = 4;
> + else
> + dsi->client[id].lanes = 3;
> +
> + if (dsi->client[id].phy_clock)
> + dphy_req_kHz = dsi->client[id].phy_clock;
> + else
> + dphy_req_kHz = mode->clock * bpp / dsi->client[id].lanes;
> +
> + lane_clock = dphy_req_kHz / 1000;
> + DRM_DEBUG("Expected : lane_clock = %llu M\n", lane_clock);
> +
> + /************************ PLL parameters config *********************/
> + /* chip spec : */
> + /* If the output data rate is below 320 Mbps, RG_BNAD_SEL should be set to 1. */
> + /* At this mode a post divider of 1/4 will be applied to VCO. */
> + if ((lane_clock >= 320) && (lane_clock <= 2500)) {
> + phy_ctrl->rg_band_sel = 0;
> + vco_div = 1;
> + } else if ((lane_clock >= 80) && (lane_clock < 320)) {
> + phy_ctrl->rg_band_sel = 1;
> + vco_div = 4;
> + } else {
> + DRM_ERROR("80M <= lane_clock< = 2500M, not support lane_clock = %llu M.\n",
> + lane_clock);
> + }
> +
> + m_n_int = lane_clock * vco_div * 1000000UL / DEFAULT_MIPI_CLK_RATE;
> + m_n_fract = ((lane_clock * vco_div * 1000000UL * 1000UL / DEFAULT_MIPI_CLK_RATE) % 1000) * 10 / 1000;
> +
> + n_pll = 2;
> +
> + m_pll = (u32)(lane_clock * vco_div * n_pll * 1000000UL / DEFAULT_MIPI_CLK_RATE);
> +
> + lane_clock = m_pll * (DEFAULT_MIPI_CLK_RATE / n_pll) / vco_div;
> + if (lane_clock > 750000000)
> + phy_ctrl->rg_cp = 3;
> + else if ((lane_clock >= 80000000) && (lane_clock <= 750000000))
> + phy_ctrl->rg_cp = 1;
> + else
> + DRM_ERROR("80M <= lane_clock< = 2500M, not support lane_clock = %llu M.\n",
> + lane_clock);
> +
> + /* chip spec : */
> + phy_ctrl->rg_pre_div = n_pll - 1;
> + phy_ctrl->rg_div = m_pll;
> + phy_ctrl->rg_0p8v = 0;
> + phy_ctrl->rg_2p5g = 1;
> + phy_ctrl->rg_320m = 0;
> + phy_ctrl->rg_lpf_r = 0;
> +
> + /* TO DO HSTX select VCM VREF */
> + phy_ctrl->rg_vrefsel_vcm = 0x5d;
> +
> + /******************** clock/data lane parameters config ******************/
> + accuracy = 10;
> + ui = (u32)(10 * 1000000000UL * accuracy / lane_clock);
> + /* unit of measurement */
> + unit_tx_byte_clk_hs = 8 * ui;
> +
> + /* D-PHY Specification : 60ns + 52*UI <= clk_post */
> + clk_post = 600 * accuracy + 52 * ui + unit_tx_byte_clk_hs + mipi->clk_post_adjust * ui;
> +
> + /* D-PHY Specification : clk_pre >= 8*UI */
> + clk_pre = 8 * ui + unit_tx_byte_clk_hs + mipi->clk_pre_adjust * ui;
> +
> + /* D-PHY Specification : clk_t_hs_exit >= 100ns */
> + clk_t_hs_exit = 1000 * accuracy + 100 * accuracy + mipi->clk_t_hs_exit_adjust * ui;
> +
> + /* clocked by TXBYTECLKHS */
> + clk_pre_delay = 0 + mipi->clk_pre_delay_adjust * ui;
> +
> + /* D-PHY Specification : clk_t_hs_trial >= 60ns */
> + /* clocked by TXBYTECLKHS */
> + clk_t_hs_trial = 600 * accuracy + 3 * unit_tx_byte_clk_hs + mipi->clk_t_hs_trial_adjust * ui;
> +
> + /* D-PHY Specification : 38ns <= clk_t_hs_prepare <= 95ns */
> + /* clocked by TXBYTECLKHS */
> + clk_t_hs_prepare = 660 * accuracy;
> +
> + /* clocked by TXBYTECLKHS */
> + data_post_delay = 0 + mipi->data_post_delay_adjust * ui;
> +
> + /* D-PHY Specification : data_t_hs_trial >= max( n*8*UI, 60ns + n*4*UI ), n = 1 */
> + /* clocked by TXBYTECLKHS */
> + data_t_hs_trial = ((600 * accuracy + 4 * ui) >= (8 * ui) ? (600 * accuracy + 4 * ui) : (8 * ui)) +
> + 2 * unit_tx_byte_clk_hs + mipi->data_t_hs_trial_adjust * ui;
> +
> + /* D-PHY Specification : 40ns + 4*UI <= data_t_hs_prepare <= 85ns + 6*UI */
> + /* clocked by TXBYTECLKHS */
> + data_t_hs_prepare = 400 * accuracy + 4 * ui;
> + /* D-PHY chip spec : clk_t_lpx + clk_t_hs_prepare > 200ns */
> + /* D-PHY Specification : clk_t_lpx >= 50ns */
> + /* clocked by TXBYTECLKHS */
> + clk_t_lpx = (uint32_t)(2000 * accuracy + 10 * accuracy + mipi->clk_t_lpx_adjust * ui - clk_t_hs_prepare);
> +
> + /* D-PHY Specification : clk_t_hs_zero + clk_t_hs_prepare >= 300 ns */
> + /* clocked by TXBYTECLKHS */
> + clk_t_hs_zero = (uint32_t)(3000 * accuracy + 3 * unit_tx_byte_clk_hs + mipi->clk_t_hs_zero_adjust * ui - clk_t_hs_prepare);
> +
> + /* D-PHY chip spec : data_t_lpx + data_t_hs_prepare > 200ns */
> + /* D-PHY Specification : data_t_lpx >= 50ns */
> + /* clocked by TXBYTECLKHS */
> + data_t_lpx = (uint32_t)(2000 * accuracy + 10 * accuracy + mipi->data_t_lpx_adjust * ui - data_t_hs_prepare);
> +
> + /* D-PHY Specification : data_t_hs_zero + data_t_hs_prepare >= 145ns + 10*UI */
> + /* clocked by TXBYTECLKHS */
> + data_t_hs_zero = (uint32_t)(1450 * accuracy + 10 * ui +
> + 3 * unit_tx_byte_clk_hs + mipi->data_t_hs_zero_adjust * ui - data_t_hs_prepare);
> +
> + phy_ctrl->clk_pre_delay = DIV_ROUND_UP(clk_pre_delay, unit_tx_byte_clk_hs);
> + phy_ctrl->clk_t_hs_prepare = DIV_ROUND_UP(clk_t_hs_prepare, unit_tx_byte_clk_hs);
> + phy_ctrl->clk_t_lpx = DIV_ROUND_UP(clk_t_lpx, unit_tx_byte_clk_hs);
> + phy_ctrl->clk_t_hs_zero = DIV_ROUND_UP(clk_t_hs_zero, unit_tx_byte_clk_hs);
> + phy_ctrl->clk_t_hs_trial = DIV_ROUND_UP(clk_t_hs_trial, unit_tx_byte_clk_hs);
> +
> + phy_ctrl->data_post_delay = DIV_ROUND_UP(data_post_delay, unit_tx_byte_clk_hs);
> + phy_ctrl->data_t_hs_prepare = DIV_ROUND_UP(data_t_hs_prepare, unit_tx_byte_clk_hs);
> + phy_ctrl->data_t_lpx = DIV_ROUND_UP(data_t_lpx, unit_tx_byte_clk_hs);
> + phy_ctrl->data_t_hs_zero = DIV_ROUND_UP(data_t_hs_zero, unit_tx_byte_clk_hs);
> + phy_ctrl->data_t_hs_trial = DIV_ROUND_UP(data_t_hs_trial, unit_tx_byte_clk_hs);
> +
> + phy_ctrl->clk_post_delay = phy_ctrl->data_t_hs_trial + DIV_ROUND_UP(clk_post, unit_tx_byte_clk_hs);
> + phy_ctrl->data_pre_delay = phy_ctrl->clk_pre_delay + 2 + phy_ctrl->clk_t_lpx +
> + phy_ctrl->clk_t_hs_prepare + phy_ctrl->clk_t_hs_zero + 8 + DIV_ROUND_UP(clk_pre, unit_tx_byte_clk_hs);
> +
> + phy_ctrl->clk_lane_lp2hs_time = phy_ctrl->clk_pre_delay + phy_ctrl->clk_t_lpx + phy_ctrl->clk_t_hs_prepare +
> + phy_ctrl->clk_t_hs_zero + 5 + 7;
> + phy_ctrl->clk_lane_hs2lp_time = phy_ctrl->clk_t_hs_trial + phy_ctrl->clk_post_delay + 8 + 4;
> + phy_ctrl->data_lane_lp2hs_time = phy_ctrl->data_pre_delay + phy_ctrl->data_t_lpx + phy_ctrl->data_t_hs_prepare +
> + phy_ctrl->data_t_hs_zero + 5 + 7;
> + phy_ctrl->data_lane_hs2lp_time = phy_ctrl->data_t_hs_trial + 8 + 5;
> +
> + phy_ctrl->phy_stop_wait_time = phy_ctrl->clk_post_delay + 4 + phy_ctrl->clk_t_hs_trial +
> + DIV_ROUND_UP(clk_t_hs_exit, unit_tx_byte_clk_hs) - (phy_ctrl->data_post_delay + 4 + phy_ctrl->data_t_hs_trial) + 3;
> +
> + phy_ctrl->lane_byte_clk = lane_clock / 8;
> + phy_ctrl->clk_division = (((phy_ctrl->lane_byte_clk / 2) % mipi->max_tx_esc_clk) > 0) ?
> + (uint32_t)(phy_ctrl->lane_byte_clk / 2 / mipi->max_tx_esc_clk + 1) :
> + (uint32_t)(phy_ctrl->lane_byte_clk / 2 / mipi->max_tx_esc_clk);
> +
> + DRM_DEBUG("DPHY clock_lane and data_lane config :\n"
> + "lane_clock = %llu, n_pll=%d, m_pll=%d\n"
> + "rg_cp=%d\n"
> + "rg_band_sel=%d\n"
> + "rg_vrefsel_vcm=%d\n"
> + "clk_pre_delay=%d\n"
> + "clk_post_delay=%d\n"
> + "clk_t_hs_prepare=%d\n"
> + "clk_t_lpx=%d\n"
> + "clk_t_hs_zero=%d\n"
> + "clk_t_hs_trial=%d\n"
> + "data_pre_delay=%d\n"
> + "data_post_delay=%d\n"
> + "data_t_hs_prepare=%d\n"
> + "data_t_lpx=%d\n"
> + "data_t_hs_zero=%d\n"
> + "data_t_hs_trial=%d\n"
> + "clk_lane_lp2hs_time=%d\n"
> + "clk_lane_hs2lp_time=%d\n"
> + "data_lane_lp2hs_time=%d\n"
> + "data_lane_hs2lp_time=%d\n"
> + "phy_stop_wait_time=%d\n",
> + lane_clock, n_pll, m_pll,
> + phy_ctrl->rg_cp,
> + phy_ctrl->rg_band_sel,
> + phy_ctrl->rg_vrefsel_vcm,
> + phy_ctrl->clk_pre_delay,
> + phy_ctrl->clk_post_delay,
> + phy_ctrl->clk_t_hs_prepare,
> + phy_ctrl->clk_t_lpx,
> + phy_ctrl->clk_t_hs_zero,
> + phy_ctrl->clk_t_hs_trial,
> + phy_ctrl->data_pre_delay,
> + phy_ctrl->data_post_delay,
> + phy_ctrl->data_t_hs_prepare,
> + phy_ctrl->data_t_lpx,
> + phy_ctrl->data_t_hs_zero,
> + phy_ctrl->data_t_hs_trial,
> + phy_ctrl->clk_lane_lp2hs_time,
> + phy_ctrl->clk_lane_hs2lp_time,
> + phy_ctrl->data_lane_lp2hs_time,
> + phy_ctrl->data_lane_hs2lp_time,
> + phy_ctrl->phy_stop_wait_time);
> +}
> +
> +static void kirin960_get_dsi_phy_ctrl(struct dw_dsi *dsi,
> + struct mipi_phy_params *phy_ctrl, u32 id)
> +{
> + struct mipi_panel_info *mipi = NULL;
> + struct drm_display_mode *mode = NULL;
> + u32 dphy_req_kHz;
> + int bpp;
> + u32 ui = 0;
> + u32 m_pll = 0;
> + u32 n_pll = 0;
> + u32 m_n_fract = 0;
> + u32 m_n_int = 0;
> + u64 lane_clock = 0;
> + u64 vco_div = 1;
> +
> + u32 accuracy = 0;
> + u32 unit_tx_byte_clk_hs = 0;
> + u32 clk_post = 0;
> + u32 clk_pre = 0;
> + u32 clk_t_hs_exit = 0;
> + u32 clk_pre_delay = 0;
> + u32 clk_t_hs_prepare = 0;
> + u32 clk_t_lpx = 0;
> + u32 clk_t_hs_zero = 0;
> + u32 clk_t_hs_trial = 0;
> + u32 data_post_delay = 0;
> + u32 data_t_hs_prepare = 0;
> + u32 data_t_hs_zero = 0;
> + u32 data_t_hs_trial = 0;
> + u32 data_t_lpx = 0;
> + u32 clk_pre_delay_reality = 0;
> + u32 clk_t_hs_zero_reality = 0;
> + u32 clk_post_delay_reality = 0;
> + u32 data_t_hs_zero_reality = 0;
> + u32 data_post_delay_reality = 0;
> + u32 data_pre_delay_reality = 0;
> +
> + WARN_ON(!phy_ctrl);
> + WARN_ON(!dsi);
> +
> + mode = &dsi->cur_mode;
> + mipi = &dsi->mipi;
> +
> + /*
> + * count phy params
> + */
> + bpp = mipi_dsi_pixel_format_to_bpp(dsi->client[id].format);
> + if (bpp < 0)
> + return;
> +
> + if (mode->clock > 80000)
> + dsi->client[id].lanes = 4;
> + else
> + dsi->client[id].lanes = 3;
> +
> + if (dsi->client[id].phy_clock)
> + dphy_req_kHz = dsi->client[id].phy_clock;
> + else
> + dphy_req_kHz = mode->clock * bpp / dsi->client[id].lanes;
> +
> + lane_clock = dphy_req_kHz / 1000;
> + DRM_INFO("Expected : lane_clock = %llu M\n", lane_clock);
> +
> + /************************ PLL parameters config *********************/
> +
> + /*
> + * chip spec :
> + * If the output data rate is below 320 Mbps,
> + * RG_BNAD_SEL should be set to 1.
> + * At this mode a post divider of 1/4 will be applied to VCO.
> + */
> + if ((lane_clock >= 320) && (lane_clock <= 2500)) {
> + phy_ctrl->rg_band_sel = 0; /* 0x1E[2] */
> + vco_div = 1;
> + } else if ((lane_clock >= 80) && (lane_clock < 320)) {
> + phy_ctrl->rg_band_sel = 1;
> + vco_div = 4;
> + } else {
> + DRM_ERROR("80M <= lane_clock< = 2500M, not support lane_clock = %llu M\n",
> + lane_clock);
> + }
> +
> + m_n_int = lane_clock * vco_div * 1000000UL / DEFAULT_MIPI_CLK_RATE;
> + m_n_fract = ((lane_clock * vco_div * 1000000UL * 1000UL / DEFAULT_MIPI_CLK_RATE) % 1000) * 10 / 1000;
> +
> + if (m_n_int % 2 == 0) {
> + if (m_n_fract * 6 >= 50) {
> + n_pll = 2;
> + m_pll = (m_n_int + 1) * n_pll;
> + } else if (m_n_fract * 6 >= 30) {
> + n_pll = 3;
> + m_pll = m_n_int * n_pll + 2;
> + } else {
> + n_pll = 1;
> + m_pll = m_n_int * n_pll;
> + }
> + } else {
> + if (m_n_fract * 6 >= 50) {
> + n_pll = 1;
> + m_pll = (m_n_int + 1) * n_pll;
> + } else if (m_n_fract * 6 >= 30) {
> + n_pll = 1;
> + m_pll = (m_n_int + 1) * n_pll;
> + } else if (m_n_fract * 6 >= 10) {
> + n_pll = 3;
> + m_pll = m_n_int * n_pll + 1;
> + } else {
> + n_pll = 2;
> + m_pll = m_n_int * n_pll;
> + }
> + }
> +
> + /* if set rg_pll_enswc=1, rg_pll_fbd_s can't be 0 */
> + if (m_pll <= 8) {
> + phy_ctrl->rg_pll_fbd_s = 1;
> + phy_ctrl->rg_pll_enswc = 0;
> +
> + if (m_pll % 2 == 0) {
> + phy_ctrl->rg_pll_fbd_p = m_pll / 2;
> + } else {
> + if (n_pll == 1) {
> + n_pll *= 2;
> + phy_ctrl->rg_pll_fbd_p = (m_pll * 2) / 2;
> + } else {
> + DRM_ERROR("phy m_pll not support!m_pll = %d\n", m_pll);
> + return;
> + }
> + }
> + } else if (m_pll <= 300) {
> + if (m_pll % 2 == 0)
> + phy_ctrl->rg_pll_enswc = 0;
> + else
> + phy_ctrl->rg_pll_enswc = 1;
> +
> + phy_ctrl->rg_pll_fbd_s = 1;
> + phy_ctrl->rg_pll_fbd_p = m_pll / 2;
> + } else if (m_pll <= 315) {
> + phy_ctrl->rg_pll_fbd_p = 150;
> + phy_ctrl->rg_pll_fbd_s = m_pll - 2 * phy_ctrl->rg_pll_fbd_p;
> + phy_ctrl->rg_pll_enswc = 1;
> + } else {
> + DRM_ERROR("phy m_pll not support!m_pll = %d\n", m_pll);
> + return;
> + }
> +
> + phy_ctrl->rg_pll_pre_p = n_pll;
> +
> + lane_clock = m_pll * (DEFAULT_MIPI_CLK_RATE / n_pll) / vco_div;
> + DRM_INFO("Config : lane_clock = %llu\n", lane_clock);
> +
> + /* FIXME : */
> + phy_ctrl->rg_pll_cp = 1; /* 0x16[7:5] */
> + phy_ctrl->rg_pll_cp_p = 3; /* 0x1E[7:5] */
> +
> + /* test_code_0x14 other parameters config */
> + phy_ctrl->rg_pll_enbwt = 0; /* 0x14[2] */
> + phy_ctrl->rg_pll_chp = 0; /* 0x14[1:0] */
> +
> + /* test_code_0x16 other parameters config, 0x16[3:2] reserved */
> + phy_ctrl->rg_pll_lpf_cs = 0; /* 0x16[4] */
> + phy_ctrl->rg_pll_refsel = 1; /* 0x16[1:0] */
> +
> + /* test_code_0x1E other parameters config */
> + phy_ctrl->reload_sel = 1; /* 0x1E[4] */
> + phy_ctrl->rg_phase_gen_en = 1; /* 0x1E[3] */
> + phy_ctrl->pll_power_down = 0; /* 0x1E[1] */
> + phy_ctrl->pll_register_override = 1; /* 0x1E[0] */
> +
> + /* HSTX select VCM VREF */
> + phy_ctrl->rg_vrefsel_vcm = 0x55;
> + if (mipi->rg_vrefsel_vcm_clk_adjust != 0)
> + phy_ctrl->rg_vrefsel_vcm = (phy_ctrl->rg_vrefsel_vcm & 0x0F) |
> + ((mipi->rg_vrefsel_vcm_clk_adjust & 0x0F) << 4);
> +
> + if (mipi->rg_vrefsel_vcm_data_adjust != 0)
> + phy_ctrl->rg_vrefsel_vcm = (phy_ctrl->rg_vrefsel_vcm & 0xF0) |
> + (mipi->rg_vrefsel_vcm_data_adjust & 0x0F);
> +
> + /* if reload_sel = 1, need to set load_command */
> + phy_ctrl->load_command = 0x5A;
> +
> + /******************** clock/data lane parameters config ******************/
> + accuracy = 10;
> + ui = 10 * 1000000000UL * accuracy / lane_clock;
> + /* unit of measurement */
> + unit_tx_byte_clk_hs = 8 * ui;
> +
> + /* D-PHY Specification : 60ns + 52*UI <= clk_post */
> + clk_post = 600 * accuracy + 52 * ui + mipi->clk_post_adjust * ui;
> +
> + /* D-PHY Specification : clk_pre >= 8*UI */
> + clk_pre = 8 * ui + mipi->clk_pre_adjust * ui;
> +
> + /* D-PHY Specification : clk_t_hs_exit >= 100ns */
> + clk_t_hs_exit = 1000 * accuracy + mipi->clk_t_hs_exit_adjust * ui;
> +
> + /* clocked by TXBYTECLKHS */
> + clk_pre_delay = 0 + mipi->clk_pre_delay_adjust * ui;
> +
> + /* D-PHY Specification : clk_t_hs_trial >= 60ns */
> + /* clocked by TXBYTECLKHS */
> + clk_t_hs_trial = 600 * accuracy + 3 * unit_tx_byte_clk_hs + mipi->clk_t_hs_trial_adjust * ui;
> +
> + /* D-PHY Specification : 38ns <= clk_t_hs_prepare <= 95ns */
> + /* clocked by TXBYTECLKHS */
> + if (mipi->clk_t_hs_prepare_adjust == 0)
> + mipi->clk_t_hs_prepare_adjust = 43;
> +
> + clk_t_hs_prepare = ((380 * accuracy + mipi->clk_t_hs_prepare_adjust * ui) <= (950 * accuracy - 8 * ui)) ?
> + (380 * accuracy + mipi->clk_t_hs_prepare_adjust * ui) : (950 * accuracy - 8 * ui);
> +
> + /* clocked by TXBYTECLKHS */
> + data_post_delay = 0 + mipi->data_post_delay_adjust * ui;
> +
> + /* D-PHY Specification : data_t_hs_trial >= max( n*8*UI, 60ns + n*4*UI ), n = 1 */
> + /* clocked by TXBYTECLKHS */
> + data_t_hs_trial = ((600 * accuracy + 4 * ui) >= (8 * ui) ? (600 * accuracy + 4 * ui) : (8 * ui)) + 8 * ui +
> + 3 * unit_tx_byte_clk_hs + mipi->data_t_hs_trial_adjust * ui;
> +
> + /* D-PHY Specification : 40ns + 4*UI <= data_t_hs_prepare <= 85ns + 6*UI */
> + /* clocked by TXBYTECLKHS */
> + if (mipi->data_t_hs_prepare_adjust == 0)
> + mipi->data_t_hs_prepare_adjust = 35;
> +
> + data_t_hs_prepare = ((400 * accuracy + 4 * ui + mipi->data_t_hs_prepare_adjust * ui) <= (850 * accuracy + 6 * ui - 8 * ui)) ?
> + (400 * accuracy + 4 * ui + mipi->data_t_hs_prepare_adjust * ui) : (850 * accuracy + 6 * ui - 8 * ui);
> +
> + /* D-PHY chip spec : clk_t_lpx + clk_t_hs_prepare > 200ns */
> + /* D-PHY Specification : clk_t_lpx >= 50ns */
> + /* clocked by TXBYTECLKHS */
> + clk_t_lpx = (((2000 * accuracy - clk_t_hs_prepare) >= 500 * accuracy) ?
> + ((2000 * accuracy - clk_t_hs_prepare)) : (500 * accuracy)) +
> + mipi->clk_t_lpx_adjust * ui;
> +
> + /* D-PHY Specification : clk_t_hs_zero + clk_t_hs_prepare >= 300 ns */
> + /* clocked by TXBYTECLKHS */
> + clk_t_hs_zero = 3000 * accuracy - clk_t_hs_prepare + 3 * unit_tx_byte_clk_hs + mipi->clk_t_hs_zero_adjust * ui;
> +
> + /* D-PHY chip spec : data_t_lpx + data_t_hs_prepare > 200ns */
> + /* D-PHY Specification : data_t_lpx >= 50ns */
> + /* clocked by TXBYTECLKHS */
> + data_t_lpx = clk_t_lpx + mipi->data_t_lpx_adjust * ui; /* 2000 * accuracy - data_t_hs_prepare; */
> +
> + /* D-PHY Specification : data_t_hs_zero + data_t_hs_prepare >= 145ns + 10*UI */
> + /* clocked by TXBYTECLKHS */
> + data_t_hs_zero = 1450 * accuracy + 10 * ui - data_t_hs_prepare +
> + 3 * unit_tx_byte_clk_hs + mipi->data_t_hs_zero_adjust * ui;
> +
> + phy_ctrl->clk_pre_delay = DIV_ROUND_UP(clk_pre_delay, unit_tx_byte_clk_hs);
> + phy_ctrl->clk_t_hs_prepare = DIV_ROUND_UP(clk_t_hs_prepare, unit_tx_byte_clk_hs);
> + phy_ctrl->clk_t_lpx = DIV_ROUND_UP(clk_t_lpx, unit_tx_byte_clk_hs);
> + phy_ctrl->clk_t_hs_zero = DIV_ROUND_UP(clk_t_hs_zero, unit_tx_byte_clk_hs);
> + phy_ctrl->clk_t_hs_trial = DIV_ROUND_UP(clk_t_hs_trial, unit_tx_byte_clk_hs);
> +
> + phy_ctrl->data_post_delay = DIV_ROUND_UP(data_post_delay, unit_tx_byte_clk_hs);
> + phy_ctrl->data_t_hs_prepare = DIV_ROUND_UP(data_t_hs_prepare, unit_tx_byte_clk_hs);
> + phy_ctrl->data_t_lpx = DIV_ROUND_UP(data_t_lpx, unit_tx_byte_clk_hs);
> + phy_ctrl->data_t_hs_zero = DIV_ROUND_UP(data_t_hs_zero, unit_tx_byte_clk_hs);
> + phy_ctrl->data_t_hs_trial = DIV_ROUND_UP(data_t_hs_trial, unit_tx_byte_clk_hs);
> + phy_ctrl->data_t_ta_go = 4;
> + phy_ctrl->data_t_ta_get = 5;
> +
> + clk_pre_delay_reality = phy_ctrl->clk_pre_delay + 2;
> + clk_t_hs_zero_reality = phy_ctrl->clk_t_hs_zero + 8;
> + data_t_hs_zero_reality = phy_ctrl->data_t_hs_zero + 4;
> + data_post_delay_reality = phy_ctrl->data_post_delay + 4;
> +
> + phy_ctrl->clk_post_delay = phy_ctrl->data_t_hs_trial + DIV_ROUND_UP(clk_post, unit_tx_byte_clk_hs);
> + phy_ctrl->data_pre_delay = clk_pre_delay_reality + phy_ctrl->clk_t_lpx +
> + phy_ctrl->clk_t_hs_prepare + clk_t_hs_zero_reality + DIV_ROUND_UP(clk_pre, unit_tx_byte_clk_hs);
> +
> + clk_post_delay_reality = phy_ctrl->clk_post_delay + 4;
> + data_pre_delay_reality = phy_ctrl->data_pre_delay + 2;
> +
> + phy_ctrl->clk_lane_lp2hs_time = clk_pre_delay_reality + phy_ctrl->clk_t_lpx +
> + phy_ctrl->clk_t_hs_prepare + clk_t_hs_zero_reality + 3;
> + phy_ctrl->clk_lane_hs2lp_time = clk_post_delay_reality + phy_ctrl->clk_t_hs_trial + 3;
> + phy_ctrl->data_lane_lp2hs_time = data_pre_delay_reality + phy_ctrl->data_t_lpx +
> + phy_ctrl->data_t_hs_prepare + data_t_hs_zero_reality + 3;
> + phy_ctrl->data_lane_hs2lp_time = data_post_delay_reality + phy_ctrl->data_t_hs_trial + 3;
> + phy_ctrl->phy_stop_wait_time = clk_post_delay_reality +
> + phy_ctrl->clk_t_hs_trial + DIV_ROUND_UP(clk_t_hs_exit, unit_tx_byte_clk_hs) -
> + (data_post_delay_reality + phy_ctrl->data_t_hs_trial) + 3;
> +
> + phy_ctrl->lane_byte_clk = lane_clock / 8;
> + phy_ctrl->clk_division = (((phy_ctrl->lane_byte_clk / 2) % mipi->max_tx_esc_clk) > 0) ?
> + (phy_ctrl->lane_byte_clk / 2 / mipi->max_tx_esc_clk + 1) :
> + (phy_ctrl->lane_byte_clk / 2 / mipi->max_tx_esc_clk);
> +
> + DRM_DEBUG("PHY clock_lane and data_lane config :\n"
> + "rg_vrefsel_vcm=%u\n"
> + "clk_pre_delay=%u\n"
> + "clk_post_delay=%u\n"
> + "clk_t_hs_prepare=%u\n"
> + "clk_t_lpx=%u\n"
> + "clk_t_hs_zero=%u\n"
> + "clk_t_hs_trial=%u\n"
> + "data_pre_delay=%u\n"
> + "data_post_delay=%u\n"
> + "data_t_hs_prepare=%u\n"
> + "data_t_lpx=%u\n"
> + "data_t_hs_zero=%u\n"
> + "data_t_hs_trial=%u\n"
> + "data_t_ta_go=%u\n"
> + "data_t_ta_get=%u\n",
> + phy_ctrl->rg_vrefsel_vcm,
> + phy_ctrl->clk_pre_delay,
> + phy_ctrl->clk_post_delay,
> + phy_ctrl->clk_t_hs_prepare,
> + phy_ctrl->clk_t_lpx,
> + phy_ctrl->clk_t_hs_zero,
> + phy_ctrl->clk_t_hs_trial,
> + phy_ctrl->data_pre_delay,
> + phy_ctrl->data_post_delay,
> + phy_ctrl->data_t_hs_prepare,
> + phy_ctrl->data_t_lpx,
> + phy_ctrl->data_t_hs_zero,
> + phy_ctrl->data_t_hs_trial,
> + phy_ctrl->data_t_ta_go,
> + phy_ctrl->data_t_ta_get);
> + DRM_DEBUG("clk_lane_lp2hs_time=%u\n"
> + "clk_lane_hs2lp_time=%u\n"
> + "data_lane_lp2hs_time=%u\n"
> + "data_lane_hs2lp_time=%u\n"
> + "phy_stop_wait_time=%u\n",
> + phy_ctrl->clk_lane_lp2hs_time,
> + phy_ctrl->clk_lane_hs2lp_time,
> + phy_ctrl->data_lane_lp2hs_time,
> + phy_ctrl->data_lane_hs2lp_time,
> + phy_ctrl->phy_stop_wait_time);
> +}
> +
> +static void dw_dsi_set_mode(struct dw_dsi *dsi, enum dsi_work_mode mode)
> +{
> + struct dsi_hw_ctx *ctx = dsi->ctx;
> + void __iomem *base = ctx->base;
> +
> + writel(RESET, base + PWR_UP);
> + writel(mode, base + MODE_CFG);
> + writel(POWERUP, base + PWR_UP);
> +}
> +
> +static void dsi_set_burst_mode(void __iomem *base, unsigned long burst_flags)
> +{
> + unsigned long flags;
> + u32 val;
> +
> + flags = burst_flags;
> + flags &= MIPI_DSI_MODE_VIDEO |
> + MIPI_DSI_MODE_VIDEO_BURST |
> + MIPI_DSI_MODE_VIDEO_SYNC_PULSE;
> +
> + if (!(flags & MIPI_DSI_MODE_VIDEO)) {
> + DRM_WARN("MIPI_DSI_MODE_VIDEO was not set! Using DSI_NON_BURST_SYNC_PULSES");
> + val = DSI_NON_BURST_SYNC_PULSES;
> + } else if (flags & MIPI_DSI_MODE_VIDEO_BURST) {
> + val = DSI_BURST_SYNC_PULSES_1;
> + } else if (flags & MIPI_DSI_MODE_VIDEO_SYNC_PULSE) {
> + val = DSI_NON_BURST_SYNC_PULSES;
> + } else {
> + val = DSI_NON_BURST_SYNC_EVENTS;
> + }
> +
> + DRM_INFO("burst_mode = 0x%x (flags: 0x%04lx)", val, burst_flags);
> + set_reg(base + MIPIDSI_VID_MODE_CFG_OFFSET, val, 2, 0);
> +}
> +
> +/*
> + * dsi phy reg write function
> + */
> +static void dsi_phy_tst_set(void __iomem *base, u32 reg, u32 val)
> +{
> + u32 reg_write = 0x10000 + reg;
> +
> + /*
> + * latch reg first
> + */
> + writel(reg_write, base + MIPIDSI_PHY_TST_CTRL1_OFFSET);
> + writel(0x02, base + MIPIDSI_PHY_TST_CTRL0_OFFSET);
> + writel(0x00, base + MIPIDSI_PHY_TST_CTRL0_OFFSET);
> +
> + /*
> + * then latch value
> + */
> + writel(val, base + MIPIDSI_PHY_TST_CTRL1_OFFSET);
> + writel(0x02, base + MIPIDSI_PHY_TST_CTRL0_OFFSET);
> + writel(0x00, base + MIPIDSI_PHY_TST_CTRL0_OFFSET);
> +}
> +
> +static void mipi_config_dphy_spec1v2_parameter(struct dw_dsi *dsi,
> + char __iomem *mipi_dsi_base,
> + u32 id)
> +{
> + u32 i;
> + u32 addr = 0;
> + u32 lanes;
> +
> + lanes = dsi->client[id].lanes - 1;
> + for (i = 0; i <= (lanes + 1); i++) {
> + /* Lane Transmission Property */
> + addr = MIPIDSI_PHY_TST_LANE_TRANSMISSION_PROPERTY + (i << 5);
> + dsi_phy_tst_set(mipi_dsi_base, addr, 0x43);
> + }
> +
> + /* pre_delay of clock lane request setting */
> + dsi_phy_tst_set(mipi_dsi_base, MIPIDSI_PHY_TST_CLK_PRE_DELAY, DSS_REDUCE(dsi->phy.clk_pre_delay));
> +
> + /* post_delay of clock lane request setting */
> + dsi_phy_tst_set(mipi_dsi_base, MIPIDSI_PHY_TST_CLK_POST_DELAY, DSS_REDUCE(dsi->phy.clk_post_delay));
> +
> + /* clock lane timing ctrl - t_lpx */
> + dsi_phy_tst_set(mipi_dsi_base, MIPIDSI_PHY_TST_CLK_TLPX, DSS_REDUCE(dsi->phy.clk_t_lpx));
> +
> + /* clock lane timing ctrl - t_hs_prepare */
> + dsi_phy_tst_set(mipi_dsi_base, MIPIDSI_PHY_TST_CLK_PREPARE, DSS_REDUCE(dsi->phy.clk_t_hs_prepare));
> +
> + /* clock lane timing ctrl - t_hs_zero */
> + dsi_phy_tst_set(mipi_dsi_base, MIPIDSI_PHY_TST_CLK_ZERO, DSS_REDUCE(dsi->phy.clk_t_hs_zero));
> +
> + /* clock lane timing ctrl - t_hs_trial */
> + dsi_phy_tst_set(mipi_dsi_base, MIPIDSI_PHY_TST_CLK_TRAIL, DSS_REDUCE(dsi->phy.clk_t_hs_trial));
> +
> + for (i = 0; i <= 4; i++) {
> + if (lanes == 2 && i == 1) /* init mipi dsi 3 lanes should skip lane3 */
> + i++;
> +
> + if (i == 2) /* skip clock lane */
> + i++; /* addr: lane0:0x60; lane1:0x80; lane2:0xC0; lane3:0xE0 */
> +
> + /* data lane pre_delay */
> + addr = MIPIDSI_PHY_TST_DATA_PRE_DELAY + (i << 5);
> + dsi_phy_tst_set(mipi_dsi_base, addr, DSS_REDUCE(dsi->phy.data_pre_delay));
> +
> + /* data lane post_delay */
> + addr = MIPIDSI_PHY_TST_DATA_POST_DELAY + (i << 5);
> + dsi_phy_tst_set(mipi_dsi_base, addr, DSS_REDUCE(dsi->phy.data_post_delay));
> +
> + /* data lane timing ctrl - t_lpx */
> + addr = MIPIDSI_PHY_TST_DATA_TLPX + (i << 5);
> + dsi_phy_tst_set(mipi_dsi_base, addr, DSS_REDUCE(dsi->phy.data_t_lpx));
> +
> + /* data lane timing ctrl - t_hs_prepare */
> + addr = MIPIDSI_PHY_TST_DATA_PREPARE + (i << 5);
> + dsi_phy_tst_set(mipi_dsi_base, addr, DSS_REDUCE(dsi->phy.data_t_hs_prepare));
> +
> + /* data lane timing ctrl - t_hs_zero */
> + addr = MIPIDSI_PHY_TST_DATA_ZERO + (i << 5);
> + dsi_phy_tst_set(mipi_dsi_base, addr, DSS_REDUCE(dsi->phy.data_t_hs_zero));
> +
> + /* data lane timing ctrl - t_hs_trial */
> + addr = MIPIDSI_PHY_TST_DATA_TRAIL + (i << 5);
> + dsi_phy_tst_set(mipi_dsi_base, addr, DSS_REDUCE(dsi->phy.data_t_hs_trial));
> +
> + DRM_DEBUG("DPHY spec1v2 config :\n"
> + "addr=0x%x\n"
> + "clk_pre_delay=%u\n"
> + "clk_t_hs_trial=%u\n"
> + "data_t_hs_zero=%u\n"
> + "data_t_lpx=%u\n"
> + "data_t_hs_prepare=%u\n",
> + addr,
> + dsi->phy.clk_pre_delay,
> + dsi->phy.clk_t_hs_trial,
> + dsi->phy.data_t_hs_zero,
> + dsi->phy.data_t_lpx,
> + dsi->phy.data_t_hs_prepare);
> + }
> +}
> +
> +static void dsi_mipi_init(struct dw_dsi *dsi, char __iomem *mipi_dsi_base,
> + u32 id)
> +{
> + struct dsi_hw_ctx *ctx = dsi->ctx;
> + u32 hline_time = 0;
> + u32 hsa_time = 0;
> + u32 hbp_time = 0;
> + u64 pixel_clk = 0;
> + unsigned long dw_jiffies = 0;
> + u32 tmp = 0;
> + bool is_ready = false;
> + struct mipi_panel_info *mipi = NULL;
> + struct dss_rect rect;
> + u32 cmp_stopstate_val = 0;
> + u32 lanes;
> +
> + WARN_ON(!dsi);
> + WARN_ON(!mipi_dsi_base);
> +
> + DRM_INFO("%s: id=%d\n", __func__, id);
> +
> + mipi = &dsi->mipi;
> +
> + if (mipi->max_tx_esc_clk == 0) {
> + DRM_INFO("max_tx_esc_clk is invalid!");
> + mipi->max_tx_esc_clk = DEFAULT_MAX_TX_ESC_CLK;
> + }
> +
> + memset(&dsi->phy, 0, sizeof(struct mipi_phy_params));
> +
> + if (ctx->g_dss_version_tag == FB_ACCEL_KIRIN970)
> + kirin970_get_dsi_dphy_ctrl(dsi, &dsi->phy, id);
> + else
> + kirin960_get_dsi_phy_ctrl(dsi, &dsi->phy, id);
> +
> + rect.x = 0;
> + rect.y = 0;
> + rect.w = dsi->cur_mode.hdisplay;
> + rect.h = dsi->cur_mode.vdisplay;
> + lanes = dsi->client[id].lanes - 1;
> + /***************Configure the DPHY start**************/
> +
> + set_reg(mipi_dsi_base + MIPIDSI_PHY_IF_CFG_OFFSET, lanes, 2, 0);
> + set_reg(mipi_dsi_base + MIPIDSI_CLKMGR_CFG_OFFSET, dsi->phy.clk_division, 8, 0);
> + set_reg(mipi_dsi_base + MIPIDSI_CLKMGR_CFG_OFFSET, dsi->phy.clk_division, 8, 8);
> +
> + writel(0x00000000, mipi_dsi_base + MIPIDSI_PHY_RSTZ_OFFSET);
> +
> + writel(0x00000000, mipi_dsi_base + MIPIDSI_PHY_TST_CTRL0_OFFSET);
> + writel(0x00000001, mipi_dsi_base + MIPIDSI_PHY_TST_CTRL0_OFFSET);
> + writel(0x00000000, mipi_dsi_base + MIPIDSI_PHY_TST_CTRL0_OFFSET);
> +
> + if (ctx->g_dss_version_tag == FB_ACCEL_KIRIN970) {
> + dsi_phy_tst_set(mipi_dsi_base, 0x0042, 0x21);
> + /* PLL configuration I */
> + dsi_phy_tst_set(mipi_dsi_base, 0x0046,
> + dsi->phy.rg_cp + (dsi->phy.rg_lpf_r << 4));
> +
> + /* PLL configuration II */
> + dsi_phy_tst_set(mipi_dsi_base, 0x0048,
> + dsi->phy.rg_0p8v + (dsi->phy.rg_2p5g << 1) +
> + (dsi->phy.rg_320m << 2) + (dsi->phy.rg_band_sel << 3));
> +
> + /* PLL configuration III */
> + dsi_phy_tst_set(mipi_dsi_base, 0x0049, dsi->phy.rg_pre_div);
> +
> + /* PLL configuration IV */
> + dsi_phy_tst_set(mipi_dsi_base, 0x004A, dsi->phy.rg_div);
> +
> + dsi_phy_tst_set(mipi_dsi_base, 0x004F, 0xf0);
> + dsi_phy_tst_set(mipi_dsi_base, 0x0050, 0xc0);
> + dsi_phy_tst_set(mipi_dsi_base, 0x0051, 0x22);
> +
> + dsi_phy_tst_set(mipi_dsi_base, 0x0053, dsi->phy.rg_vrefsel_vcm);
> +
> + /* enable BTA */
> + dsi_phy_tst_set(mipi_dsi_base, 0x0054, 0x03);
> +
> + /* PLL update control */
> + dsi_phy_tst_set(mipi_dsi_base, 0x004B, 0x1);
> +
> + /* set dphy spec parameter */
> + mipi_config_dphy_spec1v2_parameter(dsi, mipi_dsi_base, id);
> + } else {
> + int i = 0;
> +
> + /* physical configuration PLL I */
> + dsi_phy_tst_set(mipi_dsi_base, 0x14,
> + (dsi->phy.rg_pll_fbd_s << 4) + (dsi->phy.rg_pll_enswc << 3) +
> + (dsi->phy.rg_pll_enbwt << 2) + dsi->phy.rg_pll_chp);
> +
> + /* physical configuration PLL II, M */
> + dsi_phy_tst_set(mipi_dsi_base, 0x15, dsi->phy.rg_pll_fbd_p);
> +
> + /* physical configuration PLL III */
> + dsi_phy_tst_set(mipi_dsi_base, 0x16,
> + (dsi->phy.rg_pll_cp << 5) + (dsi->phy.rg_pll_lpf_cs << 4) +
> + dsi->phy.rg_pll_refsel);
> +
> + /* physical configuration PLL IV, N */
> + dsi_phy_tst_set(mipi_dsi_base, 0x17, dsi->phy.rg_pll_pre_p);
> +
> + /* sets the analog characteristic of V reference in D-PHY TX */
> + dsi_phy_tst_set(mipi_dsi_base, 0x1D, dsi->phy.rg_vrefsel_vcm);
> +
> + /* MISC AFE Configuration */
> + dsi_phy_tst_set(mipi_dsi_base, 0x1E,
> + (dsi->phy.rg_pll_cp_p << 5) + (dsi->phy.reload_sel << 4) +
> + (dsi->phy.rg_phase_gen_en << 3) + (dsi->phy.rg_band_sel << 2) +
> + (dsi->phy.pll_power_down << 1) + dsi->phy.pll_register_override);
> +
> + /* reload_command */
> + dsi_phy_tst_set(mipi_dsi_base, 0x1F, dsi->phy.load_command);
> +
> + /* pre_delay of clock lane request setting */
> + dsi_phy_tst_set(mipi_dsi_base, 0x20, DSS_REDUCE(dsi->phy.clk_pre_delay));
> +
> + /* post_delay of clock lane request setting */
> + dsi_phy_tst_set(mipi_dsi_base, 0x21, DSS_REDUCE(dsi->phy.clk_post_delay));
> +
> + /* clock lane timing ctrl - t_lpx */
> + dsi_phy_tst_set(mipi_dsi_base, 0x22, DSS_REDUCE(dsi->phy.clk_t_lpx));
> +
> + /* clock lane timing ctrl - t_hs_prepare */
> + dsi_phy_tst_set(mipi_dsi_base, 0x23, DSS_REDUCE(dsi->phy.clk_t_hs_prepare));
> +
> + /* clock lane timing ctrl - t_hs_zero */
> + dsi_phy_tst_set(mipi_dsi_base, 0x24, DSS_REDUCE(dsi->phy.clk_t_hs_zero));
> +
> + /* clock lane timing ctrl - t_hs_trial */
> + dsi_phy_tst_set(mipi_dsi_base, 0x25, dsi->phy.clk_t_hs_trial);
> +
> + for (i = 0; i <= lanes; i++) {
> + /* data lane pre_delay */
> + tmp = 0x30 + (i << 4);
> + dsi_phy_tst_set(mipi_dsi_base, tmp, DSS_REDUCE(dsi->phy.data_pre_delay));
> +
> + /* data lane post_delay */
> + tmp = 0x31 + (i << 4);
> + dsi_phy_tst_set(mipi_dsi_base, tmp, DSS_REDUCE(dsi->phy.data_post_delay));
> +
> + /* data lane timing ctrl - t_lpx */
> + dsi_phy_tst_set(mipi_dsi_base, tmp, DSS_REDUCE(dsi->phy.data_t_lpx));
> +
> + /* data lane timing ctrl - t_hs_prepare */
> + tmp = 0x33 + (i << 4);
> + dsi_phy_tst_set(mipi_dsi_base, tmp, DSS_REDUCE(dsi->phy.data_t_hs_prepare));
> +
> + /* data lane timing ctrl - t_hs_zero */
> + tmp = 0x34 + (i << 4);
> + dsi_phy_tst_set(mipi_dsi_base, tmp, DSS_REDUCE(dsi->phy.data_t_hs_zero));
> +
> + /* data lane timing ctrl - t_hs_trial */
> + tmp = 0x35 + (i << 4);
> + dsi_phy_tst_set(mipi_dsi_base, tmp, DSS_REDUCE(dsi->phy.data_t_hs_trial));
> +
> + /* data lane timing ctrl - t_ta_go */
> + tmp = 0x36 + (i << 4);
> + dsi_phy_tst_set(mipi_dsi_base, tmp, DSS_REDUCE(dsi->phy.data_t_ta_go));
> +
> + /* data lane timing ctrl - t_ta_get */
> + tmp = 0x37 + (i << 4);
> + dsi_phy_tst_set(mipi_dsi_base, tmp, DSS_REDUCE(dsi->phy.data_t_ta_get));
> + }
> + }
> +
> + writel(0x00000007, mipi_dsi_base + MIPIDSI_PHY_RSTZ_OFFSET);
> +
> + is_ready = false;
> + dw_jiffies = jiffies + HZ / 2;
> + do {
> + tmp = readl(mipi_dsi_base + MIPIDSI_PHY_STATUS_OFFSET);
> + if ((tmp & 0x00000001) == 0x00000001) {
> + is_ready = true;
> + break;
> + }
> + } while (time_after(dw_jiffies, jiffies));
Use jiffies + msecs_to_jiffies(xxx) - and drop HZ /2.
Same goes for other similar HZ uses.

> +
> + if (!is_ready) {
> + DRM_INFO("phylock is not ready!MIPIDSI_PHY_STATUS_OFFSET=0x%x.\n",
> + tmp);
> + }
> +
> + if (lanes >= DSI_4_LANES)
> + cmp_stopstate_val = (BIT(4) | BIT(7) | BIT(9) | BIT(11));
> + else if (lanes >= DSI_3_LANES)
> + cmp_stopstate_val = (BIT(4) | BIT(7) | BIT(9));
> + else if (lanes >= DSI_2_LANES)
> + cmp_stopstate_val = (BIT(4) | BIT(7));
> + else
> + cmp_stopstate_val = (BIT(4));
> +
> + is_ready = false;
> + dw_jiffies = jiffies + HZ / 2;
> + do {
> + tmp = readl(mipi_dsi_base + MIPIDSI_PHY_STATUS_OFFSET);
> + if ((tmp & cmp_stopstate_val) == cmp_stopstate_val) {
> + is_ready = true;
> + break;
> + }
> + } while (time_after(dw_jiffies, jiffies));
> +
> + if (!is_ready) {
> + DRM_INFO("phystopstateclklane is not ready! MIPIDSI_PHY_STATUS_OFFSET=0x%x.\n",
> + tmp);
> + }
> +
> + /*************************Configure the DPHY end*************************/
> +
> + /* phy_stop_wait_time */
> + set_reg(mipi_dsi_base + MIPIDSI_PHY_IF_CFG_OFFSET, dsi->phy.phy_stop_wait_time, 8, 8);
> +
> + /*--------------configuring the DPI packet transmission---------------- */
> +
> + /*
> + * 2. Configure the DPI Interface:
> + * This defines how the DPI interface interacts with the controller.
> + */
> + set_reg(mipi_dsi_base + MIPIDSI_DPI_VCID_OFFSET, mipi->vc, 2, 0);
> + set_reg(mipi_dsi_base + MIPIDSI_DPI_COLOR_CODING_OFFSET, mipi->color_mode, 4, 0);
> +
> + set_reg(mipi_dsi_base + MIPIDSI_DPI_CFG_POL_OFFSET, dsi->ldi.data_en_plr, 1, 0);
> + set_reg(mipi_dsi_base + MIPIDSI_DPI_CFG_POL_OFFSET, dsi->ldi.vsync_plr, 1, 1);
> + set_reg(mipi_dsi_base + MIPIDSI_DPI_CFG_POL_OFFSET, dsi->ldi.hsync_plr, 1, 2);
> + set_reg(mipi_dsi_base + MIPIDSI_DPI_CFG_POL_OFFSET, 0x0, 1, 3);
> + set_reg(mipi_dsi_base + MIPIDSI_DPI_CFG_POL_OFFSET, 0x0, 1, 4);
> +
> + /*
> + * 3. Select the Video Transmission Mode:
> + * This defines how the processor requires the video line to be
> + * transported through the DSI link.
> + */
> + /* video mode: low power mode */
> + set_reg(mipi_dsi_base + MIPIDSI_VID_MODE_CFG_OFFSET, 0x3f, 6, 8);
> +
> + /* TODO: fix blank display bug when set backlight */
> + set_reg(mipi_dsi_base + MIPIDSI_DPI_LP_CMD_TIM_OFFSET, 0x4, 8, 16);
> + /* video mode: send read cmd by lp mode */
> + set_reg(mipi_dsi_base + MIPIDSI_VID_MODE_CFG_OFFSET, 0x1, 1, 15);
> +
> + set_reg(mipi_dsi_base + MIPIDSI_VID_PKT_SIZE_OFFSET, rect.w, 14, 0);
> +
> + /* burst mode */
> + dsi_set_burst_mode(mipi_dsi_base, dsi->client[id].mode_flags);
> + /* for dsi read, BTA enable */
> + set_reg(mipi_dsi_base + MIPIDSI_PCKHDL_CFG_OFFSET, 0x1, 1, 2);
> +
> + /*
> + * 4. Define the DPI Horizontal timing configuration:
> + *
> + * Hsa_time = HSA*(PCLK period/Clk Lane Byte Period);
> + * Hbp_time = HBP*(PCLK period/Clk Lane Byte Period);
> + * Hline_time = (HSA+HBP+HACT+HFP)*(PCLK period/Clk Lane Byte Period);
> + */
> + pixel_clk = dsi->cur_mode.clock * 1000;
> + /* htot = dsi->cur_mode.htotal; */
> + /* vtot = dsi->cur_mode.vtotal; */
> + dsi->ldi.h_front_porch = dsi->cur_mode.hsync_start - dsi->cur_mode.hdisplay;
> + dsi->ldi.h_back_porch = dsi->cur_mode.htotal - dsi->cur_mode.hsync_end;
> + dsi->ldi.h_pulse_width = dsi->cur_mode.hsync_end - dsi->cur_mode.hsync_start;
> + dsi->ldi.v_front_porch = dsi->cur_mode.vsync_start - dsi->cur_mode.vdisplay;
> + dsi->ldi.v_back_porch = dsi->cur_mode.vtotal - dsi->cur_mode.vsync_end;
> + dsi->ldi.v_pulse_width = dsi->cur_mode.vsync_end - dsi->cur_mode.vsync_start;
> + if (dsi->ldi.v_pulse_width > 15) {
> + DRM_DEBUG_DRIVER("vsw exceeded 15\n");
> + dsi->ldi.v_pulse_width = 15;
> + }
> + hsa_time = dsi->ldi.h_pulse_width * dsi->phy.lane_byte_clk / pixel_clk;
> + hbp_time = dsi->ldi.h_back_porch * dsi->phy.lane_byte_clk / pixel_clk;
> + hline_time = DIV_ROUND_UP((dsi->ldi.h_pulse_width + dsi->ldi.h_back_porch +
> + rect.w + dsi->ldi.h_front_porch) * dsi->phy.lane_byte_clk, pixel_clk);
> +
> + DRM_INFO("hsa_time=%d, hbp_time=%d, hline_time=%d\n",
> + hsa_time, hbp_time, hline_time);
> + DRM_INFO("lane_byte_clk=%llu, pixel_clk=%llu\n",
> + dsi->phy.lane_byte_clk, pixel_clk);
> + set_reg(mipi_dsi_base + MIPIDSI_VID_HSA_TIME_OFFSET, hsa_time, 12, 0);
> + set_reg(mipi_dsi_base + MIPIDSI_VID_HBP_TIME_OFFSET, hbp_time, 12, 0);
> + set_reg(mipi_dsi_base + MIPIDSI_VID_HLINE_TIME_OFFSET, hline_time, 15, 0);
> +
> + /* Define the Vertical line configuration */
> + set_reg(mipi_dsi_base + MIPIDSI_VID_VSA_LINES_OFFSET,
> + dsi->ldi.v_pulse_width, 10, 0);
> + set_reg(mipi_dsi_base + MIPIDSI_VID_VBP_LINES_OFFSET,
> + dsi->ldi.v_back_porch, 10, 0);
> + set_reg(mipi_dsi_base + MIPIDSI_VID_VFP_LINES_OFFSET,
> + dsi->ldi.v_front_porch, 10, 0);
> + set_reg(mipi_dsi_base + MIPIDSI_VID_VACTIVE_LINES_OFFSET,
> + rect.h, 14, 0);
> + set_reg(mipi_dsi_base + MIPIDSI_TO_CNT_CFG_OFFSET,
> + 0x7FF, 16, 0);
> +
> + /* Configure core's phy parameters */
> + set_reg(mipi_dsi_base + MIPIDSI_PHY_TMR_LPCLK_CFG_OFFSET,
> + dsi->phy.clk_lane_lp2hs_time, 10, 0);
> + set_reg(mipi_dsi_base + MIPIDSI_PHY_TMR_LPCLK_CFG_OFFSET,
> + dsi->phy.clk_lane_hs2lp_time, 10, 16);
> +
> + set_reg(mipi_dsi_base + MIPIDSI_PHY_TMR_RD_CFG_OFFSET,
> + 0x7FFF, 15, 0);
> + set_reg(mipi_dsi_base + MIPIDSI_PHY_TMR_CFG_OFFSET,
> + dsi->phy.data_lane_lp2hs_time, 10, 0);
> + set_reg(mipi_dsi_base + MIPIDSI_PHY_TMR_CFG_OFFSET,
> + dsi->phy.data_lane_hs2lp_time, 10, 16);
> +
> + if (ctx->g_dss_version_tag == FB_ACCEL_KIRIN970) {
> + /* 16~19bit:pclk_en, pclk_sel, dpipclk_en, dpipclk_sel */
> + set_reg(mipi_dsi_base + MIPIDSI_CLKMGR_CFG_OFFSET, 0x5, 4, 16);
> + /* 0:dphy */
> + set_reg(mipi_dsi_base + KIRIN970_PHY_MODE, 0x0, 1, 0);
> + }
> +
> + /* Waking up Core */
> + set_reg(mipi_dsi_base + MIPIDSI_PWR_UP_OFFSET, 0x1, 1, 0);
> +}
> +
> +static void dsi_encoder_disable(struct drm_encoder *encoder)
> +{
> + struct dw_dsi *dsi = encoder_to_dsi(encoder);
> + struct dsi_hw_ctx *ctx = dsi->ctx;
> + void __iomem *base = ctx->base;
> +
> + if (!dsi->enable)
> + return;
> +
> + dw_dsi_set_mode(dsi, DSI_COMMAND_MODE);
> + /* turn off panel's backlight */
> + if (dsi->panel && drm_panel_disable(dsi->panel))
> + DRM_ERROR("failed to disable panel\n");
> +
> + /* turn off panel */
> + if (dsi->panel && drm_panel_unprepare(dsi->panel))
> + DRM_ERROR("failed to unprepare panel\n");
> +
> + writel(0, base + PWR_UP);
> + writel(0, base + LPCLK_CTRL);
> + writel(0, base + PHY_RSTZ);
> + clk_disable_unprepare(ctx->dss_dphy0_ref_clk);
> + clk_disable_unprepare(ctx->dss_dphy0_cfg_clk);
> + clk_disable_unprepare(ctx->dss_pclk_dsi0_clk);
> +
> + dsi->enable = false;
> +}
> +
> +static int mipi_dsi_on_sub1(struct dw_dsi *dsi, char __iomem *mipi_dsi_base,
> + u32 id)
> +{
> + struct dsi_hw_ctx *ctx = dsi->ctx;
> +
> + WARN_ON(!mipi_dsi_base);
> +
> + /* mipi init */
> + dsi_mipi_init(dsi, mipi_dsi_base, id);
> +
> + /* dsi memory init */
> + if (ctx->g_dss_version_tag == FB_ACCEL_KIRIN970)
> + writel(0x02600008, mipi_dsi_base + KIRIN970_DSI_MEM_CTRL);
> +
> + /* switch to cmd mode */
> + set_reg(mipi_dsi_base + MIPIDSI_MODE_CFG_OFFSET, 0x1, 1, 0);
> + /* cmd mode: low power mode */
> + set_reg(mipi_dsi_base + MIPIDSI_CMD_MODE_CFG_OFFSET, 0x7f, 7, 8);
> + set_reg(mipi_dsi_base + MIPIDSI_CMD_MODE_CFG_OFFSET, 0xf, 4, 16);
> + set_reg(mipi_dsi_base + MIPIDSI_CMD_MODE_CFG_OFFSET, 0x1, 1, 24);
> + /* disable generate High Speed clock */
> + /* delete? */
> + set_reg(mipi_dsi_base + MIPIDSI_LPCLK_CTRL_OFFSET, 0x0, 1, 0);
> +
> + return 0;
> +}
> +
> +static int mipi_dsi_on_sub2(struct dw_dsi *dsi, char __iomem *mipi_dsi_base)
> +{
> + struct dsi_hw_ctx *ctx = dsi->ctx;
> +
> + u64 pctrl_dphytx_stopcnt = 0;
> +
> + WARN_ON(!mipi_dsi_base);
> +
> + /* switch to video mode */
> + set_reg(mipi_dsi_base + MIPIDSI_MODE_CFG_OFFSET, 0x0, 1, 0);
> +
> + /* enable EOTP TX */
> + set_reg(mipi_dsi_base + MIPIDSI_PCKHDL_CFG_OFFSET, 0x1, 1, 0);
> +
> + /* enable generate High Speed clock, continue clock */
> + set_reg(mipi_dsi_base + MIPIDSI_LPCLK_CTRL_OFFSET, 0x1, 2, 0);
> +
> + if (ctx->g_dss_version_tag == FB_ACCEL_KIRIN970) {
> + /* init: wait DPHY 4 data lane stopstate */
> + pctrl_dphytx_stopcnt = (u64)(dsi->ldi.h_back_porch +
> + dsi->ldi.h_front_porch + dsi->ldi.h_pulse_width + dsi->cur_mode.hdisplay + 5) *
> + DEFAULT_PCLK_PCTRL_RATE / (dsi->cur_mode.clock * 1000);
> + DRM_DEBUG("pctrl_dphytx_stopcnt = %llu\n", pctrl_dphytx_stopcnt);
> +
> + /* FIXME: */
> + writel((u32)pctrl_dphytx_stopcnt, dsi->ctx->pctrl_base + PERI_CTRL29);
> + }
> +
> + return 0;
> +}
> +
> +static void dsi_encoder_enable(struct drm_encoder *encoder)
> +{
> + struct dw_dsi *dsi = encoder_to_dsi(encoder);
> + struct dsi_hw_ctx *ctx = dsi->ctx;
> + int ret;
> +
> + if (dsi->enable)
> + return;
> +
> + ret = clk_prepare_enable(ctx->dss_dphy0_ref_clk);
> + if (ret) {
> + DRM_ERROR("fail to enable dss_dphy0_ref_clk: %d\n", ret);
> + return;
> + }
> +
> + ret = clk_prepare_enable(ctx->dss_dphy0_cfg_clk);
> + if (ret) {
> + DRM_ERROR("fail to enable dss_dphy0_cfg_clk: %d\n", ret);
> + return;
> + }
> +
> + ret = clk_prepare_enable(ctx->dss_pclk_dsi0_clk);
> + if (ret) {
> + DRM_ERROR("fail to enable dss_pclk_dsi0_clk: %d\n", ret);
> + return;
> + }
> +
> + mipi_dsi_on_sub1(dsi, ctx->base, dsi->attached_client);
> +
> + mipi_dsi_on_sub2(dsi, ctx->base);
> +
> + /* turn on panel */
> + if (dsi->panel && drm_panel_prepare(dsi->panel))
> + DRM_ERROR("failed to prepare panel\n");
> +
> + /* dw_dsi_set_mode(dsi, DSI_VIDEO_MODE); */
> +
> + /* turn on panel's back light */
> + if (dsi->panel && drm_panel_enable(dsi->panel))
> + DRM_ERROR("failed to enable panel\n");
> +
> + dsi->enable = true;
> +}
> +
> +static enum drm_mode_status dsi_encoder_mode_valid(struct drm_encoder *encoder,
> + const struct drm_display_mode *mode)
> +
> +{
> + const struct drm_crtc_helper_funcs *crtc_funcs;
> + struct drm_display_mode adj_mode;
> + int clock = mode->clock;
> + struct drm_crtc *crtc;
> +
> + drm_for_each_crtc(crtc, encoder->dev) {
> + drm_mode_copy(&adj_mode, mode);
> +
> + crtc_funcs = crtc->helper_private;
> + if (crtc_funcs && crtc_funcs->mode_fixup) {
> + if (!crtc_funcs->mode_fixup(crtc, mode, &adj_mode)) {
> + DRM_INFO("Discarded mode: %ix%i@%i, clock: %i (adjusted to %i)",
> + mode->hdisplay, mode->vdisplay,
> + drm_mode_vrefresh(mode),
> + mode->clock, clock);
> +
> + return MODE_BAD;
> + }
> + clock = adj_mode.clock;
> + }
> + }
> +
> + DRM_INFO("Valid mode: %ix%i@%i, clock %i (adjusted to %i)",
> + mode->hdisplay, mode->vdisplay, drm_mode_vrefresh(mode),
> + mode->clock, clock);
> +
> + return MODE_OK;
> +}
> +
> +static void dsi_encoder_mode_set(struct drm_encoder *encoder,
> + struct drm_display_mode *mode,
> + struct drm_display_mode *adj_mode)
> +{
> + struct dw_dsi *dsi = encoder_to_dsi(encoder);
> +
> + drm_mode_copy(&dsi->cur_mode, adj_mode);
> +}
> +
> +static int dsi_encoder_atomic_check(struct drm_encoder *encoder,
> + struct drm_crtc_state *crtc_state,
> + struct drm_connector_state *conn_state)
> +{
> + /* do nothing */
> + return 0;
> +}
> +
> +static const struct drm_encoder_helper_funcs dw_encoder_helper_funcs = {
> + .atomic_check = dsi_encoder_atomic_check,
> + .mode_valid = dsi_encoder_mode_valid,
> + .mode_set = dsi_encoder_mode_set,
> + .enable = dsi_encoder_enable,
> + .disable = dsi_encoder_disable
> +};
> +
> +static const struct drm_encoder_funcs dw_encoder_funcs = {
> + .destroy = drm_encoder_cleanup,
> +};
> +
> +static int dw_drm_encoder_init(struct device *dev,
> + struct drm_device *drm_dev,
> + struct drm_encoder *encoder,
> + struct drm_bridge *bridge)
> +{
> + int ret;
> + u32 crtc_mask;
> +
> + drm_info(drm_dev, "%s:\n", __func__);
> +
> + /* Link drm_bridge to encoder */
> + if (!bridge) {
> + drm_info(drm_dev, "no dsi bridge to attach the encoder\n");
> + return 0;
> + }
> +
> + crtc_mask = drm_of_find_possible_crtcs(drm_dev, dev->of_node);
> + if (!crtc_mask) {
> + DRM_ERROR("failed to find crtc mask\n");
> + return -EINVAL;
> + }
> +
> + drm_info(drm_dev, "Initializing CRTC encoder: %d\n",
> + crtc_mask);
> +
> + encoder->possible_crtcs = crtc_mask;
> + encoder->possible_clones = 0;
> + ret = drm_encoder_init(drm_dev, encoder, &dw_encoder_funcs,
> + DRM_MODE_ENCODER_DSI, NULL);
> + if (ret) {
> + drm_info(drm_dev, "failed to init dsi encoder\n");
> + return ret;
> + }
> +
> + drm_encoder_helper_add(encoder, &dw_encoder_helper_funcs);
> +
> + /* associate the bridge to dsi encoder */
> + ret = drm_bridge_attach(encoder, bridge, NULL, 0);
> + if (ret) {
> + drm_info(drm_dev, "failed to attach external bridge\n");
> + drm_encoder_cleanup(encoder);
> + }
> +
> + return ret;
> +}
All the encoder stuff can, I think, be repalced by the simple encoder.
See how other drivers uses drm_simple_encoder_init()

> +
> +static int dsi_host_attach(struct mipi_dsi_host *host,
> + struct mipi_dsi_device *mdsi)
> +{
> + struct dw_dsi *dsi = host_to_dsi(host);
> + u32 id = mdsi->channel >= 1 ? OUT_PANEL : OUT_HDMI;
> +
> + if (mdsi->lanes < 1 || mdsi->lanes > 4) {
> + DRM_ERROR("dsi device params invalid\n");
> + return -EINVAL;
> + }
> +
> + dsi->client[id].lanes = mdsi->lanes;
> + dsi->client[id].format = mdsi->format;
> + dsi->client[id].mode_flags = mdsi->mode_flags;
> + dsi->client[id].phy_clock = 0;
> +
> + dsi->attached_client = id;
> +
> + DRM_DEBUG("host attach, client name=[%s], id=%d\n", mdsi->name, id);
> +
> + return 0;
> +}
> +
> +static int dsi_host_detach(struct mipi_dsi_host *host,
> + struct mipi_dsi_device *mdsi)
> +{
> + /* do nothing */
> + return 0;
> +}
> +
> +static int dsi_gen_pkt_hdr_write(void __iomem *base, u32 val)
> +{
> + u32 status;
> + int ret;
> +
> + ret = readx_poll_timeout(readl, base + CMD_PKT_STATUS, status,
> + !(status & GEN_CMD_FULL), 1000,
> + CMD_PKT_STATUS_TIMEOUT_US);
> + if (ret < 0) {
> + DRM_ERROR("failed to get available command FIFO\n");
> + return ret;
> + }
> +
> + writel(val, base + GEN_HDR);
> +
> + ret = readx_poll_timeout(readl, base + CMD_PKT_STATUS, status,
> + status & (GEN_CMD_EMPTY | GEN_PLD_W_EMPTY),
> + 1000, CMD_PKT_STATUS_TIMEOUT_US);
> + if (ret < 0) {
> + DRM_ERROR("failed to write command FIFO\n");
> + return ret;
> + }
> +
> + return 0;
> +}
> +
> +static int dsi_dcs_short_write(void __iomem *base,
> + const struct mipi_dsi_msg *msg)
> +{
> + const u16 *tx_buf = msg->tx_buf;
> + u32 val = GEN_HDATA(*tx_buf) | GEN_HTYPE(msg->type);
> +
> + if (msg->tx_len > 2) {
> + DRM_ERROR("too long tx buf length %zu for short write\n",
> + msg->tx_len);
> + return -EINVAL;
> + }
> +
> + return dsi_gen_pkt_hdr_write(base, val);
> +}
> +
> +static int dsi_dcs_long_write(void __iomem *base,
> + const struct mipi_dsi_msg *msg)
> +{
> + const u32 *tx_buf = msg->tx_buf;
> + int len = msg->tx_len, pld_data_bytes = sizeof(*tx_buf), ret;
> + u32 val = GEN_HDATA(msg->tx_len) | GEN_HTYPE(msg->type);
> + u32 remainder = 0;
> + u32 status;
> +
> + if (msg->tx_len < 3) {
> + DRM_ERROR("wrong tx buf length %zu for long write\n",
> + msg->tx_len);
> + return -EINVAL;
> + }
> +
> + while (DIV_ROUND_UP(len, pld_data_bytes)) {
> + if (len < pld_data_bytes) {
> + memcpy(&remainder, tx_buf, len);
> + writel(remainder, base + GEN_PLD_DATA);
> + len = 0;
> + } else {
> + writel(*tx_buf, base + GEN_PLD_DATA);
> + tx_buf++;
> + len -= pld_data_bytes;
> + }
> +
> + ret = readx_poll_timeout(readl, base + CMD_PKT_STATUS,
> + status, !(status & GEN_PLD_W_FULL), 1000,
> + CMD_PKT_STATUS_TIMEOUT_US);
> + if (ret < 0) {
> + DRM_ERROR("failed to get available write payload FIFO\n");
> + return ret;
> + }
> + }
> +
> + return dsi_gen_pkt_hdr_write(base, val);
> +}
> +
> +static ssize_t dsi_host_transfer(struct mipi_dsi_host *host,
> + const struct mipi_dsi_msg *msg)
> +{
> + struct dw_dsi *dsi = host_to_dsi(host);
> + struct dsi_hw_ctx *ctx = dsi->ctx;
> + void __iomem *base = ctx->base;
> + int ret;
> +
> + switch (msg->type) {
> + case MIPI_DSI_DCS_SHORT_WRITE:
> + case MIPI_DSI_DCS_SHORT_WRITE_PARAM:
> + case MIPI_DSI_SET_MAXIMUM_RETURN_PACKET_SIZE:
> + ret = dsi_dcs_short_write(base, msg);
> + break;
> + case MIPI_DSI_DCS_LONG_WRITE:
> + ret = dsi_dcs_long_write(base, msg);
> + break;
> + default:
> + DRM_ERROR("unsupported message type\n");
> + ret = -EINVAL;
> + }
> +
> + return ret;
> +}
> +
> +static const struct mipi_dsi_host_ops dsi_host_ops = {
> + .attach = dsi_host_attach,
> + .detach = dsi_host_detach,
> + .transfer = dsi_host_transfer,
> +};
> +
> +static int dsi_host_init(struct device *dev, struct dw_dsi *dsi)
> +{
> + struct mipi_dsi_host *host = &dsi->host;
> + struct mipi_panel_info *mipi = &dsi->mipi;
> + int ret;
> +
> + host->dev = dev;
> + host->ops = &dsi_host_ops;
> +
> + mipi->max_tx_esc_clk = 10 * 1000000UL;
> + mipi->vc = 0;
> + mipi->color_mode = DSI_24BITS_1;
> + mipi->clk_post_adjust = 120;
> + mipi->clk_pre_adjust = 0;
> + mipi->clk_t_hs_prepare_adjust = 0;
> + mipi->clk_t_lpx_adjust = 0;
> + mipi->clk_t_hs_trial_adjust = 0;
> + mipi->clk_t_hs_exit_adjust = 0;
> + mipi->clk_t_hs_zero_adjust = 0;
> +
> + dsi->ldi.data_en_plr = 0;
> + dsi->ldi.vsync_plr = 0;
> + dsi->ldi.hsync_plr = 0;
> +
> + ret = mipi_dsi_host_register(host);
> + if (ret) {
> + dev_info(dev, "failed to register dsi host\n");
> + return ret;
> + }
> +
> + return 0;
> +}
> +
> +static int dsi_connector_get_modes(struct drm_connector *connector)
> +{
> + struct dw_dsi *dsi = connector_to_dsi(connector);
> +
> + return drm_panel_get_modes(dsi->panel, connector);
> +}
> +
> +static enum drm_mode_status
> +dsi_connector_mode_valid(struct drm_connector *connector,
> + struct drm_display_mode *mode)
> +{
> + enum drm_mode_status mode_status = MODE_OK;
> +
> + return mode_status;
> +}
> +
> +static struct drm_encoder *
> +dsi_connector_best_encoder(struct drm_connector *connector)
> +{
> + struct dw_dsi *dsi = connector_to_dsi(connector);
> +
> + return &dsi->encoder;
> +}
> +
> +static const struct drm_connector_helper_funcs dsi_connector_helper_funcs = {
> + .get_modes = dsi_connector_get_modes,
> + .mode_valid = dsi_connector_mode_valid,
> + .best_encoder = dsi_connector_best_encoder,
> +};
> +
> +static enum drm_connector_status
> +dsi_connector_detect(struct drm_connector *connector, bool force)
> +{
> + struct dw_dsi *dsi = connector_to_dsi(connector);
> + enum drm_connector_status status;
> +
> + status = dsi->cur_client == OUT_PANEL ? connector_status_connected :
> + connector_status_disconnected;
> +
> + return status;
> +}
> +
> +static void dsi_connector_destroy(struct drm_connector *connector)
> +{
> + drm_connector_unregister(connector);
> + drm_connector_cleanup(connector);
> +}
> +
> +static struct drm_connector_funcs dsi_atomic_connector_funcs = {
> + .fill_modes = drm_helper_probe_single_connector_modes,
> + .detect = dsi_connector_detect,
> + .destroy = dsi_connector_destroy,
> + .reset = drm_atomic_helper_connector_reset,
> + .atomic_duplicate_state = drm_atomic_helper_connector_duplicate_state,
> + .atomic_destroy_state = drm_atomic_helper_connector_destroy_state,
> +};
> +
> +static int dsi_connector_init(struct drm_device *dev, struct dw_dsi *dsi)
> +{
> + struct drm_encoder *encoder = &dsi->encoder;
> + struct drm_connector *connector = &dsi->connector;
> + int ret;
> +
> + connector->polled = DRM_CONNECTOR_POLL_HPD;
> + drm_connector_helper_add(connector,
> + &dsi_connector_helper_funcs);
> +
> + ret = drm_connector_init(dev, &dsi->connector,
> + &dsi_atomic_connector_funcs,
> + DRM_MODE_CONNECTOR_DSI);
> + if (ret)
> + return ret;
> +
> + drm_dbg(dev, "Attaching CRTC encoder\n");
> + ret = drm_connector_attach_encoder(connector, encoder);
> + if (ret)
> + return ret;
> +
> + ret = drm_panel_attach(dsi->panel, connector);
> + if (ret)
> + return ret;
> +
> + drm_connector_register(&dsi->connector);
> +
> + drm_dbg(dev, "connector init\n");
> + return 0;
> +}
> +
> +static int dsi_bind(struct device *dev, struct device *master, void *data)
> +{
> + struct dsi_data *ddata = dev_get_drvdata(dev);
> + struct dw_dsi *dsi = &ddata->dsi;
> + struct drm_device *drm_dev = data;
> + int ret;
> +
> + ret = dw_drm_encoder_init(dev, drm_dev, &dsi->encoder,
> + dsi->bridge);
> + if (ret)
> + return ret;
> +
> + if (dsi->panel) {
> + ret = dsi_connector_init(drm_dev, dsi);
> + if (ret)
> + return ret;
> + }
> +
> + return 0;
> +}
> +
> +static void dsi_unbind(struct device *dev, struct device *master, void *data)
> +{
> + /* do nothing */
> +}
> +
> +static const struct component_ops dsi_ops = {
> + .bind = dsi_bind,
> + .unbind = dsi_unbind,
> +};
> +
> +static int dsi_parse_bridge_endpoint(struct dw_dsi *dsi,
> + struct device_node *endpoint)
> +{
> + struct device_node *bridge_node;
> + struct drm_bridge *bridge;
> +
> + bridge_node = of_graph_get_remote_port_parent(endpoint);
> + if (!bridge_node) {
> + DRM_ERROR("no valid bridge node\n");
> + return -ENODEV;
> + }
> + of_node_put(bridge_node);
> +
> + bridge = of_drm_find_bridge(bridge_node);
> + if (!bridge) {
> + DRM_INFO("wait for external HDMI bridge driver.\n");
> + return -EPROBE_DEFER;
> + }
> + dsi->bridge = bridge;
> +
> + return 0;
> +}
> +
> +static int dsi_parse_panel_endpoint(struct dw_dsi *dsi,
> + struct device_node *endpoint)
> +{
> + struct device_node *panel_node;
> + struct drm_panel *panel;
> +
> + panel_node = of_graph_get_remote_port_parent(endpoint);
> + if (!panel_node) {
> + DRM_ERROR("no valid panel node\n");
> + return -ENODEV;
> + }
> + of_node_put(panel_node);
> +
> + panel = of_drm_find_panel(panel_node);
> + if (!panel) {
> + DRM_DEBUG_DRIVER("skip this panel endpoint.\n");
> + return 0;
> + }
> + dsi->panel = panel;
> +
> + return 0;
> +}
> +
> +static int dsi_parse_endpoint(struct dw_dsi *dsi,
> + struct device_node *np,
> + enum dsi_output_client client)
> +{
> + struct device_node *ep_node;
> + struct of_endpoint ep;
> + int ret = 0;
> +
> + if (client == OUT_MAX)
> + return -EINVAL;
> +
> + for_each_endpoint_of_node(np, ep_node) {
> + ret = of_graph_parse_endpoint(ep_node, &ep);
> + if (ret) {
> + of_node_put(ep_node);
> + return ret;
> + }
> +
> + /* skip dsi input port, port == 0 is input port */
> + if (ep.port == 0)
> + continue;
> +
> + /* parse bridge endpoint */
> + if (client == OUT_HDMI) {
> + if (ep.id == 0) {
> + ret = dsi_parse_bridge_endpoint(dsi, ep_node);
> + if (dsi->bridge)
> + break;
> + }
> + } else { /* parse panel endpoint */
> + if (ep.id > 0) {
> + ret = dsi_parse_panel_endpoint(dsi, ep_node);
> + if (dsi->panel)
> + break;
> + }
> + }
> +
> + if (ret) {
> + of_node_put(ep_node);
> + return ret;
> + }
> + }
> +
> + if (!dsi->bridge && !dsi->panel) {
> + DRM_ERROR("at least one bridge or panel node is required\n");
> + return -ENODEV;
> + }
> +
> + return 0;
> +}
> +
> +static int dsi_parse_dt(struct platform_device *pdev, struct dw_dsi *dsi)
> +{
> + struct dsi_hw_ctx *ctx = dsi->ctx;
> + const char *compatible;
> + int ret = 0;
> + struct device_node *np = NULL;
> +
> + if (ctx->g_dss_version_tag == FB_ACCEL_KIRIN970)
> + compatible = "hisilicon,kirin970-dsi";
> + else
> + compatible = "hisilicon,kirin960-dsi";
> +
> + np = of_find_compatible_node(NULL, NULL, compatible);
> + if (!np) {
> + dev_err(&pdev->dev, "NOT FOUND device node %s!\n", compatible);
> + return -ENXIO;
> + }
> +
> + ctx->base = of_iomap(np, 0);
> + if (!(ctx->base)) {
> + dev_err(&pdev->dev, "failed to get dsi base resource.\n");
> + return -ENXIO;
> + }
> +
> + ctx->peri_crg_base = of_iomap(np, 1);
> + if (!(ctx->peri_crg_base)) {
> + dev_err(&pdev->dev, "failed to get peri_crg_base resource.\n");
> + return -ENXIO;
> + }
> +
> + if (ctx->g_dss_version_tag == FB_ACCEL_KIRIN970) {
> + ctx->pctrl_base = of_iomap(np, 2);
> + if (!(ctx->pctrl_base)) {
> + dev_err(&pdev->dev,
> + "failed to get dss pctrl_base resource.\n");
> + return -ENXIO;
> + }
> + }
> +
> + dsi->gpio_mux = devm_gpiod_get(&pdev->dev, "mux", GPIOD_OUT_HIGH);
> + if (IS_ERR(dsi->gpio_mux))
> + return PTR_ERR(dsi->gpio_mux);
> +
> + /* set dsi default output to panel */
> + dsi->cur_client = OUT_PANEL;
> + dsi->attached_client = dsi->cur_client;
> +
> + DRM_INFO("dsi cur_client is %d <0->hdmi;1->panel>\n",
> + dsi->cur_client);
> + /* dis-reset */
> + /* ip_reset_dis_dsi0, ip_reset_dis_dsi1 */
> + writel(0x30000000, ctx->peri_crg_base + PERRSTDIS3);
> +
> + ctx->dss_dphy0_ref_clk = devm_clk_get(&pdev->dev, "clk_txdphy0_ref");
> + if (IS_ERR(ctx->dss_dphy0_ref_clk)) {
> + dev_err(&pdev->dev, "failed to get dss_dphy0_ref_clk clock\n");
> + return PTR_ERR(ctx->dss_dphy0_ref_clk);
> + }
> +
> + ret = clk_set_rate(ctx->dss_dphy0_ref_clk, DEFAULT_MIPI_CLK_RATE);
> + if (ret < 0) {
> + dev_err(&pdev->dev, "dss_dphy0_ref_clk clk_set_rate(%lu) failed, error=%d!\n",
> + DEFAULT_MIPI_CLK_RATE, ret);
> + return -EINVAL;
> + }
> +
> + DRM_DEBUG("dss_dphy0_ref_clk:[%lu]->[%lu].\n",
> + DEFAULT_MIPI_CLK_RATE, clk_get_rate(ctx->dss_dphy0_ref_clk));
> +
> + ctx->dss_dphy0_cfg_clk = devm_clk_get(&pdev->dev, "clk_txdphy0_cfg");
> + if (IS_ERR(ctx->dss_dphy0_cfg_clk)) {
> + dev_err(&pdev->dev, "failed to get dss_dphy0_cfg_clk clock\n");
> + return PTR_ERR(ctx->dss_dphy0_cfg_clk);
> + }
> +
> + ret = clk_set_rate(ctx->dss_dphy0_cfg_clk, DEFAULT_MIPI_CLK_RATE);
> + if (ret < 0) {
> + dev_err(&pdev->dev, "dss_dphy0_cfg_clk clk_set_rate(%lu) failed, error=%d!\n",
> + DEFAULT_MIPI_CLK_RATE, ret);
> + return -EINVAL;
> + }
> +
> + DRM_DEBUG("dss_dphy0_cfg_clk:[%lu]->[%lu].\n",
> + DEFAULT_MIPI_CLK_RATE, clk_get_rate(ctx->dss_dphy0_cfg_clk));
> +
> + ctx->dss_pclk_dsi0_clk = devm_clk_get(&pdev->dev, "pclk_dsi0");
> + if (IS_ERR(ctx->dss_pclk_dsi0_clk)) {
> + dev_err(&pdev->dev, "failed to get dss_pclk_dsi0_clk clock\n");
> + return PTR_ERR(ctx->dss_pclk_dsi0_clk);
> + }
> +
> + return 0;
> +}
> +
> +static int dsi_probe(struct platform_device *pdev)
> +{
> + struct device_node *np = pdev->dev.of_node;
> + struct device *dev = &pdev->dev;
> + struct dsi_data *data;
> + struct dw_dsi *dsi;
> + struct dsi_hw_ctx *ctx;
> + int ret;
> +
> + data = devm_kzalloc(dev, sizeof(*data), GFP_KERNEL);
> + if (!data) {
> + dev_err(&pdev->dev, "failed to allocate dsi data.\n");
> + return -ENOMEM;
> + }
> + dsi = &data->dsi;
> + ctx = &data->ctx;
> + dsi->ctx = ctx;
> +
> + ctx->g_dss_version_tag = (long)of_device_get_match_data(dev);
> +
> + /* parse HDMI bridge endpoint */
> + ret = dsi_parse_endpoint(dsi, np, OUT_HDMI);
> + if (ret)
> + return ret;
> +
> + ret = dsi_host_init(dev, dsi);
> + if (ret)
> + return ret;
> +
> + /* parse panel endpoint */
> + ret = dsi_parse_endpoint(dsi, np, OUT_PANEL);
> + if (ret)
> + goto err_host_unregister;
> +
> + ret = dsi_parse_dt(pdev, dsi);
> + if (ret)
> + goto err_host_unregister;
> +
> + platform_set_drvdata(pdev, data);
> +
> + ret = component_add(dev, &dsi_ops);
> + if (ret)
> + goto err_host_unregister;
> +
> + return 0;
> +
> +err_host_unregister:
> + mipi_dsi_host_unregister(&dsi->host);
> + return ret;
> +}
> +
> +static int dsi_remove(struct platform_device *pdev)
> +{
> + component_del(&pdev->dev, &dsi_ops);
> +
> + return 0;
> +}
> +
> +static const struct of_device_id dsi_of_match[] = {
> + {
> + .compatible = "hisilicon,kirin960-dsi",
> + .data = (void *)FB_ACCEL_HI366x
> + }, {
> + .compatible = "hisilicon,kirin970-dsi",
> + .data = (void *)FB_ACCEL_KIRIN970
> + },
> + { }
> +};
> +MODULE_DEVICE_TABLE(of, dsi_of_match);
> +
> +static struct platform_driver dsi_driver = {
> + .probe = dsi_probe,
> + .remove = dsi_remove,
> + .driver = {
> + .name = "kirin9xx-dw-dsi",
> + .of_match_table = dsi_of_match,
> + },
> +};
> +
> +module_platform_driver(dsi_driver);
> +
> +MODULE_DESCRIPTION("DesignWare MIPI DSI Host Controller v1.02 driver");
> +MODULE_LICENSE("GPL v2");
> diff --git a/drivers/staging/hikey9xx/gpu/kirin9xx_dw_dsi_reg.h b/drivers/staging/hikey9xx/gpu/kirin9xx_dw_dsi_reg.h
> new file mode 100644
> index 000000000000..0e3971ca328c
> --- /dev/null
> +++ b/drivers/staging/hikey9xx/gpu/kirin9xx_dw_dsi_reg.h
> @@ -0,0 +1,142 @@
> +/* SPDX-License-Identifier: GPL-2.0 */
> +/*
> + * Copyright (c) 2016 Linaro Limited.
> + * Copyright (c) 2014-2016 Hisilicon Limited.
> + * Copyright (c) 2014-2020, Huawei Technologies Co., Ltd
> + */
> +
> +#ifndef __DW_DSI_REG_H__
> +#define __DW_DSI_REG_H__
> +
> +#define MASK(x) (BIT(x) - 1)
Not used, delete.

> +#define DEFAULT_MAX_TX_ESC_CLK (10 * 1000000UL)
> +/*
> + * regs
> + */
> +#define PWR_UP 0x04 /* Core power-up */
> +#define RESET 0
> +#define POWERUP BIT(0)
> +#define PHY_IF_CFG 0xA4 /* D-PHY interface configuration */
> +#define CLKMGR_CFG 0x08 /* the internal clock dividers */
> +#define PHY_RSTZ 0xA0 /* D-PHY reset control */
> +#define PHY_ENABLECLK BIT(2)
> +#define PHY_UNRSTZ BIT(1)
> +#define PHY_UNSHUTDOWNZ BIT(0)
> +#define PHY_TST_CTRL0 0xB4 /* D-PHY test interface control 0 */
> +#define PHY_TST_CTRL1 0xB8 /* D-PHY test interface control 1 */
> +#define CLK_TLPX 0x10
> +#define CLK_THS_PREPARE 0x11
> +#define CLK_THS_ZERO 0x12
> +#define CLK_THS_TRAIL 0x13
> +#define CLK_TWAKEUP 0x14
> +#define DATA_TLPX(x) (0x20 + ((x) << 4))
> +#define DATA_THS_PREPARE(x) (0x21 + ((x) << 4))
> +#define DATA_THS_ZERO(x) (0x22 + ((x) << 4))
> +#define DATA_THS_TRAIL(x) (0x23 + ((x) << 4))
> +#define DATA_TTA_GO(x) (0x24 + ((x) << 4))
> +#define DATA_TTA_GET(x) (0x25 + ((x) << 4))
> +#define DATA_TWAKEUP(x) (0x26 + ((x) << 4))
> +#define PHY_CFG_I 0x60
> +#define PHY_CFG_PLL_I 0x63
> +#define PHY_CFG_PLL_II 0x64
> +#define PHY_CFG_PLL_III 0x65
> +#define PHY_CFG_PLL_IV 0x66
> +#define PHY_CFG_PLL_V 0x67
> +#define DPI_COLOR_CODING 0x10 /* DPI color coding */
> +#define DPI_CFG_POL 0x14 /* DPI polarity configuration */
> +#define VID_HSA_TIME 0x48 /* Horizontal Sync Active time */
> +#define VID_HBP_TIME 0x4C /* Horizontal Back Porch time */
> +#define VID_HLINE_TIME 0x50 /* Line time */
> +#define VID_VSA_LINES 0x54 /* Vertical Sync Active period */
> +#define VID_VBP_LINES 0x58 /* Vertical Back Porch period */
> +#define VID_VFP_LINES 0x5C /* Vertical Front Porch period */
> +#define VID_VACTIVE_LINES 0x60 /* Vertical resolution */
> +#define VID_PKT_SIZE 0x3C /* Video packet size */
> +#define VID_MODE_CFG 0x38 /* Video mode configuration */
> +#define GEN_HDR 0x6c
> +#define GEN_HDATA(data) (((data) & 0xffff) << 8)
> +#define GEN_HDATA_MASK (0xffff << 8)
> +#define GEN_HTYPE(type) (((type) & 0xff) << 0)
> +#define GEN_HTYPE_MASK 0xff
> +#define GEN_PLD_DATA 0x70
> +#define CMD_PKT_STATUS 0x74
> +#define GEN_CMD_EMPTY BIT(0)
> +#define GEN_CMD_FULL BIT(1)
> +#define GEN_PLD_W_EMPTY BIT(2)
> +#define GEN_PLD_W_FULL BIT(3)
> +#define GEN_PLD_R_EMPTY BIT(4)
> +#define GEN_PLD_R_FULL BIT(5)
> +#define GEN_RD_CMD_BUSY BIT(6)
> +#define CMD_MODE_CFG 0x68
> +#define MAX_RD_PKT_SIZE_LP BIT(24)
> +#define DCS_LW_TX_LP BIT(19)
> +#define DCS_SR_0P_TX_LP BIT(18)
> +#define DCS_SW_1P_TX_LP BIT(17)
> +#define DCS_SW_0P_TX_LP BIT(16)
> +#define GEN_LW_TX_LP BIT(14)
> +#define GEN_SR_2P_TX_LP BIT(13)
> +#define GEN_SR_1P_TX_LP BIT(12)
> +#define GEN_SR_0P_TX_LP BIT(11)
> +#define GEN_SW_2P_TX_LP BIT(10)
> +#define GEN_SW_1P_TX_LP BIT(9)
> +#define GEN_SW_0P_TX_LP BIT(8)
> +#define EN_ACK_RQST BIT(1)
> +#define EN_TEAR_FX BIT(0)
> +#define CMD_MODE_ALL_LP (MAX_RD_PKT_SIZE_LP | \
> + DCS_LW_TX_LP | \
> + DCS_SR_0P_TX_LP | \
> + DCS_SW_1P_TX_LP | \
> + DCS_SW_0P_TX_LP | \
> + GEN_LW_TX_LP | \
> + GEN_SR_2P_TX_LP | \
> + GEN_SR_1P_TX_LP | \
> + GEN_SR_0P_TX_LP | \
> + GEN_SW_2P_TX_LP | \
> + GEN_SW_1P_TX_LP | \
> + GEN_SW_0P_TX_LP)
> +#define PHY_TMR_CFG 0x9C /* Data lanes timing configuration */
> +#define BTA_TO_CNT 0x8C /* Response timeout definition */
> +#define PHY_TMR_LPCLK_CFG 0x98 /* clock lane timing configuration */
> +#define CLK_DATA_TMR_CFG 0xCC
> +#define LPCLK_CTRL 0x94 /* Low-power in clock lane */
> +#define PHY_TXREQUESTCLKHS BIT(0)
> +#define MODE_CFG 0x34 /* Video or Command mode selection */
> +#define PHY_STATUS 0xB0 /* D-PHY PPI status interface */
> +
> +#define PHY_STOP_WAIT_TIME 0x30
> +#define CMD_PKT_STATUS_TIMEOUT_US 20000
> +
> +/*
> + * regs relevant enum
> + */
> +enum dpi_color_coding {
> + DSI_24BITS_1 = 5,
> +};
> +
> +enum dsi_video_mode_type {
> + DSI_NON_BURST_SYNC_PULSES = 0,
> + DSI_NON_BURST_SYNC_EVENTS,
> + DSI_BURST_SYNC_PULSES_1,
> + DSI_BURST_SYNC_PULSES_2
> +};
> +
> +enum dsi_work_mode {
> + DSI_VIDEO_MODE = 0,
> + DSI_COMMAND_MODE
> +};
> +
> +/*
> + * Register Write/Read Helper functions
> + */
> +static inline void dw_update_bits(void __iomem *addr, u32 bit_start,
> + u32 mask, u32 val)

Not used I think, so delete if not.

> +{
> + u32 tmp, orig;
> +
> + orig = readl(addr);
> + tmp = orig & ~(mask << bit_start);
> + tmp |= (val & mask) << bit_start;
> + writel(tmp, addr);
> +}
> +
> +#endif /* __DW_DRM_DSI_H__ */

Next message: kernel test robot: "drivers/gpu/drm/exynos/exynos_drm_dma.c:146:19: warning: variable 'mapping' is uninitialized when used here"
Previous message: Vineet Gupta: "[PATCH v2] irqchip/eznps: Fix build error for !ARC700 builds"
In reply to: Sam Ravnborg: "Re: [PATCH 00/49] DRM driver for Hikey 970"
Next in thread: Sam Ravnborg: "Re: [PATCH 00/49] DRM driver for Hikey 970"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]