Skip to content

Commit cce8be3

Browse files
committed
initialize local variables before use
fixes error: 'ctrl_reg2_data' may be used uninitialized in this function [-Werror=maybe-uninitialized] Signed-off-by: Khem Raj <[email protected]>
1 parent 5cf20df commit cce8be3

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

src/kxtj3/kxtj3.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -715,7 +715,7 @@ upm_result_t kxtj3_sensor_software_reset(const kxtj3_context dev)
715715
if (kxtj3_set_bit_on(dev, KXTJ3_CTRL_REG2, KXTJ3_CTRL_REG2_SRST) != UPM_SUCCESS)
716716
return UPM_ERROR_OPERATION_FAILED;
717717

718-
uint8_t ctrl_reg2_data;
718+
uint8_t ctrl_reg2_data = 0;
719719
kxtj3_read_register(dev, KXTJ3_CTRL_REG2, &ctrl_reg2_data);
720720

721721
uint8_t srst_counter = 0;
@@ -824,7 +824,7 @@ upm_result_t kxtj3_enable_interrupt_pin(const kxtj3_context dev, KXTJ3_INTERRUPT
824824
KXTJ3_INTERRUPT_RESPONSE_T response_type)
825825
{
826826
assert(dev != NULL);
827-
uint8_t int_reg_value;
827+
uint8_t int_reg_value = 0;
828828
kxtj3_read_register(dev, KXTJ3_INT_CTRL_REG1, &int_reg_value);
829829

830830
if (polarity)
@@ -865,7 +865,7 @@ upm_result_t kxtj3_set_interrupt_response(const kxtj3_context dev, KXTJ3_INTERRU
865865
bool kxtj3_get_interrupt_status(const kxtj3_context dev)
866866
{
867867
assert(dev != NULL);
868-
uint8_t status_reg_value;
868+
uint8_t status_reg_value = 0;
869869
kxtj3_read_register(dev, KXTJ3_STATUS_REG, &status_reg_value);
870870
if (!(status_reg_value & KXTJ3_STATUS_REG_INT))
871871
return false;
@@ -955,7 +955,7 @@ kxtj3_wakeup_axes kxtj3_get_wakeup_axis_and_direction(kxtj3_context dev)
955955
{
956956
assert(dev != NULL);
957957

958-
uint8_t int_source2_value;
958+
uint8_t int_source2_value = 0;
959959
kxtj3_read_register(dev, KXTJ3_INT_SOURCE2, &int_source2_value);
960960

961961
kxtj3_wakeup_axes wakeup_axis;

0 commit comments

Comments
 (0)