coreboot March 2017

coreboot@coreboot.org

79 participants
267 discussions

Re: [coreboot] Where do I get cables that can fit a SOIC8 clip?
by Taiidan＠gmx.com 08 Mar '17

08 Mar '17

Thanks martin but the clip I have needs the smaller cables like the ones in this photo as they didn't space the connectors enough for dupont cables. https://sc01.alicdn.com/kf/HTB1jVupNVXXXXXOXpXXq6xXFXXXm/SOIC8-SOP8-Test-Cl… https://www.aliexpress.com/store/product/SOP16-SOIC16-Flash-Chip-Test-Clips…

3 3

We need a place to hold the knowledge about the Manageability Engine.
by Persmule 08 Mar '17

08 Mar '17

4 3

Re: [coreboot] Crowdfunding Status for KGPE-D16 BMC Port Offer
by Taiidan＠gmx.com 08 Mar '17

08 Mar '17

Today I noticed a new stretch goal about flashrom support (kinda late in the game), it seems pretty important to have this so I think that the stretch goal funding should be extended. I had assumed that the BMC firmware would be stored in and loaded from the main coreboot flash image, or that there would be a shim loader installable on the bmc firmware module that would (as the BMC has DMA) load the firmware from memory for easy updating. The requiring of the firmware module may be a problem for some people because apparently there are two different SKU's from asus one of which didn't ship with it and no retailers sell it separately. Also, If the stretch goal doesn't get funded is it possible to perform a "clean" flash with a test clip?

2 1

How to access bug reports referenced with `BUG=b:XXXXXXXXX)`?
by Paul Menzel 08 Mar '17

08 Mar '17

Dear coreboot folks, Looking through the commits, I saw a syntax to reference bug reports, I hadn’t noticed before. > BUG=b:24676003 Looking through the commits, it looks like it goes back until even 2015 [1]. How can these be accessed? Could Gerrit be adapted to mark these strings up as links? Thanks, Paul [1] http://review.coreboot.org/12347

2 1

Re: [coreboot] Crowdfunding Status for KGPE-D16 BMC Port Offer
by Timothy Pearson 07 Mar '17

07 Mar '17

-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 On 02/28/2017 11:44 AM, Martin Roth wrote: > Yes, the FSF sent out an email on Friday, and that's helped get the > community funding portion from 60% up to where we are now at at 92%. > > Martin I am pleased to announce that the base OpenBMC port for the KGPE-D16 has met is pledge goal! I'd like to say thank you to all that contributed, and also note that there are currently three stretch goals that are open for funding until March 10: https://www.raptorengineering.com/coreboot/kgpe-d16-bmc-port-offer.php Martin is in the process of collecting the pledges now, and we will start work on the OpenBMC port when collection is completed. A status page for this work, complete with updates, will be made available shortly. Thanks! - -- Timothy Pearson Raptor Engineering +1 (415) 727-8645 (direct line) +1 (512) 690-0200 (switchboard) https://www.raptorengineering.com -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/ iQEcBAEBAgAGBQJYtz2DAAoJEK+E3vEXDOFbM2YH/AjCqF30c6wa8cUspBAfqH48 Gyj5FRk3olCFQ2mXwm8X0h35vEfkmoAADgZ7pDAdNRhu5IXJVLfFkiGpiozLiPbI vF8HWIzmGwWNE1+B6w3gCoGOy9xU2bkJMv7nmHLP0rgVtwBek5N08rQYQ7o7p82u mvgNMf0BlqnhtYSsM6x6Q15UHmHi45ne/2B3hWZdkMlkvWceUtejDzDo4PtwTZmC t4Tp1AEz9Alx/YsU8+5bZeWPOnreHWt8fdQkEVCUNVgRRAMluUsVVt/P2u4t9FTN 8ZrYRS9rTtFFMxQgJNwPtZcD0AdkmZW7dVMXtgfjN9jsuZxtsoJsTlOIpp+3YtM= =BqIY -----END PGP SIGNATURE-----

1 1

Re: [coreboot] coreboot community meeting minutes for March 2, 2017
by Peter Stuge 07 Mar '17

07 Mar '17

Martin Roth wrote: > * board status uploads are still only a few boards. How do we improve usage? Include it together with the default payload. //Peter

1 0

New Defects reported by Coverity Scan for coreboot
by scan-admin＠coverity.com 07 Mar '17

07 Mar '17

