[VP] fix coverity issue unchecked return value

fix coverity issue unchecked return value.
intel · Sep 24, 2024 · 9aadf63 · 9aadf63
1 parent 35c28c6
commit 9aadf63
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 6 deletions.
diff --git a/...t/agnostic/Xe2_M_plus/Xe2_LPM/vp/hal/platform_interface/vp_platform_interface_xe2_lpm.cpp b/...t/agnostic/Xe2_M_plus/Xe2_LPM/vp/hal/platform_interface/vp_platform_interface_xe2_lpm.cpp
@@ -121,7 +121,7 @@ MOS_STATUS VpPlatformInterfacsXe2_Lpm::InitVpRenderHwCaps()
         // Init native adv kernel form VP Native adv kernel Binary List
         for (auto &curKernelEntry : m_vpNativeAdvKernelBinaryList)
         {
-            VP_PUBLIC_CHK_STATUS_RETURN(InitVpNativeAdvKernels(curKernelEntry.first, curKernelEntry.second));
+            InitVpNativeAdvKernels(curKernelEntry.first, curKernelEntry.second);
         }
     }
     return MOS_STATUS_SUCCESS;

diff --git a/...et/agnostic/Xe_M_plus/Xe2_HPM/vp/hal/platform_interface/vp_platform_interface_xe2_hpm.cpp b/...et/agnostic/Xe_M_plus/Xe2_HPM/vp/hal/platform_interface/vp_platform_interface_xe2_hpm.cpp
@@ -120,7 +120,7 @@ MOS_STATUS VpPlatformInterfaceXe2_Hpm::InitVpRenderHwCaps()
         // Init native adv kernel form VP Native adv kernel Binary List
         for (auto &curKernelEntry : m_vpNativeAdvKernelBinaryList)
         {
-            VP_PUBLIC_CHK_STATUS_RETURN(InitVpNativeAdvKernels(curKernelEntry.first, curKernelEntry.second));
+            InitVpNativeAdvKernels(curKernelEntry.first, curKernelEntry.second);
         }
     }
     return MOS_STATUS_SUCCESS;

diff --git a/media_softlet/agnostic/common/vp/hal/platform_interface/vp_platform_interface.cpp b/media_softlet/agnostic/common/vp/hal/platform_interface/vp_platform_interface.cpp
@@ -339,13 +339,13 @@ MOS_STATUS VpPlatformInterface::InitVpRenderHwCaps()
         // Init native adv kernel form VP Native adv kernel Binary List
         for (auto &curKernelEntry : m_vpNativeAdvKernelBinaryList)
         {
-            VP_PUBLIC_CHK_STATUS_RETURN(InitVpNativeAdvKernels(curKernelEntry.first, curKernelEntry.second));
+            InitVpNativeAdvKernels(curKernelEntry.first, curKernelEntry.second);
         }
     }
     return MOS_STATUS_SUCCESS;
 }
 
-MOS_STATUS VpPlatformInterface::InitVpNativeAdvKernels(
+void VpPlatformInterface::InitVpNativeAdvKernels(
     std::string            kernelName,
     VP_KERNEL_BINARY_ENTRY kernelBinaryEntry)
 {
@@ -359,7 +359,7 @@ MOS_STATUS VpPlatformInterface::InitVpNativeAdvKernels(
     vpKernel.SetKernelBinSize(kernelBinaryEntry.kernelBinSize);
     m_kernelPool.insert(std::make_pair(vpKernel.GetKernelName(), vpKernel));
 
-    return MOS_STATUS_SUCCESS;
+    return;
 }
 
 MOS_STATUS VpPlatformInterface::InitVpCmKernels(

diff --git a/media_softlet/agnostic/common/vp/hal/platform_interface/vp_platform_interface.h b/media_softlet/agnostic/common/vp/hal/platform_interface/vp_platform_interface.h
@@ -180,7 +180,7 @@ class VpPlatformInterface
         std::string           postfix = "",
         uint32_t              payloadOffset = CM_PAYLOAD_OFFSET);
 
-    virtual MOS_STATUS InitVpNativeAdvKernels(
+    virtual void InitVpNativeAdvKernels(
         std::string kernelName,
         VP_KERNEL_BINARY_ENTRY kernelBinaryEntry);