]> www.infradead.org Git - users/dwmw2/linux.git/commit
can: softing: softing_startstop(): fix set but not used variable warning
authorMarc Kleine-Budde <mkl@pengutronix.de>
Sat, 8 Jan 2022 20:57:51 +0000 (21:57 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 27 Jan 2022 08:04:21 +0000 (09:04 +0100)
commit447b650f9c22ee7c94a0045db23bf62eb81ef2d9
tree5e35e064e73255d2d1f25bcef3253212ed24f935
parentf155b0fbde518857ee2d42f1fbfda6bdf0123363
can: softing: softing_startstop(): fix set but not used variable warning

[ Upstream commit 370d988cc529598ebaec6487d4f84c2115dc696b ]

In the function softing_startstop() the variable error_reporting is
assigned but not used. The code that uses this variable is commented
out. Its stated that the functionality is not finally verified.

To fix the warning:

| drivers/net/can/softing/softing_fw.c:424:9: error: variable 'error_reporting' set but not used [-Werror,-Wunused-but-set-variable]

remove the comment, activate the code, but add a "0 &&" to the if
expression and rely on the optimizer rather than the preprocessor to
remove the code.

Link: https://lore.kernel.org/all/20220109103126.1872833-1-mkl@pengutronix.de
Fixes: 03fd3cf5a179 ("can: add driver for Softing card")
Cc: Kurt Van Dijck <dev.kurt@vandijck-laurijssen.be>
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/net/can/softing/softing_fw.c