commit f2a5fa2623e550fcb9d4d9ee4942d21e3c3c90c2
author Zbigniew Kurzynski <zbigniew.kurzynski@intel.com> Mon Nov 06 13:48:11 2023 +0100
committer Zbigniew Kurzynski <zbigniew.kurzynski@intel.com> Mon Nov 06 14:17:19 2023 +0100
tree c82a8d4fd17f5b792422cfac42b77b4501c3f64e
parent 0621dc0a3abf455846bad7f6a7f399b790b96082

Add sequenced version of peci_raw method

Most of libpeci methods have its sequenced version,
in case of intensive peci method usage it allows to
avoid unnecessary file opening and closing with each
function call.

This change adds such sequenced version of peci_raw
because for some reason it was not implemented before.

Tested: I have successfully use this new method with my
tesing application.

Change-Id: Id3d8e3459e8c5f17efc803b3847ff0ae700b74d1
Signed-off-by: Zbigniew Kurzynski <zbigniew.kurzynski@intel.com>

peci.c

diff --git a/peci.c b/peci.c
index ab69234..11434ac 100644
--- a/peci.c
+++ b/peci.c
@@ -1757,11 +1757,6 @@
                      uint32_t respSize)
 {
     int peci_fd = -1;
-    struct peci_xfer_msg cmd = {0};
-    uint8_t u8TxBuf[PECI_BUFFER_SIZE] = {0};
-    uint8_t u8RxBuf[PECI_BUFFER_SIZE] = {0};
-    EPECIStatus ret = PECI_CC_SUCCESS;
-
     if (u8ReadLen && pRawResp == NULL)
     {
         return PECI_CC_INVALID_REQ;
@@ -1778,12 +1773,40 @@
         return PECI_CC_DRIVER_ERR;
     }
 
+    EPECIStatus ret = peci_raw_seq(target, u8ReadLen, pRawCmd, cmdSize,
+                                   pRawResp, respSize, peci_fd);
+    peci_Close(peci_fd);
+    return ret;
+}
+
+/*-------------------------------------------------------------------------
+ *  This function provides sequential raw PECI command access
+ *------------------------------------------------------------------------*/
+EPECIStatus peci_raw_seq(uint8_t target, uint8_t u8ReadLen,
+                         const uint8_t* pRawCmd, const uint32_t cmdSize,
+                         uint8_t* pRawResp, uint32_t respSize, int peci_fd)
+{
+    struct peci_xfer_msg cmd = {0};
+    uint8_t u8TxBuf[PECI_BUFFER_SIZE] = {0};
+    uint8_t u8RxBuf[PECI_BUFFER_SIZE] = {0};
+    EPECIStatus ret = PECI_CC_SUCCESS;
+
+    if (u8ReadLen && pRawResp == NULL)
+    {
+        return PECI_CC_INVALID_REQ;
+    }
+
+    // The target address must be in the valid range
+    if (target < MIN_CLIENT_ADDR || target > MAX_CLIENT_ADDR)
+    {
+        return PECI_CC_INVALID_REQ;
+    }
+
     // Check for valid buffer sizes
     if (cmdSize > PECI_BUFFER_SIZE || respSize < u8ReadLen ||
         u8ReadLen >
             (PECI_BUFFER_SIZE - 1)) // response buffer is data + 1 status byte
     {
-        peci_Close(peci_fd);
         return PECI_CC_INVALID_REQ;
     }
 
@@ -1802,7 +1825,6 @@
         memcpy(pRawResp, u8RxBuf, u8ReadLen);
     }
 
-    peci_Close(peci_fd);
     return ret;
 }
 

peci.h

diff --git a/peci.h b/peci.h
index c2e0200..7aea0ae 100644
--- a/peci.h
+++ b/peci.h
@@ -430,6 +430,11 @@
                      const uint32_t cmdSize, uint8_t* pRawResp,
                      uint32_t respSize);
 
+// Provides sequential raw PECI command access
+EPECIStatus peci_raw_seq(uint8_t target, uint8_t u8ReadLen,
+                         const uint8_t* pRawCmd, const uint32_t cmdSize,
+                         uint8_t* pRawResp, uint32_t respSize, int peci_fd);
+
 EPECIStatus peci_Lock(int* peci_fd, int timeout_ms);
 void peci_Unlock(int peci_fd);
 EPECIStatus peci_Ping(uint8_t target);