~aleteoryx/muditaos

82eaee53591bc26a22c3e1fb70fcfb3c2eb599d1 — Paweł Joński 4 years ago 4175025
[BH-885] Fix Bell hw

Increased stack for apps and service
Deleted invalid call to uninitialized SML in BatteryCharger
Moved torch handling from common code to pure specific
M module-bsp/board/rt1051/bellpx/hal/battery_charger/BatteryCharger.cpp => module-bsp/board/rt1051/bellpx/hal/battery_charger/BatteryCharger.cpp +0 -1
@@ 17,7 17,6 @@ namespace hal::battery
        // Mocking initial state to make system run
        Store::Battery::modify().state = Store::Battery::State::Discharging;
        Store::Battery::modify().level = getBatteryVoltage();
        eventsHandler.onStatusChanged();
    }

    int BatteryCharger::getBatteryVoltage()

M module-services/service-evtmgr/EventManager.cpp => module-services/service-evtmgr/EventManager.cpp +0 -1
@@ 233,7 233,6 @@ sys::ReturnCodes EventManagerCommon::DeinitHandler()

void EventManagerCommon::ProcessCloseReason(sys::CloseReason closeReason)
{
    bsp::torch::turn(bsp::torch::State::off);
    sendCloseReadyMessage(this);
}


M products/BellHybrid/apps/application-bell-alarm/ApplicationBellAlarm.cpp => products/BellHybrid/apps/application-bell-alarm/ApplicationBellAlarm.cpp +3 -2
@@ 26,8 26,9 @@ namespace app
                                               std::string parent,
                                               sys::phone_modes::PhoneMode mode,
                                               sys::bluetooth::BluetoothMode bluetoothMode,
                                               StartInBackground startInBackground)
        : Application(std::move(name), std::move(parent), mode, bluetoothMode, startInBackground),
                                               StartInBackground startInBackground,
                                               uint32_t stackDepth)
        : Application(std::move(name), std::move(parent), mode, bluetoothMode, startInBackground, stackDepth),
          priv{std::make_unique<app::internal::BellAlarmPriv>()}
    {}
    ApplicationBellAlarm::~ApplicationBellAlarm() = default;

M products/BellHybrid/apps/application-bell-alarm/include/application-bell-alarm/ApplicationBellAlarm.hpp => products/BellHybrid/apps/application-bell-alarm/include/application-bell-alarm/ApplicationBellAlarm.hpp +2 -1
@@ 30,7 30,8 @@ namespace app
            std::string parent                          = "",
            sys::phone_modes::PhoneMode mode            = sys::phone_modes::PhoneMode::Offline,
            sys::bluetooth::BluetoothMode bluetoothMode = sys::bluetooth::BluetoothMode::Disabled,
            StartInBackground startInBackground         = {false});
            StartInBackground startInBackground         = {false},
            std::uint32_t stackDepth                    = 8192);

        ~ApplicationBellAlarm() override;
        sys::ReturnCodes InitHandler() override;

M products/BellHybrid/apps/application-bell-main/ApplicationBellMain.cpp => products/BellHybrid/apps/application-bell-main/ApplicationBellMain.cpp +3 -2
@@ 19,8 19,9 @@ namespace app
                                             std::string parent,
                                             sys::phone_modes::PhoneMode mode,
                                             sys::bluetooth::BluetoothMode bluetoothMode,
                                             StartInBackground startInBackground)
        : Application(name, parent, mode, bluetoothMode, startInBackground)
                                             StartInBackground startInBackground,
                                             std::uint32_t stackDepth)
        : Application(name, parent, mode, bluetoothMode, startInBackground, stackDepth)
    {
        bus.channels.push_back(sys::BusChannel::ServiceDBNotifications);
        addActionReceiver(manager::actions::ShowAlarm, [this](auto &&data) {

M products/BellHybrid/apps/application-bell-main/include/application-bell-main/ApplicationBellMain.hpp => products/BellHybrid/apps/application-bell-main/include/application-bell-main/ApplicationBellMain.hpp +2 -1
@@ 24,7 24,8 @@ namespace app
            std::string parent                          = "",
            sys::phone_modes::PhoneMode mode            = sys::phone_modes::PhoneMode::Offline,
            sys::bluetooth::BluetoothMode bluetoothMode = sys::bluetooth::BluetoothMode::Disabled,
            StartInBackground startInBackground         = {false});
            StartInBackground startInBackground         = {false},
            std::uint32_t stackDepth                    = 8192);

        sys::ReturnCodes InitHandler() override;


M products/PurePhone/services/evtmgr/EventManager.cpp => products/PurePhone/services/evtmgr/EventManager.cpp +6 -0
@@ 133,6 133,12 @@ void EventManager::toggleTorchColor()
    }
}

void EventManager::ProcessCloseReason(sys::CloseReason closeReason)
{
    bsp::torch::turn(bsp::torch::State::off);
    EventManagerCommon::ProcessCloseReason(closeReason);
}

sys::MessagePointer EventManager::DataReceivedHandler(sys::DataMessage *msgl, sys::ResponseMessage *resp)
{


M products/PurePhone/services/evtmgr/include/evtmgr/EventManager.hpp => products/PurePhone/services/evtmgr/include/evtmgr/EventManager.hpp +1 -0
@@ 20,6 20,7 @@ class EventManager : public EventManagerCommon
    sys::MessagePointer DataReceivedHandler(sys::DataMessage *msgl, sys::ResponseMessage *resp) override;
    void toggleTorchOnOff();
    void toggleTorchColor();
    void ProcessCloseReason(sys::CloseReason closeReason) override;
    void handleKeyEvent(sys::Message *msg) override;
    void handleKeyMoveEvent(RawKey key);