diff --git a/model/misc/light/driver/include/light_driver.h b/model/misc/light/driver/include/light_driver.h
index 7011b43e..41ba8ba8 100644
--- a/model/misc/light/driver/include/light_driver.h
+++ b/model/misc/light/driver/include/light_driver.h
@@ -23,6 +23,9 @@
 #define LIGHT_MAKE_B_BIT    0X00000080
 #define LIGHT_WAIT_TIME     50
 
+#define LIGHT_MAX_TYPE      4
+#define LIGHT_MIN_TYPE      0
+
 #define CHECK_LIGHT_NULL_PTR_RETURN_VALUE(ptr, ret) do { \
     if ((ptr) == NULL) { \
         HDF_LOGE("%s:line %d pointer is null and return ret", __func__, __LINE__); \
diff --git a/model/misc/light/driver/src/light_driver.c b/model/misc/light/driver/src/light_driver.c
index 2c2360b4..88951a1e 100644
--- a/model/misc/light/driver/src/light_driver.c
+++ b/model/misc/light/driver/src/light_driver.c
@@ -94,6 +94,7 @@ void LightTimerEntry(uintptr_t para)
 
     drvData = GetLightDrvData();
     if (drvData == NULL) {
+        HDF_LOGE("%s: drvData is null", __func__);
         return;
     }
 
@@ -299,7 +300,7 @@ static int32_t ParseLightInfo(const struct DeviceResourceNode *node, const struc
     CHECK_LIGHT_NULL_PTR_RETURN_VALUE(parser, HDF_ERR_INVALID_PARAM);
 
     drvData->lightNum = (uint32_t)parser->GetElemNum(node, "lightType");
-    if (drvData->lightNum > LIGHT_TYPE_BUTT || drvData->lightNum < LIGHT_TYPE_NONE) {
+    if (drvData->lightNum > LIGHT_MAX_TYPE || drvData->lightNum < LIGHT_MIN_TYPE) {
         HDF_LOGE("%s: lightNum cross the border", __func__);
         return HDF_FAILURE;
     }
@@ -439,7 +440,7 @@ int32_t InitLightDriver(struct HdfDeviceObject *device)
     }
 
     if (HdfWorkInit(&drvData->work, LightWorkEntry, (void*)drvData) != HDF_SUCCESS) {
-        HDF_LOGE("%s: init workQueue fail!", __func__);
+        HDF_LOGE("%s: init work fail!", __func__);
         return HDF_FAILURE;
     }
 
@@ -453,7 +454,7 @@ int32_t InitLightDriver(struct HdfDeviceObject *device)
 
 void ReleaseLightDriver(struct HdfDeviceObject *device)
 {
-    int i;
+    int32_t i;
     struct LightDriverData *drvData = NULL;
 
     if (device == NULL) {