Re: [Intel-gfx] [PATCH 3/5] drm/i915: Remove redundant framebuffer format check

2023-01-12 Thread Ville Syrjälä
On Thu, Jan 12, 2023 at 11:07:59AM -0300, Maíra Canal wrote: > Hi, > > On 1/12/23 09:43, Ville Syrjälä wrote: > > On Mon, Jan 09, 2023 at 07:58:06AM -0300, Maíra Canal wrote: > >> Now that framebuffer_check() verifies that the format is properly > >> supported, there is no need to check it again

Re: [Intel-gfx] [PATCH 3/5] drm/i915: Remove redundant framebuffer format check

2023-01-12 Thread Maíra Canal
Hi, On 1/12/23 09:43, Ville Syrjälä wrote: On Mon, Jan 09, 2023 at 07:58:06AM -0300, Maíra Canal wrote: Now that framebuffer_check() verifies that the format is properly supported, there is no need to check it again on i915's inside helpers. Therefore, remove the redundant framebuffer format

Re: [Intel-gfx] [PATCH 3/5] drm/i915: Remove redundant framebuffer format check

2023-01-12 Thread Ville Syrjälä
On Mon, Jan 09, 2023 at 07:58:06AM -0300, Maíra Canal wrote: > Now that framebuffer_check() verifies that the format is properly > supported, there is no need to check it again on i915's inside > helpers. > > Therefore, remove the redundant framebuffer format check from the >

Re: [Intel-gfx] [PATCH 3/5] drm/i915: Remove redundant framebuffer format check

2023-01-12 Thread Simon Ser
The Intel counterpart is also: Reviewed-by: Simon Ser

[Intel-gfx] [PATCH 3/5] drm/i915: Remove redundant framebuffer format check

2023-01-09 Thread Maíra Canal
Now that framebuffer_check() verifies that the format is properly supported, there is no need to check it again on i915's inside helpers. Therefore, remove the redundant framebuffer format check from the intel_framebuffer_init() function, letting framebuffer_check() perform the framebuffer