Hi Arnd,
Thank you for the patch.
On Monday 29 September 2014 18:00:33 Arnd Bergmann wrote:
> If CONFIG_OF is disabled but we build any driver that tries to
> use of_get_display_timing, we get this link error:
>
> ERROR: "of_get_display_timing" [drivers/gpu/drm/rcar-du/rcar-du-drm.ko]
> undefined!
>
> This adds an empty inline function as an alternative, to avoid changing
> each driver using this function when we want to build them without
> CONFIG_OF.
>
> Signed-off-by: Arnd Bergmann <arnd@xxxxxxxx>
Shouldn't empty functions also be defined for of_get_display_timings and
of_display_timings_exist ?
> diff --git a/include/video/of_display_timing.h
> b/include/video/of_display_timing.h index 79e6697af6cf..02b6eb7c3ded 100644
> --- a/include/video/of_display_timing.h
> +++ b/include/video/of_display_timing.h
> @@ -15,8 +15,17 @@ struct display_timings;
>
> #define OF_USE_NATIVE_MODE -1
>
> +#ifdef CONFIG_OF
> int of_get_display_timing(struct device_node *np, const char *name,
> struct display_timing *dt);
> +#else
> +static inline int of_get_display_timing(struct device_node *np, const char
> *name,
> + struct display_timing *dt)
> +{
> + return -ENOSYS;
> +}
> +
> +#endif
> struct display_timings *of_get_display_timings(struct device_node *np);
> int of_display_timings_exist(struct device_node *np);
--
Regards,
Laurent Pinchart
--
To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
