Skip to content

boards: add support for usbcan_iso, usbcanfd solo/dual,canbridge_g473 - #219

Open
AlekseyMamontov wants to merge 1 commit into
CANnectivity:mainfrom
AlekseyMamontov:main
Open

boards: add support for usbcan_iso, usbcanfd solo/dual,canbridge_g473#219
AlekseyMamontov wants to merge 1 commit into
CANnectivity:mainfrom
AlekseyMamontov:main

Conversation

@AlekseyMamontov

@AlekseyMamontov AlekseyMamontov commented Jun 27, 2026

Copy link
Copy Markdown

Hello Brix, @henrikbrixandersen ,

This pull request adds support for the following hardware targets:

usbcan_iso (STM32F072XB)
usbcanfd_solo (STM32G431XX)
usbcanfd_dual and canbridge_g473 (STM32G473XX)

Signed-off-by: Oleksii Mamontov mamontov1@gmail.com

@henrikbrixandersen henrikbrixandersen left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please address the review comments below and split into three commits, one for each board.

@@ -0,0 +1,6 @@
# Copyright (c) 2026 Oleksii Mamontov <mamontov1@gmail.com>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
# Copyright (c) 2026 Oleksii Mamontov <mamontov1@gmail.com>
# SPDX-FileCopyrightText: Copyright (c) 2026 Oleksii Mamontov <mamontov1@gmail.com>

@@ -0,0 +1,39 @@
/*
* Copyright (c) 2026 Oleksii Mamontov <mamontov1@gmail.com>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
* Copyright (c) 2026 Oleksii Mamontov <mamontov1@gmail.com>
* SPDX-FileCopyrightText: Copyright (c) 2026 Oleksii Mamontov <mamontov1@gmail.com>

Comment thread app/boards/usbcan_iso_stm32f072xb.conf Outdated
Comment on lines +1 to +2
# Copyright (c) 2024-2025 Henrik Brix Andersen <henrik@brixandersen.dk>
# Copyright (c) 2026 Oleksii Mamontov <mamontov1@gmail.com>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
# Copyright (c) 2024-2025 Henrik Brix Andersen <henrik@brixandersen.dk>
# Copyright (c) 2026 Oleksii Mamontov <mamontov1@gmail.com>
# SPDX-FileCopyrightText: Copyright (c) 2026 Oleksii Mamontov <mamontov1@gmail.com>

Comment on lines +1 to +2
/*
* Copyright (c) 2026 Oleksii Mamontov <mamontov1@gmail.com>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
/*
* Copyright (c) 2026 Oleksii Mamontov <mamontov1@gmail.com>
/*
* SPDX-FileCopyrightText: Copyright (c) 2026 Oleksii Mamontov <mamontov1@gmail.com>

@@ -0,0 +1,6 @@
# Copyright (c) 2026 Oleksii Mamontov <mamontov1@gmail.com>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
# Copyright (c) 2026 Oleksii Mamontov <mamontov1@gmail.com>
# SPDX-FileCopyrightText: Copyright (c) 2026 Oleksii Mamontov <mamontov1@gmail.com>

@@ -0,0 +1,33 @@
/*
* Copyright (c) 2026 Oleksii Mamontov <mamontov1@gmail.com>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
* Copyright (c) 2026 Oleksii Mamontov <mamontov1@gmail.com>
* SPDX-FileCopyrightText: Copyright (c) 2026 Oleksii Mamontov <mamontov1@gmail.com>

can-controller = <&fdcan1>;
activity-leds = <&led_rx &led_tx>;
};
channel1 {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Wrong indentation.

@@ -0,0 +1,6 @@
# Copyright (c) 2026 Oleksii Mamontov <mamontov1@gmail.com>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
# Copyright (c) 2026 Oleksii Mamontov <mamontov1@gmail.com>
# SPDX-FileCopyrightText: Copyright (c) 2026 Oleksii Mamontov <mamontov1@gmail.com>

@@ -0,0 +1,28 @@
/*
* Copyright (c) 2026 Oleksii Mamontov <mamontov1@gmail.com>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
* Copyright (c) 2026 Oleksii Mamontov <mamontov1@gmail.com>
* SPDX-FileCopyrightText: Copyright (c) 2026 Oleksii Mamontov <mamontov1@gmail.com>

Comment thread app/src/led.c

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please remove this change.

@AlekseyMamontov

Copy link
Copy Markdown
Author

Hello Brix, thanks for the review! ))
I've been on the road, but I'm back now and working on fixing these issues

… and boot animation

Signed-off-by: Oleksii <35818127+AlekseyMamontov@users.noreply.github.com>
@AlekseyMamontov AlekseyMamontov changed the title boards: add support for usbcan_iso, usbcanfd solo/dual,canbridge_g473 and animation boot boards: add support for usbcan_iso, usbcanfd solo/dual,canbridge_g473 Jul 28, 2026
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.

2 participants