Messages in this thread | | | Subject | Re: [PATCH v2 2/4] iio: Documentation: Add Xilinx AMS sysfs documentation | From | Michal Simek <> | Date | Mon, 17 Sep 2018 11:56:08 +0200 |
| |
On 16.9.2018 12:12, Jonathan Cameron wrote: > On Fri, 14 Sep 2018 12:48:28 +0530 > Manish Narani <manish.narani@xilinx.com> wrote: > >> Add documentation for xilinx-ams driver. This contains information about >> various voltages and temperatures on PS (Processing System), PL >> (Programmable Logic) and AMS Control Block. >> >> Signed-off-by: Manish Narani <manish.narani@xilinx.com> > The more I look at this device the more I'm convinced it is very much a dedicated > hardware monitoring function, not a generic ADC sensing unit at all. > > Hmm. It is still fine to have it in IIO but you need to think in detail > about how you are going to interface this to hwmon via the iio-hwmon bridge. > > Some of the interface complexity should only really be apparent when we hit > hwmon perhaps rather than having so many different custom interfaces in IIO. > > Please also loop in the maintainers and lists for hwmon in the next > version so we can get their input.
Isn't there iio-hwmon driver for this?
Thanks, Michal
| |