V9 sent 23 06 2023

[kubalewski]

v8 -> v9:
[00/10] Create common DPLL configuration API

• update examples to reflect new pin-parent nest split

[01/10] dpll: documentation on DPLL subsystem interface

• fix docs build warnings
• separate netlink command/attribute list
• replace enum description with uapi header
• add brief explanation what is a DPLL
• fix EOPNOTSUPP typo
• fix typo .state_get -> .state_on_dpll_get

[02/10] dpll: spec: Add Netlink spec in YAML

• regenerate policy max values
• add missing enum descriptions
• split pin-parent nest:
  • pin-parent-device - for configuration of pin-device tuple
  • pin-parent-pin - for configuration od pin-pin tuple
• fix typos:
  • s/working-modes/working modes/
  • s/differentiate/differentiates/
  • s/valid input, auto selected by dpll/input pin auto selected by dpll/
• remove FREERUN and HOLDOVER modes

[03/10] dpll: core: Add DPLL framework base functions

• fix description in spdx header.
• remove refcount check if refcount was already set
• do not validate dpll ptr in dpll_device_put(..)
• fix return -ENOMEM on failed memory alloc
• do not validate pin ptr in dpll_pin_put(..)
• return -EINVAL in case of module/clock_id mismatch
• do not {} around one-line xa_for_each() macro
• move dpll__registration structs to dpll_core.c
• rephrase doc comment on device and pin id struct members
• remove ref in case of memory allocation fail
• check for required ops on pin/device registration
• mark pin with DPLL_REGISTERED once pin is registered with dpll

[04/10] dpll: netlink: Add DPLL framework base functions

• fix pin-id-get/device-id-get behavior
• reshuffle order of functions
• avoid forward declarations
• functions for adding pin/device handle next to each other
• pass ops callback return values to the user
• remove dpll_cmd_pin_fill_details(..) function, merge the code into
  __dpll_cmd_pin_dump_one(..)
• rename __dpll_cmd_pin_dump_one() to dpll_cmd_pin_get_one()
• use WARN_ON macro when dpll ref is missing
• remove redundant pin's dpll list not empty check
• remove double spaces inside if statement
• add extack message when set command is not possible
• do not return error when callback is not required
• WARN_ON missing ops moved to dpll_core.c
• use DPLL_REGISTERED if pin was registered with dpll
• fix pin-id-get return and add extack errors
• fix device-id-get return and add extack errors
• drop pointless init of variables
• add macro for iterating over marked pins/devices
• move dpll_set_from_nlattr() for consistent order
• use GENL_REQ_ATTR_CHECK() for checking attibute presence
• fill extack if pin/device was not found
• drop pointless init of variables
• WARN_ON if dpll not registered on send event
• rename goto labels to indicate error path
• fix docs
• drop pointless init of variables
• verify pin in notify with a mark
• prevent ops->mode_set call if missing callback
• move static dpll_msg_add_pin_handle() from pin<->netdev patch
• split pin-parent nest:
  • pin-parent-device - for configuration of pin-device tuple
  • pin-parent-pin - for configuration od pin-pin tuple

[06/10] netdev: expose DPLL pin handle for netdevice

• net_device->dpll_pin is only valid if IS_ENABLED(CONFIG_DPLL) fix the
  code in net/core/rtnetlink.c to respect that.
• move dpll_msg_add_pin_handle to "dpll: netlink" patch + export the
  function with this patch

[07/10] ice: add admin commands to access cgu configuration

• rename MAX_NETLIST_SIZE -> ICE_MAX_NETLIST_SIZE
• simplify function: s64 convert_s48_to_s64(s64 signed_48)
• do not assign 0 to field that is already 0

[08/10] ice: implement dpll interface to control cgu

• drop pointless 0 assignement
• ice_dpll_init(..) returns void instead of int
• fix context description of the functions
• fix ice_dpll_init(..) traces
• fix use package_label instead pf board_label for rclk pin
• be consistent on cgu presence naming
• remove indent in ice_dpll_deinit(..)
• remove unused struct field lock_err_num
• fix kworker resched behavior
• remove debug log from ice_dpll_deinit_worker(..)
• reorder ice internal functions
• release resources directly on error path
• remove redundant NULL checks when releasing resources
• do not assign NULL to pointers after releasing resources
• simplify variable assignement
• fix 'int ret;' declarations across the ice_dpll.c
• remove leftover ice_dpll_find(..)
• get pf pointer from dpll_priv without type cast
• improve error reporting
• fix documentation
• fix ice_dpll_update_state(..) flow
• fix return in case out of range prio set