pvr: Make pvr_cmd_pack() macro clearly internal

Add a double-underscore prefix to mark this macro as internal-only. It
should not be used directly, and only exists as a helper for the
pvr_csb_*() macros.

Signed-off-by: Matt Coster <matt.coster@imgtec.com>
Reviewed-by: Frank Binns <frank.binns@imgtec.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/17359>
This commit is contained in:
Matt Coster 2022-06-29 09:53:55 +01:00 committed by Marge Bot
parent 4084162ccb
commit 27c57b8650
1 changed files with 7 additions and 3 deletions

View File

@ -128,7 +128,11 @@ VkResult pvr_csb_emit_terminate(struct pvr_csb *csb);
#define PVRX(x) ROGUE_##x
#define pvr_cmd_length(x) PVRX(x##_length)
#define pvr_cmd_header(x) PVRX(x##_header)
#define pvr_cmd_pack(x) PVRX(x##_pack)
/* This helper is internal to this header. It should only be used as part of
* the pvr_csb_*() macros below.
*/
#define __pvr_cmd_pack(x) PVRX(x##_pack)
/**
* \brief Packs a command/state into one or more dwords and stores them in the
@ -147,7 +151,7 @@ VkResult pvr_csb_emit_terminate(struct pvr_csb *csb);
__builtin_expect(_loop_terminate != NULL, 1); \
({ \
STATIC_ASSERT(sizeof(*(_dst)) == pvr_cmd_length(cmd) * 4); \
pvr_cmd_pack(cmd)((_dst), &name); \
__pvr_cmd_pack(cmd)((_dst), &name); \
_loop_terminate = NULL; \
}))
@ -187,7 +191,7 @@ VkResult pvr_csb_emit_terminate(struct pvr_csb *csb);
*_dst = pvr_csb_alloc_dwords(csb, pvr_cmd_length(cmd)); \
__builtin_expect(_dst != NULL, 1); \
({ \
pvr_cmd_pack(cmd)(_dst, &name); \
__pvr_cmd_pack(cmd)(_dst, &name); \
_dst = NULL; \
}))