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

[Kevin Laatz <kevin.laatz@intel.com>]

On 02/06/2022 15:01, Burakov, Anatoly wrote:
> 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?

Yes, you're right - fix coming. Thanks

/Kevin