On Fri 28 Aug 2015, Nanley Chery wrote: > I should add that this patch enables this function to handle ASTC formats. > This in turn enables such formats to be used for GL calls such as > Tex*Storage*. > > - Nanley > > On Wed, Aug 26, 2015 at 2:38 PM, Nanley Chery <nanleych...@gmail.com> wrote: > > > From: Nanley Chery <nanley.g.ch...@intel.com> > > > > This function's cases for non-generic compressed formats duplicate > > the GL to MESA translation in _mesa_glenum_to_compressed_format(). > > This patch replaces the switch cases with a call to the translation > > function. > > > > Cc: Brian Paul <bri...@vmware.com> > > Cc: Chad Versace <chad.vers...@intel.com> > > Cc: Ian Romanick <ian.d.roman...@intel.com> > > Signed-off-by: Nanley Chery <nanley.g.ch...@intel.com> > > --- > > src/mesa/main/texformat.c | 94 > > +++++++---------------------------------------- > > 1 file changed, 13 insertions(+), 81 deletions(-)
This patch looks good to me. Please comment in the commit message that this change teaches the function about ASTC, thus enabling ASTC for glTex*Storage*() calls. And then this patch will be Reviewed-by: Chad Versace <chad.vers...@intel.com> _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
