From: Eason Yang <j2anfernee@gmail.com>
To: jic23@kernel.org, lars@metafoo.de, robh@kernel.org,
krzk+dt@kernel.org, conor+dt@kernel.org, dlechner@baylibre.com,
nuno.sa@analog.com, javier.carrasco.cruz@gmail.com,
tgamblin@baylibre.com, olivier.moysan@foss.st.com,
alisadariana@gmail.com, gstols@baylibre.com,
antoniu.miclaus@analog.com, eblanc@baylibre.com,
andriy.shevchenko@linux.intel.com, matteomartelli3@gmail.com,
marcelo.schmitt@analog.com, chanh@os.amperecomputing.com,
KWLIU@nuvoton.com, yhyang2@nuvoton.com
Cc: linux-iio@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, Eason Yang <j2anfernee@gmail.com>
Subject: [PATCH v8 0/2] iio: adc: add Nuvoton NCT7201 ADC driver
Date: Tue, 29 Apr 2025 10:55:03 +0800 [thread overview]
Message-ID: <20250429025505.3278016-1-j2anfernee@gmail.com> (raw)
Change since version 8:
- dev_err_probe(), move the definition from device.h to dev_printk.h
- Use USEC_PER_MSEC rather than the hard coded value of 1000
- Use one dev for both regmap and regmap16
Change since version 7:
- Fix comments
- Derive dev from the respective regmap
- Generate the mask from the number of voltage input channels
Change since version 6:
- Fix comments
- Add use_single_read in regmap_config
- Remove unused definitions
- Do not shadow the return code by -EIO and let regmap API caller to decide
- Use simple English in all error messages
- Use a local variable for the struct device pointers. This increases
code readability with shortened lines.
- Use `fsleep` instead of `mdelay`
- Use 16 bits type __le16 instead of u8 data[2]
Change since version 5:
- Fix comments
- Add NUVOTON NCT7201 IIO DRIVER section in MAINTAINERS
- Add vdd-supply and vref-supply to the DT example
- Remove mutex since the regmap should already have an internal lock
- Remove redundant assigning values
- Check errors on regmap_write
Change since version 4:
- Fix comments
- Add interrupts and reset-gpios to the DT example
- Use the FIELD_PREP and FIELD_GET
- Add use_single_write in regmap_config
- Use regmap_access_table
Change since version 3:
- Fix comments
- Don't put nct720"x" in the name, just call it nct7201
- Remove differential inputs until conversions are finished
- Add NCT7201_ prefix in all macros and avoid the tables
- Correct event threshold values in raw units
- Add with and without interrupt callback function to have the event
config part and one that doesn't
- Remove print an error message if regmap_wirte failed case
Change since version 2:
- Remvoe read-vin-data-size property, default use read word vin data
- Use regmap instead of i2c smbus API
- IIO should be IIO_CHAN_INFO_RAW and _SCALE not _PROCESSED
- Use dev_xxx_probe in probe function and dev_xxx in other functions
- Use devm_iio_device_register replace of iio_device_register
- Use guard(mutex) replace of mutex_lock
- Use get_unaligned_le16 conversion API
Changes since version 1:
- Add new property in iio:adc binding document
- Add new driver for Nuvoton NCT720x driver
Eason Yang (2):
dt-bindings: iio: adc: add NCT7201 ADCs
iio: adc: add support for Nuvoton NCT7201
.../bindings/iio/adc/nuvoton,nct7201.yaml | 70 +++
MAINTAINERS | 7 +
drivers/iio/adc/Kconfig | 11 +
drivers/iio/adc/Makefile | 1 +
drivers/iio/adc/nct7201.c | 462 ++++++++++++++++++
5 files changed, 551 insertions(+)
create mode 100644 Documentation/devicetree/bindings/iio/adc/nuvoton,nct7201.yaml
create mode 100644 drivers/iio/adc/nct7201.c
--
2.34.1
next reply other threads:[~2025-04-29 2:55 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-04-29 2:55 Eason Yang [this message]
2025-04-29 2:55 ` [PATCH v8 1/2] dt-bindings: iio: adc: add NCT7201 ADCs Eason Yang
2025-04-29 7:36 ` Krzysztof Kozlowski
2025-04-29 16:09 ` David Lechner
2025-04-29 17:43 ` Krzysztof Kozlowski
2025-04-29 17:50 ` David Lechner
2025-04-29 2:55 ` [PATCH v8 2/2] iio: adc: add support for Nuvoton NCT7201 Eason Yang
2025-04-29 16:32 ` David Lechner
2025-05-02 9:23 ` Andy Shevchenko
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=20250429025505.3278016-1-j2anfernee@gmail.com \
--to=j2anfernee@gmail.com \
--cc=KWLIU@nuvoton.com \
--cc=alisadariana@gmail.com \
--cc=andriy.shevchenko@linux.intel.com \
--cc=antoniu.miclaus@analog.com \
--cc=chanh@os.amperecomputing.com \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=dlechner@baylibre.com \
--cc=eblanc@baylibre.com \
--cc=gstols@baylibre.com \
--cc=javier.carrasco.cruz@gmail.com \
--cc=jic23@kernel.org \
--cc=krzk+dt@kernel.org \
--cc=lars@metafoo.de \
--cc=linux-iio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=marcelo.schmitt@analog.com \
--cc=matteomartelli3@gmail.com \
--cc=nuno.sa@analog.com \
--cc=olivier.moysan@foss.st.com \
--cc=robh@kernel.org \
--cc=tgamblin@baylibre.com \
--cc=yhyang2@nuvoton.com \
/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: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox