[PATCH v5 1/4] drm/xe: Define XE_REG_IS_VALID
Nilawar, Badal
badal.nilawar at intel.com
Thu Apr 4 04:04:31 UTC 2024
On 04-04-2024 09:24, Karthik Poosa wrote:
> Add macro to check if struct xe_reg has valid address. (Lucas, Badal).
Usually we don't add (reviewer names) in top level commit message. You
can use Cc: or Suggested-by: which ever appropriate.
>
> Signed-off-by: Karthik Poosa <karthik.poosa at intel.com>
> ---
> drivers/gpu/drm/xe/regs/xe_reg_defs.h | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/gpu/drm/xe/regs/xe_reg_defs.h b/drivers/gpu/drm/xe/regs/xe_reg_defs.h
> index c89ef2b79a3f..42078643be6b 100644
> --- a/drivers/gpu/drm/xe/regs/xe_reg_defs.h
> +++ b/drivers/gpu/drm/xe/regs/xe_reg_defs.h
> @@ -131,4 +131,6 @@ struct xe_reg_mcr {
> .__reg = XE_REG_INITIALIZER(r_, ##__VA_ARGS__, .mcr = 1) \
> })
>
> +#define XE_REG_IS_VALID(r) ((r.addr) ? true : false)
Looks good to me. Please maintain this RB if you float new revisions.
Reviewed-by: Badal Nilawar <badal.nilawar at intel.com>
> +
> #endif
More information about the Intel-xe
mailing list