Skip to content

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

Open
wiktormowinski wants to merge 1 commit into
dasharofrom
eMMC_toggle_protectli_vp2430
Open

configs/config.protectli_vp2430_no_emmc: config variant without eMMC#900
wiktormowinski wants to merge 1 commit into
dasharofrom
eMMC_toggle_protectli_vp2430

Conversation

@wiktormowinski
Copy link
Copy Markdown
Contributor

@wiktormowinski wiktormowinski commented May 21, 2026

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 2 times, most recently from c843a27 to eae9728 Compare May 25, 2026 06:26
…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>
@wiktormowinski wiktormowinski force-pushed the eMMC_toggle_protectli_vp2430 branch from eae9728 to 6b173a0 Compare May 25, 2026 07:00
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.

1 participant