RE: [PATCH net-next v4 1/2] dsa: lan9303: Whitespace Only

From: Jerry.Ray
Date: Fri Dec 09 2022 - 14:25:33 EST


> Please name the commit message something more specific than "Whitespace Only",
> that is likely to not be confused with some other patch. A "Whitespace Only"
> patch can take place anywhere in this file. Like "align dsa_switch_ops members".
>
> Do you use a text editor which highlights tabs? The members above this
> line are aligned with tabs, the ones below with spaces. Still not
> exactly what I'd call "consistent".
>

Hi Vladimir,

Thank you for your comments. I will rename the patch to be more explicit and
will address the tabs-spaces issue you pointed out.

I'll look for a better Linux-based text editor this weekend. Do you have a
recommendation?

Regards,
Jerry.