Re: [Intel-gfx] [PATCH] drm/i915: fix an error code in intel_modeset_all_tiles()
On Wed, 08 Jan 2020, Chris Wilson wrote:
> Quoting Jani Nikula (2020-01-08 14:33:22)
>> On Tue, 07 Jan 2020, Chris Wilson wrote:
>> > Quoting Souza, Jose (2020-01-07 14:01:40)
>> >> On Tue, 2020-01-07 at 16:03 +0300, Dan Carpenter wrote:
>> >> > There is a cut and paste bug so we return the wrong error code.
>> >> >
>> >> > Fixes: a603f5bd1691 ("drm/i915/dp: Make sure all tiled connectors get
>> >> > added to the state with full modeset")
>> >>
>> >> Reviewed-by: José Roberto de Souza
>> >>
>> >> > Signed-off-by: Dan Carpenter
>> >
>> > Pushed. Thanks for the patch,
>>
>> Hrmh, we are still supposed to wait for CI results for every patch, no
>> exceptions.
>
> There is no coverage of that path in CI.
Maybe not, but making that assessment is not obvious to me.
I don't want to start on the slippery slope of people pushing untested
patches claiming "no coverage".
BR,
Jani.
--
Jani Nikula, Intel Open Source Graphics Center
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Re: [Intel-gfx] [PATCH] drm/i915: fix an error code in intel_modeset_all_tiles()
Quoting Jani Nikula (2020-01-08 14:33:22)
> On Tue, 07 Jan 2020, Chris Wilson wrote:
> > Quoting Souza, Jose (2020-01-07 14:01:40)
> >> On Tue, 2020-01-07 at 16:03 +0300, Dan Carpenter wrote:
> >> > There is a cut and paste bug so we return the wrong error code.
> >> >
> >> > Fixes: a603f5bd1691 ("drm/i915/dp: Make sure all tiled connectors get
> >> > added to the state with full modeset")
> >>
> >> Reviewed-by: José Roberto de Souza
> >>
> >> > Signed-off-by: Dan Carpenter
> >
> > Pushed. Thanks for the patch,
>
> Hrmh, we are still supposed to wait for CI results for every patch, no
> exceptions.
There is no coverage of that path in CI.
-Chris
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Re: [Intel-gfx] [PATCH] drm/i915: fix an error code in intel_modeset_all_tiles()
On Tue, 07 Jan 2020, Chris Wilson wrote:
> Quoting Souza, Jose (2020-01-07 14:01:40)
>> On Tue, 2020-01-07 at 16:03 +0300, Dan Carpenter wrote:
>> > There is a cut and paste bug so we return the wrong error code.
>> >
>> > Fixes: a603f5bd1691 ("drm/i915/dp: Make sure all tiled connectors get
>> > added to the state with full modeset")
>>
>> Reviewed-by: José Roberto de Souza
>>
>> > Signed-off-by: Dan Carpenter
>
> Pushed. Thanks for the patch,
Hrmh, we are still supposed to wait for CI results for every patch, no
exceptions.
If it's not "every patch", it's going to be a subjective assessment, and
I really don't like where that would lead. Who's going to say what's
going to be just fine. Even if it's "obviously correct".
As it is, this patch won't get the CI results at all now because BAT
fails with "patch already applied". Tracing any issues back to this
patch would mean bisecting the results from drm-tip runs.
---
If the problem is that CI is too heavy, slow and itself error prone for
small patches, then that's what we need to address instead of just
bypassing CI.
I also don't like the prospect of spending time on hacking dim to
double-check the patches passed CI before pushing.
BR,
Jani.
--
Jani Nikula, Intel Open Source Graphics Center
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Re: [Intel-gfx] [PATCH] drm/i915: fix an error code in intel_modeset_all_tiles()
Quoting Souza, Jose (2020-01-07 14:01:40)
> On Tue, 2020-01-07 at 16:03 +0300, Dan Carpenter wrote:
> > There is a cut and paste bug so we return the wrong error code.
> >
> > Fixes: a603f5bd1691 ("drm/i915/dp: Make sure all tiled connectors get
> > added to the state with full modeset")
>
> Reviewed-by: José Roberto de Souza
>
> > Signed-off-by: Dan Carpenter
Pushed. Thanks for the patch,
-Chris
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Re: [Intel-gfx] [PATCH] drm/i915: fix an error code in intel_modeset_all_tiles()
On Tue, 2020-01-07 at 16:03 +0300, Dan Carpenter wrote:
> There is a cut and paste bug so we return the wrong error code.
>
> Fixes: a603f5bd1691 ("drm/i915/dp: Make sure all tiled connectors get
> added to the state with full modeset")
Reviewed-by: José Roberto de Souza
> Signed-off-by: Dan Carpenter
> ---
> drivers/gpu/drm/i915/display/intel_display.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_display.c
> b/drivers/gpu/drm/i915/display/intel_display.c
> index da5266e76738..a96bee699a5e 100644
> --- a/drivers/gpu/drm/i915/display/intel_display.c
> +++ b/drivers/gpu/drm/i915/display/intel_display.c
> @@ -14424,7 +14424,7 @@ intel_modeset_all_tiles(struct
> intel_atomic_state *state, int tile_grp_id)
> crtc_state = drm_atomic_get_crtc_state(&state->base,
> conn_state-
> >crtc);
> if (IS_ERR(crtc_state)) {
> - ret = PTR_ERR(conn_state);
> + ret = PTR_ERR(crtc_state);
> break;
> }
> crtc_state->mode_changed = true;
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
