March 2010 - coreboot - mail.coreboot.org

[Patch] remove some warnings
by Myles Watson March 24, 2010

March 24, 2010

1. gcc doesn't like a character array being cast to a struct. -extern unsigned char AmlCode[]; +extern void* AmlCode; 2. Include <console/console.h> if you're using PRINTK_IN_CAR (not ROMCC) 3. Add a few more "nuisance" prototypes. Signed-off-by: Myles Watson <mylesgw(a)gmail.com> I think it would be nice to add the number of warnings for each board on one of the build pages. It looks like there are many that could compile without any warnings, and some that look like they have a few too many. Here's the list after this patch: coreboot-builds/a-trend_atc-6220/make.log:2 coreboot-builds/a-trend_atc-6240/make.log:2 coreboot-builds/abit_be6-ii_v2_0/make.log:2 coreboot-builds/advantech_pcm-5820/make.log:76 coreboot-builds/amd_db800/make.log:111 coreboot-builds/amd_dbm690t/make.log:132 coreboot-builds/amd_mahogany/make.log:182 coreboot-builds/amd_mahogany_fam10/make.log:378 coreboot-builds/amd_norwich/make.log:110 coreboot-builds/amd_pistachio/make.log:127 coreboot-builds/amd_rumba/make.log:65 coreboot-builds/amd_serengeti_cheetah/make.log:74 coreboot-builds/amd_serengeti_cheetah_fam10/make.log:270 coreboot-builds/arima_hdama/make.log:49 coreboot-builds/artecgroup_dbe61/make.log:112 coreboot-builds/asi_mb_5blgp/make.log:2 coreboot-builds/asi_mb_5blmp/make.log:2 coreboot-builds/asus_a8n_e/make.log:54 coreboot-builds/asus_a8v-e_se/make.log:74 coreboot-builds/asus_m2v-mx_se/make.log:103 coreboot-builds/asus_mew-am/make.log:9 coreboot-builds/asus_mew-vm/make.log:10 coreboot-builds/asus_p2b-d/make.log:3 coreboot-builds/asus_p2b-ds/make.log:3 coreboot-builds/asus_p2b-f/make.log:2 coreboot-builds/asus_p2b-ls/make.log:2 coreboot-builds/asus_p2b/make.log:2 coreboot-builds/asus_p3b-f/make.log:2 coreboot-builds/axus_tc320/make.log:2 coreboot-builds/azza_pt-6ibd/make.log:2 coreboot-builds/bcom_winnet100/make.log:2 coreboot-builds/bcom_winnetp680/make.log:100 coreboot-builds/biostar_m6tba/make.log:2 coreboot-builds/broadcom_blast/make.log:78 coreboot-builds/compaq_deskpro_en_sff_p600/make.log:2 coreboot-builds/dell_s1850/make.log:5 coreboot-builds/digitallogic_adl855pc/make.log:10 coreboot-builds/digitallogic_msm586seg/make.log:12 coreboot-builds/digitallogic_msm800sev/make.log:111 coreboot-builds/eaglelion_5bcm/make.log:2 coreboot-builds/emulation_qemu-x86/make.log:0 coreboot-builds/gigabyte_ga-6bxc/make.log:2 coreboot-builds/gigabyte_ga_2761gxdk/make.log:128 coreboot-builds/gigabyte_m57sli/make.log:133 coreboot-builds/hp_dl145_g3/make.log:110 coreboot-builds/hp_e_vectra_p2706t/make.log:9 coreboot-builds/ibm_e325/make.log:48 coreboot-builds/ibm_e326/make.log:50 coreboot-builds/iei_juki-511p/make.log:2 coreboot-builds/iei_nova4899r/make.log:2 coreboot-builds/iei_pcisa-lx-800-r10/make.log:111 coreboot-builds/intel_d945gclf/make.log:24 coreboot-builds/intel_eagleheights/make.log:49 coreboot-builds/intel_jarrell/make.log:13 coreboot-builds/intel_mtarvon/make.log:6 coreboot-builds/intel_truxton/make.log:6 coreboot-builds/intel_xe7501devkit/make.log:15 coreboot-builds/iwill_dk8_htx/make.log:51 coreboot-builds/iwill_dk8s2/make.log:58 coreboot-builds/iwill_dk8x/make.log:53 coreboot-builds/jetway_j7f24/make.log:99 coreboot-builds/kontron_986lcd-m/make.log:15 coreboot-builds/kontron_kt690/make.log:131 coreboot-builds/lippert_frontrunner/make.log:42 coreboot-builds/lippert_roadrunner-lx/make.log:113 coreboot-builds/lippert_spacerunner-lx/make.log:114 coreboot-builds/mitac_6513wu/make.log:9 coreboot-builds/msi_ms6119/make.log:2 coreboot-builds/msi_ms6147/make.log:2 coreboot-builds/msi_ms6156/make.log:2 coreboot-builds/msi_ms6178/make.log:9 coreboot-builds/msi_ms7135/make.log:58 coreboot-builds/msi_ms7260/make.log:131 coreboot-builds/msi_ms9185/make.log:108 coreboot-builds/msi_ms9282/make.log:132 coreboot-builds/msi_ms9652_fam10/make.log:298 coreboot-builds/nec_powermate2000/make.log:9 coreboot-builds/newisys_khepri/make.log:48 coreboot-builds/nvidia_l1_2pvv/make.log:135 coreboot-builds/olpc_btest/make.log:72 coreboot-builds/olpc_rev_a/make.log:71 coreboot-builds/pcengines_alix1c/make.log:112 coreboot-builds/rca_rm4100/make.log:1 coreboot-builds/roda_rk886ex/make.log:25 coreboot-builds/soyo_sy-6ba-plus-iii/make.log:2 coreboot-builds/sunw_ultra40/make.log:59 coreboot-builds/supermicro_h8dme/make.log:126 coreboot-builds/supermicro_h8dmr/make.log:124 coreboot-builds/supermicro_h8dmr_fam10/make.log:293 coreboot-builds/supermicro_h8qme_fam10/make.log:293 coreboot-builds/supermicro_x6dai_g/make.log:13 coreboot-builds/supermicro_x6dhe_g/make.log:17 coreboot-builds/supermicro_x6dhe_g2/make.log:9 coreboot-builds/supermicro_x6dhr_ig/make.log:6 coreboot-builds/supermicro_x6dhr_ig2/make.log:6 coreboot-builds/technexion_tim5690/make.log:130 coreboot-builds/technexion_tim8690/make.log:130 coreboot-builds/technologic_ts5300/make.log:10 coreboot-builds/televideo_tc7020/make.log:2 coreboot-builds/thomson_ip1000/make.log:0 coreboot-builds/tyan_s1846/make.log:2 coreboot-builds/tyan_s2735/make.log:78 coreboot-builds/tyan_s2850/make.log:52 coreboot-builds/tyan_s2875/make.log:50 coreboot-builds/tyan_s2880/make.log:54 coreboot-builds/tyan_s2881/make.log:53 coreboot-builds/tyan_s2882/make.log:51 coreboot-builds/tyan_s2885/make.log:49 coreboot-builds/tyan_s2891/make.log:58 coreboot-builds/tyan_s2892/make.log:4 coreboot-builds/tyan_s2895/make.log:49 coreboot-builds/tyan_s2912/make.log:136 coreboot-builds/tyan_s2912_fam10/make.log:298 coreboot-builds/tyan_s4880/make.log:53 coreboot-builds/tyan_s4882/make.log:44 coreboot-builds/via_epia-cn/make.log:48 coreboot-builds/via_epia-m/make.log:135 coreboot-builds/via_epia-m700/make.log:204 coreboot-builds/via_epia-n/make.log:132 coreboot-builds/via_epia/make.log:62 coreboot-builds/via_pc2500e/make.log:45 coreboot-builds/via_vt8454c/make.log:60 coreboot-builds/winent_pl6064/make.log:110 Thanks, Myles

