Skip to content

Update board definitions for SC-OBC Module A1 #90564

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

Merged

Conversation

sasataku
Copy link
Contributor

@sasataku sasataku commented May 26, 2025

In the EM (Engineering Model), the board was referred to as Space Cubics Module 1.
Starting with the FM (Flight Model) release, we are officially renaming it to: SC-OBC Module A1
Accordingly, the board name is updated from scobc_module1 to scobc_a1.

In addition to renaming the board to SC-OBC Module A1, the following changes are made:

  • Updated UART IP core register addresses
  • Modified HRMEM (SRAM) size
  • Changed the number of IRQs
  • Updated FPGA System Register / System Monitor Register address to DTS

These changes are based on the specifications described in the following manual:
https://sc-obc-fpga-technical-reference-manual.readthedocs.io/ja/latest/

And we also decided to introduce a Revision, setting the EM to 1.0.0, the FM to 2.0.0, and using 2.0.0 as the default.

Copy link
Collaborator

@nordicjm nordicjm left a comment

Choose a reason for hiding this comment

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

should add a deprecated board entry in boards/deprecated.cmake

@sasataku sasataku force-pushed the update-board-info-for-scobc-a1 branch 7 times, most recently from 8ce593b to 17db4d6 Compare June 2, 2025 10:45
@sasataku sasataku marked this pull request as ready for review June 3, 2025 13:54
@sasataku sasataku requested a review from nordicjm June 4, 2025 11:03
nordicjm
nordicjm previously approved these changes Jun 4, 2025
@@ -21,7 +21,7 @@ Hardware
Supported Features
==================

The Space Cubics OBC module 1 provides the following hardware features:
The SC-OBC Module A1 provides the following hardware features:

+-----------+------------+------------------------------------+
Copy link
Collaborator

Choose a reason for hiding this comment

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

n.b. this section should be replaced with the hw-directive block instead (for a future PR)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks a lot! I looked at the other boards and gave it a try in the hw-directive block.

@nordicjm nordicjm requested a review from kartben June 4, 2025 11:11
@@ -35,7 +35,7 @@ The Space Cubics OBC module 1 provides the following hardware features:
+-----------+------------+------------------------------------+

The default configuration for the board can be found in the defconfig file:
:file:`boards/arm/scobc_module1/scobc_module1_defconfig`.
:file:`boards/arm/scobc_a1/scobc_a1_defconfig`.
Copy link
Collaborator

Choose a reason for hiding this comment

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

path does not exist, so is probably easier to replace section with hw-directive block

Copy link
Collaborator

Choose a reason for hiding this comment

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

yes +1, the board doc page already has a direct link to "Browse board sources" which should be more than enough

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, I replaced. Thanks!

@sasataku sasataku force-pushed the update-board-info-for-scobc-a1 branch from 17db4d6 to 7ca4208 Compare June 4, 2025 11:34
sasataku added 6 commits June 4, 2025 22:45
In the EM (Engineering Model), the board was referred to as
"Space Cubics Module 1". Starting with the FM (Flight Model) release,
we are officially renaming it to:

  `SC-OBC Module A1`

Accordingly, the board name is updated from `scobc_module1` to
`scobc_a1`.

Signed-off-by: Takuya Sasaki <[email protected]>
Rename the board name to `scobc-a1`, and update the board's full name
to `SC-OBC Module A1`. And update the output of the `Hello World`
sample to match the latest Zephyr version.
Then, The supported feature section was replaced with a hw-directive
block.

Signed-off-by: Takuya Sasaki <[email protected]>
Increase number of IRQs for SC-OBC Module A1 to 32, including
reserved entries.

Signed-off-by: Takuya Sasaki <[email protected]>
The following changes have been made in the FM (Flight Model) of the
`SC-OBC Module A1`:

  - Change in the UART Core register address
    Link: https://sc-obc-fpga-technical-reference-manual.readthedocs.io/ja/latest/#orgecb2e01

  - Change in the size of HRMEM (High-Reliability Memory)
    Link: https://sc-obc-fpga-technical-reference-manual.readthedocs.io/ja/latest/#orgc3b1ecc

As a result, we have defined the EM board revision as `1.0.0` and the
FM as `2.0.0`. The EM configuration has been moved to
`scobc_a1_1_0_0.overlay`.

Signed-off-by: Takuya Sasaki <[email protected]>
Fixes the YAMLLint error as below:

   YAMLLint (indentation):wrong indentation: expected 4 but found 2
   File:boards/sc/scobc_a1/board.yml
   Line:12
   Column:3

Signed-off-by: Takuya Sasaki <[email protected]>
Adds SC-OBC Module A1 FPGA System Register and System Monitor
definitions to DTS.

  - System Register
    Registers for system control of SC-OBC Module A1
    Link: https://sc-obc-fpga-technical-reference-manual.readthedocs.io/ja/latest/#org19067cd

  - System Monitor
    Module for monitoring the SC-OBC Module A1 system
    Link: https://sc-obc-fpga-technical-reference-manual.readthedocs.io/ja/latest/#org8832978

Signed-off-by: Takuya Sasaki <[email protected]>
@sasataku sasataku force-pushed the update-board-info-for-scobc-a1 branch from 7ca4208 to bba2d88 Compare June 4, 2025 13:45
Copy link

sonarqubecloud bot commented Jun 4, 2025

@sasataku sasataku requested a review from nordicjm June 5, 2025 01:51
@sasataku
Copy link
Contributor Author

sasataku commented Jun 9, 2025

@nordicjm
I got approval earlier, but I made updates to the docs.
Sorry for the trouble, but could you take another look when you have time?
Thanks in advance.

@fabiobaltieri fabiobaltieri merged commit 330848c into zephyrproject-rtos:main Jun 9, 2025
27 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.

4 participants