From: Adrian Ratiu <adrian.ratiu@collabora.com>
To: adrian61 <pop.adrian61@gmail.com>
Cc: linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org,
Jernej Skrabec <jernej.skrabec@siol.net>,
Andrzej Hajda <a.hajda@samsung.com>,
Jonas Karlman <jonas@kwiboo.se>,
linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org,
linux-rockchip@lists.infradead.org, linux-imx@nxp.com,
kernel@collabora.com, linux-stm32@st-md-mailman.stormreply.com,
Laurent Pinchart <Laurent.pinchart@ideasonboard.com>
Subject: Re: [PATCH v5 3/5] drm: bridge: synopsis: add dsi v1.01 support
Date: Tue, 31 Mar 2020 00:18:20 +0300 [thread overview]
Message-ID: <87d08t35k3.fsf@collabora.com> (raw)
In-Reply-To: <CAP-HsdSXPBMG5vJwtpXBuZy9zp8HA+UvYPR5hyvdyjZfz-1dww@mail.gmail.com>
On Mon, 30 Mar 2020, adrian61 <pop.adrian61@gmail.com> wrote:
> Hello Adrian,
>
> Here i get a compile error:
I neglected to test with CONFIG_DEBUG_FS, oops!
Will fix in v6, thanks!
>
> On Mon, Mar 30, 2020 at 2:36 PM Adrian Ratiu <adrian.ratiu@collabora.com> wrote:
>>
>> The Synopsis MIPI DSI v1.01 host controller is quite widely used
>> on platforms like i.mx6 and is not very different from the other
>> versions like the 1.31/1.30 used on rockchip/stm. The protocols
>> appear to be the same, only the register layout is different and
>> the newer versions have new features symbolized by new registers
>> so adding support for it is just a matter of defining the new
>> layout and adding a couple of dsi version checks.
>>
>> Signed-off-by: Adrian Ratiu <adrian.ratiu@collabora.com>
>> ---
>> New in v5.
>> ---
>> drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c | 125 +++++++++++++++++-
>> 1 file changed, 119 insertions(+), 6 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c b/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c
>> index 5b78ff925af0..fb9dbc4fd0f5 100644
>> --- a/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c
>> +++ b/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c
>> @@ -32,6 +32,7 @@
>>
>> #define HWVER_131 0x31333100 /* IP version 1.31 */
>> #define HWVER_130 0x31333000 /* IP version 1.30 */
>> +#define HWVER_101 0x31303000 /* IP version 1.01 */
>>
>> #define DSI_VERSION 0x00
>> #define VERSION GENMASK(31, 8)
>> @@ -100,6 +101,25 @@
>> #define DSI_EDPI_CMD_SIZE 0x64
>>
>> #define DSI_CMD_MODE_CFG 0x68
>> +
>> +#define DSI_DPI_CFG 0x0c
>> +#define DSI_TMR_LINE_CFG 0x28
>> +#define DSI_VTIMING_CFG 0x2c
>> +#define DSI_PHY_TMR_CFG_V101 0x30
>> +#define DSI_PHY_IF_CFG_V101 0x58
>> +#define DSI_PHY_IF_CTRL 0x5c
>> +#define DSI_PHY_RSTZ_V101 0x54
>> +#define DSI_PHY_STATUS_V101 0x60
>> +#define DSI_PHY_TST_CTRL0_V101 0x64
>> +#define DSI_GEN_HDR_V101 0x34
>> +#define DSI_GEN_PLD_DATA_V101 0x38
>> +#define DSI_CMD_MODE_CFG_V101 0x24
>> +#define DSI_CMD_PKT_STATUS_V101 0x3c
>> +#define DSI_VID_PKT_CFG 0x20
>> +#define DSI_VID_MODE_CFG_V101 0x1c
>> +#define DSI_TO_CNT_CFG_V101 0x40
>> +#define DSI_PCKHDL_CFG_V101 0x18
>> +
>> #define MAX_RD_PKT_SIZE_LP BIT(24)
>> #define DCS_LW_TX_LP BIT(19)
>> #define DCS_SR_0P_TX_LP BIT(18)
>> @@ -127,6 +147,33 @@
>> GEN_SW_1P_TX_LP | \
>> GEN_SW_0P_TX_LP)
>>
>> +#define EN_TEAR_FX_V101 BIT(14)
>> +#define DCS_LW_TX_LP_V101 BIT(12)
>> +#define GEN_LW_TX_LP_V101 BIT(11)
>> +#define MAX_RD_PKT_SIZE_LP_V101 BIT(10)
>> +#define DCS_SW_2P_TX_LP_V101 BIT(9)
>> +#define DCS_SW_1P_TX_LP_V101 BIT(8)
>> +#define DCS_SW_0P_TX_LP_V101 BIT(7)
>> +#define GEN_SR_2P_TX_LP_V101 BIT(6)
>> +#define GEN_SR_1P_TX_LP_V101 BIT(5)
>> +#define GEN_SR_0P_TX_LP_V101 BIT(4)
>> +#define GEN_SW_2P_TX_LP_V101 BIT(3)
>> +#define GEN_SW_1P_TX_LP_V101 BIT(2)
>> +#define GEN_SW_0P_TX_LP_V101 BIT(1)
>> +
>> +#define CMD_MODE_ALL_LP_V101 (DCS_LW_TX_LP_V101 | \
>> + GEN_LW_TX_LP_V101 | \
>> + MAX_RD_PKT_SIZE_LP_V101 | \
>> + DCS_SW_2P_TX_LP_V101 | \
>> + DCS_SW_1P_TX_LP_V101 | \
>> + DCS_SW_0P_TX_LP_V101 | \
>> + GEN_SR_2P_TX_LP_V101 | \
>> + GEN_SR_1P_TX_LP_V101 | \
>> + GEN_SR_0P_TX_LP_V101 | \
>> + GEN_SW_2P_TX_LP_V101 | \
>> + GEN_SW_1P_TX_LP_V101 | \
>> + GEN_SW_0P_TX_LP_V101)
>> +
>> #define DSI_GEN_HDR 0x6c
>> #define DSI_GEN_PLD_DATA 0x70
>>
>> @@ -165,6 +212,11 @@
>> #define DSI_INT_MSK0 0xc4
>> #define DSI_INT_MSK1 0xc8
>>
>> +#define DSI_ERROR_ST0_V101 0x44
>> +#define DSI_ERROR_ST1_V101 0x48
>> +#define DSI_ERROR_MSK0_V101 0x4c
>> +#define DSI_ERROR_MSK1_V101 0x50
>> +
>> #define DSI_PHY_TMR_RD_CFG 0xf4
>>
>> #define PHY_STATUS_TIMEOUT_US 10000
>> @@ -359,6 +411,49 @@ static const struct dw_mipi_dsi_variant dw_mipi_dsi_v130_v131_layout = {
>> .cfg_gen_payload = REG_FIELD(DSI_GEN_PLD_DATA, 0, 31),
>> };
>>
>> +static const struct dw_mipi_dsi_variant dw_mipi_dsi_v101_layout = {
>> + .cfg_dpi_vid = REG_FIELD(DSI_DPI_CFG, 0, 1),
>> + .cfg_dpi_color_coding = REG_FIELD(DSI_DPI_CFG, 2, 4),
>> + .cfg_dpi_18loosely_en = REG_FIELD(DSI_DPI_CFG, 10, 10),
>> + .cfg_dpi_vsync_active_low = REG_FIELD(DSI_DPI_CFG, 6, 6),
>> + .cfg_dpi_hsync_active_low = REG_FIELD(DSI_DPI_CFG, 7, 7),
>> + .cfg_cmd_mode_en = REG_FIELD(DSI_CMD_MODE_CFG_V101, 0, 0),
>> + .cfg_cmd_mode_all_lp_en = REG_FIELD(DSI_CMD_MODE_CFG_V101, 1, 12),
>> + .cfg_cmd_mode_ack_rqst_en = REG_FIELD(DSI_CMD_MODE_CFG_V101, 13, 13),
>> + .cfg_cmd_pkt_status = REG_FIELD(DSI_CMD_PKT_STATUS_V101, 0, 14),
>> + .cfg_vid_mode_en = REG_FIELD(DSI_VID_MODE_CFG_V101, 0, 0),
>> + .cfg_vid_mode_type = REG_FIELD(DSI_VID_MODE_CFG_V101, 1, 2),
>> + .cfg_vid_mode_low_power = REG_FIELD(DSI_VID_MODE_CFG_V101, 3, 8),
>> + .cfg_vid_pkt_size = REG_FIELD(DSI_VID_PKT_CFG, 0, 10),
>> + .cfg_vid_hsa_time = REG_FIELD(DSI_TMR_LINE_CFG, 0, 8),
>> + .cfg_vid_hbp_time = REG_FIELD(DSI_TMR_LINE_CFG, 9, 17),
>> + .cfg_vid_hline_time = REG_FIELD(DSI_TMR_LINE_CFG, 18, 31),
>> + .cfg_vid_vsa_time = REG_FIELD(DSI_VTIMING_CFG, 0, 3),
>> + .cfg_vid_vbp_time = REG_FIELD(DSI_VTIMING_CFG, 4, 9),
>> + .cfg_vid_vfp_time = REG_FIELD(DSI_VTIMING_CFG, 10, 15),
>> + .cfg_vid_vactive_time = REG_FIELD(DSI_VTIMING_CFG, 16, 26),
>> + .cfg_phy_txrequestclkhs = REG_FIELD(DSI_PHY_IF_CTRL, 0, 0),
>> + .cfg_phy_bta_time = REG_FIELD(DSI_PHY_TMR_CFG_V101, 0, 11),
>> + .cfg_phy_lp2hs_time = REG_FIELD(DSI_PHY_TMR_CFG_V101, 12, 19),
>> + .cfg_phy_hs2lp_time = REG_FIELD(DSI_PHY_TMR_CFG_V101, 20, 27),
>> + .cfg_phy_testclr = REG_FIELD(DSI_PHY_TST_CTRL0_V101, 0, 0),
>> + .cfg_phy_unshutdownz = REG_FIELD(DSI_PHY_RSTZ_V101, 0, 0),
>> + .cfg_phy_unrstz = REG_FIELD(DSI_PHY_RSTZ_V101, 1, 1),
>> + .cfg_phy_enableclk = REG_FIELD(DSI_PHY_RSTZ_V101, 2, 2),
>> + .cfg_phy_nlanes = REG_FIELD(DSI_PHY_IF_CFG_V101, 0, 1),
>> + .cfg_phy_stop_wait_time = REG_FIELD(DSI_PHY_IF_CFG_V101, 2, 9),
>> + .cfg_phy_status = REG_FIELD(DSI_PHY_STATUS_V101, 0, 0),
>> + .cfg_pckhdl_cfg = REG_FIELD(DSI_PCKHDL_CFG_V101, 0, 4),
>> + .cfg_hstx_timeout_counter = REG_FIELD(DSI_TO_CNT_CFG_V101, 0, 15),
>> + .cfg_lprx_timeout_counter = REG_FIELD(DSI_TO_CNT_CFG_V101, 16, 31),
>> + .cfg_int_stat0 = REG_FIELD(DSI_ERROR_ST0_V101, 0, 20),
>> + .cfg_int_stat1 = REG_FIELD(DSI_ERROR_ST1_V101, 0, 17),
>> + .cfg_int_mask0 = REG_FIELD(DSI_ERROR_MSK0_V101, 0, 20),
>> + .cfg_int_mask1 = REG_FIELD(DSI_ERROR_MSK1_V101, 0, 17),
>> + .cfg_gen_hdr = REG_FIELD(DSI_GEN_HDR_V101, 0, 31),
>> + .cfg_gen_payload = REG_FIELD(DSI_GEN_PLD_DATA_V101, 0, 31),
>> +};
>> +
>> /*
>> * Check if either a link to a master or slave is present
>> */
>> @@ -466,6 +561,9 @@ static void dw_mipi_message_config(struct dw_mipi_dsi *dsi,
>> case HWVER_131:
>> cmd_mode_lp = CMD_MODE_ALL_LP;
>> break;
>> + case HWVER_101:
>> + cmd_mode_lp = CMD_MODE_ALL_LP_V101;
>> + break;
>> }
>>
>> if (msg->flags & MIPI_DSI_MSG_REQ_ACK)
>> @@ -644,7 +742,7 @@ static void dw_mipi_dsi_video_mode_config(struct dw_mipi_dsi *dsi)
>> VID_MODE_TYPE_NON_BURST_SYNC_EVENTS);
>>
>> #ifdef CONFIG_DEBUG_FS
>> - if (dsi->vpg) {
>> + if (dsi->hw_version > HWVER_101 && dsi->vpg) {
>> regmap_field_write(dsi->regs, dsi->field_vid_mode_vpg_en, 1);
>> regmap_field_write(dsi->regs, dsi->field_vid_mode_vpg_horiz,
>> dsi->vpg_horizontal ? 1 : 0);
>
> "regmap_field_write" passed 3 arguments, but takes just 2.
>
>
>> @@ -662,9 +760,15 @@ static void dw_mipi_dsi_set_mode(struct dw_mipi_dsi *dsi,
>>
>> dw_mipi_dsi_video_mode_config(dsi);
>>
>> + if (dsi->hw_version == HWVER_101)
>> + regmap_field_write(dsi->field_vid_mode_en, 1);
>> +
>> regmap_field_write(dsi->field_phy_txrequestclkhs, 1);
>> } else {
>> regmap_field_write(dsi->field_cmd_mode_en, 1);
>> +
>> + if (dsi->hw_version == HWVER_101)
>> + regmap_field_write(dsi->field_vid_mode_en, 0);
>> }
>>
>> regmap_write(dsi->regs, DSI_PWR_UP, POWERUP);
>> @@ -857,9 +961,13 @@ static void dw_mipi_dsi_dphy_timing_config(struct dw_mipi_dsi *dsi)
>> regmap_field_write(dsi->field_phy_lp2hs_time, timing.data_lp2hs);
>> regmap_field_write(dsi->field_phy_hs2lp_time, timing.data_hs2lp);
>>
>> - regmap_field_write(dsi->field_phy_max_rd_time, 10000);
>> - regmap_field_write(dsi->field_phy_clkhs2lp_time, timing.clk_hs2lp);
>> - regmap_field_write(dsi->field_phy_clklp2hs_time, timing.clk_lp2hs);
>> + if (dsi->hw_version > HWVER_101) {
>> + regmap_field_write(dsi->field_phy_max_rd_time, 10000);
>> + regmap_field_write(dsi->field_phy_clkhs2lp_time,
>> + timing.clk_hs2lp);
>> + regmap_field_write(dsi->field_phy_clklp2hs_time,
>> + timing.clk_lp2hs);
>> + }
>> }
>>
>> static void dw_mipi_dsi_dphy_interface_config(struct dw_mipi_dsi *dsi)
>> @@ -880,7 +988,8 @@ static void dw_mipi_dsi_dphy_init(struct dw_mipi_dsi *dsi)
>> regmap_field_write(dsi->field_phy_unrstz, 0);
>> regmap_field_write(dsi->field_phy_unshutdownz, 0);
>>
>> - regmap_field_write(dsi->field_phy_forcepll, 0);
>> + if (dsi->hw_version > HWVER_101)
>> + regmap_field_write(dsi->field_phy_forcepll, 0);
>>
>> regmap_field_write(dsi->field_phy_testclr, 0);
>> regmap_field_write(dsi->field_phy_testclr, 1);
>> @@ -896,7 +1005,8 @@ static void dw_mipi_dsi_dphy_enable(struct dw_mipi_dsi *dsi)
>> regmap_field_write(dsi->field_phy_unrstz, 1);
>> regmap_field_write(dsi->field_phy_unshutdownz, 1);
>>
>> - regmap_field_write(dsi->field_phy_forcepll, 1);
>> + if (dsi->hw_version > HWVER_101)
>> + regmap_field_write(dsi->field_phy_forcepll, 1);
>>
>> ret = regmap_field_read_poll_timeout(dsi->field_phy_status,
>> val, val & PHY_LOCK,
>> @@ -1131,6 +1241,9 @@ static int dw_mipi_dsi_regmap_fields_init(struct dw_mipi_dsi *dsi)
>> case HWVER_131:
>> variant = &dw_mipi_dsi_v130_v131_layout;
>> break;
>> + case HWVER_101:
>> + variant = &dw_mipi_dsi_v101_layout;
>> + break;
>> default:
>> DRM_ERROR("Unrecognized DSI host controller HW revision\n");
>> return -ENODEV;
>> --
>> 2.26.0
>>
>>
>> _______________________________________________
>> linux-arm-kernel mailing list
>> linux-arm-kernel@lists.infradead.org
>> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>
> Regards,
> Adrian
WARNING: multiple messages have this Message-ID (diff)
From: Adrian Ratiu <adrian.ratiu@collabora.com>
To: adrian61 <pop.adrian61@gmail.com>
Cc: devicetree@vger.kernel.org,
Jernej Skrabec <jernej.skrabec@siol.net>,
Jonas Karlman <jonas@kwiboo.se>,
linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org,
Andrzej Hajda <a.hajda@samsung.com>,
linux-imx@nxp.com, linux-rockchip@lists.infradead.org,
kernel@collabora.com, linux-stm32@st-md-mailman.stormreply.com,
linux-arm-kernel@lists.infradead.org,
Laurent Pinchart <Laurent.pinchart@ideasonboard.com>
Subject: Re: [PATCH v5 3/5] drm: bridge: synopsis: add dsi v1.01 support
Date: Tue, 31 Mar 2020 00:18:20 +0300 [thread overview]
Message-ID: <87d08t35k3.fsf@collabora.com> (raw)
In-Reply-To: <CAP-HsdSXPBMG5vJwtpXBuZy9zp8HA+UvYPR5hyvdyjZfz-1dww@mail.gmail.com>
On Mon, 30 Mar 2020, adrian61 <pop.adrian61@gmail.com> wrote:
> Hello Adrian,
>
> Here i get a compile error:
I neglected to test with CONFIG_DEBUG_FS, oops!
Will fix in v6, thanks!
>
> On Mon, Mar 30, 2020 at 2:36 PM Adrian Ratiu <adrian.ratiu@collabora.com> wrote:
>>
>> The Synopsis MIPI DSI v1.01 host controller is quite widely used
>> on platforms like i.mx6 and is not very different from the other
>> versions like the 1.31/1.30 used on rockchip/stm. The protocols
>> appear to be the same, only the register layout is different and
>> the newer versions have new features symbolized by new registers
>> so adding support for it is just a matter of defining the new
>> layout and adding a couple of dsi version checks.
>>
>> Signed-off-by: Adrian Ratiu <adrian.ratiu@collabora.com>
>> ---
>> New in v5.
>> ---
>> drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c | 125 +++++++++++++++++-
>> 1 file changed, 119 insertions(+), 6 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c b/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c
>> index 5b78ff925af0..fb9dbc4fd0f5 100644
>> --- a/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c
>> +++ b/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c
>> @@ -32,6 +32,7 @@
>>
>> #define HWVER_131 0x31333100 /* IP version 1.31 */
>> #define HWVER_130 0x31333000 /* IP version 1.30 */
>> +#define HWVER_101 0x31303000 /* IP version 1.01 */
>>
>> #define DSI_VERSION 0x00
>> #define VERSION GENMASK(31, 8)
>> @@ -100,6 +101,25 @@
>> #define DSI_EDPI_CMD_SIZE 0x64
>>
>> #define DSI_CMD_MODE_CFG 0x68
>> +
>> +#define DSI_DPI_CFG 0x0c
>> +#define DSI_TMR_LINE_CFG 0x28
>> +#define DSI_VTIMING_CFG 0x2c
>> +#define DSI_PHY_TMR_CFG_V101 0x30
>> +#define DSI_PHY_IF_CFG_V101 0x58
>> +#define DSI_PHY_IF_CTRL 0x5c
>> +#define DSI_PHY_RSTZ_V101 0x54
>> +#define DSI_PHY_STATUS_V101 0x60
>> +#define DSI_PHY_TST_CTRL0_V101 0x64
>> +#define DSI_GEN_HDR_V101 0x34
>> +#define DSI_GEN_PLD_DATA_V101 0x38
>> +#define DSI_CMD_MODE_CFG_V101 0x24
>> +#define DSI_CMD_PKT_STATUS_V101 0x3c
>> +#define DSI_VID_PKT_CFG 0x20
>> +#define DSI_VID_MODE_CFG_V101 0x1c
>> +#define DSI_TO_CNT_CFG_V101 0x40
>> +#define DSI_PCKHDL_CFG_V101 0x18
>> +
>> #define MAX_RD_PKT_SIZE_LP BIT(24)
>> #define DCS_LW_TX_LP BIT(19)
>> #define DCS_SR_0P_TX_LP BIT(18)
>> @@ -127,6 +147,33 @@
>> GEN_SW_1P_TX_LP | \
>> GEN_SW_0P_TX_LP)
>>
>> +#define EN_TEAR_FX_V101 BIT(14)
>> +#define DCS_LW_TX_LP_V101 BIT(12)
>> +#define GEN_LW_TX_LP_V101 BIT(11)
>> +#define MAX_RD_PKT_SIZE_LP_V101 BIT(10)
>> +#define DCS_SW_2P_TX_LP_V101 BIT(9)
>> +#define DCS_SW_1P_TX_LP_V101 BIT(8)
>> +#define DCS_SW_0P_TX_LP_V101 BIT(7)
>> +#define GEN_SR_2P_TX_LP_V101 BIT(6)
>> +#define GEN_SR_1P_TX_LP_V101 BIT(5)
>> +#define GEN_SR_0P_TX_LP_V101 BIT(4)
>> +#define GEN_SW_2P_TX_LP_V101 BIT(3)
>> +#define GEN_SW_1P_TX_LP_V101 BIT(2)
>> +#define GEN_SW_0P_TX_LP_V101 BIT(1)
>> +
>> +#define CMD_MODE_ALL_LP_V101 (DCS_LW_TX_LP_V101 | \
>> + GEN_LW_TX_LP_V101 | \
>> + MAX_RD_PKT_SIZE_LP_V101 | \
>> + DCS_SW_2P_TX_LP_V101 | \
>> + DCS_SW_1P_TX_LP_V101 | \
>> + DCS_SW_0P_TX_LP_V101 | \
>> + GEN_SR_2P_TX_LP_V101 | \
>> + GEN_SR_1P_TX_LP_V101 | \
>> + GEN_SR_0P_TX_LP_V101 | \
>> + GEN_SW_2P_TX_LP_V101 | \
>> + GEN_SW_1P_TX_LP_V101 | \
>> + GEN_SW_0P_TX_LP_V101)
>> +
>> #define DSI_GEN_HDR 0x6c
>> #define DSI_GEN_PLD_DATA 0x70
>>
>> @@ -165,6 +212,11 @@
>> #define DSI_INT_MSK0 0xc4
>> #define DSI_INT_MSK1 0xc8
>>
>> +#define DSI_ERROR_ST0_V101 0x44
>> +#define DSI_ERROR_ST1_V101 0x48
>> +#define DSI_ERROR_MSK0_V101 0x4c
>> +#define DSI_ERROR_MSK1_V101 0x50
>> +
>> #define DSI_PHY_TMR_RD_CFG 0xf4
>>
>> #define PHY_STATUS_TIMEOUT_US 10000
>> @@ -359,6 +411,49 @@ static const struct dw_mipi_dsi_variant dw_mipi_dsi_v130_v131_layout = {
>> .cfg_gen_payload = REG_FIELD(DSI_GEN_PLD_DATA, 0, 31),
>> };
>>
>> +static const struct dw_mipi_dsi_variant dw_mipi_dsi_v101_layout = {
>> + .cfg_dpi_vid = REG_FIELD(DSI_DPI_CFG, 0, 1),
>> + .cfg_dpi_color_coding = REG_FIELD(DSI_DPI_CFG, 2, 4),
>> + .cfg_dpi_18loosely_en = REG_FIELD(DSI_DPI_CFG, 10, 10),
>> + .cfg_dpi_vsync_active_low = REG_FIELD(DSI_DPI_CFG, 6, 6),
>> + .cfg_dpi_hsync_active_low = REG_FIELD(DSI_DPI_CFG, 7, 7),
>> + .cfg_cmd_mode_en = REG_FIELD(DSI_CMD_MODE_CFG_V101, 0, 0),
>> + .cfg_cmd_mode_all_lp_en = REG_FIELD(DSI_CMD_MODE_CFG_V101, 1, 12),
>> + .cfg_cmd_mode_ack_rqst_en = REG_FIELD(DSI_CMD_MODE_CFG_V101, 13, 13),
>> + .cfg_cmd_pkt_status = REG_FIELD(DSI_CMD_PKT_STATUS_V101, 0, 14),
>> + .cfg_vid_mode_en = REG_FIELD(DSI_VID_MODE_CFG_V101, 0, 0),
>> + .cfg_vid_mode_type = REG_FIELD(DSI_VID_MODE_CFG_V101, 1, 2),
>> + .cfg_vid_mode_low_power = REG_FIELD(DSI_VID_MODE_CFG_V101, 3, 8),
>> + .cfg_vid_pkt_size = REG_FIELD(DSI_VID_PKT_CFG, 0, 10),
>> + .cfg_vid_hsa_time = REG_FIELD(DSI_TMR_LINE_CFG, 0, 8),
>> + .cfg_vid_hbp_time = REG_FIELD(DSI_TMR_LINE_CFG, 9, 17),
>> + .cfg_vid_hline_time = REG_FIELD(DSI_TMR_LINE_CFG, 18, 31),
>> + .cfg_vid_vsa_time = REG_FIELD(DSI_VTIMING_CFG, 0, 3),
>> + .cfg_vid_vbp_time = REG_FIELD(DSI_VTIMING_CFG, 4, 9),
>> + .cfg_vid_vfp_time = REG_FIELD(DSI_VTIMING_CFG, 10, 15),
>> + .cfg_vid_vactive_time = REG_FIELD(DSI_VTIMING_CFG, 16, 26),
>> + .cfg_phy_txrequestclkhs = REG_FIELD(DSI_PHY_IF_CTRL, 0, 0),
>> + .cfg_phy_bta_time = REG_FIELD(DSI_PHY_TMR_CFG_V101, 0, 11),
>> + .cfg_phy_lp2hs_time = REG_FIELD(DSI_PHY_TMR_CFG_V101, 12, 19),
>> + .cfg_phy_hs2lp_time = REG_FIELD(DSI_PHY_TMR_CFG_V101, 20, 27),
>> + .cfg_phy_testclr = REG_FIELD(DSI_PHY_TST_CTRL0_V101, 0, 0),
>> + .cfg_phy_unshutdownz = REG_FIELD(DSI_PHY_RSTZ_V101, 0, 0),
>> + .cfg_phy_unrstz = REG_FIELD(DSI_PHY_RSTZ_V101, 1, 1),
>> + .cfg_phy_enableclk = REG_FIELD(DSI_PHY_RSTZ_V101, 2, 2),
>> + .cfg_phy_nlanes = REG_FIELD(DSI_PHY_IF_CFG_V101, 0, 1),
>> + .cfg_phy_stop_wait_time = REG_FIELD(DSI_PHY_IF_CFG_V101, 2, 9),
>> + .cfg_phy_status = REG_FIELD(DSI_PHY_STATUS_V101, 0, 0),
>> + .cfg_pckhdl_cfg = REG_FIELD(DSI_PCKHDL_CFG_V101, 0, 4),
>> + .cfg_hstx_timeout_counter = REG_FIELD(DSI_TO_CNT_CFG_V101, 0, 15),
>> + .cfg_lprx_timeout_counter = REG_FIELD(DSI_TO_CNT_CFG_V101, 16, 31),
>> + .cfg_int_stat0 = REG_FIELD(DSI_ERROR_ST0_V101, 0, 20),
>> + .cfg_int_stat1 = REG_FIELD(DSI_ERROR_ST1_V101, 0, 17),
>> + .cfg_int_mask0 = REG_FIELD(DSI_ERROR_MSK0_V101, 0, 20),
>> + .cfg_int_mask1 = REG_FIELD(DSI_ERROR_MSK1_V101, 0, 17),
>> + .cfg_gen_hdr = REG_FIELD(DSI_GEN_HDR_V101, 0, 31),
>> + .cfg_gen_payload = REG_FIELD(DSI_GEN_PLD_DATA_V101, 0, 31),
>> +};
>> +
>> /*
>> * Check if either a link to a master or slave is present
>> */
>> @@ -466,6 +561,9 @@ static void dw_mipi_message_config(struct dw_mipi_dsi *dsi,
>> case HWVER_131:
>> cmd_mode_lp = CMD_MODE_ALL_LP;
>> break;
>> + case HWVER_101:
>> + cmd_mode_lp = CMD_MODE_ALL_LP_V101;
>> + break;
>> }
>>
>> if (msg->flags & MIPI_DSI_MSG_REQ_ACK)
>> @@ -644,7 +742,7 @@ static void dw_mipi_dsi_video_mode_config(struct dw_mipi_dsi *dsi)
>> VID_MODE_TYPE_NON_BURST_SYNC_EVENTS);
>>
>> #ifdef CONFIG_DEBUG_FS
>> - if (dsi->vpg) {
>> + if (dsi->hw_version > HWVER_101 && dsi->vpg) {
>> regmap_field_write(dsi->regs, dsi->field_vid_mode_vpg_en, 1);
>> regmap_field_write(dsi->regs, dsi->field_vid_mode_vpg_horiz,
>> dsi->vpg_horizontal ? 1 : 0);
>
> "regmap_field_write" passed 3 arguments, but takes just 2.
>
>
>> @@ -662,9 +760,15 @@ static void dw_mipi_dsi_set_mode(struct dw_mipi_dsi *dsi,
>>
>> dw_mipi_dsi_video_mode_config(dsi);
>>
>> + if (dsi->hw_version == HWVER_101)
>> + regmap_field_write(dsi->field_vid_mode_en, 1);
>> +
>> regmap_field_write(dsi->field_phy_txrequestclkhs, 1);
>> } else {
>> regmap_field_write(dsi->field_cmd_mode_en, 1);
>> +
>> + if (dsi->hw_version == HWVER_101)
>> + regmap_field_write(dsi->field_vid_mode_en, 0);
>> }
>>
>> regmap_write(dsi->regs, DSI_PWR_UP, POWERUP);
>> @@ -857,9 +961,13 @@ static void dw_mipi_dsi_dphy_timing_config(struct dw_mipi_dsi *dsi)
>> regmap_field_write(dsi->field_phy_lp2hs_time, timing.data_lp2hs);
>> regmap_field_write(dsi->field_phy_hs2lp_time, timing.data_hs2lp);
>>
>> - regmap_field_write(dsi->field_phy_max_rd_time, 10000);
>> - regmap_field_write(dsi->field_phy_clkhs2lp_time, timing.clk_hs2lp);
>> - regmap_field_write(dsi->field_phy_clklp2hs_time, timing.clk_lp2hs);
>> + if (dsi->hw_version > HWVER_101) {
>> + regmap_field_write(dsi->field_phy_max_rd_time, 10000);
>> + regmap_field_write(dsi->field_phy_clkhs2lp_time,
>> + timing.clk_hs2lp);
>> + regmap_field_write(dsi->field_phy_clklp2hs_time,
>> + timing.clk_lp2hs);
>> + }
>> }
>>
>> static void dw_mipi_dsi_dphy_interface_config(struct dw_mipi_dsi *dsi)
>> @@ -880,7 +988,8 @@ static void dw_mipi_dsi_dphy_init(struct dw_mipi_dsi *dsi)
>> regmap_field_write(dsi->field_phy_unrstz, 0);
>> regmap_field_write(dsi->field_phy_unshutdownz, 0);
>>
>> - regmap_field_write(dsi->field_phy_forcepll, 0);
>> + if (dsi->hw_version > HWVER_101)
>> + regmap_field_write(dsi->field_phy_forcepll, 0);
>>
>> regmap_field_write(dsi->field_phy_testclr, 0);
>> regmap_field_write(dsi->field_phy_testclr, 1);
>> @@ -896,7 +1005,8 @@ static void dw_mipi_dsi_dphy_enable(struct dw_mipi_dsi *dsi)
>> regmap_field_write(dsi->field_phy_unrstz, 1);
>> regmap_field_write(dsi->field_phy_unshutdownz, 1);
>>
>> - regmap_field_write(dsi->field_phy_forcepll, 1);
>> + if (dsi->hw_version > HWVER_101)
>> + regmap_field_write(dsi->field_phy_forcepll, 1);
>>
>> ret = regmap_field_read_poll_timeout(dsi->field_phy_status,
>> val, val & PHY_LOCK,
>> @@ -1131,6 +1241,9 @@ static int dw_mipi_dsi_regmap_fields_init(struct dw_mipi_dsi *dsi)
>> case HWVER_131:
>> variant = &dw_mipi_dsi_v130_v131_layout;
>> break;
>> + case HWVER_101:
>> + variant = &dw_mipi_dsi_v101_layout;
>> + break;
>> default:
>> DRM_ERROR("Unrecognized DSI host controller HW revision\n");
>> return -ENODEV;
>> --
>> 2.26.0
>>
>>
>> _______________________________________________
>> linux-arm-kernel mailing list
>> linux-arm-kernel@lists.infradead.org
>> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>
> Regards,
> Adrian
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
WARNING: multiple messages have this Message-ID (diff)
From: Adrian Ratiu <adrian.ratiu@collabora.com>
To: adrian61 <pop.adrian61@gmail.com>
Cc: devicetree@vger.kernel.org,
Jernej Skrabec <jernej.skrabec@siol.net>,
Jonas Karlman <jonas@kwiboo.se>,
linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org,
Andrzej Hajda <a.hajda@samsung.com>,
linux-imx@nxp.com, linux-rockchip@lists.infradead.org,
kernel@collabora.com, linux-stm32@st-md-mailman.stormreply.com,
linux-arm-kernel@lists.infradead.org,
Laurent Pinchart <Laurent.pinchart@ideasonboard.com>
Subject: Re: [PATCH v5 3/5] drm: bridge: synopsis: add dsi v1.01 support
Date: Tue, 31 Mar 2020 00:18:20 +0300 [thread overview]
Message-ID: <87d08t35k3.fsf@collabora.com> (raw)
In-Reply-To: <CAP-HsdSXPBMG5vJwtpXBuZy9zp8HA+UvYPR5hyvdyjZfz-1dww@mail.gmail.com>
On Mon, 30 Mar 2020, adrian61 <pop.adrian61@gmail.com> wrote:
> Hello Adrian,
>
> Here i get a compile error:
I neglected to test with CONFIG_DEBUG_FS, oops!
Will fix in v6, thanks!
>
> On Mon, Mar 30, 2020 at 2:36 PM Adrian Ratiu <adrian.ratiu@collabora.com> wrote:
>>
>> The Synopsis MIPI DSI v1.01 host controller is quite widely used
>> on platforms like i.mx6 and is not very different from the other
>> versions like the 1.31/1.30 used on rockchip/stm. The protocols
>> appear to be the same, only the register layout is different and
>> the newer versions have new features symbolized by new registers
>> so adding support for it is just a matter of defining the new
>> layout and adding a couple of dsi version checks.
>>
>> Signed-off-by: Adrian Ratiu <adrian.ratiu@collabora.com>
>> ---
>> New in v5.
>> ---
>> drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c | 125 +++++++++++++++++-
>> 1 file changed, 119 insertions(+), 6 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c b/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c
>> index 5b78ff925af0..fb9dbc4fd0f5 100644
>> --- a/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c
>> +++ b/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c
>> @@ -32,6 +32,7 @@
>>
>> #define HWVER_131 0x31333100 /* IP version 1.31 */
>> #define HWVER_130 0x31333000 /* IP version 1.30 */
>> +#define HWVER_101 0x31303000 /* IP version 1.01 */
>>
>> #define DSI_VERSION 0x00
>> #define VERSION GENMASK(31, 8)
>> @@ -100,6 +101,25 @@
>> #define DSI_EDPI_CMD_SIZE 0x64
>>
>> #define DSI_CMD_MODE_CFG 0x68
>> +
>> +#define DSI_DPI_CFG 0x0c
>> +#define DSI_TMR_LINE_CFG 0x28
>> +#define DSI_VTIMING_CFG 0x2c
>> +#define DSI_PHY_TMR_CFG_V101 0x30
>> +#define DSI_PHY_IF_CFG_V101 0x58
>> +#define DSI_PHY_IF_CTRL 0x5c
>> +#define DSI_PHY_RSTZ_V101 0x54
>> +#define DSI_PHY_STATUS_V101 0x60
>> +#define DSI_PHY_TST_CTRL0_V101 0x64
>> +#define DSI_GEN_HDR_V101 0x34
>> +#define DSI_GEN_PLD_DATA_V101 0x38
>> +#define DSI_CMD_MODE_CFG_V101 0x24
>> +#define DSI_CMD_PKT_STATUS_V101 0x3c
>> +#define DSI_VID_PKT_CFG 0x20
>> +#define DSI_VID_MODE_CFG_V101 0x1c
>> +#define DSI_TO_CNT_CFG_V101 0x40
>> +#define DSI_PCKHDL_CFG_V101 0x18
>> +
>> #define MAX_RD_PKT_SIZE_LP BIT(24)
>> #define DCS_LW_TX_LP BIT(19)
>> #define DCS_SR_0P_TX_LP BIT(18)
>> @@ -127,6 +147,33 @@
>> GEN_SW_1P_TX_LP | \
>> GEN_SW_0P_TX_LP)
>>
>> +#define EN_TEAR_FX_V101 BIT(14)
>> +#define DCS_LW_TX_LP_V101 BIT(12)
>> +#define GEN_LW_TX_LP_V101 BIT(11)
>> +#define MAX_RD_PKT_SIZE_LP_V101 BIT(10)
>> +#define DCS_SW_2P_TX_LP_V101 BIT(9)
>> +#define DCS_SW_1P_TX_LP_V101 BIT(8)
>> +#define DCS_SW_0P_TX_LP_V101 BIT(7)
>> +#define GEN_SR_2P_TX_LP_V101 BIT(6)
>> +#define GEN_SR_1P_TX_LP_V101 BIT(5)
>> +#define GEN_SR_0P_TX_LP_V101 BIT(4)
>> +#define GEN_SW_2P_TX_LP_V101 BIT(3)
>> +#define GEN_SW_1P_TX_LP_V101 BIT(2)
>> +#define GEN_SW_0P_TX_LP_V101 BIT(1)
>> +
>> +#define CMD_MODE_ALL_LP_V101 (DCS_LW_TX_LP_V101 | \
>> + GEN_LW_TX_LP_V101 | \
>> + MAX_RD_PKT_SIZE_LP_V101 | \
>> + DCS_SW_2P_TX_LP_V101 | \
>> + DCS_SW_1P_TX_LP_V101 | \
>> + DCS_SW_0P_TX_LP_V101 | \
>> + GEN_SR_2P_TX_LP_V101 | \
>> + GEN_SR_1P_TX_LP_V101 | \
>> + GEN_SR_0P_TX_LP_V101 | \
>> + GEN_SW_2P_TX_LP_V101 | \
>> + GEN_SW_1P_TX_LP_V101 | \
>> + GEN_SW_0P_TX_LP_V101)
>> +
>> #define DSI_GEN_HDR 0x6c
>> #define DSI_GEN_PLD_DATA 0x70
>>
>> @@ -165,6 +212,11 @@
>> #define DSI_INT_MSK0 0xc4
>> #define DSI_INT_MSK1 0xc8
>>
>> +#define DSI_ERROR_ST0_V101 0x44
>> +#define DSI_ERROR_ST1_V101 0x48
>> +#define DSI_ERROR_MSK0_V101 0x4c
>> +#define DSI_ERROR_MSK1_V101 0x50
>> +
>> #define DSI_PHY_TMR_RD_CFG 0xf4
>>
>> #define PHY_STATUS_TIMEOUT_US 10000
>> @@ -359,6 +411,49 @@ static const struct dw_mipi_dsi_variant dw_mipi_dsi_v130_v131_layout = {
>> .cfg_gen_payload = REG_FIELD(DSI_GEN_PLD_DATA, 0, 31),
>> };
>>
>> +static const struct dw_mipi_dsi_variant dw_mipi_dsi_v101_layout = {
>> + .cfg_dpi_vid = REG_FIELD(DSI_DPI_CFG, 0, 1),
>> + .cfg_dpi_color_coding = REG_FIELD(DSI_DPI_CFG, 2, 4),
>> + .cfg_dpi_18loosely_en = REG_FIELD(DSI_DPI_CFG, 10, 10),
>> + .cfg_dpi_vsync_active_low = REG_FIELD(DSI_DPI_CFG, 6, 6),
>> + .cfg_dpi_hsync_active_low = REG_FIELD(DSI_DPI_CFG, 7, 7),
>> + .cfg_cmd_mode_en = REG_FIELD(DSI_CMD_MODE_CFG_V101, 0, 0),
>> + .cfg_cmd_mode_all_lp_en = REG_FIELD(DSI_CMD_MODE_CFG_V101, 1, 12),
>> + .cfg_cmd_mode_ack_rqst_en = REG_FIELD(DSI_CMD_MODE_CFG_V101, 13, 13),
>> + .cfg_cmd_pkt_status = REG_FIELD(DSI_CMD_PKT_STATUS_V101, 0, 14),
>> + .cfg_vid_mode_en = REG_FIELD(DSI_VID_MODE_CFG_V101, 0, 0),
>> + .cfg_vid_mode_type = REG_FIELD(DSI_VID_MODE_CFG_V101, 1, 2),
>> + .cfg_vid_mode_low_power = REG_FIELD(DSI_VID_MODE_CFG_V101, 3, 8),
>> + .cfg_vid_pkt_size = REG_FIELD(DSI_VID_PKT_CFG, 0, 10),
>> + .cfg_vid_hsa_time = REG_FIELD(DSI_TMR_LINE_CFG, 0, 8),
>> + .cfg_vid_hbp_time = REG_FIELD(DSI_TMR_LINE_CFG, 9, 17),
>> + .cfg_vid_hline_time = REG_FIELD(DSI_TMR_LINE_CFG, 18, 31),
>> + .cfg_vid_vsa_time = REG_FIELD(DSI_VTIMING_CFG, 0, 3),
>> + .cfg_vid_vbp_time = REG_FIELD(DSI_VTIMING_CFG, 4, 9),
>> + .cfg_vid_vfp_time = REG_FIELD(DSI_VTIMING_CFG, 10, 15),
>> + .cfg_vid_vactive_time = REG_FIELD(DSI_VTIMING_CFG, 16, 26),
>> + .cfg_phy_txrequestclkhs = REG_FIELD(DSI_PHY_IF_CTRL, 0, 0),
>> + .cfg_phy_bta_time = REG_FIELD(DSI_PHY_TMR_CFG_V101, 0, 11),
>> + .cfg_phy_lp2hs_time = REG_FIELD(DSI_PHY_TMR_CFG_V101, 12, 19),
>> + .cfg_phy_hs2lp_time = REG_FIELD(DSI_PHY_TMR_CFG_V101, 20, 27),
>> + .cfg_phy_testclr = REG_FIELD(DSI_PHY_TST_CTRL0_V101, 0, 0),
>> + .cfg_phy_unshutdownz = REG_FIELD(DSI_PHY_RSTZ_V101, 0, 0),
>> + .cfg_phy_unrstz = REG_FIELD(DSI_PHY_RSTZ_V101, 1, 1),
>> + .cfg_phy_enableclk = REG_FIELD(DSI_PHY_RSTZ_V101, 2, 2),
>> + .cfg_phy_nlanes = REG_FIELD(DSI_PHY_IF_CFG_V101, 0, 1),
>> + .cfg_phy_stop_wait_time = REG_FIELD(DSI_PHY_IF_CFG_V101, 2, 9),
>> + .cfg_phy_status = REG_FIELD(DSI_PHY_STATUS_V101, 0, 0),
>> + .cfg_pckhdl_cfg = REG_FIELD(DSI_PCKHDL_CFG_V101, 0, 4),
>> + .cfg_hstx_timeout_counter = REG_FIELD(DSI_TO_CNT_CFG_V101, 0, 15),
>> + .cfg_lprx_timeout_counter = REG_FIELD(DSI_TO_CNT_CFG_V101, 16, 31),
>> + .cfg_int_stat0 = REG_FIELD(DSI_ERROR_ST0_V101, 0, 20),
>> + .cfg_int_stat1 = REG_FIELD(DSI_ERROR_ST1_V101, 0, 17),
>> + .cfg_int_mask0 = REG_FIELD(DSI_ERROR_MSK0_V101, 0, 20),
>> + .cfg_int_mask1 = REG_FIELD(DSI_ERROR_MSK1_V101, 0, 17),
>> + .cfg_gen_hdr = REG_FIELD(DSI_GEN_HDR_V101, 0, 31),
>> + .cfg_gen_payload = REG_FIELD(DSI_GEN_PLD_DATA_V101, 0, 31),
>> +};
>> +
>> /*
>> * Check if either a link to a master or slave is present
>> */
>> @@ -466,6 +561,9 @@ static void dw_mipi_message_config(struct dw_mipi_dsi *dsi,
>> case HWVER_131:
>> cmd_mode_lp = CMD_MODE_ALL_LP;
>> break;
>> + case HWVER_101:
>> + cmd_mode_lp = CMD_MODE_ALL_LP_V101;
>> + break;
>> }
>>
>> if (msg->flags & MIPI_DSI_MSG_REQ_ACK)
>> @@ -644,7 +742,7 @@ static void dw_mipi_dsi_video_mode_config(struct dw_mipi_dsi *dsi)
>> VID_MODE_TYPE_NON_BURST_SYNC_EVENTS);
>>
>> #ifdef CONFIG_DEBUG_FS
>> - if (dsi->vpg) {
>> + if (dsi->hw_version > HWVER_101 && dsi->vpg) {
>> regmap_field_write(dsi->regs, dsi->field_vid_mode_vpg_en, 1);
>> regmap_field_write(dsi->regs, dsi->field_vid_mode_vpg_horiz,
>> dsi->vpg_horizontal ? 1 : 0);
>
> "regmap_field_write" passed 3 arguments, but takes just 2.
>
>
>> @@ -662,9 +760,15 @@ static void dw_mipi_dsi_set_mode(struct dw_mipi_dsi *dsi,
>>
>> dw_mipi_dsi_video_mode_config(dsi);
>>
>> + if (dsi->hw_version == HWVER_101)
>> + regmap_field_write(dsi->field_vid_mode_en, 1);
>> +
>> regmap_field_write(dsi->field_phy_txrequestclkhs, 1);
>> } else {
>> regmap_field_write(dsi->field_cmd_mode_en, 1);
>> +
>> + if (dsi->hw_version == HWVER_101)
>> + regmap_field_write(dsi->field_vid_mode_en, 0);
>> }
>>
>> regmap_write(dsi->regs, DSI_PWR_UP, POWERUP);
>> @@ -857,9 +961,13 @@ static void dw_mipi_dsi_dphy_timing_config(struct dw_mipi_dsi *dsi)
>> regmap_field_write(dsi->field_phy_lp2hs_time, timing.data_lp2hs);
>> regmap_field_write(dsi->field_phy_hs2lp_time, timing.data_hs2lp);
>>
>> - regmap_field_write(dsi->field_phy_max_rd_time, 10000);
>> - regmap_field_write(dsi->field_phy_clkhs2lp_time, timing.clk_hs2lp);
>> - regmap_field_write(dsi->field_phy_clklp2hs_time, timing.clk_lp2hs);
>> + if (dsi->hw_version > HWVER_101) {
>> + regmap_field_write(dsi->field_phy_max_rd_time, 10000);
>> + regmap_field_write(dsi->field_phy_clkhs2lp_time,
>> + timing.clk_hs2lp);
>> + regmap_field_write(dsi->field_phy_clklp2hs_time,
>> + timing.clk_lp2hs);
>> + }
>> }
>>
>> static void dw_mipi_dsi_dphy_interface_config(struct dw_mipi_dsi *dsi)
>> @@ -880,7 +988,8 @@ static void dw_mipi_dsi_dphy_init(struct dw_mipi_dsi *dsi)
>> regmap_field_write(dsi->field_phy_unrstz, 0);
>> regmap_field_write(dsi->field_phy_unshutdownz, 0);
>>
>> - regmap_field_write(dsi->field_phy_forcepll, 0);
>> + if (dsi->hw_version > HWVER_101)
>> + regmap_field_write(dsi->field_phy_forcepll, 0);
>>
>> regmap_field_write(dsi->field_phy_testclr, 0);
>> regmap_field_write(dsi->field_phy_testclr, 1);
>> @@ -896,7 +1005,8 @@ static void dw_mipi_dsi_dphy_enable(struct dw_mipi_dsi *dsi)
>> regmap_field_write(dsi->field_phy_unrstz, 1);
>> regmap_field_write(dsi->field_phy_unshutdownz, 1);
>>
>> - regmap_field_write(dsi->field_phy_forcepll, 1);
>> + if (dsi->hw_version > HWVER_101)
>> + regmap_field_write(dsi->field_phy_forcepll, 1);
>>
>> ret = regmap_field_read_poll_timeout(dsi->field_phy_status,
>> val, val & PHY_LOCK,
>> @@ -1131,6 +1241,9 @@ static int dw_mipi_dsi_regmap_fields_init(struct dw_mipi_dsi *dsi)
>> case HWVER_131:
>> variant = &dw_mipi_dsi_v130_v131_layout;
>> break;
>> + case HWVER_101:
>> + variant = &dw_mipi_dsi_v101_layout;
>> + break;
>> default:
>> DRM_ERROR("Unrecognized DSI host controller HW revision\n");
>> return -ENODEV;
>> --
>> 2.26.0
>>
>>
>> _______________________________________________
>> linux-arm-kernel mailing list
>> linux-arm-kernel@lists.infradead.org
>> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>
> Regards,
> Adrian
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2020-03-30 21:17 UTC|newest]
Thread overview: 80+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <CGME20200330113455eucas1p1441dc79d44de5081e9d90079e2020ca0@eucas1p1.samsung.com>
2020-03-30 11:35 ` [PATCH v5 0/5] Genericize DW MIPI DSI bridge and add i.MX 6 driver Adrian Ratiu
2020-03-30 11:35 ` Adrian Ratiu
2020-03-30 11:35 ` Adrian Ratiu
2020-03-30 11:35 ` Adrian Ratiu
2020-03-30 11:35 ` [PATCH v5 1/5] drm: bridge: dw_mipi_dsi: add initial regmap infrastructure Adrian Ratiu
2020-03-30 11:35 ` Adrian Ratiu
2020-03-30 11:35 ` Adrian Ratiu
2020-03-30 15:58 ` adrian61
2020-03-30 15:58 ` adrian61
2020-03-30 15:58 ` adrian61
2020-03-30 16:13 ` adrian61
2020-03-30 16:13 ` adrian61
2020-03-30 16:13 ` adrian61
2020-03-30 21:16 ` Adrian Ratiu
2020-03-30 21:16 ` Adrian Ratiu
2020-03-30 21:16 ` Adrian Ratiu
2020-03-30 21:16 ` Adrian Ratiu
2020-03-30 11:35 ` [PATCH v5 2/5] drm: bridge: dw_mipi_dsi: abstract register access using reg_fields Adrian Ratiu
2020-03-30 11:35 ` Adrian Ratiu
2020-03-30 11:35 ` Adrian Ratiu
2020-03-30 11:35 ` Adrian Ratiu
2020-04-06 15:39 ` Andrzej Hajda
2020-04-06 15:39 ` Andrzej Hajda
2020-04-06 15:39 ` Andrzej Hajda
2020-04-10 10:22 ` Adrian Ratiu
2020-04-10 10:22 ` Adrian Ratiu
2020-04-10 10:22 ` Adrian Ratiu
2020-04-10 10:22 ` Adrian Ratiu
2020-03-30 11:35 ` [PATCH v5 3/5] drm: bridge: synopsis: add dsi v1.01 support Adrian Ratiu
2020-03-30 11:35 ` Adrian Ratiu
2020-03-30 11:35 ` Adrian Ratiu
2020-03-30 16:08 ` adrian61
2020-03-30 16:08 ` adrian61
2020-03-30 16:08 ` adrian61
2020-03-30 16:08 ` adrian61
2020-03-30 21:18 ` Adrian Ratiu [this message]
2020-03-30 21:18 ` Adrian Ratiu
2020-03-30 21:18 ` Adrian Ratiu
2020-03-30 11:35 ` [PATCH v5 4/5] drm: imx: Add i.MX 6 MIPI DSI host platform driver Adrian Ratiu
2020-03-30 11:35 ` Adrian Ratiu
2020-03-30 11:35 ` Adrian Ratiu
2020-03-30 11:49 ` Fabio Estevam
2020-03-30 11:49 ` Fabio Estevam
2020-03-30 11:49 ` Fabio Estevam
2020-03-30 11:49 ` Fabio Estevam
2020-03-30 13:51 ` Ezequiel Garcia
2020-03-30 13:51 ` Ezequiel Garcia
2020-03-30 13:51 ` Ezequiel Garcia
2020-03-30 13:51 ` Ezequiel Garcia
2020-03-30 21:31 ` Adrian Ratiu
2020-03-30 21:31 ` Adrian Ratiu
2020-03-30 21:31 ` Adrian Ratiu
2020-03-30 21:31 ` Adrian Ratiu
2020-03-31 4:05 ` Ezequiel Garcia
2020-03-31 4:05 ` Ezequiel Garcia
2020-03-31 4:05 ` Ezequiel Garcia
2020-03-31 4:05 ` Ezequiel Garcia
2020-03-31 7:19 ` Adrian Ratiu
2020-03-31 7:19 ` Adrian Ratiu
2020-03-31 7:19 ` Adrian Ratiu
2020-03-31 7:19 ` Adrian Ratiu
2020-03-30 21:20 ` Adrian Ratiu
2020-03-30 21:20 ` Adrian Ratiu
2020-03-30 21:20 ` Adrian Ratiu
2020-03-30 21:20 ` Adrian Ratiu
2020-03-30 11:35 ` [PATCH v5 5/5] dt-bindings: display: add i.MX6 MIPI DSI host controller doc Adrian Ratiu
2020-03-30 11:35 ` Adrian Ratiu
2020-03-30 11:35 ` Adrian Ratiu
2020-03-30 11:52 ` Fabio Estevam
2020-03-30 11:52 ` Fabio Estevam
2020-03-30 11:52 ` Fabio Estevam
2020-03-30 11:52 ` Fabio Estevam
2020-03-30 15:44 ` Rob Herring
2020-03-30 15:44 ` Rob Herring
2020-03-30 15:44 ` Rob Herring
2020-03-30 15:44 ` Rob Herring
2020-04-06 14:23 ` [PATCH v5 0/5] Genericize DW MIPI DSI bridge and add i.MX 6 driver Andrzej Hajda
2020-04-06 14:23 ` Andrzej Hajda
2020-04-06 14:23 ` Andrzej Hajda
2020-04-06 14:23 ` Andrzej Hajda
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87d08t35k3.fsf@collabora.com \
--to=adrian.ratiu@collabora.com \
--cc=Laurent.pinchart@ideasonboard.com \
--cc=a.hajda@samsung.com \
--cc=devicetree@vger.kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=jernej.skrabec@siol.net \
--cc=jonas@kwiboo.se \
--cc=kernel@collabora.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-imx@nxp.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-rockchip@lists.infradead.org \
--cc=linux-stm32@st-md-mailman.stormreply.com \
--cc=pop.adrian61@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.