Re: [PATCH v3 1/6] staging: rtl8192e: fix coding style issues (merge broken strings)

From: Dan Carpenter
Date: Mon Mar 16 2015 - 04:44:22 EST


On Fri, Mar 13, 2015 at 12:53:16AM +0100, Mateusz Kulikowski wrote:
> - 'WARNING: break quoted strings at a space character'

> @@ -1361,9 +1365,10 @@ static u8 HwRateToMRate90(bool bIsHT, u8 rate)
> break;
>
> default:
> - RT_TRACE(COMP_RECV, "HwRateToMRate90(): Non supported"
> - "Rate [%x], bIsHT = %d!!!\n", rate, bIsHT);
> - break;
> + RT_TRACE(COMP_RECV,
> + "HwRateToMRate90(): Non supportedRate [%x], bIsHT = %d!!!\n",

There should be a space character between "supported" and "Rate". You
can fix it in a later patch though, it's minor. And really no one cares
about RT_TRACE() output.

regards,
dan carpenter

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/