7 30

build service results for r5285
by coreboot information March 24, 2010

March 24, 2010

Dear coreboot readers! This is the automatic build system of coreboot. The developer "stepan" checked in revision 5285 to the coreboot repository. This caused the following changes: Change Log: Some keyboards need a longer timeout. Also increase error level, because a not-connected keyboard should normally not raise an error. Signed-off-by: Stefan Reinauer <stepan(a)coresystems.de> Acked-by: Myles Watson <mylesgw(a)gmail.com> Build Log: Compilation of soyo:sy-6ba-plus-iii is still broken See the error log at http://qa.coreboot.org/log_buildbrd.php?revision=5285&device=sy-6ba-plus-ii… If something broke during this checkin please be a pain in stepan's neck until the issue is fixed. If this issue is not fixed within 24h the revision should be backed out. Best regards, coreboot automatic build system

1 0

build service results for r5284
by coreboot information March 24, 2010

March 24, 2010

Dear coreboot readers! This is the automatic build system of coreboot. The developer "stepan" checked in revision 5284 to the coreboot repository. This caused the following changes: Change Log: Drop unneeded code that checks for CONFIG_ variables in build_opt_tbl. Signed-off-by: Stefan Reinauer <stepan(a)coresystems.de> Acked-by: Myles Watson <mylesgw(a)gmail.com> Build Log: Compilation of soyo:sy-6ba-plus-iii is still broken See the error log at http://qa.coreboot.org/log_buildbrd.php?revision=5284&device=sy-6ba-plus-ii… If something broke during this checkin please be a pain in stepan's neck until the issue is fixed. If this issue is not fixed within 24h the revision should be backed out. Best regards, coreboot automatic build system

