diff options
| author | Mario Six <mario.six@gdsys.cc> | 2019-03-29 10:18:13 +0100 |
|---|---|---|
| committer | Mario Six <mario.six@gdsys.cc> | 2019-05-21 08:03:38 +0200 |
| commit | ba75772fcbf32194db81c1907898657f84aa5182 (patch) | |
| tree | b14393ccd29e56f77ef1d41586bc497bfed61dc4 /board/gdsys | |
| parent | db1d03a299811267bcd0f38fe9cd0f833ab29cd9 (diff) | |
| download | u-boot-ba75772fcbf32194db81c1907898657f84aa5182.tar.gz | |
gdsys: cmd_ioloop: Introduce commenting enum
Replace the boolean parameter of io_check_status that controls whether
the status is printed or not with a documenting enum.
Signed-off-by: Mario Six <mario.six@gdsys.cc>
Diffstat (limited to 'board/gdsys')
| -rw-r--r-- | board/gdsys/common/cmd_ioloop.c | 13 |
1 files changed, 9 insertions, 4 deletions
diff --git a/board/gdsys/common/cmd_ioloop.c b/board/gdsys/common/cmd_ioloop.c index a53c80c0ad..8f70c1eda2 100644 --- a/board/gdsys/common/cmd_ioloop.c +++ b/board/gdsys/common/cmd_ioloop.c @@ -10,6 +10,11 @@ #include <gdsys_fpga.h> +enum status_print_type { + STATUS_LOUD = 0, + STATUS_SILENT = 1, +}; + enum { STATE_TX_PACKET_BUILDING = BIT(0), STATE_TX_TRANSMITTING = BIT(1), @@ -52,7 +57,7 @@ unsigned long long rx_ctr; unsigned long long tx_ctr; unsigned long long err_ctr; -static void io_check_status(uint fpga, u16 status, bool silent) +static void io_check_status(uint fpga, u16 status, enum status_print_type type) { u16 mask = STATE_RX_DIST_ERR | STATE_RX_LENGTH_ERR | STATE_RX_FRAME_CTR_ERR | STATE_RX_FCS_ERR | @@ -66,7 +71,7 @@ static void io_check_status(uint fpga, u16 status, bool silent) err_ctr++; FPGA_SET_REG(fpga, ep.rx_tx_status, status); - if (silent) + if (type == STATUS_SILENT) return; if (status & STATE_RX_PACKET_DROPPED) @@ -193,7 +198,7 @@ int do_ioreflect(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) FPGA_GET_REG(fpga, top_interrupt, &top_int); FPGA_GET_REG(fpga, ep.rx_tx_status, &rx_tx_status); - io_check_status(fpga, rx_tx_status, true); + io_check_status(fpga, rx_tx_status, STATUS_SILENT); if ((top_int & IRQ_CPU_RECEIVE_DATA_AVAILABLE_STATUS) && (top_int & IRQ_CPU_TRANSMITBUFFER_FREE_STATUS)) io_reflect(fpga); @@ -260,7 +265,7 @@ int do_ioloop(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) FPGA_GET_REG(fpga, top_interrupt, &top_int); FPGA_GET_REG(fpga, ep.rx_tx_status, &rx_tx_status); - io_check_status(fpga, rx_tx_status, false); + io_check_status(fpga, rx_tx_status, STATUS_LOUD); if (top_int & IRQ_CPU_TRANSMITBUFFER_FREE_STATUS) io_send(fpga, size); if (top_int & IRQ_CPU_RECEIVE_DATA_AVAILABLE_STATUS) |