Message ID | 20191104141439.26312-4-jani.nikula@intel.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | [v2,1/4] drm/i915/dsc: make parameter arrays const | expand |
On Mon, Nov 04, 2019 at 04:14:39PM +0200, Jani Nikula wrote: > Use intel_dsc_ prefix. No functional changes. > > Cc: Manasi Navare <manasi.d.navare@intel.com> > Signed-off-by: Jani Nikula <jani.nikula@intel.com> > --- > drivers/gpu/drm/i915/display/intel_vdsc.c | 12 ++++++------ > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_vdsc.c b/drivers/gpu/drm/i915/display/intel_vdsc.c > index ac10736a076a..b23ba8d108db 100644 > --- a/drivers/gpu/drm/i915/display/intel_vdsc.c > +++ b/drivers/gpu/drm/i915/display/intel_vdsc.c > @@ -501,8 +501,8 @@ intel_dsc_power_domain(const struct intel_crtc_state *crtc_state) > return POWER_DOMAIN_TRANSCODER(cpu_transcoder); > } > > -static void intel_configure_pps_for_dsc_encoder(struct intel_encoder *encoder, > - const struct intel_crtc_state *crtc_state) > +static void intel_dsc_pps_configure(struct intel_encoder *encoder, > + const struct intel_crtc_state *crtc_state) > { > struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc); > struct drm_i915_private *dev_priv = to_i915(encoder->base.dev); > @@ -901,8 +901,8 @@ static void intel_configure_pps_for_dsc_encoder(struct intel_encoder *encoder, > } > } > > -static void intel_dp_write_dsc_pps_sdp(struct intel_encoder *encoder, > - const struct intel_crtc_state *crtc_state) > +static void intel_dsc_dp_pps_write(struct intel_encoder *encoder, > + const struct intel_crtc_state *crtc_state) This function writes pps header and packs pps payload, if that also applies to dsi dsc then we can get rid of the _dp in this function. if it doesnt apply, the rename looks good to me and in that case: Reviewed-by: Manasi Navare <manasi.d.navare@intel.com> Manasi > { > struct intel_dp *intel_dp = enc_to_intel_dp(&encoder->base); > struct intel_digital_port *intel_dig_port = dp_to_dig_port(intel_dp); > @@ -937,9 +937,9 @@ void intel_dsc_enable(struct intel_encoder *encoder, > intel_display_power_get(dev_priv, > intel_dsc_power_domain(crtc_state)); > > - intel_configure_pps_for_dsc_encoder(encoder, crtc_state); > + intel_dsc_pps_configure(encoder, crtc_state); > > - intel_dp_write_dsc_pps_sdp(encoder, crtc_state); > + intel_dsc_dp_pps_write(encoder, crtc_state); > > if (crtc_state->cpu_transcoder == TRANSCODER_EDP) { > dss_ctl1_reg = DSS_CTL1; > -- > 2.20.1 >
On Mon, 04 Nov 2019, Manasi Navare <manasi.d.navare@intel.com> wrote: > On Mon, Nov 04, 2019 at 04:14:39PM +0200, Jani Nikula wrote: >> Use intel_dsc_ prefix. No functional changes. >> >> Cc: Manasi Navare <manasi.d.navare@intel.com> >> Signed-off-by: Jani Nikula <jani.nikula@intel.com> >> --- >> drivers/gpu/drm/i915/display/intel_vdsc.c | 12 ++++++------ >> 1 file changed, 6 insertions(+), 6 deletions(-) >> >> diff --git a/drivers/gpu/drm/i915/display/intel_vdsc.c b/drivers/gpu/drm/i915/display/intel_vdsc.c >> index ac10736a076a..b23ba8d108db 100644 >> --- a/drivers/gpu/drm/i915/display/intel_vdsc.c >> +++ b/drivers/gpu/drm/i915/display/intel_vdsc.c >> @@ -501,8 +501,8 @@ intel_dsc_power_domain(const struct intel_crtc_state *crtc_state) >> return POWER_DOMAIN_TRANSCODER(cpu_transcoder); >> } >> >> -static void intel_configure_pps_for_dsc_encoder(struct intel_encoder *encoder, >> - const struct intel_crtc_state *crtc_state) >> +static void intel_dsc_pps_configure(struct intel_encoder *encoder, >> + const struct intel_crtc_state *crtc_state) >> { >> struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc); >> struct drm_i915_private *dev_priv = to_i915(encoder->base.dev); >> @@ -901,8 +901,8 @@ static void intel_configure_pps_for_dsc_encoder(struct intel_encoder *encoder, >> } >> } >> >> -static void intel_dp_write_dsc_pps_sdp(struct intel_encoder *encoder, >> - const struct intel_crtc_state *crtc_state) >> +static void intel_dsc_dp_pps_write(struct intel_encoder *encoder, >> + const struct intel_crtc_state *crtc_state) > > This function writes pps header and packs pps payload, if that also applies to dsi dsc > then we can get rid of the _dp in this function. > > if it doesnt apply, the rename looks good to me and in that case: It's slightly different for DSI. You'll see. ;) > > Reviewed-by: Manasi Navare <manasi.d.navare@intel.com> Thanks. BR, Jani. > > Manasi > >> { >> struct intel_dp *intel_dp = enc_to_intel_dp(&encoder->base); >> struct intel_digital_port *intel_dig_port = dp_to_dig_port(intel_dp); >> @@ -937,9 +937,9 @@ void intel_dsc_enable(struct intel_encoder *encoder, >> intel_display_power_get(dev_priv, >> intel_dsc_power_domain(crtc_state)); >> >> - intel_configure_pps_for_dsc_encoder(encoder, crtc_state); >> + intel_dsc_pps_configure(encoder, crtc_state); >> >> - intel_dp_write_dsc_pps_sdp(encoder, crtc_state); >> + intel_dsc_dp_pps_write(encoder, crtc_state); >> >> if (crtc_state->cpu_transcoder == TRANSCODER_EDP) { >> dss_ctl1_reg = DSS_CTL1; >> -- >> 2.20.1 >> > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/intel-gfx
diff --git a/drivers/gpu/drm/i915/display/intel_vdsc.c b/drivers/gpu/drm/i915/display/intel_vdsc.c index ac10736a076a..b23ba8d108db 100644 --- a/drivers/gpu/drm/i915/display/intel_vdsc.c +++ b/drivers/gpu/drm/i915/display/intel_vdsc.c @@ -501,8 +501,8 @@ intel_dsc_power_domain(const struct intel_crtc_state *crtc_state) return POWER_DOMAIN_TRANSCODER(cpu_transcoder); } -static void intel_configure_pps_for_dsc_encoder(struct intel_encoder *encoder, - const struct intel_crtc_state *crtc_state) +static void intel_dsc_pps_configure(struct intel_encoder *encoder, + const struct intel_crtc_state *crtc_state) { struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc); struct drm_i915_private *dev_priv = to_i915(encoder->base.dev); @@ -901,8 +901,8 @@ static void intel_configure_pps_for_dsc_encoder(struct intel_encoder *encoder, } } -static void intel_dp_write_dsc_pps_sdp(struct intel_encoder *encoder, - const struct intel_crtc_state *crtc_state) +static void intel_dsc_dp_pps_write(struct intel_encoder *encoder, + const struct intel_crtc_state *crtc_state) { struct intel_dp *intel_dp = enc_to_intel_dp(&encoder->base); struct intel_digital_port *intel_dig_port = dp_to_dig_port(intel_dp); @@ -937,9 +937,9 @@ void intel_dsc_enable(struct intel_encoder *encoder, intel_display_power_get(dev_priv, intel_dsc_power_domain(crtc_state)); - intel_configure_pps_for_dsc_encoder(encoder, crtc_state); + intel_dsc_pps_configure(encoder, crtc_state); - intel_dp_write_dsc_pps_sdp(encoder, crtc_state); + intel_dsc_dp_pps_write(encoder, crtc_state); if (crtc_state->cpu_transcoder == TRANSCODER_EDP) { dss_ctl1_reg = DSS_CTL1;
Use intel_dsc_ prefix. No functional changes. Cc: Manasi Navare <manasi.d.navare@intel.com> Signed-off-by: Jani Nikula <jani.nikula@intel.com> --- drivers/gpu/drm/i915/display/intel_vdsc.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-)