diff --git a/components/drivers/include/drivers/sensor.h b/components/drivers/include/drivers/sensor.h
index e09d8d8c94..615dd1f367 100644
--- a/components/drivers/include/drivers/sensor.h
+++ b/components/drivers/include/drivers/sensor.h
@@ -238,8 +238,8 @@ struct rt_sensor_data
 
 struct rt_sensor_ops
 {
-    rt_size_t (*fetch_data)(struct rt_sensor_device *sensor, void *buf, rt_size_t len);
-    rt_err_t (*control)(struct rt_sensor_device *sensor, int cmd, void *arg);
+    rt_size_t (*fetch_data)(rt_sensor_t sensor, void *buf, rt_size_t len);
+    rt_err_t (*control)(rt_sensor_t sensor, int cmd, void *arg);
 };
 
 int rt_hw_sensor_register(rt_sensor_t sensor,
diff --git a/components/drivers/sensors/SConscript b/components/drivers/sensors/SConscript
index b56ae5ce6e..afafc6d2ce 100644
--- a/components/drivers/sensors/SConscript
+++ b/components/drivers/sensors/SConscript
@@ -9,6 +9,6 @@ CPPPATH = [cwd, cwd + '/../include']
 if GetDepend('RT_USING_SENSOR_CMD'):
     src += ['sensor_cmd.c']
 
-group = DefineGroup('Sensors', src, depend = ['RT_USING_SENSOR', 'RT_USING_DEVICE'], CPPPATH = CPPPATH)
+group = DefineGroup('DeviceDrivers', src, depend = ['RT_USING_SENSOR', 'RT_USING_DEVICE'], CPPPATH = CPPPATH)
 
 Return('group')
diff --git a/components/drivers/sensors/sensor.c b/components/drivers/sensors/sensor.c
index cbd149317e..25fd5681be 100644
--- a/components/drivers/sensors/sensor.c
+++ b/components/drivers/sensors/sensor.c
@@ -43,11 +43,8 @@ static char *const sensor_name_str[] =
     "bp_"        /* Blood Pressure    */
 };
 
-/* Sensor interrupt correlation function */
-/*
- * Sensor interrupt handler function
- */
-void rt_sensor_cb(rt_sensor_t sen)
+/* sensor interrupt handler function */
+static void _sensor_cb(rt_sensor_t sen)
 {
     if (sen->parent.rx_indicate == RT_NULL)
     {
@@ -76,7 +73,7 @@ void rt_sensor_cb(rt_sensor_t sen)
 }
 
 /* ISR for sensor interrupt */
-static void irq_callback(void *args)
+static void _irq_callback(void *args)
 {
     rt_sensor_t sensor = (rt_sensor_t)args;
     rt_uint8_t i;
@@ -86,17 +83,17 @@ static void irq_callback(void *args)
         /* Invoke a callback for all sensors in the module */
         for (i = 0; i < sensor->module->sen_num; i++)
         {
-            rt_sensor_cb(sensor->module->sen[i]);
+            _sensor_cb(sensor->module->sen[i]);
         }
     }
     else
     {
-        rt_sensor_cb(sensor);
+        _sensor_cb(sensor);
     }
 }
 
 /* Sensor interrupt initialization function */
