Fix casting for srgba-read.cpp test

pi_bool is uint32_t and ur_bool_t is uint8_t, so to make sure
correct functionality is maintain, use uint32_t as replacement
for pi_bool, instead of ur_bool_t.

Also, add back check for urMemImageCreate that was before in
piMemImageCreate.

Signed-off-by: Jaime Arteaga <jaime.a.arteaga.molina@intel.com>

Author: Jaime Arteaga

sycl/plugins/unified_runtime/ur/adapters/level_zero/ur_level_zero_context.cpp

@@ -13,8 +13,6 @@
 
 #include "ur_level_zero.hpp"
 #include "ur_level_zero_context.hpp"
-#include "ur_level_zero.hpp"
-  
 
 UR_APIEXPORT ur_result_t UR_APICALL urContextCreate(
     uint32_t DeviceCount, ///< [in] the number of devices given in phDevices
@@ -108,10 +106,10 @@ UR_APIEXPORT ur_result_t UR_APICALL urContextGetInfo(
     return ReturnValue(uint32_t{Context->RefCount.load()});
   case UR_CONTEXT_INFO_USM_MEMCPY2D_SUPPORT:
     // 2D USM memcpy is supported.
-    return ReturnValue(ur_bool_t{UseMemcpy2DOperations});
+    return ReturnValue(uint32_t{UseMemcpy2DOperations});
   case UR_CONTEXT_INFO_USM_FILL2D_SUPPORT:
     // 2D USM fill is not supported.
-    return ReturnValue(ur_bool_t{false});
+    return ReturnValue(uint32_t{false});
   case UR_CONTEXT_INFO_ATOMIC_MEMORY_ORDER_CAPABILITIES: {
 
     ur_memory_order_capability_flags_t Capabilities =

sycl/plugins/unified_runtime/ur/adapters/level_zero/ur_level_zero_device.cpp

@@ -10,7 +10,6 @@
 #include "ur_level_zero.hpp"
 #include <algorithm>
 #include <climits>
-  
 
 UR_APIEXPORT ur_result_t UR_APICALL urDeviceGet(
     ur_platform_handle_t Platform, ///< [in] handle of the platform instance
@@ -771,9 +770,9 @@ UR_APIEXPORT ur_result_t UR_APICALL urDeviceGetInfo(
     return ReturnValue(capabilities);
   }
   case UR_DEVICE_INFO_MEM_CHANNEL_SUPPORT:
-    return ReturnValue(ur_bool_t{false});
+    return ReturnValue(uint32_t{false});
   case UR_DEVICE_INFO_IMAGE_SRGB:
-    return ReturnValue(ur_bool_t{false});
+    return ReturnValue(uint32_t{false});
 
   case UR_DEVICE_INFO_QUEUE_ON_DEVICE_PROPERTIES:
   case UR_DEVICE_INFO_QUEUE_ON_HOST_PROPERTIES: {
@@ -1197,7 +1196,7 @@ UR_APIEXPORT ur_result_t UR_APICALL urDeviceSelectBinary(
 
   // Look for GEN binary, which we known can only be handled by Level-Zero now.
   const char *BinaryTarget =
-      UR_DEVICE_BINARY_TARGET_SPIRV64_GEN; //UR_DEVICE_BINARY_TARGET_SPIRV64_GEN;
+      UR_DEVICE_BINARY_TARGET_SPIRV64_GEN; // UR_DEVICE_BINARY_TARGET_SPIRV64_GEN;
 
   uint32_t *SelectedBinaryInd = SelectedBinary;
 

sycl/plugins/unified_runtime/ur/adapters/level_zero/ur_level_zero_mem.cpp

@@ -13,8 +13,6 @@
 #include "ur_level_zero.hpp"
 #include "ur_level_zero_context.hpp"
 #include "ur_level_zero_event.hpp"
-#include "ur_level_zero.hpp"
-  
 
 // Default to using compute engine for fill operation, but allow to
 // override this with an environment variable.
@@ -1604,6 +1602,12 @@ UR_APIEXPORT ur_result_t UR_APICALL urMemImageCreate(
     void *Host,                       ///< [in] pointer to the buffer data
     ur_mem_handle_t *Mem ///< [out] pointer to handle of image object created
 ) {
+  // TODO: implement read-only, write-only
+  if ((Flags & UR_MEM_FLAG_READ_WRITE) == 0) {
+    die("urMemImageCreate: Level-Zero implements only read-write buffer,"
+        "no read-only or write-only yet.");
+  }
+
   std::shared_lock<ur_shared_mutex> Lock(Context->Mutex);
 
   ZeStruct<ze_image_desc_t> ZeImageDesc;