Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feature/add hpmicro device support #46329

Conversation

willChuai
Copy link

This PR adds support for HPMicro HPM6750 SoC and hpm6750evkmini board that is use the SoC.

  • Add HPM6750 SoC support.
  • Add HPM6360 SoC support.
  • Add HPM6750EVKMINI support
  • Drivers: Serial, GPIO,Pinctrl

hal_hpmicro module request:
#46301

@zephyrbot
Copy link
Collaborator

zephyrbot commented Jun 7, 2022

The following west manifest projects have been modified in this Pull Request:

Name Old Revision New Revision Diff
hal_hpmicro N/A zephyrproject-rtos/hal_hpmicro#2 zephyrproject-rtos/hal_hpmicro#2/files

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@zephyrbot zephyrbot added area: RISCV RISCV Architecture (32-bit & 64-bit) area: Pinctrl area: GPIO area: UART Universal Asynchronous Receiver-Transmitter labels Jun 7, 2022
@willChuai willChuai force-pushed the feature/add_hpmicro_device_support branch 2 times, most recently from 05f33cb to e9cfec4 Compare June 8, 2022 02:38
reg = <0x1180000 DT_SIZE_K(256)>;
};

noncache: memory@11c0000 {
Copy link
Contributor

@Cherish-Gww Cherish-Gww Jul 31, 2023

Choose a reason for hiding this comment

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

I don't think it's necessary to define a noncache node.
Using zephyr/arch/common/nocache.ld instead, it will automatic allocate on need. I have tested by using nocache.ld in ethernet driver.

select ATOMIC_OPERATIONS_BUILTIN
select INCLUDE_RESET_VECTOR
select RISCV_ISA_EXT_M
select RISCV_ISA_EXT_A
Copy link
Contributor

@Cherish-Gww Cherish-Gww Aug 3, 2023

Choose a reason for hiding this comment

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

According to the user manual description SOC_HPM6700 and SOC_HPM6360 support:
RV32I, M, A, F, D, C, P(SIMD and DSP)
so you have lost:
RISCV_ISA_EXT_F
RISCV_ISA_EXT_D
RISCV_ISA_EXT_C

This is related to link the libraries from the toolchain.

@github-actions
Copy link

github-actions bot commented Oct 4, 2023

This pull request has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 14 days. Note, that you can always re-open a closed pull request at any time.

@github-actions github-actions bot added the Stale label Oct 4, 2023
@ddavidebor
Copy link

ddavidebor commented Oct 5, 2023

Hey HPMicro! Just wanted to let you know we're excited about this PR at my company, and we look forward to buy lots and lots of your microcontrollers !

(if no Zephyr support, we will use micros from other companies)

@github-actions github-actions bot removed the Stale label Oct 6, 2023
@Cherish-Gww
Copy link
Contributor

Cherish-Gww commented Oct 19, 2023

I organized this PR here: #64117
@ddavidebor You can follow this PR

@Cherish-Gww
Copy link
Contributor

Hey HPMicro! Just wanted to let you know we're excited about this PR at my company, and we look forward to buy lots and lots of your microcontrollers !

(if no Zephyr support, we will use micros from other companies)

I am using hpmicro chip too, I have debugged the vast majority of peripheral drivers.

@willChuai willChuai dismissed stale reviews from cfriedt and fkokosinski via f80c458 October 30, 2023 03:55
@willChuai willChuai force-pushed the feature/add_hpmicro_device_support branch from f6cf1b8 to f80c458 Compare October 30, 2023 03:55
@@ -34,4 +34,8 @@ zephyr_library_sources_ifdef(CONFIG_PINCTRL_TI_K3 pinctrl_ti_k3.c)
zephyr_library_sources_ifdef(CONFIG_PINCTRL_EMSDP pinctrl_emsdp.c)
zephyr_library_sources_ifdef(CONFIG_PINCTRL_TI_CC32XX pinctrl_ti_cc32xx.c)
zephyr_library_sources_ifdef(CONFIG_PINCTRL_NUMAKER pinctrl_numaker.c)
<<<<<<< HEAD
Copy link
Contributor

Choose a reason for hiding this comment

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

Conflict introduced

zephyr_library_sources_ifdef(CONFIG_PINCTRL_QUICKLOGIC_EOS_S3 pinctrl_eos_s3.c)
=======
Copy link
Contributor

Choose a reason for hiding this comment

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

Please keep them all

drivers/pinctrl/Kconfig Outdated Show resolved Hide resolved
- add hal hpmicro hpm_sdk software packages to zephyr

Signed-off-by: Yawei CHUAI <[email protected]>
- add hpmicro series mcu to zephyr

Signed-off-by: Yawei CHUAI <[email protected]>
- add hpmicro company name to vendor-prefixes

Signed-off-by: Yawei CHUAI <[email protected]>
- add support for hpmicro hpm6750

Signed-off-by: Yawei CHUAI <[email protected]>

6750
- add support for hpmicro hpm6360

Signed-off-by: Yawei CHUAI <[email protected]>

6360
- Adding HPMicro hpm6750 device tree support

Signed-off-by: Yawei CHUAI <[email protected]>
- add hpmicro series mcu clock driver
- add clock dtsi

Signed-off-by: Yawei CHUAI <[email protected]>
- add pinctrl support for hpmicro
- add pinctrl dtsi

Signed-off-by: Yawei CHUAI <[email protected]>

pinctrl
- add hpm-uart driver
- add uart dtsi

Signed-off-by: Yawei CHUAI <[email protected]>
- add hpm-gpio driver
- add gpio dtsi

Signed-off-by: Yawei CHUAI <[email protected]>
- add support for serial
- add support for gpio
- add usage instructions

Signed-off-by: Yawei CHUAI <[email protected]>
- hpm6750evkmini gpio test passed

Signed-off-by: Yawei CHUAI <[email protected]>
@willChuai willChuai force-pushed the feature/add_hpmicro_device_support branch from f80c458 to 2297476 Compare October 31, 2023 07:14
Copy link

This pull request has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 14 days. Note, that you can always re-open a closed pull request at any time.

@github-actions github-actions bot added the Stale label Dec 31, 2023
@github-actions github-actions bot closed this Jan 14, 2024
@willChuai
Copy link
Author

I will create a new pull request because the current pull request process is taking too long and is too confusing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: ADC Analog-to-Digital Converter (ADC) area: Boards area: Clock Control area: Devicetree Binding PR modifies or adds a Device Tree binding area: Devicetree area: Documentation area: GPIO area: I2C area: Pinctrl area: PWM Pulse Width Modulation area: RISCV RISCV Architecture (32-bit & 64-bit) area: UART Universal Asynchronous Receiver-Transmitter DNM This PR should not be merged (Do Not Merge) manifest manifest-hal_hpmicro Stale
Projects
None yet
Development

Successfully merging this pull request may close these issues.