Skip to content
Snippets Groups Projects
Commit ac8ea675 authored by Davide Mor's avatar Davide Mor
Browse files

[SX1278] Remove TRACE and changed it with LOG_ERR

parent b9553359
No related branches found
No related tags found
1 merge request!240[SX1278] Remove TRACE and changed it with LOG_ERR
Pipeline #7878 passed
......@@ -63,9 +63,15 @@ bool SX1278Fsk::checkVersion()
SPITransaction spi(getSpiSlave());
uint8_t version = spi.readRegister(REG_VERSION);
TRACE("[sx1278] Chip id: %d\n", version);
return version == 0x12;
if (version == 0x12)
{
return true;
}
else
{
LOG_ERR(logger, "Wrong chip id: {}", version);
return false;
}
}
SX1278Fsk::Error SX1278Fsk::configure(const Config &config)
......
......@@ -212,7 +212,7 @@ private:
bool crc_enabled;
long long last_tx = 0;
float last_rx_rssi = 0.0f;
PrintLogger logger = Logging::getLogger("sx1278");
PrintLogger logger = Logging::getLogger("sx1278-fsk");
};
} // namespace Boardcore
......@@ -135,7 +135,6 @@ SX1278Lora::Error SX1278Lora::init(const Config &config)
// First probe for the device
if (!checkVersion())
{
TRACE("[sx1278] Wrong chipid\n");
return Error::BAD_VALUE;
}
......@@ -152,9 +151,15 @@ bool SX1278Lora::checkVersion()
SPITransaction spi(getSpiSlave());
uint8_t version = spi.readRegister(REG_VERSION);
TRACE("[sx1278] Chip id: %d\n", version);
return version == 0x12;
if (version == 0x12)
{
return true;
}
else
{
LOG_ERR(logger, "Wrong chip id: {}", version);
return false;
}
}
SX1278Lora::Error SX1278Lora::configure(const Config &config)
......
......@@ -22,6 +22,8 @@
#pragma once
#include <diagnostic/PrintLogger.h>
#include "SX1278Common.h"
#include "SX1278LoraTimings.h"
......@@ -203,9 +205,8 @@ private:
void setMode(Mode mode) override;
void setMapping(SX1278::DioMapping mapping) override;
void setFreqRF(int freq_rf);
bool crc_enabled;
PrintLogger logger = Logging::getLogger("sx1278-lora");
};
} // namespace Boardcore
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment