On Wed, Apr 06, 2022 at 09:49:08AM +0200, Miquel Raynal wrote: > andriy.shevchenko@xxxxxxxxxxxxxxx wrote on Tue, 5 Apr 2022 17:50:57 > +0300: > > On Tue, Apr 05, 2022 at 10:19:07AM +0200, Miquel Raynal wrote: ... > > > +#define RZN1_DMAMUX_SPLIT 16 > > > > I would name it more explicitly: > > > > #define RZN1_DMAMUX_SPLIT_1_0 16 > > I am sorry but I don't understand this suffix, which probably means > that it is not as clear as we wish. Do you mind if I stick to > RZN1_DMAMUX_SPLIT? The suffix to show that this is the value between part 0 (indexed by 0) and part 1 (indexed by 1) as far as I can see they are different by size. Since they are not equal, the original name without suffix is confusing (I would expect indexing up to 4 in such case). -- With Best Regards, Andy Shevchenko