Re: [PATCH 00/16] Adding NV12 support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Feb 13, 2018 at 8:59 PM Vidya Srinivas <vidya.srinivas@xxxxxxxxx>
wrote:

> This patch series is adding NV12 support for Broxton display after
rebasing on
> latest drm-tip.
> Initial series of the patches can be found here:
> https://lists.freedesktop.org/archives/intel-gfx/2015-May/066786.html

I still don't see support for advertising modifiers in this patch set. The
series should handle NV12 in the supported_modifiers function to indicate
which tiling and/or compression modifiers are supported for NV12
framebuffers.

Kristian




> Previous revision history:
> The first version of patches were reviewed when floated by Chandra in 2015
> but currently there was a design change with respect to
> - the way fb offset is handled
> - the way rotation is handled
> Current NV12 patch series has been ported as per the
> current changes on drm-tip

> Review comments from Ville (12th June 2017) have been addressed Review
> comments from Clinton A Taylor (7th July 2017) have been addressed

> Review comments from Clinton A Taylor (10th July 2017)
>          have been addressed. Had missed out tested-by/reviewed-by in the
patches.

>          Fixed that error in this series.
>          Review comments from Ville (11th July 2017) addressed.
>          Review comments from Paauwe, Bob (29th July 2017) addressed.

> Update from rev 28 Aug 2017
>          Rebased the series.
>          Tested with IGT for rotation, sprite and tiling combinations.
>          IGT Links:
>          https://patchwork.kernel.org/patch/9995943/
>          https://patchwork.kernel.org/patch/9995945/
>          Review comments by Maarten are addressed in this series.
>          NV12 enabled for Gen10.
>          Review comments from Shashank Sharma are addressed.

> Update from last rev:
>          IGT debug_fs test failure fixed.

> Chandra Konduru (6):
>    drm/i915: Set scaler mode for NV12
>    drm/i915: Update format_is_yuv() to include NV12
>    drm/i915: Upscale scaler max scale for NV12
>    drm/i915: Add NV12 as supported format for primary plane
>    drm/i915: Add NV12 as supported format for sprite plane
>    drm/i915: Add NV12 support to intel_framebuffer_init

> Mahesh Kumar (9):
>    drm/i915/skl+: rename skl_wm_values struct to skl_ddb_values
>    drm/i915/skl+: refactor WM calculation for NV12
>    drm/i915/skl+: add NV12 in skl_format_to_fourcc
>    drm/i915/skl+: support verification of DDB HW state for NV12
>    drm/i915/skl+: NV12 related changes for WM
>    drm/i915/skl+: pass skl_wm_level struct to wm compute func
>    drm/i915/skl+: make sure higher latency level has higher wm value
>    drm/i915/skl+: nv12 workaround disable WM level 1-7
>    drm/i915/skl: split skl_compute_ddb function

> Vidya Srinivas (1):
>    drm/i915: Enable YUV to RGB for Gen10 in Plane Ctrl Reg

>   drivers/gpu/drm/i915/i915_drv.h      |  10 +-
>   drivers/gpu/drm/i915/i915_reg.h      |   7 +
>   drivers/gpu/drm/i915/intel_atomic.c  |   8 +-
>   drivers/gpu/drm/i915/intel_display.c |  59 ++++-
>   drivers/gpu/drm/i915/intel_drv.h     |   9 +-
>   drivers/gpu/drm/i915/intel_pm.c      | 433
++++++++++++++++++++++-------------
>   drivers/gpu/drm/i915/intel_sprite.c  |  18 +-
>   7 files changed, 357 insertions(+), 187 deletions(-)

> --
> 2.7.4

> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux