[dpdk-stable] patch 'net/ena/base: fix documentation of functions' has been queued to LTS release 18.11.9

Kevin Traynor ktraynor at redhat.com
Thu May 28 18:22:53 CEST 2020


Hi,

FYI, your patch has been queued to LTS release 18.11.9

Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
It will be pushed if I get no objections before 06/03/20. So please
shout if anyone has objections.

Also note that after the patch there's a diff of the upstream commit vs the
patch applied to the branch. This will indicate if there was any rebasing
needed to apply to the stable branch. If there were code changes for rebasing
(ie: not only metadata diffs), please double check that the rebase was
correctly done.

Queued patches are on a temporary branch at:
https://github.com/kevintraynor/dpdk-stable-queue

This queued commit can be viewed at:
https://github.com/kevintraynor/dpdk-stable-queue/commit/a1c41ac2eba79d42bba4c52f33126b2a45b91cc2

Thanks.

Kevin.

---
>From a1c41ac2eba79d42bba4c52f33126b2a45b91cc2 Mon Sep 17 00:00:00 2001
From: Michal Krawczyk <mk at semihalf.com>
Date: Wed, 8 Apr 2020 10:29:01 +0200
Subject: [PATCH] net/ena/base: fix documentation of functions

[ upstream commit b118993abd4cae7af3f8a66e0ad4fda1bee84c09 ]

The documentation format was aligned and few typos were fixed.

Fixes: 99ecfbf845b3 ("ena: import communication layer")

Signed-off-by: Michal Krawczyk <mk at semihalf.com>
Reviewed-by: Igor Chauskin <igorch at amazon.com>
Reviewed-by: Guy Tzalik <gtzalik at amazon.com>
---
 drivers/net/ena/base/ena_com.h | 24 ++++++++++++------------
 1 file changed, 12 insertions(+), 12 deletions(-)

