I've fixed 45 format-related warnings/errors of IDS_HDT_CONSOLE at AMD
16h vendorcode. Now a coreboot for fam16 could be built successfully
with IDS_HDT_CONSOLE enabled (IDSOPT_IDS_ENABLED TRUE and
IDSOPT_TRACING_ENABLED TRUE at board/OptionsIds.h) even with config
WARNINGS_ARE_ERRORS. Joined errors log (before a fix) could be seen at
attachments and here - https://pastebin.com/RZQBvTEc , will upload a
patch to review.coreboot.org ASAP. If it's successful, I could try
doing the same for the other AGESA platforms.

P.S. IDS_HDT_CONSOLE could be really helpful for debugging, although
so much logging causes a board to boot very slowly - i.e. 25 minutes.
These logs could be captured using USB EHCI dongle like FT232H.

On Sun, Nov 10, 2019 at 6:16 PM Sergej Ivanov <getin...@gmail.com> wrote:
>
> I have same issue, memtest86+ failed on all fam16 family for me(AM1ML, AMD 
> devboard and so on) (usually in first 10Mb and near a dedicated video 
> memory). BTW i'm choose to use last free memtest86 (without plus), it work's.
>
> вс, 10 нояб. 2019 г. в 21:07, Mike Banon <mikeb...@gmail.com>:
>>
>> No problem (this log is really huge for some reason). At the beginning
>> of it - the most interesting part - I've found that "MemBusFreqLimit :
>> 800" and "MemoryClockSelect : 667". Perhaps I'd try again forcing
>> either 933 (for 1866, although don't know if possible on this
>> platform) or 800 (for 1600) at the vendorcode. Another interesting
>> thing is that memtest86+ is failing but all the OS are working
>> correctly.
>>
>> On Sun, Nov 10, 2019 at 7:16 PM Kyösti Mälkki <kyosti.mal...@gmail.com> 
>> wrote:
>> >
>> > On Sun, Nov 10, 2019 at 3:52 PM Mike Banon <mikeb...@gmail.com> wrote:
>> > >
>> > > Sorry it took me so long: using FT232H and screen Ctrl+A h option,
>> > > finally I got a complete boot log on AM1I-A Fam16h with these IDS
>> > > options enabled. Please could you take a look, to see if it contains
>> > > any hints why this 1866MHz CL9 RAM has been initialized as 1333MHz CL9
>> > > only?
>> >
>> > Not really, I expect it to involve several hours of work with little or no 
>> > gain.
>> >
>> > Kyösti
>> _______________________________________________
>> coreboot mailing list -- coreboot@coreboot.org
>> To unsubscribe send an email to coreboot-le...@coreboot.org
>
>
>
> --
> Мастерство Поклона 7 \\ Увеличения нападают на власть, используя поклоны.
# ===> ids_01.txt

    CC         
libagesa/vendorcode/amd/agesa/f16kb/Proc/CPU/Family/0x16/cpuF16MmioMap.o
In file included from src/include/assert.h:20,
                 from src/vendorcode/amd/include/Porting.h:276,
                 from src/vendorcode/amd/agesa/f16kb/AGESA.h:47,
                 from 
