[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <cover.1757971454.git.marilene.agarcia@gmail.com>
Date: Mon, 15 Sep 2025 19:14:39 -0300
From: Marilene Andrade Garcia <marilene.agarcia@...il.com>
To: linux-iio@...r.kernel.org,
linux-kernel@...r.kernel.org,
devicetree@...r.kernel.org
Cc: Marilene Andrade Garcia <marilene.agarcia@...il.com>,
Kim Seer Paller <kimseer.paller@...log.com>,
Lars-Peter Clausen <lars@...afoo.de>,
Michael Hennerich <Michael.Hennerich@...log.com>,
Jonathan Cameron <jic23@...nel.org>,
David Lechner <dlechner@...libre.com>,
Nuno Sá <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.schmitt1@...il.com>,
Marcelo Schmitt <Marcelo.Schmitt@...log.com>,
Ceclan Dumitru <dumitru.ceclan@...log.com>,
Jonathan Santos <Jonathan.Santos@...log.com>,
Dragos Bogdan <dragos.bogdan@...log.com>
Subject: [PATCH v11 0/3] Add MAX14001/MAX14002 support
Hello maintainers,
Thank you for reviewing v10, for your suggestions, and for answering
my questions.
I believe I’ve addressed most of the requested code changes. There was only
one that I haven’t fixed, I’ve included the reasons in the patch message.
I’d also like to use this cover letter to address some of the remaining
questions from previous cover letters.
Regarding regmap: thank you, David, for your response. I’ve implemented it,
and the code seems to be working fine. I tested it on the Raspberry Pi
modified kernel version rpi-6.12 with Raspberry Pi 5 hardware, using the
MAX14001PMB evaluation board.
As for in_Y_mean_raw, the issue is that I don’t have the file
/sys/bus/iio/devices/iio:device0/in_0_mean_raw; instead, I have
/sys/bus/iio/devices/iio:device0/in_voltage0_mean_raw. I was thinking of
adding in_voltageY_mean_raw to the documentation, so I am submitting a
patch with this change in the current patch set.
Thank you also for the explanations about the extra analog frontend
circuitry. I plan to study this further and send a dedicated patch to cover
it in the future.
Thank you Jonathan for the two possible solutions to set the number of ADC
readings used in the mean calculation. I’ll study both approaches and send
a dedicated patch to implement one of them in the next steps.
I intend to continue sending patches to implement all the features of the
MAX14001/MAX14002. Since I mostly work on weekends, I’ll be submitting
patches at a low frequency, but consistently.
Thank you for your time.
Best regards,
Marilene Andrade Garcia
Marilene Andrade Garcia (3):
dt-bindings: iio: adc: add max14001
iio: adc: max14001: New driver
iio: ABI: Add voltage mean raw attribute
Documentation/ABI/testing/sysfs-bus-iio | 1 +
.../bindings/iio/adc/adi,max14001.yaml | 87 +++++
MAINTAINERS | 9 +
drivers/iio/adc/Kconfig | 10 +
drivers/iio/adc/Makefile | 1 +
drivers/iio/adc/max14001.c | 356 ++++++++++++++++++
6 files changed, 464 insertions(+)
create mode 100644 Documentation/devicetree/bindings/iio/adc/adi,max14001.yaml
create mode 100644 drivers/iio/adc/max14001.c
base-commit: 671b9b6d7f4fe17a174c410397e72253877ca64e
--
2.34.1
Powered by blists - more mailing lists