From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: linux-fbdev@vger.kernel.org
Subject: Re: [PATCH 22/23] OMAPDSS: change internal dispc functions to static
Date: Sun, 13 Dec 2015 20:22:33 +0000 [thread overview]
Message-ID: <2219457.fCRnFmI3e4@avalon> (raw)
In-Reply-To: <1449676791-26304-23-git-send-email-tomi.valkeinen@ti.com>
Hi Tomi,
Thank you for the patch.
On Wednesday 09 December 2015 17:59:50 Tomi Valkeinen wrote:
> A bunch of dispc functions are only used inside dispc, so we can make
> them static.
>
> Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
> ---
> drivers/video/fbdev/omap2/dss/dispc.c | 15 ++++++++++-----
> drivers/video/fbdev/omap2/dss/dss.h | 5 -----
> 2 files changed, 10 insertions(+), 10 deletions(-)
>
> diff --git a/drivers/video/fbdev/omap2/dss/dispc.c
> b/drivers/video/fbdev/omap2/dss/dispc.c index 822add50f92e..346822b55947
> 100644
> --- a/drivers/video/fbdev/omap2/dss/dispc.c
> +++ b/drivers/video/fbdev/omap2/dss/dispc.c
> @@ -251,6 +251,11 @@ struct color_conv_coef {
> int full_range;
> };
>
> +static unsigned long dispc_fclk_rate(void);
> +static unsigned long dispc_core_clk_rate(void);
> +static unsigned long dispc_mgr_lclk_rate(enum omap_channel channel);
> +static unsigned long dispc_mgr_pclk_rate(enum omap_channel channel);
> +
Looks like dispc.c should be refactored to avoid forward declarations. That's
unrelated to this patch though, so
Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> static unsigned long dispc_plane_pclk_rate(enum omap_plane plane);
> static unsigned long dispc_plane_lclk_rate(enum omap_plane plane);
>
> @@ -2952,7 +2957,7 @@ static void dispc_mgr_set_lcd_type_tft(enum
> omap_channel channel) mgr_fld_write(channel, DISPC_MGR_FLD_STNTFT, 1);
> }
>
> -void dispc_set_loadmode(enum omap_dss_load_mode mode)
> +static void dispc_set_loadmode(enum omap_dss_load_mode mode)
> {
> REG_FLD_MOD(DISPC_CONFIG, mode, 2, 1);
> }
> @@ -3311,7 +3316,7 @@ static void dispc_mgr_get_lcd_divisor(enum
> omap_channel channel, int *lck_div, *pck_div = FLD_GET(l, 7, 0);
> }
>
> -unsigned long dispc_fclk_rate(void)
> +static unsigned long dispc_fclk_rate(void)
> {
> struct dss_pll *pll;
> unsigned long r = 0;
> @@ -3342,7 +3347,7 @@ unsigned long dispc_fclk_rate(void)
> return r;
> }
>
> -unsigned long dispc_mgr_lclk_rate(enum omap_channel channel)
> +static unsigned long dispc_mgr_lclk_rate(enum omap_channel channel)
> {
> struct dss_pll *pll;
> int lcd;
> @@ -3383,7 +3388,7 @@ unsigned long dispc_mgr_lclk_rate(enum omap_channel
> channel) }
> }
>
> -unsigned long dispc_mgr_pclk_rate(enum omap_channel channel)
> +static unsigned long dispc_mgr_pclk_rate(enum omap_channel channel)
> {
> unsigned long r;
>
> @@ -3408,7 +3413,7 @@ void dispc_set_tv_pclk(unsigned long pclk)
> dispc.tv_pclk_rate = pclk;
> }
>
> -unsigned long dispc_core_clk_rate(void)
> +static unsigned long dispc_core_clk_rate(void)
> {
> return dispc.core_clk_rate;
> }
> diff --git a/drivers/video/fbdev/omap2/dss/dss.h
> b/drivers/video/fbdev/omap2/dss/dss.h index 2406bcdb831a..fec68d8939bc
> 100644
> --- a/drivers/video/fbdev/omap2/dss/dss.h
> +++ b/drivers/video/fbdev/omap2/dss/dss.h
> @@ -378,7 +378,6 @@ void dispc_lcd_enable_signal(bool enable);
> void dispc_pck_free_enable(bool enable);
> void dispc_enable_fifomerge(bool enable);
> void dispc_enable_gamma_table(bool enable);
> -void dispc_set_loadmode(enum omap_dss_load_mode mode);
>
> typedef bool (*dispc_div_calc_func)(int lckd, int pckd, unsigned long lck,
> unsigned long pck, void *data);
> @@ -388,7 +387,6 @@ bool dispc_div_calc(unsigned long dispc,
>
> bool dispc_mgr_timings_ok(enum omap_channel channel,
> const struct omap_video_timings *timings);
> -unsigned long dispc_fclk_rate(void);
> int dispc_calc_clock_rates(unsigned long dispc_fclk_rate,
> struct dispc_clock_info *cinfo);
>
> @@ -398,9 +396,6 @@ void dispc_ovl_compute_fifo_thresholds(enum omap_plane
> plane, u32 *fifo_low, u32 *fifo_high, bool use_fifomerge,
> bool manual_update);
>
> -unsigned long dispc_mgr_lclk_rate(enum omap_channel channel);
> -unsigned long dispc_mgr_pclk_rate(enum omap_channel channel);
> -unsigned long dispc_core_clk_rate(void);
> void dispc_mgr_set_clock_div(enum omap_channel channel,
> const struct dispc_clock_info *cinfo);
> int dispc_mgr_get_clock_div(enum omap_channel channel,
--
Regards,
Laurent Pinchart
prev parent reply other threads:[~2015-12-13 20:22 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-12-09 15:59 [PATCH 22/23] OMAPDSS: change internal dispc functions to static Tomi Valkeinen
2015-12-13 20:22 ` Laurent Pinchart [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=2219457.fCRnFmI3e4@avalon \
--to=laurent.pinchart@ideasonboard.com \
--cc=linux-fbdev@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).