| Message ID | 20210712212111.18896-1-andreas@kemnade.info (mailing list archive) |
|---|---|
| Headers | show |
| Series | mfd: rn5t618: Extend ADC support | expand |
On Mon, 12 Jul 2021, Andreas Kemnade wrote: > Add iio map to make voltage_now related channels accessible to power > driver. > > Changes in v3: > - use scale functions > - add acks > > Changes in v2: > - use iio_map instead of devicetree to allow mapping which does not > block future extension by devicetree. > > > *** BLURB HERE *** Doh! > Andreas Kemnade (2): > iio: adc: rn5t618: Add iio map > power: supply: rn5t618: Add voltage_now property > > drivers/iio/adc/rn5t618-adc.c | 23 +++++++++++++++++ > drivers/power/supply/Kconfig | 2 ++ > drivers/power/supply/rn5t618_power.c | 38 ++++++++++++++++++++++++++++ > 3 files changed, 63 insertions(+) Not sure I get this. Firstly, the cover-letter is marked as MFD, but no MFD changes occur. Secondly, I am only in receipt of the 0th patch which seems odd. IMHO patch sets should be sent threaded and all parties should see discussion on all patches regardless of whether they maintain them or not, since the overall conversation is often useful/relevant to all.
On Tue, 13 Jul 2021 09:27:27 +0100 Lee Jones <lee.jones@linaro.org> wrote: > On Mon, 12 Jul 2021, Andreas Kemnade wrote: > > > Add iio map to make voltage_now related channels accessible to power > > driver. > > > > Changes in v3: > > - use scale functions > > - add acks > > > > Changes in v2: > > - use iio_map instead of devicetree to allow mapping which does not > > block future extension by devicetree. > > > > > > *** BLURB HERE *** > > Doh! > > > Andreas Kemnade (2): > > iio: adc: rn5t618: Add iio map > > power: supply: rn5t618: Add voltage_now property > > > > drivers/iio/adc/rn5t618-adc.c | 23 +++++++++++++++++ > > drivers/power/supply/Kconfig | 2 ++ > > drivers/power/supply/rn5t618_power.c | 38 ++++++++++++++++++++++++++++ > > 3 files changed, 63 insertions(+) > > Not sure I get this. > > Firstly, the cover-letter is marked as MFD, but no MFD changes occur. So this is a bit of a fun corner case. The series 'used' to include an mfd change that made that labelling relevant. Then that went away as I suggested that we could do it in a simpler fashion. Under the circumstances the series needed a rename! > Secondly, I am only in receipt of the 0th patch which seems odd. IMHO > patch sets should be sent threaded and all parties should see > discussion on all patches regardless of whether they maintain them or > not, since the overall conversation is often useful/relevant to all. > Absolutely agree for small series like this. Anyhow, as far as I'm concerned this is now in the hands of the power supply maintainers anyway so ignoring it ;) Jonathan
Hi, On Mon, Jul 12, 2021 at 11:21:09PM +0200, Andreas Kemnade wrote: > Add iio map to make voltage_now related channels accessible to power > driver. > > Changes in v3: > - use scale functions > - add acks > > Changes in v2: > - use iio_map instead of devicetree to allow mapping which does not > block future extension by devicetree. Thanks, queued. -- Sebastian > *** BLURB HERE *** :) > Andreas Kemnade (2): > iio: adc: rn5t618: Add iio map > power: supply: rn5t618: Add voltage_now property > > drivers/iio/adc/rn5t618-adc.c | 23 +++++++++++++++++ > drivers/power/supply/Kconfig | 2 ++ > drivers/power/supply/rn5t618_power.c | 38 ++++++++++++++++++++++++++++ > 3 files changed, 63 insertions(+) > > -- > 2.30.2 >