src/vendorcode/amd/agesa/f16kb/Proc/CPU/Family/0x16/cpuF16MmioMap.c:49:
src/vendorcode/amd/agesa/f16kb/Proc/CPU/Family/0x16/cpuF16MmioMap.c: In 
function 'cpuF16AddingMmioMap':
src/vendorcode/amd/agesa/f16kb/Proc/CPU/Family/0x16/cpuF16MmioMap.c:158:33: 
error: format '%x' expects argument of type 'unsigned int', but argument 3 has 
type 'UINT64' {aka 'long long unsigned int'} [-Werror=format=]
     IDS_HDT_CONSOLE (MAIN_FLOW, "%08x%08x    %08x%08x", 
(MmioRange[MmioPair].Base >> 32) & 0xFFFFFFFF,
                                 ^~~~~~~~~~~~~~~~~~~~~~  
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/CPU/Family/0x16/cpuF16MmioMap.c:158:5: 
note: in expansion of macro 'IDS_HDT_CONSOLE'
     IDS_HDT_CONSOLE (MAIN_FLOW, "%08x%08x    %08x%08x", 
(MmioRange[MmioPair].Base >> 32) & 0xFFFFFFFF,
     ^~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/CPU/Family/0x16/cpuF16MmioMap.c:158:33: 
error: format '%x' expects argument of type 'unsigned int', but argument 4 has 
type 'UINT64' {aka 'long long unsigned int'} [-Werror=format=]
     IDS_HDT_CONSOLE (MAIN_FLOW, "%08x%08x    %08x%08x", 
(MmioRange[MmioPair].Base >> 32) & 0xFFFFFFFF,
                                 ^~~~~~~~~~~~~~~~~~~~~~
                                                          
MmioRange[MmioPair].Base & 0xFFFFFFFF,
                                                          
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/CPU/Family/0x16/cpuF16MmioMap.c:158:5: 
note: in expansion of macro 'IDS_HDT_CONSOLE'
     IDS_HDT_CONSOLE (MAIN_FLOW, "%08x%08x    %08x%08x", 
(MmioRange[MmioPair].Base >> 32) & 0xFFFFFFFF,
     ^~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/CPU/Family/0x16/cpuF16MmioMap.c:158:33: 
error: format '%x' expects argument of type 'unsigned int', but argument 5 has 
type 'UINT64' {aka 'long long unsigned int'} [-Werror=format=]
     IDS_HDT_CONSOLE (MAIN_FLOW, "%08x%08x    %08x%08x", 
(MmioRange[MmioPair].Base >> 32) & 0xFFFFFFFF,
                                 ^~~~~~~~~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/CPU/Family/0x16/cpuF16MmioMap.c:160:57:
                                                         
(MmioRange[MmioPair].Limit >> 32) & 0xFFFFFFFF,
                                                         
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/CPU/Family/0x16/cpuF16MmioMap.c:158:5: 
note: in expansion of macro 'IDS_HDT_CONSOLE'
     IDS_HDT_CONSOLE (MAIN_FLOW, "%08x%08x    %08x%08x", 
(MmioRange[MmioPair].Base >> 32) & 0xFFFFFFFF,
     ^~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/CPU/Family/0x16/cpuF16MmioMap.c:158:33: 
error: format '%x' expects argument of type 'unsigned int', but argument 6 has 
type 'UINT64' {aka 'long long unsigned int'} [-Werror=format=]
     IDS_HDT_CONSOLE (MAIN_FLOW, "%08x%08x    %08x%08x", 
(MmioRange[MmioPair].Base >> 32) & 0xFFFFFFFF,
                                 ^~~~~~~~~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/CPU/Family/0x16/cpuF16MmioMap.c:161:58:
                                                          
MmioRange[MmioPair].Limit & 0xFFFFFFFF);
                                                          
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/CPU/Family/0x16/cpuF16MmioMap.c:158:5: 
note: in expansion of macro 'IDS_HDT_CONSOLE'
     IDS_HDT_CONSOLE (MAIN_FLOW, "%08x%08x    %08x%08x", 
(MmioRange[MmioPair].Base >> 32) & 0xFFFFFFFF,
     ^~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/CPU/Family/0x16/cpuF16MmioMap.c:175:31: 
error: format '%x' expects argument of type 'unsigned int', but argument 3 has 
type 'UINT64' {aka 'long long unsigned int'} [-Werror=format=]
   IDS_HDT_CONSOLE (MAIN_FLOW, "req %08x%08x    %08x%08x\n", (NewMmioRange.Base 
>> 32) & 0xFFFFFFFF,
                               ^~~~~~~~~~~~~~~~~~~~~~~~~~~~  
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/CPU/Family/0x16/cpuF16MmioMap.c:175:3: 
note: in expansion of macro 'IDS_HDT_CONSOLE'
   IDS_HDT_CONSOLE (MAIN_FLOW, "req %08x%08x    %08x%08x\n", (NewMmioRange.Base 
>> 32) & 0xFFFFFFFF,
   ^~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/CPU/Family/0x16/cpuF16MmioMap.c:175:31: 
error: format '%x' expects argument of type 'unsigned int', but argument 4 has 
type 'UINT64' {aka 'long long unsigned int'} [-Werror=format=]
   IDS_HDT_CONSOLE (MAIN_FLOW, "req %08x%08x    %08x%08x\n", (NewMmioRange.Base 
>> 32) & 0xFFFFFFFF,
                               ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
                                                          NewMmioRange.Base & 
0xFFFFFFFF,
                                                          
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/CPU/Family/0x16/cpuF16MmioMap.c:175:3: 
note: in expansion of macro 'IDS_HDT_CONSOLE'
   IDS_HDT_CONSOLE (MAIN_FLOW, "req %08x%08x    %08x%08x\n", (NewMmioRange.Base 
>> 32) & 0xFFFFFFFF,
   ^~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/CPU/Family/0x16/cpuF16MmioMap.c:175:31: 
error: format '%x' expects argument of type 'unsigned int', but argument 5 has 
type 'UINT64' {aka 'long long unsigned int'} [-Werror=format=]
   IDS_HDT_CONSOLE (MAIN_FLOW, "req %08x%08x    %08x%08x\n", (NewMmioRange.Base 
>> 32) & 0xFFFFFFFF,
                               ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/CPU/Family/0x16/cpuF16MmioMap.c:177:57:
                                                         (NewMmioRange.Limit >> 
32) & 0xFFFFFFFF,
                                                         
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/CPU/Family/0x16/cpuF16MmioMap.c:175:3: 
note: in expansion of macro 'IDS_HDT_CONSOLE'
   IDS_HDT_CONSOLE (MAIN_FLOW, "req %08x%08x    %08x%08x\n", (NewMmioRange.Base 
>> 32) & 0xFFFFFFFF,
   ^~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/CPU/Family/0x16/cpuF16MmioMap.c:175:31: 
error: format '%x' expects argument of type 'unsigned int', but argument 6 has 
type 'UINT64' {aka 'long long unsigned int'} [-Werror=format=]
   IDS_HDT_CONSOLE (MAIN_FLOW, "req %08x%08x    %08x%08x\n", (NewMmioRange.Base 
>> 32) & 0xFFFFFFFF,
                               ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/CPU/Family/0x16/cpuF16MmioMap.c:178:58:
                                                          NewMmioRange.Limit & 
0xFFFFFFFF);
                                                          
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/CPU/Family/0x16/cpuF16MmioMap.c:175:3: 
note: in expansion of macro 'IDS_HDT_CONSOLE'
   IDS_HDT_CONSOLE (MAIN_FLOW, "req %08x%08x    %08x%08x\n", (NewMmioRange.Base 
>> 32) & 0xFFFFFFFF,
   ^~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/CPU/Family/0x16/cpuF16MmioMap.c:417:41: 
error: format '%x' expects argument of type 'unsigned int', but argument 3 has 
type 'UINT64' {aka 'long long unsigned int'} [-Werror=format=]
             IDS_HDT_CONSOLE (MAIN_FLOW, "%08x%08x    %08x%08x", 
(MmioRange[MmioPair].Base >> 32) & 0xFFFFFFFF,
                                         ^~~~~~~~~~~~~~~~~~~~~~  
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/CPU/Family/0x16/cpuF16MmioMap.c:417:13: 
note: in expansion of macro 'IDS_HDT_CONSOLE'
             IDS_HDT_CONSOLE (MAIN_FLOW, "%08x%08x    %08x%08x", 
(MmioRange[MmioPair].Base >> 32) & 0xFFFFFFFF,
             ^~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/CPU/Family/0x16/cpuF16MmioMap.c:417:41: 
error: format '%x' expects argument of type 'unsigned int', but argument 4 has 
type 'UINT64' {aka 'long long unsigned int'} [-Werror=format=]
             IDS_HDT_CONSOLE (MAIN_FLOW, "%08x%08x    %08x%08x", 
(MmioRange[MmioPair].Base >> 32) & 0xFFFFFFFF,
                                         ^~~~~~~~~~~~~~~~~~~~~~
                                                                  
MmioRange[MmioPair].Base & 0xFFFFFFFF,
                                                                  
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/CPU/Family/0x16/cpuF16MmioMap.c:417:13: 
note: in expansion of macro 'IDS_HDT_CONSOLE'
             IDS_HDT_CONSOLE (MAIN_FLOW, "%08x%08x    %08x%08x", 
(MmioRange[MmioPair].Base >> 32) & 0xFFFFFFFF,
             ^~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/CPU/Family/0x16/cpuF16MmioMap.c:417:41: 
error: format '%x' expects argument of type 'unsigned int', but argument 5 has 
type 'UINT64' {aka 'long long unsigned int'} [-Werror=format=]
             IDS_HDT_CONSOLE (MAIN_FLOW, "%08x%08x    %08x%08x", 
(MmioRange[MmioPair].Base >> 32) & 0xFFFFFFFF,
                                         ^~~~~~~~~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/CPU/Family/0x16/cpuF16MmioMap.c:419:65:
                                                                 
(MmioRange[MmioPair].Limit >> 32) & 0xFFFFFFFF,
                                                                 
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/CPU/Family/0x16/cpuF16MmioMap.c:417:13: 
note: in expansion of macro 'IDS_HDT_CONSOLE'
             IDS_HDT_CONSOLE (MAIN_FLOW, "%08x%08x    %08x%08x", 
(MmioRange[MmioPair].Base >> 32) & 0xFFFFFFFF,
             ^~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/CPU/Family/0x16/cpuF16MmioMap.c:417:41: 
error: format '%x' expects argument of type 'unsigned int', but argument 6 has 
type 'UINT64' {aka 'long long unsigned int'} [-Werror=format=]
             IDS_HDT_CONSOLE (MAIN_FLOW, "%08x%08x    %08x%08x", 
(MmioRange[MmioPair].Base >> 32) & 0xFFFFFFFF,
                                         ^~~~~~~~~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/CPU/Family/0x16/cpuF16MmioMap.c:420:66:
                                                                  
MmioRange[MmioPair].Limit & 0xFFFFFFFF);
                                                                  
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/CPU/Family/0x16/cpuF16MmioMap.c:417:13: 
note: in expansion of macro 'IDS_HDT_CONSOLE'
             IDS_HDT_CONSOLE (MAIN_FLOW, "%08x%08x    %08x%08x", 
(MmioRange[MmioPair].Base >> 32) & 0xFFFFFFFF,
             ^~~~~~~~~~~~~~~

# ===> ids_02.txt

    CC         libagesa/vendorcode/amd/agesa/f16kb/Proc/CPU/cpuMicrocodePatch.o
In file included from src/include/assert.h:20,
                 from src/vendorcode/amd/include/Porting.h:276,
                 from src/vendorcode/amd/agesa/f16kb/AGESA.h:47,
                 from 
src/vendorcode/amd/agesa/f16kb/Proc/CPU/cpuMicrocodePatch.c:49:
src/vendorcode/amd/agesa/f16kb/Proc/CPU/cpuMicrocodePatch.c: In function 
'LoadMicrocodePatch':
src/vendorcode/amd/agesa/f16kb/Proc/CPU/cpuMicrocodePatch.c:151:37: error: 
format '%x' expects argument of type 'unsigned int', but argument 3 has type 
'MICROCODE_PATCH *' {aka 'struct <anonymous> *'} [-Werror=format=]
         IDS_HDT_CONSOLE (CPU_TRACE, "  Force Ucode loaded from offset %x\n", 
ForceLoadMicrocodePatchPtr);
                                     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~  
~~~~~~~~~~~~~~~~~~~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/CPU/cpuMicrocodePatch.c:151:9: note: in 
expansion of macro 'IDS_HDT_CONSOLE'
         IDS_HDT_CONSOLE (CPU_TRACE, "  Force Ucode loaded from offset %x\n", 
ForceLoadMicrocodePatchPtr);
         ^~~~~~~~~~~~~~~

# ===> ids_03.txt

    CC         libagesa/vendorcode/amd/agesa/f16kb/Proc/Common/AmdInitReset.o
In file included from src/include/assert.h:20,
                 from src/vendorcode/amd/include/Porting.h:276,
                 from src/vendorcode/amd/agesa/f16kb/AGESA.h:47,
                 from 
src/vendorcode/amd/agesa/f16kb/Proc/Common/AmdInitReset.c:49:
src/vendorcode/amd/agesa/f16kb/Proc/Common/AmdInitReset.c: In function 
'AmdInitReset':
src/vendorcode/amd/agesa/f16kb/Proc/Common/AmdInitReset.c:157:31: error: format 
'%s' expects argument of type 'char *', but argument 3 has type 
'AMD_CODE_HEADER *' {aka 'struct <anonymous> *'} [-Werror=format=]
   IDS_HDT_CONSOLE (MAIN_FLOW, "\n*** %s ***\n\n", &UserOptions.VersionString);
                               ^~~~~~~~~~~~~~~~~~  ~~~~~~~~~~~~~~~~~~~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/Common/AmdInitReset.c:157:3: note: in 
expansion of macro 'IDS_HDT_CONSOLE'
   IDS_HDT_CONSOLE (MAIN_FLOW, "\n*** %s ***\n\n", &UserOptions.VersionString);
   ^~~~~~~~~~~~~~~

# ===> ids_04.txt

    CC         libagesa/vendorcode/amd/agesa/f16kb/Proc/Common/S3RestoreState.o
In file included from src/include/assert.h:20,
                 from src/vendorcode/amd/include/Porting.h:276,
                 from 
src/vendorcode/amd/agesa/f16kb/Proc/Common/S3RestoreState.c:50:
src/vendorcode/amd/agesa/f16kb/Proc/Common/S3RestoreState.c: In function 
'S3RestoreStateFromTable':
src/vendorcode/amd/agesa/f16kb/Proc/Common/S3RestoreState.c:168:30: error: too 
many arguments for format [-Werror=format-extra-args]
   IDS_HDT_CONSOLE (S3_TRACE, "Start S3 restore\n", ((S3_WRITE_OP_HEADER *) 
S3SaveTableRecordPtr)->Address);
                              ^~~~~~~~~~~~~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/Common/S3RestoreState.c:168:3: note: in 
expansion of macro 'IDS_HDT_CONSOLE'
   IDS_HDT_CONSOLE (S3_TRACE, "Start S3 restore\n", ((S3_WRITE_OP_HEADER *) 
S3SaveTableRecordPtr)->Address);
   ^~~~~~~~~~~~~~~

# ===> ids_05.txt

    CC         libagesa/vendorcode/amd/agesa/f16kb/Proc/Common/S3SaveState.o
In file included from 
src/vendorcode/amd/agesa/f16kb/Proc/Common/S3SaveState.c:56:
src/vendorcode/amd/agesa/f16kb/Proc/Common/S3SaveState.c: In function 
'S3SaveStateSaveWriteOp':
src/vendorcode/amd/agesa/f16kb/Proc/Common/S3SaveState.c:271:32: error: format 
'%x' expects argument of type 'unsigned int', but argument 4 has type 'UINT64' 
{aka 'long long unsigned int'} [-Werror=format=]
     IDS_HDT_CONSOLE (S3_TRACE, "  S3 Save: %s Address: 0x%08x Data: ", 
S3SaveDebugOpcodeString (StdHeader, OpCode), Address);
                                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~          
                                     ~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/Common/S3SaveState.h:52:38: note: in 
definition of macro 'S3_SCRIPT_DEBUG_CODE'
   #define S3_SCRIPT_DEBUG_CODE(Code) Code
                                      ^~~~
src/vendorcode/amd/agesa/f16kb/Include/Ids.h:744:40: note: in expansion of 
macro 'printk'
     #define IDS_HDT_CONSOLE(f, s, ...) printk (BIOS_DEBUG, s, ##__VA_ARGS__);
                                        ^~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/Common/S3SaveState.c:271:5: note: in 
expansion of macro 'IDS_HDT_CONSOLE'
     IDS_HDT_CONSOLE (S3_TRACE, "  S3 Save: %s Address: 0x%08x Data: ", 
S3SaveDebugOpcodeString (StdHeader, OpCode), Address);
     ^~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/Common/S3SaveState.c: In function 
'S3SaveStateSaveReadWriteOp':
src/vendorcode/amd/agesa/f16kb/Proc/Common/S3SaveState.c:336:32: error: format 
'%x' expects argument of type 'unsigned int', but argument 4 has type 'UINT64' 
{aka 'long long unsigned int'} [-Werror=format=]
     IDS_HDT_CONSOLE (S3_TRACE, "  S3 Save: %s Address: 0x%08x Data: ", 
S3SaveDebugOpcodeString (StdHeader, OpCode), Address);
                                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~          
                                     ~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/Common/S3SaveState.h:52:38: note: in 
definition of macro 'S3_SCRIPT_DEBUG_CODE'
   #define S3_SCRIPT_DEBUG_CODE(Code) Code
                                      ^~~~
src/vendorcode/amd/agesa/f16kb/Include/Ids.h:744:40: note: in expansion of 
macro 'printk'
     #define IDS_HDT_CONSOLE(f, s, ...) printk (BIOS_DEBUG, s, ##__VA_ARGS__);
                                        ^~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/Common/S3SaveState.c:336:5: note: in 
expansion of macro 'IDS_HDT_CONSOLE'
     IDS_HDT_CONSOLE (S3_TRACE, "  S3 Save: %s Address: 0x%08x Data: ", 
S3SaveDebugOpcodeString (StdHeader, OpCode), Address);
     ^~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/Common/S3SaveState.c: In function 
'S3SaveStateSavePollOp':
src/vendorcode/amd/agesa/f16kb/Proc/Common/S3SaveState.c:412:32: error: format 
'%x' expects argument of type 'unsigned int', but argument 4 has type 'UINT64' 
{aka 'long long unsigned int'} [-Werror=format=]
     IDS_HDT_CONSOLE (S3_TRACE, "  S3 Save: %s Address: 0x%08x Data: ", 
S3SaveDebugOpcodeString (StdHeader, OpCode), Address);
                                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~          
                                     ~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/Common/S3SaveState.h:52:38: note: in 
definition of macro 'S3_SCRIPT_DEBUG_CODE'
   #define S3_SCRIPT_DEBUG_CODE(Code) Code
                                      ^~~~
src/vendorcode/amd/agesa/f16kb/Include/Ids.h:744:40: note: in expansion of 
macro 'printk'
     #define IDS_HDT_CONSOLE(f, s, ...) printk (BIOS_DEBUG, s, ##__VA_ARGS__);
                                        ^~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/Common/S3SaveState.c:412:5: note: in 
expansion of macro 'IDS_HDT_CONSOLE'
     IDS_HDT_CONSOLE (S3_TRACE, "  S3 Save: %s Address: 0x%08x Data: ", 
S3SaveDebugOpcodeString (StdHeader, OpCode), Address);
     ^~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/Common/S3SaveState.c: In function 
'S3SaveStateSaveInfoOp':
src/vendorcode/amd/agesa/f16kb/Proc/Common/S3SaveState.c:484:32: error: format 
'%s' expects argument of type 'char *', but argument 3 has type 'VOID *' {aka 
'void *'} [-Werror=format=]
     IDS_HDT_CONSOLE (S3_TRACE, "  S3 Save: Info: %s \n", Information);
                                ^~~~~~~~~~~~~~~~~~~~~~~~  ~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/Common/S3SaveState.h:52:38: note: in 
definition of macro 'S3_SCRIPT_DEBUG_CODE'
   #define S3_SCRIPT_DEBUG_CODE(Code) Code
                                      ^~~~
src/vendorcode/amd/agesa/f16kb/Include/Ids.h:744:40: note: in expansion of 
macro 'printk'
     #define IDS_HDT_CONSOLE(f, s, ...) printk (BIOS_DEBUG, s, ##__VA_ARGS__);
                                        ^~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/Common/S3SaveState.c:484:5: note: in 
expansion of macro 'IDS_HDT_CONSOLE'
     IDS_HDT_CONSOLE (S3_TRACE, "  S3 Save: Info: %s \n", Information);
     ^~~~~~~~~~~~~~~

# ===> ids_06.txt

    CC         
libagesa/vendorcode/amd/agesa/f16kb/Proc/Fch/Interface/InitEnvDef.o
In file included from src/include/assert.h:20,
                 from src/vendorcode/amd/include/Porting.h:276,
                 from src/vendorcode/amd/agesa/f16kb/AGESA.h:47,
                 from src/vendorcode/amd/agesa/f16kb/Proc/Fch/FchPlatform.h:49,
                 from 
src/vendorcode/amd/agesa/f16kb/Proc/Fch/Interface/InitEnvDef.c:43:
src/vendorcode/amd/agesa/f16kb/Proc/Fch/Interface/InitEnvDef.c: In function 
'FchInitEnvCreatePrivateData':
src/vendorcode/amd/agesa/f16kb/Proc/Fch/Interface/InitEnvDef.c:141:31: error: 
format '%x' expects argument of type 'unsigned int', but argument 4 has type 
'FCH_DATA_BLOCK *' {aka 'struct _FCH_DATA_BLOCK *'} [-Werror=format=]
   IDS_HDT_CONSOLE (FCH_TRACE, "    FCH Data Block Allocation: [0x%x], Ptr = 
0x%08x\n", AgesaStatus, FchParams);
                               
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~               ~~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/Fch/Interface/InitEnvDef.c:141:3: note: in 
expansion of macro 'IDS_HDT_CONSOLE'
   IDS_HDT_CONSOLE (FCH_TRACE, "    FCH Data Block Allocation: [0x%x], Ptr = 
0x%08x\n", AgesaStatus, FchParams);
   ^~~~~~~~~~~~~~~

# ===> ids_07.txt

    CC         
libagesa/vendorcode/amd/agesa/f16kb/Proc/Fch/Interface/InitResetDef.o
In file included from src/include/assert.h:20,
                 from src/vendorcode/amd/include/Porting.h:276,
                 from src/vendorcode/amd/agesa/f16kb/AGESA.h:47,
                 from src/vendorcode/amd/agesa/f16kb/Proc/Fch/FchPlatform.h:49,
                 from 
src/vendorcode/amd/agesa/f16kb/Proc/Fch/Interface/InitResetDef.c:43:
src/vendorcode/amd/agesa/f16kb/Proc/Fch/Interface/InitResetDef.c: In function 
'FchInitResetLoadPrivateDefault':
src/vendorcode/amd/agesa/f16kb/Proc/Fch/Interface/InitResetDef.c:74:31: error: 
format '%x' expects argument of type 'unsigned int', but argument 4 has type 
'FCH_RESET_DATA_BLOCK *' {aka 'struct _FCH_RESET_DATA_BLOCK *'} 
[-Werror=format=]
   IDS_HDT_CONSOLE (FCH_TRACE, "    FCH Reset Data Block Allocation: [0x%x], 
Ptr = 0x%08x\n", AgesaStatus, FchParams);
                               
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~               
~~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/Fch/Interface/InitResetDef.c:74:3: note: in 
expansion of macro 'IDS_HDT_CONSOLE'
   IDS_HDT_CONSOLE (FCH_TRACE, "    FCH Reset Data Block Allocation: [0x%x], 
Ptr = 0x%08x\n", AgesaStatus, FchParams);
   ^~~~~~~~~~~~~~~

# ===> ids_08.txt

    CC         
libagesa/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbCommonLib/GnbLib.o
In file included from src/include/assert.h:20,
                 from src/vendorcode/amd/include/Porting.h:276,
                 from src/vendorcode/amd/agesa/f16kb/AGESA.h:47,
                 from 
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbCommonLib/GnbLib.c:50:
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbCommonLib/GnbLib.c: In 
function 'GnbLibDebugDumpBuffer':
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbCommonLib/GnbLib.c:485:35: 
error: conversion lacks type at end of format [-Werror=format=]
       IDS_HDT_CONSOLE (GNB_TRACE, "%08x%08", *(UINT32 *) ((UINT8 *) Buffer + 
Index), *(UINT32 *) ((UINT8 *) Buffer + Index + 4));
                                   ^~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbCommonLib/GnbLib.c:485:7: 
note: in expansion of macro 'IDS_HDT_CONSOLE'
       IDS_HDT_CONSOLE (GNB_TRACE, "%08x%08", *(UINT32 *) ((UINT8 *) Buffer + 
Index), *(UINT32 *) ((UINT8 *) Buffer + Index + 4));
       ^~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbCommonLib/GnbLib.c:485:35: 
error: too many arguments for format [-Werror=format-extra-args]
       IDS_HDT_CONSOLE (GNB_TRACE, "%08x%08", *(UINT32 *) ((UINT8 *) Buffer + 
Index), *(UINT32 *) ((UINT8 *) Buffer + Index + 4));
                                   ^~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbCommonLib/GnbLib.c:485:7: 
note: in expansion of macro 'IDS_HDT_CONSOLE'
       IDS_HDT_CONSOLE (GNB_TRACE, "%08x%08", *(UINT32 *) ((UINT8 *) Buffer + 
Index), *(UINT32 *) ((UINT8 *) Buffer + Index + 4));
       ^~~~~~~~~~~~~~~

# ===> ids_09.txt

    CC         
libagesa/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbGfxConfig/GfxConfigLib.o
In file included from src/include/assert.h:20,
                 from src/vendorcode/amd/include/Porting.h:276,
                 from src/vendorcode/amd/agesa/f16kb/AGESA.h:47,
                 from 
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbGfxConfig/GfxConfigLib.c:50:
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbGfxConfig/GfxConfigLib.c: In 
function 'GfxConfigDebugDump':
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbGfxConfig/GfxConfigLib.c:258:32:
 error: format '%x' expects argument of type 'unsigned int', but argument 3 has 
type 'UINT64' {aka 'long long unsigned int'} [-Werror=format=]
     IDS_HDT_CONSOLE (GFX_MISC, "  UmaBase - 0x%x\n", Gfx->UmaInfo.UmaBase);
                                ^~~~~~~~~~~~~~~~~~~~  ~~~~~~~~~~~~~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbGfxConfig/GfxConfigLib.c:258:5:
 note: in expansion of macro 'IDS_HDT_CONSOLE'
     IDS_HDT_CONSOLE (GFX_MISC, "  UmaBase - 0x%x\n", Gfx->UmaInfo.UmaBase);
     ^~~~~~~~~~~~~~~

# ===> ids_10.txt

    CC         
libagesa/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbGfxIntTableV3/GfxIntegratedInfoTable.o
In file included from src/include/assert.h:20,
                 from src/vendorcode/amd/include/Porting.h:276,
                 from src/vendorcode/amd/agesa/f16kb/AGESA.h:47,
                 from 
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbGfxIntTableV3/GfxIntegratedInfoTable.c:49:
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbGfxIntTableV3/GfxIntegratedInfoTable.c:
 In function 'GfxIntInfoTableDebugDumpV3':
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbGfxIntTableV3/GfxIntegratedInfoTable.c:498:30:
 error: format '%d' expects argument of type 'int', but argument 3 has type 
'long unsigned int' [-Werror=format=]
   IDS_HDT_CONSOLE (GFX_MISC, "                    SizeOf %d\n", 
sizeof(ATOM_PPLIB_EXTENDEDHEADER));
                              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~  
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbGfxIntTableV3/GfxIntegratedInfoTable.c:498:3:
 note: in expansion of macro 'IDS_HDT_CONSOLE'
   IDS_HDT_CONSOLE (GFX_MISC, "                    SizeOf %d\n", 
sizeof(ATOM_PPLIB_EXTENDEDHEADER));
   ^~~~~~~~~~~~~~~

# ===> ids_11.txt

    CC         
libagesa/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbGfxIntTableV3/GfxPwrPlayTable.o
In file included from src/include/assert.h:20,
                 from src/vendorcode/amd/include/Porting.h:276,
                 from src/vendorcode/amd/agesa/f16kb/AGESA.h:47,
                 from 
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbGfxIntTableV3/GfxPwrPlayTable.c:51:
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbGfxIntTableV3/GfxPwrPlayTable.c:
 In function 'GfxIntDebugDumpPpTable':
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbGfxIntTableV3/GfxPwrPlayTable.c:1161:32:
 error: format '%d' expects argument of type 'int', but argument 3 has type 
'long unsigned int' [-Werror=format=]
     IDS_HDT_CONSOLE (GFX_MISC, "  State #%d\n", Index + 1
                                ^~~~~~~~~~~~~~~  ~~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbGfxIntTableV3/GfxPwrPlayTable.c:1161:5:
 note: in expansion of macro 'IDS_HDT_CONSOLE'
     IDS_HDT_CONSOLE (GFX_MISC, "  State #%d\n", Index + 1
     ^~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbGfxIntTableV3/GfxPwrPlayTable.c:1176:33:
 error: format '%x' expects argument of type 'unsigned int', but argument 3 has 
type 'ATOM_PPLIB_EXTENDEDHEADER *' {aka 'struct <anonymous> *'} 
[-Werror=format=]
     IDS_HDT_CONSOLE (GNB_TRACE, "ExtendedHeader = %08x\n", ExtendedHeader);
                                 ^~~~~~~~~~~~~~~~~~~~~~~~~  ~~~~~~~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbGfxIntTableV3/GfxPwrPlayTable.c:1176:5:
 note: in expansion of macro 'IDS_HDT_CONSOLE'
     IDS_HDT_CONSOLE (GNB_TRACE, "ExtendedHeader = %08x\n", ExtendedHeader);
     ^~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbGfxIntTableV3/GfxPwrPlayTable.c:1206:34:
 error: format '%d' expects argument of type 'int', but argument 3 has type 
'UINTN' {aka 'long unsigned int'} [-Werror=format=]
       IDS_HDT_CONSOLE (GFX_MISC, "  VCE Voltage Record #%d\n", Index
                                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~  ~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbGfxIntTableV3/GfxPwrPlayTable.c:1206:7:
 note: in expansion of macro 'IDS_HDT_CONSOLE'
       IDS_HDT_CONSOLE (GFX_MISC, "  VCE Voltage Record #%d\n", Index
       ^~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbGfxIntTableV3/GfxPwrPlayTable.c:1219:34:
 error: format '%d' expects argument of type 'int', but argument 3 has type 
'UINTN' {aka 'long unsigned int'} [-Werror=format=]
       IDS_HDT_CONSOLE (GFX_MISC, "  SAMU Voltage Record #%d\n", Index
                                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~  ~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbGfxIntTableV3/GfxPwrPlayTable.c:1219:7:
 note: in expansion of macro 'IDS_HDT_CONSOLE'
       IDS_HDT_CONSOLE (GFX_MISC, "  SAMU Voltage Record #%d\n", Index
       ^~~~~~~~~~~~~~~

# ===> ids_12.txt

    CC         
libagesa/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbNbInitLibV4/GnbNbInitLibV4.o
In file included from src/include/assert.h:20,
                 from src/vendorcode/amd/include/Porting.h:276,
                 from src/vendorcode/amd/agesa/f16kb/AGESA.h:47,
                 from 
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbNbInitLibV4/GnbNbInitLibV4.c:50:
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbNbInitLibV4/GnbNbInitLibV4.c:
 In function 'GnbEnableIommuMmioV4':
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbNbInitLibV4/GnbNbInitLibV4.c:374:35:
 error: format '%x' expects argument of type 'unsigned int', but argument 3 has 
type 'UINT64' {aka 'long long unsigned int'} [-Werror=format=]
       IDS_HDT_CONSOLE (GNB_TRACE, "  Enable IOMMU MMIO at address %x for 
Socket %d Silicon %d\n", BaseAddress, GnbGetSocketId (GnbHandle) , 
GnbGetSiliconId (GnbHandle));
                                   
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~  ~~~~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbNbInitLibV4/GnbNbInitLibV4.c:374:7:
 note: in expansion of macro 'IDS_HDT_CONSOLE'
       IDS_HDT_CONSOLE (GNB_TRACE, "  Enable IOMMU MMIO at address %x for 
Socket %d Silicon %d\n", BaseAddress, GnbGetSocketId (GnbHandle) , 
GnbGetSiliconId (GnbHandle));
       ^~~~~~~~~~~~~~~

# ===> ids_13.txt

    CC         
libagesa/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbPcieConfig/PcieConfigLib.o
In file included from src/include/assert.h:20,
                 from src/vendorcode/amd/include/Porting.h:276,
                 from src/vendorcode/amd/agesa/f16kb/AGESA.h:47,
                 from 
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbPcieConfig/PcieConfigLib.c:50:
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbPcieConfig/PcieConfigLib.c: 
In function 'PcieConfigEngineDebugDump':
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbPcieConfig/PcieConfigLib.c:560:33:
 error: format '%x' expects argument of type 'unsigned int', but argument 3 has 
type 'PCIe_PORT_MISC_CONTROL' {aka 'struct <anonymous>'} [-Werror=format=]
     IDS_HDT_CONSOLE (PCIE_MISC, "          Misc Control  - 0x%02x\n", 
EngineList->Type.Port.PortData.MiscControls);
                                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~  
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbPcieConfig/PcieConfigLib.c:560:5:
 note: in expansion of macro 'IDS_HDT_CONSOLE'
     IDS_HDT_CONSOLE (PCIE_MISC, "          Misc Control  - 0x%02x\n", 
EngineList->Type.Port.PortData.MiscControls);
     ^~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbPcieConfig/PcieConfigLib.c: 
In function 'PcieUserDescriptorConfigDump':
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbPcieConfig/PcieConfigLib.c:739:33:
 error: format '%x' expects argument of type 'unsigned int', but argument 12 
has type 'PCIe_PORT_MISC_CONTROL' {aka 'struct <anonymous>'} [-Werror=format=]
     IDS_HDT_CONSOLE (PCIE_MISC, "    PortPresent - %d\n    ChannelType  - %d\n 
   DeviceNumber - %d\n    FunctionNumber - %d\n    LinkSpeedCapability - %d\n   
 LinkAspm - %d\n    LinkHotplug - %d\n    ResetId - %d\n    SB link - %d\n    
MiscControls - 0x%02x\n" ,
                                 
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbPcieConfig/PcieConfigLib.c:749:7:
       ((PCIe_PORT_DESCRIPTOR *) EngineDescriptor)->Port.MiscControls
       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbPcieConfig/PcieConfigLib.c:739:5:
 note: in expansion of macro 'IDS_HDT_CONSOLE'
     IDS_HDT_CONSOLE (PCIE_MISC, "    PortPresent - %d\n    ChannelType  - %d\n 
   DeviceNumber - %d\n    FunctionNumber - %d\n    LinkSpeedCapability - %d\n   
 LinkAspm - %d\n    LinkHotplug - %d\n    ResetId - %d\n    SB link - %d\n    
MiscControls - 0x%02x\n" ,
     ^~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbPcieConfig/PcieConfigLib.c: 
In function 'PcieUserConfigConfigDump':
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbPcieConfig/PcieConfigLib.c:787:33:
 error: format '%d' expects argument of type 'int', but argument 4 has type 
'UINTN' {aka 'long unsigned int'} [-Werror=format=]
     IDS_HDT_CONSOLE (PCIE_MISC, "  ComplexDescriptor SocketId - %d\n  
NumberOfEngines - %d\n",
                                 
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbPcieConfig/PcieConfigLib.c:789:9:
         NumberOfEngines
         ~~~~~~~~~~~~~~~          
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbPcieConfig/PcieConfigLib.c:787:5:
 note: in expansion of macro 'IDS_HDT_CONSOLE'
     IDS_HDT_CONSOLE (PCIE_MISC, "  ComplexDescriptor SocketId - %d\n  
NumberOfEngines - %d\n",
     ^~~~~~~~~~~~~~~

# ===> ids_14.txt

    CC         
libagesa/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbScsLibV1/GnbScsLibV1.o
In file included from src/include/assert.h:20,
                 from src/vendorcode/amd/include/Porting.h:276,
                 from src/vendorcode/amd/agesa/f16kb/AGESA.h:47,
                 from 
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbScsLibV1/GnbScsLibV1.c:50:
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbScsLibV1/GnbScsLibV1.c: In 
function 'GnbSmuInitLibV7139_fun0':
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbScsLibV1/GnbScsLibV1.c:107:35:
 error: too many arguments for format [-Werror=format-extra-args]
       IDS_HDT_CONSOLE (GNB_TRACE, "Verify SCS Binary fail\n", ScsDataPtr);
                                   ^~~~~~~~~~~~~~~~~~~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbScsLibV1/GnbScsLibV1.c:107:7:
 note: in expansion of macro 'IDS_HDT_CONSOLE'
       IDS_HDT_CONSOLE (GNB_TRACE, "Verify SCS Binary fail\n", ScsDataPtr);
       ^~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbScsLibV1/GnbScsLibV1.c:112:33:
 error: format '%x' expects argument of type 'unsigned int', but argument 3 has 
type 'UINT8 *' {aka 'unsigned char *'} [-Werror=format=]
     IDS_HDT_CONSOLE (GNB_TRACE, "Load SCS @%08x\n", ScsDataPtr);
                                 ^~~~~~~~~~~~~~~~~~  ~~~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbScsLibV1/GnbScsLibV1.c:112:5:
 note: in expansion of macro 'IDS_HDT_CONSOLE'
     IDS_HDT_CONSOLE (GNB_TRACE, "Load SCS @%08x\n", ScsDataPtr);
     ^~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbScsLibV1/GnbScsLibV1.c:119:33:
 error: too many arguments for format [-Werror=format-extra-args]
     IDS_HDT_CONSOLE (GNB_TRACE, "Get SCS Result\n", ScsDataPtr);
                                 ^~~~~~~~~~~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbScsLibV1/GnbScsLibV1.c:119:5:
 note: in expansion of macro 'IDS_HDT_CONSOLE'
     IDS_HDT_CONSOLE (GNB_TRACE, "Get SCS Result\n", ScsDataPtr);
     ^~~~~~~~~~~~~~~

# ===> ids_15.txt

    CC         libagesa/vendorcode/amd/agesa/f16kb/Proc/IDS/Debug/IdsDebug.o
In file included from src/include/assert.h:20,
                 from src/vendorcode/amd/include/Porting.h:276,
                 from src/vendorcode/amd/agesa/f16kb/AGESA.h:47,
                 from 
src/vendorcode/amd/agesa/f16kb/Proc/IDS/Debug/IdsDebug.c:45:
src/vendorcode/amd/agesa/f16kb/Proc/IDS/Debug/IdsDebug.c: In function 
'IdsAssert':
src/vendorcode/amd/agesa/f16kb/Proc/IDS/Debug/IdsDebug.c:132:31: error: format 
'%x' expects argument of type 'unsigned int', but argument 3 has type 'long 
unsigned int' [-Werror=format=]
   IDS_HDT_CONSOLE (MAIN_FLOW, "ASSERT on File[%x] Line[%x]\n", (UINTN) file, 
(UINTN) line);
                               ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~  ~~~~~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/IDS/Debug/IdsDebug.c:132:3: note: in 
expansion of macro 'IDS_HDT_CONSOLE'
   IDS_HDT_CONSOLE (MAIN_FLOW, "ASSERT on File[%x] Line[%x]\n", (UINTN) file, 
(UINTN) line);
   ^~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/IDS/Debug/IdsDebug.c:132:31: error: format 
'%x' expects argument of type 'unsigned int', but argument 4 has type 'long 
unsigned int' [-Werror=format=]
   IDS_HDT_CONSOLE (MAIN_FLOW, "ASSERT on File[%x] Line[%x]\n", (UINTN) file, 
(UINTN) line);
                               ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~                
~~~~~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/IDS/Debug/IdsDebug.c:132:3: note: in 
expansion of macro 'IDS_HDT_CONSOLE'
   IDS_HDT_CONSOLE (MAIN_FLOW, "ASSERT on File[%x] Line[%x]\n", (UINTN) file, 
(UINTN) line);
   ^~~~~~~~~~~~~~~

# ===> ids_16.txt

    CC         
libagesa/vendorcode/amd/agesa/f16kb/Proc/Mem/Feat/RDWR2DTRAINING/KB/mfRdWr2DKb.o
In file included from src/include/assert.h:20,
                 from src/vendorcode/amd/include/Porting.h:276,
                 from src/vendorcode/amd/agesa/f16kb/AGESA.h:47,
                 from 
src/vendorcode/amd/agesa/f16kb/Proc/Mem/Feat/RDWR2DTRAINING/KB/mfRdWr2DKb.c:50:
src/vendorcode/amd/agesa/f16kb/Proc/Mem/Feat/RDWR2DTRAINING/KB/mfRdWr2DKb.c: In 
function 'MemFRdWr2DProgramVrefKB':
src/vendorcode/amd/agesa/f16kb/Proc/Mem/Feat/RDWR2DTRAINING/KB/mfRdWr2DKb.c:224:32:
 error: repeated ' ' flag in format [-Werror=format=]
     IDS_HDT_CONSOLE (MEM_FLOW, "\t\t\tDimm Vref = %c%d%    ", (Va.AdjustValue 
< 0) ? '-':'+', (Va.AdjustValue < 0) ? (0 - Va.AdjustValue) : Va.AdjustValue );
                                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/Mem/Feat/RDWR2DTRAINING/KB/mfRdWr2DKb.c:224:5:
 note: in expansion of macro 'IDS_HDT_CONSOLE'
     IDS_HDT_CONSOLE (MEM_FLOW, "\t\t\tDimm Vref = %c%d%    ", (Va.AdjustValue 
< 0) ? '-':'+', (Va.AdjustValue < 0) ? (0 - Va.AdjustValue) : Va.AdjustValue );
     ^~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/Mem/Feat/RDWR2DTRAINING/KB/mfRdWr2DKb.c:224:32:
 error: repeated ' ' flag in format [-Werror=format=]
     IDS_HDT_CONSOLE (MEM_FLOW, "\t\t\tDimm Vref = %c%d%    ", (Va.AdjustValue 
< 0) ? '-':'+', (Va.AdjustValue < 0) ? (0 - Va.AdjustValue) : Va.AdjustValue );
                                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/Mem/Feat/RDWR2DTRAINING/KB/mfRdWr2DKb.c:224:5:
 note: in expansion of macro 'IDS_HDT_CONSOLE'
     IDS_HDT_CONSOLE (MEM_FLOW, "\t\t\tDimm Vref = %c%d%    ", (Va.AdjustValue 
< 0) ? '-':'+', (Va.AdjustValue < 0) ? (0 - Va.AdjustValue) : Va.AdjustValue );
     ^~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/Mem/Feat/RDWR2DTRAINING/KB/mfRdWr2DKb.c:224:32:
 error: repeated ' ' flag in format [-Werror=format=]
     IDS_HDT_CONSOLE (MEM_FLOW, "\t\t\tDimm Vref = %c%d%    ", (Va.AdjustValue 
< 0) ? '-':'+', (Va.AdjustValue < 0) ? (0 - Va.AdjustValue) : Va.AdjustValue );
                                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/Mem/Feat/RDWR2DTRAINING/KB/mfRdWr2DKb.c:224:5:
 note: in expansion of macro 'IDS_HDT_CONSOLE'
     IDS_HDT_CONSOLE (MEM_FLOW, "\t\t\tDimm Vref = %c%d%    ", (Va.AdjustValue 
< 0) ? '-':'+', (Va.AdjustValue < 0) ? (0 - Va.AdjustValue) : Va.AdjustValue );
     ^~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/Mem/Feat/RDWR2DTRAINING/KB/mfRdWr2DKb.c:224:32:
 error: conversion lacks type at end of format [-Werror=format=]
     IDS_HDT_CONSOLE (MEM_FLOW, "\t\t\tDimm Vref = %c%d%    ", (Va.AdjustValue 
< 0) ? '-':'+', (Va.AdjustValue < 0) ? (0 - Va.AdjustValue) : Va.AdjustValue );
                                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/Mem/Feat/RDWR2DTRAINING/KB/mfRdWr2DKb.c:224:5:
 note: in expansion of macro 'IDS_HDT_CONSOLE'
     IDS_HDT_CONSOLE (MEM_FLOW, "\t\t\tDimm Vref = %c%d%    ", (Va.AdjustValue 
< 0) ? '-':'+', (Va.AdjustValue < 0) ? (0 - Va.AdjustValue) : Va.AdjustValue );
     ^~~~~~~~~~~~~~~

# ===> ids_17.txt

In file included from 
src/vendorcode/amd/agesa/f16kb/Proc/Mem/Feat/RDWR2DTRAINING/mfRdWr2DTraining.c:54:
src/vendorcode/amd/agesa/f16kb/Proc/Mem/Feat/RDWR2DTRAINING/mfRdWr2DTraining.c: 
In function 'MemFRdWr2DFindCsVrefMargin':
src/vendorcode/amd/agesa/f16kb/Proc/Mem/Feat/RDWR2DTRAINING/mfRdWr2DTraining.c:1245:34:
 error: format '%x' expects a matching 'unsigned int' argument [-Werror=format=]
       IDS_HDT_CONSOLE (MEM_FLOW, "\t\t\tSmallest Max Negative Vref Offset from 
V-Nom for ChipSel %02x = 00\n");
                                  
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Include/Ids.h:747:45: note: in definition of 
macro 'IDS_HDT_CONSOLE_DEBUG_CODE'
   #define IDS_HDT_CONSOLE_DEBUG_CODE(Code)  Code
                                             ^~~~
src/vendorcode/amd/agesa/f16kb/Include/Ids.h:744:40: note: in expansion of 
macro 'printk'
     #define IDS_HDT_CONSOLE(f, s, ...) printk (BIOS_DEBUG, s, ##__VA_ARGS__);
                                        ^~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/Mem/Feat/RDWR2DTRAINING/mfRdWr2DTraining.c:1245:7:
 note: in expansion of macro 'IDS_HDT_CONSOLE'
       IDS_HDT_CONSOLE (MEM_FLOW, "\t\t\tSmallest Max Negative Vref Offset from 
V-Nom for ChipSel %02x = 00\n");
       ^~~~~~~~~~~~~~~

# ===> ids_18.txt

    CC         libagesa/vendorcode/amd/agesa/f16kb/Proc/Mem/Main/mmflow.o
In file included from src/include/assert.h:20,
                 from src/vendorcode/amd/include/Porting.h:276,
                 from src/vendorcode/amd/agesa/f16kb/AGESA.h:47,
                 from src/vendorcode/amd/agesa/f16kb/Proc/Mem/Main/mmflow.c:53:
src/vendorcode/amd/agesa/f16kb/Proc/Mem/Main/mmflow.c: In function 
'MemSPDDataProcess':
src/vendorcode/amd/agesa/f16kb/Proc/Mem/Main/mmflow.c:392:40: error: format 
'%x' expects argument of type 'unsigned int', but argument 6 has type 'UINT8 *' 
{aka 'unsigned char *'} [-Werror=format=]
             IDS_HDT_CONSOLE (MEM_FLOW, "SPD Socket %d Channel %d Dimm %d: 
%08x\n", Socket, Channel, Dimm, SpdParam.Buffer);
                                        
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~                         
~~~~~~~~~~~~~~~
src/include/console/console.h:72:54: note: in definition of macro 'printk'
 #define printk(LEVEL, fmt, args...) do_printk(LEVEL, fmt, ##args)
                                                      ^~~
src/vendorcode/amd/agesa/f16kb/Proc/Mem/Main/mmflow.c:392:13: note: in 
expansion of macro 'IDS_HDT_CONSOLE'
             IDS_HDT_CONSOLE (MEM_FLOW, "SPD Socket %d Channel %d Dimm %d: 
%08x\n", Socket, Channel, Dimm, SpdParam.Buffer);
             ^~~~~~~~~~~~~~~

# ===> ids_19.txt

    CC         
libagesa/vendorcode/amd/agesa/f16kb/Proc/Mem/Tech/mttRdDqs2DTraining.o
In file included from 
src/vendorcode/amd/agesa/f16kb/Proc/Mem/Tech/mttRdDqs2DTraining.c:60:
src/vendorcode/amd/agesa/f16kb/Proc/Mem/Tech/mttRdDqs2DTraining.c: In function 
'MemT2DRdDQSFindCsVrefMargin':
src/vendorcode/amd/agesa/f16kb/Proc/Mem/Tech/mttRdDqs2DTraining.c:1075:34: 
error: format '%x' expects a matching 'unsigned int' argument [-Werror=format=]
       IDS_HDT_CONSOLE (MEM_FLOW, "Smallest Max Negative Vref Offset from V-Nom 
for ChipSel %02x = 00\n");
                                  
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/vendorcode/amd/agesa/f16kb/Include/Ids.h:747:45: note: in definition of 
macro 'IDS_HDT_CONSOLE_DEBUG_CODE'
   #define IDS_HDT_CONSOLE_DEBUG_CODE(Code)  Code
                                             ^~~~
src/vendorcode/amd/agesa/f16kb/Include/Ids.h:744:40: note: in expansion of 
macro 'printk'
     #define IDS_HDT_CONSOLE(f, s, ...) printk (BIOS_DEBUG, s, ##__VA_ARGS__);
                                        ^~~~~~
src/vendorcode/amd/agesa/f16kb/Proc/Mem/Tech/mttRdDqs2DTraining.c:1075:7: note: 
in expansion of macro 'IDS_HDT_CONSOLE'
       IDS_HDT_CONSOLE (MEM_FLOW, "Smallest Max Negative Vref Offset from V-Nom 
for ChipSel %02x = 00\n");
       ^~~~~~~~~~~~~~~

# ===
_______________________________________________
coreboot mailing list -- coreboot@coreboot.org
To unsubscribe send an email to coreboot-le...@coreboot.org

Reply via email to