quick: Fixing a few warnings

Fixing some warnings shown in GCC and Clang compilations. These are pretty minor and unlikely to cause a problem overall but the code should be as warning free as possible. Signed-off-by: Tyler Erickson <tyler.erickson@seagate.com>
Seagate · Aug 21, 2023 · 766adac · 766adac
1 parent 0a36c18
commit 766adac
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/subprojects/opensea-operations b/subprojects/opensea-operations
diff --git a/utils/C/openSeaChest/openSeaChest_Configure.c b/utils/C/openSeaChest/openSeaChest_Configure.c
@@ -33,7 +33,7 @@
 //  Global Variables  //
 ////////////////////////
 const char *util_name = "openSeaChest_Configure";
-const char *buildVersion = "2.4.4";
+const char *buildVersion = "2.4.5";
 
 ////////////////////////////
 //  functions to declare  //
@@ -1904,6 +1904,7 @@ int32_t main(int argc, char *argv[])
         if (ATA_DCO_DISABLE_FEATURES || ATA_DCO_SETMAXMODE || ATA_DCO_SETMAXLBA)
         {
             dcoData dco;
+            bool scsiAtaInSync = false;
             memset(&dco, 0, sizeof(dcoData));
             switch (dco_Identify(&deviceList[deviceIter], &dco))
             {
@@ -2104,7 +2105,6 @@ int32_t main(int argc, char *argv[])
                         dco.feat2.extendedPowerConditions = false;
                     }
                 }
-                bool scsiAtaInSync = false;
                 switch (dco_Set(&deviceList[deviceIter], &dco))
                 {
                 case SUCCESS:

diff --git a/utils/C/openSeaChest/openSeaChest_PassthroughTest.c b/utils/C/openSeaChest/openSeaChest_PassthroughTest.c
@@ -33,7 +33,7 @@
 //  Global Variables  //
 ////////////////////////
 const char *util_name = "openSeaChest_PassthroughTest";
-const char *buildVersion = "1.3.2";
+const char *buildVersion = "1.3.3";
 
 ////////////////////////////
 //  functions to declare  //
@@ -7128,7 +7128,7 @@ static bool test_SAT_Capabilities(ptrPassthroughTestParams inputs, ptrScsiDevInf
             set_Console_Colors(true, LIKELY_HACK_COLOR);
             printf("HACK FOUND: SPIO\n");
             set_Console_Colors(true, DEFAULT);
-            inputs->device->drive_info.passThroughHacks.ataPTHacks.singleSectorPIOOnly;
+            inputs->device->drive_info.passThroughHacks.ataPTHacks.singleSectorPIOOnly = true;
         }
 
         //In next tests, need to make sure we actually get response information that makes sense.