diff --git a/drivers/net/ena/base/ena_com.h b/drivers/net/ena/base/ena_com.h
index f58cd86a87..2e80ad563f 100644
--- a/drivers/net/ena/base/ena_com.h
+++ b/drivers/net/ena/base/ena_com.h
@@ -381,5 +381,5 @@ extern "C" {
 int ena_com_mmio_reg_read_request_init(struct ena_com_dev *ena_dev);
 
-/* ena_com_set_mmio_read_mode - Enable/disable the mmio reg read mechanism
+/* ena_com_set_mmio_read_mode - Enable/disable the indirect mmio reg read mechanism
  * @ena_dev: ENA communication layer struct
  * @readless_supported: readless mode (enable/disable)
@@ -507,5 +507,5 @@ bool ena_com_get_ena_admin_polling_mode(struct ena_com_dev *ena_dev);
  * @ena_dev: ENA communication layer struct
  *
- * This method go over the admin completion queue and wake up all the pending
+ * This method goes over the admin completion queue and wakes up all the pending
  * threads that wait on the commands wait event.
  *
@@ -517,5 +517,5 @@ void ena_com_admin_q_comp_intr_handler(struct ena_com_dev *ena_dev);
  * @ena_dev: ENA communication layer struct
  *
- * This method go over the async event notification queue and call the proper
+ * This method goes over the async event notification queue and calls the proper
  * aenq handler.
  */
@@ -534,5 +534,5 @@ void ena_com_abort_admin_commands(struct ena_com_dev *ena_dev);
  * @ena_dev: ENA communication layer struct
  *
- * This method wait until all the outstanding admin commands will be completed.
+ * This method waits until all the outstanding admin commands are completed.
  */
 void ena_com_wait_for_abort_completion(struct ena_com_dev *ena_dev);
@@ -541,5 +541,5 @@ void ena_com_wait_for_abort_completion(struct ena_com_dev *ena_dev);
  * @ena_dev: ENA communication layer struct
  *
- * This method validate the device parameters are the same as the saved
+ * This method verifies the device parameters are the same as the saved
  * parameters in ena_dev.
  * This method is useful after device reset, to validate the device mac address
@@ -718,5 +718,5 @@ int ena_com_set_hash_ctrl(struct ena_com_dev *ena_dev);
  * Retrieve the hash control from the device.
  *
- * @note, If the caller called ena_com_fill_hash_ctrl but didn't flash
+ * @note: If the caller called ena_com_fill_hash_ctrl but didn't flash
  * it to the device, the new configuration will be lost.
  *
@@ -770,5 +770,5 @@ int ena_com_indirect_table_set(struct ena_com_dev *ena_dev);
  * Retrieve the RSS indirection table from the device.
  *
- * @note: If the caller called ena_com_indirect_table_fill_entry but didn't flash
+ * @note: If the caller called ena_com_indirect_table_fill_entry but didn't flush
  * it to the device, the new configuration will be lost.
  *
@@ -796,5 +796,5 @@ int ena_com_allocate_debug_area(struct ena_com_dev *ena_dev,
  * @ena_dev: ENA communication layer struct
  *
- * Free the allocate debug area.
+ * Free the allocated debug area.
  */
 void ena_com_delete_debug_area(struct ena_com_dev *ena_dev);
@@ -803,5 +803,5 @@ void ena_com_delete_debug_area(struct ena_com_dev *ena_dev);
  * @ena_dev: ENA communication layer struct
  *
- * Free the allocate host info.
+ * Free the allocated host info.
  */
 void ena_com_delete_host_info(struct ena_com_dev *ena_dev);
@@ -844,7 +844,7 @@ int ena_com_destroy_io_cq(struct ena_com_dev *ena_dev,
  * @cmd_comp_size: command completion size.
 
- * Submit an admin command and then wait until the device will return a
+ * Submit an admin command and then wait until the device returns a
  * completion.
- * The completion will be copyed into cmd_comp.
+ * The completion will be copied into cmd_comp.
  *
  * @return - 0 on success, negative value on failure.
@@ -1028,5 +1028,5 @@ static inline void ena_com_calculate_interrupt_delay(struct ena_com_dev *ena_dev
  * @rx_delay_interval: Rx interval in usecs
  * @tx_delay_interval: Tx interval in usecs
- * @unmask: unask enable/disable
+ * @unmask: unmask enable/disable
  *
  * Prepare interrupt update register with the supplied parameters.
-- 
2.21.3

---
  Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- -	2020-05-28 17:13:02.515704682 +0100
+++ 0066-net-ena-base-fix-documentation-of-functions.patch	2020-05-28 17:12:59.144555427 +0100
@@ -1 +1 @@
-From b118993abd4cae7af3f8a66e0ad4fda1bee84c09 Mon Sep 17 00:00:00 2001
+From a1c41ac2eba79d42bba4c52f33126b2a45b91cc2 Mon Sep 17 00:00:00 2001
@@ -5,0 +6,2 @@
+[ upstream commit b118993abd4cae7af3f8a66e0ad4fda1bee84c09 ]
+
@@ -18 +20 @@
-index 07f63f44af..6c9943df79 100644
+index f58cd86a87..2e80ad563f 100644
@@ -21 +23 @@
-@@ -371,5 +371,5 @@ extern "C" {
+@@ -381,5 +381,5 @@ extern "C" {
@@ -28 +30 @@
-@@ -505,5 +505,5 @@ void ena_com_set_admin_auto_polling_mode(struct ena_com_dev *ena_dev,
+@@ -507,5 +507,5 @@ bool ena_com_get_ena_admin_polling_mode(struct ena_com_dev *ena_dev);
@@ -35 +37 @@
-@@ -515,5 +515,5 @@ void ena_com_admin_q_comp_intr_handler(struct ena_com_dev *ena_dev);
+@@ -517,5 +517,5 @@ void ena_com_admin_q_comp_intr_handler(struct ena_com_dev *ena_dev);
@@ -42 +44 @@
-@@ -532,5 +532,5 @@ void ena_com_abort_admin_commands(struct ena_com_dev *ena_dev);
+@@ -534,5 +534,5 @@ void ena_com_abort_admin_commands(struct ena_com_dev *ena_dev);
@@ -49 +51 @@
-@@ -539,5 +539,5 @@ void ena_com_wait_for_abort_completion(struct ena_com_dev *ena_dev);
+@@ -541,5 +541,5 @@ void ena_com_wait_for_abort_completion(struct ena_com_dev *ena_dev);
@@ -56 +58 @@
-@@ -716,5 +716,5 @@ int ena_com_set_hash_ctrl(struct ena_com_dev *ena_dev);
+@@ -718,5 +718,5 @@ int ena_com_set_hash_ctrl(struct ena_com_dev *ena_dev);
@@ -63 +65 @@
-@@ -768,5 +768,5 @@ int ena_com_indirect_table_set(struct ena_com_dev *ena_dev);
+@@ -770,5 +770,5 @@ int ena_com_indirect_table_set(struct ena_com_dev *ena_dev);
@@ -70 +72 @@
-@@ -794,5 +794,5 @@ int ena_com_allocate_debug_area(struct ena_com_dev *ena_dev,
+@@ -796,5 +796,5 @@ int ena_com_allocate_debug_area(struct ena_com_dev *ena_dev,
@@ -77 +79 @@
-@@ -801,5 +801,5 @@ void ena_com_delete_debug_area(struct ena_com_dev *ena_dev);
+@@ -803,5 +803,5 @@ void ena_com_delete_debug_area(struct ena_com_dev *ena_dev);
@@ -84 +86 @@
-@@ -842,7 +842,7 @@ int ena_com_destroy_io_cq(struct ena_com_dev *ena_dev,
+@@ -844,7 +844,7 @@ int ena_com_destroy_io_cq(struct ena_com_dev *ena_dev,
@@ -94 +96 @@
-@@ -933,5 +933,5 @@ static inline void ena_com_disable_adaptive_moderation(struct ena_com_dev *ena_d
+@@ -1028,5 +1028,5 @@ static inline void ena_com_calculate_interrupt_delay(struct ena_com_dev *ena_dev



More information about the stable mailing list