Skip to content

Conversation

@helloeagleyang
Copy link
Contributor

  • added the hpm_sdk online package

@Rbb666 Rbb666 requested a review from Copilot September 19, 2025 12:54
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for the HPM SDK online package to the RT-Thread package system. The HPM SDK is a hardware abstraction layer SDK from HPMicro for RT-Thread integration.

  • Adds a new HPM SDK package configuration with version selection support
  • Includes package metadata with repository information and download URLs
  • Integrates the package into the existing HAL SDK menu structure

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
peripherals/hal-sdk/hpmicro/package.json Package metadata defining HPM SDK with repository URLs and version information
peripherals/hal-sdk/hpmicro/Kconfig Configuration options for HPM SDK package with version selection
peripherals/hal-sdk/Kconfig Integration of HPM SDK into the main HAL SDK configuration menu

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

},
"license": "BSD-3-Clause",
"repository": "https://github.com/hpmicro/rtt-pkg-hpm-sdk",
"icon": "unknown",
Copy link

Copilot AI Sep 19, 2025

Choose a reason for hiding this comment

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

[nitpick] The icon field is set to 'unknown'. Consider providing a proper icon URL or removing this field if no icon is available.

Suggested change
"icon": "unknown",

Copilot uses AI. Check for mistakes.
"repository": "https://github.com/hpmicro/rtt-pkg-hpm-sdk",
"icon": "unknown",
"homepage": "https://github.com/hpmicro/rtt-pkg-hpm-sdk#readme",
"doc": "unknown",
Copy link

Copilot AI Sep 19, 2025

Choose a reason for hiding this comment

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

[nitpick] The doc field is set to 'unknown'. Consider providing a proper documentation URL or removing this field if no documentation is available.

Suggested change
"doc": "unknown",

Copilot uses AI. Check for mistakes.
- added the hpm_sdk online package

Signed-off-by: Fan YANG <[email protected]>
@Rbb666 Rbb666 merged commit 7117818 into RT-Thread:master Sep 19, 2025
11 of 12 checks passed
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