u-boot.git/arch/sandbox/include/asm/bitops.h, branch v2015.01Unnamed repository; edit this file 'description' to name the repository.http://cgit.235523.xyz/u-boot.git/atom/arch/sandbox/include/asm/bitops.h?h=v2015.012014-06-23T21:37:24Zsandbox: change local_irq_save() to macro2014-06-23T21:37:24ZMasahiro Yamadayamada.m@jp.panasonic.com2014-06-12T03:26:15Zurn:sha1:1638d98052e0d03e46d504b21ec1b88ecfcd87aa
local_irq_save() should be a macro, not a function
because local_irq_save() saves flag to the given argument.
GCC is silent about this issue, but Clang warns:
In file included from lib/asm-offsets.c:15:
In file included from include/common.h:20:
In file included from include/linux/bitops.h:110:
arch/sandbox/include/asm/bitops.h:59:17:
warning: variable 'flags' is uninitialized when used here
[-Wuninitialized]
local_irq_save(flags);
^~~~~
That change causes another warning:
In file included from include/linux/bitops.h:110:0,
from include/common.h:20,
from lib/asm-offsets.c:15:
arch/sandbox/include/asm/bitops.h: In function ‘test_and_set_bit’:
arch/sandbox/include/asm/bitops.h:56:16: warning: unused variable ‘flags’ [-Wunused-variable]
So, flags should be set to __always_unused.
Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
Cc: Simon Glass <sjg@chromium.org>
Cc: Jeroen Hofstee <jeroen@myspectrum.nl>
Acked-by: Simon Glass <sjg@chromium.org>
Acked-by: Jeroen Hofstee <jeroen@myspectrum.nl>
sandbox: Add architecture header files2011-10-17T20:45:50ZSimon Glasssjg@chromium.org2011-10-10T08:22:14Zurn:sha1:744d9859a71c515118457fecb7a58674b600e81a
This adds required header files for the sandbox architecture, and a basic
description of what sandbox is (README.sandbox).
Signed-off-by: Simon Glass <sjg@chromium.org>