From: Thomas Monjalon
> 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.
Yes, this one tried to do the cleaning, I will handle it in the first patch (of
this series) accessing this file.
> > 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.
>
>
