[dpdk-test-report] |WARNING| pw98397 [PATCH v2] net/ice: add 1PPS support for E810

[dpdk-test-report] |WARNING| pw98397 [PATCH v2] net/ice: add 1PPS support for E810

[checkpatch]

Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/98397

_coding style issues_


--- a/doc/guides/rel_notes/release_21_11.rst
+++ b/doc/guides/rel_notes/release_21_11.rst

[dpdk-test-report] |WARNING| pw98397 [PATCH] [v2] net/ice: add 1PPS support for E810

[dpdklab]

[-- Attachment #1: Type: text/plain, Size: 3235 bytes --]

Test-Label: iol-testing
Test-Status: WARNING
http://dpdk.org/patch/98397

_apply patch failure_

Submitter: Simei Su <simei.su@intel.com>
Date: Thursday, September 09 2021 01:30:49 
Applied on: CommitID:2508c18cff9ffb217ac40e44071dc2762d257f31
Apply patch set 98397 failed:

Checking patch doc/guides/nics/ice.rst...
error: while searching for:

  These ICE_DBG_XXX are defined in ``drivers/net/ice/base/ice_type.h``.

Driver compilation and testing
------------------------------


error: patch failed: doc/guides/nics/ice.rst:219
Checking patch doc/guides/rel_notes/release_21_11.rst...
error: doc/guides/rel_notes/release_21_11.rst: does not exist in index
Checking patch drivers/net/ice/ice_ethdev.c...
error: while searching for:
#define ICE_PIPELINE_MODE_SUPPORT_ARG  "pipeline-mode-support"
#define ICE_PROTO_XTR_ARG         "proto_xtr"
#define ICE_HW_DEBUG_MASK_ARG     "hw_debug_mask"

#define ICE_CYCLECOUNTER_MASK  0xffffffffffffffffULL

static const char * const ice_valid_args[] = {
	ICE_SAFE_MODE_SUPPORT_ARG,
	ICE_PIPELINE_MODE_SUPPORT_ARG,
	ICE_PROTO_XTR_ARG,
	ICE_HW_DEBUG_MASK_ARG,
	NULL
};


error: patch failed: drivers/net/ice/ice_ethdev.c:30
Hunk #2 succeeded at 1836 (offset 24 lines).
Hunk #3 succeeded at 1991 (offset 19 lines).
Hunk #4 succeeded at 2419 (offset -17 lines).
Hunk #5 succeeded at 2458 (offset -17 lines).
Hunk #6 succeeded at 3444 (offset -37 lines).
Hunk #7 succeeded at 3571 (offset -37 lines).
Checking patch drivers/net/ice/ice_ethdev.h...
Hunk #1 succeeded at 143 (offset -1 lines).
Hunk #2 succeeded at 156 (offset -1 lines).
Hunk #3 succeeded at 470 (offset -1 lines).
Hunk #4 succeeded at 480 (offset -1 lines).
Applying patch doc/guides/nics/ice.rst with 1 reject...
Rejected hunk #1.
Applying patch drivers/net/ice/ice_ethdev.c with 1 reject...
Rejected hunk #1.
Hunk #2 applied cleanly.
Hunk #3 applied cleanly.
Hunk #4 applied cleanly.
Hunk #5 applied cleanly.
Hunk #6 applied cleanly.
Hunk #7 applied cleanly.
Applied patch drivers/net/ice/ice_ethdev.h cleanly.
diff a/doc/guides/nics/ice.rst b/doc/guides/nics/ice.rst	(rejected hunks)
@@ -219,6 +219,14 @@ Runtime Config Options
 
   These ICE_DBG_XXX are defined in ``drivers/net/ice/base/ice_type.h``.
 
+- ``1PPS out support``
+
+  The E810 supports four single-ended GPIO signals (SDP[20:23]). The 1PPS
+  signal outputs via SDP[20:23]. User can select GPIO pin index flexibly.
+  Pin index 0 means SDP20, 1 means SDP21 and so on. For example::
+
+    -a af:00.0,pps_out='[pin:0]'
+
 Driver compilation and testing
 ------------------------------
 
diff a/drivers/net/ice/ice_ethdev.c b/drivers/net/ice/ice_ethdev.c	(rejected hunks)
@@ -30,14 +30,18 @@
 #define ICE_PIPELINE_MODE_SUPPORT_ARG  "pipeline-mode-support"
 #define ICE_PROTO_XTR_ARG         "proto_xtr"
 #define ICE_HW_DEBUG_MASK_ARG     "hw_debug_mask"
+#define ICE_ONE_PPS_OUT_ARG       "pps_out"
 
 #define ICE_CYCLECOUNTER_MASK  0xffffffffffffffffULL
 
+#define PPS_OUT_DELAY_NS  1
+
 static const char * const ice_valid_args[] = {
 	ICE_SAFE_MODE_SUPPORT_ARG,
 	ICE_PIPELINE_MODE_SUPPORT_ARG,
 	ICE_PROTO_XTR_ARG,
 	ICE_HW_DEBUG_MASK_ARG,
+	ICE_ONE_PPS_OUT_ARG,
 	NULL
 };
 

https://lab.dpdk.org/results/dashboard/patchsets/18633/

UNH-IOL DPDK Community Lab