* [PATCH] misc compiler warning fixes
@ 2012-10-17 20:20 Sascha Hauer
2012-10-17 20:20 ` [PATCH 1/3] mips: remove unused variable Sascha Hauer
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Sascha Hauer @ 2012-10-17 20:20 UTC (permalink / raw)
To: barebox
Some compiler warning fixes
----------------------------------------------------------------
Sascha Hauer (3):
mips: remove unused variable
sandbox: only test for sdl libs when sdl is enabled
net gianfar: remove unused variable
arch/mips/include/asm/mipsregs.h | 3 ---
arch/sandbox/Makefile | 7 +++++--
drivers/net/gianfar.c | 1 -
3 files changed, 5 insertions(+), 6 deletions(-)
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH 1/3] mips: remove unused variable
2012-10-17 20:20 [PATCH] misc compiler warning fixes Sascha Hauer
@ 2012-10-17 20:20 ` Sascha Hauer
2012-10-17 20:20 ` [PATCH 2/3] sandbox: only test for sdl libs when sdl is enabled Sascha Hauer
2012-10-17 20:20 ` [PATCH 3/3] net gianfar: remove unused variable Sascha Hauer
2 siblings, 0 replies; 4+ messages in thread
From: Sascha Hauer @ 2012-10-17 20:20 UTC (permalink / raw)
To: barebox
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
arch/mips/include/asm/mipsregs.h | 3 ---
1 file changed, 3 deletions(-)
diff --git a/arch/mips/include/asm/mipsregs.h b/arch/mips/include/asm/mipsregs.h
index 87c48ed..608f19b 100644
--- a/arch/mips/include/asm/mipsregs.h
+++ b/arch/mips/include/asm/mipsregs.h
@@ -566,7 +566,6 @@ do { \
#define __read_64bit_c0_split(source, sel) \
({ \
unsigned long long __val; \
- unsigned long __flags; \
\
if (sel == 0) \
__asm__ __volatile__( \
@@ -592,8 +591,6 @@ do { \
#define __write_64bit_c0_split(source, sel, val) \
do { \
- unsigned long __flags; \
- \
if (sel == 0) \
__asm__ __volatile__( \
".set\tmips64\n\t" \
--
1.7.10.4
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH 2/3] sandbox: only test for sdl libs when sdl is enabled
2012-10-17 20:20 [PATCH] misc compiler warning fixes Sascha Hauer
2012-10-17 20:20 ` [PATCH 1/3] mips: remove unused variable Sascha Hauer
@ 2012-10-17 20:20 ` Sascha Hauer
2012-10-17 20:20 ` [PATCH 3/3] net gianfar: remove unused variable Sascha Hauer
2 siblings, 0 replies; 4+ messages in thread
From: Sascha Hauer @ 2012-10-17 20:20 UTC (permalink / raw)
To: barebox
Otherwise we get the following warning when sdl is not installed
on the compile host:
Package sdl was not found in the pkg-config search path.
Perhaps you should add the directory containing `sdl.pc'
to the PKG_CONFIG_PATH environment variable
No package 'sdl' found
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
arch/sandbox/Makefile | 7 +++++--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/arch/sandbox/Makefile b/arch/sandbox/Makefile
index c0aa8c6..596c8f8 100644
--- a/arch/sandbox/Makefile
+++ b/arch/sandbox/Makefile
@@ -39,10 +39,13 @@ archprepare: maketools
PHONY += maketools
-SDL_LIBS-$(CONFIG_DRIVER_VIDEO_SDL) := $(shell pkg-config sdl --libs)
+ifeq ($(CONFIG_DRIVER_VIDEO_SDL),y)
+SDL_LIBS := $(shell pkg-config sdl --libs)
+endif
+
cmd_barebox__ = $(CC) -o $@ -Wl,-T,$(barebox-lds) \
-Wl,--start-group $(barebox-common) -Wl,--end-group \
- -lrt -lpthread $(SDL_LIBS-y)
+ -lrt -lpthread $(SDL_LIBS)
common-y += $(BOARD) arch/sandbox/os/
--
1.7.10.4
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH 3/3] net gianfar: remove unused variable
2012-10-17 20:20 [PATCH] misc compiler warning fixes Sascha Hauer
2012-10-17 20:20 ` [PATCH 1/3] mips: remove unused variable Sascha Hauer
2012-10-17 20:20 ` [PATCH 2/3] sandbox: only test for sdl libs when sdl is enabled Sascha Hauer
@ 2012-10-17 20:20 ` Sascha Hauer
2 siblings, 0 replies; 4+ messages in thread
From: Sascha Hauer @ 2012-10-17 20:20 UTC (permalink / raw)
To: barebox
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
drivers/net/gianfar.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/net/gianfar.c b/drivers/net/gianfar.c
index bf834b6..55675ba 100644
--- a/drivers/net/gianfar.c
+++ b/drivers/net/gianfar.c
@@ -82,7 +82,6 @@ static void gfar_adjust_link(struct eth_device *edev)
struct gfar_private *priv = edev->priv;
void __iomem *regs = priv->regs;
u32 ecntrl, maccfg2;
- uint32_t status;
priv->link = edev->phydev->link;
priv->duplexity =edev->phydev->duplex;
--
1.7.10.4
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2012-10-17 20:20 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-10-17 20:20 [PATCH] misc compiler warning fixes Sascha Hauer
2012-10-17 20:20 ` [PATCH 1/3] mips: remove unused variable Sascha Hauer
2012-10-17 20:20 ` [PATCH 2/3] sandbox: only test for sdl libs when sdl is enabled Sascha Hauer
2012-10-17 20:20 ` [PATCH 3/3] net gianfar: remove unused variable Sascha Hauer
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox