Commit be1dcdff authored by Ohad Sharabi's avatar Ohad Sharabi Committed by Ashutosh Dixit
Browse files

drm/xe/oa: Don't use hardcoded values 



The current implementation uses hardcoded values instead of common defines.

v2:
 - Make the commit a regular commit instead of a fixup commit
 - slightly modify commit message

Signed-off-by: default avatarOhad Sharabi <osharabi@habana.ai>
Reviewed-by: default avatarAshutosh Dixit <ashutosh.dixit@intel.com>
Signed-off-by: default avatarAshutosh Dixit <ashutosh.dixit@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20240721071335.101234-1-osharabi@habana.ai
parent d930c19f

drivers/gpu/drm/xe/xe_oa.c

+1 −1
Original line number Diff line number Diff line
@@ -641,7 +641,7 @@ static void xe_oa_store_flex(struct xe_oa_stream *stream, struct xe_lrc *lrc, 
	u32 offset = xe_bo_ggtt_addr(lrc->bo);



	do {

		bb->cs[bb->len++] = MI_STORE_DATA_IMM | BIT(22) /* GGTT */ | 2;

		bb->cs[bb->len++] = MI_STORE_DATA_IMM | MI_SDI_GGTT | MI_SDI_NUM_DW(1);

		bb->cs[bb->len++] = offset + flex->offset * sizeof(u32);

		bb->cs[bb->len++] = 0;

		bb->cs[bb->len++] = flex->value;