Skip to content

Conversation

@miguelmanguerra
Copy link

Copied motor controller 3 component and changed all the make files and etc to be torpedo board.

Copy link
Contributor

@cartercooper cartercooper left a comment

Choose a reason for hiding this comment

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

Make sure everything builds still and then you can merge this.

@@ -0,0 +1,11 @@
<?xml version="1.0" encoding="UTF-8"?>
Copy link
Contributor

Choose a reason for hiding this comment

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

Don't change this file

static const CAN_HWConfig_S CAN_HWConfig =
{
// GPIO
.rxPin = 8,
Copy link
Contributor

Choose a reason for hiding this comment

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

You are not using these pins for CAN in your schematic.

static const debug_HWConfig_S debug_HWConfig =
{
// GPIO
.rxPin = 12,
Copy link
Contributor

Choose a reason for hiding this comment

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

You are not using these pins for UART in your schematic.


[MESSAGE_HANDLER_RX_CHANNEL_TESTER] =
{
.messageID = protocol_MID_MC_deviceName,
Copy link
Contributor

Choose a reason for hiding this comment

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

This is the message ID for motor controller. Use the one for torpedo board. Create one if it doesn't exist.

.callbackEnable = true,
.initValue =
{
.MC_deviceName =
Copy link
Contributor

Choose a reason for hiding this comment

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

Change to Torpedo Board

{
[MESSAGE_HANDLER_TX_CHANNEL_RID] =
{
.messageID = protocol_MID_MC_deviceName,
Copy link
Contributor

Choose a reason for hiding this comment

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

Torpedo board

{
case MESSAGE_HANDLER_TX_CHANNEL_RID:
{
memcpy(message->MC_deviceName.name, "MTR_BRD", 7U);
Copy link
Contributor

Choose a reason for hiding this comment

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

Torpedo board

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.

4 participants