On Wed, 29 Aug 2012 13:50:42 +0100 "Lespiau, Damien" <[email protected]> wrote:
> On Fri, Aug 24, 2012 at 12:14 AM, Ben Widawsky <[email protected]> > wrote: > >> +#define FDI_RXA_DEBUG 0xf0020 > >> +#define FDI_RXB_DEBUG 0xf1020 > > [snip] > > > My only gripe with this patch is in the naming. Can you please use > > something find-able in the BSPEC, like: FDI_RX_DEBUG_B instead of > > FDI_RXB_DEBUG > > I used the same convention as all the others FDI_[R,T]x_[A,B,C]_* > registers, if you look closely: FDX_TXA_CTL, FDX_TXA_MISC, ... I > believe this is because those registers have been introduced with > ironlake and the ILK spec names them like this while later specs adopt > the _A, _B, _C suffixes. I went for consistency. > Generally, I agree with this approach, but in this case I'd rather shift everything over to the new naming instead. -- Ben Widawsky, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
