On Fri, May 31, 2013 at 11:30:29PM +0200, Arnd Bergmann wrote: > On Thursday 30 May 2013, ludovic.desroc...@atmel.com wrote: > > +/* > > + * Source and/or destination peripheral ID > > + */ > > +#define AT91_DMA_CFG_PER_ID_MASK (0xff) > > +#define AT91_DMA_CFG_PER_ID(id) (id & > > AT91_DMA_CFG_PER_ID_MASK) > > + > > I'm a little worried about macros like this one spreading in the name > of readability, and I don't see this as actual value-add here: > > The output of the macro is always the same as the input, and passing > an invalid number still results in an invalid output and no error message, > it actually prevents us from warning about the mistake at run-time. >
You are right, this macro is here only for readability in order to not have something like this: dmas = <&dma0 AT91_DMA_CFG_FIFOCFG_ASAP | 13> In this case, I prefer having a macro instead of using a numeric value which can stand for other things than the peripheral id and which could conflict with other parameters such as the FIFO configuration. If adding this macro is really an issue, it could be removed of course but in this case I am in favour of readability. Ludovic > The other macros are fine. > > Arnd _______________________________________________ devicetree-discuss mailing list devicetree-discuss@lists.ozlabs.org https://lists.ozlabs.org/listinfo/devicetree-discuss