[PATCH v3 0/4] staging: iio: adc: ad7816: code cleanups and modernization

Md. Mahmudul Hasan Mabud posted 4 patches 1 day, 13 hours ago
There is a newer version of this series
drivers/staging/iio/adc/ad7816.c | 111 +++++++++++++++----------------
1 file changed, 53 insertions(+), 58 deletions(-)
[PATCH v3 0/4] staging: iio: adc: ad7816: code cleanups and modernization
Posted by Md. Mahmudul Hasan Mabud 1 day, 13 hours ago
Changes in v3:
- Added missing Signed-off-by tags and detailed patch descriptions to
  all commits as requested by the automated patch-bot.
- No functional changes since v2.

Changes in v2:
- Renamed sysfs callback functions (e.g., ad7816_show_mode to mode_show)
  to satisfy the requirements of modern IIO attribute macros.
- Fixed usage of sysfs_match_string() by removing the unnecessary 
  ARRAY_SIZE parameter.
- Fixed vertical alignment of function parameters to match open parenthesis.
- Replaced sprintf() with sysfs_emit() across all show attributes.

Md. Mahmudul Hasan Mabud (4):
  staging: iio: adc: ad7816: redefine mode constants to start from 0
  staging: iio: adc: ad7816: use sysfs_match_string() for mode selection
  staging: iio: adc: ad7816: replace sprintf() with sysfs_emit()
  staging: iio: adc: ad7816: use modern IIO attribute macros

 drivers/staging/iio/adc/ad7816.c | 111 +++++++++++++++----------------
 1 file changed, 53 insertions(+), 58 deletions(-)

-- 
2.34.1
Re: [PATCH v3 0/4] staging: iio: adc: ad7816: code cleanups and modernization
Posted by Andy Shevchenko 1 day ago
On Tue, Mar 31, 2026 at 12:17:06AM +0600, Md. Mahmudul Hasan Mabud wrote:

Please, also put a small description of what is done in the series as a whole.
Nevertheless, this version is much better now, but some small tweaks are
required. I believe the next one will be good enough.

> Changes in v3:
> - Added missing Signed-off-by tags and detailed patch descriptions to
>   all commits as requested by the automated patch-bot.
> - No functional changes since v2.

-- 
With Best Regards,
Andy Shevchenko