Thread (17 messages) flat view 17 messages, 3 authors, 2014-08-20

[PATCH 0/8] mtd: nand: Support for new DT NAND driver

From: Lee Jones <hidden>
Date: 2014-08-20 06:48:20
Also in: lkml

On Tue, 19 Aug 2014, Brian Norris wrote:
On Mon, Aug 18, 2014 at 09:39:53AM +0100, Lee Jones wrote:
quoted
On Thu, 14 Aug 2014, Brian Norris wrote:
quoted
drivers/mtd/nand/stm_nand_bch.c: In function 'bch_calc_timing_registers':
drivers/mtd/nand/stm_nand_bch.c:1022:6: warning: variable 'ren_half_off' set but not used [-Wunused-but-set-variable]
drivers/mtd/nand/stm_nand_bch.c:1021:6: warning: variable 'ren_half_on' set but not used [-Wunused-but-set-variable]
This I have never seen.  I guess -Wunused-but-set-variable is normally
off.  This issue has been present since the beginning of this endeavour
and exists in the original (internal) code.
BTW, you can get extra warnings by building with the W=[123] option to
the Makefile (e.g., 'make W=1').

$ make help
[...]
  make W=n   [targets] Enable extra gcc checks, n=1,2,3 where
		1: warnings which may be relevant and do not occur too often
		2: warnings which occur quite often but may still be relevant
		3: more obscure warnings, can most likely be ignored
		Multiple levels can be combined with W=12 or W=123
Already taken care of.

I will reply in more detail to your other (build warnings) email.

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org ? Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help