On Thu, Sep 04, 2014 at 01:32:24PM +0100, Damien Lespiau wrote: > On Thu, Sep 04, 2014 at 03:11:24PM +0300, Jani Nikula wrote: > > On Thu, 04 Sep 2014, Damien Lespiau <damien.lespiau@xxxxxxxxx> wrote: > > > The new version of the macro does a few things better: > > > - protect the arguments, > > > - only evaluate the arguments once, > > > - check that the arguments are of the same type, > > > > > > Change LC_FREQ_2K to be a unsigned 64bit constant and removed the '()' > > > from the caller as a result. > > > > > > Signed-off-by: Damien Lespiau <damien.lespiau@xxxxxxxxx> > > > > Reviewed-by: Jani Nikula <jani.nikula@xxxxxxxxx> > > > > Can be picked up for dinq separate from this series, right? > > Yes, it can indeed. s/can/must/ If the platform enabling changes shared code, that part must go in through the normal merge process and has the same merge window cutoff deadlines like everything else. Queued for -next, thanks for the patch. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
