llarm-emu clang_tidy fixes

Author: kernelwernel

llarm-emu/src/cpu/core/registers.hpp

@@ -84,8 +84,8 @@ struct REGISTERS {
 
     void write(const id::cpsr cpsr_id, const u8 cpsr_value);
     void write(const id::reg reg_id);
-    void write(const id::reg, const id::reg);
-    void write(const id::reg, const u32);
+    void write(const id::reg destination_reg_id, const id::reg source_reg_id);
+    void write(const id::reg register_id, const u32 value);
     void write(const u32 code, const u8 start, const u8 end, const u32 value);
     void write(const u8 reg_bits, const u32 value);
     void force_write(const id::reg register_id, const u32 value);

llarm-emu/src/cpu/instructions/operation.cpp

@@ -12,31 +12,31 @@ bool operation::carry_add(const u64 sum) {
     return (sum > max);
 }
 
-bool operation::carry_add(const u32 sum1, const u32 sum2) {
+bool operation::carry_add(const u32 a, const u32 b) {
     constexpr u64 max = ((1ULL << 32) - 1);
-    return ((sum1 + sum2) > max);
+    return ((a + b) > max);
 }
 
 // CHECK IF THIS WORKS TODO
-bool operation::carry_add(const u32 sum1, const u32 sum2, const u32 sum3) {
+bool operation::carry_add(const u32 a, const u32 b, const u32 c) {
     constexpr u64 max = ((1ULL << 32) - 1);
-    return ((sum1 + sum2 + sum3) > max);
+    return ((a + b + c) > max);
 }
 
 // TODO: COMPLETE
-bool operation::borrow_add(const u32 p1, const u32 p2) {
-    (void)p1;
-    (void)p2;
+bool operation::borrow_add(const u32 a, const u32 b) {
+    (void)a;
+    (void)b;
     return true; // TODO: COMPLETE THIS SHIT
 }
 
-bool operation::borrow_sub(const u32 p1, const u32 p2) {
-    return (p1 < p2);
+bool operation::borrow_sub(const u32 a, const u32 b) {
+    return (a < b);
 }
 
 // CHECK IF THIS WORKS
-bool operation::borrow_sub(const u32 p1, const u32 p2, const u32 p3) {
-    return (p1 < p2) || (p1 - p2 < p3);
+bool operation::borrow_sub(const u32 a, const u32 b, const u32 c) {
+    return (a < b) || (a - b < c);
 }
 
 bool operation::overflow_add(const u32 a, const u32 b) {

llarm-emu/src/cpu/instructions/operation.hpp

@@ -7,21 +7,21 @@
 #include <llarm/shared/types.hpp>
 
 namespace operation {
-    bool carry_add(const u64);
-    bool carry_add(const u32, const u32);
-    bool carry_add(const u32, const u32, const u32);
+    bool carry_add(const u64 sum);
+    bool carry_add(const u32 a, const u32 b);
+    bool carry_add(const u32 a, const u32 b, const u32 c);
 
-    bool borrow_add(const u32, const u32);
-    bool borrow_sub(const u32, const u32);
-    bool borrow_sub(const u32, const u32, const u32);
+    bool borrow_add(const u32 a, const u32 b);
+    bool borrow_sub(const u32 a, const u32 b);
+    bool borrow_sub(const u32 a, const u32 b, const u32 c);
 
-    bool overflow_add(const u32, const u32, const u32);
-    bool overflow_add(const u32, const u32);
-    bool overflow_sub(const u32, const u32);
-    bool overflow_sub(const u32, const u32, const u32);
+    bool overflow_add(const u32 a, const u32 b, const u32 c);
+    bool overflow_add(const u32 a, const u32 b);
+    bool overflow_sub(const u32 a, const u32 b);
+    bool overflow_sub(const u32 a, const u32 b, const u32 c);
 
-    bool signed_overflow_sub(const i32, const i32);
-    bool signed_overflow_add(const i32, const i32);
+    bool signed_overflow_sub(const i32 a, const i32 b);
+    bool signed_overflow_add(const i32 a, const i32 b);
 
     i32 sign_extend(const u32 value, const u8 sign_index = 31);
     i32 signed_sat(const u32 x, const u32 n);

llarm-emu/src/peripherals/uart/uart.cpp

@@ -159,11 +159,11 @@ void UART::write(const u32 address, const u32 value) {
             return;
         }
 
-        case OFFSET_UARTRSR:   // UARTECR on write: any write clears all error bits
+        case OFFSET_UARTRSR: // UARTECR on write: any write clears all error bits
             rsr = 0;
             return;
 
-        case OFFSET_UARTFR: return;  // read-only
+        case OFFSET_UARTFR: return; // read-only
 
         case OFFSET_UARTILPR: ilpr = value & 0xFFU; return;
         case OFFSET_UARTIBRD: ibrd = value & 0xFFFFU; return;

llarm-emu/src/peripherals/uart/uart.hpp

@@ -105,20 +105,20 @@ struct UART {
 
     // TX FIFO: 32 x 8-bit entries
     std::queue<u8>  tx_fifo;
-    // RX FIFO: 32 x 12-bit entries — bits [11:8] = error flags, bits [7:0] = data
+    // RX FIFO: 32 x 12-bit entries, bits [11:8] = error flags, bits [7:0] = data
     std::queue<u16> rx_fifo;
 
     // registers with TRM reset values
-    u32 rsr = 0x00;    // UARTRSR — receive status / error bits
-    u32 ilpr = 0x00;    // UARTILPR — IrDA low-power counter
-    u32 ibrd = 0x0000;  // UARTIBRD — integer baud rate divisor
-    u32 fbrd = 0x00;    // UARTFBRD — fractional baud rate divisor
-    u32 lcr_h = 0x00;    // UARTLCR_H — line control
-    u32 cr = 0x0300;  // UARTCR — control register (TXE=1, RXE=1, UARTEN=0)
-    u32 ifls = 0x12;    // UARTIFLS — interrupt FIFO level select (TX=1/2, RX=1/2)
-    u32 imsc = 0x000;   // UARTIMSC — interrupt mask
-    u32 ris = 0x000;   // UARTRIS — raw interrupt status
-    u32 dmacr = 0x00;    // UARTDMACR — DMA control
+    u32 rsr = 0x00; // UARTRSR: receive status / error bits
+    u32 ilpr = 0x00; // UARTILPR: IrDA low-power counter
+    u32 ibrd = 0x0000; // UARTIBRD: integer baud rate divisor
+    u32 fbrd = 0x00; // UARTFBRD: fractional baud rate divisor
+    u32 lcr_h = 0x00; // UARTLCR_H: line control
+    u32 cr = 0x0300; // UARTCR: control register (TXE=1, RXE=1, UARTEN=0)
+    u32 ifls = 0x12; // UARTIFLS: interrupt FIFO level select (TX=1/2, RX=1/2)
+    u32 imsc = 0x000; // UARTIMSC: interrupt mask
+    u32 ris = 0x000; // UARTRIS: raw interrupt status
+    u32 dmacr = 0x00; // UARTDMACR: DMA control
 
     u32 flag_register() const;
     u8 tx_trigger_level() const;