1 0

[PATCH] libpayload coreboot framebuffer
by Stefan Reinauer March 24, 2010

March 24, 2010

See patch.. -- coresystems GmbH . Brahmsstr. 16 . D-79104 Freiburg i. Br. Tel.: +49 761 7668825 . Fax: +49 761 7664613 Email: info(a)coresystems.de . http://www.coresystems.de/ Registergericht: Amtsgericht Freiburg . HRB 7656 Geschäftsführer: Stefan Reinauer . Ust-IdNr.: DE245674866

2 1

[PATCH] libpayload lpgcc arch fix
by Stefan Reinauer March 24, 2010

March 24, 2010

See patch... -- coresystems GmbH . Brahmsstr. 16 . D-79104 Freiburg i. Br. Tel.: +49 761 7668825 . Fax: +49 761 7664613 Email: info(a)coresystems.de . http://www.coresystems.de/ Registergericht: Amtsgericht Freiburg . HRB 7656 Geschäftsführer: Stefan Reinauer . Ust-IdNr.: DE245674866

2 1

[PATCH] libpayload out of tree build
by Stefan Reinauer March 24, 2010

March 24, 2010

See patch -- coresystems GmbH . Brahmsstr. 16 . D-79104 Freiburg i. Br. Tel.: +49 761 7668825 . Fax: +49 761 7664613 Email: info(a)coresystems.de . http://www.coresystems.de/ Registergericht: Amtsgericht Freiburg . HRB 7656 Geschäftsführer: Stefan Reinauer . Ust-IdNr.: DE245674866

2 1

[PATCH] libpayload timer overflow
by Stefan Reinauer March 24, 2010

March 24, 2010

See patch -- coresystems GmbH . Brahmsstr. 16 . D-79104 Freiburg i. Br. Tel.: +49 761 7668825 . Fax: +49 761 7664613 Email: info(a)coresystems.de . http://www.coresystems.de/ Registergericht: Amtsgericht Freiburg . HRB 7656 Geschäftsführer: Stefan Reinauer . Ust-IdNr.: DE245674866

2 1

[PATCH] libpayload hid layer
by Stefan Reinauer March 24, 2010

March 24, 2010

