[Intel-gfx] [PATCH v8 4/7] drm/i915: use a macro to define MOCS entries
Lucas De Marchi
lucas.de.marchi at gmail.com
Tue Jan 22 21:33:25 UTC 2019
On Tue, Jan 22, 2019 at 6:32 AM Chris Wilson <chris at chris-wilson.co.uk> wrote:
>
> Quoting Lucas De Marchi (2019-01-22 05:12:24)
> > Let's use a macro to make tables smaller and at the same time allow us
> > to add fields that apply to all entries in future.
> >
> > For the sake of readability, I'm calling an exception on 80 chars limit.
> > Lines are aligned for easy comparison of the entry values.
>
> > + MOCS_ENTRY(I915_MOCS_UNCACHED, LE_1_UC | LE_TC_2_LLC_ELLC, \
> > + L3_1_UC), \
>
> MOCS_ENTRY(I915_MOCS_UNCACHED,
> LE_1_UC | LE_TC_2_LLC_ELLC, L3_1_UC), \
My intention was to split the lines for each *value*, so it's easy to
see what control_value vs l3cc_value is set to
(too difficult to spot mistakes on adding a comma rather than a |).
But I'm not strongly against your version, so I'll switch to that.
thanks
Lucas De Marchi
>
> is even more readable with the visual clustering of attribute flags.
> -Chris
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
--
Lucas De Marchi
More information about the Intel-gfx
mailing list