Hi,
> Am 26.06.2016 um 11:52 schrieb Jonathan Cameron <jic23@xxxxxxxxxx>:
>
> On 22/06/16 21:53, Linus Walleij wrote:
>> This adds device tree support to the BMP085, BMP180 and BMP280
>> pressure sensors. Tested on the Qualcomm APQ8060 Dragonboard:
>>
>> iio:device1$ cat in_temp_input
>> 26700
>> iio:device1$ cat in_pressure_input
>> 99.185000000
>>
> Please also confirm the value of 'name' given the mess we have had
> over that recently.
Here is what I get on a bmp085 equipped device:
root@letux:/sys/bus/iio/devices/iio:device0# for i in *
> do
> echo $i $(cat $i)
> done
dev 252:0
in_pressure_input 94.778000000
in_pressure_oversampling_ratio 8
in_pressure_oversampling_ratio_available 1 2 4 8
in_temp_input 33200
in_temp_oversampling_ratio 1
in_temp_oversampling_ratio_available 1
name bmp085
of_node
power
subsystem
uevent MAJOR=252 MINOR=0 DEVNAME=iio:device0 DEVTYPE=iio_device OF_NAME=bmp085 OF_FULLNAME=/ocp/i2c@48072000/bmp085@77 OF_COMPATIBLE_0=bosch,bmp085 OF_COMPATIBLE_N=1
root@letux:/sys/bus/iio/devices/iio:device0#
So "name" is the name of the chip variant.
>
>
>> Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx>
>> ---
>> ChangeLog v1->v2:
>> - Fix some BMP085 misspelled as BMP058 errors.
>> - Add the new BME280 sensor as well
>> ---
>> drivers/iio/pressure/bmp280.c | 14 ++++++++++++++
>> 1 file changed, 14 insertions(+)
>>
>> diff --git a/drivers/iio/pressure/bmp280.c b/drivers/iio/pressure/bmp280.c
>> index dbbcd6d83e3b..655e888ef8f9 100644
>> --- a/drivers/iio/pressure/bmp280.c
>> +++ b/drivers/iio/pressure/bmp280.c
>> @@ -18,6 +18,7 @@
>> #include <linux/module.h>
>> #include <linux/i2c.h>
>> #include <linux/acpi.h>
>> +#include <linux/of.h>
>> #include <linux/regmap.h>
>> #include <linux/delay.h>
>> #include <linux/iio/iio.h>
>> @@ -1094,6 +1095,18 @@ static const struct acpi_device_id bmp280_acpi_match[] = {
>> };
>> MODULE_DEVICE_TABLE(acpi, bmp280_acpi_match);
>>
>> +#ifdef CONFIG_OF
>> +static const struct of_device_id bmp280_of_match[] = {
>> + { .compatible = "bosch,bme280", .data = (void *)BME280_CHIP_ID },
>> + { .compatible = "bosch,bmp280", .data = (void *)BMP280_CHIP_ID },
>> + { .compatible = "bosch,bmp180", .data = (void *)BMP180_CHIP_ID },
>> + { .compatible = "bosch,bmp085", .data = (void *)BMP180_CHIP_ID },
>> +};
>> +MODULE_DEVICE_TABLE(of, bmp280_of_match);
>> +#else
>> +#define bmp280_of_match NULL
>> +#endif
>> +
>> static const struct i2c_device_id bmp280_id[] = {
>> {"bmp280", BMP280_CHIP_ID },
>> {"bmp180", BMP180_CHIP_ID },
>> @@ -1107,6 +1120,7 @@ static struct i2c_driver bmp280_driver = {
>> .driver = {
>> .name = "bmp280",
>> .acpi_match_table = ACPI_PTR(bmp280_acpi_match),
>> + .of_match_table = of_match_ptr(bmp280_of_match),
>> },
>> .probe = bmp280_probe,
>> .id_table = bmp280_id,
>>
>
BR,
Nikolaus--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
