Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for Raytac MDBT40 module #297

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

nhnifong
Copy link

@nhnifong nhnifong commented Aug 2, 2018

Copies the generic nrf51 and changes the pinout according to the spec for the MDBT40, the module that ships with the XYZrobot 6 DOF Robotic Arm Kit (https://www.pololu.com/product/2743)

I confirmed this allowed me to program one of those modules with an ST-Link V2, flash softdevice 110, and load the serial example. UART communication was working as expected.

@nhnifong nhnifong closed this Aug 2, 2018
@nhnifong
Copy link
Author

nhnifong commented Aug 2, 2018

The script builds when I run locally with
./arduino --verbose-build --verify --board sandeepmistry:nRF5:Raytac_MDBT40 examples/01.Basics/Blink/Blink.ino;

The relevant error from travis seems to be a linker error.

"/home/travis/.arduino15/packages/sandeepmistry/tools/gcc-arm-none-eabi/5_2-2015q4/bin/arm-none-eabi-gcc" "-L/tmp/arduino_build_751411" -Os -Wl,--gc-sections -save-temps "-L/home/travis/Arduino/hardware/sandeepmistry/nRF5/cores/nRF5/SDK/components/toolchain/gcc/" "-L/home/travis/Arduino/hardware/sandeepmistry/nRF5/cores/nRF5/SDK/components/softdevice/{softdevice}/toolchain/armgcc/" "-Tnrf51_{build.chip}.ld" "-Wl,-Map,/tmp/arduino_build_751411/Blink.ino.map" --specs=nano.specs --specs=nosys.specs -mcpu=cortex-m0 -mthumb -Wl,--cref -Wl,--check-sections -Wl,--gc-sections -Wl,--unresolved-symbols=report-all -Wl,--warn-common -Wl,--warn-section-align -o "/tmp/arduino_build_751411/Blink.ino.elf" "/tmp/arduino_build_751411/sketch/Blink.ino.cpp.o" "/tmp/arduino_build_751411/core/variant.cpp.o" -Wl,--start-group -lm "/tmp/arduino_build_751411/core/core.a" -Wl,--end-group
/home/travis/.arduino15/packages/sandeepmistry/tools/gcc-arm-none-eabi/5_2-2015q4/bin/../lib/gcc/arm-none-eabi/5.2.1/../../../../arm-none-eabi/bin/ld: cannot open linker script file nrf51_{build.chip}.ld: No such file or directory

@nhnifong nhnifong reopened this Aug 9, 2018
@dlabun
Copy link
Collaborator

dlabun commented Aug 9, 2018

It's failing in the Travis CI because you didn't specific a chip type in the config file.

@dlabun dlabun self-requested a review August 9, 2018 23:37
Copy link
Collaborator

@dlabun dlabun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Travis CI config incorrect, does not specify chip type.

@nhnifong
Copy link
Author

I've specified the chip for the test. thanks for the help.

@sandeepmistry
Copy link
Owner

@nhnifong @dlabun any thoughts on my comments from: #280 (comment)

I'm not sure what the advantage of having a separate variant from the generic is for a module. Variants are more intended for developer boards.

@dlabun
Copy link
Collaborator

dlabun commented Aug 19, 2018

@sandeepmistry Just replied on the #280 thread.

@dlabun dlabun mentioned this pull request Aug 31, 2018
*/

#ifndef _VARIANT_GENERIC_
#define _VARIANT_GENERIC_
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The above too lines need to be updated.

@nhnifong
Copy link
Author

I've made the small change. Sorry for not getting to this for two weeks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants