DAC960_Failure prints a newline at the end of the message string, so the message string does not need to include a newline explicitly. Done using Coccinelle. In the first case, the preceding spaces were also converted to a tab. In the second case, doing so would upset the indentation, so it was left as is. Signed-off-by: Julia Lawall <Julia.Lawall@xxxxxxx> --- drivers/block/DAC960.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/block/DAC960.c b/drivers/block/DAC960.c index 442e777..89cd83a 100644 --- a/drivers/block/DAC960.c +++ b/drivers/block/DAC960.c @@ -1344,7 +1344,7 @@ static bool DAC960_V1_EnableMemoryMailboxInterface(DAC960_Controller_T CommandMailbox.TypeX.CommandOpcode2 = 0x10; break; default: - DAC960_Failure(Controller, "Unknown Controller Type\n"); + DAC960_Failure(Controller, "Unknown Controller Type"); break; } return false; @@ -1527,7 +1527,7 @@ static bool DAC960_V2_EnableMemoryMailboxInterface(DAC960_Controller_T DAC960_LP_AcknowledgeHardwareMailboxStatus(ControllerBaseAddress); break; default: - DAC960_Failure(Controller, "Unknown Controller Type\n"); + DAC960_Failure(Controller, "Unknown Controller Type"); CommandStatus = DAC960_V2_AbormalCompletion; break; } -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html