From 1c6b1f755447ee64f46cdb89c9ad916d87707710 Mon Sep 17 00:00:00 2001
From: Daniel Agar <daniel@agar.ca>
Date: Fri, 19 Nov 2021 21:06:06 -0500
Subject: [PATCH] pwm_out: parameter failure error messages

---
 src/drivers/pwm_out/PWMOut.cpp | 21 ++++++++++++++++++++-
 1 file changed, 20 insertions(+), 1 deletion(-)

diff --git a/src/drivers/pwm_out/PWMOut.cpp b/src/drivers/pwm_out/PWMOut.cpp
index 0dd2f411ec..2ae68f4a4e 100644
--- a/src/drivers/pwm_out/PWMOut.cpp
+++ b/src/drivers/pwm_out/PWMOut.cpp
@@ -576,6 +576,9 @@ void PWMOut::update_params()
 				} else if (pwm_default_channel_mask & 1 << i) {
 					_mixing_output.minValue(i) = pwm_min_default;
 				}
+
+			} else {
+				PX4_ERR("param %s not found", str);
 			}
 		}
 
@@ -596,6 +599,9 @@ void PWMOut::update_params()
 				} else if (pwm_default_channel_mask & 1 << i) {
 					_mixing_output.maxValue(i) = pwm_max_default;
 				}
+
+			} else {
+				PX4_ERR("param %s not found", str);
 			}
 		}
 
@@ -613,6 +619,9 @@ void PWMOut::update_params()
 						param_set(param_find(str), &pwm_fail_new);
 					}
 				}
+
+			} else {
+				PX4_ERR("param %s not found", str);
 			}
 		}
 
@@ -633,6 +642,9 @@ void PWMOut::update_params()
 				} else if (pwm_default_channel_mask & 1 << i) {
 					_mixing_output.disarmedValue(i) = pwm_disarmed_default;
 				}
+
+			} else {
+				PX4_ERR("param %s not found", str);
 			}
 
 			if (_mixing_output.disarmedValue(i) > 0) {
@@ -654,6 +666,9 @@ void PWMOut::update_params()
 				} else {
 					reverse_pwm_mask = reverse_pwm_mask & ~(1 << i);
 				}
+
+			} else {
+				PX4_ERR("param %s not found", str);
 			}
 		}
 	}
@@ -665,7 +680,11 @@ void PWMOut::update_params()
 			sprintf(str, "%s_TRIM%u", prefix, i + 1);
 
 			float pval = 0.0f;
-			param_get(param_find(str), &pval);
+
+			if (param_get(param_find(str), &pval) != PX4_OK) {
+				PX4_ERR("param %s not found", str);
+			}
+
 			values[i] = roundf(10000 * pval);
 		}