[PATCH v5 0/3] iio: frequency: ad9523: fix checkpatch warnings

Bhargav Joshi posted 3 patches 1 month, 2 weeks ago
Only 2 patches received!
There is a newer version of this series
drivers/iio/frequency/ad9523.c | 88 +++++++++++++---------------------
1 file changed, 33 insertions(+), 55 deletions(-)
[PATCH v5 0/3] iio: frequency: ad9523: fix checkpatch warnings
Posted by Bhargav Joshi 1 month, 2 weeks ago
These patches address several checkpatch warnings in the ad9523 driver.

Patch 1: Updated the macros to properly use their argument x.
Patch 2: Fixed the multi-line pointer dereferences.
Patch 3: Updated symbolic permissions to octal (0444/0200).

Changes in v5:
- Reflowed all commit messages to properly wrap near 72 characters as
requested.
- Collected Reviewed-by tags for patches 1 and 3.

Changes in v4:
- Used full name for SoB 

Changes in v3:
- Patch 1: updated macros to use '(x)' instead of 'x'.
- Patch 2: collected reviewed-by (no code changes).
- Patch 3: fixed vertical spacing and broken indentation.

bhargav (3):
  iio: frequency: ad9523: fix implicit variable usage in macros
  iio: frequency: ad9523: avoid multiple line dereferences for pdata
  iio: frequency: ad9523: fix checkpatch warnings for symbolic
    permissions

 drivers/iio/frequency/ad9523.c | 88 +++++++++++++---------------------
 1 file changed, 33 insertions(+), 55 deletions(-)

-- 
2.53.0
Re: [PATCH v5 0/3] iio: frequency: ad9523: fix checkpatch warnings
Posted by Andy Shevchenko 1 month, 2 weeks ago
On Wed, Feb 25, 2026 at 9:41 PM Bhargav Joshi <rougueprince47@gmail.com> wrote:
>
> These patches address several checkpatch warnings in the ad9523 driver.
>
> Patch 1: Updated the macros to properly use their argument x.
> Patch 2: Fixed the multi-line pointer dereferences.
> Patch 3: Updated symbolic permissions to octal (0444/0200).

Thanks for the update, it still needs more amendments (something even
went wrong in patch 3).

-- 
With Best Regards,
Andy Shevchenko