>From: Jiri Pirko <jiri@xxxxxxxxxxx> >Sent: Monday, October 2, 2023 4:53 PM > >Wed, Sep 27, 2023 at 11:24:33AM CEST, arkadiusz.kubalewski@xxxxxxxxx wrote: >>Add new pin's attributes to dpll netlink spec: >>- phase-offset - measured difference between phase of signals on pin >> and dpll >>- phase-adjust - adjustable value of pin's signal phase >>- phase-adjust-min / phase-adjust-max - values for determining limits >> for phase-adjust >> >>Signed-off-by: Arkadiusz Kubalewski <arkadiusz.kubalewski@xxxxxxxxx> >>--- >> Documentation/netlink/specs/dpll.yaml | 33 ++++++++++++++++++++++++++- >> drivers/dpll/dpll_nl.c | 8 ++++--- >> drivers/dpll/dpll_nl.h | 2 +- >> include/uapi/linux/dpll.h | 8 ++++++- >> 4 files changed, 45 insertions(+), 6 deletions(-) >> >>diff --git a/Documentation/netlink/specs/dpll.yaml >>b/Documentation/netlink/specs/dpll.yaml >>index 8b86b28b47a6..dc057494101f 100644 >>--- a/Documentation/netlink/specs/dpll.yaml >>+++ b/Documentation/netlink/specs/dpll.yaml >>@@ -1,7 +1,7 @@ >> # SPDX-License-Identifier: ((GPL-2.0 WITH Linux-syscall-note) OR BSD-3- >Clause) >> >> name: dpll >>- >>+version: 2 > >How is this supposed to work. You bump the version with every uapi >extension? First time I see this. > Well, wasn't sure if this needs to be updated, but sure will remove this part. Thank you! Arkadiusz >[...] > > >>diff --git a/include/uapi/linux/dpll.h b/include/uapi/linux/dpll.h >>index 20ef0718f8dc..050f51b48ef8 100644 >>--- a/include/uapi/linux/dpll.h >>+++ b/include/uapi/linux/dpll.h >>@@ -7,7 +7,7 @@ >> #define _UAPI_LINUX_DPLL_H >> >> #define DPLL_FAMILY_NAME "dpll" >>-#define DPLL_FAMILY_VERSION 1 >>+#define DPLL_FAMILY_VERSION 2