Re: [PATCH][drm-next] drm/tve200: make two functions static

From	Linus Walleij <linus.walleij@linaro.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH][drm-next] drm/tve200: make two functions static
Date	2017-09-25 17:50 +0200
Message-ID	<utDNU-8jd-15@gated-at.bofh.it> (permalink)
References	<usyGB-8dH-1@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Fri, Sep 22, 2017 at 6:05 PM, Colin King <colin.king@canonical.com> wrote:

> From: Colin Ian King <colin.king@canonical.com>
>
> The functions tve200_display_disable and tve200_display_funcs are
> local to the source and do not need to be in global scope, so make
> them static.
>
> Cleans up sparse warnings:
> symbol 'tve200_display_disable' was not declared. Should it be static?
> symbol 'tve200_display_funcs' was not declared. Should it be static?
>
> Signed-off-by: Colin Ian King <colin.king@canonical.com>

Reviewed-by: Linus Walleij <linus.walleij@linaro.org>

I am travelling right now but will apply the patch on monday in
one week from now.

Yours,
Linus Walleij

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH][drm-next] drm/tve200: make two functions static Colin King <colin.king@canonical.com> - 2017-09-22 18:10 +0200
  Re: [PATCH][drm-next] drm/tve200: make two functions static Linus Walleij <linus.walleij@linaro.org> - 2017-09-25 17:50 +0200

csiph-web