erebion/coreboot
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

mainboard/intel/ptlrvp: Remove UFS support

Browse source 
Unified Flash Storage (UFS) has been descoped from Panther Lake
configurations. This commit removes UFS-related configurations and GPIO
pad settings across relevant files.

BUG=b:442891168

Change-Id: I5de2878aa44e2d48879b9ecf274aebedfbf551ca
Signed-off-by: Jeremy Compostella <jeremy.compostella@intel.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/88989
Reviewed-by: Jamie Ryu <jamie.m.ryu@intel.com>
Reviewed-by: Subrata Banik <subratabanik@google.com>
Reviewed-by: Wonkyu Kim <wonkyu.kim@intel.com>
Reviewed-by: Cliff Huang <cliff.huang@intel.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
Jeremy Compostella 2025-08-28 16:13:49 -07:00 • committed by Subrata Banik
commit 5b46caef93
3 changed files with 3 additions and 19 deletions
Download patch file Download diff file Expand all files Collapse all files

8
src/mainboard/intel/ptlrvp/variants/ptlrvp/fw_config.c
View file

@ -248,11 +248,6 @@ static const struct pad_config gen4_ssd_pads[] = {
PAD_CFG_GPO(GPP_B09, 1, PLTRST),
};
static const struct pad_config ufs_enable_pads[] = {
/* GPP_D21: GPP_D21_UFS_REFCLK */
PAD_CFG_NF(GPP_D21, NONE, DEEP, NF1),
};
/* Gen5 NVME: at the bottom M.2 slot */
static const struct pad_config pre_mem_gen5_ssd_pwr_pads[] = {
/* GPP_B16: GEN5_SSD_PWREN */
@ -601,11 +596,10 @@ void fw_config_gpio_padbased_override(struct pad_config *padbased_table)
} else if (fw_config_probe(FW_CONFIG(STORAGE, STORAGE_NVME_GEN5))) {
GPIO_PADBASED_OVERRIDE(padbased_table, gen5_ssd_pads);
} else if (fw_config_probe(FW_CONFIG(STORAGE, STORAGE_UFS))) {
GPIO_PADBASED_OVERRIDE(padbased_table, ufs_enable_pads);
die("UFS is not supported on Panther Lake\n");
} else if (fw_config_probe(FW_CONFIG(STORAGE, STORAGE_UNKNOWN))) {
GPIO_PADBASED_OVERRIDE(padbased_table, gen4_ssd_pads);
GPIO_PADBASED_OVERRIDE(padbased_table, gen5_ssd_pads);
GPIO_PADBASED_OVERRIDE(padbased_table, ufs_enable_pads);
}
if (fw_config_probe(FW_CONFIG(AUDIO, AUDIO_NONE))) {

7
src/mainboard/intel/ptlrvp/variants/ptlrvp/overridetree.cb
View file

@ -39,7 +39,7 @@ fw_config
option STORAGE_UNKNOWN 0
option STORAGE_NVME_GEN4 1
option STORAGE_NVME_GEN5 2
option STORAGE_UFS 3
option STORAGE_UFS 3 # Not supported
end
field FP 17
option FP_ABSENT 0
@ -518,11 +518,6 @@ chip soc/intel/pantherlake
end
end
device ref ufs on
probe STORAGE STORAGE_UFS
probe STORAGE STORAGE_UNKNOWN
end
device ref pcie_rp2 on
probe CELLULAR CELLULAR_PCIE
register "pcie_rp[PCIE_RP(2)]" = "{

7
src/mainboard/intel/ptlrvp/variants/ptlrvp_chromeec/overridetree.cb
View file

@ -39,7 +39,7 @@ fw_config
option STORAGE_UNKNOWN 0
option STORAGE_NVME_GEN4 1
option STORAGE_NVME_GEN5 2
option STORAGE_UFS 3
option STORAGE_UFS 3 # Not supported
end
field FP 17
option FP_ABSENT 0
@ -518,11 +518,6 @@ chip soc/intel/pantherlake
end
end
device ref ufs on
probe STORAGE STORAGE_UFS
probe STORAGE STORAGE_UNKNOWN
end
device ref pcie_rp2 on
probe CELLULAR CELLULAR_PCIE
register "pcie_rp[PCIE_RP(2)]" = "{