Skip to content

Complete MCU services implementation#595

Open
ZeroSkill1 wants to merge 4 commits into
devkitPro:masterfrom
ZeroSkill1:mcu_impl
Open

Complete MCU services implementation#595
ZeroSkill1 wants to merge 4 commits into
devkitPro:masterfrom
ZeroSkill1:mcu_impl

Conversation

@ZeroSkill1
Copy link
Copy Markdown
Contributor

Implements all 9 MCU services and adds related definitions.

Comment thread libctru/source/services/mcuhid.c Outdated
return &mcuHidHandle;
}

Result MCUHID_SetSensorConfiguration(MCU_SensorConfig config)
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Are you sure this sets both accel on and pedometer on bits, or just accel on? I suspect it's just accel on

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It's indeed just accelerometer on, as it does & 1 when writing to the register.

Comment thread libctru/include/3ds/services/mcugpu.h Outdated
* @brief Gets the event handle for MCUGPU related events.
* @param out_event Pointer to output the event handle to.
*/
Result MCUGPU_GetEventHandle(Handle *out_event);
Copy link
Copy Markdown
Collaborator

@TuxSH TuxSH May 20, 2026

Choose a reason for hiding this comment

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

event -> interrupt?

Copy link
Copy Markdown
Contributor Author

@ZeroSkill1 ZeroSkill1 May 20, 2026

Choose a reason for hiding this comment

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

Yes, I originally based most of the naming on what I saw on 3dbrew.

#include <3ds/types.h>

/// SRV notifications published by the MCU system module.
enum MCU_InterruptNotification
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

typedef enum Name { } Name; (a few occurences of this)

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