Skip to content

Clean up the BMC Kconfig - #620

Open
DannyIsa wants to merge 1 commit into
sonic-net:masterfrom
DannyIsa:master_sonic_aspeed_bmc_kconfig_cleanup
Open

Clean up the BMC Kconfig#620
DannyIsa wants to merge 1 commit into
sonic-net:masterfrom
DannyIsa:master_sonic_aspeed_bmc_kconfig_cleanup

Conversation

@DannyIsa

@DannyIsa DannyIsa commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Why I did it

Clean up the ###-> nvidia_aspeed_bmc block in config.local/arm64/config.sonic-aspeed:

  • remove eight options already set to the same value earlier in the same file
  • move CONFIG_MODVERSIONS out of the block, since it is a common kernel config
  • drop CONFIG_MSDOS_FS and CONFIG_FAT_DEFAULT_UTF8

These are the changes requested in the review of #609 and merged there on 202605. master carries the same block, but never received them.

How I did it

How to verify it

@DannyIsa
DannyIsa requested a review from a team as a code owner September 3, 2026 07:23
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@mssonicbld

Copy link
Copy Markdown

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).

@DannyIsa
DannyIsa force-pushed the master_sonic_aspeed_bmc_kconfig_cleanup branch from 6cadf5b to 1fb9d88 Compare September 3, 2026 07:30
@DannyIsa DannyIsa changed the title arm64/config.sonic-aspeed: Clean up the nvidia_aspeed_bmc Kconfig block arm64/config.sonic-aspeed: Clean up the BMC Kconfig block Sep 3, 2026
@mssonicbld

Copy link
Copy Markdown

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).

Signed-off-by: Daniel Isakov <disakov@nvidia.com>
@DannyIsa
DannyIsa force-pushed the master_sonic_aspeed_bmc_kconfig_cleanup branch from 1fb9d88 to 4a94a67 Compare September 3, 2026 07:33
@DannyIsa DannyIsa changed the title arm64/config.sonic-aspeed: Clean up the BMC Kconfig block Clean up the BMC Kconfig Sep 3, 2026
@mssonicbld

Copy link
Copy Markdown

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).

@liat-grozovik

Copy link
Copy Markdown
Collaborator

@saiarcot895 could you please help review?

# CONFIG_CRYPTO_DEV_ASPEED_HACE is not set
# CONFIG_CRYPTO_DEV_ASPEED_RSSS is not set
# CONFIG_CRYPTO_DEV_ASPEED_ECDSA is not set
CONFIG_MODVERSIONS=y

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is already set by Debian's own default configs, and shouldn't need to be set here.

@mssonicbld

Copy link
Copy Markdown

This PR has backport request label(s) for branch(es): 202605, but is missing required test information. Please make sure you tick the tested branch(es) in the Tested branch section and provide test evidence (e.g., 202605: <test result>) in the Test result section as well in your PR description.

---Powered by SONiC BuildBot

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.

5 participants