March 2013 - coreboot-gerrit - mail.coreboot.org

Patch merged into coreboot/master: 1cc4737 f15tn/Include/OptionIdsInstall.h: Remove idle `… || )`
by gerrit＠coreboot.org March 20, 2013

March 20, 2013

the following patch was just integrated into master: commit 1cc4737c3bc7e0b1e560f3d049f4edbef9340177 Author: Siyuan Wang <wangsiyuanbuaa(a)gmail.com> Date: Wed Mar 20 15:28:31 2013 +0800 f15tn/Include/OptionIdsInstall.h: Remove idle `… || )` Change-Id: I4aba6cc490ab24c6db345c0c5a64a6a9985ed0ab Signed-off-by: Siyuan Wang <SiYuan.Wang(a)amd.com> Signed-off-by: Siyuan Wang <wangsiyuanbuaa(a)gmail.com> Reviewed-on: http://review.coreboot.org/2864 Reviewed-by: Paul Menzel <paulepanter(a)users.sourceforge.net> Tested-by: build bot (Jenkins) Reviewed-by: Marc Jones <marc.jones(a)se-eng.com> Build-Tested: build bot (Jenkins) at Wed Mar 20 10:54:19 2013, giving +1 Reviewed-By: Marc Jones <marc.jones(a)se-eng.com> at Wed Mar 20 17:50:02 2013, giving +2 See http://review.coreboot.org/2864 for details. -gerrit

1 0

Patch set updated for coreboot: b39c4d9 f15tn/Include/OptionIdsInstall.h: Remove idle `… || )`
by Paul Menzel March 20, 2013

March 20, 2013

Paul Menzel (paulepanter(a)users.sourceforge.net) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2864 -gerrit commit b39c4d9a8146d208adb76d7260edd1b65188b2de Author: Siyuan Wang <wangsiyuanbuaa(a)gmail.com> Date: Wed Mar 20 15:28:31 2013 +0800 f15tn/Include/OptionIdsInstall.h: Remove idle `… || )` Change-Id: I4aba6cc490ab24c6db345c0c5a64a6a9985ed0ab Signed-off-by: Siyuan Wang <SiYuan.Wang(a)amd.com> Signed-off-by: Siyuan Wang <wangsiyuanbuaa(a)gmail.com> --- src/vendorcode/amd/agesa/f15tn/Include/OptionIdsInstall.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/vendorcode/amd/agesa/f15tn/Include/OptionIdsInstall.h b/src/vendorcode/amd/agesa/f15tn/Include/OptionIdsInstall.h index 0ad2df2..de512f9 100644 --- a/src/vendorcode/amd/agesa/f15tn/Include/OptionIdsInstall.h +++ b/src/vendorcode/amd/agesa/f15tn/Include/OptionIdsInstall.h @@ -82,7 +82,7 @@ #if (IDSOPT_IDS_ENABLED == TRUE) #if (IDSOPT_CONTROL_ENABLED == TRUE) // Check for all families which include HT Features. - #if ((OPTION_FAMILY10H == TRUE) || (OPTION_FAMILY15H_OR == TRUE) || ) && (AGESA_ENTRY_INIT_POST == TRUE) + #if ((OPTION_FAMILY10H == TRUE) || (OPTION_FAMILY15H_OR == TRUE)) && (AGESA_ENTRY_INIT_POST == TRUE) #undef M_HTIDS_PORT_OVERRIDE_HOOK #define M_HTIDS_PORT_OVERRIDE_HOOK HtIdsGetPortOverride #endif

1 0

New patch to review for coreboot: 0cd3fad clean up code in f15tn/Include/OptionIdsInstall.h
by Siyuan Wang March 20, 2013

March 20, 2013

Siyuan Wang (wangsiyuanbuaa(a)gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2864 -gerrit commit 0cd3fad471c4e3811019f68b4e24978fcb35b966 Author: Siyuan Wang <wangsiyuanbuaa(a)gmail.com> Date: Wed Mar 20 15:28:31 2013 +0800 clean up code in f15tn/Include/OptionIdsInstall.h Change-Id: I4aba6cc490ab24c6db345c0c5a64a6a9985ed0ab Signed-off-by: Siyuan Wang <SiYuan.Wang(a)amd.com> Signed-off-by: Siyuan Wang <wangsiyuanbuaa(a)gmail.com> --- src/vendorcode/amd/agesa/f15tn/Include/OptionIdsInstall.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/vendorcode/amd/agesa/f15tn/Include/OptionIdsInstall.h b/src/vendorcode/amd/agesa/f15tn/Include/OptionIdsInstall.h index 0ad2df2..de512f9 100644 --- a/src/vendorcode/amd/agesa/f15tn/Include/OptionIdsInstall.h +++ b/src/vendorcode/amd/agesa/f15tn/Include/OptionIdsInstall.h @@ -82,7 +82,7 @@ #if (IDSOPT_IDS_ENABLED == TRUE) #if (IDSOPT_CONTROL_ENABLED == TRUE) // Check for all families which include HT Features. - #if ((OPTION_FAMILY10H == TRUE) || (OPTION_FAMILY15H_OR == TRUE) || ) && (AGESA_ENTRY_INIT_POST == TRUE) + #if ((OPTION_FAMILY10H == TRUE) || (OPTION_FAMILY15H_OR == TRUE)) && (AGESA_ENTRY_INIT_POST == TRUE) #undef M_HTIDS_PORT_OVERRIDE_HOOK #define M_HTIDS_PORT_OVERRIDE_HOOK HtIdsGetPortOverride #endif

1 0