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. [...] >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