[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2] hw/display/edid: Add missing 'qdev-properties.h' header
From: |
Richard Henderson |
Subject: |
Re: [PATCH v2] hw/display/edid: Add missing 'qdev-properties.h' header |
Date: |
Tue, 19 May 2020 08:28:24 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.7.0 |
On 5/18/20 10:45 AM, Philippe Mathieu-Daudé wrote:
> When trying to consume the DEFINE_EDID_PROPERTIES() macro
> by including "hw/display/edid.h", we get this build failure:
>
> include/hw/display/edid.h:24:5: error: implicit declaration of
> function ‘DEFINE_PROP_UINT32’ [-Werror=implicit-function-declaration]
> 24 | DEFINE_PROP_UINT32("xres", _state, _edid_info.prefx, 0), \
> | ^~~~~~~~~~~~~~~~~~
>
> Headers should be self-contained, and one shouldn't have to
> dig to find the missing headers.
> In this case "hw/qdev-properties.h" is missing. Add it.
>
> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
> ---
> Since v1: reworded description (rth)
>
> Supersedes: <address@hidden>
> ---
> include/hw/display/edid.h | 1 +
> 1 file changed, 1 insertion(+)
Reviewed-by: Richard Henderson <address@hidden>
r~