syslog: print the server address with print_ip()

The status line built the dotted quad by hand, which is what
print_ip() from cmd_parser.c already does. Reusing it drops seven
calls to one and takes 48 bytes off BANK2.

The prototype is not __banked because cmd_parser.c and syslog.c are
both in BANK2; a plain call is what the compiler needs here. Should
either file move to another bank, this has to become __banked with it.
This commit is contained in:
d00f committed 2026-09-03 21:27:02 +02:00
1 parent 40ce742d04
commit 2e2529be27
2 files changed
+3 -2

No files matched your search

+1
View File
@@ -13,6 +13,7 @@ void cmd_tokenize(void) __banked;
void cmd_parser(void) __banked;
void execute_config(void) __banked;
void execute_commands(__xdata uint8_t *p) __banked;
void print_ip(__xdata uint8_t *ptr);
void print_sw_version(void) __banked;
void clear_command_history(void) __banked;
+2 -2
View File
@@ -1,5 +1,6 @@
#include "machine.h"
#include "syslog.h"
#include "cmd_parser.h"
#include "uip/uip.h"
#include "rtl837x_common.h"
@@ -35,8 +36,7 @@ void syslog_start(void) __banked
return;
}
print_string_newline_no_syslog("Started syslog to IP ");
itoa(state.server_ip[0]); write_char('.'); itoa(state.server_ip[1]); write_char('.');
itoa(state.server_ip[2]); write_char('.'); itoa(state.server_ip[3]);
print_ip(state.server_ip);
write_char(':'); itoa_short(state.server_port); write_char('\n');
state.enabled = 1;
}