Apply suggestions from code review

Author: Kerilk

loader/icd.c

@@ -544,7 +544,7 @@ void khrIcdDeinitialize(void) {
 #if defined(CL_ENABLE_LAYERS)
     // free layers first in reverse order of their creation (front to back)
     // they may still need to use vendors while terminating
-    KHR_ICD_TRACE("Finalizing and unloading layers\n");
+    KHR_ICD_TRACE("finalizing and unloading layers\n");
     struct KHRLayer *head = khrFirstLayer;
     deinitLayer.dispatch = khrDeinitDispatch;
     khrFirstLayer = &deinitLayer;
@@ -555,7 +555,13 @@ void khrIcdDeinitialize(void) {
         free(cur->libraryName);
 #endif
         if (cur->p_clDeinitLayer)
-            cur->p_clDeinitLayer();
+        {
+            cl_int res = cur->p_clDeinitLayer();
+            if (CL_SUCCESS != res)
+            {
+                KHR_ICD_TRACE("error reported in layer deinitialization\n");
+            }
+        }
         if (!khrDisableLibraryUnloading)
             khrIcdOsLibraryUnload(cur->library);
         head = cur->next;
@@ -564,7 +570,7 @@ void khrIcdDeinitialize(void) {
 #endif // defined(CL_ENABLE_LAYERS)
 
     // free vendor in reverse order of their creation (back to front)
-    KHR_ICD_TRACE("Finalizing and unloading vendors\n");
+    KHR_ICD_TRACE("finalizing and unloading vendors\n");
     while (lastVendor) {
         KHRicdVendor *cur = lastVendor;
         free(cur->suffix);