From b581d14823fd580432dfe850d9fe689c34bb219f Mon Sep 17 00:00:00 2001 From: MCUdude Date: Sun, 23 Jan 2022 22:44:50 +0100 Subject: [PATCH] String formatting Allign colons, only print clocks that are actually present (>0 kHz) --- src/jtag3.c | 34 +++++++++++++++++++++++----------- 1 file changed, 23 insertions(+), 11 deletions(-) diff --git a/src/jtag3.c b/src/jtag3.c index 4f22d824..b5008d01 100644 --- a/src/jtag3.c +++ b/src/jtag3.c @@ -2364,8 +2364,8 @@ static void jtag3_display(PROGRAMMER * pgm, const char * p) memmove(resp, resp + 3, status - 3); resp[status - 3] = 0; - avrdude_message(MSG_INFO, "%sICE hardware version: %d\n", p, parms[0]); - avrdude_message(MSG_INFO, "%sICE firmware version: %d.%02d (rel. %d)\n", p, + avrdude_message(MSG_INFO, "%sICE HW version : %d\n", p, parms[0]); + avrdude_message(MSG_INFO, "%sICE FW version : %d.%02d (rel. %d)\n", p, parms[1], parms[2], (parms[3] | (parms[4] << 8))); avrdude_message(MSG_INFO, "%sSerial number : %s", p, resp); @@ -2381,27 +2381,39 @@ static void jtag3_print_parms1(PROGRAMMER * pgm, const char * p) return; avrdude_message(MSG_INFO, "%sVtarget : %.2f V\n", p, - b2_to_u16(buf) / 1000.0); + b2_to_u16(buf) / 1000.0); if (jtag3_getparm(pgm, SCOPE_AVR, 1, PARM3_CLK_MEGA_PROG, buf, 2) < 0) return; - avrdude_message(MSG_INFO, "%sJTAG clock megaAVR/program: %u kHz\n", p, - b2_to_u16(buf)); + + if (b2_to_u16(buf) > 0) { + avrdude_message(MSG_INFO, "%sJTAG clock megaAVR/program : %u kHz\n", p, + b2_to_u16(buf)); + } if (jtag3_getparm(pgm, SCOPE_AVR, 1, PARM3_CLK_MEGA_DEBUG, buf, 2) < 0) return; - avrdude_message(MSG_INFO, "%sJTAG clock megaAVR/debug: %u kHz\n", p, - b2_to_u16(buf)); + + if (b2_to_u16(buf) > 0) { + avrdude_message(MSG_INFO, "%sJTAG clock megaAVR/debug : %u kHz\n", p, + b2_to_u16(buf)); + } if (jtag3_getparm(pgm, SCOPE_AVR, 1, PARM3_CLK_XMEGA_JTAG, buf, 2) < 0) return; - avrdude_message(MSG_INFO, "%sJTAG clock Xmega: %u kHz\n", p, - b2_to_u16(buf)); + + if (b2_to_u16(buf) > 0) { + avrdude_message(MSG_INFO, "%sJTAG clock Xmega : %u kHz\n", p, + b2_to_u16(buf)); + } if (jtag3_getparm(pgm, SCOPE_AVR, 1, PARM3_CLK_XMEGA_PDI, buf, 2) < 0) return; - avrdude_message(MSG_INFO, "%sPDI/UPDI clock Xmega : %u kHz\n\n", p, - b2_to_u16(buf)); + + if (b2_to_u16(buf) > 0) { + avrdude_message(MSG_INFO, "%sPDI/UPDI clock Xmega/megaAVR : %u kHz\n\n", p, + b2_to_u16(buf)); + } } static void jtag3_print_parms(PROGRAMMER * pgm)