Skip to content

Commit 71cdf25

Browse files
committed
[L0] Return error if not exactly RGB or RGX
Signed-off-by: Neil R. Spruit <neil.r.spruit@intel.com>
1 parent ff79add commit 71cdf25

File tree

1 file changed

+16
-15
lines changed

1 file changed

+16
-15
lines changed

source/adapters/level_zero/image.cpp

Lines changed: 16 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -108,24 +108,25 @@ ur_result_t ze2urImageFormat(const ze_image_desc_t *ZeImageDesc,
108108
case ZE_IMAGE_FORMAT_LAYOUT_8_8_8:
109109
case ZE_IMAGE_FORMAT_LAYOUT_16_16_16:
110110
case ZE_IMAGE_FORMAT_LAYOUT_32_32_32:
111-
if (ZeImageFormat.x != ZE_IMAGE_FORMAT_SWIZZLE_R &&
112-
ZeImageFormat.y != ZE_IMAGE_FORMAT_SWIZZLE_G) {
111+
if (ZeImageFormat.x == ZE_IMAGE_FORMAT_SWIZZLE_R &&
112+
ZeImageFormat.y == ZE_IMAGE_FORMAT_SWIZZLE_G) {
113+
switch (ZeImageFormat.z) {
114+
case ZE_IMAGE_FORMAT_SWIZZLE_B:
115+
ChannelOrder = UR_IMAGE_CHANNEL_ORDER_RGB;
116+
break;
117+
case ZE_IMAGE_FORMAT_SWIZZLE_X:
118+
ChannelOrder = UR_IMAGE_CHANNEL_ORDER_RGX;
119+
break;
120+
default:
121+
logger::error(
122+
"ze2urImageFormat: unexpected image format channel z: z = {}\n",
123+
ZeImageFormat.z);
124+
return UR_RESULT_ERROR_INVALID_VALUE;
125+
}
126+
} else {
113127
logger::error("ze2urImageFormat: unexpected image format channel");
114128
return UR_RESULT_ERROR_INVALID_VALUE;
115129
}
116-
switch (ZeImageFormat.z) {
117-
case ZE_IMAGE_FORMAT_SWIZZLE_B:
118-
ChannelOrder = UR_IMAGE_CHANNEL_ORDER_RGB;
119-
break;
120-
case ZE_IMAGE_FORMAT_SWIZZLE_X:
121-
ChannelOrder = UR_IMAGE_CHANNEL_ORDER_RGX;
122-
break;
123-
default:
124-
logger::error(
125-
"ze2urImageFormat: unexpected image format channel z: z = {}\n",
126-
ZeImageFormat.z);
127-
return UR_RESULT_ERROR_INVALID_VALUE;
128-
}
129130
break;
130131
case ZE_IMAGE_FORMAT_LAYOUT_8_8_8_8:
131132
case ZE_IMAGE_FORMAT_LAYOUT_16_16_16_16:

0 commit comments

Comments
 (0)