[U-Boot] [PATCH] ddr:fsl: Fix warnings on gcc-6.x
Tom Rini
trini at konsulko.com
Fri Feb 10 21:46:22 UTC 2017
On Fri, Feb 10, 2017 at 04:45:12PM +0000, york sun wrote:
> On 02/09/2017 03:10 PM, Tom Rini wrote:
> > With gcc-6.x we will see many warnings like:
> > warning: ‘dual_0S’ defined but not used [-Wunused-const-variable=]
> >
> > Depending on exactly what DDR choices are or are not enabled when we use
> > this file. So we use slightly more exact #ifdef tests in order to
> > silence the warnings.
> >
> > Reported-by: Thomas Schaefer <Thomas.Schaefer at kontron.com>
> > Cc: York Sun <york.sun at nxp.com>
> > Signed-off-by: Tom Rini <trini at konsulko.com>
> > ---
> > drivers/ddr/fsl/options.c | 27 ++++++++++++++++++++++++---
> > 1 file changed, 24 insertions(+), 3 deletions(-)
> >
> > diff --git a/drivers/ddr/fsl/options.c b/drivers/ddr/fsl/options.c
> > index d6a8fcb216a4..7f7c16ced6f3 100644
> > --- a/drivers/ddr/fsl/options.c
> > +++ b/drivers/ddr/fsl/options.c
> > @@ -29,10 +29,12 @@ struct dynamic_odt {
> > unsigned int odt_rtt_wr;
> > };
> >
> > -#ifdef CONFIG_SYS_FSL_DDR4
> > +#if defined(CONFIG_SYS_FSL_DDR4) && defined(CONFIG_DIMM_SLOTS_PER_CTLR)
> > /* Quad rank is not verified yet due availability.
> > * Replacing 20 OHM with 34 OHM since DDR4 doesn't have 20 OHM option
> > */
> > +#if (CONFIG_DIMM_SLOTS_PER_CTLR == 1) || ((CONFIG_DIMM_SLOTS_PER_CTLR == 2) && \
> > + defined(CONFIG_FSL_DDR_FIRST_SLOT_QUAD_CAPABLE))
> > static const struct dynamic_odt single_Q[4] = {
> > { /* cs0 */
> > FSL_DDR_ODT_NEVER,
> > @@ -59,7 +61,9 @@ static const struct dynamic_odt single_Q[4] = {
> > DDR4_RTT_120_OHM
> > }
> > };
> > +#endif
> >
> > +#if (CONFIG_DIMM_SLOTS_PER_CTLR == 1)
> > static const struct dynamic_odt single_D[4] = {
> > { /* cs0 */
> > FSL_DDR_ODT_NEVER,
> > @@ -89,6 +93,8 @@ static const struct dynamic_odt single_S[4] = {
> > {0, 0, 0, 0},
> > };
> >
> > +#elif (CONFIG_DIMM_SLOTS_PER_CTLR == 2)
> > +
>
> <snip>
>
> Tom,
>
> What do you think about using __maybe_unused instead? As Thomas Schaefer
> has verified that GCC optimized out unused constants.
Yeah, that ends up being clearer I think esp since we're not concerned
with the data being included by accident. Thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170210/2c92f466/attachment.sig>
More information about the U-Boot
mailing list