From: Jonathan Cameron <Jonathan.Cameron@huawei.com> To: Peter Rosin <peda@axentia.se> Cc: Jonathan Cameron <jic23@kernel.org>, <linux-kernel@vger.kernel.org>, Hartmut Knaack <knaack.h@gmx.de>, Lars-Peter Clausen <lars@metafoo.de>, "Peter Meerwald-Stadler" <pmeerw@pmeerw.net>, Rob Herring <robh+dt@kernel.org>, "Mark Rutland" <mark.rutland@arm.com>, "David S. Miller" <davem@davemloft.net>, Mauro Carvalho Chehab <mchehab@kernel.org>, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, Linus Walleij <linus.walleij@linaro.org>, Andrew Morton <akpm@linux-foundation.org>, Randy Dunlap <rdunlap@infradead.org>, "Andrew F . Davis" <afd@ti.com>, Fabio Estevam <festevam@gmail.com>, <linux-iio@vger.kernel.org>, <devicetree@vger.kernel.org> Subject: Re: [PATCH v3 2/2] iio: afe: unit-converter: new driver Date: Wed, 18 Apr 2018 10:37:21 +0100 [thread overview] Message-ID: <20180418103721.00004171@huawei.com> (raw) In-Reply-To: <81e69701-d49b-6675-010d-99d5d3eaf827@axentia.se> On Mon, 16 Apr 2018 09:12:45 +0200 Peter Rosin <peda@axentia.se> wrote: > On 2018-04-15 19:31, Jonathan Cameron wrote: > > On Tue, 10 Apr 2018 17:28:02 +0200 > > Peter Rosin <peda@axentia.se> wrote: > > > >> If an ADC channel measures the midpoint of a voltage divider, the > >> interesting voltage is often the voltage over the full resistance. > >> E.g. if the full voltage is too big for the ADC to handle. > >> Likewise, if an ADC channel measures the voltage across a shunt > >> resistor, the interesting value is often the current through the > >> resistor. > >> > >> This driver solves both problems by allowing to linearly scale a channel > >> and by allowing changes to the type of the channel. Or both. > >> > >> Signed-off-by: Peter Rosin <peda@axentia.se> > > So I 'think' the only outstanding question is Andrew's one about the driver > > name. We aren't in a hurry at this point in the kernel cycle, so lets > > wait until that discussion has ended. Assuming that we do possibly end > > up with a change, then please roll all the patches up into a single series > > to avoid me getting confusion. > > Yeah, sure, sorry for the split series, but the lt6106 that's present in > one of our newer designs didn't occur to me until just seconds after > firing the first half of the series. Which is kind of typical... > > Anyway, about the driver naming. The suggestion I like best so far is > linear-scaler from Linus W, but thinking about it some more I think I > like iio-rescale even better. > > Any objections to iio-rescale? Works for me. But then I rarely care 'that much' about naming and am responsible for plenty of previous confusing choices ;) Jonathan > > Cheers, > Peter > -- > To unsubscribe from this list: send the line "unsubscribe linux-iio" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html
WARNING: multiple messages have this Message-ID (diff)
From: Jonathan Cameron <Jonathan.Cameron@huawei.com> To: Peter Rosin <peda@axentia.se> Cc: Jonathan Cameron <jic23@kernel.org>, linux-kernel@vger.kernel.org, Hartmut Knaack <knaack.h@gmx.de>, Lars-Peter Clausen <lars@metafoo.de>, Peter Meerwald-Stadler <pmeerw@pmeerw.net>, Rob Herring <robh+dt@kernel.org>, Mark Rutland <mark.rutland@arm.com>, "David S. Miller" <davem@davemloft.net>, Mauro Carvalho Chehab <mchehab@kernel.org>, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, Linus Walleij <linus.walleij@linaro.org>, Andrew Morton <akpm@linux-foundation.org>, Randy Dunlap <rdunlap@infradead.org>, "Andrew F . Davis" <afd@ti.com>, Fabio Estevam <festevam@gmail.com>, linux-iio@vger.kernel.org, devicetree@vger.kernel.org Subject: Re: [PATCH v3 2/2] iio: afe: unit-converter: new driver Date: Wed, 18 Apr 2018 10:37:21 +0100 [thread overview] Message-ID: <20180418103721.00004171@huawei.com> (raw) In-Reply-To: <81e69701-d49b-6675-010d-99d5d3eaf827@axentia.se> On Mon, 16 Apr 2018 09:12:45 +0200 Peter Rosin <peda@axentia.se> wrote: > On 2018-04-15 19:31, Jonathan Cameron wrote: > > On Tue, 10 Apr 2018 17:28:02 +0200 > > Peter Rosin <peda@axentia.se> wrote: > > > >> If an ADC channel measures the midpoint of a voltage divider, the > >> interesting voltage is often the voltage over the full resistance. > >> E.g. if the full voltage is too big for the ADC to handle. > >> Likewise, if an ADC channel measures the voltage across a shunt > >> resistor, the interesting value is often the current through the > >> resistor. > >> > >> This driver solves both problems by allowing to linearly scale a channel > >> and by allowing changes to the type of the channel. Or both. > >> > >> Signed-off-by: Peter Rosin <peda@axentia.se> > > So I 'think' the only outstanding question is Andrew's one about the driver > > name. We aren't in a hurry at this point in the kernel cycle, so lets > > wait until that discussion has ended. Assuming that we do possibly end > > up with a change, then please roll all the patches up into a single series > > to avoid me getting confusion. > > Yeah, sure, sorry for the split series, but the lt6106 that's present in > one of our newer designs didn't occur to me until just seconds after > firing the first half of the series. Which is kind of typical... > > Anyway, about the driver naming. The suggestion I like best so far is > linear-scaler from Linus W, but thinking about it some more I think I > like iio-rescale even better. > > Any objections to iio-rescale? Works for me. But then I rarely care 'that much' about naming and am responsible for plenty of previous confusing choices ;) Jonathan > > Cheers, > Peter > -- > To unsubscribe from this list: send the line "unsubscribe linux-iio" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2018-04-18 9:37 UTC|newest] Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top 2018-04-10 15:28 [PATCH v3 0/2] iio: add unit converter Peter Rosin 2018-04-10 15:28 ` [PATCH v3 1/2] dt-bindings: iio: afe: add current-sense-shunt and voltage-divider Peter Rosin 2018-04-13 21:42 ` Rob Herring 2018-04-16 14:00 ` Peter Rosin 2018-04-21 14:34 ` Jonathan Cameron 2018-04-10 15:28 ` [PATCH v3 2/2] iio: afe: unit-converter: new driver Peter Rosin 2018-04-15 17:31 ` Jonathan Cameron 2018-04-16 7:12 ` Peter Rosin 2018-04-18 9:37 ` Jonathan Cameron [this message] 2018-04-18 9:37 ` Jonathan Cameron 2018-04-11 14:15 ` [PATCH v3 0/2] iio: add unit converter Peter Rosin 2018-04-11 14:15 ` [PATCH 1/2] dt-bindings: iio: afe: add binding for adi,lt6106 Peter Rosin 2018-04-16 18:44 ` Rob Herring 2018-04-11 14:15 ` [PATCH 2/2] iio: afe: unit-converter: add support " Peter Rosin 2018-04-11 15:43 ` Andrew F. Davis 2018-04-11 15:43 ` Andrew F. Davis 2018-04-11 15:51 ` Lars-Peter Clausen 2018-04-11 16:13 ` Andrew F. Davis 2018-04-11 16:13 ` Andrew F. Davis 2018-04-12 14:29 ` Peter Rosin 2018-04-12 15:35 ` Andrew F. Davis 2018-04-12 15:35 ` Andrew F. Davis 2018-04-12 22:31 ` Peter Rosin 2018-04-13 8:11 ` Lars-Peter Clausen 2018-04-16 7:29 ` Peter Rosin 2018-04-13 14:47 ` Andrew F. Davis 2018-04-13 14:47 ` Andrew F. Davis 2018-04-16 7:17 ` Peter Rosin 2018-04-12 14:04 ` Peter Rosin 2018-04-11 15:34 ` [PATCH v3 0/2] iio: add unit converter Andrew F. Davis 2018-04-11 15:34 ` Andrew F. Davis 2018-04-15 17:41 ` Jonathan Cameron
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20180418103721.00004171@huawei.com \ --to=jonathan.cameron@huawei.com \ --cc=afd@ti.com \ --cc=akpm@linux-foundation.org \ --cc=davem@davemloft.net \ --cc=devicetree@vger.kernel.org \ --cc=festevam@gmail.com \ --cc=gregkh@linuxfoundation.org \ --cc=jic23@kernel.org \ --cc=knaack.h@gmx.de \ --cc=lars@metafoo.de \ --cc=linus.walleij@linaro.org \ --cc=linux-iio@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=mark.rutland@arm.com \ --cc=mchehab@kernel.org \ --cc=peda@axentia.se \ --cc=pmeerw@pmeerw.net \ --cc=rdunlap@infradead.org \ --cc=robh+dt@kernel.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.