[PATCH i-g-t v2 08/10] lib/intel_cmds_info: Introduce render tilings

Zbigniew Kempczyński zbigniew.kempczynski at intel.com
Tue May 7 05:49:04 UTC 2024


On Fri, Apr 26, 2024 at 03:18:25PM +0200, Karolina Stolarek wrote:
> 
> On 26.04.2024 11:01, Zbigniew Kempczyński wrote:> +struct render_tiling_info
> {
> > +	uint32_t supported_tiling;
> > +	uint32_t supported_compressed_tiling;
> > +};
> > +
> >   struct intel_cmds_info {
> >   	struct blt_cmd_info const *blt_cmds[__BLT_MAX_CMD];
> > +	struct render_tiling_info const *render_tilings;
> 
> These two structs should share the naming convention, as they describe
> the same thing but for two different copy operations.

Not exactly. blt_cmd_info keeps relation between commands and tilings
whereas render_tiling_info is just a list of supported tilings
(there's single render command so there's useless to define it as
a single element array to keep the convention).

--
Zbigniew

> 
> There are two ways about it -- either rename blt_cmd_info to
> blt_tiling_info in a separate patch or make the render one to be
> something like render_cmd_info.
> 
> All the best,
> Karolina
> 
> >   };
> >   extern const struct intel_cmds_info pre_gen6_cmds_info;


More information about the igt-dev mailing list