Update parameters in getRadioParameters (zsmartsystems#1409)

Signed-off-by: Chris Jackson <chris@cd-jackson.com>
passmanSA · Dec 2, 2024 · c324494 · c324494
1 parent 36b1761
commit c324494
Show file tree

Hide file tree

Showing 5 changed files with 23 additions and 22 deletions.
diff --git a/...ber/src/main/java/com/zsmartsystems/zigbee/console/ember/EmberConsoleNcpStateCommand.java b/...ber/src/main/java/com/zsmartsystems/zigbee/console/ember/EmberConsoleNcpStateCommand.java
@@ -92,7 +92,7 @@ private void ncpState(ZigBeeNetworkManager networkManager, PrintStream out) {
         int nwkAddress = ncp.getNwkAddress();
         EzspGetNetworkParametersResponse nwkParameterResponse = ncp.getNetworkParameters();
         EmberNetworkParameters nwkParameters = nwkParameterResponse.getParameters();
-        EzspGetRadioParametersResponse radioParametersResponse = ncp.getRadioParameters();
+        EzspGetRadioParametersResponse radioParametersResponse = ncp.getRadioParameters(0);
         EmberMultiPhyRadioParameters radioParameters = radioParametersResponse.getParameters();
 
         String mfgName = ncp.getMfgName();

diff --git a/com.zsmartsystems.zigbee.dongle.ember.autocode/src/main/resources/ezsp_protocol.xml b/com.zsmartsystems.zigbee.dongle.ember.autocode/src/main/resources/ezsp_protocol.xml
@@ -3706,15 +3706,15 @@
 		<command_parameters>
 			<parameter>
 				<data_type>uint8_t</data_type>
-				<name>childCount</name>
-				<description>The number of children the node currently has.</description>
+				<name>phyIndex</name>
+				<description>The physical radio to query.</description>
 			</parameter>
 		</command_parameters>
 		<response_parameters>
 			<parameter>
 				<data_type>EmberStatus</data_type>
 				<name>status</name>
-				<description>The result of the CBKE operation.</description>
+				<description>The result of the operation.</description>
 			</parameter>
 			<parameter>
 				<data_type>EmberMultiPhyRadioParameters</data_type>

diff --git a/...ems.zigbee.dongle.ember/src/main/java/com/zsmartsystems/zigbee/dongle/ember/EmberNcp.java b/...ems.zigbee.dongle.ember/src/main/java/com/zsmartsystems/zigbee/dongle/ember/EmberNcp.java
@@ -173,11 +173,12 @@ public EzspGetNetworkParametersResponse getNetworkParameters() {
     /**
      * Gets the current radio parameters, or an empty parameters class if there's an error
      *
+     * @param phyIndex the physical radio index
      * @return {@link EzspGetRadioParametersResponse} or null on error
      */
-    public EzspGetRadioParametersResponse getRadioParameters() {
+    public EzspGetRadioParametersResponse getRadioParameters(int phyIndex) {
         EzspGetRadioParametersRequest request = new EzspGetRadioParametersRequest();
-        request.setChildCount(1);
+        request.setPhyIndex(phyIndex);
         EzspTransaction transaction = zigBeeDongleEzsp.getProtocolHandler().sendEzspTransaction(
                 new EzspSingleResponseTransaction(request, EzspGetRadioParametersResponse.class));
         return (EzspGetRadioParametersResponse) transaction.getResponse();

diff --git a/...ava/com/zsmartsystems/zigbee/dongle/ember/ezsp/command/EzspGetRadioParametersRequest.java b/...ava/com/zsmartsystems/zigbee/dongle/ember/ezsp/command/EzspGetRadioParametersRequest.java
@@ -25,11 +25,11 @@ public class EzspGetRadioParametersRequest extends EzspFrameRequest {
     public static final int FRAME_ID = 0xFD;
 
     /**
-     * The number of children the node currently has.
+     * The physical radio to query.
      * <p>
      * EZSP type is <i>uint8_t</i> - Java type is {@link int}
      */
-    private int childCount;
+    private int phyIndex;
 
     /**
      * Serialiser used to serialise to binary line data
@@ -45,23 +45,23 @@ public EzspGetRadioParametersRequest() {
     }
 
     /**
-     * The number of children the node currently has.
+     * The physical radio to query.
      * <p>
      * EZSP type is <i>uint8_t</i> - Java type is {@link int}
      *
-     * @return the current childCount as {@link int}
+     * @return the current phyIndex as {@link int}
      */
-    public int getChildCount() {
-        return childCount;
+    public int getPhyIndex() {
+        return phyIndex;
     }
 
     /**
-     * The number of children the node currently has.
+     * The physical radio to query.
      *
-     * @param childCount the childCount to set as {@link int}
+     * @param phyIndex the phyIndex to set as {@link int}
      */
-    public void setChildCount(int childCount) {
-        this.childCount = childCount;
+    public void setPhyIndex(int phyIndex) {
+        this.phyIndex = phyIndex;
     }
 
     @Override
@@ -70,7 +70,7 @@ public int[] serialize() {
         serializeHeader(serializer);
 
         // Serialize the fields
-        serializer.serializeUInt8(childCount);
+        serializer.serializeUInt8(phyIndex);
         return serializer.getPayload();
     }
 
@@ -79,8 +79,8 @@ public String toString() {
         final StringBuilder builder = new StringBuilder(82);
         builder.append("EzspGetRadioParametersRequest [networkId=");
         builder.append(networkId);
-        builder.append(", childCount=");
-        builder.append(childCount);
+        builder.append(", phyIndex=");
+        builder.append(phyIndex);
         builder.append(']');
         return builder.toString();
     }

diff --git a/...va/com/zsmartsystems/zigbee/dongle/ember/ezsp/command/EzspGetRadioParametersResponse.java b/...va/com/zsmartsystems/zigbee/dongle/ember/ezsp/command/EzspGetRadioParametersResponse.java
@@ -26,7 +26,7 @@ public class EzspGetRadioParametersResponse extends EzspFrameResponse {
     public static final int FRAME_ID = 0xFD;
 
     /**
-     * The result of the CBKE operation.
+     * The result of the operation.
      * <p>
      * EZSP type is <i>EmberStatus</i> - Java type is {@link EmberStatus}
      */
@@ -52,7 +52,7 @@ public EzspGetRadioParametersResponse(int[] inputBuffer) {
     }
 
     /**
-     * The result of the CBKE operation.
+     * The result of the operation.
      * <p>
      * EZSP type is <i>EmberStatus</i> - Java type is {@link EmberStatus}
      *
@@ -63,7 +63,7 @@ public EmberStatus getStatus() {
     }
 
     /**
-     * The result of the CBKE operation.
+     * The result of the operation.
      *
      * @param status the status to set as {@link EmberStatus}
      */