* [PATCH net-next] dpll: zl3073x: add chip-specific minimum input reference frequency
@ 2026-08-14 15:32 Ivan Vecera
2026-08-15 22:31 ` Vadim Fedorenko
2026-08-16 11:04 ` Vincent Jardin
0 siblings, 2 replies; 3+ messages in thread
From: Ivan Vecera @ 2026-08-14 15:32 UTC (permalink / raw)
To: netdev
Cc: Vincent Jardin, Min Li, Vadim Fedorenko, Arkadiusz Kubalewski,
Jiri Pirko, open list
Commit 24e4aff8983fe6 ("dpll: zl3073x: recognize the ZL30643 chip ID
(0x0E3B)") added support for the ZL30643, a ZL3064x line-card variant
that is register-compatible with ZL3073x. However, ZL3064x chips
require a minimum input reference frequency of 1 kHz, not 1 Hz like
other ZL3073x parts.
Add a min_ref_freq field to zl3073x_chip_info and validate it in both
the DT property parser and the ref-sync pair configuration to prevent
configuring unsupported frequencies on ZL3064x parts.
Signed-off-by: Ivan Vecera <ivecera@redhat.com>
---
drivers/dpll/zl3073x/core.c | 43 +++++++++++++++++++------------------
drivers/dpll/zl3073x/core.h | 2 ++
drivers/dpll/zl3073x/dpll.c | 10 +++++----
drivers/dpll/zl3073x/prop.c | 10 ++++++---
4 files changed, 37 insertions(+), 28 deletions(-)
diff --git a/drivers/dpll/zl3073x/core.c b/drivers/dpll/zl3073x/core.c
index 5b2d77f2c2288e..c0b3b0d579d524 100644
--- a/drivers/dpll/zl3073x/core.c
+++ b/drivers/dpll/zl3073x/core.c
@@ -20,29 +20,30 @@
#include "dpll.h"
#include "regs.h"
-#define ZL_CHIP_INFO(_id, _nchannels, _flags) \
- { .id = (_id), .num_channels = (_nchannels), .flags = (_flags) }
+#define ZL_CHIP_INFO(_id, _nchannels, _flags, _min_freq) \
+ { .id = (_id), .num_channels = (_nchannels), .flags = (_flags), \
+ .min_ref_freq = (_min_freq) }
static const struct zl3073x_chip_info zl3073x_chip_ids[] = {
- ZL_CHIP_INFO(0x0E30, 2, ZL3073X_FLAG_REF_PHASE_COMP_32),
- ZL_CHIP_INFO(0x0E3B, 3, ZL3073X_FLAG_REF_PHASE_COMP_32),
- ZL_CHIP_INFO(0x0E93, 1, ZL3073X_FLAG_REF_PHASE_COMP_32),
- ZL_CHIP_INFO(0x0E94, 2, ZL3073X_FLAG_REF_PHASE_COMP_32),
- ZL_CHIP_INFO(0x0E95, 3, ZL3073X_FLAG_REF_PHASE_COMP_32),
- ZL_CHIP_INFO(0x0E96, 4, ZL3073X_FLAG_REF_PHASE_COMP_32),
- ZL_CHIP_INFO(0x0E97, 5, ZL3073X_FLAG_REF_PHASE_COMP_32),
- ZL_CHIP_INFO(0x1E93, 1, ZL3073X_FLAG_DIE_TEMP),
- ZL_CHIP_INFO(0x1E94, 2, ZL3073X_FLAG_DIE_TEMP),
- ZL_CHIP_INFO(0x1E95, 3, ZL3073X_FLAG_DIE_TEMP),
- ZL_CHIP_INFO(0x1E96, 4, ZL3073X_FLAG_DIE_TEMP),
- ZL_CHIP_INFO(0x1E97, 5, ZL3073X_FLAG_DIE_TEMP),
- ZL_CHIP_INFO(0x1F60, 2, ZL3073X_FLAG_REF_PHASE_COMP_32),
- ZL_CHIP_INFO(0x2E93, 1, ZL3073X_FLAG_DIE_TEMP),
- ZL_CHIP_INFO(0x2E94, 2, ZL3073X_FLAG_DIE_TEMP),
- ZL_CHIP_INFO(0x2E95, 3, ZL3073X_FLAG_DIE_TEMP),
- ZL_CHIP_INFO(0x2E96, 4, ZL3073X_FLAG_DIE_TEMP),
- ZL_CHIP_INFO(0x2E97, 5, ZL3073X_FLAG_DIE_TEMP),
- ZL_CHIP_INFO(0x3FC4, 2, ZL3073X_FLAG_DIE_TEMP),
+ ZL_CHIP_INFO(0x0E30, 2, ZL3073X_FLAG_REF_PHASE_COMP_32, 1),
+ ZL_CHIP_INFO(0x0E3B, 3, ZL3073X_FLAG_REF_PHASE_COMP_32, 1000),
+ ZL_CHIP_INFO(0x0E93, 1, ZL3073X_FLAG_REF_PHASE_COMP_32, 1),
+ ZL_CHIP_INFO(0x0E94, 2, ZL3073X_FLAG_REF_PHASE_COMP_32, 1),
+ ZL_CHIP_INFO(0x0E95, 3, ZL3073X_FLAG_REF_PHASE_COMP_32, 1),
+ ZL_CHIP_INFO(0x0E96, 4, ZL3073X_FLAG_REF_PHASE_COMP_32, 1),
+ ZL_CHIP_INFO(0x0E97, 5, ZL3073X_FLAG_REF_PHASE_COMP_32, 1),
+ ZL_CHIP_INFO(0x1E93, 1, ZL3073X_FLAG_DIE_TEMP, 1),
+ ZL_CHIP_INFO(0x1E94, 2, ZL3073X_FLAG_DIE_TEMP, 1),
+ ZL_CHIP_INFO(0x1E95, 3, ZL3073X_FLAG_DIE_TEMP, 1),
+ ZL_CHIP_INFO(0x1E96, 4, ZL3073X_FLAG_DIE_TEMP, 1),
+ ZL_CHIP_INFO(0x1E97, 5, ZL3073X_FLAG_DIE_TEMP, 1),
+ ZL_CHIP_INFO(0x1F60, 2, ZL3073X_FLAG_REF_PHASE_COMP_32, 1),
+ ZL_CHIP_INFO(0x2E93, 1, ZL3073X_FLAG_DIE_TEMP, 1),
+ ZL_CHIP_INFO(0x2E94, 2, ZL3073X_FLAG_DIE_TEMP, 1),
+ ZL_CHIP_INFO(0x2E95, 3, ZL3073X_FLAG_DIE_TEMP, 1),
+ ZL_CHIP_INFO(0x2E96, 4, ZL3073X_FLAG_DIE_TEMP, 1),
+ ZL_CHIP_INFO(0x2E97, 5, ZL3073X_FLAG_DIE_TEMP, 1),
+ ZL_CHIP_INFO(0x3FC4, 2, ZL3073X_FLAG_DIE_TEMP, 1),
};
#define ZL_RANGE_OFFSET 0x80
diff --git a/drivers/dpll/zl3073x/core.h b/drivers/dpll/zl3073x/core.h
index 78dc208f3eea2a..51014463377356 100644
--- a/drivers/dpll/zl3073x/core.h
+++ b/drivers/dpll/zl3073x/core.h
@@ -42,11 +42,13 @@ enum zl3073x_flags {
* @id: chip ID
* @num_channels: number of DPLL channels supported by this variant
* @flags: chip variant flags
+ * @min_ref_freq: minimum input reference frequency in Hz
*/
struct zl3073x_chip_info {
u16 id;
u8 num_channels;
unsigned long flags;
+ u32 min_ref_freq;
};
/**
diff --git a/drivers/dpll/zl3073x/dpll.c b/drivers/dpll/zl3073x/dpll.c
index 83bd3027dbaa1e..ee077c3e68da83 100644
--- a/drivers/dpll/zl3073x/dpll.c
+++ b/drivers/dpll/zl3073x/dpll.c
@@ -281,7 +281,8 @@ zl3073x_dpll_input_pin_ref_sync_set(const struct dpll_pin *dpll_pin,
sync_freq = zl3073x_ref_freq_get(sync_ref);
/* Sync signal must be 8 kHz or less and clock reference
- * must be 1 kHz or more and higher than the sync signal.
+ * must meet the chip's minimum frequency requirement and be
+ * higher than the sync signal.
*/
if (sync_freq > 8000) {
NL_SET_ERR_MSG(extack,
@@ -289,9 +290,10 @@ zl3073x_dpll_input_pin_ref_sync_set(const struct dpll_pin *dpll_pin,
rc = -EINVAL;
goto unlock;
}
- if (ref_freq < 1000) {
- NL_SET_ERR_MSG(extack,
- "clock frequency must be 1 kHz or more");
+ if (ref_freq < zldev->info->min_ref_freq) {
+ NL_SET_ERR_MSG_FMT(extack,
+ "clock frequency must be %u Hz or more",
+ zldev->info->min_ref_freq);
rc = -EINVAL;
goto unlock;
}
diff --git a/drivers/dpll/zl3073x/prop.c b/drivers/dpll/zl3073x/prop.c
index ac9d41d0f978ef..cdceddcf353e46 100644
--- a/drivers/dpll/zl3073x/prop.c
+++ b/drivers/dpll/zl3073x/prop.c
@@ -20,9 +20,9 @@
* @freq: frequency to check
*
* The function checks the given frequency is valid for the device. For input
- * pins it checks that the frequency can be factorized using supported base
- * frequencies. For output pins it checks that the frequency divides connected
- * synth frequency without remainder.
+ * pins it checks that the frequency is above the chip's minimum and can be
+ * factorized using supported base frequencies. For output pins it checks that
+ * the frequency divides connected synth frequency without remainder.
*
* Return: true if the frequency is valid, false if not.
*/
@@ -36,6 +36,10 @@ zl3073x_pin_check_freq(struct zl3073x_dev *zldev, enum dpll_pin_direction dir,
if (dir == DPLL_PIN_DIRECTION_INPUT) {
int rc;
+ /* Check minimum frequency */
+ if (freq < zldev->info->min_ref_freq)
+ goto err_inv_freq;
+
/* Check if the frequency can be factorized */
rc = zl3073x_ref_freq_factorize(freq, NULL, NULL);
if (rc)
base-commit: 4f93b12cf7b25fbf8e73d222722805b049f0a6d3
--
2.54.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH net-next] dpll: zl3073x: add chip-specific minimum input reference frequency
2026-08-14 15:32 [PATCH net-next] dpll: zl3073x: add chip-specific minimum input reference frequency Ivan Vecera
@ 2026-08-15 22:31 ` Vadim Fedorenko
2026-08-16 11:04 ` Vincent Jardin
1 sibling, 0 replies; 3+ messages in thread
From: Vadim Fedorenko @ 2026-08-15 22:31 UTC (permalink / raw)
To: Ivan Vecera, netdev
Cc: Vincent Jardin, Min Li, Arkadiusz Kubalewski, Jiri Pirko,
open list
On 14/08/2026 16:32, Ivan Vecera wrote:
> Commit 24e4aff8983fe6 ("dpll: zl3073x: recognize the ZL30643 chip ID
> (0x0E3B)") added support for the ZL30643, a ZL3064x line-card variant
> that is register-compatible with ZL3073x. However, ZL3064x chips
> require a minimum input reference frequency of 1 kHz, not 1 Hz like
> other ZL3073x parts.
>
> Add a min_ref_freq field to zl3073x_chip_info and validate it in both
> the DT property parser and the ref-sync pair configuration to prevent
> configuring unsupported frequencies on ZL3064x parts.
>
> Signed-off-by: Ivan Vecera <ivecera@redhat.com>
> ---
> drivers/dpll/zl3073x/core.c | 43 +++++++++++++++++++------------------
> drivers/dpll/zl3073x/core.h | 2 ++
> drivers/dpll/zl3073x/dpll.c | 10 +++++----
> drivers/dpll/zl3073x/prop.c | 10 ++++++---
> 4 files changed, 37 insertions(+), 28 deletions(-)
>
> diff --git a/drivers/dpll/zl3073x/core.c b/drivers/dpll/zl3073x/core.c
> index 5b2d77f2c2288e..c0b3b0d579d524 100644
> --- a/drivers/dpll/zl3073x/core.c
> +++ b/drivers/dpll/zl3073x/core.c
> @@ -20,29 +20,30 @@
> #include "dpll.h"
> #include "regs.h"
>
> -#define ZL_CHIP_INFO(_id, _nchannels, _flags) \
> - { .id = (_id), .num_channels = (_nchannels), .flags = (_flags) }
> +#define ZL_CHIP_INFO(_id, _nchannels, _flags, _min_freq) \
> + { .id = (_id), .num_channels = (_nchannels), .flags = (_flags), \
> + .min_ref_freq = (_min_freq) }
>
> static const struct zl3073x_chip_info zl3073x_chip_ids[] = {
> - ZL_CHIP_INFO(0x0E30, 2, ZL3073X_FLAG_REF_PHASE_COMP_32),
> - ZL_CHIP_INFO(0x0E3B, 3, ZL3073X_FLAG_REF_PHASE_COMP_32),
> - ZL_CHIP_INFO(0x0E93, 1, ZL3073X_FLAG_REF_PHASE_COMP_32),
> - ZL_CHIP_INFO(0x0E94, 2, ZL3073X_FLAG_REF_PHASE_COMP_32),
> - ZL_CHIP_INFO(0x0E95, 3, ZL3073X_FLAG_REF_PHASE_COMP_32),
> - ZL_CHIP_INFO(0x0E96, 4, ZL3073X_FLAG_REF_PHASE_COMP_32),
> - ZL_CHIP_INFO(0x0E97, 5, ZL3073X_FLAG_REF_PHASE_COMP_32),
> - ZL_CHIP_INFO(0x1E93, 1, ZL3073X_FLAG_DIE_TEMP),
> - ZL_CHIP_INFO(0x1E94, 2, ZL3073X_FLAG_DIE_TEMP),
> - ZL_CHIP_INFO(0x1E95, 3, ZL3073X_FLAG_DIE_TEMP),
> - ZL_CHIP_INFO(0x1E96, 4, ZL3073X_FLAG_DIE_TEMP),
> - ZL_CHIP_INFO(0x1E97, 5, ZL3073X_FLAG_DIE_TEMP),
> - ZL_CHIP_INFO(0x1F60, 2, ZL3073X_FLAG_REF_PHASE_COMP_32),
> - ZL_CHIP_INFO(0x2E93, 1, ZL3073X_FLAG_DIE_TEMP),
> - ZL_CHIP_INFO(0x2E94, 2, ZL3073X_FLAG_DIE_TEMP),
> - ZL_CHIP_INFO(0x2E95, 3, ZL3073X_FLAG_DIE_TEMP),
> - ZL_CHIP_INFO(0x2E96, 4, ZL3073X_FLAG_DIE_TEMP),
> - ZL_CHIP_INFO(0x2E97, 5, ZL3073X_FLAG_DIE_TEMP),
> - ZL_CHIP_INFO(0x3FC4, 2, ZL3073X_FLAG_DIE_TEMP),
> + ZL_CHIP_INFO(0x0E30, 2, ZL3073X_FLAG_REF_PHASE_COMP_32, 1),
> + ZL_CHIP_INFO(0x0E3B, 3, ZL3073X_FLAG_REF_PHASE_COMP_32, 1000),
> + ZL_CHIP_INFO(0x0E93, 1, ZL3073X_FLAG_REF_PHASE_COMP_32, 1),
> + ZL_CHIP_INFO(0x0E94, 2, ZL3073X_FLAG_REF_PHASE_COMP_32, 1),
> + ZL_CHIP_INFO(0x0E95, 3, ZL3073X_FLAG_REF_PHASE_COMP_32, 1),
> + ZL_CHIP_INFO(0x0E96, 4, ZL3073X_FLAG_REF_PHASE_COMP_32, 1),
> + ZL_CHIP_INFO(0x0E97, 5, ZL3073X_FLAG_REF_PHASE_COMP_32, 1),
> + ZL_CHIP_INFO(0x1E93, 1, ZL3073X_FLAG_DIE_TEMP, 1),
> + ZL_CHIP_INFO(0x1E94, 2, ZL3073X_FLAG_DIE_TEMP, 1),
> + ZL_CHIP_INFO(0x1E95, 3, ZL3073X_FLAG_DIE_TEMP, 1),
> + ZL_CHIP_INFO(0x1E96, 4, ZL3073X_FLAG_DIE_TEMP, 1),
> + ZL_CHIP_INFO(0x1E97, 5, ZL3073X_FLAG_DIE_TEMP, 1),
> + ZL_CHIP_INFO(0x1F60, 2, ZL3073X_FLAG_REF_PHASE_COMP_32, 1),
> + ZL_CHIP_INFO(0x2E93, 1, ZL3073X_FLAG_DIE_TEMP, 1),
> + ZL_CHIP_INFO(0x2E94, 2, ZL3073X_FLAG_DIE_TEMP, 1),
> + ZL_CHIP_INFO(0x2E95, 3, ZL3073X_FLAG_DIE_TEMP, 1),
> + ZL_CHIP_INFO(0x2E96, 4, ZL3073X_FLAG_DIE_TEMP, 1),
> + ZL_CHIP_INFO(0x2E97, 5, ZL3073X_FLAG_DIE_TEMP, 1),
> + ZL_CHIP_INFO(0x3FC4, 2, ZL3073X_FLAG_DIE_TEMP, 1),
> };
>
> #define ZL_RANGE_OFFSET 0x80
> diff --git a/drivers/dpll/zl3073x/core.h b/drivers/dpll/zl3073x/core.h
> index 78dc208f3eea2a..51014463377356 100644
> --- a/drivers/dpll/zl3073x/core.h
> +++ b/drivers/dpll/zl3073x/core.h
> @@ -42,11 +42,13 @@ enum zl3073x_flags {
> * @id: chip ID
> * @num_channels: number of DPLL channels supported by this variant
> * @flags: chip variant flags
> + * @min_ref_freq: minimum input reference frequency in Hz
> */
> struct zl3073x_chip_info {
> u16 id;
> u8 num_channels;
> unsigned long flags;
> + u32 min_ref_freq;
> };
>
> /**
> diff --git a/drivers/dpll/zl3073x/dpll.c b/drivers/dpll/zl3073x/dpll.c
> index 83bd3027dbaa1e..ee077c3e68da83 100644
> --- a/drivers/dpll/zl3073x/dpll.c
> +++ b/drivers/dpll/zl3073x/dpll.c
> @@ -281,7 +281,8 @@ zl3073x_dpll_input_pin_ref_sync_set(const struct dpll_pin *dpll_pin,
> sync_freq = zl3073x_ref_freq_get(sync_ref);
>
> /* Sync signal must be 8 kHz or less and clock reference
> - * must be 1 kHz or more and higher than the sync signal.
> + * must meet the chip's minimum frequency requirement and be
> + * higher than the sync signal.
> */
> if (sync_freq > 8000) {
> NL_SET_ERR_MSG(extack,
> @@ -289,9 +290,10 @@ zl3073x_dpll_input_pin_ref_sync_set(const struct dpll_pin *dpll_pin,
> rc = -EINVAL;
> goto unlock;
> }
> - if (ref_freq < 1000) {
> - NL_SET_ERR_MSG(extack,
> - "clock frequency must be 1 kHz or more");
> + if (ref_freq < zldev->info->min_ref_freq) {
> + NL_SET_ERR_MSG_FMT(extack,
> + "clock frequency must be %u Hz or more",
> + zldev->info->min_ref_freq);
> rc = -EINVAL;
> goto unlock;
> }
> diff --git a/drivers/dpll/zl3073x/prop.c b/drivers/dpll/zl3073x/prop.c
> index ac9d41d0f978ef..cdceddcf353e46 100644
> --- a/drivers/dpll/zl3073x/prop.c
> +++ b/drivers/dpll/zl3073x/prop.c
> @@ -20,9 +20,9 @@
> * @freq: frequency to check
> *
> * The function checks the given frequency is valid for the device. For input
> - * pins it checks that the frequency can be factorized using supported base
> - * frequencies. For output pins it checks that the frequency divides connected
> - * synth frequency without remainder.
> + * pins it checks that the frequency is above the chip's minimum and can be
> + * factorized using supported base frequencies. For output pins it checks that
> + * the frequency divides connected synth frequency without remainder.
> *
> * Return: true if the frequency is valid, false if not.
> */
> @@ -36,6 +36,10 @@ zl3073x_pin_check_freq(struct zl3073x_dev *zldev, enum dpll_pin_direction dir,
> if (dir == DPLL_PIN_DIRECTION_INPUT) {
> int rc;
>
> + /* Check minimum frequency */
> + if (freq < zldev->info->min_ref_freq)
> + goto err_inv_freq;
> +
> /* Check if the frequency can be factorized */
> rc = zl3073x_ref_freq_factorize(freq, NULL, NULL);
> if (rc)
>
> base-commit: 4f93b12cf7b25fbf8e73d222722805b049f0a6d3
LGTM,
Reviewed-by: Vadim Fedorenko <vadim.fedorenko@linux.dev>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH net-next] dpll: zl3073x: add chip-specific minimum input reference frequency
2026-08-14 15:32 [PATCH net-next] dpll: zl3073x: add chip-specific minimum input reference frequency Ivan Vecera
2026-08-15 22:31 ` Vadim Fedorenko
@ 2026-08-16 11:04 ` Vincent Jardin
1 sibling, 0 replies; 3+ messages in thread
From: Vincent Jardin @ 2026-08-16 11:04 UTC (permalink / raw)
To: Ivan Vecera
Cc: netdev, Min Li, Vadim Fedorenko, Arkadiusz Kubalewski, Jiri Pirko,
open list
Hi Ivan,
Thanks for leading it. Below some few minor comments.
> diff --git a/drivers/dpll/zl3073x/core.c b/drivers/dpll/zl3073x/core.c
> index 5b2d77f2c2288e..c0b3b0d579d524 100644
> --- a/drivers/dpll/zl3073x/core.c
> +++ b/drivers/dpll/zl3073x/core.c
> @@ -20,29 +20,30 @@
> #include "dpll.h"
> #include "regs.h"
>
> -#define ZL_CHIP_INFO(_id, _nchannels, _flags) \
> - { .id = (_id), .num_channels = (_nchannels), .flags = (_flags) }
> +#define ZL_CHIP_INFO(_id, _nchannels, _flags, _min_freq) \
> + { .id = (_id), .num_channels = (_nchannels), .flags = (_flags), \
> + .min_ref_freq = (_min_freq) }
Do we really need this C macro ? Everytime we'll add something, it leads to a
wide numbder of line changes for zl3073x_chip_ids[]. What's about using directly the named
fields below and leaving 0/nothing for default cases such as the min_ref_freq ( something like
min_ref_freq ?: 1; ) ?
... return zldev->info->min_ref_freq ?: 1; ...
If you prefer to keep the C macro, then let's use a variadic, something like:
#define ZL_CHIP_INFO(_id, _nchannels, _flags, ...) \
{ .id = (_id), .num_channels = (_nchannels), .flags = (_flags),\
##__VA_ARGS__ }
then only the ZL30643 entry needs to be touched:
ZL_CHIP_INFO(0x0E3B, 3, ZL3073X_FLAG_REF_PHASE_COMP_32,
.min_ref_freq = 1000),
>
> static const struct zl3073x_chip_info zl3073x_chip_ids[] = {
> - ZL_CHIP_INFO(0x0E30, 2, ZL3073X_FLAG_REF_PHASE_COMP_32),
> - ZL_CHIP_INFO(0x0E3B, 3, ZL3073X_FLAG_REF_PHASE_COMP_32),
...
> - ZL_CHIP_INFO(0x2E97, 5, ZL3073X_FLAG_DIE_TEMP),
> - ZL_CHIP_INFO(0x3FC4, 2, ZL3073X_FLAG_DIE_TEMP),
> + ZL_CHIP_INFO(0x0E30, 2, ZL3073X_FLAG_REF_PHASE_COMP_32, 1),
> + ZL_CHIP_INFO(0x0E3B, 3, ZL3073X_FLAG_REF_PHASE_COMP_32, 1000),
> + ZL_CHIP_INFO(0x0E93, 1, ZL3073X_FLAG_REF_PHASE_COMP_32, 1),
...
> + ZL_CHIP_INFO(0x2E97, 5, ZL3073X_FLAG_DIE_TEMP, 1),
> + ZL_CHIP_INFO(0x3FC4, 2, ZL3073X_FLAG_DIE_TEMP, 1),
That's a wide set of changes just to add/update 1 line. Since an update is needed, I feel the macro
should be avoided so only
ZL_CHIP_INFO(0x0E3B, 3, ZL3073X_FLAG_REF_PHASE_COMP_32, 1000),
is needed.
Then, the init code could set to 1 if the value of .min_ref_freq is 0.
> --- a/drivers/dpll/zl3073x/dpll.c
> +++ b/drivers/dpll/zl3073x/dpll.c
> @@ -281,7 +281,8 @@ zl3073x_dpll_input_pin_ref_sync_set(const struct dpll_pin *dpll_pin,
> sync_freq = zl3073x_ref_freq_get(sync_ref);
>
> /* Sync signal must be 8 kHz or less and clock reference
> - * must be 1 kHz or more and higher than the sync signal.
> + * must meet the chip's minimum frequency requirement and be
> + * higher than the sync signal.
> */
> if (sync_freq > 8000) {
> NL_SET_ERR_MSG(extack,
> @@ -289,9 +290,10 @@ zl3073x_dpll_input_pin_ref_sync_set(const struct dpll_pin *dpll_pin,
> rc = -EINVAL;
> goto unlock;
> }
> - if (ref_freq < 1000) {
> - NL_SET_ERR_MSG(extack,
> - "clock frequency must be 1 kHz or more");
> + if (ref_freq < zldev->info->min_ref_freq) {
> + NL_SET_ERR_MSG_FMT(extack,
> + "clock frequency must be %u Hz or more",
> + zldev->info->min_ref_freq);
> rc = -EINVAL;
> goto unlock;
> }
TBC: to be considered if 0 means 1 (à la zldev->info->min_ref_freq ?: 1)
> diff --git a/drivers/dpll/zl3073x/prop.c b/drivers/dpll/zl3073x/prop.c
> index ac9d41d0f978ef..cdceddcf353e46 100644
> --- a/drivers/dpll/zl3073x/prop.c
> +++ b/drivers/dpll/zl3073x/prop.c
> @@ -20,9 +20,9 @@
> * @freq: frequency to check
> *
> * The function checks the given frequency is valid for the device. For input
> - * pins it checks that the frequency can be factorized using supported base
> - * frequencies. For output pins it checks that the frequency divides connected
> - * synth frequency without remainder.
> + * pins it checks that the frequency is above the chip's minimum and can be
> + * factorized using supported base frequencies. For output pins it checks that
> + * the frequency divides connected synth frequency without remainder.
> *
> * Return: true if the frequency is valid, false if not.
> */
> @@ -36,6 +36,10 @@ zl3073x_pin_check_freq(struct zl3073x_dev *zldev, enum dpll_pin_direction dir,
> if (dir == DPLL_PIN_DIRECTION_INPUT) {
> int rc;
>
> + /* Check minimum frequency */
This comment is not needed, the code tells us.
> + if (freq < zldev->info->min_ref_freq)
> + goto err_inv_freq;
> +
> /* Check if the frequency can be factorized */
> rc = zl3073x_ref_freq_factorize(freq, NULL, NULL);
> if (rc)
>
Best regards,
Vincent
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2026-08-16 11:04 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-08-14 15:32 [PATCH net-next] dpll: zl3073x: add chip-specific minimum input reference frequency Ivan Vecera
2026-08-15 22:31 ` Vadim Fedorenko
2026-08-16 11:04 ` Vincent Jardin
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.