Software/SystemK
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix for #12: Transition from "Configuring" to "Ready" due to "Configured" event not possible (#13)

Browse source 
This pull request fixes #12.

The root cause was [`BLE_HandleCommonEvents()`](https://git.ktag.clubk.club/Software/SystemK/src/tag/SystemK_v01.01/BLE/BLE_Utils.c#L57) was causing BLE events to be marked as "not new" by [`BLE_IsPacketNew()`](https://git.ktag.clubk.club/Software/SystemK/src/tag/SystemK_v01.01/BLE/BLE_Packet_Tracker.c#L53), even though these packets were left unhandled by the common event handler.

The solution is to separate the querying of whether a packet is new from the marking it as already seen, since we don't want to mark a packet as seen until it has been properly _handled_.  I added the function `BLE_RememberPacket()` for the latter purpose.

I also cleaned up some of the logging code, now that logging is configurable (on ESP32 platform, anyway).

Co-authored-by: Joe Kearney <joe@clubk.club>
Reviewed-on: #13
This commit is contained in:
Joe Kearney 2026-02-28 02:51:00 +00:00
parent 45df6f952a
commit daa1a2a1ec
6 changed files with 28 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

2
States/Playing/State_Playing.c
View file

@ -206,6 +206,8 @@ void HandleBLETagPacket(const BLE_TagPacket_T *const packet)
{
if (BLE_IsPacketNew(packet->BD_ADDR, BLE_PACKET_TYPE_TAG, packet->event_number))
{
BLE_RememberPacket(packet->BD_ADDR, BLE_PACKET_TYPE_TAG, packet->event_number);
Health_In_Percent = (int16_t)Get_Health();
if (packet->damage > 0)

2
States/State_Configuring.c
View file

@ -376,6 +376,8 @@ void HandleBLEConfigurationPacket(const BLE_ParametersPacket_T *const packet)
{
if (BLE_IsPacketNew(packet->BD_ADDR, BLE_PACKET_TYPE_PARAMETERS, packet->event_number))
{
BLE_RememberPacket(packet->BD_ADDR, BLE_PACKET_TYPE_PARAMETERS, packet->event_number);
if (packet->subtype == BLE_REQUEST_PARAMETER_CHANGE)
{
SystemKResult_T result = SYSTEMK_RESULT_SUCCESS;

10
States/State_Machine.c
View file

@ -135,6 +135,8 @@ void HandleBLEEventPacket(const BLE_EventPacket_T *const packet, StateMachineCon
{
if (BLE_IsPacketNew(packet->BD_ADDR, BLE_PACKET_TYPE_EVENT, packet->event_number))
{
BLE_RememberPacket(packet->BD_ADDR, BLE_PACKET_TYPE_EVENT, packet->event_number);
if (packet->event_ID == BLE_EVENT_CONFIGURED)
{
if (context->States.Current_State == STATE_CONFIGURING)
@ -146,6 +148,10 @@ void HandleBLEEventPacket(const BLE_EventPacket_T *const packet, StateMachineCon
};
Transition_For_Event(context, STATE_READY, &Event);
}
else
{
KLOG_WARN("STATE", "Ignoring a BLE Configured event--not in the Configuring State!");
}
}
else if (packet->event_ID == BLE_EVENT_CONFIGURE)
{
@ -158,6 +164,10 @@ void HandleBLEEventPacket(const BLE_EventPacket_T *const packet, StateMachineCon
};
Transition_For_Event(context, STATE_CONFIGURING, &Event);
}
else
{
KLOG_WARN("STATE", "Ignoring a BLE Configure event--not in the Ready State!");
}
}
else if (packet->event_ID == BLE_EVENT_WRAPUP_COMPLETE)
{