debugprobe icon indicating copy to clipboard operation
debugprobe copied to clipboard

Add pin names to binary information

Open fduxiao opened this issue 1 year ago • 4 comments
trafficstars

I added the pin names to the binary information in case someone downloaded the firmware ten years ago and forget which pins to use.

fduxiao avatar Jan 26 '24 18:01 fduxiao

Can you update this PR on top of master and add UART_CTS as well?

P33M avatar Mar 18 '24 16:03 P33M

@P33M done as you wish. I also added baud rate and interface to features. Now the output looks like

$ picotool info debugprobe.uf2 -a
File debugprobe.uf2:

Program Information
 name:          debugprobe
 features:      PROBE UART BAUDRATE 115200
                PROBE UART INTERFACE uart1
                UART stdin / stdout
 binary start:  0x10000000
 binary end:    0x1000a2d8

Fixed Pin Information
 0:   UART0 TX, UART0 TX
 1:   UART0 RX
 4:   PROBE UART TX
 5:   PROBE UART RX
 12:  PROBE SWCLK
 13:  PROBE SWDI
 14:  PROBE SWDIO
 25:  LED

Build Information
 sdk version:       1.5.1
 pico_board:        pico
 boot2_name:        boot2_w25q080
 build date:        Mar 18 2024
 build attributes:  Release

Besides, I notice a naming difference in include/board_example_config.h.

/* PIO config for PROBE_IO_OEN - note that SWDIOEN and SWCLK are both side_set signals, so must be consecutive. */
#if defined(PROBE_IO_SWDIOEN)
#define PROBE_PIN_SWDIOEN (PROBE_PIN_OFFSET + 0)
#define PROBE_PIN_SWCLK (PROBE_PIN_OFFSET + 1)
#define PROBE_PIN_SWDIO (PROBE_PIN_OFFSET + 2)
#define PROBE_PIN_SWDI (PROBE_PIN_OFFSET + 3)
#endif

See the difference between PROBE_IO_SWDIOEN and PROBE_IO_OEN. Is this a mistake? I can open another pull request for it.

fduxiao avatar Mar 18 '24 21:03 fduxiao

The compile guard is wrong - it should be PROBE_IO_OEN. Please add another commit to this PR which fixes the typo.

P33M avatar Mar 19 '24 14:03 P33M

@P33M Fixed.

fduxiao avatar Mar 20 '24 00:03 fduxiao

Thanks, applied.

P33M avatar Mar 26 '24 19:03 P33M