staging: pi433: print rf69 debug message more detail

When pi433 failed for rf69 configuration, Debug message should help for
finding which value is incorrect. But it's hard to know because it just
prints "illegal value". This patch make print message more detail.

Signed-off-by: Sidong Yang <realwakka@gmail.com>
Link: https://lore.kernel.org/r/20211107105947.21928-1-realwakka@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/staging/pi433/rf69.c b/drivers/staging/pi433/rf69.c
index 7d86bb8be24535a99763b6bd50de1d3c7d67d5eb..dc047bcdf44b05d6dd51a8c456b027be93b3691b 100644 (file)
--- a/drivers/staging/pi433/rf69.c
+++ b/drivers/staging/pi433/rf69.c
@@ -113,7 +113,7 @@ int rf69_set_mode(struct spi_device *spi, enum mode mode)
        };
 
        if (unlikely(mode >= ARRAY_SIZE(mode_map))) {
-               dev_dbg(&spi->dev, "set: illegal input param");
+               dev_dbg(&spi->dev, "set: illegal mode %u", mode);
                return -EINVAL;
        }
 
@@ -143,7 +143,7 @@ int rf69_set_modulation(struct spi_device *spi, enum modulation modulation)
        };
 
        if (unlikely(modulation >= ARRAY_SIZE(modulation_map))) {
-               dev_dbg(&spi->dev, "set: illegal input param");
+               dev_dbg(&spi->dev, "set: illegal modulation %u", modulation);
                return -EINVAL;
        }
 
@@ -191,7 +191,7 @@ int rf69_set_modulation_shaping(struct spi_device *spi,
                                                   MASK_DATAMODUL_MODULATION_SHAPE,
                                                   DATAMODUL_MODULATION_SHAPE_0_3);
                default:
-                       dev_dbg(&spi->dev, "set: illegal input param");
+                       dev_dbg(&spi->dev, "set: illegal mod shaping for FSK %u", mod_shaping);
                        return -EINVAL;
                }
        case OOK:
@@ -209,7 +209,7 @@ int rf69_set_modulation_shaping(struct spi_device *spi,
                                                   MASK_DATAMODUL_MODULATION_SHAPE,
                                                   DATAMODUL_MODULATION_SHAPE_2BR);
                default:
-                       dev_dbg(&spi->dev, "set: illegal input param");
+                       dev_dbg(&spi->dev, "set: illegal mod shaping for OOK %u", mod_shaping);
                        return -EINVAL;
                }
        default:
@@ -392,7 +392,7 @@ int rf69_set_output_power_level(struct spi_device *spi, u8 power_level)
        return rf69_read_mod_write(spi, REG_PALEVEL, MASK_PALEVEL_OUTPUT_POWER,
                                   power_level);
 failed:
-       dev_dbg(&spi->dev, "set: illegal input param");
+       dev_dbg(&spi->dev, "set: illegal power level %u", power_level);
        return -EINVAL;
 }
 
@@ -417,7 +417,7 @@ int rf69_set_pa_ramp(struct spi_device *spi, enum pa_ramp pa_ramp)
        };
 
        if (unlikely(pa_ramp >= ARRAY_SIZE(pa_ramp_map))) {
-               dev_dbg(&spi->dev, "set: illegal input param");
+               dev_dbg(&spi->dev, "set: illegal pa_ramp %u", pa_ramp);
                return -EINVAL;
        }
 
@@ -433,7 +433,7 @@ int rf69_set_antenna_impedance(struct spi_device *spi,
        case two_hundred_ohm:
                return rf69_set_bit(spi, REG_LNA, MASK_LNA_ZIN);
        default:
-               dev_dbg(&spi->dev, "set: illegal input param");
+               dev_dbg(&spi->dev, "set: illegal antenna impedance %u", antenna_impedance);
                return -EINVAL;
        }
 }
@@ -451,7 +451,7 @@ int rf69_set_lna_gain(struct spi_device *spi, enum lna_gain lna_gain)
        };
 
        if (unlikely(lna_gain >= ARRAY_SIZE(lna_gain_map))) {
-               dev_dbg(&spi->dev, "set: illegal input param");
+               dev_dbg(&spi->dev, "set: illegal lna gain %u", lna_gain);
                return -EINVAL;
        }
 
