On Wed, Jun 14, 2017 at 1:06 AM, Daniel Stone <dan...@fooishbar.org> wrote:
> Hi, > > On 14 June 2017 at 07:46, Jason Ekstrand <ja...@jlekstrand.net> wrote: > > On June 13, 2017 23:19:28 Chad Versace <c...@kiwitree.net> wrote: > >> This function looks good to me. > >> > >> Alternatively, we could define two orthogonal functions: > >> > >> bool > >> isl_tiling_from_drm_format_mod(uint64_t mod, enum isl_tiling > *tiling); > >> > >> bool > >> isl_aux_usage_from_drm_format_mod(uint64_t mod, enum isl_aux_usage > >> *aux_usage); > >> > >> I'm just thinking out loud, though. Don't interpret that as a > suggesting. > > > > I've got an even better plan... While reworking Ben's patches for the CCS > > modifier in the dri driver, I added an isl_drm_modifier_info struct and a > > function to look up the info. I'm hoping to get those patches sent out > this > > week. I can provide a branch tomorrow. > > Ah, missed this reply. I'll leave that in your hands then and wait for > your patches, as well as Chad's ones to update the new protocol in a > few days. Happy to review those when they arrive. > For your reading pleasure: https://cgit.freedesktop.org/~jekstrand/mesa/log/?h=wip/i965-ccs-mod-v3 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. --Jason
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
