[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