Capitalized local constexpr variables to be more consistent
This commit is contained in:
40
hardware.hpp
40
hardware.hpp
@@ -46,24 +46,24 @@ class Hardware {
|
||||
public:
|
||||
static void init() FORCE_INLINE
|
||||
{
|
||||
constexpr auto baudVal = calcBaud();
|
||||
constexpr auto BaudVal = calcBaud();
|
||||
|
||||
*getRegPtr<Registers::BAUD_REG_H_ADDR>() = static_cast<uint8_t>(baudVal >> 8);
|
||||
*getRegPtr<Registers::BAUD_REG_L_ADDR>() = static_cast<uint8_t>(baudVal);
|
||||
*getRegPtr<Registers::BAUD_REG_H_ADDR>() = static_cast<uint8_t>(BaudVal >> 8);
|
||||
*getRegPtr<Registers::BAUD_REG_L_ADDR>() = static_cast<uint8_t>(BaudVal);
|
||||
|
||||
constexpr auto dataBitsVal = calcDataBits();
|
||||
constexpr auto parityVal = calcParity();
|
||||
constexpr auto stopBitsVal = calcStopBits();
|
||||
constexpr auto modeVal = calcMode();
|
||||
constexpr auto enableRx = calcRxState<true>();
|
||||
constexpr auto enableTx = calcTxState<true>();
|
||||
constexpr auto interruptVal = calcInterrupt();
|
||||
constexpr auto DataBitsValues = calcDataBits();
|
||||
constexpr auto ParityVal = calcParity();
|
||||
constexpr auto StopBitsVal = calcStopBits();
|
||||
constexpr auto ModeVal = calcMode();
|
||||
constexpr auto EnableRx = calcRxState<true>();
|
||||
constexpr auto EnableTx = calcTxState<true>();
|
||||
constexpr auto InterruptVal = calcInterrupt();
|
||||
|
||||
constexpr uint8_t controlRegB = dataBitsVal.regBVal | enableRx | enableTx | interruptVal;
|
||||
constexpr uint8_t controlRegC = dataBitsVal.regCVal | parityVal | stopBitsVal | modeVal;
|
||||
constexpr uint8_t ControlRegB = DataBitsValues.regBVal | EnableRx | EnableTx | InterruptVal;
|
||||
constexpr uint8_t ControlRegC = DataBitsValues.regCVal | ParityVal | StopBitsVal | ModeVal;
|
||||
|
||||
*getRegPtr<Registers::CTRL_STAT_REG_B_ADDR>() = controlRegB;
|
||||
*getRegPtr<Registers::CTRL_STAT_REG_C_ADDR>() = controlRegC;
|
||||
*getRegPtr<Registers::CTRL_STAT_REG_B_ADDR>() = ControlRegB;
|
||||
*getRegPtr<Registers::CTRL_STAT_REG_C_ADDR>() = ControlRegC;
|
||||
}
|
||||
|
||||
static bool rxByteBlocking(typename cfg::data_t &byte) FORCE_INLINE
|
||||
@@ -137,8 +137,8 @@ class Hardware {
|
||||
static constexpr auto calcBaud()
|
||||
{
|
||||
// The actual formula is (F_CPU / (16 * baudRate)) - 1, but this one has the advantage of rounding correctly
|
||||
constexpr auto baudVal = (F_CPU + 8 * cfg::BAUD_RATE) / (16 * cfg::BAUD_RATE) - 1;
|
||||
return baudVal;
|
||||
constexpr auto BaudVal = (F_CPU + 8 * cfg::BAUD_RATE) / (16 * cfg::BAUD_RATE) - 1;
|
||||
return BaudVal;
|
||||
}
|
||||
|
||||
static constexpr auto calcDataBits()
|
||||
@@ -202,23 +202,23 @@ class Hardware {
|
||||
return modeVal;
|
||||
}
|
||||
|
||||
template <bool enable>
|
||||
template <bool Enable>
|
||||
static constexpr auto calcRxState()
|
||||
{
|
||||
uint8_t enableVal = 0;
|
||||
|
||||
if (enable)
|
||||
if (Enable)
|
||||
enableVal = (1 << CtrlFlagsB::RX_ENABLE);
|
||||
|
||||
return enableVal;
|
||||
}
|
||||
|
||||
template <bool enable>
|
||||
template <bool Enable>
|
||||
static constexpr auto calcTxState()
|
||||
{
|
||||
uint8_t enableVal = 0;
|
||||
|
||||
if (enable)
|
||||
if (Enable)
|
||||
enableVal = (1 << CtrlFlagsB::TX_ENABLE);
|
||||
|
||||
return enableVal;
|
||||
|
||||
Reference in New Issue
Block a user