-static rt_err_t rt_sensor_irq_init(rt_sensor_t sensor)
+static rt_err_t _sensor_irq_init(rt_sensor_t sensor)
 {
     if (sensor->config.irq_pin.pin == RT_PIN_NONE)
     {
@@ -107,15 +104,15 @@ static rt_err_t rt_sensor_irq_init(rt_sensor_t sensor)
 
     if (sensor->config.irq_pin.mode == PIN_MODE_INPUT_PULLDOWN)
     {
-        rt_pin_attach_irq(sensor->config.irq_pin.pin, PIN_IRQ_MODE_RISING, irq_callback, (void *)sensor);
+        rt_pin_attach_irq(sensor->config.irq_pin.pin, PIN_IRQ_MODE_RISING, _irq_callback, (void *)sensor);
     }
     else if (sensor->config.irq_pin.mode == PIN_MODE_INPUT_PULLUP)
     {
-        rt_pin_attach_irq(sensor->config.irq_pin.pin, PIN_IRQ_MODE_FALLING, irq_callback, (void *)sensor);
+        rt_pin_attach_irq(sensor->config.irq_pin.pin, PIN_IRQ_MODE_FALLING, _irq_callback, (void *)sensor);
     }
     else if (sensor->config.irq_pin.mode == PIN_MODE_INPUT)
     {
-        rt_pin_attach_irq(sensor->config.irq_pin.pin, PIN_IRQ_MODE_RISING_FALLING, irq_callback, (void *)sensor);
+        rt_pin_attach_irq(sensor->config.irq_pin.pin, PIN_IRQ_MODE_RISING_FALLING, _irq_callback, (void *)sensor);
     }
 
     rt_pin_irq_enable(sensor->config.irq_pin.pin, RT_TRUE);
@@ -125,31 +122,30 @@ static rt_err_t rt_sensor_irq_init(rt_sensor_t sensor)
     return 0;
 }
 
-// local rt_sensor_ops
-
-static rt_size_t local_fetch_data(struct rt_sensor_device *sensor, void *buf, rt_size_t len)
+/* sensor local ops */
+static rt_size_t _local_fetch_data(rt_sensor_t sensor, void *buf, rt_size_t len)
 {
     LOG_D("Undefined fetch_data");
     return 0;
 }
-static rt_err_t local_control(struct rt_sensor_device *sensor, int cmd, void *arg)
+static rt_err_t _local_control(rt_sensor_t sensor, int cmd, void *arg)
 {
     LOG_D("Undefined control");
     return RT_ERROR;
 }
 static struct rt_sensor_ops local_ops =
 {
-    .fetch_data = local_fetch_data,
-    .control = local_control
+    .fetch_data = _local_fetch_data,
+    .control = _local_control
 };
 
 /* RT-Thread Device Interface */
-static rt_err_t rt_sensor_open(rt_device_t dev, rt_uint16_t oflag)
+static rt_err_t _sensor_open(rt_device_t dev, rt_uint16_t oflag)
 {
     rt_sensor_t sensor = (rt_sensor_t)dev;
     RT_ASSERT(dev != RT_NULL);
     rt_err_t res = RT_EOK;
-    rt_err_t (*local_ctrl)(struct rt_sensor_device * sensor, int cmd, void *arg) =  local_control;
+    rt_err_t (*local_ctrl)(rt_sensor_t sensor, int cmd, void *arg) =  _local_control;
 
     if (sensor->module)
     {
@@ -184,7 +180,7 @@ static rt_err_t rt_sensor_open(rt_device_t dev, rt_uint16_t oflag)
         if (local_ctrl(sensor, RT_SENSOR_CTRL_SET_MODE, (void *)RT_SENSOR_MODE_INT) == RT_EOK)
         {
             /* Initialization sensor interrupt */
-            rt_sensor_irq_init(sensor);
+            _sensor_irq_init(sensor);
             sensor->config.mode = RT_SENSOR_MODE_INT;
         }
     }
@@ -194,7 +190,7 @@ static rt_err_t rt_sensor_open(rt_device_t dev, rt_uint16_t oflag)
         if (local_ctrl(sensor, RT_SENSOR_CTRL_SET_MODE, (void *)RT_SENSOR_MODE_FIFO) == RT_EOK)
         {
             /* Initialization sensor interrupt */
-            rt_sensor_irq_init(sensor);
+            _sensor_irq_init(sensor);
             sensor->config.mode = RT_SENSOR_MODE_FIFO;
         }
     }
@@ -220,11 +216,11 @@ __exit:
     return res;
 }
 
-static rt_err_t rt_sensor_close(rt_device_t dev)
+static rt_err_t _sensor_close(rt_device_t dev)
 {
     rt_sensor_t sensor = (rt_sensor_t)dev;
     int i;
-    rt_err_t (*local_ctrl)(struct rt_sensor_device * sensor, int cmd, void *arg) = local_control;
+    rt_err_t (*local_ctrl)(rt_sensor_t sensor, int cmd, void *arg) = _local_control;
 
     RT_ASSERT(dev != RT_NULL);
 
@@ -279,7 +275,7 @@ __exit:
     return RT_EOK;
 }
 
-static rt_size_t rt_sensor_read(rt_device_t dev, rt_off_t pos, void *buf, rt_size_t len)
+static rt_size_t _sensor_read(rt_device_t dev, rt_off_t pos, void *buf, rt_size_t len)
 {
     rt_sensor_t sensor = (rt_sensor_t)dev;
     rt_size_t result = 0;
@@ -326,12 +322,12 @@ static rt_size_t rt_sensor_read(rt_device_t dev, rt_off_t pos, void *buf, rt_siz
     return result;
 }
 
-static rt_err_t rt_sensor_control(rt_device_t dev, int cmd, void *args)
+static rt_err_t _sensor_control(rt_device_t dev, int cmd, void *args)
 {
     rt_sensor_t sensor = (rt_sensor_t)dev;
     rt_err_t result = RT_EOK;
     RT_ASSERT(dev != RT_NULL);
-    rt_err_t (*local_ctrl)(struct rt_sensor_device * sensor, int cmd, void *arg) = local_control;
+    rt_err_t (*local_ctrl)(rt_sensor_t sensor, int cmd, void *arg) = _local_control;
 
     if (sensor->module)
     {
@@ -413,11 +409,11 @@ static rt_err_t rt_sensor_control(rt_device_t dev, int cmd, void *args)
 const static struct rt_device_ops rt_sensor_ops =
 {
     RT_NULL,
-    rt_sensor_open,
-    rt_sensor_close,
-    rt_sensor_read,
+    _sensor_open,
+    _sensor_close,
+    _sensor_read,
     RT_NULL,
-    rt_sensor_control
+    _sensor_control
 };
 #endif
 
@@ -470,11 +466,11 @@ int rt_hw_sensor_register(rt_sensor_t sensor,
     device->ops         = &rt_sensor_ops;
 #else
     device->init        = RT_NULL;
-    device->open        = rt_sensor_open;
-    device->close       = rt_sensor_close;
-    device->read        = rt_sensor_read;
+    device->open        = _sensor_open;
+    device->close       = _sensor_close;
+    device->read        = _sensor_read;
     device->write       = RT_NULL;
-    device->control     = rt_sensor_control;
+    device->control     = _sensor_control;
 #endif
     device->type        = RT_Device_Class_Sensor;
     device->rx_indicate = RT_NULL;