03.09.2020 14:06, Andy Shevchenko пишет: > On Thu, Sep 3, 2020 at 3:53 AM Dmitry Osipenko <digetx@xxxxxxxxx> 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!
