Re: [PATCH v3 03/22] i2c: tegra: Clean up messages in the code

Dmitry Osipenko
2020-09-03T15:03:21Z

03.09.2020 14:06, Andy Shevchenko пишет:
> On Thu, Sep 3, 2020 at 3:53 AM Dmitry Osipenko <digetx@gmail.com> wrote:
>>
>> This patch unifies style of all messages in the driver by starting them
>> with a lowercase letter and using consistent capitalization and wording
>> for all messages.
>
> I didn't look at the rest (yet) but this series has a patch ordering issue.
> Why do we first do some little, non-critical clean ups?
>
> The preferred way is to arrange like:
> - fixes that may be backported
> - fixes that are likely not going to be backported
> - features
> - cleanups
>
> In its turn cleanups go by severity:
> - code affected ones (and maybe logical changers)
> - ...
> - commentary / indentation fixes
>

That's a good suggestion! All patches in this version are ordered by the
time they were created ans since none of these patches should be
worthwhile to backport (IMO) and because majority of patches do minor
changes, it appeared to me that it should be okay as-is.

I agree that it should be worthwhile to reorder the patches now, after
the series grew up a tad in regards to amount of patches. I'll change
the order in the next version, thanks!