[Intel-gfx] Pushing commits to drm-intel-next-queued and questions about CI

[Hans de Goede]

Hi,

On 07-11-17 17:50, Jani Nikula wrote:
> On Tue, 07 Nov 2017, Hans de Goede <hdegoede at redhat.com> wrote:
>> Hi,
>>
>> On 03-11-17 20:11, Hans de Goede wrote:
>>> Hi,
>>>
>>> On 03-11-17 19:51, Hans de Goede wrote:
>>>> Hi,
>>>>
>>>> On 03-11-17 18:40, Rodrigo Vivi wrote:
>>>>> Hi Hans,
>>>>>
>>>>> On Fri, Nov 03, 2017 at 10:41:55AM +0000, Hans de Goede wrote:
>>>>>> Hi Daniel,
>>>>>>
>>>>>> A while ago I was granted commit rights to the drm-intel repo.
>>>>>> So far I've not used these, but since no-one seems to be
>>>>>> pushing these 2 sets, I guess now might be a good time to
>>>>>> learn how to push things myself.
>>>>>>
>>>>>> I'm talking about these 2 patch-sets:
>>>>>>
>>>>>> https://patchwork.freedesktop.org/series/32274/
>>>>>
>>>>> For this one we got the ack from Ingo, but I missed the reviewed-by
>>>>> on the second one that is the i915 part of it.
>>>>> Do we need it or just that ack was enough for both patches?
>>>>
>>>> With "this one" I assume you mean the one below:
>>>>
>>>>>> https://patchwork.freedesktop.org/series/32288/
>>>>
>>>> As that is the one which has an ack from Ingo, the second
>>>> one has had a reviewed-by from Imre Deak for a few
>>>> versions already, it is right there in patch work.
>>>>
>>>> Thank you for rescheduling the tests, this one passes
>>>> Fi.CI.BAT now, but I did not see Fi.CI.IGD getting run?
>>>
>>> Ok I just got a success report for Fi.CI.IGD too, so I
>>> guess I can go and push this now ?
>>
>> Erm, ping? So given the patches have a Reviewed-by /
>> Acked-by and Fi.CI.Bat and Fi.CI.IGD both pass:
>>
>> https://patchwork.freedesktop.org/series/32288/
>>
>> Can I push these (and others in similar circumstances in
>> the future) ?
> 
> Yes, as long as you use dim for that [1].

Ah ok, thank you for pointing that out, it was told me when I
got commit rights, but that was a while ago. Ok, I've pushed
the 2 patches from:

https://patchwork.freedesktop.org/series/32288/

Using dim now.

Regards,

Hans