<div dir="auto">Thank you. <div dir="auto"><br></div><div dir="auto">I have closed <a href="https://bugs.freedesktop.org/show_bug.cgi?id=102552">https://bugs.freedesktop.org/show_bug.cgi?id=102552</a><br><div data-smartmail="gmail_signature" dir="auto"><br>Best regards,<br>                  Denis.</div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sep 14, 2017 2:01 AM, "Marek Olšák" <<a href="mailto:maraeo@gmail.com">maraeo@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I pushed the patch. Thanks.<br>
<br>
Marek<br>
<br>
On Wed, Sep 13, 2017 at 9:27 PM, Denis Pauk <<a href="mailto:pauk.denis@gmail.com">pauk.denis@gmail.com</a>> wrote:<br>
> Could you please commit this changes?<br>
><br>
> I have not found what i need to do next after recieve "approve" for changes<br>
> in <a href="https://www.mesa3d.org/submittingpatches.html#reviewing" rel="noreferrer" target="_blank">https://www.mesa3d.org/<wbr>submittingpatches.html#<wbr>reviewing</a>.<br>
> Do i need to send new mail with "Reviewed-by: " mark?<br>
><br>
> On Wed, Sep 13, 2017 at 6:36 PM, Marek Olšák <<a href="mailto:maraeo@gmail.com">maraeo@gmail.com</a>> wrote:<br>
>><br>
>> I've changed my mind. The patch is OK:<br>
>><br>
>> Reviewed-by: Marek Olšák <<a href="mailto:marek.olsak@amd.com">marek.olsak@amd.com</a>><br>
>><br>
>> Marek<br>
>><br>
>> On Wed, Sep 13, 2017 at 4:45 PM, Denis Pauk <<a href="mailto:pauk.denis@gmail.com">pauk.denis@gmail.com</a>> wrote:<br>
>> > What do you think about replace all checks in patch to asserts?<br>
>> ><br>
>> ><br>
>> > Best regards,<br>
>> >                   Denis.<br>
>> ><br>
>> > On Sep 13, 2017 1:00 PM, "Marek Olšák" <<a href="mailto:maraeo@gmail.com">maraeo@gmail.com</a>> wrote:<br>
>> >><br>
>> >> On Wed, Sep 13, 2017 at 6:54 AM, Денис Паук <<a href="mailto:pauk.denis@gmail.com">pauk.denis@gmail.com</a>><br>
>> >> wrote:<br>
>> >> > Do you mean delete check in u_format.c:: util_format_is_supported?<br>
>> >> > Could<br>
>> >> > you<br>
>> >> > please explain more?<br>
>> >><br>
>> >> I mean delete everything. Invalid formats shouldn't be passed to<br>
>> >> pipe_screen::is_format_<wbr>supported. The code doing it needs to be fixed.<br>
>> >><br>
>> >> Marek<br>
><br>
><br>
><br>
><br>
> --<br>
> Best regards,<br>
>                   Denis.<br>
</blockquote></div></div>