[v5,12/19] net/bnxt: replace use of term sanity

Message ID 20231129173122.126895-13-stephen@networkplumber.org (mailing list archive)
State Superseded, archived
Delegated to: Thomas Monjalon
Headers
Series remove use of the term "sanity check" |

Checks

Context Check Description
ci/checkpatch success coding style OK

Commit Message

Stephen Hemminger Nov. 29, 2023, 5:26 p.m. UTC
  Do not use non-inclusive terms.

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 drivers/net/bnxt/bnxt_hwrm.c           | 2 +-
 drivers/net/bnxt/hsi_struct_def_dpdk.h | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
  

Comments

Somnath Kotur Nov. 30, 2023, 3:31 a.m. UTC | #1
On Wed, Nov 29, 2023 at 11:01 PM Stephen Hemminger
<stephen@networkplumber.org> wrote:
>
> Do not use non-inclusive terms.
>
> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
> ---
>  drivers/net/bnxt/bnxt_hwrm.c           | 2 +-
>  drivers/net/bnxt/hsi_struct_def_dpdk.h | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/net/bnxt/bnxt_hwrm.c b/drivers/net/bnxt/bnxt_hwrm.c
> index 06f196760fa8..e1254375b650 100644
> --- a/drivers/net/bnxt/bnxt_hwrm.c
> +++ b/drivers/net/bnxt/bnxt_hwrm.c
> @@ -262,7 +262,7 @@ static int bnxt_hwrm_send_message(struct bnxt *bp, void *msg,
>
>                 done = bnxt_check_cq_hwrm_done(cpr, is_tx, is_rx,
>                                                i == timeout - 1);
> -               /* Sanity check on the resp->resp_len */
> +               /* Input check on the resp->resp_len */
>                 rte_io_rmb();
>                 if (resp->resp_len && resp->resp_len <= bp->max_resp_len) {
>                         /* Last byte of resp contains the valid key */
> diff --git a/drivers/net/bnxt/hsi_struct_def_dpdk.h b/drivers/net/bnxt/hsi_struct_def_dpdk.h
> index 9afdd056ce7a..a8076a6fc223 100644
> --- a/drivers/net/bnxt/hsi_struct_def_dpdk.h
> +++ b/drivers/net/bnxt/hsi_struct_def_dpdk.h
> @@ -55482,7 +55482,7 @@ struct hwrm_tfc_ident_alloc_output {
>   * Requests the firmware to free a TFC resource identifier.
>   * A resource subtype and session id are passed in.
>   * An identifier (previously allocated) corresponding to all these is
> - * freed, only after various sanity checks are completed.
> + * freed, only after various checks are completed.
>   */
>  /* hwrm_tfc_ident_free_input (size:192b/24B) */
>  struct hwrm_tfc_ident_free_input {
> --
> 2.42.0
>
Reviewed-by: Somnath Kotur <somnath.kotur@broadcom.com>
  

Patch

diff --git a/drivers/net/bnxt/bnxt_hwrm.c b/drivers/net/bnxt/bnxt_hwrm.c
index 06f196760fa8..e1254375b650 100644
--- a/drivers/net/bnxt/bnxt_hwrm.c
+++ b/drivers/net/bnxt/bnxt_hwrm.c
@@ -262,7 +262,7 @@  static int bnxt_hwrm_send_message(struct bnxt *bp, void *msg,
 
 		done = bnxt_check_cq_hwrm_done(cpr, is_tx, is_rx,
 					       i == timeout - 1);
-		/* Sanity check on the resp->resp_len */
+		/* Input check on the resp->resp_len */
 		rte_io_rmb();
 		if (resp->resp_len && resp->resp_len <= bp->max_resp_len) {
 			/* Last byte of resp contains the valid key */
diff --git a/drivers/net/bnxt/hsi_struct_def_dpdk.h b/drivers/net/bnxt/hsi_struct_def_dpdk.h
index 9afdd056ce7a..a8076a6fc223 100644
--- a/drivers/net/bnxt/hsi_struct_def_dpdk.h
+++ b/drivers/net/bnxt/hsi_struct_def_dpdk.h
@@ -55482,7 +55482,7 @@  struct hwrm_tfc_ident_alloc_output {
  * Requests the firmware to free a TFC resource identifier.
  * A resource subtype and session id are passed in.
  * An identifier (previously allocated) corresponding to all these is
- * freed, only after various sanity checks are completed.
+ * freed, only after various checks are completed.
  */
 /* hwrm_tfc_ident_free_input (size:192b/24B) */
 struct hwrm_tfc_ident_free_input {