Re: [PATCH for 6.19 2/4] PM: EM: Rename em.yaml to dev-energymodel.yaml

[Donald Hunter]

Changwoo Min <changwoo@xxxxxxxxxx> writes:

> The EM YNL specification used many acronyms, including ‘em’, ‘pd’,
> ‘ps’, etc. While the acronyms are short and convenient, they could be
> confusing. So, let’s spell them out to be more specific. The following
> changes were made in the spec. Note that the protocol name cannot exceed
> GENL_NAMSIZ (16).
>
>   em           -> dev-energymodel
>   pds          -> perf-domains
>   pd           -> perf-domain
>   pd-id        -> perf-domain-id
>   pd-table     -> perf-table
>   ps           -> perf-state
>   get-pds      -> get-perf-domains
>   get-pd-table -> get-perf-table
>   pd-created   -> perf-domain-created
>   pd-updated   -> perf-domain-updated
>   pd-deleted   -> perf-domain-deleted
>
> In addition. doc strings were added to the spec. based on the comments in
> energy_model.h. Two flag attributes (perf-state-flags and
> perf-domain-flags) were added for easily interpreting the bit flags.
>
> Finally, the autogenerated files and em_netlink.c were updated accordingly
> to reflect the name changes.
>
> Suggested-by: Donald Hunter <donald.hunter@xxxxxxxxx>
> Signed-off-by: Changwoo Min <changwoo@xxxxxxxxxx>
> ---
>  .../netlink/specs/dev-energymodel.yaml        | 175 ++++++++++++++++++
>  Documentation/netlink/specs/em.yaml           | 116 ------------
>  MAINTAINERS                                   |   8 +-

YNL spec and MAINTAINERS

Reviewed-by: Donald Hunter <donald.hunter@xxxxxxxxx>

>  include/uapi/linux/dev_energymodel.h          |  89 +++++++++
>  include/uapi/linux/energy_model.h             |  63 -------
>  kernel/power/em_netlink.c                     | 135 ++++++++------
>  kernel/power/em_netlink_autogen.c             |  44 ++---
>  kernel/power/em_netlink_autogen.h             |  20 +-
>  8 files changed, 384 insertions(+), 266 deletions(-)
>  create mode 100644 Documentation/netlink/specs/dev-energymodel.yaml
>  delete mode 100644 Documentation/netlink/specs/em.yaml
>  create mode 100644 include/uapi/linux/dev_energymodel.h
>  delete mode 100644 include/uapi/linux/energy_model.h