From 5cc970a503122e5d9d466ba032d432d470d801aa Mon Sep 17 00:00:00 2001 From: Jaime Arteaga Date: Wed, 18 Aug 2021 21:12:39 +0000 Subject: [PATCH] Minor cleanups Signed-off-by: Jaime Arteaga --- level_zero/core/source/event/event_impl.inl | 11 ++--------- 1 file changed, 2 insertions(+), 9 deletions(-) diff --git a/level_zero/core/source/event/event_impl.inl b/level_zero/core/source/event/event_impl.inl index 3c76726895..fbe0192347 100644 --- a/level_zero/core/source/event/event_impl.inl +++ b/level_zero/core/source/event/event_impl.inl @@ -103,7 +103,6 @@ template ze_result_t EventImp::queryStatus() { uint64_t *hostAddr = static_cast(hostAddress); uint32_t queryVal = Event::STATE_CLEARED; - ze_result_t retVal; if (metricStreamer != nullptr) { *hostAddr = metricStreamer->getNotificationState(); @@ -113,13 +112,7 @@ ze_result_t EventImp::queryStatus() { return queryStatusKernelTimestamp(); } memcpy_s(static_cast(&queryVal), sizeof(uint32_t), static_cast(hostAddr), sizeof(uint32_t)); - retVal = (queryVal == Event::STATE_CLEARED) ? ZE_RESULT_NOT_READY : ZE_RESULT_SUCCESS; - - if (retVal == ZE_RESULT_NOT_READY) { - return retVal; - } - - return retVal; + return (queryVal == Event::STATE_CLEARED) ? ZE_RESULT_NOT_READY : ZE_RESULT_SUCCESS; } template @@ -138,7 +131,7 @@ ze_result_t EventImp::hostEventSetValueTimestamps(TagSizeT eventVal) { }; for (uint32_t i = 0; i < kernelCount; i++) { uint32_t packetsToSet = kernelTimestampsData[i].getPacketsUsed(); - for (uint32_t i = 0; i < packetsToSet; i++) { + for (uint32_t j = 0; j < packetsToSet; j++) { eventTsSetFunc(baseAddr + NEO::TimestampPackets::getContextStartOffset()); eventTsSetFunc(baseAddr + NEO::TimestampPackets::getGlobalStartOffset()); eventTsSetFunc(baseAddr + NEO::TimestampPackets::getContextEndOffset());