Re: [PATCH v7 5/9] dmaengine: dw: dmamux: Introduce RZN1 DMA router support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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





[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux