Re: [PATCH] iio: adc: ad9467: refine mismatch vs unknown chip-id messages

From: Jonathan Cameron
Date: Wed Sep 16 2020 - 14:15:51 EST


On Wed, 16 Sep 2020 11:31:28 +0300
Alexandru Ardelean <alexandru.ardelean@xxxxxxxxxx> wrote:

> We should probably print what the expected chip-ID is. We already have
> that information available, based on the device specified via
> device-tree.
>
> Signed-off-by: Alexandru Ardelean <alexandru.ardelean@xxxxxxxxxx>
applied.

thanks,

Jonathan

> ---
> drivers/iio/adc/ad9467.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/iio/adc/ad9467.c b/drivers/iio/adc/ad9467.c
> index 8a6682b567be..f068256cfca9 100644
> --- a/drivers/iio/adc/ad9467.c
> +++ b/drivers/iio/adc/ad9467.c
> @@ -390,7 +390,8 @@ static int ad9467_probe(struct spi_device *spi)
>
> id = ad9467_spi_read(spi, AN877_ADC_REG_CHIP_ID);
> if (id != conv->chip_info->id) {
> - dev_err(&spi->dev, "Unrecognized CHIP_ID 0x%X\n", id);
> + dev_err(&spi->dev, "Mismatch CHIP_ID, got 0x%X, expected 0x%X\n",
> + id, conv->chip_info->id);
> return -ENODEV;
> }
>