drivers/mrf24j40: replace && with || in TX power validation

Stopkaa · Stopkaa · commit 27ceb0eb8c99 · 2025-11-21T12:15:37.000+01:00
diff --git a/drivers/mrf24j40/mrf24j40_radio_hal.c b/drivers/mrf24j40/mrf24j40_radio_hal.c
@@ -254,7 +254,7 @@ static int _config_phy(ieee802154_dev_t *hal, const ieee802154_phy_conf_t *conf)
     mrf24j40_t *dev = hal->priv;
     int8_t pow = conf->pow;
     uint8_t channel = conf->channel;
-    if (pow < MRF24J40_MIN_TXPOWER && pow > MRF24J40_MAX_TXPOWER) {
+    if (pow < MRF24J40_MIN_TXPOWER || pow > MRF24J40_MAX_TXPOWER) {
         return -EINVAL;
     }
     mrf24j40_set_txpower(dev, pow);

Original file line number	Diff line number	Diff line change
`@@ -254,7 +254,7 @@ static int _config_phy(ieee802154_dev_t hal, const ieee802154_phy_conf_t conf)`
`254`	`254`	`mrf24j40_t *dev = hal->priv;`
`255`	`255`	`int8_t pow = conf->pow;`
`256`	`256`	`uint8_t channel = conf->channel;`
`257`		`- if (pow < MRF24J40_MIN_TXPOWER && pow > MRF24J40_MAX_TXPOWER) {`
	`257`	`+ if (pow < MRF24J40_MIN_TXPOWER \|\| pow > MRF24J40_MAX_TXPOWER) {`
`258`	`258`	`return -EINVAL;`
`259`	`259`	`}`
`260`	`260`	`mrf24j40_set_txpower(dev, pow);`