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

Add RTL8730E board revision 5th #6180

Merged
merged 2 commits into from May 13, 2024

Conversation

ewoodev
Copy link
Contributor

@ewoodev ewoodev commented May 10, 2024

No description provided.

os/board/Kconfig Outdated Show resolved Hide resolved
os/board/rtl8730e/src/rtl8730e_st7789.c Outdated Show resolved Hide resolved
@ewoodev ewoodev force-pushed the WORK-240510_add_revision branch 3 times, most recently from 8bf9acb to b70bf87 Compare May 13, 2024 07:42
@ewoodev ewoodev force-pushed the WORK-240510_add_revision branch 2 times, most recently from 46d0aab to 8342e28 Compare May 13, 2024 11:54
@@ -36,6 +36,13 @@ config BOARD_FLASH_16M

endchoice

config RTL8730E_REVISION
Copy link
Contributor

Choose a reason for hiding this comment

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

  1. RTL8730E_BOARD_REVISION?
  2. But this may make someone mis-understand the chipset revision....

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Changed!

@@ -36,6 +36,13 @@ config BOARD_FLASH_16M

endchoice

config RTL8730E_REVISION
int "Select target board revision version"
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's remove "version" here as well.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

int "Select target board revision version"
default 0
---help---
Select the board revision version.
Copy link
Contributor

Choose a reason for hiding this comment

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

same as before

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done~

@ewoodev ewoodev force-pushed the WORK-240510_add_revision branch 2 times, most recently from 29a7219 to 3368897 Compare May 13, 2024 12:09
The pins of the periphery may vary for each new board.
To adjust the pins of the periphery for each board, a CONFIG_RTL8730E_BOARD_REVISION configuration is added.

Signed-off-by: eunwoo.nam <eunwoo.nam@samsung.com>
Add the revised 5th board.
The pin of UART1 has been changed. The ALC1019 I2C port has been changed.

Signed-off-by: eunwoo.nam <eunwoo.nam@samsung.com>
@sunghan-chang sunghan-chang merged commit 5f99ade into Samsung:master May 13, 2024
11 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.

None yet

3 participants