| Message ID | 1491329765-14340-1-git-send-email-dhinakaran.pandiyan@intel.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Tue, 04 Apr 2017, Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com> wrote: > Noticed this while I was looking at some debug output, > [drm:intel_hdmi_compute_config [i915]] picking bpc to 12 for HDMI output > [drm:intel_hdmi_compute_config [i915]] forcing pipe bpc to 36 for HDMI > > I believe the second line should be pipe *bpp* > > Signed-off-by: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com> Reviewed-by: Jani Nikula <jani.nikula@intel.com> > --- > drivers/gpu/drm/i915/intel_hdmi.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/intel_hdmi.c b/drivers/gpu/drm/i915/intel_hdmi.c > index 1d623b5..6efc3cb 100644 > --- a/drivers/gpu/drm/i915/intel_hdmi.c > +++ b/drivers/gpu/drm/i915/intel_hdmi.c > @@ -1392,7 +1392,7 @@ bool intel_hdmi_compute_config(struct intel_encoder *encoder, > } > > if (!pipe_config->bw_constrained) { > - DRM_DEBUG_KMS("forcing pipe bpc to %i for HDMI\n", desired_bpp); > + DRM_DEBUG_KMS("forcing pipe bpp to %i for HDMI\n", desired_bpp); > pipe_config->pipe_bpp = desired_bpp; > }
On Wed, 05 Apr 2017, Jani Nikula <jani.nikula@linux.intel.com> wrote: > On Tue, 04 Apr 2017, Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com> wrote: >> Noticed this while I was looking at some debug output, >> [drm:intel_hdmi_compute_config [i915]] picking bpc to 12 for HDMI output >> [drm:intel_hdmi_compute_config [i915]] forcing pipe bpc to 36 for HDMI >> >> I believe the second line should be pipe *bpp* >> >> Signed-off-by: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com> > > Reviewed-by: Jani Nikula <jani.nikula@intel.com> And pushed, thanks for the patch. BR, Jani. > > >> --- >> drivers/gpu/drm/i915/intel_hdmi.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/gpu/drm/i915/intel_hdmi.c b/drivers/gpu/drm/i915/intel_hdmi.c >> index 1d623b5..6efc3cb 100644 >> --- a/drivers/gpu/drm/i915/intel_hdmi.c >> +++ b/drivers/gpu/drm/i915/intel_hdmi.c >> @@ -1392,7 +1392,7 @@ bool intel_hdmi_compute_config(struct intel_encoder *encoder, >> } >> >> if (!pipe_config->bw_constrained) { >> - DRM_DEBUG_KMS("forcing pipe bpc to %i for HDMI\n", desired_bpp); >> + DRM_DEBUG_KMS("forcing pipe bpp to %i for HDMI\n", desired_bpp); >> pipe_config->pipe_bpp = desired_bpp; >> }
diff --git a/drivers/gpu/drm/i915/intel_hdmi.c b/drivers/gpu/drm/i915/intel_hdmi.c index 1d623b5..6efc3cb 100644 --- a/drivers/gpu/drm/i915/intel_hdmi.c +++ b/drivers/gpu/drm/i915/intel_hdmi.c @@ -1392,7 +1392,7 @@ bool intel_hdmi_compute_config(struct intel_encoder *encoder, } if (!pipe_config->bw_constrained) { - DRM_DEBUG_KMS("forcing pipe bpc to %i for HDMI\n", desired_bpp); + DRM_DEBUG_KMS("forcing pipe bpp to %i for HDMI\n", desired_bpp); pipe_config->pipe_bpp = desired_bpp; }
Noticed this while I was looking at some debug output, [drm:intel_hdmi_compute_config [i915]] picking bpc to 12 for HDMI output [drm:intel_hdmi_compute_config [i915]] forcing pipe bpc to 36 for HDMI I believe the second line should be pipe *bpp* Signed-off-by: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com> --- drivers/gpu/drm/i915/intel_hdmi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)