commit 28a2ca4bbe235a3a4d5838368a405972101040de
author Jayanth Othayoth <ojayanth@gmail.com> Fri Dec 13 05:12:31 2024 -0600
committer Jayanth Othayoth <ojayanth@gmail.com> Mon Jan 06 06:54:18 2025 +0000
tree 73f6ea9b40f829b3ea3e93b2a079af9df2eb95c9
parent 5ed2eb4c375c7037c95a4a635c180af60eac28e5

clang-tidy: Remove unused private field

The following errors were reported during clang-tidy enablement due
to unused private field.

'''
 tools/i2c/i2c.hpp:43:13: error: private field 'busId' is not used [-Werror
'''

The fix involves removing these unused private field 'busId'.

Tested: Build and unit testing verified.

Change-Id: I1fac353707e250ff661d4f42abdb1a2e9bb3a689
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>

tools/i2c/i2c.hpp

diff --git a/tools/i2c/i2c.hpp b/tools/i2c/i2c.hpp
index 4e43065..0f745c3 100644
--- a/tools/i2c/i2c.hpp
+++ b/tools/i2c/i2c.hpp
@@ -24,7 +24,7 @@
     explicit I2CDevice(uint8_t busId, uint8_t devAddr,
                        InitialState initialState = InitialState::OPEN,
                        int maxRetries = 0) :
-        busId(busId), devAddr(devAddr), maxRetries(maxRetries),
+        devAddr(devAddr), maxRetries(maxRetries),
         busStr("/dev/i2c-" + std::to_string(busId))
     {
         if (initialState == InitialState::OPEN)
@@ -39,9 +39,6 @@
     /** @brief Empty adapter functionality value with no bit flags set */
     static constexpr unsigned long NO_FUNCS = 0;
 
-    /** @brief The I2C bus ID */
-    uint8_t busId;
-
     /** @brief The i2c device address in the bus */
     uint8_t devAddr;