On Wed, 03 Mar 2021, "Shankar, Uma" <uma.shankar@xxxxxxxxx> wrote: > Looks Good to me. > Reviewed-by: Uma Shankar <uma.shankar@xxxxxxxxx> Thanks for the review, pushed the series. >> diff --git a/drivers/gpu/drm/i915/display/intel_display_types.h >> b/drivers/gpu/drm/i915/display/intel_display_types.h >> index 1a76e1d9de7a..9f1945018712 100644 >> --- a/drivers/gpu/drm/i915/display/intel_display_types.h >> +++ b/drivers/gpu/drm/i915/display/intel_display_types.h >> @@ -1166,6 +1166,13 @@ struct intel_crtc_state { >> u8 pipeline_full; >> u16 flipline, vmin, vmax; >> } vrr; >> + >> + /* Stream Splitter for eDP MSO */ >> + struct { >> + bool enable; >> + u8 link_count; >> + u8 pixel_overlap; >> + } splitter; >> }; Due to certain amount of urgency, I decided to push the series with this naming in crtc state. I think it's accurate now, and workable for legacy DSI as well. However, I'm not hung up on it, and I'm open to renaming when the time comes to convert legacy DSI to using crtc state for this stuff. BR, Jani. -- Jani Nikula, Intel Open Source Graphics Center _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx