Re: [U-Boot] X86 Broadwell-DE override FSP configuration
Hi Vincenzo, On Thu, Jan 18, 2018 at 4:45 AM, vnktux wrote: > Thank you Bin, > > I managed to fully boot Broadwell-DE on platform with memory down enabled :) > > Now before submitting the patch I have to solve a small problem. > In either case memory down is enable or not the booting process takes 1 hour > and 20 minutes. On the output there are no error at all. I tried to > enable/disable fastboot and MRC but this didn't solve the problem. > 1 hour and 20 minutes seems too long. Is this Intel CamelbackMountain CRB or your custom board? Please avoid top-posting next time. Thanks! Regards, Bin ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] X86 Broadwell-DE override FSP configuration
Thank you Bin, I managed to fully boot Broadwell-DE on platform with memory down enabled :) Now before submitting the patch I have to solve a small problem. In either case memory down is enable or not the booting process takes 1 hour and 20 minutes. On the output there are no error at all. I tried to enable/disable fastboot and MRC but this didn't solve the problem. Sent with [ProtonMail](https://protonmail.com) Secure Email. Original Message On 16 January 2018 2:01 PM, Bin Meng wrote: > +ML > > Hi Vincenzo, > > On Tue, Jan 16, 2018 at 8:21 PM, vnktux vnk...@protonmail.com wrote: > >> Hi all, >> I managed to fully boot a bare metal U-Boot on the Intel Camelback Mountain >> CRB (Broadwell-DE). > > This is great! > >> However I have a small problem, basically U-Boot does not override the FSP >> UPD data when the platform boot. > > U-Boot can override the FSP UPD data. > >> For prototyping I am using a patched FSP binary I got in the company where I >> work. Now that it boot on the camelback mountain I need it to run on another >> platform (also broadwell-de) that has memory down installed on the >> motherboard. That should be easy, all I have to do is to enable memory down >> in the fsp_config.c file and provide the address of the SPD binary file. I >> have done that, but for some reason it doesn't work. During the booting >> process the platform give an error about the Dimm module not being >> supported. That make sense since the pached FSP binary I am using has memory >> down disabled. The part I don't understand is why U-Boot doesnt override the >> UPD settings specified in my fsp_config.c file. >> Below you will find my implementation, can somebody explain me what am I >> doing wrong? >> https://raw.githubusercontent.com/WarOfDevil/u-boot.x86_64-broadwell-de/master/arch/x86/cpu/broadwell-de/fsp_configs.c >> https://raw.githubusercontent.com/WarOfDevil/u-boot.x86_64-broadwell-de/master/arch/x86/dts/poseidon.dts > > It looks you have commented out the FSP node in the device tree, then > your fsp_configs.c won't do the UPD override. > >> Once I manage to make this work, I can create a patch for U-Boot. > > Regards, > Bin ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] X86 Broadwell-DE override FSP configuration
+ML Hi Vincenzo, On Tue, Jan 16, 2018 at 8:21 PM, vnktux wrote: > Hi all, > > I managed to fully boot a bare metal U-Boot on the Intel Camelback Mountain > CRB (Broadwell-DE). > This is great! > However I have a small problem, basically U-Boot does not override the FSP > UPD data when the platform boot. U-Boot can override the FSP UPD data. > For prototyping I am using a patched FSP binary I got in the company where I > work. Now that it boot on the camelback mountain I need it to run on another > platform (also broadwell-de) that has memory down installed on the > motherboard. That should be easy, all I have to do is to enable memory down > in the fsp_config.c file and provide the address of the SPD binary file. I > have done that, but for some reason it doesn't work. During the booting > process the platform give an error about the Dimm module not being > supported. That make sense since the pached FSP binary I am using has memory > down disabled. The part I don't understand is why U-Boot doesnt override the > UPD settings specified in my fsp_config.c file. > > Below you will find my implementation, can somebody explain me what am I > doing wrong? > > https://raw.githubusercontent.com/WarOfDevil/u-boot.x86_64-broadwell-de/master/arch/x86/cpu/broadwell-de/fsp_configs.c > > https://raw.githubusercontent.com/WarOfDevil/u-boot.x86_64-broadwell-de/master/arch/x86/dts/poseidon.dts It looks you have commented out the FSP node in the device tree, then your fsp_configs.c won't do the UPD override. > > Once I manage to make this work, I can create a patch for U-Boot. > Regards, Bin ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
[U-Boot] X86 Broadwell-DE override FSP configuration
Hi all,
I managed to fully boot a bare metal U-Boot on the Intel Camelback Mountain CRB
(Broadwell-DE).
However I have a small problem, basically U-Boot does not override the FSP UPD
data when the platform boot.
For prototyping I am using a patched FSP binary I got in the company where I
work. Now that it boot on the camelback mountain I need it to run on another
platform (also broadwell-de) that has memory down installed on the motherboard.
That should be easy, all I have to do is to enable memory down in the
fsp_config.c file and provide the address of the SPD binary file. I have done
that, but for some reason it doesn't work. During the booting process the
platform give an error about the Dimm module not being supported. That make
sense since the pached FSP binary I am using has memory down disabled. The part
I don't understand is why U-Boot doesnt override the UPD settings specified in
my fsp_config.c file.
Below you will find my implementation, can somebody explain me what am I doing
wrong?
/*
* Copyright (C) 2017, Vincenzo Bove <
vincenzo.b...@prodrive-technologies.com
>
*
* SPDX-License-Identifier: GPL-2.0+
*/
#include
#include
#include
#define CONFIG_FSP_MEMORY_DOWN 1
unsigned int px86a_c0_d0_spd[] = {
0x30, 0x30, 0x30, 0x30, 0x30, 0x30, 0x30, 0x3a, 0x20, 0x32, 0x33, 0x20,
0x30, 0x30, 0x20, 0x30, 0x63, 0x20, 0x30, 0x33, 0x20, 0x38, 0x35, 0x20,
};
unsigned int px86a_c0_d0_spd_len = 2400;
DECLARE_GLOBAL_DATA_PTR;
/**
* Override the FSP's configuration data.
* If the device tree does not specify an integer setting, use the default
* provided by Prodrive BroadwellDE.rom
*/
void update_fsp_configs(struct fsp_config_data *config,
struct fspinit_rtbuf *rt_buf)
{
struct upd_region *fsp_upd = &config->fsp_upd;
const void *blob = gd->fdt_blob;
int node;
/* Initialize runtime buffer for fsp_init() */
rt_buf->common.stack_top = config->common.stack_top - 32;
rt_buf->common.boot_mode = config->common.boot_mode;
rt_buf->common.upd_data = &config->fsp_upd;
node = fdtdec_next_compatible(blob, 0, COMPAT_INTEL_BROADWELL_DE_FSP);
if (node < 0) {
debug("%s: Cannot find FSP node\n", __func__);
return;
}
debug("%s: FSP node found!\n", __func__);
fsp_upd->memEccSupport = fdtdec_get_int(blob, node,
"fsp,memEccSupport",
MEM_ECC_SUPPORT_AUTO);
fsp_upd->memDdrMemoryType = fdtdec_get_int(blob, node,
"fsp,memDdrMemoryType",
MEM_DDR_MEMORY_TYPE_UDIMM_AND_RDIMM);
fsp_upd->memRankMultiplication = fdtdec_get_int(blob, node,
"fsp,memRankMultiplication",
MEM_RANK_MULTIPLICATION_AUTO);
fsp_upd->memRankMarginTool = fdtdec_get_int(blob, node,
"fsp,memRankMarginTool",
MEM_RANK_MARGIN_TOOL_AUTO);
fsp_upd->memScrambling = fdtdec_get_int(blob, node,
"fsp,memScrambling",
MEM_SCRAMBLING_AUTO);
fsp_upd->memRefreshMode = fdtdec_get_int(blob, node,
"fsp,memRefreshMode",
MEM_REFRESH_MODE_ACC_SELF_REFRESH);
fsp_upd->memMcOdtOverride = fdtdec_get_int(blob, node,
"fsp,memMcOdtOverride",
MEM_MC0DT_OVERRIDE_AUTO);
fsp_upd->memCAParity = fdtdec_get_int(blob, node,
"fsp,memCAParity",
MEM_CA_PARITY_AUTO);
fsp_upd->memThermalThrottling = fdtdec_get_int(blob, node,
"fsp,memThermalThrottling",
MEM_THERMAL_THROTTLING_CLOSEDLOOP);
fsp_upd->memPowerSavingsMode = fdtdec_get_int(blob, node,
"fsp,memPowerSavingsMode",
MEM_POWER_SAVINGS_MODE_AUTO);
fsp_upd->memElectricalThrottling = fdtdec_get_int(blob, node,
"fsp,memElectricalThrottling",
MEM_ELECTRICAL_THROTTLING_DISABLED);
fsp_upd->memPagePolicy = fdtdec_get_int(blob, node,
"fsp,memPagePolicy",
MEM_PAGE_POLICY_AUTO);
fsp_upd->memSocketInterleaveBelow4G = fdtdec_get_int(blob, node,
"fsp,memSocketInterleaveBelow4G",
MEM_SOCKET_INTERLEAVE_BELOW_4G_DISABLED);
fsp_upd->memChannelInterleave = fdtdec_get_int(blob, node,
"fsp,memChannelInterleave",
MEM_CHANNEL_INTERLEAVE_AUTO);
fsp_upd->memRankInterleave = fdtdec_get_int(blob, node,
"fsp,memRankInterleave",
MEM_RANK_INTERLEAVE_AUTO);
if(CONFIG_FSP_MEMORY_DOWN){
//memcpy ((int *)0xfffd, px86a_c0_d0_spd, sizeof (px86a_c0_d0_spd));
fsp_upd->memDownEnable = fdtdec_get_bool(blob, node,
"fsp,memDownEnable");
fsp_upd->memDownCh0Dimm0Sp
