[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20260111120635.46f7216e@jic23-huawei>
Date: Sun, 11 Jan 2026 12:06:35 +0000
From: Jonathan Cameron <jic23@...nel.org>
To: Matti Vaittinen <mazziesaccount@...il.com>
Cc: Maxime Chevallier <maxime.chevallier@...tlin.com>, David Lechner
<dlechner@...libre.com>, nuno.sa@...log.com, Andy Shevchenko
<andy@...nel.org>, Rob Herring <robh@...nel.org>, Krzysztof Kozlowski
<krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>, Marcelo Schmitt
<marcelo.schmitt@...log.com>, Antoniu Miclaus <antoniu.miclaus@...log.com>,
Angelo Dureghello <adureghello@...libre.com>, Tobias Sperling
<tobias.sperling@...ting.com>, Eason Yang <j2anfernee@...il.com>, Marilene
Andrade Garcia <marilene.agarcia@...il.com>, Cosmin Tanislav
<cosmin-gabriel.tanislav.xa@...esas.com>, duje@...emihanovic.xyz,
herve.codina@...tlin.com, Rodolfo Giometti <giometti@...eenne.com>,
linux-iio@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org, thomas.petazzoni@...tlin.com
Subject: Re: [PATCH 2/2] iio: adc: add driver for Texas Instruments TLA2528
adc
On Fri, 2 Jan 2026 09:13:51 +0200
Matti Vaittinen <mazziesaccount@...il.com> wrote:
> On 31/12/2025 19:12, Jonathan Cameron wrote:
> > On Mon, 29 Dec 2025 10:20:23 +0200
> > Matti Vaittinen <mazziesaccount@...il.com> wrote:
> >
> >> On 23/12/2025 17:55, Maxime Chevallier wrote:
> >>> This adds a new driver for the TI TLA2528 ADC chip. It ha 8 12-bit
> >>> channels, that can also be configured as 16-bit averaging channels.
> >>>
> >>> Add a very simple driver for it, allowing reading raw values for each
> >>> channel.
> >>>
> >>> Signed-off-by: Rodolfo Giometti <giometti@...eenne.com>
> >>> Signed-off-by: Maxime Chevallier <maxime.chevallier@...tlin.com>
> >>> ---
> >>> MAINTAINERS | 7 ++
> >>> drivers/iio/adc/Kconfig | 10 ++
> >>> drivers/iio/adc/Makefile | 1 +
> >>> drivers/iio/adc/ti-tla2528.c | 209 +++++++++++++++++++++++++++++++++++
> >>> 4 files changed, 227 insertions(+)
> >>> create mode 100644 drivers/iio/adc/ti-tla2528.c
> >>>
> >>> diff --git a/MAINTAINERS b/MAINTAINERS
> >>> index dc731d37c8fe..5c382ae216c7 100644
> >>> --- a/MAINTAINERS
> >>> +++ b/MAINTAINERS
> >>> @@ -25866,6 +25866,13 @@ F: include/dt-bindings/soc/ti,sci_pm_domain.h
> >>> F: include/linux/soc/ti/ti_sci_inta_msi.h
> >>> F: include/linux/soc/ti/ti_sci_protocol.h
> >>>
> >>> +TEXAS INSTRUMENTS' TLA2528 ADC DRIVER
> >>> +M: Maxime Chevallier <maxime.chevallier@...tlin.com>
> >>> +L: linux-iio@...r.kernel.org
> >>> +S: Supported
> >>> +F: Documentation/devicetree/bindings/iio/adc/ti,tla2528.yaml
> >>> +F: drivers/iio/adc/ti-tla2528.c
> >>> +
> >>> TEXAS INSTRUMENTS' TMP117 TEMPERATURE SENSOR DRIVER
> >>> M: Puranjay Mohan <puranjay@...nel.org>
> >>> L: linux-iio@...r.kernel.org
> >>> diff --git a/drivers/iio/adc/Kconfig b/drivers/iio/adc/Kconfig
> >>> index 58da8255525e..67376de410bf 100644
> >>
> >> Hmm. Would it ease merging if MAINTAINERS changes were in their own patch?
> >
> > Not particularly. Though I personally slightly prefer the logic
> > of bringing the entry in with the first file, then adding additional files
> > in later patches.
> >
> > Given it is huge and in alphabetical order, conflicts in MAINTAINERS are
> > fairly rare and trivial to resolve.
> >
>
> Thanks for this clarification :)
>
> I don't know where I had picked up this idea, but I thought that the
> volume of changes in MAINTAINERs was somewhat annoying source of
> conflicts. I sit and type corrected :)
Low chance for any given patch, but high chance overall as so many patches
touch it! Resolving it isn't really any easier if it is inside a bigger
patch or not.
Jonathan
>
>
> Yours,
> -- Matti
>
> ---
> Matti Vaittinen
> Linux kernel developer at ROHM Semiconductors
> Oulu Finland
>
> ~~ When things go utterly wrong vim users can always type :help! ~~
Powered by blists - more mailing lists