From: Jani Nikula <jani.nikula@intel.com>
To: "Souza, Jose" <jose.souza@intel.com>,
"intel-gfx@lists.freedesktop.org"
<intel-gfx@lists.freedesktop.org>
Subject: Re: [Intel-gfx] [PATCH] drm/i915/ddi: add use_edp_hobl() and use_edp_low_vswing() helpers
Date: Wed, 08 Dec 2021 12:16:47 +0200 [thread overview]
Message-ID: <87v8zz4d0w.fsf@intel.com> (raw)
In-Reply-To: <c655648a3ff0ae5d6067f90067a4e1ad6b1700df.camel@intel.com>
On Fri, 03 Dec 2021, "Souza, Jose" <jose.souza@intel.com> wrote:
> On Fri, 2021-12-03 at 15:13 +0200, Jani Nikula wrote:
>> Localize HOBL and low vswing VBT lookups to a couple of small helpers,
>> and get rid of a bunch of local variables.
>
> Reviewed-by: José Roberto de Souza <jose.souza@intel.com>
Thanks, pushed.
>
>>
>> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
>> ---
>> .../drm/i915/display/intel_ddi_buf_trans.c | 94 +++++++------------
>> 1 file changed, 36 insertions(+), 58 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/i915/display/intel_ddi_buf_trans.c b/drivers/gpu/drm/i915/display/intel_ddi_buf_trans.c
>> index 78cd8f77b49d..4ecbfa625891 100644
>> --- a/drivers/gpu/drm/i915/display/intel_ddi_buf_trans.c
>> +++ b/drivers/gpu/drm/i915/display/intel_ddi_buf_trans.c
>> @@ -1032,6 +1032,21 @@ bool is_hobl_buf_trans(const struct intel_ddi_buf_trans *table)
>> return table == &tgl_combo_phy_trans_edp_hbr2_hobl;
>> }
>>
>> +static bool use_edp_hobl(struct intel_encoder *encoder)
>> +{
>> + struct drm_i915_private *i915 = to_i915(encoder->base.dev);
>> + struct intel_dp *intel_dp = enc_to_intel_dp(encoder);
>> +
>> + return i915->vbt.edp.hobl && !intel_dp->hobl_failed;
>> +}
>> +
>> +static bool use_edp_low_vswing(struct intel_encoder *encoder)
>> +{
>> + struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
>> +
>> + return dev_priv->vbt.edp.low_vswing;
>> +}
>> +
>> static const struct intel_ddi_buf_trans *
>> intel_get_buf_trans(const struct intel_ddi_buf_trans *trans, int *num_entries)
>> {
>> @@ -1057,14 +1072,12 @@ bdw_get_buf_trans(struct intel_encoder *encoder,
>> const struct intel_crtc_state *crtc_state,
>> int *n_entries)
>> {
>> - struct drm_i915_private *i915 = to_i915(encoder->base.dev);
>> -
>> if (intel_crtc_has_type(crtc_state, INTEL_OUTPUT_ANALOG))
>> return intel_get_buf_trans(&bdw_trans_fdi, n_entries);
>> else if (intel_crtc_has_type(crtc_state, INTEL_OUTPUT_HDMI))
>> return intel_get_buf_trans(&bdw_trans_hdmi, n_entries);
>> else if (intel_crtc_has_type(crtc_state, INTEL_OUTPUT_EDP) &&
>> - i915->vbt.edp.low_vswing)
>> + use_edp_low_vswing(encoder))
>> return intel_get_buf_trans(&bdw_trans_edp, n_entries);
>> else
>> return intel_get_buf_trans(&bdw_trans_dp, n_entries);
>> @@ -1094,12 +1107,10 @@ skl_y_get_buf_trans(struct intel_encoder *encoder,
>> const struct intel_crtc_state *crtc_state,
>> int *n_entries)
>> {
>> - struct drm_i915_private *i915 = to_i915(encoder->base.dev);
>> -
>> if (intel_crtc_has_type(crtc_state, INTEL_OUTPUT_HDMI))
>> return intel_get_buf_trans(&skl_y_trans_hdmi, n_entries);
>> else if (intel_crtc_has_type(crtc_state, INTEL_OUTPUT_EDP) &&
>> - i915->vbt.edp.low_vswing)
>> + use_edp_low_vswing(encoder))
>> return _skl_get_buf_trans_dp(encoder, &skl_y_trans_edp, n_entries);
>> else
>> return _skl_get_buf_trans_dp(encoder, &skl_y_trans_dp, n_entries);
>> @@ -1110,12 +1121,10 @@ skl_u_get_buf_trans(struct intel_encoder *encoder,
>> const struct intel_crtc_state *crtc_state,
>> int *n_entries)
>> {
>> - struct drm_i915_private *i915 = to_i915(encoder->base.dev);
>> -
>> if (intel_crtc_has_type(crtc_state, INTEL_OUTPUT_HDMI))
>> return intel_get_buf_trans(&skl_trans_hdmi, n_entries);
>> else if (intel_crtc_has_type(crtc_state, INTEL_OUTPUT_EDP) &&
>> - i915->vbt.edp.low_vswing)
>> + use_edp_low_vswing(encoder))
>> return _skl_get_buf_trans_dp(encoder, &skl_u_trans_edp, n_entries);
>> else
>> return _skl_get_buf_trans_dp(encoder, &skl_u_trans_dp, n_entries);
>> @@ -1126,12 +1135,10 @@ skl_get_buf_trans(struct intel_encoder *encoder,
>> const struct intel_crtc_state *crtc_state,
>> int *n_entries)
>> {
>> - struct drm_i915_private *i915 = to_i915(encoder->base.dev);
>> -
>> if (intel_crtc_has_type(crtc_state, INTEL_OUTPUT_HDMI))
>> return intel_get_buf_trans(&skl_trans_hdmi, n_entries);
>> else if (intel_crtc_has_type(crtc_state, INTEL_OUTPUT_EDP) &&
>> - i915->vbt.edp.low_vswing)
>> + use_edp_low_vswing(encoder))
>> return _skl_get_buf_trans_dp(encoder, &skl_trans_edp, n_entries);
>> else
>> return _skl_get_buf_trans_dp(encoder, &skl_trans_dp, n_entries);
>> @@ -1142,12 +1149,10 @@ kbl_y_get_buf_trans(struct intel_encoder *encoder,
>> const struct intel_crtc_state *crtc_state,
>> int *n_entries)
>> {
>> - struct drm_i915_private *i915 = to_i915(encoder->base.dev);
>> -
>> if (intel_crtc_has_type(crtc_state, INTEL_OUTPUT_HDMI))
>> return intel_get_buf_trans(&skl_y_trans_hdmi, n_entries);
>> else if (intel_crtc_has_type(crtc_state, INTEL_OUTPUT_EDP) &&
>> - i915->vbt.edp.low_vswing)
>> + use_edp_low_vswing(encoder))
>> return _skl_get_buf_trans_dp(encoder, &skl_y_trans_edp, n_entries);
>> else
>> return _skl_get_buf_trans_dp(encoder, &kbl_y_trans_dp, n_entries);
>> @@ -1158,12 +1163,10 @@ kbl_u_get_buf_trans(struct intel_encoder *encoder,
>> const struct intel_crtc_state *crtc_state,
>> int *n_entries)
>> {
>> - struct drm_i915_private *i915 = to_i915(encoder->base.dev);
>> -
>> if (intel_crtc_has_type(crtc_state, INTEL_OUTPUT_HDMI))
>> return intel_get_buf_trans(&skl_trans_hdmi, n_entries);
>> else if (intel_crtc_has_type(crtc_state, INTEL_OUTPUT_EDP) &&
>> - i915->vbt.edp.low_vswing)
>> + use_edp_low_vswing(encoder))
>> return _skl_get_buf_trans_dp(encoder, &skl_u_trans_edp, n_entries);
>> else
>> return _skl_get_buf_trans_dp(encoder, &kbl_u_trans_dp, n_entries);
>> @@ -1174,12 +1177,10 @@ kbl_get_buf_trans(struct intel_encoder *encoder,
>> const struct intel_crtc_state *crtc_state,
>> int *n_entries)
>> {
>> - struct drm_i915_private *i915 = to_i915(encoder->base.dev);
>> -
>> if (intel_crtc_has_type(crtc_state, INTEL_OUTPUT_HDMI))
>> return intel_get_buf_trans(&skl_trans_hdmi, n_entries);
>> else if (intel_crtc_has_type(crtc_state, INTEL_OUTPUT_EDP) &&
>> - i915->vbt.edp.low_vswing)
>> + use_edp_low_vswing(encoder))
>> return _skl_get_buf_trans_dp(encoder, &skl_trans_edp, n_entries);
>> else
>> return _skl_get_buf_trans_dp(encoder, &kbl_trans_dp, n_entries);
>> @@ -1190,12 +1191,10 @@ bxt_get_buf_trans(struct intel_encoder *encoder,
>> const struct intel_crtc_state *crtc_state,
>> int *n_entries)
>> {
>> - struct drm_i915_private *i915 = to_i915(encoder->base.dev);
>> -
>> if (intel_crtc_has_type(crtc_state, INTEL_OUTPUT_HDMI))
>> return intel_get_buf_trans(&bxt_trans_hdmi, n_entries);
>> else if (intel_crtc_has_type(crtc_state, INTEL_OUTPUT_EDP) &&
>> - i915->vbt.edp.low_vswing)
>> + use_edp_low_vswing(encoder))
>> return intel_get_buf_trans(&bxt_trans_edp, n_entries);
>> else
>> return intel_get_buf_trans(&bxt_trans_dp, n_entries);
>> @@ -1215,12 +1214,10 @@ icl_get_combo_buf_trans_edp(struct intel_encoder *encoder,
>> const struct intel_crtc_state *crtc_state,
>> int *n_entries)
>> {
>> - struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
>> -
>> if (crtc_state->port_clock > 540000) {
>> return intel_get_buf_trans(&icl_combo_phy_trans_dp_hbr2_edp_hbr3,
>> n_entries);
>> - } else if (dev_priv->vbt.edp.low_vswing) {
>> + } else if (use_edp_low_vswing(encoder)) {
>> return intel_get_buf_trans(&icl_combo_phy_trans_edp_hbr2,
>> n_entries);
>> }
>> @@ -1282,12 +1279,10 @@ ehl_get_combo_buf_trans(struct intel_encoder *encoder,
>> const struct intel_crtc_state *crtc_state,
>> int *n_entries)
>> {
>> - struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
>> -
>> if (intel_crtc_has_type(crtc_state, INTEL_OUTPUT_HDMI))
>> return intel_get_buf_trans(&icl_combo_phy_trans_hdmi, n_entries);
>> else if (intel_crtc_has_type(crtc_state, INTEL_OUTPUT_EDP) &&
>> - dev_priv->vbt.edp.low_vswing)
>> + use_edp_low_vswing(encoder))
>> return ehl_get_combo_buf_trans_edp(encoder, crtc_state, n_entries);
>> else
>> return intel_get_buf_trans(&ehl_combo_phy_trans_dp, n_entries);
>> @@ -1309,12 +1304,10 @@ jsl_get_combo_buf_trans(struct intel_encoder *encoder,
>> const struct intel_crtc_state *crtc_state,
>> int *n_entries)
>> {
>> - struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
>> -
>> if (intel_crtc_has_type(crtc_state, INTEL_OUTPUT_HDMI))
>> return intel_get_buf_trans(&icl_combo_phy_trans_hdmi, n_entries);
>> else if (intel_crtc_has_type(crtc_state, INTEL_OUTPUT_EDP) &&
>> - dev_priv->vbt.edp.low_vswing)
>> + use_edp_low_vswing(encoder))
>> return jsl_get_combo_buf_trans_edp(encoder, crtc_state, n_entries);
>> else
>> return intel_get_buf_trans(&icl_combo_phy_trans_dp_hbr2_edp_hbr3, n_entries);
>> @@ -1346,16 +1339,13 @@ tgl_get_combo_buf_trans_edp(struct intel_encoder *encoder,
>> const struct intel_crtc_state *crtc_state,
>> int *n_entries)
>> {
>> - struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
>> - struct intel_dp *intel_dp = enc_to_intel_dp(encoder);
>> -
>> if (crtc_state->port_clock > 540000) {
>> return intel_get_buf_trans(&icl_combo_phy_trans_dp_hbr2_edp_hbr3,
>> n_entries);
>> - } else if (dev_priv->vbt.edp.hobl && !intel_dp->hobl_failed) {
>> + } else if (use_edp_hobl(encoder)) {
>> return intel_get_buf_trans(&tgl_combo_phy_trans_edp_hbr2_hobl,
>> n_entries);
>> - } else if (dev_priv->vbt.edp.low_vswing) {
>> + } else if (use_edp_low_vswing(encoder)) {
>> return intel_get_buf_trans(&icl_combo_phy_trans_edp_hbr2,
>> n_entries);
>> }
>> @@ -1394,16 +1384,13 @@ dg1_get_combo_buf_trans_edp(struct intel_encoder *encoder,
>> const struct intel_crtc_state *crtc_state,
>> int *n_entries)
>> {
>> - struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
>> - struct intel_dp *intel_dp = enc_to_intel_dp(encoder);
>> -
>> if (crtc_state->port_clock > 540000)
>> return intel_get_buf_trans(&icl_combo_phy_trans_dp_hbr2_edp_hbr3,
>> n_entries);
>> - else if (dev_priv->vbt.edp.hobl && !intel_dp->hobl_failed)
>> + else if (use_edp_hobl(encoder))
>> return intel_get_buf_trans(&tgl_combo_phy_trans_edp_hbr2_hobl,
>> n_entries);
>> - else if (dev_priv->vbt.edp.low_vswing)
>> + else if (use_edp_low_vswing(encoder))
>> return intel_get_buf_trans(&icl_combo_phy_trans_edp_hbr2,
>> n_entries);
>> else
>> @@ -1439,16 +1426,13 @@ rkl_get_combo_buf_trans_edp(struct intel_encoder *encoder,
>> const struct intel_crtc_state *crtc_state,
>> int *n_entries)
>> {
>> - struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
>> - struct intel_dp *intel_dp = enc_to_intel_dp(encoder);
>> -
>> if (crtc_state->port_clock > 540000) {
>> return intel_get_buf_trans(&icl_combo_phy_trans_dp_hbr2_edp_hbr3,
>> n_entries);
>> - } else if (dev_priv->vbt.edp.hobl && !intel_dp->hobl_failed) {
>> + } else if (use_edp_hobl(encoder)) {
>> return intel_get_buf_trans(&tgl_combo_phy_trans_edp_hbr2_hobl,
>> n_entries);
>> - } else if (dev_priv->vbt.edp.low_vswing) {
>> + } else if (use_edp_low_vswing(encoder)) {
>> return intel_get_buf_trans(&icl_combo_phy_trans_edp_hbr2,
>> n_entries);
>> }
>> @@ -1485,14 +1469,11 @@ adls_get_combo_buf_trans_edp(struct intel_encoder *encoder,
>> const struct intel_crtc_state *crtc_state,
>> int *n_entries)
>> {
>> - struct drm_i915_private *i915 = to_i915(encoder->base.dev);
>> - struct intel_dp *intel_dp = enc_to_intel_dp(encoder);
>> -
>> if (crtc_state->port_clock > 540000)
>> return intel_get_buf_trans(&adls_combo_phy_trans_edp_hbr3, n_entries);
>> - else if (i915->vbt.edp.hobl && !intel_dp->hobl_failed)
>> + else if (use_edp_hobl(encoder))
>> return intel_get_buf_trans(&tgl_combo_phy_trans_edp_hbr2_hobl, n_entries);
>> - else if (i915->vbt.edp.low_vswing)
>> + else if (use_edp_low_vswing(encoder))
>> return intel_get_buf_trans(&adls_combo_phy_trans_edp_hbr2, n_entries);
>> else
>> return adls_get_combo_buf_trans_dp(encoder, crtc_state, n_entries);
>> @@ -1527,16 +1508,13 @@ adlp_get_combo_buf_trans_edp(struct intel_encoder *encoder,
>> const struct intel_crtc_state *crtc_state,
>> int *n_entries)
>> {
>> - struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
>> - struct intel_dp *intel_dp = enc_to_intel_dp(encoder);
>> -
>> if (crtc_state->port_clock > 540000) {
>> return intel_get_buf_trans(&adlp_combo_phy_trans_edp_hbr3,
>> n_entries);
>> - } else if (dev_priv->vbt.edp.hobl && !intel_dp->hobl_failed) {
>> + } else if (use_edp_hobl(encoder)) {
>> return intel_get_buf_trans(&tgl_combo_phy_trans_edp_hbr2_hobl,
>> n_entries);
>> - } else if (dev_priv->vbt.edp.low_vswing) {
>> + } else if (use_edp_low_vswing(encoder)) {
>> return intel_get_buf_trans(&adlp_combo_phy_trans_edp_up_to_hbr2,
>> n_entries);
>> }
>
--
Jani Nikula, Intel Open Source Graphics Center
next prev parent reply other threads:[~2021-12-08 10:16 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-12-03 13:13 [Intel-gfx] [PATCH] drm/i915/ddi: add use_edp_hobl() and use_edp_low_vswing() helpers Jani Nikula
2021-12-03 13:36 ` Souza, Jose
2021-12-08 10:16 ` Jani Nikula [this message]
2021-12-03 15:13 ` [Intel-gfx] ✓ Fi.CI.BAT: success for " Patchwork
2021-12-03 21:40 ` [Intel-gfx] ✓ Fi.CI.IGT: " 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=87v8zz4d0w.fsf@intel.com \
--to=jani.nikula@intel.com \
--cc=intel-gfx@lists.freedesktop.org \
--cc=jose.souza@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