Re: [Mesa-dev] [PATCH mesa] gallium: simplify util_format_is_supported()

2018-06-18 Thread Marek Olšák
The function can be removed completely, because it doesn't do anything now. Marek On Mon, Jun 18, 2018 at 10:55 AM, Eric Engestrom wrote: > On Monday, 2018-06-18 07:31:49 -0700, Matt Turner wrote: > > On Mon, Jun 18, 2018 at 3:15 AM Eric Engestrom > wrote: > > > > > > As of 66673bef941af344314

Re: [Mesa-dev] [PATCH mesa] gallium: simplify util_format_is_supported()

2018-06-18 Thread Eric Engestrom
On Monday, 2018-06-18 07:31:49 -0700, Matt Turner wrote: > On Mon, Jun 18, 2018 at 3:15 AM Eric Engestrom > wrote: > > > > As of 66673bef941af344314f ("mesa: Unconditionally enable floating-point > > textures"), the `bind` argument is no longer used, and the format check > > is now a simple enum

Re: [Mesa-dev] [PATCH mesa] gallium: simplify util_format_is_supported()

2018-06-18 Thread Matt Turner
On Mon, Jun 18, 2018 at 3:15 AM Eric Engestrom wrote: > > As of 66673bef941af344314f ("mesa: Unconditionally enable floating-point > textures"), the `bind` argument is no longer used, and the format check > is now a simple enum range check, so inline it and remove the > unnecessary argument from c

Re: [Mesa-dev] [PATCH mesa] gallium: simplify util_format_is_supported()

2018-06-18 Thread Timothy Arceri
Reviewed-by: Timothy Arceri On 18/06/18 20:15, Eric Engestrom wrote: As of 66673bef941af344314f ("mesa: Unconditionally enable floating-point textures"), the `bind` argument is no longer used, and the format check is now a simple enum range check, so inline it and remove the unnecessary argumen

[Mesa-dev] [PATCH mesa] gallium: simplify util_format_is_supported()

2018-06-18 Thread Eric Engestrom
As of 66673bef941af344314f ("mesa: Unconditionally enable floating-point textures"), the `bind` argument is no longer used, and the format check is now a simple enum range check, so inline it and remove the unnecessary argument from callers. Cc: Timothy Arceri Cc: Matt Turner Cc: Ian Romanick S