[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20241109143945.2616c5d5@jic23-huawei>
Date: Sat, 9 Nov 2024 14:39:45 +0000
From: Jonathan Cameron <jic23@...nel.org>
To: Zicheng Qu <quzicheng@...wei.com>
Cc: <lars@...afoo.de>, <Michael.Hennerich@...log.com>,
<gregkh@...uxfoundation.org>, <daniel.baluta@...el.com>,
<shubhrajyoti@...com>, <linux-iio@...r.kernel.org>,
<linux-staging@...ts.linux.dev>, <linux-kernel@...r.kernel.org>,
<tanghui20@...wei.com>, <zhangqiao22@...wei.com>, <judy.chenhui@...wei.com>
Subject: Re: [PATCH v2 0/2] Fix phase range check in AD9832 and AD9834
drivers
On Thu, 7 Nov 2024 01:10:13 +0000
Zicheng Qu <quzicheng@...wei.com> wrote:
> Hi,
>
> The previous link is: https://lore.kernel.org/linux-iio/20241105140359.2465656-1-quzicheng@huawei.com/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
Powered by blists - more mailing lists