[PATCH v2 21/43] drm/xe/guc: Convert register access to use xe_mmio
Rodrigo Vivi
rodrigo.vivi at intel.com
Tue Sep 10 18:48:48 UTC 2024
On Fri, Sep 06, 2024 at 05:08:10PM -0700, Matt Roper wrote:
> Stop using GT pointers for register access.
>
> Since GuC was the only part of the driver using xe_mmio_wait32_not(), we
> can also drop the _Generic wrapper macro for that function as well.
>
> Signed-off-by: Matt Roper <matthew.d.roper at intel.com>
> ---
> drivers/gpu/drm/xe/xe_guc.c | 60 ++++++++++++++++++---------------
> drivers/gpu/drm/xe/xe_guc_ads.c | 2 +-
> drivers/gpu/drm/xe/xe_guc_pc.c | 34 +++++++++----------
> drivers/gpu/drm/xe/xe_mmio.c | 4 +--
> drivers/gpu/drm/xe/xe_mmio.h | 6 ++--
> 5 files changed, 54 insertions(+), 52 deletions(-)
>
> diff --git a/drivers/gpu/drm/xe/xe_guc.c b/drivers/gpu/drm/xe/xe_guc.c
> index 5599464013bd..1eb5bb7e8771 100644
> --- a/drivers/gpu/drm/xe/xe_guc.c
> +++ b/drivers/gpu/drm/xe/xe_guc.c
> @@ -236,10 +236,10 @@ static void guc_write_params(struct xe_guc *guc)
>
> xe_force_wake_assert_held(gt_to_fw(gt), XE_FW_GT);
>
> - xe_mmio_write32(gt, SOFT_SCRATCH(0), 0);
> + xe_mmio_write32(>->mmio, SOFT_SCRATCH(0), 0);
>
> for (i = 0; i < GUC_CTL_MAX_DWORDS; i++)
> - xe_mmio_write32(gt, SOFT_SCRATCH(1 + i), guc->params[i]);
> + xe_mmio_write32(>->mmio, SOFT_SCRATCH(1 + i), guc->params[i]);
> }
>
> static void guc_fini_hw(void *arg)
> @@ -425,6 +425,7 @@ int xe_guc_post_load_init(struct xe_guc *guc)
> int xe_guc_reset(struct xe_guc *guc)
> {
> struct xe_gt *gt = guc_to_gt(guc);
> + struct xe_mmio *mmio = >->mmio;
> u32 guc_status, gdrst;
> int ret;
>
> @@ -433,15 +434,15 @@ int xe_guc_reset(struct xe_guc *guc)
> if (IS_SRIOV_VF(gt_to_xe(gt)))
> return xe_gt_sriov_vf_bootstrap(gt);
>
> - xe_mmio_write32(gt, GDRST, GRDOM_GUC);
> + xe_mmio_write32(mmio, GDRST, GRDOM_GUC);
>
> - ret = xe_mmio_wait32(gt, GDRST, GRDOM_GUC, 0, 5000, &gdrst, false);
> + ret = xe_mmio_wait32(mmio, GDRST, GRDOM_GUC, 0, 5000, &gdrst, false);
> if (ret) {
> xe_gt_err(gt, "GuC reset timed out, GDRST=%#x\n", gdrst);
> goto err_out;
> }
>
> - guc_status = xe_mmio_read32(gt, GUC_STATUS);
> + guc_status = xe_mmio_read32(mmio, GUC_STATUS);
> if (!(guc_status & GS_MIA_IN_RESET)) {
> xe_gt_err(gt, "GuC status: %#x, MIA core expected to be in reset\n",
> guc_status);
> @@ -459,6 +460,7 @@ int xe_guc_reset(struct xe_guc *guc)
> static void guc_prepare_xfer(struct xe_guc *guc)
> {
> struct xe_gt *gt = guc_to_gt(guc);
> + struct xe_mmio *mmio = >->mmio;
> struct xe_device *xe = guc_to_xe(guc);
> u32 shim_flags = GUC_ENABLE_READ_CACHE_LOGIC |
> GUC_ENABLE_READ_CACHE_FOR_SRAM_DATA |
> @@ -473,12 +475,12 @@ static void guc_prepare_xfer(struct xe_guc *guc)
> shim_flags |= REG_FIELD_PREP(GUC_MOCS_INDEX_MASK, gt->mocs.uc_index);
>
> /* Must program this register before loading the ucode with DMA */
> - xe_mmio_write32(gt, GUC_SHIM_CONTROL, shim_flags);
> + xe_mmio_write32(mmio, GUC_SHIM_CONTROL, shim_flags);
>
> - xe_mmio_write32(gt, GT_PM_CONFIG, GT_DOORBELL_ENABLE);
> + xe_mmio_write32(mmio, GT_PM_CONFIG, GT_DOORBELL_ENABLE);
>
> /* Make sure GuC receives ARAT interrupts */
> - xe_mmio_rmw32(gt, PMINTRMSK, ARAT_EXPIRED_INTRMSK, 0);
> + xe_mmio_rmw32(mmio, PMINTRMSK, ARAT_EXPIRED_INTRMSK, 0);
> }
>
> /*
> @@ -494,7 +496,7 @@ static int guc_xfer_rsa(struct xe_guc *guc)
> if (guc->fw.rsa_size > 256) {
> u32 rsa_ggtt_addr = xe_bo_ggtt_addr(guc->fw.bo) +
> xe_uc_fw_rsa_offset(&guc->fw);
> - xe_mmio_write32(gt, UOS_RSA_SCRATCH(0), rsa_ggtt_addr);
> + xe_mmio_write32(>->mmio, UOS_RSA_SCRATCH(0), rsa_ggtt_addr);
> return 0;
> }
>
> @@ -503,7 +505,7 @@ static int guc_xfer_rsa(struct xe_guc *guc)
> return -ENOMEM;
>
> for (i = 0; i < UOS_RSA_SCRATCH_COUNT; i++)
> - xe_mmio_write32(gt, UOS_RSA_SCRATCH(i), rsa[i]);
> + xe_mmio_write32(>->mmio, UOS_RSA_SCRATCH(i), rsa[i]);
>
> return 0;
> }
> @@ -593,6 +595,7 @@ static s32 guc_pc_get_cur_freq(struct xe_guc_pc *guc_pc)
> static void guc_wait_ucode(struct xe_guc *guc)
> {
> struct xe_gt *gt = guc_to_gt(guc);
> + struct xe_mmio *mmio = >->mmio;
> struct xe_guc_pc *guc_pc = >->uc.guc.pc;
> ktime_t before, after, delta;
> int load_done;
> @@ -619,7 +622,7 @@ static void guc_wait_ucode(struct xe_guc *guc)
> * timeouts rather than allowing a huge timeout each time. So basically, need
> * to treat a timeout no different to a value change.
> */
> - ret = xe_mmio_wait32_not(gt, GUC_STATUS, GS_UKERNEL_MASK | GS_BOOTROM_MASK,
> + ret = xe_mmio_wait32_not(mmio, GUC_STATUS, GS_UKERNEL_MASK | GS_BOOTROM_MASK,
> last_status, 1000 * 1000, &status, false);
> if (ret < 0)
> count++;
> @@ -657,7 +660,7 @@ static void guc_wait_ucode(struct xe_guc *guc)
> switch (bootrom) {
> case XE_BOOTROM_STATUS_NO_KEY_FOUND:
> xe_gt_err(gt, "invalid key requested, header = 0x%08X\n",
> - xe_mmio_read32(gt, GUC_HEADER_INFO));
> + xe_mmio_read32(mmio, GUC_HEADER_INFO));
> break;
>
> case XE_BOOTROM_STATUS_RSA_FAILED:
> @@ -672,7 +675,7 @@ static void guc_wait_ucode(struct xe_guc *guc)
> switch (ukernel) {
> case XE_GUC_LOAD_STATUS_EXCEPTION:
> xe_gt_err(gt, "firmware exception. EIP: %#x\n",
> - xe_mmio_read32(gt, SOFT_SCRATCH(13)));
> + xe_mmio_read32(mmio, SOFT_SCRATCH(13)));
> break;
>
> case XE_GUC_LOAD_STATUS_INIT_MMIO_SAVE_RESTORE_INVALID:
> @@ -824,10 +827,10 @@ static void guc_handle_mmio_msg(struct xe_guc *guc)
>
> xe_force_wake_assert_held(gt_to_fw(gt), XE_FW_GT);
>
> - msg = xe_mmio_read32(gt, SOFT_SCRATCH(15));
> + msg = xe_mmio_read32(>->mmio, SOFT_SCRATCH(15));
> msg &= XE_GUC_RECV_MSG_EXCEPTION |
> XE_GUC_RECV_MSG_CRASH_DUMP_POSTED;
> - xe_mmio_write32(gt, SOFT_SCRATCH(15), 0);
> + xe_mmio_write32(>->mmio, SOFT_SCRATCH(15), 0);
>
> if (msg & XE_GUC_RECV_MSG_CRASH_DUMP_POSTED)
> xe_gt_err(gt, "Received early GuC crash dump notification!\n");
> @@ -844,14 +847,14 @@ static void guc_enable_irq(struct xe_guc *guc)
> REG_FIELD_PREP(ENGINE1_MASK, GUC_INTR_GUC2HOST);
>
> /* Primary GuC and media GuC share a single enable bit */
> - xe_mmio_write32(gt, GUC_SG_INTR_ENABLE,
> + xe_mmio_write32(>->mmio, GUC_SG_INTR_ENABLE,
> REG_FIELD_PREP(ENGINE1_MASK, GUC_INTR_GUC2HOST));
>
> /*
> * There are separate mask bits for primary and media GuCs, so use
> * a RMW operation to avoid clobbering the other GuC's setting.
> */
> - xe_mmio_rmw32(gt, GUC_SG_INTR_MASK, events, 0);
> + xe_mmio_rmw32(>->mmio, GUC_SG_INTR_MASK, events, 0);
> }
>
> int xe_guc_enable_communication(struct xe_guc *guc)
> @@ -907,7 +910,7 @@ void xe_guc_notify(struct xe_guc *guc)
> * additional payload data to the GuC but this capability is not
> * used by the firmware yet. Use default value in the meantime.
> */
> - xe_mmio_write32(gt, guc->notify_reg, default_notify_data);
> + xe_mmio_write32(>->mmio, guc->notify_reg, default_notify_data);
> }
>
> int xe_guc_auth_huc(struct xe_guc *guc, u32 rsa_addr)
> @@ -925,6 +928,7 @@ int xe_guc_mmio_send_recv(struct xe_guc *guc, const u32 *request,
> {
> struct xe_device *xe = guc_to_xe(guc);
> struct xe_gt *gt = guc_to_gt(guc);
> + struct xe_mmio *mmio = >->mmio;
> u32 header, reply;
> struct xe_reg reply_reg = xe_gt_is_media_type(gt) ?
> MED_VF_SW_FLAG(0) : VF_SW_FLAG(0);
> @@ -947,19 +951,19 @@ int xe_guc_mmio_send_recv(struct xe_guc *guc, const u32 *request,
> /* Not in critical data-path, just do if else for GT type */
> if (xe_gt_is_media_type(gt)) {
> for (i = 0; i < len; ++i)
> - xe_mmio_write32(gt, MED_VF_SW_FLAG(i),
> + xe_mmio_write32(mmio, MED_VF_SW_FLAG(i),
> request[i]);
> - xe_mmio_read32(gt, MED_VF_SW_FLAG(LAST_INDEX));
> + xe_mmio_read32(mmio, MED_VF_SW_FLAG(LAST_INDEX));
> } else {
> for (i = 0; i < len; ++i)
> - xe_mmio_write32(gt, VF_SW_FLAG(i),
> + xe_mmio_write32(mmio, VF_SW_FLAG(i),
> request[i]);
> - xe_mmio_read32(gt, VF_SW_FLAG(LAST_INDEX));
> + xe_mmio_read32(mmio, VF_SW_FLAG(LAST_INDEX));
> }
>
> xe_guc_notify(guc);
>
> - ret = xe_mmio_wait32(gt, reply_reg, GUC_HXG_MSG_0_ORIGIN,
> + ret = xe_mmio_wait32(mmio, reply_reg, GUC_HXG_MSG_0_ORIGIN,
> FIELD_PREP(GUC_HXG_MSG_0_ORIGIN, GUC_HXG_ORIGIN_GUC),
> 50000, &reply, false);
> if (ret) {
> @@ -969,7 +973,7 @@ int xe_guc_mmio_send_recv(struct xe_guc *guc, const u32 *request,
> return ret;
> }
>
> - header = xe_mmio_read32(gt, reply_reg);
> + header = xe_mmio_read32(mmio, reply_reg);
> if (FIELD_GET(GUC_HXG_MSG_0_TYPE, header) ==
> GUC_HXG_TYPE_NO_RESPONSE_BUSY) {
> /*
> @@ -985,7 +989,7 @@ int xe_guc_mmio_send_recv(struct xe_guc *guc, const u32 *request,
> BUILD_BUG_ON(FIELD_MAX(GUC_HXG_MSG_0_TYPE) != GUC_HXG_TYPE_RESPONSE_SUCCESS);
> BUILD_BUG_ON((GUC_HXG_TYPE_RESPONSE_SUCCESS ^ GUC_HXG_TYPE_RESPONSE_FAILURE) != 1);
>
> - ret = xe_mmio_wait32(gt, reply_reg, resp_mask, resp_mask,
> + ret = xe_mmio_wait32(mmio, reply_reg, resp_mask, resp_mask,
> 1000000, &header, false);
>
> if (unlikely(FIELD_GET(GUC_HXG_MSG_0_ORIGIN, header) !=
> @@ -1032,7 +1036,7 @@ int xe_guc_mmio_send_recv(struct xe_guc *guc, const u32 *request,
>
> for (i = 1; i < VF_SW_FLAG_COUNT; i++) {
> reply_reg.addr += sizeof(u32);
> - response_buf[i] = xe_mmio_read32(gt, reply_reg);
> + response_buf[i] = xe_mmio_read32(mmio, reply_reg);
> }
> }
>
> @@ -1155,7 +1159,7 @@ void xe_guc_print_info(struct xe_guc *guc, struct drm_printer *p)
> if (err)
> return;
>
> - status = xe_mmio_read32(gt, GUC_STATUS);
> + status = xe_mmio_read32(>->mmio, GUC_STATUS);
>
> drm_printf(p, "\nGuC status 0x%08x:\n", status);
> drm_printf(p, "\tBootrom status = 0x%x\n",
> @@ -1170,7 +1174,7 @@ void xe_guc_print_info(struct xe_guc *guc, struct drm_printer *p)
> drm_puts(p, "\nScratch registers:\n");
> for (i = 0; i < SOFT_SCRATCH_COUNT; i++) {
> drm_printf(p, "\t%2d: \t0x%x\n",
> - i, xe_mmio_read32(gt, SOFT_SCRATCH(i)));
> + i, xe_mmio_read32(>->mmio, SOFT_SCRATCH(i)));
> }
>
> xe_force_wake_put(gt_to_fw(gt), XE_FW_GT);
> diff --git a/drivers/gpu/drm/xe/xe_guc_ads.c b/drivers/gpu/drm/xe/xe_guc_ads.c
> index d1902a8581ca..66d4e5e95abd 100644
> --- a/drivers/gpu/drm/xe/xe_guc_ads.c
> +++ b/drivers/gpu/drm/xe/xe_guc_ads.c
> @@ -684,7 +684,7 @@ static void guc_doorbell_init(struct xe_guc_ads *ads)
>
> if (GRAPHICS_VER(xe) >= 12 && !IS_DGFX(xe)) {
> u32 distdbreg =
> - xe_mmio_read32(gt, DIST_DBS_POPULATED);
> + xe_mmio_read32(>->mmio, DIST_DBS_POPULATED);
>
> ads_blob_write(ads,
> system_info.generic_gt_sysinfo[GUC_GENERIC_GT_SYSINFO_DOORBELL_COUNT_PER_SQIDI],
> diff --git a/drivers/gpu/drm/xe/xe_guc_pc.c b/drivers/gpu/drm/xe/xe_guc_pc.c
> index 034b29984d5e..2b654f820ae2 100644
> --- a/drivers/gpu/drm/xe/xe_guc_pc.c
> +++ b/drivers/gpu/drm/xe/xe_guc_pc.c
> @@ -262,7 +262,7 @@ static void pc_set_manual_rp_ctrl(struct xe_guc_pc *pc, bool enable)
> u32 state = enable ? RPSWCTL_ENABLE : RPSWCTL_DISABLE;
>
> /* Allow/Disallow punit to process software freq requests */
> - xe_mmio_write32(gt, RP_CONTROL, state);
> + xe_mmio_write32(>->mmio, RP_CONTROL, state);
> }
>
> static void pc_set_cur_freq(struct xe_guc_pc *pc, u32 freq)
> @@ -274,7 +274,7 @@ static void pc_set_cur_freq(struct xe_guc_pc *pc, u32 freq)
>
> /* Req freq is in units of 16.66 Mhz */
> rpnswreq = REG_FIELD_PREP(REQ_RATIO_MASK, encode_freq(freq));
> - xe_mmio_write32(gt, RPNSWREQ, rpnswreq);
> + xe_mmio_write32(>->mmio, RPNSWREQ, rpnswreq);
>
> /* Sleep for a small time to allow pcode to respond */
> usleep_range(100, 300);
> @@ -334,9 +334,9 @@ static void mtl_update_rpe_value(struct xe_guc_pc *pc)
> u32 reg;
>
> if (xe_gt_is_media_type(gt))
> - reg = xe_mmio_read32(gt, MTL_MPE_FREQUENCY);
> + reg = xe_mmio_read32(>->mmio, MTL_MPE_FREQUENCY);
> else
> - reg = xe_mmio_read32(gt, MTL_GT_RPE_FREQUENCY);
> + reg = xe_mmio_read32(>->mmio, MTL_GT_RPE_FREQUENCY);
>
> pc->rpe_freq = decode_freq(REG_FIELD_GET(MTL_RPE_MASK, reg));
> }
> @@ -353,9 +353,9 @@ static void tgl_update_rpe_value(struct xe_guc_pc *pc)
> * PCODE at a different register
> */
> if (xe->info.platform == XE_PVC)
> - reg = xe_mmio_read32(gt, PVC_RP_STATE_CAP);
> + reg = xe_mmio_read32(>->mmio, PVC_RP_STATE_CAP);
> else
> - reg = xe_mmio_read32(gt, FREQ_INFO_REC);
> + reg = xe_mmio_read32(>->mmio, FREQ_INFO_REC);
>
> pc->rpe_freq = REG_FIELD_GET(RPE_MASK, reg) * GT_FREQUENCY_MULTIPLIER;
> }
> @@ -392,10 +392,10 @@ u32 xe_guc_pc_get_act_freq(struct xe_guc_pc *pc)
>
> /* When in RC6, actual frequency reported will be 0. */
> if (GRAPHICS_VERx100(xe) >= 1270) {
> - freq = xe_mmio_read32(gt, MTL_MIRROR_TARGET_WP1);
> + freq = xe_mmio_read32(>->mmio, MTL_MIRROR_TARGET_WP1);
> freq = REG_FIELD_GET(MTL_CAGF_MASK, freq);
> } else {
> - freq = xe_mmio_read32(gt, GT_PERF_STATUS);
> + freq = xe_mmio_read32(>->mmio, GT_PERF_STATUS);
> freq = REG_FIELD_GET(CAGF_MASK, freq);
> }
>
> @@ -425,7 +425,7 @@ int xe_guc_pc_get_cur_freq(struct xe_guc_pc *pc, u32 *freq)
> if (ret)
> return ret;
>
> - *freq = xe_mmio_read32(gt, RPNSWREQ);
> + *freq = xe_mmio_read32(>->mmio, RPNSWREQ);
>
> *freq = REG_FIELD_GET(REQ_RATIO_MASK, *freq);
> *freq = decode_freq(*freq);
> @@ -612,10 +612,10 @@ enum xe_gt_idle_state xe_guc_pc_c_status(struct xe_guc_pc *pc)
> u32 reg, gt_c_state;
>
> if (GRAPHICS_VERx100(gt_to_xe(gt)) >= 1270) {
> - reg = xe_mmio_read32(gt, MTL_MIRROR_TARGET_WP1);
> + reg = xe_mmio_read32(>->mmio, MTL_MIRROR_TARGET_WP1);
> gt_c_state = REG_FIELD_GET(MTL_CC_MASK, reg);
> } else {
> - reg = xe_mmio_read32(gt, GT_CORE_STATUS);
> + reg = xe_mmio_read32(>->mmio, GT_CORE_STATUS);
> gt_c_state = REG_FIELD_GET(RCN_MASK, reg);
> }
>
> @@ -638,7 +638,7 @@ u64 xe_guc_pc_rc6_residency(struct xe_guc_pc *pc)
> struct xe_gt *gt = pc_to_gt(pc);
> u32 reg;
>
> - reg = xe_mmio_read32(gt, GT_GFX_RC6);
> + reg = xe_mmio_read32(>->mmio, GT_GFX_RC6);
>
> return reg;
> }
> @@ -652,7 +652,7 @@ u64 xe_guc_pc_mc6_residency(struct xe_guc_pc *pc)
> struct xe_gt *gt = pc_to_gt(pc);
> u64 reg;
>
> - reg = xe_mmio_read32(gt, MTL_MEDIA_MC6);
> + reg = xe_mmio_read32(>->mmio, MTL_MEDIA_MC6);
>
> return reg;
> }
> @@ -665,9 +665,9 @@ static void mtl_init_fused_rp_values(struct xe_guc_pc *pc)
> xe_device_assert_mem_access(pc_to_xe(pc));
>
> if (xe_gt_is_media_type(gt))
> - reg = xe_mmio_read32(gt, MTL_MEDIAP_STATE_CAP);
> + reg = xe_mmio_read32(>->mmio, MTL_MEDIAP_STATE_CAP);
> else
> - reg = xe_mmio_read32(gt, MTL_RP_STATE_CAP);
> + reg = xe_mmio_read32(>->mmio, MTL_RP_STATE_CAP);
>
> pc->rp0_freq = decode_freq(REG_FIELD_GET(MTL_RP0_CAP_MASK, reg));
>
> @@ -683,9 +683,9 @@ static void tgl_init_fused_rp_values(struct xe_guc_pc *pc)
> xe_device_assert_mem_access(pc_to_xe(pc));
>
> if (xe->info.platform == XE_PVC)
> - reg = xe_mmio_read32(gt, PVC_RP_STATE_CAP);
> + reg = xe_mmio_read32(>->mmio, PVC_RP_STATE_CAP);
> else
> - reg = xe_mmio_read32(gt, RP_STATE_CAP);
> + reg = xe_mmio_read32(>->mmio, RP_STATE_CAP);
> pc->rp0_freq = REG_FIELD_GET(RP0_MASK, reg) * GT_FREQUENCY_MULTIPLIER;
> pc->rpn_freq = REG_FIELD_GET(RPN_MASK, reg) * GT_FREQUENCY_MULTIPLIER;
> }
> diff --git a/drivers/gpu/drm/xe/xe_mmio.c b/drivers/gpu/drm/xe/xe_mmio.c
> index 29f4e3759106..ccf53a7840d9 100644
> --- a/drivers/gpu/drm/xe/xe_mmio.c
> +++ b/drivers/gpu/drm/xe/xe_mmio.c
> @@ -430,8 +430,8 @@ int __xe_mmio_wait32(struct xe_mmio *mmio, struct xe_reg reg, u32 mask, u32 val,
> * This function works exactly like xe_mmio_wait32() with the exception that
> * @val is expected not to be matched.
> */
> -int __xe_mmio_wait32_not(struct xe_mmio *mmio, struct xe_reg reg, u32 mask, u32 val, u32 timeout_us,
> - u32 *out_val, bool atomic)
> +int xe_mmio_wait32_not(struct xe_mmio *mmio, struct xe_reg reg, u32 mask, u32 val, u32 timeout_us,
> + u32 *out_val, bool atomic)
> {
> return ____xe_mmio_wait32(mmio, reg, mask, val, timeout_us, out_val, atomic, false);
> }
> diff --git a/drivers/gpu/drm/xe/xe_mmio.h b/drivers/gpu/drm/xe/xe_mmio.h
> index 99e3b58c9bb2..2e97dc811d82 100644
> --- a/drivers/gpu/drm/xe/xe_mmio.h
> +++ b/drivers/gpu/drm/xe/xe_mmio.h
> @@ -56,10 +56,8 @@ int __xe_mmio_wait32(struct xe_mmio *mmio, struct xe_reg reg, u32 mask, u32 val,
> #define xe_mmio_wait32(p, reg, mask, val, timeout_us, out_val, atomic) \
> __xe_mmio_wait32(__to_xe_mmio(p), reg, mask, val, timeout_us, out_val, atomic)
>
> -int __xe_mmio_wait32_not(struct xe_mmio *mmio, struct xe_reg reg, u32 mask,
> - u32 val, u32 timeout_us, u32 *out_val, bool atomic);
> -#define xe_mmio_wait32_not(p, reg, mask, val, timeout_us, out_val, atomic) \
> - __xe_mmio_wait32_not(__to_xe_mmio(p), reg, mask, val, timeout_us, out_val, atomic)
> +int xe_mmio_wait32_not(struct xe_mmio *mmio, struct xe_reg reg, u32 mask,
> + u32 val, u32 timeout_us, u32 *out_val, bool atomic);
>
shouldn't this go with the last patch?
Or avoid the last patch and then in every last usage of a case, like this,
you remove the unused compat layer?
> static inline u32 __xe_mmio_adjusted_addr(const struct xe_mmio *mmio, u32 addr)
> {
> --
> 2.45.2
>
More information about the Intel-xe
mailing list