@@ -466,14 +466,14 @@ static int rf69_set_bandwidth_intern(struct spi_device *spi, u8 reg,
 
        // check value for mantisse and exponent
        if (exponent > 7) {
-               dev_dbg(&spi->dev, "set: illegal input param");
+               dev_dbg(&spi->dev, "set: illegal bandwidth exponent %u", exponent);
                return -EINVAL;
        }
 
        if ((mantisse != mantisse16) &&
            (mantisse != mantisse20) &&
            (mantisse != mantisse24)) {
-               dev_dbg(&spi->dev, "set: illegal input param");
+               dev_dbg(&spi->dev, "set: illegal bandwidth mantisse %u", mantisse);
                return -EINVAL;
        }
 
@@ -531,7 +531,7 @@ int rf69_set_ook_threshold_dec(struct spi_device *spi,
        };
 
        if (unlikely(threshold_decrement >= ARRAY_SIZE(td_map))) {
-               dev_dbg(&spi->dev, "set: illegal input param");
+               dev_dbg(&spi->dev, "set: illegal OOK threshold decrement %u", threshold_decrement);
                return -EINVAL;
        }
 
@@ -578,7 +578,7 @@ int rf69_set_dio_mapping(struct spi_device *spi, u8 dio_number, u8 value)
                dio_addr = REG_DIOMAPPING2;
                break;
        default:
-       dev_dbg(&spi->dev, "set: illegal input param");
+               dev_dbg(&spi->dev, "set: illegal dio number %u", dio_number);
                return -EINVAL;
        }
 
@@ -681,7 +681,7 @@ int rf69_set_fifo_fill_condition(struct spi_device *spi,
                return rf69_clear_bit(spi, REG_SYNC_CONFIG,
                                      MASK_SYNC_CONFIG_FIFO_FILL_CONDITION);
        default:
-               dev_dbg(&spi->dev, "set: illegal input param");
+               dev_dbg(&spi->dev, "set: illegal fifo fill condition %u", fifo_fill_condition);
                return -EINVAL;
        }
 }
@@ -690,7 +690,7 @@ int rf69_set_sync_size(struct spi_device *spi, u8 sync_size)
 {
        // check input value
        if (sync_size > 0x07) {
-               dev_dbg(&spi->dev, "set: illegal input param");
+               dev_dbg(&spi->dev, "set: illegal sync size %u", sync_size);
                return -EINVAL;
        }
 
@@ -727,7 +727,7 @@ int rf69_set_packet_format(struct spi_device *spi,
                return rf69_clear_bit(spi, REG_PACKETCONFIG1,
                                      MASK_PACKETCONFIG1_PACKET_FORMAT_VARIABLE);
        default:
-               dev_dbg(&spi->dev, "set: illegal input param");
+               dev_dbg(&spi->dev, "set: illegal packet format %u", packet_format);
                return -EINVAL;
        }
 }
@@ -753,7 +753,7 @@ int rf69_set_address_filtering(struct spi_device *spi,
        };
 
        if (unlikely(address_filtering >= ARRAY_SIZE(af_map))) {
-               dev_dbg(&spi->dev, "set: illegal input param");
+               dev_dbg(&spi->dev, "set: illegal address filtering %u", address_filtering);
                return -EINVAL;
        }
 
@@ -788,7 +788,7 @@ int rf69_set_tx_start_condition(struct spi_device *spi,
                return rf69_set_bit(spi, REG_FIFO_THRESH,
                                    MASK_FIFO_THRESH_TXSTART);
        default:
-               dev_dbg(&spi->dev, "set: illegal input param");
+               dev_dbg(&spi->dev, "set: illegal tx start condition %u", tx_start_condition);
                return -EINVAL;
        }
 }
@@ -799,7 +799,7 @@ int rf69_set_fifo_threshold(struct spi_device *spi, u8 threshold)
 
        /* check input value */
        if (threshold & 0x80) {
-               dev_dbg(&spi->dev, "set: illegal input param");
+               dev_dbg(&spi->dev, "set: illegal fifo threshold %u", threshold);
                return -EINVAL;
        }
 
@@ -826,7 +826,7 @@ int rf69_set_dagc(struct spi_device *spi, enum dagc dagc)
        };
 
        if (unlikely(dagc >= ARRAY_SIZE(dagc_map))) {
-               dev_dbg(&spi->dev, "set: illegal input param");
+               dev_dbg(&spi->dev, "set: illegal dagc %u", dagc);
                return -EINVAL;
        }