# Re: \[PATCH v2\] video: of\_display\_timing.h: Declare 'display\_timing'

- From: Tomi Valkeinen \<hidden\>
- Date: 2013-06-26 13:36:27 
- List: linux-fbdev
- Page: https://ratatoskr.run/linux-fbdev/2013/06/13465040
- Thread: https://ratatoskr.run/linux-fbdev/2013/06/13465039/t.md

```text
On 26/06/13 16:34, Fabio Estevam wrote:
> Commit ffa3fd21de ("videomode: implement public of_get_display_timing()") causes
> the following build warning:
> 
> include/video/of_display_timing.h:18:10: warning: 'struct display_timing' declared inside parameter list [enabled by default]
> include/video/of_display_timing.h:18:10: warning: its scope is only this definition or declaration, which is probably not what you want [enabled by default]
> 
> Declare 'display_timing' to avoid the build warning.
> 
> Signed-off-by: Fabio Estevam <redacted>
> ---
> Changes since v1:
> - Add 'struct display_timing;' instead of '#include <video/display_timing.h>'
>  include/video/of_display_timing.h | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/include/video/of_display_timing.h b/include/video/of_display_timing.h
> index 6562ad9..79e6697 100644
> --- a/include/video/of_display_timing.h
> +++ b/include/video/of_display_timing.h
> @@ -10,6 +10,7 @@
>  #define __LINUX_OF_DISPLAY_TIMING_H
>  
>  struct device_node;
> +struct display_timing;
>  struct display_timings;
>  
>  #define OF_USE_NATIVE_MODE -1
> 

Thanks. I've added this to fbdev-3.11 branch.

 Tomi


```

## Attachments

- [signature.asc](https://ratatoskr.run/linux-fbdev/2013/06/13465040/attachment/0) (application/pgp-signature, 901 bytes)