<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Jun 14, 2017 at 1:06 AM, Daniel Stone <span dir="ltr"><<a href="mailto:daniel@fooishbar.org" target="_blank">daniel@fooishbar.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi,<br>
<span class="gmail-"><br>
On 14 June 2017 at 07:46, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
> On June 13, 2017 23:19:28 Chad Versace <<a href="mailto:chad@kiwitree.net">chad@kiwitree.net</a>> wrote:<br>
</span><span class="gmail-">>> This function looks good to me.<br>
>><br>
>> Alternatively, we could define two orthogonal functions:<br>
>><br>
>>     bool<br>
>>     isl_tiling_from_drm_format_<wbr>mod(uint64_t mod, enum isl_tiling *tiling);<br>
>><br>
>>     bool<br>
>>     isl_aux_usage_from_drm_format_<wbr>mod(uint64_t mod, enum isl_aux_usage<br>
>> *aux_usage);<br>
>><br>
>> I'm just thinking out loud, though. Don't interpret that as a suggesting.<br>
><br>
> I've got an even better plan... While reworking Ben's patches for the CCS<br>
> modifier in the dri driver, I added an isl_drm_modifier_info struct and a<br>
> function to look up the info.  I'm hoping to get those patches sent out this<br>
> week.  I can provide a branch tomorrow.<br>
<br>
</span>Ah, missed this reply. I'll leave that in your hands then and wait for<br>
your patches, as well as Chad's ones to update the new protocol in a<br>
few days. Happy to review those when they arrive.<br></blockquote><div><br></div><div>For your reading pleasure:<br><br><a href="https://cgit.freedesktop.org/~jekstrand/mesa/log/?h=wip/i965-ccs-mod-v3">https://cgit.freedesktop.org/~jekstrand/mesa/log/?h=wip/i965-ccs-mod-v3</a><br><br></div><div>That contains the isl_drm_modifier_info struct I described above.  I'll be sending it to the list once I run kmscube through it to ensure that it actually does CCS as advertised.<br><br></div><div>--Jason <br></div></div><br></div></div>