diff --git a/light/v1_0/ILightInterface.idl b/light/v1_0/ILightInterface.idl
index fb2f0eb9daefc41988e859ca630fc51a031e5395..23b6602c91e75a0dcb020833c5b16ab0fd4ef17f 100644
--- a/light/v1_0/ILightInterface.idl
+++ b/light/v1_0/ILightInterface.idl
@@ -61,7 +61,8 @@ interface ILightInterface {
      *
      * @param lightId Indicates the light id. For details, see {@link HdfLightId}.
      *
-     * @param effect Indicates the pointer to the lighting effect. For details, see {@link HdfLightEffect}.
+     * @param effect Indicates the pointer to the lighting effect, if the lightbrightness field is 0,
+     * light brightness according to the defaultBrightness configured by HCS. For details, see {@link LightEffect}.
      *
      * @return Returns <b>0</b> if the operation is successful.
      * @return Returns <b>-1</b> if the light id is not supported.
diff --git a/light/v1_0/LightTypes.idl b/light/v1_0/LightTypes.idl
index 29b4db5f6e5089cdf4c1a9b1a22a9c38a4434d41..e3aebc6f9e2bb66cc0e25a099a6afebac657cf82 100644
--- a/light/v1_0/LightTypes.idl
+++ b/light/v1_0/LightTypes.idl
@@ -91,7 +91,8 @@ struct HdfLightFlashEffect {
  * @since 3.1
  */
 struct HdfLightEffect {
-    int lightBrightness;    /** Brightness value. The value 1 of the most significant bit indicates a color.
-                                Bits 16–31 for red, bits 8–15 for green, and bits 0–7 for blue. */
+    int lightBrightness;    /** Brightness value. Bits 24–31 for extended bit, Bits 16–23 for red,
+                                bits 8–15 for green, and bits 0–7 for blue. if the byte segment is not equal to 0,
+                                indicates turn on light of the corresponding color. */
     struct HdfLightFlashEffect flashEffect;    /** Blinking mode. For details, see {@link LightFlashEffect}. */
 };