On Tue, Sep 8, 2020 at 3:55 PM Dmitry Osipenko <digetx@xxxxxxxxx> wrote: > 08.09.2020 11:43, Andy Shevchenko пишет: > > On Tue, Sep 8, 2020 at 5:11 AM Dmitry Osipenko <digetx@xxxxxxxxx> wrote: > >> > >> Reorder location of functions in the code in order to have definition > >> of functions closer to the place of the invocation. This change makes > >> easier to navigate around the code and removes the need to have a > >> prototype for tegra_i2c_init(). > > > > Still seems to have ordering issues (it moves pieces you actually > > change in the next patches). > > It's unavoidable that this patch either touches previous or next > changes. You actually had an opposite complaint to v5. Nice :-) > This diver has a long overdue for the refactoring. I think it's a good > time to do it right now because driver works stable and it's quite > unlikely that there will ever be fixes for the older code. Hence it > should be good to have the code reordered now, so that we could have a > nicer code base for the future changes. I agree. > > Have you considered which one looks neat this variant or if you move > > it closer to the end of the series? > This variant should be more logical because changes have this order: > > 1. fixes > 2. features > 3. code reorder > 4. more important cleanups > 5. less important cleanups > > The 3 merely shuffles code around without any changes to the code, hence > it's a safe change. > > The 4 has a potential danger since there are cleanups of the outdated > features and some changes may change semantic in a case of a bug. Hence, > if in the future we'll happen to find a problem in one of the 4 patches, > then there is a better chance that the problem could be fixed with a > simple revert. If 3 is placed after 4, then the chance is completely gone. Thanks for elaboration. If you think so (and I guess you are), go ahead! -- With Best Regards, Andy Shevchenko