Reviewed-By: Wladimir J. van der Laan <laanwj@xxxxxxxxx> I do wonder whether we'll need the split formats in practice - e.g. the GC3000 on the i.MX6qp, for which I suppose this is being done because of tiled buffers support in the PRE, has the "single buffer" feature which allows rendering to a single buffer with multiple pixel pipes. This is what the Vivante driver uses. But it can't hurt reserving formats for them. Wladimir > > Vivante GC hardware uses simple 4x4 tiled and nested 64x64 supertiled > > formats as well as so-called split-tiled variants for dual-pipe > > hardware, where even and odd tiles start at different base addresses. > > > > Signed-off-by: Philipp Zabel <p.zabel@xxxxxxxxxxxxxx> > > --- > > include/uapi/drm/drm_fourcc.h | 41 +++++++++++++++++++++++++++++++++++++++++ > > 1 file changed, 41 insertions(+) > > > > diff --git a/include/uapi/drm/drm_fourcc.h b/include/uapi/drm/drm_fourcc.h > > index a5890bf44c0af..ec0498cf61756 100644 > > --- a/include/uapi/drm/drm_fourcc.h > > +++ b/include/uapi/drm/drm_fourcc.h > > @@ -159,6 +159,7 @@ extern "C" { > > #define DRM_FORMAT_MOD_VENDOR_NV 0x03 > > #define DRM_FORMAT_MOD_VENDOR_SAMSUNG 0x04 > > #define DRM_FORMAT_MOD_VENDOR_QCOM 0x05 > > +#define DRM_FORMAT_MOD_VENDOR_VIVANTE 0x06 > > /* add more to the end as needed */ > > > > #define fourcc_mod_code(vendor, val) \ > > @@ -233,6 +234,46 @@ extern "C" { > > */ > > #define DRM_FORMAT_MOD_SAMSUNG_64_32_TILE fourcc_mod_code(SAMSUNG, 1) > > > > +/* Vivante framebuffer modifiers */ > > + > > +/* > > + * Vivante 4x4 tiling layout > > + * > > + * This is a simple tiled layout using tiles of 4x4 pixels in a row-major > > + * layout. > > + */ > > +#define DRM_FORMAT_MOD_VIVANTE_TILED fourcc_mod_code(VIVANTE, 1) > > + > > +/* > > + * Vivante 64x64 super-tiling layout > > + * > > + * This is a tiled layout using 64x64 pixel super-tiles, where each super-tile > > + * contains 8x4 groups of 2x4 tiles of 4x4 pixels (like above) each, all in row- > > + * major layout. > > + * > > + * For more information: see > > + * https://github.com/etnaviv/etna_viv/blob/master/doc/hardware.md#texture-tiling > > + */ > > +#define DRM_FORMAT_MOD_VIVANTE_SUPER_TILED fourcc_mod_code(VIVANTE, 2) > > + > > +/* > > + * Vivante 4x4 tiling layout for dual-pipe > > + * > > + * Same as the 4x4 tiling layout, except every second 4x4 pixel tile starts at a > > + * different base address. Offsets from the base addresses are therefore halved > > + * compared to the non-split tiled layout. > > + */ > > +#define DRM_FORMAT_MOD_VIVANTE_SPLIT_TILED fourcc_mod_code(VIVANTE, 3) > > + > > +/* > > + * Vivante 64x64 super-tiling layout for dual-pipe > > + * > > + * Same as the 64x64 super-tiling layout, except every second 4x4 pixel tile > > + * starts at a different base address. Offsets from the base addresses are > > + * therefore halved compared to the non-split super-tiled layout. > > + */ > > +#define DRM_FORMAT_MOD_VIVANTE_SPLIT_SUPER_TILED fourcc_mod_code(VIVANTE, 4) > > Does this mean the dual-pipe stuff has 2 planes? Do you need Ville's > driver-private format stuff for this? > > Also, someone else with clue needs to review this before I can pull it in > :-) > -Daniel > > > + > > #if defined(__cplusplus) > > } > > #endif > > -- > > 2.11.0 > > > > _______________________________________________ > > dri-devel mailing list > > dri-devel@xxxxxxxxxxxxxxxxxxxxx > > https://lists.freedesktop.org/mailman/listinfo/dri-devel > > -- > Daniel Vetter > Software Engineer, Intel Corporation > http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel