Re: [PATCH] USB: serial: iuu_phoenix: fix quoted string split across lines
From: Utkarsh Verma <hidden>
Date: 2021-08-06 13:12:26
On Thu, Aug 05, 2021 at 04:20:56PM +0200, Lukas Bulwahn wrote:
On Thu, Aug 5, 2021 at 3:53 PM Dwaipayan Ray [off-list ref] wrote:quoted
On Tue, Aug 3, 2021 at 7:51 PM Utkarsh Verma [off-list ref] wrote:quoted
Fix checkpatch warning "quoted string split across lines".You could write a line about why this change is required. And that there are no functional changes.... and does this change actually increase the line length beyond 80, or even 100 characters? So, did you run checkpatch on your patch?
Yes, I did run the checkpatch on my patch, and all the warnings regarding "quoted string split across lines" were removed. Although the line length increased to more than 100 characters, there were no warnings from the checkpatch.
If you are just swapping one violation of one rule by the violation of another rule, you will need to make a judgement which rule is more important and provide a good reasoning for that.
Since there were no "LONG_LINE" warnings, so I didn't mention about it in the commit description. But will rectify this in v2.
Other than that, it looks good to me. Lukas
_______________________________________________ Linux-kernel-mentees mailing list Linux-kernel-mentees@lists.linuxfoundation.org https://lists.linuxfoundation.org/mailman/listinfo/linux-kernel-mentees