From: "Souza, Jose" <jose.souza@intel.com>
To: "ville.syrjala@linux.intel.com" <ville.syrjala@linux.intel.com>,
"intel-gfx@lists.freedesktop.org"
<intel-gfx@lists.freedesktop.org>
Subject: Re: [Intel-gfx] [PATCH 14/16] drm/i915: Use intel_de_rmw() for icl mg phy programming
Date: Fri, 29 Oct 2021 22:02:11 +0000 [thread overview]
Message-ID: <95ce65f6982748402fabeccf8ac43b26a356ff0d.camel@intel.com> (raw)
In-Reply-To: <20211006204937.30774-15-ville.syrjala@linux.intel.com>
On Wed, 2021-10-06 at 23:49 +0300, Ville Syrjala wrote:
> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
>
> Streamline the code by using intel_de_rmw().
Some lines above 100 cols, other than that:
Reviewed-by: José Roberto de Souza <jose.souza@intel.com>
>
> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> ---
> drivers/gpu/drm/i915/display/intel_ddi.c | 111 ++++++++---------------
> 1 file changed, 39 insertions(+), 72 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_ddi.c b/drivers/gpu/drm/i915/display/intel_ddi.c
> index 3c1b289df2c0..ce8c85701cff 100644
> --- a/drivers/gpu/drm/i915/display/intel_ddi.c
> +++ b/drivers/gpu/drm/i915/display/intel_ddi.c
> @@ -1165,7 +1165,6 @@ static void icl_mg_phy_set_signal_levels(struct intel_encoder *encoder,
> enum tc_port tc_port = intel_port_to_tc(dev_priv, encoder->port);
> const struct intel_ddi_buf_trans *trans;
> int n_entries, ln;
> - u32 val;
>
> if (intel_tc_port_in_tbt_alt_mode(enc_to_dig_port(encoder)))
> return;
> @@ -1174,15 +1173,11 @@ static void icl_mg_phy_set_signal_levels(struct intel_encoder *encoder,
> if (drm_WARN_ON_ONCE(&dev_priv->drm, !trans))
> return;
>
> - /* Set MG_TX_LINK_PARAMS cri_use_fs32 to 0. */
> for (ln = 0; ln < 2; ln++) {
> - val = intel_de_read(dev_priv, MG_TX1_LINK_PARAMS(ln, tc_port));
> - val &= ~CRI_USE_FS32;
> - intel_de_write(dev_priv, MG_TX1_LINK_PARAMS(ln, tc_port), val);
> -
> - val = intel_de_read(dev_priv, MG_TX2_LINK_PARAMS(ln, tc_port));
> - val &= ~CRI_USE_FS32;
> - intel_de_write(dev_priv, MG_TX2_LINK_PARAMS(ln, tc_port), val);
> + intel_de_rmw(dev_priv, MG_TX1_LINK_PARAMS(ln, tc_port),
> + CRI_USE_FS32, 0);
> + intel_de_rmw(dev_priv, MG_TX2_LINK_PARAMS(ln, tc_port),
> + CRI_USE_FS32, 0);
> }
>
> /* Program MG_TX_SWINGCTRL with values from vswing table */
> @@ -1191,19 +1186,15 @@ static void icl_mg_phy_set_signal_levels(struct intel_encoder *encoder,
>
> level = intel_ddi_level(encoder, crtc_state, 2*ln+0);
>
> - val = intel_de_read(dev_priv, MG_TX1_SWINGCTRL(ln, tc_port));
> - val &= ~CRI_TXDEEMPH_OVERRIDE_17_12_MASK;
> - val |= CRI_TXDEEMPH_OVERRIDE_17_12(
> - trans->entries[level].mg.cri_txdeemph_override_17_12);
> - intel_de_write(dev_priv, MG_TX1_SWINGCTRL(ln, tc_port), val);
> + intel_de_rmw(dev_priv, MG_TX1_SWINGCTRL(ln, tc_port),
> + CRI_TXDEEMPH_OVERRIDE_17_12_MASK,
> + CRI_TXDEEMPH_OVERRIDE_17_12(trans->entries[level].mg.cri_txdeemph_override_17_12));
>
> level = intel_ddi_level(encoder, crtc_state, 2*ln+1);
>
> - val = intel_de_read(dev_priv, MG_TX2_SWINGCTRL(ln, tc_port));
> - val &= ~CRI_TXDEEMPH_OVERRIDE_17_12_MASK;
> - val |= CRI_TXDEEMPH_OVERRIDE_17_12(
> - trans->entries[level].mg.cri_txdeemph_override_17_12);
> - intel_de_write(dev_priv, MG_TX2_SWINGCTRL(ln, tc_port), val);
> + intel_de_rmw(dev_priv, MG_TX2_SWINGCTRL(ln, tc_port),
> + CRI_TXDEEMPH_OVERRIDE_17_12_MASK,
> + CRI_TXDEEMPH_OVERRIDE_17_12(trans->entries[level].mg.cri_txdeemph_override_17_12));
> }
>
> /* Program MG_TX_DRVCTRL with values from vswing table */
> @@ -1212,27 +1203,21 @@ static void icl_mg_phy_set_signal_levels(struct intel_encoder *encoder,
>
> level = intel_ddi_level(encoder, crtc_state, 2*ln+0);
>
> - val = intel_de_read(dev_priv, MG_TX1_DRVCTRL(ln, tc_port));
> - val &= ~(CRI_TXDEEMPH_OVERRIDE_11_6_MASK |
> - CRI_TXDEEMPH_OVERRIDE_5_0_MASK);
> - val |= CRI_TXDEEMPH_OVERRIDE_5_0(
> - trans->entries[level].mg.cri_txdeemph_override_5_0) |
> - CRI_TXDEEMPH_OVERRIDE_11_6(
> - trans->entries[level].mg.cri_txdeemph_override_11_6) |
> - CRI_TXDEEMPH_OVERRIDE_EN;
> - intel_de_write(dev_priv, MG_TX1_DRVCTRL(ln, tc_port), val);
> + intel_de_rmw(dev_priv, MG_TX1_DRVCTRL(ln, tc_port),
> + CRI_TXDEEMPH_OVERRIDE_11_6_MASK |
> + CRI_TXDEEMPH_OVERRIDE_5_0_MASK,
> + CRI_TXDEEMPH_OVERRIDE_5_0(trans->entries[level].mg.cri_txdeemph_override_5_0) |
> + CRI_TXDEEMPH_OVERRIDE_11_6(trans->entries[level].mg.cri_txdeemph_override_11_6) |
> + CRI_TXDEEMPH_OVERRIDE_EN);
>
> level = intel_ddi_level(encoder, crtc_state, 2*ln+1);
>
> - val = intel_de_read(dev_priv, MG_TX2_DRVCTRL(ln, tc_port));
> - val &= ~(CRI_TXDEEMPH_OVERRIDE_11_6_MASK |
> - CRI_TXDEEMPH_OVERRIDE_5_0_MASK);
> - val |= CRI_TXDEEMPH_OVERRIDE_5_0(
> - trans->entries[level].mg.cri_txdeemph_override_5_0) |
> - CRI_TXDEEMPH_OVERRIDE_11_6(
> - trans->entries[level].mg.cri_txdeemph_override_11_6) |
> - CRI_TXDEEMPH_OVERRIDE_EN;
> - intel_de_write(dev_priv, MG_TX2_DRVCTRL(ln, tc_port), val);
> + intel_de_rmw(dev_priv, MG_TX2_DRVCTRL(ln, tc_port),
> + CRI_TXDEEMPH_OVERRIDE_11_6_MASK |
> + CRI_TXDEEMPH_OVERRIDE_5_0_MASK,
> + CRI_TXDEEMPH_OVERRIDE_5_0(trans->entries[level].mg.cri_txdeemph_override_5_0) |
> + CRI_TXDEEMPH_OVERRIDE_11_6(trans->entries[level].mg.cri_txdeemph_override_11_6) |
> + CRI_TXDEEMPH_OVERRIDE_EN);
>
> /* FIXME: Program CRI_LOADGEN_SEL after the spec is updated */
> }
> @@ -1243,50 +1228,32 @@ static void icl_mg_phy_set_signal_levels(struct intel_encoder *encoder,
> * values from table for which TX1 and TX2 enabled.
> */
> for (ln = 0; ln < 2; ln++) {
> - val = intel_de_read(dev_priv, MG_CLKHUB(ln, tc_port));
> - if (crtc_state->port_clock < 300000)
> - val |= CFG_LOW_RATE_LKREN_EN;
> - else
> - val &= ~CFG_LOW_RATE_LKREN_EN;
> - intel_de_write(dev_priv, MG_CLKHUB(ln, tc_port), val);
> + intel_de_rmw(dev_priv, MG_CLKHUB(ln, tc_port),
> + CFG_LOW_RATE_LKREN_EN,
> + crtc_state->port_clock < 300000 ? CFG_LOW_RATE_LKREN_EN : 0);
> }
>
> /* Program the MG_TX_DCC<LN, port being used> based on the link frequency */
> for (ln = 0; ln < 2; ln++) {
> - val = intel_de_read(dev_priv, MG_TX1_DCC(ln, tc_port));
> - val &= ~CFG_AMI_CK_DIV_OVERRIDE_VAL_MASK;
> - if (crtc_state->port_clock <= 500000) {
> - val &= ~CFG_AMI_CK_DIV_OVERRIDE_EN;
> - } else {
> - val |= CFG_AMI_CK_DIV_OVERRIDE_EN |
> - CFG_AMI_CK_DIV_OVERRIDE_VAL(1);
> - }
> - intel_de_write(dev_priv, MG_TX1_DCC(ln, tc_port), val);
> + intel_de_rmw(dev_priv, MG_TX1_DCC(ln, tc_port),
> + CFG_AMI_CK_DIV_OVERRIDE_VAL_MASK |
> + CFG_AMI_CK_DIV_OVERRIDE_EN,
> + crtc_state->port_clock > 500000 ?
> + CFG_AMI_CK_DIV_OVERRIDE_EN | CFG_AMI_CK_DIV_OVERRIDE_VAL(1) : 0);
>
> - val = intel_de_read(dev_priv, MG_TX2_DCC(ln, tc_port));
> - val &= ~CFG_AMI_CK_DIV_OVERRIDE_VAL_MASK;
> - if (crtc_state->port_clock <= 500000) {
> - val &= ~CFG_AMI_CK_DIV_OVERRIDE_EN;
> - } else {
> - val |= CFG_AMI_CK_DIV_OVERRIDE_EN |
> - CFG_AMI_CK_DIV_OVERRIDE_VAL(1);
> - }
> - intel_de_write(dev_priv, MG_TX2_DCC(ln, tc_port), val);
> + intel_de_rmw(dev_priv, MG_TX2_DCC(ln, tc_port),
> + CFG_AMI_CK_DIV_OVERRIDE_VAL_MASK |
> + CFG_AMI_CK_DIV_OVERRIDE_EN,
> + crtc_state->port_clock > 500000 ?
> + CFG_AMI_CK_DIV_OVERRIDE_EN | CFG_AMI_CK_DIV_OVERRIDE_VAL(1) : 0);
> }
>
> /* Program MG_TX_PISO_READLOAD with values from vswing table */
> for (ln = 0; ln < 2; ln++) {
> - val = intel_de_read(dev_priv,
> - MG_TX1_PISO_READLOAD(ln, tc_port));
> - val |= CRI_CALCINIT;
> - intel_de_write(dev_priv, MG_TX1_PISO_READLOAD(ln, tc_port),
> - val);
> -
> - val = intel_de_read(dev_priv,
> - MG_TX2_PISO_READLOAD(ln, tc_port));
> - val |= CRI_CALCINIT;
> - intel_de_write(dev_priv, MG_TX2_PISO_READLOAD(ln, tc_port),
> - val);
> + intel_de_rmw(dev_priv, MG_TX1_PISO_READLOAD(ln, tc_port),
> + 0, CRI_CALCINIT);
> + intel_de_rmw(dev_priv, MG_TX2_PISO_READLOAD(ln, tc_port),
> + 0, CRI_CALCINIT);
> }
> }
>
next prev parent reply other threads:[~2021-10-29 22:02 UTC|newest]
Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-10-06 20:49 [Intel-gfx] [PATCH 00/16] drm/i915: DP per-lane drive settings for icl+ Ville Syrjala
2021-10-06 20:49 ` [Intel-gfx] [PATCH 01/16] drm/i915: Remove pointless extra namespace from dkl/snps buf trans structs Ville Syrjala
2021-10-08 10:18 ` Jani Nikula
2021-10-06 20:49 ` [Intel-gfx] [PATCH 02/16] drm/i915: Shrink {icl_mg, tgl_dkl}_phy_ddi_buf_trans Ville Syrjala
2021-10-08 10:19 ` Jani Nikula
2021-10-06 20:49 ` [Intel-gfx] [PATCH 03/16] drm/i915: Use standard form terminating condition for lane for loops Ville Syrjala
2021-10-08 10:19 ` Jani Nikula
2021-10-06 20:49 ` [Intel-gfx] [PATCH 04/16] drm/i915: Add all per-lane register definitions for icl combo phy Ville Syrjala
2021-10-08 10:21 ` Jani Nikula
2021-10-08 10:29 ` Ville Syrjälä
2021-10-06 20:49 ` [Intel-gfx] [PATCH 05/16] drm/i915: Remove dead DKL_TX_LOADGEN_SHARING_PMD_DISABLE stuff Ville Syrjala
2021-10-08 10:23 ` Jani Nikula
2021-10-06 20:49 ` [Intel-gfx] [PATCH 06/16] drm/i915: Extract icl_combo_phy_loadgen_select() Ville Syrjala
2021-10-08 10:25 ` Jani Nikula
2021-10-06 20:49 ` [Intel-gfx] [PATCH 07/16] drm/i915: Stop using group access when progrmming icl combo phy TX Ville Syrjala
2021-10-29 21:53 ` Souza, Jose
2021-10-06 20:49 ` [Intel-gfx] [PATCH 08/16] drm/i915: Query the vswing levels per-lane for icl combo phy Ville Syrjala
2021-10-29 21:57 ` Souza, Jose
2021-11-01 10:11 ` Ville Syrjälä
2021-11-01 17:36 ` Souza, Jose
2021-10-06 20:49 ` [Intel-gfx] [PATCH 09/16] drm/i915: Query the vswing levels per-lane for icl mg phy Ville Syrjala
2021-10-29 21:59 ` Souza, Jose
2021-11-01 9:56 ` Ville Syrjälä
2021-10-06 20:49 ` [Intel-gfx] [PATCH 10/16] drm/i915: Query the vswing levels per-lane for tgl dkl phy Ville Syrjala
2021-10-29 21:59 ` Souza, Jose
2021-10-06 20:49 ` [Intel-gfx] [PATCH 11/16] drm/i915: Query the vswing levels per-lane for snps phy Ville Syrjala
2021-10-29 22:00 ` Souza, Jose
2021-10-06 20:49 ` [Intel-gfx] [PATCH 12/16] drm/i915: Enable per-lane drive settings for icl+ Ville Syrjala
2021-10-29 22:04 ` Souza, Jose
2021-10-06 20:49 ` [Intel-gfx] [PATCH 13/16] drm/i915: Use intel_de_rmw() for tgl dkl phy programming Ville Syrjala
2021-10-29 22:01 ` Souza, Jose
2021-10-06 20:49 ` [Intel-gfx] [PATCH 14/16] drm/i915: Use intel_de_rmw() for icl mg " Ville Syrjala
2021-10-29 22:02 ` Souza, Jose [this message]
2021-10-06 20:49 ` [Intel-gfx] [PATCH 15/16] drm/i915: Use intel_de_rmw() for icl combo " Ville Syrjala
2021-10-29 22:02 ` Souza, Jose
2021-10-06 20:49 ` [Intel-gfx] [PATCH 16/16] drm/i915: Fix icl+ combo phy static lane power down setup Ville Syrjala
2021-10-28 13:25 ` Imre Deak
2021-10-28 17:43 ` Jani Nikula
2021-10-07 0:18 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915: DP per-lane drive settings for icl+ Patchwork
2021-10-07 0:20 ` [Intel-gfx] ✗ Fi.CI.SPARSE: " Patchwork
2021-10-07 0:52 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
2021-10-07 3:08 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
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=95ce65f6982748402fabeccf8ac43b26a356ff0d.camel@intel.com \
--to=jose.souza@intel.com \
--cc=intel-gfx@lists.freedesktop.org \
--cc=ville.syrjala@linux.intel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox