linux-kernel - Re: [PATCH] clk: ti: dra7: remove two unused symbols

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20200514200219.GQ37466@atomide.com>
Date:   Thu, 14 May 2020 13:02:19 -0700
From:   Tony Lindgren <tony@...mide.com>
To:     Stephen Boyd <sboyd@...nel.org>
Cc:     Tero Kristo <t-kristo@...com>, Jason Yan <yanaijie@...wei.com>,
        mturquette@...libre.com, linux-omap@...r.kernel.org,
        linux-clk@...r.kernel.org, linux-kernel@...r.kernel.org,
        Hulk Robot <hulkci@...wei.com>
Subject: Re: [PATCH] clk: ti: dra7: remove two unused symbols

* Stephen Boyd <sboyd@...nel.org> [200514 19:59]:
> Quoting Tony Lindgren (2020-04-17 08:33:42)
> > Tero,
> > 
> > * Jason Yan <yanaijie@...wei.com> [200417 07:09]:
> > > Fix the following gcc warning:
> > > 
> > > drivers/clk/ti/clk-7xx.c:320:43: warning: \u2018dra7_gpu_sys_clk_data\u2019
> > > defined but not used [-Wunused-const-variable=]
> > >  static const struct omap_clkctrl_div_data dra7_gpu_sys_clk_data
> > > __initconst = {
> > >                                            ^~~~~~~~~~~~~~~~~~~~~
> > > drivers/clk/ti/clk-7xx.c:315:27: warning: \u2018dra7_gpu_sys_clk_parents\u2019
> > > defined but not used [-Wunused-const-variable=]
> > >  static const char * const dra7_gpu_sys_clk_parents[] __initconst = {
> > >                            ^~~~~~~~~~~~~~~~~~~~~~~~
> > 
> > Looks like this started happening with commit 957ad44ff5f2
> > ("clk: ti: add clkctrl data dra7 sgx") as I did it based on
> > omap5. But the clocks seem slightly different, care to check
> > sgx clocking against the TRM in case I also messed up something
> > else?
> 
> Is that an ack?

Acked-by: Tony Lindgren <tony@...mide.com>