]> git.dujemihanovic.xyz Git - u-boot.git/commit
linux/compat.h: Remove debug() from spin_lock_irqsave()
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Thu, 19 Nov 2020 19:26:20 +0000 (21:26 +0200)
committerTom Rini <trini@konsulko.com>
Wed, 2 Dec 2020 21:21:58 +0000 (16:21 -0500)
commit3cbb026f17332b70d352ae3659179ca2b51f418c
tree5e4ce20626f7bf2495402e5882684ee782e463df
parentc2ba01c082b4ab21e43bb15e1c3a7e19573d1133
linux/compat.h: Remove debug() from spin_lock_irqsave()

It seems nobody tested the debug() option in spin_lock_irqsave().
Currently, when #define DEBUG, it spoils the compiler with

In file included from drivers/usb/dwc3/gadget.c:18:
drivers/usb/dwc3/gadget.c: In function ‘dwc3_gadget_set_selfpowered’:
include/log.h:235:4: warning: ‘flags’ is used uninitialized in this function [-Wuninitialized]
  235 |    printf(pr_fmt(fmt), ##args); \
      |    ^~~~~~
drivers/usb/dwc3/gadget.c:1347:17: note: ‘flags’ was declared here
 1347 |  unsigned long  flags;
      |                 ^~~~~

and so on...
Drop useless debug() call to make compiler happy.

Fixes: 0c06db598367 ("lib, linux: move linux specific defines to linux/compat.h")
Cc: Heiko Schocher <hs@denx.de>
Cc: Tom Rini <trini@ti.com>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Reviewed-by: Oleksandr Andrushchenko <oleksandr_andrushchenko@epam.com>
Reviewed-by: Heiko Schocher <hs@denx.de>
include/linux/compat.h