Align naming in Media Input cluster XML with the spec. (#24504)

* Align naming in Media Input cluster XML with the spec. * Regenerate generated code.
project-chip · Aug 2, 2023 · 1068020 · 1068020
1 parent 9de00e5
commit 1068020
Show file tree

Hide file tree

Showing 33 changed files with 195 additions and 140 deletions.
diff --git a/examples/all-clusters-app/all-clusters-common/all-clusters-app.matter b/examples/all-clusters-app/all-clusters-common/all-clusters-app.matter
@@ -3293,14 +3293,14 @@ server cluster MediaInput = 1287 {
     kNameUpdates = 0x1;
   }
 
-  struct InputInfo {
+  struct InputInfoStruct {
     int8u index = 0;
     InputTypeEnum inputType = 1;
     char_string<32> name = 2;
     char_string<32> description = 3;
   }
 
-  readonly attribute InputInfo inputList[] = 0;
+  readonly attribute InputInfoStruct inputList[] = 0;
   readonly attribute int8u currentInput = 1;
   readonly attribute bitmap32 featureMap = 65532;
   readonly attribute int16u clusterRevision = 65533;

diff --git a/examples/all-clusters-minimal-app/all-clusters-common/all-clusters-minimal-app.matter b/examples/all-clusters-minimal-app/all-clusters-common/all-clusters-minimal-app.matter
@@ -2649,14 +2649,14 @@ server cluster MediaInput = 1287 {
     kNameUpdates = 0x1;
   }
 
-  struct InputInfo {
+  struct InputInfoStruct {
     int8u index = 0;
     InputTypeEnum inputType = 1;
     char_string<32> name = 2;
     char_string<32> description = 3;
   }
 
-  readonly attribute InputInfo inputList[] = 0;
+  readonly attribute InputInfoStruct inputList[] = 0;
   readonly attribute int8u currentInput = 1;
   readonly attribute bitmap32 featureMap = 65532;
   readonly attribute int16u clusterRevision = 65533;

diff --git a/examples/tv-app/android/java/MediaInputManager.cpp b/examples/tv-app/android/java/MediaInputManager.cpp
@@ -72,7 +72,7 @@ CHIP_ERROR MediaInputManager::HandleGetInputList(chip::app::AttributeValueEncode
         jint size = env->GetArrayLength(inputArray);
         for (int i = 0; i < size; i++)
         {
-            app::Clusters::MediaInput::Structs::InputInfo::Type mediaInput;
+            app::Clusters::MediaInput::Structs::InputInfoStruct::Type mediaInput;
 
             jobject inputObj  = env->GetObjectArrayElement(inputArray, i);
             jclass inputClass = env->GetObjectClass(inputObj);

diff --git a/examples/tv-app/linux/include/media-input/MediaInputManager.h b/examples/tv-app/linux/include/media-input/MediaInputManager.h
@@ -22,7 +22,7 @@
 #include <app/clusters/media-input-server/media-input-server.h>
 #include <vector>
 
-using InputInfoType = chip::app::Clusters::MediaInput::Structs::InputInfo::Type;
+using InputInfoType = chip::app::Clusters::MediaInput::Structs::InputInfoStruct::Type;
 
 class MediaInputManager : public chip::app::Clusters::MediaInput::Delegate
 {
@@ -39,7 +39,7 @@ class MediaInputManager : public chip::app::Clusters::MediaInput::Delegate
 
 protected:
     uint8_t mCurrentInput;
-    std::vector<chip::app::Clusters::MediaInput::Structs::InputInfo::Type> mInputs;
+    std::vector<chip::app::Clusters::MediaInput::Structs::InputInfoStruct::Type> mInputs;
     // Magic numbers are here on purpose, please allocate memory
     char mCharDataBuffer[10][32];
 };
diff --git a/examples/tv-app/tv-common/tv-app.matter b/examples/tv-app/tv-common/tv-app.matter
@@ -1744,14 +1744,14 @@ server cluster MediaInput = 1287 {
     kNameUpdates = 0x1;
   }
 
-  struct InputInfo {
+  struct InputInfoStruct {
     int8u index = 0;
     InputTypeEnum inputType = 1;
     char_string<32> name = 2;
     char_string<32> description = 3;
   }
 
-  readonly attribute InputInfo inputList[] = 0;
+  readonly attribute InputInfoStruct inputList[] = 0;
   readonly attribute int8u currentInput = 1;
   readonly attribute bitmap32 featureMap = 65532;
   readonly attribute int16u clusterRevision = 65533;

diff --git a/examples/tv-casting-app/tv-casting-common/tv-casting-app.matter b/examples/tv-casting-app/tv-casting-common/tv-casting-app.matter
@@ -1816,14 +1816,14 @@ client cluster MediaInput = 1287 {
     kNameUpdates = 0x1;
   }
 
-  struct InputInfo {
+  struct InputInfoStruct {
     int8u index = 0;
     InputTypeEnum inputType = 1;
     char_string<32> name = 2;
     char_string<32> description = 3;
   }
 
-  readonly attribute InputInfo inputList[] = 0;
+  readonly attribute InputInfoStruct inputList[] = 0;
   readonly attribute int16u clusterRevision = 65533;
 
   request struct SelectInputRequest {

diff --git a/src/app/clusters/media-input-server/media-input-delegate.h b/src/app/clusters/media-input-server/media-input-delegate.h
@@ -34,7 +34,7 @@ namespace MediaInput {
 class Delegate
 {
 public:
-    // no easy way to handle the return memory of app::Clusters::MediaInput::Structs::InputInfo::Type list, so encoder is used
+    // no easy way to handle the return memory of app::Clusters::MediaInput::Structs::InputInfoStruct::Type list, so encoder is used
     virtual CHIP_ERROR HandleGetInputList(app::AttributeValueEncoder & aEncoder)     = 0;
     virtual uint8_t HandleGetCurrentInput()                                          = 0;
     virtual bool HandleSelectInput(const uint8_t index)                              = 0;

diff --git a/src/app/tests/suites/TV_MediaInputCluster.yaml b/src/app/tests/suites/TV_MediaInputCluster.yaml
@@ -35,16 +35,16 @@ tests:
           value:
               [
                   {
-                      index: 1,
-                      inputType: 4,
-                      name: "HDMI",
-                      description: "High-Definition Multimedia Interface",
+                      Index: 1,
+                      InputType: 4,
+                      Name: "HDMI",
+                      Description: "High-Definition Multimedia Interface",
                   },
                   {
-                      index: 2,
-                      inputType: 4,
-                      name: "HDMI",
-                      description: "High-Definition Multimedia Interface",
+                      Index: 2,
+                      InputType: 4,
+                      Name: "HDMI",
+                      Description: "High-Definition Multimedia Interface",
                   },
               ]
 
@@ -83,15 +83,15 @@ tests:
           value:
               [
                   {
-                      index: 1,
-                      inputType: 4,
-                      name: "HDMI Test",
-                      description: "High-Definition Multimedia Interface",
+                      Index: 1,
+                      InputType: 4,
+                      Name: "HDMI Test",
+                      Description: "High-Definition Multimedia Interface",
                   },
                   {
-                      index: 2,
-                      inputType: 4,
-                      name: "HDMI",
-                      description: "High-Definition Multimedia Interface",
+                      Index: 2,
+                      InputType: 4,
+                      Name: "HDMI",
+                      Description: "High-Definition Multimedia Interface",
                   },
               ]
diff --git a/src/app/zap-templates/zcl/data-model/chip/media-input-cluster.xml b/src/app/zap-templates/zcl/data-model/chip/media-input-cluster.xml
@@ -25,12 +25,12 @@ limitations under the License.
     <server init="false" tick="false">true</server>
     <!-- TODO: Add feature map once it is supported -->
     <description>This cluster provides an interface for controlling the Input Selector on a media device such as a TV.</description>
-    <attribute side="server" code="0x0000" define="MEDIA_INPUT_LIST"          type="ARRAY" entryType="InputInfo" length="254"           writable="false" optional="false">InputList</attribute>
-    <attribute side="server" code="0x0001" define="MEDIA_INPUT_CURRENT_INPUT" type="INT8U" default="0x00"        min="0x00" max="0xFF"  writable="false" optional="true">CurrentInput</attribute>
+    <attribute side="server" code="0x0000" define="MEDIA_INPUT_LIST"          type="ARRAY" entryType="InputInfoStruct" length="254"           writable="false" optional="false">InputList</attribute>
+    <attribute side="server" code="0x0001" define="MEDIA_INPUT_CURRENT_INPUT" type="INT8U" default="0x00"              min="0x00" max="0xFF"  writable="false" optional="false">CurrentInput</attribute>
 
     <command source="client" code="0x00" name="SelectInput" optional="false">
       <description>Upon receipt, this SHALL change the input on the media device to the input at a specific index in the Input List.</description>
-      <arg name="index" type="INT8U"/>
+      <arg name="Index" type="INT8U"/>
     </command>
 
     <command source="client" code="0x01" name="ShowInputStatus" optional="false">
@@ -43,18 +43,18 @@ limitations under the License.
 
     <command source="client" code="0x03" name="RenameInput" optional="true">
       <description>Upon receipt, this SHALL rename the input at a specific index in the Input List. Updates to the input name SHALL appear in the TV settings menus.    </description>
-      <arg name="index" type="INT8U"/>
-      <arg name="name" type="CHAR_STRING"/>
+      <arg name="Index" type="INT8U"/>
+      <arg name="Name" type="CHAR_STRING"/>
     </command>
 
   </cluster>
 
-  <struct name="InputInfo">
+  <struct name="InputInfoStruct">
     <cluster code="0x0507"/>
-    <item name="index" type="INT8U"/>
-    <item name="inputType" type="InputTypeEnum"/>
-    <item name="name" type="CHAR_STRING" length="32"/>
-    <item name="description" type="CHAR_STRING" length="32"/>
+    <item name="Index" type="INT8U"/>
+    <item name="InputType" type="InputTypeEnum"/>
+    <item name="Name" type="CHAR_STRING" length="32"/>
+    <item name="Description" type="CHAR_STRING" length="32"/>
   </struct>
 
   <enum name="InputTypeEnum" type="ENUM8">
@@ -63,13 +63,13 @@ limitations under the License.
     <item name="Aux" value="0x01"/>
     <item name="Coax" value="0x02"/>
     <item name="Composite" value="0x03"/>
-    <item name="Hdmi" value="0x04"/>
+    <item name="HDMI" value="0x04"/>
     <item name="Input" value="0x05"/>
     <item name="Line" value="0x06"/>
     <item name="Optical" value="0x07"/>
     <item name="Video" value="0x08"/>
-    <item name="Scart" value="0x09"/>
-    <item name="Usb" value="0x0A"/>
+    <item name="SCART" value="0x09"/>
+    <item name="USB" value="0x0A"/>
     <item name="Other" value="0x0B"/>
   </enum>
 

diff --git a/src/controller/data_model/controller-clusters.matter b/src/controller/data_model/controller-clusters.matter
@@ -3696,14 +3696,14 @@ client cluster MediaInput = 1287 {
     kNameUpdates = 0x1;
   }
 
-  struct InputInfo {
+  struct InputInfoStruct {
     int8u index = 0;
     InputTypeEnum inputType = 1;
     char_string<32> name = 2;
     char_string<32> description = 3;
   }
 
-  readonly attribute InputInfo inputList[] = 0;
+  readonly attribute InputInfoStruct inputList[] = 0;
   readonly attribute int8u currentInput = 1;
   readonly attribute command_id generatedCommandList[] = 65528;
   readonly attribute command_id acceptedCommandList[] = 65529;

diff --git a/src/controller/java/zap-generated/CHIPAttributeTLVValueDecoder.cpp b/src/controller/java/zap-generated/CHIPAttributeTLVValueDecoder.cpp
diff --git a/src/controller/java/zap-generated/CHIPReadCallbacks.cpp b/src/controller/java/zap-generated/CHIPReadCallbacks.cpp
diff --git a/src/controller/java/zap-generated/CHIPReadCallbacks.h b/src/controller/java/zap-generated/CHIPReadCallbacks.h
diff --git a/src/controller/java/zap-generated/chip/devicecontroller/ChipClusters.java b/src/controller/java/zap-generated/chip/devicecontroller/ChipClusters.java
diff --git a/src/controller/java/zap-generated/chip/devicecontroller/ChipStructs.java b/src/controller/java/zap-generated/chip/devicecontroller/ChipStructs.java
diff --git a/src/controller/java/zap-generated/chip/devicecontroller/ClusterInfoMapping.java b/src/controller/java/zap-generated/chip/devicecontroller/ClusterInfoMapping.java