On Sun, Nov 03, 2024 at 11:11:13AM -0500, Aren wrote: > On Sun, Nov 03, 2024 at 11:31:03AM +0000, Jonathan Cameron wrote: > > On Sat, 2 Nov 2024 15:50:39 -0400 > > Aren Moynihan <aren@xxxxxxxxxxxxxxxxx> wrote: ... > > For this Andy was asking for consistency. Generally we don't insist on a > > particular ordering in IIO drivers, but we do prefer them to be the same. > > Your new ordering is inconsistent between resume and suspend. Whilst > > existing code may be inconsistent, you can still pick most common ordering > > and use that for your new code. > > > > If the existing driver is inconsistent then feel free to tidy that up but > > do it in a precursor patch so there is a consistent style for you to then > > carry on. > > Oh right, the order of declarations in stk3310_suspend also needs to be > flipped. Is that simple enough that you can fix it when applying this? > > Apparently I was being dense, I checked the rest of the driver to see > what it did (it's consistent about putting shorter lines & ones without > an assignment first), and fixed the case Andy pointed out to match that, > but failed to check the rest of the patch. Thanks! You may ignore my comment about RCT order if it's not that one that being commonly used in the driver. -- With Best Regards, Andy Shevchenko
