bl808_linux icon indicating copy to clipboard operation
bl808_linux copied to clipboard

Checked in binaries for bflb_iot_tool, bflb_mcu_tool, genromfs, and openocd

Open robertlipe opened this issue 3 years ago • 2 comments

https://github.com/bouffalolab/bl808_linux/tree/main/bl_mcu_sdk_bl808/tools/bflb_flash_tool

has binaries checked in for . This is clearly a mistake as it'll never work on ChromeOS, Raspberry Pi, MacOS, self-hosting, or lots of other environments used by the kinds of people working with this kind of code.

PLEASE check in the source, not the binaries.

JLink has similar issues, but that might not be totally up to you. You should at least link out to the Segger page so people can download the "right" JLink.

Please. Just look in the tree for anything with dll, exe, or similar platform-specific artificacts. This bugreport applies to all of them.

bl_mcu_sdk_bl808 is another checked in binary. Source, pelase.

Also, the capitalization inside https://github.com/bouffalolab/bl808_linux/tree/main/bl_mcu_sdk_bl808/drivers/bl808_driver/risc-v is weird. Common convention is to not use upper case in filenames.

robertlipe avatar Oct 15 '22 07:10 robertlipe

Hey @robertlipe

https://github.com/bouffalolab/bl808_linux/tree/main/bl_mcu_sdk_bl808/tools/bflb_flash_tool

You can find sources here: https://pypi.org/project/bflb-mcu-tool/

csi_dsp blob

This is work of T-Head, and probably there is some NDA under it, so they can't release the source code.

gamelaster avatar Oct 15 '22 07:10 gamelaster

@robertlipe bflb_flash_tool is removed. Thanks.

YafeiJin avatar Oct 17 '22 06:10 YafeiJin