Skip to content

configs/config.protectli_vp2430_no_emmc: config variant without eMMC#900

Merged
philipanda merged 1 commit into
dasharofrom
eMMC_toggle_protectli_vp2430
Jun 2, 2026
Merged

configs/config.protectli_vp2430_no_emmc: config variant without eMMC#900
philipanda merged 1 commit into
dasharofrom
eMMC_toggle_protectli_vp2430

Conversation

@wiktormowinski

@wiktormowinski wiktormowinski commented May 21, 2026

Copy link
Copy Markdown
Contributor

Adds ENABLE_EMMC option (default y) to support VP2430e, a hardware variant without eMMC. SMBIOS product name gets an "e" suffix when the option is disabled.

ScsEmmcEnabled is set by soc_silicon_init_params() via is_devfn_enabled(PCH_DEVFN_EMMC) before mainboard_silicon_init_params() is called, so overriding it directly in FSP UPD at the end of mainboard_silicon_init_params() is sufficient.

pcidev_path_on_root(PCH_DEVFN_EMMC)->enabled is also cleared to prevent the device from appearing in ACPI tables.

@wiktormowinski wiktormowinski requested a review from mkopec May 21, 2026 09:49
@wiktormowinski wiktormowinski force-pushed the eMMC_toggle_protectli_vp2430 branch 3 times, most recently from eae9728 to 6b173a0 Compare May 25, 2026 07:00
@philipanda philipanda self-requested a review May 26, 2026 11:02
philipanda
philipanda previously approved these changes May 26, 2026
@philipanda

philipanda commented May 29, 2026

Copy link
Copy Markdown
Contributor

Seems the tag protectli_vp2430_v0.9.1-rc1 was not updated after the force pushes

@philipanda philipanda left a comment

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.

tag fixed for rc2

…MMC removed

Adds ENABLE_EMMC option (default y) to support VP2430e a hardware
variant without eMMC. SMBIOS product name gets an "e" suffix when the
option is disabled.

`ScsEmmcEnabled` is set by `soc_silicon_init_params()` via
`is_devfn_enabled(PCH_DEVFN_EMMC)` before `mainboard_silicon_init_params()`
is called, so overriding it directly in FSP UPD at the end of
`mainboard_silicon_init_params()` is sufficient.

`pcidev_path_on_root(PCH_DEVFN_EMMC)->enabled` is also cleared to
prevent the device from appearing in ACPI tables.

Upstream-Status: Inappropriate [Dasharo downstream]

Signed-off-by: Wiktor Mowinski <wiktor.mowinski@3mdeb.com>
@philipanda philipanda force-pushed the eMMC_toggle_protectli_vp2430 branch from 51e331a to 0d14b14 Compare June 2, 2026 06:04
@philipanda philipanda merged commit 0d14b14 into dasharo Jun 2, 2026
38 of 39 checks passed
@philipanda philipanda deleted the eMMC_toggle_protectli_vp2430 branch June 2, 2026 06:04
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