Skip to main content
Topic: How to contribute BPv4 development? (+ some bugfix) (Read 2048 times) previous topic - next topic

How to contribute BPv4 development? (+ some bugfix)

Hello everyone!

I found some bugs around in Bus Pirate v4 code and want to send patch for it.

First, It does not compile using MPLABX IDE. I searched forum a bit and heard that file is quite old.
So, I tried to fix file paths in project config and successfully managed to build 6.3-beta firmware.

Here is a diff of that fix.
https://github.com/perillamint/bus_pira ... 9425660806

Additionally, BPv4 gives false positive "Buffer overflow warning". I searched code a bit, and I found that warning is produced by this code.
https://github.com/perillamint/bus_pira ... ART.c#L262

Although this code works well on BPv3, which uses FTDI and UART to communicate with PC, BPv4 does not use FTDI and UART for communicating with PC. So warning criteria (U2BRG<U1BRG) will be always true. (BPv4 does not uses U2BRG)

I want to fix this but I don't have programming experience with PIC24, I don't know how to fix this false positive warning right now. (To fix this, I need to find which register provides USB baud rate and use it instead of U2BRG)

**EDIT:
patch:
https://github.com/perillamint/bus_pira ... d9e4415c58
I think this patch is enough. Since there are no "Baud rate" stuff exist in USB CDC and since USB CDC is fast enough then UART communication, disabling it when it is not in debug mode(debug mode uses AUX0,1 as UART port to communicate with PC)


Additionally, DP google code repository goes read only due to googles decision. Is there any other live public SVN repository?

p.s. Sorry for my poor English. (I'm not native English speaker)

Re: How to contribute BPv4 development? (+ some bugfix)

Reply #1
Thanks a lot for bugfixes. DP repo moved to github as far as I know so I believe you can access it over there.

Yeah, MPLAB IDE is dead, I did the switch mostly but keeping IDE for DP related stuff. I'm sure Ian will catch up soon. :)

Note: Deleted the previous post.