diff mbox

[U-Boot] FPGA: drivers/fpga/ivm_core.c: incorrect printf

Message ID 20170415131540.25655-1-xypron.glpk@gmx.de
State Accepted
Commit ea1e3f96c3666245b3651ab8022738763b97e6a8
Delegated to: Tom Rini
Headers show

Commit Message

Heinrich Schuchardt April 15, 2017, 1:15 p.m. UTC 
The number of arguments for printf does not match the
format string.

The problem was indicated by cppcheck.

Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
---
 drivers/fpga/ivm_core.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Tom Rini April 15, 2017, 4:12 p.m. UTC | #1 
On Sat, Apr 15, 2017 at 03:15:40PM +0200, Heinrich Schuchardt wrote:

> The number of arguments for printf does not match the
> format string.
> 
> The problem was indicated by cppcheck.
> 
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>

Reviewed-by: Tom Rini <trini@konsulko.com>
Tom Rini April 19, 2017, 1:04 p.m. UTC | #2 
On Sat, Apr 15, 2017 at 03:15:40PM +0200, xypron.glpk@gmx.de wrote:

> The number of arguments for printf does not match the
> format string.
> 
> The problem was indicated by cppcheck.
> 
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> Reviewed-by: Tom Rini <trini@konsulko.com>

Applied to u-boot/master, thanks!
diff mbox

Patch

diff --git a/drivers/fpga/ivm_core.c b/drivers/fpga/ivm_core.c
index 03aea625d5..78e9cc46ac 100644
--- a/drivers/fpga/ivm_core.c
+++ b/drivers/fpga/ivm_core.c
@@ -3142,7 +3142,7 @@  signed char ispVMProcessLVDS(unsigned short a_usLVDSCount)
 	}
 
 #ifdef DEBUG
-	printf(");\n", a_usLVDSCount);
+	printf(");\n");
 #endif /* DEBUG */
 
 	return 0;