add 0xcb helios #1174

Closed
Conor-Burns wants to merge 1 commit from add-0xcb-helios into develop
Conor-Burns commented 2023-01-14 12:25:33 -06:00 (Migrated from github.com)

Adding 0xCB Helios

source

Adding 0xCB Helios [source](https://github.com/0xCB-dev/0xCB-Helios) ![](https://github.com/0xCB-dev/0xCB-Helios/raw/main/rev1.0/helios.webp)
kilograham (Migrated from github.com) reviewed 2023-01-19 16:21:00 -06:00
@ -0,0 +16,4 @@
#define _BOARDS_0XCB_HELIOS_H
// For board detection
#define 0XCB_HELIOS
kilograham (Migrated from github.com) commented 2023-01-19 16:21:00 -06:00

Um; that surely doesn't compile!

Um; that surely doesn't compile!
kilograham (Migrated from github.com) requested changes 2023-01-23 12:51:47 -06:00
kilograham (Migrated from github.com) left a comment

as per comment, this header won't compile

as per comment, this header won't compile
peterharperuk (Migrated from github.com) reviewed 2023-05-16 12:29:16 -05:00
@ -0,0 +16,4 @@
#define _BOARDS_0XCB_HELIOS_H
// For board detection
#define 0XCB_HELIOS
peterharperuk (Migrated from github.com) commented 2023-05-16 12:29:16 -05:00

error: macro names must be identifiers

error: macro names must be identifiers
kilograham commented 2023-05-25 15:06:05 -05:00 (Migrated from github.com)

please fix this if you want it added

please fix this if you want it added
kilograham commented 2024-05-19 17:34:51 -05:00 (Migrated from github.com)

As OP doesn't seem to be around, I suggest we just add an _ to the define and commit this (i would have done so in passing, but i can't edit the commit in the original repo)

As OP doesn't seem to be around, I suggest we just add an `_` to the define and commit this (i would have done so in passing, but i can't edit the commit in the original repo)
lurch commented 2024-06-05 13:31:17 -05:00 (Migrated from github.com)

As OP doesn't seem to be around, I suggest we just add an _ to the define and commit this (i would have done so in passing, but i can't edit the commit in the original repo)

Done that in #1724

> As OP doesn't seem to be around, I suggest we just add an `_` to the define and commit this (i would have done so in passing, but i can't edit the commit in the original repo) Done that in #1724
kilograham commented 2024-06-21 14:27:22 -05:00 (Migrated from github.com)

#1724 has been merged into develop

#1724 has been merged into develop

Pull request closed

Sign in to join this conversation.
No reviewers
No milestone
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
gh-raspberrypi/pico-sdk!1174
No description provided.