Re: [PATCH v2 0/2] Fix phase range check in AD9832 and AD9834 drivers

From: Jonathan Cameron
Date: Sat Nov 09 2024 - 09:40:02 EST


On Thu, 7 Nov 2024 01:10:13 +0000
Zicheng Qu <quzicheng@xxxxxxxxxx> wrote:

> Hi,
>
> The previous link is: https://lore.kernel.org/linux-iio/20241105140359.2465656-1-quzicheng@xxxxxxxxxx/T/#u
>
> This patch (v2) addresses an issue in both the AD9832 and AD9834 drivers
> where the phase range check allows an invalid value of 4096. The
> condition has been corrected to ensure only valid 12-bit values (0-4095)
> are accepted.
>
> These changes ensure compliance with datasheet specifications and
> prevent potential issues with incorrect phase settings.
>
> Zicheng Qu (2):
> staging: iio: ad9834: Correct phase range check
> staging: iio: ad9832: Correct phase range check
>
> drivers/staging/iio/frequency/ad9832.c | 2 +-
> drivers/staging/iio/frequency/ad9834.c | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> Thank you for reviewing this patch.
>
For future reference, for IIO at least (and most of the kernel) don't
reply to previous version when posting a new version. That tends to
make it likely your patch set will be lost way up the pile of email
in a maintainers in box.

Many maintainers start at latest and work backwards until they run out
of time :)

Jonathan

> Best regards,
>
> Zicheng