Re: [PATCH v5 2/4] lib/power: add get and set API for pause duration

["Burakov, Anatoly" <anatoly.burakov@intel.com>]

On 31-May-22 10:59 AM, Kevin Laatz wrote:
> Add new get/set API for configuring 'pause_duration' which used to adjust
> the pause mode callback duration.
> 
> Signed-off-by: Kevin Laatz <kevin.laatz@intel.com>
> Acked-by: Ray Kinsella <mdr@ashroe.eu>
> Acked-by: Anatoly Burakov <anatoly.burakov@intel.com>
> Tested-by: David Hunt <david.hunt@intel.com>
> 
> ---
> v5: add doc entry for new API
> v3: changed printf to RTE_LOG
> ---
>   doc/guides/prog_guide/power_man.rst |  6 ++++++
>   lib/power/rte_power_pmd_mgmt.c      | 25 +++++++++++++++++++++--
>   lib/power/rte_power_pmd_mgmt.h      | 31 +++++++++++++++++++++++++++++
>   lib/power/version.map               |  2 ++
>   4 files changed, 62 insertions(+), 2 deletions(-)
> 
> diff --git a/doc/guides/prog_guide/power_man.rst b/doc/guides/prog_guide/power_man.rst
> index 5d345f6eb9..f22513b324 100644
> --- a/doc/guides/prog_guide/power_man.rst
> +++ b/doc/guides/prog_guide/power_man.rst
> @@ -258,6 +258,12 @@ API Overview for Ethernet PMD Power Management
>   * **Set Emptypoll Max**: Set the number of empty polls to wait before entering
>     sleep state.
>   
> +* **Get Pause Duration**: Get the configured duration (ms) to be used in the
> +  Pause callback.
> +
> +* **Set Pause Duration**: Set the duration of the pause (ms) used in the Pause
> +  mode callback.
> +

Is it not microseconds rather than milliseconds?

-- 
Thanks,
Anatoly