See patch -- coresystems GmbH . Brahmsstr. 16 . D-79104 Freiburg i. Br. Tel.: +49 761 7668825 . Fax: +49 761 7664613 Email: info(a)coresystems.de . http://www.coresystems.de/ Registergericht: Amtsgericht Freiburg . HRB 7656 Geschäftsführer: Stefan Reinauer . Ust-IdNr.: DE245674866

2 1

[commit] r5285 - trunk/src/pc80
by repository service March 24, 2010

March 24, 2010

Author: stepan Date: Wed Mar 24 22:24:17 2010 New Revision: 5285 URL: https://tracker.coreboot.org/trac/coreboot/changeset/5285 Log: Some keyboards need a longer timeout. Also increase error level, because a not-connected keyboard should normally not raise an error. Signed-off-by: Stefan Reinauer <stepan(a)coresystems.de> Acked-by: Myles Watson <mylesgw(a)gmail.com> Modified: trunk/src/pc80/keyboard.c Modified: trunk/src/pc80/keyboard.c ============================================================================== --- trunk/src/pc80/keyboard.c Wed Mar 24 22:21:33 2010 (r5284) +++ trunk/src/pc80/keyboard.c Wed Mar 24 22:24:17 2010 (r5285) @@ -60,8 +60,8 @@ #define KBD_REPLY_ACK 0xFA // Command ACK #define KBD_REPLY_RESEND 0xFE // Command NACK, send command again -/* Wait 200ms for keyboard controller answers */ -#define KBC_TIMEOUT_IN_MS 200 +/* Wait 400ms for keyboard controller answers */ +#define KBC_TIMEOUT_IN_MS 400 static int kbc_input_buffer_empty(void) { @@ -85,7 +85,7 @@ } if (!timeout) { - printk(BIOS_WARNING, "Keyboard controller output buffer result timeout\n"); + printk(BIOS_INFO, "Keyboard controller output buffer result timeout\n"); } return !!timeout; }

1 0

[commit] r5284 - trunk/util/options
by repository service March 24, 2010

March 24, 2010

Author: stepan Date: Wed Mar 24 22:21:33 2010 New Revision: 5284 URL: https://tracker.coreboot.org/trac/coreboot/changeset/5284 Log: Drop unneeded code that checks for CONFIG_ variables in build_opt_tbl. Signed-off-by: Stefan Reinauer <stepan(a)coresystems.de> Acked-by: Myles Watson <mylesgw(a)gmail.com> Modified: trunk/util/options/build_opt_tbl.c Modified: trunk/util/options/build_opt_tbl.c ============================================================================== --- trunk/util/options/build_opt_tbl.c Wed Mar 24 21:59:43 2010 (r5283) +++ trunk/util/options/build_opt_tbl.c Wed Mar 24 22:21:33 2010 (r5284) @@ -472,25 +472,6 @@ fprintf(stderr, "Error - location is to big in line\n%s\n", line); exit(1); } - /* And since we are not ready to be fully general purpose yet.. */ - if ((cs->range_start/8) != CONFIG_LB_CKS_RANGE_START) { - fprintf(stderr, "Error - Range start((value in file #%d), which is byte #%d)" - "does not match the value of the config variable LB_CKS_RANGE_START)(%d) in line\n%s\n", - cs->range_start, cs->range_start/8, CONFIG_LB_CKS_RANGE_START, line); - exit(1); - } - if ((cs->range_end/8) != CONFIG_LB_CKS_RANGE_END) { - fprintf(stderr, "Error - Range end ((value in file #%d), which is byte #%d)" - "does not match the value of the config variable LB_CKS_RANGE_END (%d) in line\n%s\n", - cs->range_end, cs->range_end/8, - CONFIG_LB_CKS_RANGE_END, line); - exit(1); - } - if ((cs->location/8) != CONFIG_LB_CKS_LOC) { - fprintf(stderr, "Error - Location ((value in file #%d), which is byte #%d) does not match LB_CKS_LOC (%d) in line\n%s\n", - cs->location, cs->location/8, CONFIG_LB_CKS_LOC, line); - exit(1); - } cs->tag = LB_TAG_OPTION_CHECKSUM; cs->size = sizeof(*cs);

1 0