On Monday 14 April 2014 22:35:05 Ben Dooks wrote: > Add the DMA resource IDs for the R8A7790 Audio and SYS DMA controllers > for use when specifying DMA handles. > > Signed-off-by: Ben Dooks <ben.dooks@xxxxxxxxxxxxxxx> I really hate these lists in bindings: > +#define R8A7790_DMA_SCIFA0_TX (0x21) > +#define R8A7790_DMA_SCIFA0_RX (0x22) > +#define R8A7790_DMA_SCIFA1_TX (0x25) > +#define R8A7790_DMA_SCIFA1_RX (0x26) > +#define R8A7790_DMA_SCIFA2_TX (0x27) > +#define R8A7790_DMA_SCIFA2_RX (0x28) > + > +#define R8A7790_DMA_SCIFB0_TX (0x3D) > +#define R8A7790_DMA_SCIFB0_RX (0x3E) > +#define R8A7790_DMA_SCIFB1_TX (0x19) > +#define R8A7790_DMA_SCIFB1_RX (0x1A) > +#define R8A7790_DMA_SCIFB2_TX (0x1D) > +#define R8A7790_DMA_SCIFB2_RX (0x1E) These are all hardware constants, they should come from the data sheet and get put into the dtsi file. The driver doesn't care what they are, and the binding doesn't care, this only serves to make the binding less generic. > +#define CHCR_RX_32BIT SHDMA_ARM_CHCR_RX(SHDMA_ARM_SZ_32BIT) > +#define CHCR_TX_32BIT SHDMA_ARM_CHCR_TX(SHDMA_ARM_SZ_32BIT) > +#define CHCR_RX_256BIT SHDMA_ARM_CHCR_RX(SHDMA_ARM_SZ_256BIT) > +#define CHCR_TX_256BIT SHDMA_ARM_CHCR_TX(SHDMA_ARM_SZ_256BIT) For these, it's different: There is nothing wrong putting macros like these into DT, because they will be the same across all SoCs. However, as I mentioned in my reply to patch 8, I don't think that information belongs into DT, and it should be in the device driver instead, because that already knows the direction and transfer width and has to set it through the slave config interface anyway. Arnd -- To unsubscribe from this list: send the line "unsubscribe dmaengine" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html