Although I'd prefer to kill it, this patch is right, so Reviewed-by: Rodrigo Vivi <[email protected]>
On Tue, Jun 13, 2017 at 12:33 PM, Paulo Zanoni <[email protected]> wrote: > The macro takes a port as an argument, not a pipe. > > Signed-off-by: Paulo Zanoni <[email protected]> > --- > drivers/gpu/drm/i915/i915_reg.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h > index a1d3cca..1983b75 100644 > --- a/drivers/gpu/drm/i915/i915_reg.h > +++ b/drivers/gpu/drm/i915/i915_reg.h > @@ -62,7 +62,7 @@ static inline bool i915_mmio_reg_valid(i915_reg_t reg) > > #define _PORT(port, a, b) ((a) + (port)*((b)-(a))) > #define _MMIO_PORT(port, a, b) _MMIO(_PORT(port, a, b)) > -#define _MMIO_PORT3(pipe, a, b, c) _MMIO(_PICK(pipe, a, b, c)) > +#define _MMIO_PORT3(port, a, b, c) _MMIO(_PICK(port, a, b, c)) > #define _MMIO_PORT6(port, a, b, c, d, e, f) _MMIO(_PICK(port, a, b, c, d, e, > f)) > #define _MMIO_PORT6_LN(port, ln, a0, a1, b, c, d, e, f) > \ > _MMIO(_PICK(port, a0, b, c, d, e, f) + (ln * (a1 - a0))) > -- > 2.9.4 > > _______________________________________________ > Intel-gfx mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Rodrigo Vivi Blog: http://blog.vivi.eng.br _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
