From nobody Fri Jan 2 06:54:49 2026 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id A55F9CDB483 for ; Sun, 15 Oct 2023 15:18:21 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230162AbjJOPSV (ORCPT ); Sun, 15 Oct 2023 11:18:21 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:36034 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230142AbjJOPSO (ORCPT ); Sun, 15 Oct 2023 11:18:14 -0400 Received: from mail-wm1-x32b.google.com (mail-wm1-x32b.google.com [IPv6:2a00:1450:4864:20::32b]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id D6AF1FB; Sun, 15 Oct 2023 08:18:10 -0700 (PDT) Received: by mail-wm1-x32b.google.com with SMTP id 5b1f17b1804b1-40776b200e5so16255555e9.2; Sun, 15 Oct 2023 08:18:10 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1697383089; x=1697987889; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=NMdJnM8oHQ7eHl+UF9y/ARFxYpaLioFllg8ThKJYIfg=; b=M+UtqsvAtnViL34/Rjoh/Riubp27o6OMLkaELkSqKabmDk9Hx6aytZdkFYZ3PsvhIg 07Rwaa7mu5YW5gDPLiqrPUZlG1ukUVz4JyldU8WprJWVNRknYHDxzn4+9v6LI5k7XaKK sU6wLuc9aK6sENiCTdIEd1P+4bmSCOUoQDThmS0PnVq0W0rkQoFXVOqUKLp8Bt22QCOW 3w4/dUJgizxT09s0HDUUOtN8VzUowkNDvzvbH0jmdEH+GFDO/9RJEl7X2T0W5tFN0Y3+ IY6C7z/9yClKXyoFezHjbPqDVcnfsMqiQ4rrknNnWqAHT2YTcokQ9bXPZi/e7828oq+F TcBA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1697383089; x=1697987889; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=NMdJnM8oHQ7eHl+UF9y/ARFxYpaLioFllg8ThKJYIfg=; b=CTuaJA3Lao6Dk6fmeeoR4mS3pp+0bvl7Aysn64zOgl82bigFiFFZySiZQeb/0p2T94 G4cueCf+T+jBm2l+tKRr0pdk3Oufl4d16mcuPQH2nwwDgueITvE34Xx7qh01ywLXJ0uz 6cTmxq4F0amYPJM/i7glGY6wzCtTbobruLCveNoDfGctOXEhSl3sWl+xliHDe9azIbop j96YeeJq3yJ5dEkZkr64yYbrTNR3eMwFeYP7h0lIjnMD09yNZtO9QkYnJAswAgP4p95T JoX8eiZEx8UftagCfIPifpjG027TB/yZvxrihV5jXdVcmvfeD3WavMkFq5Hbtf0pydKd 3SHA== X-Gm-Message-State: AOJu0Yx4gMlPsKsoS/2qRBcOFM1NgTByt7JXJJEbUjm3KOGatfZZO1eG TwLmyORjTEAocIu5ycG5hjxRWa1r2Zk= X-Google-Smtp-Source: AGHT+IEHmVYTABMP6Rxh9SuqXGxC1tzOfrC1ZjC7nUsGzMqE3KM8pSLzVxxYQow+zZinwWWmWuP8zg== X-Received: by 2002:a05:600c:3586:b0:406:849c:52c3 with SMTP id p6-20020a05600c358600b00406849c52c3mr26286281wmq.22.1697383088044; Sun, 15 Oct 2023 08:18:08 -0700 (PDT) Received: from localhost.localdomain (13.red-83-35-58.dynamicip.rima-tde.net. [83.35.58.13]) by smtp.gmail.com with ESMTPSA id n35-20020a05600c3ba300b003fc16ee2864sm4712455wms.48.2023.10.15.08.18.07 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 15 Oct 2023 08:18:07 -0700 (PDT) From: Angel Iglesias To: linux-iio@vger.kernel.org Cc: Biju Das , linux-kernel@vger.kernel.org, Angel Iglesias , Jonathan Cameron , Lars-Peter Clausen , Andy Shevchenko , =?UTF-8?q?Uwe=20Kleine-K=C3=B6nig?= Subject: [PATCH 4/5] iio: pressure: bmp280: Allow multiple chips id per family of devices Date: Sun, 15 Oct 2023 17:16:26 +0200 Message-ID: <9f8489d82325b2dfb5c8c71c3d558d509b2b01bf.1697381932.git.ang.iglesiasg@gmail.com> X-Mailer: git-send-email 2.42.0 In-Reply-To: References: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" Improve device detection in certain chip families known to have various chip ids. Signed-off-by: Angel Iglesias diff --git a/drivers/iio/pressure/bmp280-core.c b/drivers/iio/pressure/bmp2= 80-core.c index ea02a623bb58..e3bb4d7906a9 100644 --- a/drivers/iio/pressure/bmp280-core.c +++ b/drivers/iio/pressure/bmp280-core.c @@ -38,6 +38,7 @@ #include #include /* For irq_get_irq_data() */ #include +#include #include #include =20 @@ -794,10 +795,12 @@ static int bmp280_chip_config(struct bmp280_data *dat= a) } =20 static const int bmp280_oversampling_avail[] =3D { 1, 2, 4, 8, 16 }; +static const u8 bmp280_chip_ids[] =3D { BMP280_CHIP_ID }; =20 const struct bmp280_chip_info bmp280_chip_info =3D { .id_reg =3D BMP280_REG_ID, - .chip_id =3D BMP280_CHIP_ID, + .chip_id =3D bmp280_chip_ids, + .num_chip_id =3D ARRAY_SIZE(bmp280_chip_ids), .regmap_config =3D &bmp280_regmap_config, .start_up_time =3D 2000, .channels =3D bmp280_channels, @@ -846,9 +849,12 @@ static int bme280_chip_config(struct bmp280_data *data) return bmp280_chip_config(data); } =20 +static const u8 bme280_chip_ids[] =3D { BME280_CHIP_ID }; + const struct bmp280_chip_info bme280_chip_info =3D { .id_reg =3D BMP280_REG_ID, - .chip_id =3D BME280_CHIP_ID, + .chip_id =3D bme280_chip_ids, + .num_chip_id =3D ARRAY_SIZE(bme280_chip_ids), .regmap_config =3D &bmp280_regmap_config, .start_up_time =3D 2000, .channels =3D bmp280_channels, @@ -1220,10 +1226,12 @@ static int bmp380_chip_config(struct bmp280_data *d= ata) =20 static const int bmp380_oversampling_avail[] =3D { 1, 2, 4, 8, 16, 32 }; static const int bmp380_iir_filter_coeffs_avail[] =3D { 1, 2, 4, 8, 16, 32= , 64, 128}; +static const u8 bmp380_chip_ids[] =3D { BMP380_CHIP_ID }; =20 const struct bmp280_chip_info bmp380_chip_info =3D { .id_reg =3D BMP380_REG_ID, - .chip_id =3D BMP380_CHIP_ID, + .chip_id =3D bmp380_chip_ids, + .num_chip_id =3D ARRAY_SIZE(bmp380_chip_ids), .regmap_config =3D &bmp380_regmap_config, .start_up_time =3D 2000, .channels =3D bmp380_channels, @@ -1720,10 +1728,12 @@ static int bmp580_chip_config(struct bmp280_data *d= ata) } =20 static const int bmp580_oversampling_avail[] =3D { 1, 2, 4, 8, 16, 32, 64,= 128 }; +static const u8 bmp580_chip_ids[] =3D { BMP580_CHIP_ID, BMP580_CHIP_ID_ALT= }; =20 const struct bmp280_chip_info bmp580_chip_info =3D { .id_reg =3D BMP580_REG_CHIP_ID, - .chip_id =3D BMP580_CHIP_ID, + .chip_id =3D bmp580_chip_ids, + .num_chip_id =3D ARRAY_SIZE(bmp580_chip_ids), .regmap_config =3D &bmp580_regmap_config, .start_up_time =3D 2000, .channels =3D bmp380_channels, @@ -1983,10 +1993,12 @@ static int bmp180_chip_config(struct bmp280_data *d= ata) =20 static const int bmp180_oversampling_temp_avail[] =3D { 1 }; static const int bmp180_oversampling_press_avail[] =3D { 1, 2, 4, 8 }; +static const u8 bmp180_chip_ids[] =3D { BMP180_CHIP_ID }; =20 const struct bmp280_chip_info bmp180_chip_info =3D { .id_reg =3D BMP280_REG_ID, - .chip_id =3D BMP180_CHIP_ID, + .chip_id =3D bmp180_chip_ids, + .num_chip_id =3D ARRAY_SIZE(bmp180_chip_ids), .regmap_config =3D &bmp180_regmap_config, .start_up_time =3D 2000, .channels =3D bmp280_channels, @@ -2077,6 +2089,7 @@ int bmp280_common_probe(struct device *dev, struct bmp280_data *data; struct gpio_desc *gpiod; unsigned int chip_id; + unsigned int i; int ret; =20 indio_dev =3D devm_iio_device_alloc(dev, sizeof(*data)); @@ -2142,10 +2155,30 @@ int bmp280_common_probe(struct device *dev, ret =3D regmap_read(regmap, data->chip_info->id_reg, &chip_id); if (ret < 0) return ret; - if (chip_id !=3D data->chip_info->chip_id) { - dev_err(dev, "bad chip id: expected %x got %x\n", - data->chip_info->chip_id, chip_id); - return -EINVAL; + + for (i =3D 0; i < data->chip_info->num_chip_id; i++) { + if (chip_id =3D=3D data->chip_info->chip_id[i]) + break; + } + + if (i =3D=3D data->chip_info->num_chip_id) { + size_t nbuf; + char *buf; + + // 0x, so four chars per number plus one space + ENDL + if (check_mul_overflow(data->chip_info->num_chip_id, 5, &nbuf)) + return ret; + + buf =3D kmalloc_array(data->chip_info->num_chip_id, 5, GFP_KERNEL); + if (!buf) + return ret; + + for (i =3D 0; i < data->chip_info->num_chip_id; i++) + snprintf(&buf[i*5], nbuf - i*5, "0x%x ", data->chip_info->chip_id[i]); + + dev_err(dev, "bad chip id: expected one of [ %s ] got 0x%x\n", buf, chip= _id); + kfree(buf); + return ret; } =20 if (data->chip_info->preinit) { diff --git a/drivers/iio/pressure/bmp280.h b/drivers/iio/pressure/bmp280.h index 5c0563ce7572..a230fcfc4a85 100644 --- a/drivers/iio/pressure/bmp280.h +++ b/drivers/iio/pressure/bmp280.h @@ -418,7 +418,8 @@ struct bmp280_data { =20 struct bmp280_chip_info { unsigned int id_reg; - const unsigned int chip_id; + const u8 *chip_id; + int num_chip_id; =20 const struct regmap_config *regmap_config; =20 --=20 2.42.0