[PATCH] smbios: address build warning

Heinrich Schuchardt xypron.glpk at gmx.de
Tue Dec 3 14:28:14 CET 2024


On 03.12.24 14:16, Peng Fan wrote:
>> Subject: Re: [PATCH] smbios: address build warning
>>
>> On 03.12.24 14:42, Peng Fan (OSS) wrote:
>>> From: Peng Fan <peng.fan at nxp.com>
>>>
>>> include display_options.h to address build warning:
>>
>> build warning occurring with CONFIG_LOG=y
>>
>>> lib/smbios.c: In function ‘smbios_update_version’:
>>> lib/smbios.c:305:9: warning: implicit declaration of function
>> ‘print_buffer’
>>> [-Wimplicit-function-declaration]
>>>            print_buffer((ulong)ptr, ptr, 1, old_len + 1, 0);
>>>            ^~~~~~~~~~~~
>>>
>>> Signed-off-by: Peng Fan <peng.fan at nxp.com>
>>
>> Reviewed-by: Heinrich Schuchardt <xypron.glpk at gmx.de>
>>
>
> Thanks, please let me know whether I need to post V2
> to update commit log to include
> "build warning occurring with CONFIG_LOG=y", or
> Maintainers could help update when apply the patch.
>
> Thanks,
> Peng.

To me the patch is ok as is.
I was just wondering, why we did not see the issue in CI.

Best regards

Heinrich

>
>>> ---
>>>    lib/smbios.c | 1 +
>>>    1 file changed, 1 insertion(+)
>>>
>>> diff --git a/lib/smbios.c b/lib/smbios.c index
>>> a36d4b4e54a..defb6b42f45 100644
>>> --- a/lib/smbios.c
>>> +++ b/lib/smbios.c
>>> @@ -7,6 +7,7 @@
>>>
>>>    #define LOG_CATEGORY	LOGC_BOARD
>>>
>>> +#include <display_options.h>
>>>    #include <dm.h>
>>>    #include <env.h>
>>>    #include <linux/stringify.h>
>



More information about the U-Boot mailing list