Message ID | 20190912172218.23335-4-clg@kaod.org |
---|---|
State | Superseded |
Headers | show |
Series | xive: new interfaces, fixes and cleanups | expand |
Context | Check | Description |
---|---|---|
snowpatch_ozlabs/apply_patch | success | Successfully applied on branch master (470ffb5f29d741c3bed600f7bb7bf0cbb270e05a) |
snowpatch_ozlabs/snowpatch_job_snowpatch-skiboot | success | Test snowpatch/job/snowpatch-skiboot on branch master |
snowpatch_ozlabs/snowpatch_job_snowpatch-skiboot-dco | success | Signed-off-by present |
diff --git a/hw/xive.c b/hw/xive.c index 174e648ab9b2..ca8044985662 100644 --- a/hw/xive.c +++ b/hw/xive.c @@ -2529,7 +2529,7 @@ static int64_t xive_sync(struct xive *x) /* XXX Add timeout */ for (;;) { - r = xive_regrx(x, VC_EQC_CONFIG); + r = xive_regr(x, VC_EQC_CONFIG); if ((r & SYNC_MASK) == SYNC_MASK) break; cpu_relax();
There is no reason to issue loads on XSCOM when syncing the interrupt controller. All should be in place to use MMIOs. Signed-off-by: Cédric Le Goater <clg@kaod.org> --- hw/xive.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)