29/04/2021 17:43, Matan Azrad:
> From: Dekel Peled <dek...@nvidia.com>
>
> This patch adds support of write operation to NIC registers.
>
> Signed-off-by: Dekel Peled <dek...@nvidia.com>
> Acked-by: Matan Azrad <ma...@nvidia.com>
> ---
> --- a/drivers/common/mlx5/version.map
> +++ b/drivers/common/mlx5/version.map
> @@ -13,8 +13,17 @@ INTERNAL {
> mlx5_dev_to_pci_addr; # WINDOWS_NO_EXPORT
>
> mlx5_devx_cmd_alloc_pd;
> + mlx5_devx_alloc_uar;
Please keep alphabetical order.
> +
> mlx5_devx_cmd_create_cq;
> + mlx5_devx_cmd_create_credential_obj;
> + mlx5_devx_cmd_create_crypto_login_obj;
> + mlx5_devx_cmd_create_dek_obj;
> mlx5_devx_cmd_create_flex_parser;
> + mlx5_devx_cmd_create_flow_hit_aso_obj;
> + mlx5_devx_cmd_create_flow_meter_aso_obj;
> + mlx5_devx_cmd_create_geneve_tlv_option;
> + mlx5_devx_cmd_create_import_kek_obj;
> mlx5_devx_cmd_create_qp;
> mlx5_devx_cmd_create_rq;
> mlx5_devx_cmd_create_rqt;
> @@ -24,13 +33,6 @@ INTERNAL {
> mlx5_devx_cmd_create_tis;
> mlx5_devx_cmd_create_virtio_q_counters; # WINDOWS_NO_EXPORT
> mlx5_devx_cmd_create_virtq;
> - mlx5_devx_cmd_create_flow_hit_aso_obj;
> - mlx5_devx_cmd_create_flow_meter_aso_obj;
> - mlx5_devx_cmd_create_geneve_tlv_option;
> - mlx5_devx_cmd_create_dek_obj;
> - mlx5_devx_cmd_create_import_kek_obj;
> - mlx5_devx_cmd_create_credential_obj;
> - mlx5_devx_cmd_create_crypto_login_obj;
Are they changes of previous patches?
It seems some cleaning is required in previous patches.
> mlx5_devx_cmd_destroy;
> mlx5_devx_cmd_flow_counter_alloc;
> mlx5_devx_cmd_flow_counter_query;
> @@ -51,12 +53,17 @@ INTERNAL {
> mlx5_devx_cmd_queue_counter_alloc; # WINDOWS_NO_EXPORT
> mlx5_devx_cmd_queue_counter_query; # WINDOWS_NO_EXPORT
> mlx5_devx_cmd_register_read;
> + mlx5_devx_cmd_register_write;
> mlx5_devx_cmd_wq_query; # WINDOWS_NO_EXPORT
> mlx5_devx_get_out_command_status;
> mlx5_devx_alloc_uar; # WINDOWS_NO_EXPORT
>
> +
> mlx5_devx_cq_create;
There is an extra blank line here.
