Skip to content

Commit

Permalink
Mark ClusterRevision global attribute in XML files to be `side=eith…
Browse files Browse the repository at this point in the history
…er` (#31009)

* Switch globalAttribute 0xFFFD (revision) to be either side in xml

* Zap regen

---------

Co-authored-by: Andrei Litvin <[email protected]>
  • Loading branch information
andy31415 and andreilitvin authored Jan 4, 2024
1 parent 787db00 commit 8bef061
Show file tree
Hide file tree
Showing 22 changed files with 39 additions and 39 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -2426,7 +2426,7 @@ cluster TimeSynchronization = 56 {
Two types of switch devices are supported: latching switch (e.g. rocker switch) and momentary switch (e.g. push button), distinguished with their feature flags.
Interactions with the switch device are exposed as attributes (for the latching switch) and as events (for both types of switches). An interested party MAY subscribe to these attributes/events and thus be informed of the interactions, and can perform actions based on this, for example by sending commands to perform an action such as controlling a light or a window shade. */
cluster Switch = 59 {
revision 1; // NOTE: Default/not specifically set
revision 1;

bitmap Feature : bitmap32 {
kLatchingSwitch = 0x1;
Expand Down Expand Up @@ -2952,7 +2952,7 @@ cluster RefrigeratorAndTemperatureControlledCabinetMode = 82 {

/** This cluster supports remotely monitoring and controling the different typs of functionality available to a washing device, such as a washing machine. */
cluster LaundryWasherControls = 83 {
revision 1; // NOTE: Default/not specifically set
revision 1;

enum NumberOfRinsesEnum : enum8 {
kNone = 0;
Expand Down Expand Up @@ -3425,7 +3425,7 @@ provisional cluster MicrowaveOvenMode = 94 {

/** This cluster supports remotely monitoring and, where supported, changing the operational state of any device where a state machine is a part of the operation. */
cluster OperationalState = 96 {
revision 1; // NOTE: Default/not specifically set
revision 1;

enum ErrorStateEnum : enum8 {
kNoError = 0;
Expand Down Expand Up @@ -3491,7 +3491,7 @@ cluster OperationalState = 96 {

/** This cluster supports remotely monitoring and, where supported, changing the operational state of a Robotic Vacuum. */
cluster RvcOperationalState = 97 {
revision 1; // NOTE: Default/not specifically set
revision 1;

enum ErrorStateEnum : enum8 {
kFailedToFindChargingDock = 64;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2158,7 +2158,7 @@ cluster EthernetNetworkDiagnostics = 55 {
Two types of switch devices are supported: latching switch (e.g. rocker switch) and momentary switch (e.g. push button), distinguished with their feature flags.
Interactions with the switch device are exposed as attributes (for the latching switch) and as events (for both types of switches). An interested party MAY subscribe to these attributes/events and thus be informed of the interactions, and can perform actions based on this, for example by sending commands to perform an action such as controlling a light or a window shade. */
cluster Switch = 59 {
revision 1; // NOTE: Default/not specifically set
revision 1;

bitmap Feature : bitmap32 {
kLatchingSwitch = 0x1;
Expand Down
2 changes: 1 addition & 1 deletion examples/bridge-app/bridge-common/bridge-app.matter
Original file line number Diff line number Diff line change
Expand Up @@ -1550,7 +1550,7 @@ cluster EthernetNetworkDiagnostics = 55 {
Two types of switch devices are supported: latching switch (e.g. rocker switch) and momentary switch (e.g. push button), distinguished with their feature flags.
Interactions with the switch device are exposed as attributes (for the latching switch) and as events (for both types of switches). An interested party MAY subscribe to these attributes/events and thus be informed of the interactions, and can perform actions based on this, for example by sending commands to perform an action such as controlling a light or a window shade. */
cluster Switch = 59 {
revision 1; // NOTE: Default/not specifically set
revision 1;

bitmap Feature : bitmap32 {
kLatchingSwitch = 0x1;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1040,7 +1040,7 @@ cluster GroupKeyManagement = 63 {

/** This cluster supports remotely monitoring and, where supported, changing the operational state of any device where a state machine is a part of the operation. */
cluster OperationalState = 96 {
revision 1; // NOTE: Default/not specifically set
revision 1;

enum ErrorStateEnum : enum8 {
kNoError = 0;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -671,7 +671,7 @@ cluster GeneralDiagnostics = 51 {
Two types of switch devices are supported: latching switch (e.g. rocker switch) and momentary switch (e.g. push button), distinguished with their feature flags.
Interactions with the switch device are exposed as attributes (for the latching switch) and as events (for both types of switches). An interested party MAY subscribe to these attributes/events and thus be informed of the interactions, and can perform actions based on this, for example by sending commands to perform an action such as controlling a light or a window shade. */
cluster Switch = 59 {
revision 1; // NOTE: Default/not specifically set
revision 1;

bitmap Feature : bitmap32 {
kLatchingSwitch = 0x1;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1040,7 +1040,7 @@ cluster GroupKeyManagement = 63 {

/** This cluster supports remotely monitoring and, where supported, changing the operational state of any device where a state machine is a part of the operation. */
cluster OperationalState = 96 {
revision 1; // NOTE: Default/not specifically set
revision 1;

enum ErrorStateEnum : enum8 {
kNoError = 0;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1112,7 +1112,7 @@ cluster RvcCleanMode = 85 {

/** This cluster supports remotely monitoring and, where supported, changing the operational state of a Robotic Vacuum. */
cluster RvcOperationalState = 97 {
revision 1; // NOTE: Default/not specifically set
revision 1;

enum ErrorStateEnum : enum8 {
kFailedToFindChargingDock = 64;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1138,7 +1138,7 @@ cluster GroupKeyManagement = 63 {

/** This cluster supports remotely monitoring and, where supported, changing the operational state of any device where a state machine is a part of the operation. */
cluster OperationalState = 96 {
revision 1; // NOTE: Default/not specifically set
revision 1;

enum ErrorStateEnum : enum8 {
kNoError = 0;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1769,7 +1769,7 @@ cluster TimeSynchronization = 56 {
Two types of switch devices are supported: latching switch (e.g. rocker switch) and momentary switch (e.g. push button), distinguished with their feature flags.
Interactions with the switch device are exposed as attributes (for the latching switch) and as events (for both types of switches). An interested party MAY subscribe to these attributes/events and thus be informed of the interactions, and can perform actions based on this, for example by sending commands to perform an action such as controlling a light or a window shade. */
cluster Switch = 59 {
revision 1; // NOTE: Default/not specifically set
revision 1;

bitmap Feature : bitmap32 {
kLatchingSwitch = 0x1;
Expand Down
2 changes: 1 addition & 1 deletion examples/lighting-app/lighting-common/lighting-app.matter
Original file line number Diff line number Diff line change
Expand Up @@ -1675,7 +1675,7 @@ cluster EthernetNetworkDiagnostics = 55 {
Two types of switch devices are supported: latching switch (e.g. rocker switch) and momentary switch (e.g. push button), distinguished with their feature flags.
Interactions with the switch device are exposed as attributes (for the latching switch) and as events (for both types of switches). An interested party MAY subscribe to these attributes/events and thus be informed of the interactions, and can perform actions based on this, for example by sending commands to perform an action such as controlling a light or a window shade. */
cluster Switch = 59 {
revision 1; // NOTE: Default/not specifically set
revision 1;

bitmap Feature : bitmap32 {
kLatchingSwitch = 0x1;
Expand Down
4 changes: 2 additions & 2 deletions examples/placeholder/linux/apps/app1/config.matter
Original file line number Diff line number Diff line change
Expand Up @@ -2544,7 +2544,7 @@ cluster BridgedDeviceBasicInformation = 57 {
Two types of switch devices are supported: latching switch (e.g. rocker switch) and momentary switch (e.g. push button), distinguished with their feature flags.
Interactions with the switch device are exposed as attributes (for the latching switch) and as events (for both types of switches). An interested party MAY subscribe to these attributes/events and thus be informed of the interactions, and can perform actions based on this, for example by sending commands to perform an action such as controlling a light or a window shade. */
cluster Switch = 59 {
revision 1; // NOTE: Default/not specifically set
revision 1;

bitmap Feature : bitmap32 {
kLatchingSwitch = 0x1;
Expand Down Expand Up @@ -2599,7 +2599,7 @@ cluster Switch = 59 {
Two types of switch devices are supported: latching switch (e.g. rocker switch) and momentary switch (e.g. push button), distinguished with their feature flags.
Interactions with the switch device are exposed as attributes (for the latching switch) and as events (for both types of switches). An interested party MAY subscribe to these attributes/events and thus be informed of the interactions, and can perform actions based on this, for example by sending commands to perform an action such as controlling a light or a window shade. */
cluster Switch = 59 {
revision 1; // NOTE: Default/not specifically set
revision 1;

bitmap Feature : bitmap32 {
kLatchingSwitch = 0x1;
Expand Down
4 changes: 2 additions & 2 deletions examples/placeholder/linux/apps/app2/config.matter
Original file line number Diff line number Diff line change
Expand Up @@ -2501,7 +2501,7 @@ cluster BridgedDeviceBasicInformation = 57 {
Two types of switch devices are supported: latching switch (e.g. rocker switch) and momentary switch (e.g. push button), distinguished with their feature flags.
Interactions with the switch device are exposed as attributes (for the latching switch) and as events (for both types of switches). An interested party MAY subscribe to these attributes/events and thus be informed of the interactions, and can perform actions based on this, for example by sending commands to perform an action such as controlling a light or a window shade. */
cluster Switch = 59 {
revision 1; // NOTE: Default/not specifically set
revision 1;

bitmap Feature : bitmap32 {
kLatchingSwitch = 0x1;
Expand Down Expand Up @@ -2556,7 +2556,7 @@ cluster Switch = 59 {
Two types of switch devices are supported: latching switch (e.g. rocker switch) and momentary switch (e.g. push button), distinguished with their feature flags.
Interactions with the switch device are exposed as attributes (for the latching switch) and as events (for both types of switches). An interested party MAY subscribe to these attributes/events and thus be informed of the interactions, and can perform actions based on this, for example by sending commands to perform an action such as controlling a light or a window shade. */
cluster Switch = 59 {
revision 1; // NOTE: Default/not specifically set
revision 1;

bitmap Feature : bitmap32 {
kLatchingSwitch = 0x1;
Expand Down
2 changes: 1 addition & 1 deletion examples/rvc-app/rvc-common/rvc-app.matter
Original file line number Diff line number Diff line change
Expand Up @@ -1035,7 +1035,7 @@ cluster RvcCleanMode = 85 {

/** This cluster supports remotely monitoring and, where supported, changing the operational state of a Robotic Vacuum. */
cluster RvcOperationalState = 97 {
revision 1; // NOTE: Default/not specifically set
revision 1;

enum ErrorStateEnum : enum8 {
kFailedToFindChargingDock = 64;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ limitations under the License.
<server init="false" tick="false">true</server>
<description>This cluster allows a client to manage the power draw of a device. An example of such a client could be an Energy Management System (EMS) which controls an Energy Smart Appliance (ESA).</description>

<globalAttribute side="server" code="0xFFFD" value="2" />
<globalAttribute side="either" code="0xFFFD" value="2" />

<!--Attributes-->
<attribute code="0x0000" side="server" type="ESATypeEnum" define="ESA_TYPE" writable="false" optional="false">ESAType</attribute>
Expand Down Expand Up @@ -189,4 +189,4 @@ limitations under the License.
<item fieldId="3" name="MaximumEnergy" type="energy_mwh" optional="true"/>
<item fieldId="4" name="LoadControl" type="int8s" optional="true"/>
</struct>
</configurator>
</configurator>
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ limitations under the License.
<server init="false" tick="false">true</server>
<description>Electric Vehicle Supply Equipment (EVSE) is equipment used to charge an Electric Vehicle (EV) or Plug-In Hybrid Electric Vehicle. This cluster provides an interface to the functionality of Electric Vehicle Supply Equipment (EVSE) management.</description>

<globalAttribute side="server" code="0xFFFD" value="2" />
<globalAttribute side="either" code="0xFFFD" value="2" />

<!--Attributes-->
<attribute code="0x0000" side="server" type="StateEnum" define="STATE" isNullable="true" writable="false" optional="false">State</attribute>
Expand Down Expand Up @@ -214,4 +214,4 @@ limitations under the License.
<field name="RFID" mask="0x8"/>
<field name="V2X" mask="0x10"/>
</bitmap>
</configurator>
</configurator>
Original file line number Diff line number Diff line change
Expand Up @@ -36,9 +36,9 @@ limitations under the License.
<description>This cluster supports remotely monitoring and controling the different typs of
functionality available to a drying device, such as a laundry dryer.</description>

<globalAttribute side="server" code="0xFFFD" value="1" />
<globalAttribute side="either" code="0xFFFD" value="1" />

<attribute side="server" code="0x0000" define="SUPPORTED_DRYNESS_LEVELS" type="array" entryType="DrynessLevelEnum" writable="false" isNullable="false" optional="false" min="0x01" max="0x04">SupportedDrynessLevels</attribute>
<attribute side="server" code="0x0001" define="SELECTED_DRYNESS_LEVEL" type="DrynessLevelEnum" writable="true" isNullable="true" optional="false">SelectedDrynessLevel</attribute>
</cluster>
</configurator>
</configurator>
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ limitations under the License.
<server init="false" tick="false">true</server>
<description>This cluster supports remotely monitoring and, where supported, changing the operational state of any device where a state machine is a part of the operation.</description>

<globalAttribute side="server" code="0xFFFD" value="1" />
<globalAttribute side="either" code="0xFFFD" value="1" />

<attribute side="server" code="0x0000" define="PHASE_LIST" type="array" entryType="char_string" writable="false" isNullable="true" optional="false">PhaseList</attribute>
<attribute side="server" code="0x0001" define="CURRENT_PHASE" type="int8u" min="0x00" max="0x1F" writable="false" isNullable="true" optional="false">CurrentPhase</attribute>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ limitations under the License.
<server init="false" tick="false">true</server>
<description>This cluster supports remotely monitoring and, where supported, changing the operational state of a Robotic Vacuum.</description>

<globalAttribute side="server" code="0xFFFD" value="1" />
<globalAttribute side="either" code="0xFFFD" value="1" />

<attribute side="server" code="0x0000" define="PHASE_LIST" type="array" entryType="char_string" writable="false" isNullable="true" optional="false">PhaseList</attribute>
<attribute side="server" code="0x0001" define="CURRENT_PHASE" type="int8u" min="0x00" max="0x1F" writable="false" isNullable="true" optional="false">CurrentPhase</attribute>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ Two types of switch devices are supported: latching switch (e.g. rocker switch)
Interactions with the switch device are exposed as attributes (for the latching switch) and as events (for both types of switches). An interested party MAY subscribe to these attributes/events and thus be informed of the interactions, and can perform actions based on this, for example by sending commands to perform an action such as controlling a light or a window shade.</description>

<globalAttribute side="server" code="0xFFFC" value="0" />
<globalAttribute side="server" code="0xFFFD" value="1" />
<globalAttribute side="either" code="0xFFFD" value="1" />

<attribute side="server" code="0x0000" define="NUMBER_OF_POSITIONS" type="int8u" writable="false" optional="false" default="2" min="2">NumberOfPositions</attribute>
<attribute side="server" code="0x0001" define="CURRENT_POSITION" type="int8u" writable="false" reportable="true" optional="false">CurrentPosition</attribute>
Expand Down
4 changes: 2 additions & 2 deletions src/app/zap-templates/zcl/data-model/chip/timer-cluster.xml
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ limitations under the License.
<server init="false" tick="false">true</server>
<description>This cluster supports creating a simple timer functionality.</description>

<globalAttribute side="server" code="0xFFFD" value="1" />
<globalAttribute side="either" code="0xFFFD" value="1" />

<attribute side="server" code="0x0000" define="SET_TIME" type="elapsed_s" writable="false" isNullable="false" optional="false">SetTime</attribute>
<attribute side="server" code="0x0001" define="TIME_REMAINING" type="elapsed_s" writable="false" isNullable="false" optional="false">TimeRemaining</attribute>
Expand Down Expand Up @@ -73,4 +73,4 @@ limitations under the License.
<arg name="TimeReduction" type="elapsed_s" optional="false"/>
</command>
</cluster>
</configurator>
</configurator>
Original file line number Diff line number Diff line change
Expand Up @@ -40,12 +40,12 @@ limitations under the License.
<server init="false" tick="false">true</server>
<description>This cluster supports remotely monitoring and controling the different typs of functionality available to a washing device, such as a washing machine.</description>

<globalAttribute side="server" code="0xFFFD" value="1" />
<globalAttribute side="either" code="0xFFFD" value="1" />

<attribute side="server" code="0x0000" define="SPIN_SPEEDS" type="array" entryType="char_string" writable="false" isNullable="false" optional="true">SpinSpeeds</attribute>
<attribute side="server" code="0x0001" define="SPIN_SPEED_CURRENT" type="int8u" min="0x00" max="0x1F" writable="true" isNullable="true" optional="true">SpinSpeedCurrent</attribute>
<attribute side="server" code="0x0002" define="NUMBER_OF_RINSES" type="NumberOfRinsesEnum" writable="true" isNullable="false" optional="true">NumberOfRinses</attribute>
<attribute side="server" code="0x0003" define="SUPPORTED_RINSES" type="array" entryType="NumberOfRinsesEnum" writable="false" isNullable="false" optional="true">SupportedRinses</attribute>
</cluster>
</configurator>


Loading

0 comments on commit 8bef061

Please sign in to comment.