On Wed, 9 Dec 2020 11:48:40 -0800 Guenter Roeck <linux@xxxxxxxxxxxx> wrote: > On 12/9/20 12:11 AM, Alexandru Ardelean wrote: > > On Tue, Dec 8, 2020 at 6:10 PM Puranjay Mohan <puranjay12@xxxxxxxxx> wrote: > >> > >> I have this TI's TMP117 sensor with me and I was thinking about writing an > >> IIO driver for it as a hobby project. Is the IIO subsystem the correct > >> place for this driver? if yes, can someone help me get started with this, > >> I haven't written an IIO driver before. I have this sensor and also a > >> raspberry pi with me for testing. > > > > This could also fit into drivers/hwmon. > > Looking at the HWMON subsystem there are more TMP drivers there > > (TMP102/103/108/401/513). > > The first 3 seem a bit more similar to TMP117 (in terms of register map). > > > > It would probably be better suited for hwmon (it has limit registers, > suggesting a common use as hardware monitoring device). It is a curious part. I suspect TI based their design for a medical grade digital thermometer chip on an existing hwmon part. The limit registers are very simple so could be supported by IIO. This sits somewhere in the middle of high end thermocouple chips which tend to be in IIO and typically lower accuracy / range hwmon parts. It's in the fuzzy borderline region so I doubt anyone would raise strong objections to which subsystem it was in. Guenter has fallen on the hwmon side of things and I'm fine with that. Jonathan > > > Let's see what others have to add. > > But, all-in-all whatever driver you end up writing, the easiest method > > is to copy an existing similar driver and extend it. > > Sometimes, a part can be added to an existing driver. > > At a quick scan through existing drivers, it doesn't look like TMP117 > > is similar to existing drivers, so it may require a new driver > > altogether. > > I don't see an immediate match either, but the tmp102 hwmon driver > might be a good start. > > Guenter > > > I may have missed something though. > > > > Thanks > > Alex > > > >> > >> -- > >> Thanks and Regards > >> > >> Yours Truly, > >> > >> Puranjay Mohan >