Hi, Please find the latest report on new defect(s) introduced to coreboot found with Coverity Scan. 12 new defect(s) introduced to coreboot found with Coverity Scan. 1 defect(s), reported by Coverity Scan earlier, were marked fixed in the recent build analyzed by Coverity Scan. New defect(s) Reported-by: Coverity Scan Showing 12 of 12 defect(s) ** CID 1371825: Uninitialized variables (UNINIT) /src/mainboard/intel/kblrvp/romstage.c: 58 in mainboard_memory_init_params() ________________________________________________________________________________________________________ *** CID 1371825: Uninitialized variables (UNINIT) /src/mainboard/intel/kblrvp/romstage.c: 58 in mainboard_memory_init_params() 52 } else { /* for CONFIG_BOARD_INTEL_KBLRVP7 */ 53 struct spd_block blk; 54 55 mem_cfg->DqPinsInterleaved = 1; 56 get_spd_smbus(&blk); 57 mem_cfg->MemorySpdDataLen = blk.len; >>> CID 1371825: Uninitialized variables (UNINIT) >>> Using uninitialized element of array "blk.spd_array". 58 mem_cfg->MemorySpdPtr00 = (u32)blk.spd_array[0]; 59 } 60 mem_cfg->MemorySpdPtr10 = mem_cfg->MemorySpdPtr00; ** CID 1371824: Control flow issues (NO_EFFECT) /src/lib/spd_bin.c: 160 in get_spd_smbus() ________________________________________________________________________________________________________ *** CID 1371824: Control flow issues (NO_EFFECT) /src/lib/spd_bin.c: 160 in get_spd_smbus() 154 155 void get_spd_smbus(struct spd_block *blk) 156 { 157 u8 i; 158 unsigned char *spd_data_ptr = car_get_var_ptr(&spd_data); 159 >>> CID 1371824: Control flow issues (NO_EFFECT) >>> This less-than-zero comparison of an unsigned value is never true. "i < 0". 160 for (i = 0 ; i < CONFIG_DIMM_MAX; i++) { 161 get_spd(spd_data_ptr + i * CONFIG_DIMM_SPD_SIZE, 162 0xA0 + (i << 1)); 163 blk->spd_array[i] = spd_data_ptr + i * CONFIG_DIMM_SPD_SIZE; 164 } 165 ** CID 1371823: Control flow issues (NO_EFFECT) /src/lib/spd_bin.c: 104 in update_spd_len() ________________________________________________________________________________________________________ *** CID 1371823: Control flow issues (NO_EFFECT) /src/lib/spd_bin.c: 104 in update_spd_len() 98 } 99 } 100 101 static void update_spd_len(struct spd_block *blk) 102 { 103 u8 i, j = 0; >>> CID 1371823: Control flow issues (NO_EFFECT) >>> This less-than-zero comparison of an unsigned value is never true. "i < 0". 104 for (i = 0 ; i < CONFIG_DIMM_MAX; i++) 105 if (blk->spd_array[i] != NULL) 106 j |= blk->spd_array[i][SPD_DRAM_TYPE]; 107 108 /* If spd used is DDR4, then its length is 512 byte. */ 109 if (j == SPD_DRAM_DDR4) ** CID 1371822: Control flow issues (NO_EFFECT) /src/lib/spd_bin.c: 30 in dump_spd_info() ________________________________________________________________________________________________________ *** CID 1371822: Control flow issues (NO_EFFECT) /src/lib/spd_bin.c: 30 in dump_spd_info() 24 static u8 spd_data[CONFIG_DIMM_MAX * CONFIG_DIMM_SPD_SIZE] CAR_GLOBAL; 25 26 void dump_spd_info(struct spd_block *blk) 27 { 28 u8 i; 29 >>> CID 1371822: Control flow issues (NO_EFFECT) >>> This less-than-zero comparison of an unsigned value is never true. "i < 0". 30 for (i = 0; i < CONFIG_DIMM_MAX; i++) 31 if (blk->spd_array[i] != NULL && blk->spd_array[i][0] != 0) { 32 printk(BIOS_DEBUG, "SPD @ 0x%02X\n", 0xA0|(i << 1)); 33 print_spd_info(blk->spd_array[i]); 34 } 35 } ** CID 1371821: Control flow issues (DEADCODE) /src/lib/spd_bin.c: 105 in update_spd_len() ________________________________________________________________________________________________________ *** CID 1371821: Control flow issues (DEADCODE) /src/lib/spd_bin.c: 105 in update_spd_len() 99 } 100 101 static void update_spd_len(struct spd_block *blk) 102 { 103 u8 i, j = 0; 104 for (i = 0 ; i < CONFIG_DIMM_MAX; i++) >>> CID 1371821: Control flow issues (DEADCODE) >>> Execution cannot reach this statement: "if (blk->spd_array[i] != NU...". 105 if (blk->spd_array[i] != NULL) 106 j |= blk->spd_array[i][SPD_DRAM_TYPE]; 107 108 /* If spd used is DDR4, then its length is 512 byte. */ 109 if (j == SPD_DRAM_DDR4) 110 blk->len = SPD_PAGE_LEN_DDR4; ** CID 1371820: Control flow issues (DEADCODE) /src/lib/spd_bin.c: 161 in get_spd_smbus() ________________________________________________________________________________________________________ *** CID 1371820: Control flow issues (DEADCODE) /src/lib/spd_bin.c: 161 in get_spd_smbus() 155 void get_spd_smbus(struct spd_block *blk) 156 { 157 u8 i; 158 unsigned char *spd_data_ptr = car_get_var_ptr(&spd_data); 159 160 for (i = 0 ; i < CONFIG_DIMM_MAX; i++) { >>> CID 1371820: Control flow issues (DEADCODE) >>> Execution cannot reach this statement: "get_spd(spd_data_ptr + i * ...". 161 get_spd(spd_data_ptr + i * CONFIG_DIMM_SPD_SIZE, 162 0xA0 + (i << 1)); 163 blk->spd_array[i] = spd_data_ptr + i * CONFIG_DIMM_SPD_SIZE; 164 } 165 166 update_spd_len(blk); ** CID 1371819: Control flow issues (DEADCODE) /src/lib/spd_bin.c: 110 in update_spd_len() ________________________________________________________________________________________________________ *** CID 1371819: Control flow issues (DEADCODE) /src/lib/spd_bin.c: 110 in update_spd_len() 104 for (i = 0 ; i < CONFIG_DIMM_MAX; i++) 105 if (blk->spd_array[i] != NULL) 106 j |= blk->spd_array[i][SPD_DRAM_TYPE]; 107 108 /* If spd used is DDR4, then its length is 512 byte. */ 109 if (j == SPD_DRAM_DDR4) >>> CID 1371819: Control flow issues (DEADCODE) >>> Execution cannot reach this statement: "blk->len = 512;". 110 blk->len = SPD_PAGE_LEN_DDR4; 111 else 112 blk->len = SPD_PAGE_LEN; 113 } 114 115 int get_spd_cbfs_rdev(struct region_device *spd_rdev, u8 spd_index) ** CID 1371818: Control flow issues (DEADCODE) /src/lib/spd_bin.c: 31 in dump_spd_info() ________________________________________________________________________________________________________ *** CID 1371818: Control flow issues (DEADCODE) /src/lib/spd_bin.c: 31 in dump_spd_info() 25 26 void dump_spd_info(struct spd_block *blk) 27 { 28 u8 i; 29 30 for (i = 0; i < CONFIG_DIMM_MAX; i++) >>> CID 1371818: Control flow issues (DEADCODE) >>> Execution cannot reach this statement: "if (blk->spd_array[i] != NU...". 31 if (blk->spd_array[i] != NULL && blk->spd_array[i][0] != 0) { 32 printk(BIOS_DEBUG, "SPD @ 0x%02X\n", 0xA0|(i << 1)); 33 print_spd_info(blk->spd_array[i]); 34 } 35 } 36 ** CID 1371817: Integer handling issues (CONSTANT_EXPRESSION_RESULT) /src/lib/spd_bin.c: 104 in update_spd_len() ________________________________________________________________________________________________________ *** CID 1371817: Integer handling issues (CONSTANT_EXPRESSION_RESULT) /src/lib/spd_bin.c: 104 in update_spd_len() 98 } 99 } 100 101 static void update_spd_len(struct spd_block *blk) 102 { 103 u8 i, j = 0; >>> CID 1371817: Integer handling issues (CONSTANT_EXPRESSION_RESULT) >>> "i < 0" is always false regardless of the values of its operands. This occurs as the logical operand of "for". 104 for (i = 0 ; i < CONFIG_DIMM_MAX; i++) 105 if (blk->spd_array[i] != NULL) 106 j |= blk->spd_array[i][SPD_DRAM_TYPE]; 107 108 /* If spd used is DDR4, then its length is 512 byte. */ 109 if (j == SPD_DRAM_DDR4) ** CID 1371816: Integer handling issues (CONSTANT_EXPRESSION_RESULT) /src/lib/spd_bin.c: 160 in get_spd_smbus() ________________________________________________________________________________________________________ *** CID 1371816: Integer handling issues (CONSTANT_EXPRESSION_RESULT) /src/lib/spd_bin.c: 160 in get_spd_smbus() 154 155 void get_spd_smbus(struct spd_block *blk) 156 { 157 u8 i; 158 unsigned char *spd_data_ptr = car_get_var_ptr(&spd_data); 159 >>> CID 1371816: Integer handling issues (CONSTANT_EXPRESSION_RESULT) >>> "i < 0" is always false regardless of the values of its operands. This occurs as the logical operand of "for". 160 for (i = 0 ; i < CONFIG_DIMM_MAX; i++) { 161 get_spd(spd_data_ptr + i * CONFIG_DIMM_SPD_SIZE, 162 0xA0 + (i << 1)); 163 blk->spd_array[i] = spd_data_ptr + i * CONFIG_DIMM_SPD_SIZE; 164 } 165 ** CID 1371815: Integer handling issues (CONSTANT_EXPRESSION_RESULT) /src/lib/spd_bin.c: 30 in dump_spd_info() ________________________________________________________________________________________________________ *** CID 1371815: Integer handling issues (CONSTANT_EXPRESSION_RESULT) /src/lib/spd_bin.c: 30 in dump_spd_info() 24 static u8 spd_data[CONFIG_DIMM_MAX * CONFIG_DIMM_SPD_SIZE] CAR_GLOBAL; 25 26 void dump_spd_info(struct spd_block *blk) 27 { 28 u8 i; 29 >>> CID 1371815: Integer handling issues (CONSTANT_EXPRESSION_RESULT) >>> "i < 0" is always false regardless of the values of its operands. This occurs as the logical operand of "for". 30 for (i = 0; i < CONFIG_DIMM_MAX; i++) 31 if (blk->spd_array[i] != NULL && blk->spd_array[i][0] != 0) { 32 printk(BIOS_DEBUG, "SPD @ 0x%02X\n", 0xA0|(i << 1)); 33 print_spd_info(blk->spd_array[i]); 34 } 35 } ** CID 1371814: Memory - illegal accesses (BUFFER_SIZE_WARNING) /src/soc/intel/common/smbios.c: 31 in dimm_info_fill() ________________________________________________________________________________________________________ *** CID 1371814: Memory - illegal accesses (BUFFER_SIZE_WARNING) /src/soc/intel/common/smbios.c: 31 in dimm_info_fill() 25 { 26 dimm->dimm_size = dimm_capacity; 27 dimm->ddr_type = ddr_type; 28 dimm->ddr_frequency = frequency; 29 dimm->channel_num = channel_id; 30 dimm->dimm_num = dimm_id; >>> CID 1371814: Memory - illegal accesses (BUFFER_SIZE_WARNING) >>> Calling strncpy with a maximum size argument of 19 bytes on destination array "dimm->module_part_number" of size 19 bytes might leave the destination string unterminated. 31 strncpy((char *)dimm->module_part_number, 32 module_part_num, 33 sizeof(dimm->module_part_number)); 34 switch (data_width) { 35 case 8: 36 dimm->bus_width = MEMORY_BUS_WIDTH_8; ________________________________________________________________________________________________________ To view the defects in Coverity Scan visit, https://u2389337.ct.sendgrid.net/wf/click?upn=08onrYu34A-2BWcWUl-2F-2BfV0V0… To manage Coverity Scan email notifications for "coreboot(a)coreboot.org", click https://u2389337.ct.sendgrid.net/wf/click?upn=08onrYu34A-2BWcWUl-2F-2BfV0V0…

1 0

Re: [coreboot] Inteal Leafhill : Linux SATA driver fails when used with coreboot+grub
by Paul Menzel 07 Mar '17

07 Mar '17

Dear Gailu, Am Montag, den 06.03.2017, 00:28 +0530 schrieb Gailu Singh: > I tried to find out the details for following error > > ata1: SATA link down (SStatus 4 SControl 300) > > As per status register description > > SStatus 4 : Phy in offline mode as a result of the interface being disabled > or running in a BIST loopback mode > > Is there any chance that coreboot/grub/Linux is putting SATA in to BIST > loopback mode? > > I am trying to understand who is responsible for SATA Linux status 4 and > possible candidates are > a) coreboot > b) grub > c) Linux Please try the latest Linux kernel version [1] – currently 4.10.1 – to rule out any problems in that regard. Please post the logs from that Linux kernel. Thanks, Paul PS: Please just send plain text messages to mailing lists. [1] https://www.kernel.org/

1 0

Where to get ME image/flash descriptors for the x220?
by Taiidan＠gmx.com 07 Mar '17

07 Mar '17

I want generic ones, not the sketchy extracted ones that came with my fleabay laptop. The lenovo website doesn't work on my computer BTW

4 5

Anyone using the free VBIOS implemention for AMD GPUs?
by i1w5d7gf38keg＠tutanota.com 07 Mar '17

07 Mar '17

Is someone here using the free VBIOS implementation for AMD PCIe GPUs? Does digital Video output also work on some cards or just VGA(D-sub)? https://sourceforge.net/p/openradeonbios/wiki/Home

2 1

← Newer
1
...
19
20
21
22
23
24
25
26
27
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

coreboot March 2017