diff --git a/model/input/driver/hdf_encoder.c b/model/input/driver/hdf_encoder.c
index 134b8f610a9e4ee57e0f6863a2dece797201b1b0..85ddcb7822e1e5089134259bcc8a7c5170b59bae 100644
--- a/model/input/driver/hdf_encoder.c
+++ b/model/input/driver/hdf_encoder.c
@@ -18,12 +18,13 @@
 
 static void EncoderTimerFunc(uintptr_t arg)
 {
+    int32_t ret;
     EncoderDriver *encoderDrv = (EncoderDriver *)arg;
     uint16_t gpioClk = encoderDrv->encoderCfg->gpioClk;
     uint16_t gpioData = encoderDrv->encoderCfg->gpioData;
     uint16_t gpioSW = encoderDrv->encoderCfg->gpioSW;
 
-    int32_t ret = GpioRead(gpioClk, &encoderDrv->encoderClkNowSta);
+    ret = GpioRead(gpioClk, &encoderDrv->encoderClkNowSta);
     if (ret != HDF_SUCCESS) {
         HDF_LOGE("%s: gpio read failed, ret %d", __func__, ret);
         return;
@@ -64,12 +65,13 @@ static void EncoderTimerFunc(uintptr_t arg)
 
 static EncoderCfg *EncoderConfigInstance(struct HdfDeviceObject *device)
 {
+    int32_t ret;
     EncoderCfg *encoderCfg = (EncoderCfg *)OsalMemAlloc(sizeof(EncoderCfg));
     if (encoderCfg == NULL) {
         HDF_LOGE("%s: malloc encoder config failed", __func__);
         return NULL;
     }
-    int32_t ret = memset_s(encoderCfg, sizeof(EncoderCfg), 0, sizeof(EncoderCfg));
+    ret = memset_s(encoderCfg, sizeof(EncoderCfg), 0, sizeof(EncoderCfg));
     if (ret != 0) {
         HDF_LOGE("%s: memset_s encoder config failed", __func__);
         OsalMemFree(encoderCfg);
@@ -87,12 +89,13 @@ static EncoderCfg *EncoderConfigInstance(struct HdfDeviceObject *device)
 
 static EncoderDriver *EncoderDriverInstance(EncoderCfg *encoderCfg)
 {
+    int32_t ret;
     EncoderDriver *encoderDrv = (EncoderDriver *)OsalMemAlloc(sizeof(EncoderDriver));
     if (encoderDrv == NULL) {
         HDF_LOGE("%s: malloc key driver failed", __func__);
         return NULL;
     }
-    int32_t ret = memset_s(encoderDrv, sizeof(EncoderDriver), 0, sizeof(EncoderDriver));
+    ret = memset_s(encoderDrv, sizeof(EncoderDriver), 0, sizeof(EncoderDriver));
     if (ret != 0) {
         HDF_LOGE("%s: memset encoder driver failed", __func__);
         OsalMemFree(encoderDrv);
@@ -107,6 +110,7 @@ static EncoderDriver *EncoderDriverInstance(EncoderCfg *encoderCfg)
 
 static int32_t EncoderInit(EncoderDriver *EncoderDrv)
 {
+    int32_t ret;
     uint16_t gpioClk = EncoderDrv->encoderCfg->gpioClk;
     uint16_t gpioData = EncoderDrv->encoderCfg->gpioData;
     uint16_t gpioSW = EncoderDrv->encoderCfg->gpioSW;
@@ -114,7 +118,7 @@ static int32_t EncoderInit(EncoderDriver *EncoderDrv)
     GpioSetDir(gpioData, GPIO_DIR_IN);
     GpioSetDir(gpioSW, GPIO_DIR_IN);
 
-    int32_t ret = OsalTimerCreate(&EncoderDrv->timer, TIMER_INTERVAL_ENCODER, EncoderTimerFunc, EncoderDrv);
+    ret = OsalTimerCreate(&EncoderDrv->timer, TIMER_INTERVAL_ENCODER, EncoderTimerFunc, EncoderDrv);
     if (ret != HDF_SUCCESS) {
         HDF_LOGE("%s: create timer failed, ret = %d\n", __func__, ret);
         return HDF_FAILURE;
@@ -149,12 +153,13 @@ static int32_t EncoderInit(EncoderDriver *EncoderDrv)
 
 static InputDevice *InputDeviceInstance(EncoderDriver *encoderDrv)
 {
+    int32_t ret;
     InputDevice *inputDev = (InputDevice *)OsalMemAlloc(sizeof(InputDevice));
     if (inputDev == NULL) {
         HDF_LOGE("%s: malloc input device failed", __func__);
         return NULL;
     }
-    int32_t ret = memset_s(inputDev, sizeof(InputDevice), 0, sizeof(InputDevice));
+    ret = memset_s(inputDev, sizeof(InputDevice), 0, sizeof(InputDevice));
     if (ret != 0) {
         HDF_LOGE("%s: memset encoder driver failed", __func__);
         OsalMemFree(inputDev);
@@ -178,6 +183,7 @@ static InputDevice *InputDeviceInstance(EncoderDriver *encoderDrv)
 
 static int32_t RegisterEncoderDevice(EncoderCfg *encoderCfg, struct HdfDeviceObject *device)
 {
+    int32_t ret;
     EncoderDriver *EncoderDrv = EncoderDriverInstance(encoderCfg);
     if (EncoderDrv == NULL) {
         HDF_LOGE("%s: instance encoder driver failed", __func__);
@@ -185,7 +191,7 @@ static int32_t RegisterEncoderDevice(EncoderCfg *encoderCfg, struct HdfDeviceObj
     }
     device->priv = (void *)EncoderDrv;
 
-    int32_t ret = EncoderInit(EncoderDrv);
+    ret = EncoderInit(EncoderDrv);
     if (ret != HDF_SUCCESS) {
         HDF_LOGE("%s: key driver init failed, ret %d", __func__, ret);
         goto EXIT;
@@ -213,6 +219,7 @@ EXIT:
 
 static int32_t HdfEnCoderDriverInit(struct HdfDeviceObject *device)
 {
+    int32_t ret;
     if (device == NULL) {
         HDF_LOGE("%s: param is null", __func__);
         return HDF_ERR_INVALID_PARAM;
@@ -224,7 +231,7 @@ static int32_t HdfEnCoderDriverInit(struct HdfDeviceObject *device)
         return HDF_ERR_MALLOC_FAIL;
     }
 
-    int32_t ret = RegisterEncoderDevice(encoderCfg, device);
+    ret = RegisterEncoderDevice(encoderCfg, device);
     if (ret != HDF_SUCCESS) {
         goto EXIT;
     }
diff --git a/model/input/driver/hdf_input_device_manager.c b/model/input/driver/hdf_input_device_manager.c
index c9fcaa95a3859c45aabd5ee353c07835edd704a1..365313745303bc7da3f4efe8892cbda70a3988d8 100644
--- a/model/input/driver/hdf_input_device_manager.c
+++ b/model/input/driver/hdf_input_device_manager.c
@@ -103,12 +103,14 @@ static int32_t CreateDeviceNode(InputDevice *inputDev)
 
 static void DeleteDeviceNode(InputDevice *inputDev)
 {
+    int32_t len;
+    int32_t ret;
     if (IsHidDevice(inputDev->devType)) {
         char svcName[SERVICE_NAME_LEN] = {0};
         const char *moduleName = "HDF_HID";
 
-        int32_t len = (inputDev->devId < PLACEHOLDER_LIMIT) ? 1 : PLACEHOLDER_LENGTH;
-        int32_t ret = snprintf_s(svcName, SERVICE_NAME_LEN, strlen("hdf_input_event") + len, "%s%u",
+        len = (inputDev->devId < PLACEHOLDER_LIMIT) ? 1 : PLACEHOLDER_LENGTH;
+        ret = snprintf_s(svcName, SERVICE_NAME_LEN, strlen("hdf_input_event") + len, "%s%u",
             "hdf_input_event", inputDev->devId);
         if (ret < 0) {
             HDF_LOGE("%s: snprintf_s failed", __func__);
diff --git a/model/input/driver/hdf_key.c b/model/input/driver/hdf_key.c
index 7df3cd819bd09b3bd707ae155e2709992a1f17d4..56ba4cb97c40b928f4511818593070c1f57d5119 100644
--- a/model/input/driver/hdf_key.c
+++ b/model/input/driver/hdf_key.c
@@ -23,6 +23,7 @@
 
 int32_t KeyIrqHandle(uint16_t intGpioNum, void *data)
 {
+    int32_t ret;
     uint16_t gpioValue = 0;
     KeyDriver *driver = (KeyDriver *)data;
     if (driver == NULL) {
@@ -32,7 +33,7 @@ int32_t KeyIrqHandle(uint16_t intGpioNum, void *data)
     if (event == NULL) {
         return HDF_FAILURE;
     }
-    int32_t ret = GpioDisableIrq(intGpioNum);
+    ret = GpioDisableIrq(intGpioNum);
 
     if (ret != HDF_SUCCESS) {
         HDF_LOGE("%s: disable irq failed, ret %d", __func__, ret);
@@ -143,13 +144,14 @@ static InputDevice *InputDeviceInstance(KeyDriver *keyDrv)
 
 static int32_t RegisterKeyDevice(KeyChipCfg *keyCfg)
 {
+    int32_t ret;
     KeyDriver *keyDrv = KeyDriverInstance(keyCfg);
     if (keyDrv == NULL) {
         HDF_LOGE("%s: instance key config failed", __func__);
         return HDF_ERR_MALLOC_FAIL;
     }
 
-    int32_t ret = KeyInit(keyDrv);
+    ret = KeyInit(keyDrv);
     if (ret != HDF_SUCCESS) {
         HDF_LOGE("%s: key driver init failed, ret %d", __func__, ret);
         goto EXIT;
@@ -178,6 +180,7 @@ EXIT:
 
 static int32_t HdfKeyDriverInit(struct HdfDeviceObject *device)
 {
+    int32_t ret;
     HDF_LOGI("%s: enter", __func__);
     if (device == NULL) {
         return HDF_ERR_INVALID_PARAM;
@@ -189,7 +192,7 @@ static int32_t HdfKeyDriverInit(struct HdfDeviceObject *device)
         return HDF_ERR_MALLOC_FAIL;
     }
 
-    int32_t ret = RegisterKeyDevice(keyCfg);
+    ret = RegisterKeyDevice(keyCfg);
     if (ret != HDF_SUCCESS) {
         goto EXIT;
     }
diff --git a/model/input/driver/hdf_touch.c b/model/input/driver/hdf_touch.c
index 9f67dea5038e6e04660132fbc5c5f8ffae9c4999..b12ebcf9edd06ca1ea5e8ece91647b7435c90b44 100644
--- a/model/input/driver/hdf_touch.c
+++ b/model/input/driver/hdf_touch.c
@@ -456,13 +456,14 @@ static int32_t TouchGetDeviceStrInfo(TouchDriver *driver, int32_t cmd, struct Hd
 
 static int32_t TouchGetDeviceAttr(TouchDriver *driver, struct HdfSBuf *reply)
 {
+    char *tempStr = "main_touch";
+    int32_t ret;
     if (driver->inputDev == NULL) {
         return HDF_FAILURE;
     }
 
     HDF_LOGE("%s: enter", __func__);
-    char *tempStr = "main_touch";
-    int32_t ret = strncpy_s(driver->inputDev->attrSet.devName, DEV_NAME_LEN, tempStr, strlen(tempStr));
+    ret = strncpy_s(driver->inputDev->attrSet.devName, DEV_NAME_LEN, tempStr, strlen(tempStr));
     if (ret != 0) {
         HDF_LOGE("%s: strncpy dev attr failed", __func__);
         return HDF_FAILURE;
diff --git a/model/input/driver/input_config_parser.c b/model/input/driver/input_config_parser.c
index 01bc6524ebe413f334aa06ee4d56a6e63b8de810..17f65dd27edbd23f2d3fbc824b04b6c7f481cef0 100644
--- a/model/input/driver/input_config_parser.c
+++ b/model/input/driver/input_config_parser.c
@@ -25,12 +25,13 @@
 
 int32_t ParseKeyConfig(const struct DeviceResourceNode *node, KeyChipCfg *config)
 {
+    int32_t ret;
+    struct DeviceResourceIface *parser = NULL;
     if (node == NULL || config == NULL) {
         HDF_LOGE("%s: param is null", __func__);
         return HDF_FAILURE;
     }
 
-    struct DeviceResourceIface *parser = NULL;
     parser = DeviceResourceGetIfaceInstance(HDF_CONFIG_SOURCE);
     if (parser == NULL) {
         HDF_LOGE("%s: instance parser failed", __func__);
@@ -38,7 +39,7 @@ int32_t ParseKeyConfig(const struct DeviceResourceNode *node, KeyChipCfg *config
     }
 
     const struct DeviceResourceNode *keyNode = node;
-    int32_t ret = parser->GetString(keyNode, "keyName", &config->keyName, NULL);
+    ret = parser->GetString(keyNode, "keyName", &config->keyName, NULL);
     CHECK_PARSER_RET(ret, "GetString");
     ret = parser->GetUint8(keyNode, "inputType", &config->devType, 0);
     CHECK_PARSER_RET(ret, "GetUint8");
diff --git a/model/input/driver/touchscreen/touch_ft6336.c b/model/input/driver/touchscreen/touch_ft6336.c
index 87a006fbe9ae201821a827a713b1b69753fc0da2..8742debaba3ba7290d4fae59f3b342010a066abf 100644
--- a/model/input/driver/touchscreen/touch_ft6336.c
+++ b/model/input/driver/touchscreen/touch_ft6336.c
@@ -92,10 +92,11 @@ static int32_t ChipDataHandle(ChipDevice *device)
     uint8_t buf[POINT_BUFFER_LEN] = {0};
     uint8_t reg = 0x0;
     uint8_t pointNum;
+    int32_t ret;
     InputI2cClient *i2cClient = &device->driver->i2cClient;
     FrameData *frame = &device->driver->frameData;
 
-    int32_t ret = InputI2cRead(i2cClient, &reg, 1, buf, POINT_BUFFER_LEN);
+    ret = InputI2cRead(i2cClient, &reg, 1, buf, POINT_BUFFER_LEN);
     CHIP_CHECK_RETURN(ret);
 
     OsalMutexLock(&device->driver->mutex);