Op 05-01-18 om 10:45 schreef Shashank Sharma: > Currently, we are using a bool in CRTC state (state->ycbcr420), > to indicate modeset, that the output format is YCBCR 4:2:0. Now in > order to support other YCBCR formats, we will need more such flags. > > The idea behind this patch is to replace this bool with an enum, > and plug this in in the existing YCBCR 4:2:0 framework in such a > way that this can be re-used for YCBCR 4:4:4 and other output formats too. > > This patch adds a new enum for CRTC output formats, and then plugs it > in the existing modeset framework. > > V3: Added this patch in the series, to address review comments from > second patchset. > > Cc: Ville Syrjala <[email protected]> > Cc: Maarten Lankhorst <[email protected]> > Signed-off-by: Shashank Sharma <[email protected]> Series looks good to me now, so feel free to add my r-b to where its missing. But you still need an ack from Ville I think. :)
Reviewed-by: Maarten Lankhorst <[email protected]> ~Maarten _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
