On Tue, Oct 23, 2012 at 9:29 PM, Paulo Zanoni <przanoni at gmail.com> wrote: > +#define TRANS_DDI_FUNC_CTL_A 0x60400 > +#define TRANS_DDI_FUNC_CTL_B 0x61400 > +#define TRANS_DDI_FUNC_CTL_C 0x62400 > +#define TRANS_DDI_FUNC_CTL_EDP 0x6F400 > +#define DDI_FUNC_CTL(tran) _TRANSCODER(tran, TRANS_DDI_FUNC_CTL_A, \ > + TRANS_DDI_FUNC_CTL_B) I was also thinking that it'd be nice to know what we need to give to that macro at a glimpse where we call it, so maybe name it TRANS_DDI_FUNC_CTL(trans)? -- Damien