* [PATCH 1/3] drm/i915: Make i9xx_load_ycbcr_conversion_matrix() static @ 2017-09-01 14:31 ville.syrjala 2017-09-01 14:31 ` [PATCH 2/3] drm/i915: Make i2c lock ops static ville.syrjala ` (5 more replies) 0 siblings, 6 replies; 10+ messages in thread From: ville.syrjala @ 2017-09-01 14:31 UTC (permalink / raw) To: intel-gfx From: Ville Syrjälä <ville.syrjala@linux.intel.com> Make i9xx_load_ycbcr_conversion_matrix() static to appease sparse: intel_color.c:110:6: warning: symbol 'i9xx_load_ycbcr_conversion_matrix' was not declared. Should it be static? Cc: Shashank Sharma <shashank.sharma@intel.com> Fixes: 25edf91501b8 ("drm/i915: prepare csc unit for YCBCR420 output") Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> --- drivers/gpu/drm/i915/intel_color.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/intel_color.c b/drivers/gpu/drm/i915/intel_color.c index 8e4e829682b9..ff9ecd211abb 100644 --- a/drivers/gpu/drm/i915/intel_color.c +++ b/drivers/gpu/drm/i915/intel_color.c @@ -107,7 +107,7 @@ static void ctm_mult_by_limited(uint64_t *result, int64_t *input) } } -void i9xx_load_ycbcr_conversion_matrix(struct intel_crtc *intel_crtc) +static void i9xx_load_ycbcr_conversion_matrix(struct intel_crtc *intel_crtc) { int pipe = intel_crtc->pipe; struct drm_i915_private *dev_priv = to_i915(intel_crtc->base.dev); -- 2.13.5 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 2/3] drm/i915: Make i2c lock ops static 2017-09-01 14:31 [PATCH 1/3] drm/i915: Make i9xx_load_ycbcr_conversion_matrix() static ville.syrjala @ 2017-09-01 14:31 ` ville.syrjala 2017-09-01 14:38 ` Chris Wilson 2017-09-01 14:31 ` [PATCH 3/3] drm/i915: Fix enum pipe vs. enum transcoder for the PCH transcoder ville.syrjala ` (4 subsequent siblings) 5 siblings, 1 reply; 10+ messages in thread From: ville.syrjala @ 2017-09-01 14:31 UTC (permalink / raw) To: intel-gfx; +Cc: Daniel Vetter From: Ville Syrjälä <ville.syrjala@linux.intel.com> Make gmbus_lock_ops and proxy_lock_ops static to appease sparse intel_i2c.c:652:34: warning: symbol 'gmbus_lock_ops' was not declared. Should it be static? intel_sdvo.c:2981:34: warning: symbol 'proxy_lock_ops' was not declared. Should it be static? Cc: Daniel Vetter <daniel.vetter@ffwll.ch> Fixes: a85066840d29 ("drm/i915: Rework sdvo proxy i2c locking") Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> --- drivers/gpu/drm/i915/intel_i2c.c | 2 +- drivers/gpu/drm/i915/intel_sdvo.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_i2c.c b/drivers/gpu/drm/i915/intel_i2c.c index 6698826954e1..eb5827110d8f 100644 --- a/drivers/gpu/drm/i915/intel_i2c.c +++ b/drivers/gpu/drm/i915/intel_i2c.c @@ -649,7 +649,7 @@ static void gmbus_unlock_bus(struct i2c_adapter *adapter, mutex_unlock(&dev_priv->gmbus_mutex); } -const struct i2c_lock_operations gmbus_lock_ops = { +static const struct i2c_lock_operations gmbus_lock_ops = { .lock_bus = gmbus_lock_bus, .trylock_bus = gmbus_trylock_bus, .unlock_bus = gmbus_unlock_bus, diff --git a/drivers/gpu/drm/i915/intel_sdvo.c b/drivers/gpu/drm/i915/intel_sdvo.c index 9509a18eb00d..7437944b388f 100644 --- a/drivers/gpu/drm/i915/intel_sdvo.c +++ b/drivers/gpu/drm/i915/intel_sdvo.c @@ -2978,7 +2978,7 @@ static void proxy_unlock_bus(struct i2c_adapter *adapter, sdvo->i2c->lock_ops->unlock_bus(sdvo->i2c, flags); } -const struct i2c_lock_operations proxy_lock_ops = { +static const struct i2c_lock_operations proxy_lock_ops = { .lock_bus = proxy_lock_bus, .trylock_bus = proxy_trylock_bus, .unlock_bus = proxy_unlock_bus, -- 2.13.5 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH 2/3] drm/i915: Make i2c lock ops static 2017-09-01 14:31 ` [PATCH 2/3] drm/i915: Make i2c lock ops static ville.syrjala @ 2017-09-01 14:38 ` Chris Wilson 0 siblings, 0 replies; 10+ messages in thread From: Chris Wilson @ 2017-09-01 14:38 UTC (permalink / raw) To: ville.syrjala, intel-gfx; +Cc: Daniel Vetter Quoting ville.syrjala@linux.intel.com (2017-09-01 15:31:22) > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > > Make gmbus_lock_ops and proxy_lock_ops static to appease sparse > intel_i2c.c:652:34: warning: symbol 'gmbus_lock_ops' was not declared. Should it be static? > intel_sdvo.c:2981:34: warning: symbol 'proxy_lock_ops' was not declared. Should it be static? > > Cc: Daniel Vetter <daniel.vetter@ffwll.ch> > Fixes: a85066840d29 ("drm/i915: Rework sdvo proxy i2c locking") > Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk> -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 3/3] drm/i915: Fix enum pipe vs. enum transcoder for the PCH transcoder 2017-09-01 14:31 [PATCH 1/3] drm/i915: Make i9xx_load_ycbcr_conversion_matrix() static ville.syrjala 2017-09-01 14:31 ` [PATCH 2/3] drm/i915: Make i2c lock ops static ville.syrjala @ 2017-09-01 14:31 ` ville.syrjala 2017-09-01 14:43 ` Chris Wilson 2017-09-01 14:37 ` [PATCH 1/3] drm/i915: Make i9xx_load_ycbcr_conversion_matrix() static Chris Wilson ` (3 subsequent siblings) 5 siblings, 1 reply; 10+ messages in thread From: ville.syrjala @ 2017-09-01 14:31 UTC (permalink / raw) To: intel-gfx; +Cc: Matthias Kaehlcke From: Ville Syrjälä <ville.syrjala@linux.intel.com> Use enum pipe for PCH transcoders also in the FIFO underrun code. Fixes the following new sparse warnings: intel_fifo_underrun.c:340:49: warning: mixing different enum types intel_fifo_underrun.c:340:49: int enum pipe versus intel_fifo_underrun.c:340:49: int enum transcoder intel_fifo_underrun.c:344:49: warning: mixing different enum types intel_fifo_underrun.c:344:49: int enum pipe versus intel_fifo_underrun.c:344:49: int enum transcoder intel_fifo_underrun.c:397:57: warning: mixing different enum types intel_fifo_underrun.c:397:57: int enum pipe versus intel_fifo_underrun.c:397:57: int enum transcoder intel_fifo_underrun.c:398:17: warning: mixing different enum types intel_fifo_underrun.c:398:17: int enum pipe versus intel_fifo_underrun.c:398:17: int enum transcoder Cc: Matthias Kaehlcke <mka@chromium.org> Fixes: a21960339c8c ("drm/i915: Consistently use enum pipe for PCH transcoders") Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> --- drivers/gpu/drm/i915/i915_trace.h | 4 ++-- drivers/gpu/drm/i915/intel_fifo_underrun.c | 20 ++++++++++---------- 2 files changed, 12 insertions(+), 12 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_trace.h b/drivers/gpu/drm/i915/i915_trace.h index b24a83d43559..6fd5c57e21f6 100644 --- a/drivers/gpu/drm/i915/i915_trace.h +++ b/drivers/gpu/drm/i915/i915_trace.h @@ -38,7 +38,7 @@ TRACE_EVENT(intel_cpu_fifo_underrun, ); TRACE_EVENT(intel_pch_fifo_underrun, - TP_PROTO(struct drm_i915_private *dev_priv, enum transcoder pch_transcoder), + TP_PROTO(struct drm_i915_private *dev_priv, enum pipe pch_transcoder), TP_ARGS(dev_priv, pch_transcoder), TP_STRUCT__entry( @@ -48,7 +48,7 @@ TRACE_EVENT(intel_pch_fifo_underrun, ), TP_fast_assign( - enum pipe pipe = (enum pipe)pch_transcoder; + enum pipe pipe = pch_transcoder; __entry->pipe = pipe; __entry->frame = dev_priv->drm.driver->get_vblank_counter(&dev_priv->drm, pipe); __entry->scanline = intel_get_crtc_scanline(intel_get_crtc_for_pipe(dev_priv, pipe)); diff --git a/drivers/gpu/drm/i915/intel_fifo_underrun.c b/drivers/gpu/drm/i915/intel_fifo_underrun.c index 5a7cca32c0fa..04689600e337 100644 --- a/drivers/gpu/drm/i915/intel_fifo_underrun.c +++ b/drivers/gpu/drm/i915/intel_fifo_underrun.c @@ -187,11 +187,11 @@ static void broadwell_set_fifo_underrun_reporting(struct drm_device *dev, } static void ibx_set_fifo_underrun_reporting(struct drm_device *dev, - enum transcoder pch_transcoder, + enum pipe pch_transcoder, bool enable) { struct drm_i915_private *dev_priv = to_i915(dev); - uint32_t bit = (pch_transcoder == TRANSCODER_A) ? + uint32_t bit = (pch_transcoder == PIPE_A) ? SDE_TRANSA_FIFO_UNDER : SDE_TRANSB_FIFO_UNDER; if (enable) @@ -203,7 +203,7 @@ static void ibx_set_fifo_underrun_reporting(struct drm_device *dev, static void cpt_check_pch_fifo_underruns(struct intel_crtc *crtc) { struct drm_i915_private *dev_priv = to_i915(crtc->base.dev); - enum transcoder pch_transcoder = (enum transcoder) crtc->pipe; + enum pipe pch_transcoder = crtc->pipe; uint32_t serr_int = I915_READ(SERR_INT); lockdep_assert_held(&dev_priv->irq_lock); @@ -215,12 +215,12 @@ static void cpt_check_pch_fifo_underruns(struct intel_crtc *crtc) POSTING_READ(SERR_INT); trace_intel_pch_fifo_underrun(dev_priv, pch_transcoder); - DRM_ERROR("pch fifo underrun on pch transcoder %s\n", - transcoder_name(pch_transcoder)); + DRM_ERROR("pch fifo underrun on pch transcoder %c\n", + pipe_name(pch_transcoder)); } static void cpt_set_fifo_underrun_reporting(struct drm_device *dev, - enum transcoder pch_transcoder, + enum pipe pch_transcoder, bool enable, bool old) { struct drm_i915_private *dev_priv = to_i915(dev); @@ -238,8 +238,8 @@ static void cpt_set_fifo_underrun_reporting(struct drm_device *dev, if (old && I915_READ(SERR_INT) & SERR_INT_TRANS_FIFO_UNDERRUN(pch_transcoder)) { - DRM_ERROR("uncleared pch fifo underrun on pch transcoder %s\n", - transcoder_name(pch_transcoder)); + DRM_ERROR("uncleared pch fifo underrun on pch transcoder %c\n", + pipe_name(pch_transcoder)); } } } @@ -395,8 +395,8 @@ void intel_pch_fifo_underrun_irq_handler(struct drm_i915_private *dev_priv, if (intel_set_pch_fifo_underrun_reporting(dev_priv, pch_transcoder, false)) { trace_intel_pch_fifo_underrun(dev_priv, pch_transcoder); - DRM_ERROR("PCH transcoder %s FIFO underrun\n", - transcoder_name(pch_transcoder)); + DRM_ERROR("PCH transcoder %c FIFO underrun\n", + pipe_name(pch_transcoder)); } } -- 2.13.5 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH 3/3] drm/i915: Fix enum pipe vs. enum transcoder for the PCH transcoder 2017-09-01 14:31 ` [PATCH 3/3] drm/i915: Fix enum pipe vs. enum transcoder for the PCH transcoder ville.syrjala @ 2017-09-01 14:43 ` Chris Wilson 0 siblings, 0 replies; 10+ messages in thread From: Chris Wilson @ 2017-09-01 14:43 UTC (permalink / raw) To: ville.syrjala, intel-gfx; +Cc: Matthias Kaehlcke Quoting ville.syrjala@linux.intel.com (2017-09-01 15:31:23) > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > > Use enum pipe for PCH transcoders also in the FIFO underrun code. > > Fixes the following new sparse warnings: > intel_fifo_underrun.c:340:49: warning: mixing different enum types > intel_fifo_underrun.c:340:49: int enum pipe versus > intel_fifo_underrun.c:340:49: int enum transcoder > intel_fifo_underrun.c:344:49: warning: mixing different enum types > intel_fifo_underrun.c:344:49: int enum pipe versus > intel_fifo_underrun.c:344:49: int enum transcoder > intel_fifo_underrun.c:397:57: warning: mixing different enum types > intel_fifo_underrun.c:397:57: int enum pipe versus > intel_fifo_underrun.c:397:57: int enum transcoder > intel_fifo_underrun.c:398:17: warning: mixing different enum types > intel_fifo_underrun.c:398:17: int enum pipe versus > intel_fifo_underrun.c:398:17: int enum transcoder > > Cc: Matthias Kaehlcke <mka@chromium.org> > Fixes: a21960339c8c ("drm/i915: Consistently use enum pipe for PCH transcoders") > Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> C types are consistent, so Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk> I have not checked if the semantics are correct... -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/3] drm/i915: Make i9xx_load_ycbcr_conversion_matrix() static 2017-09-01 14:31 [PATCH 1/3] drm/i915: Make i9xx_load_ycbcr_conversion_matrix() static ville.syrjala 2017-09-01 14:31 ` [PATCH 2/3] drm/i915: Make i2c lock ops static ville.syrjala 2017-09-01 14:31 ` [PATCH 3/3] drm/i915: Fix enum pipe vs. enum transcoder for the PCH transcoder ville.syrjala @ 2017-09-01 14:37 ` Chris Wilson 2017-09-01 16:50 ` Ville Syrjälä 2017-09-01 15:13 ` ✓ Fi.CI.BAT: success for series starting with [1/3] " Patchwork ` (2 subsequent siblings) 5 siblings, 1 reply; 10+ messages in thread From: Chris Wilson @ 2017-09-01 14:37 UTC (permalink / raw) To: ville.syrjala, intel-gfx Quoting ville.syrjala@linux.intel.com (2017-09-01 15:31:21) > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > > Make i9xx_load_ycbcr_conversion_matrix() static to appease sparse: > intel_color.c:110:6: warning: symbol 'i9xx_load_ycbcr_conversion_matrix' was not declared. Should it be static? > > Cc: Shashank Sharma <shashank.sharma@intel.com> > Fixes: 25edf91501b8 ("drm/i915: prepare csc unit for YCBCR420 output") > Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk> -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/3] drm/i915: Make i9xx_load_ycbcr_conversion_matrix() static 2017-09-01 14:37 ` [PATCH 1/3] drm/i915: Make i9xx_load_ycbcr_conversion_matrix() static Chris Wilson @ 2017-09-01 16:50 ` Ville Syrjälä 0 siblings, 0 replies; 10+ messages in thread From: Ville Syrjälä @ 2017-09-01 16:50 UTC (permalink / raw) To: Chris Wilson; +Cc: intel-gfx On Fri, Sep 01, 2017 at 03:37:31PM +0100, Chris Wilson wrote: > Quoting ville.syrjala@linux.intel.com (2017-09-01 15:31:21) > > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > > > > Make i9xx_load_ycbcr_conversion_matrix() static to appease sparse: > > intel_color.c:110:6: warning: symbol 'i9xx_load_ycbcr_conversion_matrix' was not declared. Should it be static? > > > > Cc: Shashank Sharma <shashank.sharma@intel.com> > > Fixes: 25edf91501b8 ("drm/i915: prepare csc unit for YCBCR420 output") > > Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> > Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk> Entire series pushed to dinq. Thanks for the review. -- Ville Syrjälä Intel OTC _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 10+ messages in thread
* ✓ Fi.CI.BAT: success for series starting with [1/3] drm/i915: Make i9xx_load_ycbcr_conversion_matrix() static 2017-09-01 14:31 [PATCH 1/3] drm/i915: Make i9xx_load_ycbcr_conversion_matrix() static ville.syrjala ` (2 preceding siblings ...) 2017-09-01 14:37 ` [PATCH 1/3] drm/i915: Make i9xx_load_ycbcr_conversion_matrix() static Chris Wilson @ 2017-09-01 15:13 ` Patchwork 2017-09-01 18:04 ` ✓ Fi.CI.IGT: " Patchwork 2017-09-04 4:17 ` [PATCH 1/3] " Sharma, Shashank 5 siblings, 0 replies; 10+ messages in thread From: Patchwork @ 2017-09-01 15:13 UTC (permalink / raw) To: ville.syrjala; +Cc: intel-gfx == Series Details == Series: series starting with [1/3] drm/i915: Make i9xx_load_ycbcr_conversion_matrix() static URL : https://patchwork.freedesktop.org/series/29694/ State : success == Summary == Series 29694v1 series starting with [1/3] drm/i915: Make i9xx_load_ycbcr_conversion_matrix() static https://patchwork.freedesktop.org/api/1.0/series/29694/revisions/1/mbox/ Test kms_cursor_legacy: Subgroup basic-busy-flip-before-cursor-atomic: fail -> PASS (fi-snb-2600) fdo#100215 Test kms_flip: Subgroup basic-flip-vs-wf_vblank: pass -> FAIL (fi-blb-e6850) fdo#101599 fdo#100215 https://bugs.freedesktop.org/show_bug.cgi?id=100215 fdo#101599 https://bugs.freedesktop.org/show_bug.cgi?id=101599 fi-bdw-5557u total:288 pass:268 dwarn:0 dfail:0 fail:0 skip:20 time:453s fi-bdw-gvtdvm total:288 pass:265 dwarn:0 dfail:0 fail:0 skip:23 time:444s fi-blb-e6850 total:288 pass:223 dwarn:1 dfail:0 fail:1 skip:63 time:354s fi-bsw-n3050 total:288 pass:243 dwarn:0 dfail:0 fail:0 skip:45 time:561s fi-bwr-2160 total:288 pass:184 dwarn:0 dfail:0 fail:0 skip:104 time:256s fi-bxt-j4205 total:288 pass:260 dwarn:0 dfail:0 fail:0 skip:28 time:519s fi-byt-j1900 total:288 pass:254 dwarn:1 dfail:0 fail:0 skip:33 time:524s fi-elk-e7500 total:288 pass:230 dwarn:0 dfail:0 fail:0 skip:58 time:442s fi-glk-2a total:288 pass:260 dwarn:0 dfail:0 fail:0 skip:28 time:611s fi-hsw-4770 total:288 pass:263 dwarn:0 dfail:0 fail:0 skip:25 time:447s fi-hsw-4770r total:288 pass:263 dwarn:0 dfail:0 fail:0 skip:25 time:424s fi-ilk-650 total:288 pass:229 dwarn:0 dfail:0 fail:0 skip:59 time:429s fi-ivb-3520m total:288 pass:261 dwarn:0 dfail:0 fail:0 skip:27 time:504s fi-ivb-3770 total:288 pass:261 dwarn:0 dfail:0 fail:0 skip:27 time:477s fi-kbl-7500u total:288 pass:264 dwarn:1 dfail:0 fail:0 skip:23 time:517s fi-kbl-7560u total:288 pass:269 dwarn:0 dfail:0 fail:0 skip:19 time:596s fi-kbl-r total:288 pass:261 dwarn:0 dfail:0 fail:0 skip:27 time:599s fi-pnv-d510 total:288 pass:223 dwarn:1 dfail:0 fail:0 skip:64 time:527s fi-skl-6260u total:288 pass:269 dwarn:0 dfail:0 fail:0 skip:19 time:468s fi-skl-6700k total:288 pass:265 dwarn:0 dfail:0 fail:0 skip:23 time:541s fi-skl-6770hq total:288 pass:269 dwarn:0 dfail:0 fail:0 skip:19 time:490s fi-skl-gvtdvm total:288 pass:266 dwarn:0 dfail:0 fail:0 skip:22 time:445s fi-skl-x1585l total:288 pass:268 dwarn:0 dfail:0 fail:0 skip:20 time:489s fi-snb-2520m total:288 pass:251 dwarn:0 dfail:0 fail:0 skip:37 time:548s fi-snb-2600 total:288 pass:249 dwarn:0 dfail:0 fail:1 skip:38 time:408s fi-byt-n2820 failed to connect after reboot 8b962a9a16b3a4e3cc2a56cba82ee9c0dc0941b5 drm-tip: 2017y-09m-01d-13h-56m-06s UTC integration manifest 1360ff920a07 drm/i915: Fix enum pipe vs. enum transcoder for the PCH transcoder 4c61baa65e56 drm/i915: Make i2c lock ops static dc1138cd174e drm/i915: Make i9xx_load_ycbcr_conversion_matrix() static == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_5564/ _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 10+ messages in thread
* ✓ Fi.CI.IGT: success for series starting with [1/3] drm/i915: Make i9xx_load_ycbcr_conversion_matrix() static 2017-09-01 14:31 [PATCH 1/3] drm/i915: Make i9xx_load_ycbcr_conversion_matrix() static ville.syrjala ` (3 preceding siblings ...) 2017-09-01 15:13 ` ✓ Fi.CI.BAT: success for series starting with [1/3] " Patchwork @ 2017-09-01 18:04 ` Patchwork 2017-09-04 4:17 ` [PATCH 1/3] " Sharma, Shashank 5 siblings, 0 replies; 10+ messages in thread From: Patchwork @ 2017-09-01 18:04 UTC (permalink / raw) To: ville.syrjala; +Cc: intel-gfx == Series Details == Series: series starting with [1/3] drm/i915: Make i9xx_load_ycbcr_conversion_matrix() static URL : https://patchwork.freedesktop.org/series/29694/ State : success == Summary == Test kms_atomic_transition: Subgroup plane-use-after-nonblocking-unbind: incomplete -> FAIL (shard-hsw) fdo#101847 Test kms_setmode: Subgroup basic: pass -> FAIL (shard-hsw) fdo#99912 fdo#101847 https://bugs.freedesktop.org/show_bug.cgi?id=101847 fdo#99912 https://bugs.freedesktop.org/show_bug.cgi?id=99912 shard-hsw total:2265 pass:1232 dwarn:0 dfail:0 fail:17 skip:1016 time:9672s == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_5564/shards.html _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/3] drm/i915: Make i9xx_load_ycbcr_conversion_matrix() static 2017-09-01 14:31 [PATCH 1/3] drm/i915: Make i9xx_load_ycbcr_conversion_matrix() static ville.syrjala ` (4 preceding siblings ...) 2017-09-01 18:04 ` ✓ Fi.CI.IGT: " Patchwork @ 2017-09-04 4:17 ` Sharma, Shashank 5 siblings, 0 replies; 10+ messages in thread From: Sharma, Shashank @ 2017-09-04 4:17 UTC (permalink / raw) To: ville.syrjala, intel-gfx On 9/1/2017 8:01 PM, ville.syrjala@linux.intel.com wrote: > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > > Make i9xx_load_ycbcr_conversion_matrix() static to appease sparse: > intel_color.c:110:6: warning: symbol 'i9xx_load_ycbcr_conversion_matrix' was not declared. Should it be static? > > Cc: Shashank Sharma <shashank.sharma@intel.com> > Fixes: 25edf91501b8 ("drm/i915: prepare csc unit for YCBCR420 output") > Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> > --- > drivers/gpu/drm/i915/intel_color.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/intel_color.c b/drivers/gpu/drm/i915/intel_color.c > index 8e4e829682b9..ff9ecd211abb 100644 > --- a/drivers/gpu/drm/i915/intel_color.c > +++ b/drivers/gpu/drm/i915/intel_color.c > @@ -107,7 +107,7 @@ static void ctm_mult_by_limited(uint64_t *result, int64_t *input) > } > } > > -void i9xx_load_ycbcr_conversion_matrix(struct intel_crtc *intel_crtc) > +static void i9xx_load_ycbcr_conversion_matrix(struct intel_crtc *intel_crtc) > { Thanks for this patch. Reviewed-by: Shashank Sharma <shashank.sharma@intel.com> > int pipe = intel_crtc->pipe; > struct drm_i915_private *dev_priv = to_i915(intel_crtc->base.dev); _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2017-09-04 4:17 UTC | newest] Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2017-09-01 14:31 [PATCH 1/3] drm/i915: Make i9xx_load_ycbcr_conversion_matrix() static ville.syrjala 2017-09-01 14:31 ` [PATCH 2/3] drm/i915: Make i2c lock ops static ville.syrjala 2017-09-01 14:38 ` Chris Wilson 2017-09-01 14:31 ` [PATCH 3/3] drm/i915: Fix enum pipe vs. enum transcoder for the PCH transcoder ville.syrjala 2017-09-01 14:43 ` Chris Wilson 2017-09-01 14:37 ` [PATCH 1/3] drm/i915: Make i9xx_load_ycbcr_conversion_matrix() static Chris Wilson 2017-09-01 16:50 ` Ville Syrjälä 2017-09-01 15:13 ` ✓ Fi.CI.BAT: success for series starting with [1/3] " Patchwork 2017-09-01 18:04 ` ✓ Fi.CI.IGT: " Patchwork 2017-09-04 4:17 ` [PATCH 1/3] " Sharma, Shashank
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.