linux-kernel - Re: [PATCH] iio: adc: Add TI ADS1015 ADC driver support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Thu, 14 Jan 2016 09:34:34 -0800
From:	Guenter Roeck <linux@...ck-us.net>
To:	Michael Welling <mwelling@...e.org>,
	Daniel Baluta <daniel.baluta@...el.com>
Cc:	jic23@...nel.org, knaack.h@....de, lars@...afoo.de,
	pmeerw@...erw.net, linux-kernel@...r.kernel.org,
	linux-iio@...r.kernel.org, lucas.demarchi@...el.com,
	eibach@...ys.de
Subject: Re: [PATCH] iio: adc: Add TI ADS1015 ADC driver support

On 01/14/2016 08:51 AM, Michael Welling wrote:
> On Thu, Jan 14, 2016 at 06:25:08PM +0200, Daniel Baluta wrote:
>> The driver has sysfs readings with runtime PM support for power saving.
>> It also offers buffer support that can be used together with IIO software
>> triggers.
>>
>
> It should be noted that the hwmon driver subsystem has support for this device.
>
> http://lxr.free-electrons.com/source/drivers/hwmon/ads1015.c
>
> The driver could likely be removed if this is accepted into iio.
> The hwmon subsystem has an iio wrapper driver.
>
> http://lxr.free-electrons.com/source/drivers/hwmon/iio_hwmon.c
>
> The only hurdle is adding support for the ads1115 before removing the other driver.
>
Yes, plus one would hope that there are not going to be a different set of
devicetree properties associated with the new driver, and that the new driver
will support (at least) the same properties.

You'll also have to make sure that only one of the two drivers can be selected.

Guenter