Re: [PATCH v5 0/2] add two commands for testpmd application

[Ferruh Yigit <ferruh.yigit@amd.com>]

On 10/10/2024 7:42 AM, Chaoyong He wrote:
> This patch series aims to add two commands for the testpmd application:
> - testpmd> set port <port-id> eeprom <accept_risk> magic <magic> \
>           value <value> offset <offset>
> - testpmd> set port <port-id> led <on/off>
> 
> ---
> v5:
> * Use 'accept_risk' rather than 'confirm' to make it clear enough.
> * Use dynamic memory alloc rather than VLA.
> * Place the logic in the right place following the reviewer's request.
> v4:
> * Make the log more readable by using 'rte_strerror()'.
> * Make the modification of document following the rules.
> v3:
> * Add acknowledgement for the set eeprom command.
> * Modify logic following the request from reviewer.
> v2:
> * Solve the conflict in document file.
> ---
> 
> James Hershaw (2):
>   app/testpmd: add support for setting device EEPROM
>   app/testpmd: add set dev led on/off command
>

For series,
Acked-by: Ferruh Yigit <ferruh.yigit@amd.com>

Series applied to dpdk-next-net/main, thanks.