Re: [PATCH v4 00/22] drm/i915: Organize most GPU features by platform

[Rodrigo Vivi <rodrigo.vivi@gmail.com>]

All patches reviewed and merged to dinq last week. Thanks for the
patches and sorry by the delayed update.

On Wed, Aug 17, 2016 at 12:30 PM, Carlos Santa <carlos.santa@intel.com> wrote:
>  - organize most GPU features so that they are easy to group by platforms.
>    It seems some of the ground work was already done for Gen7 features.
>
>  - make most of these GPU features now a device_info flag also based on
>    previous work done by others. The idea is here is to have a central place where
>    to add new features, now it should also be possible to see what the supported
>    features are for a given platform by dumping of the struct definitions.
>    The list of the features that were converted to a device_info flag include: PSR,
>    RUNTIME_PM, CORE_RING_FREQ, CSR, GUC, RESOURCE_STREAMER, RC6,
>    RC6p, DP_MST, GMBUS_IRQ, FW_BLC, HW_CONTEXTS, LOGICAL_RING_CONTEXTS, L3_DPF,
>    HWS_NEEDS_PHYSICAL and GMCH_DISPLAY.
>
> - Changes since v3
>
>         * Improve the commit message for GENX_FEATURES macro (GEN6,GEN5,GEN4,GEN3,and GEN2)
>                 - drm/i915: Introduce GEN6_FEATURES for device info
>                 - drm/i915: Introduce GEN5_FEATURES for device info
>                 - drm/i915: Introduce GEN4_FEATURES for device info
>                 - drm/i915: Introduce GEN3_FEATURES for device info
>                 - drm/i915: Introduce GEN2_FEATURES for device info
>
>         * Split removal of runtime PM for SNB into a separate patch in case it needs to be reverted
>                 - drm/i915: Remove runtime PM for SNB
>
>         * Fix the way HAS_LLC() was getting invoked
>                 - Get rid of HAS_CORE_RING_FREQ
>
> Carlos Santa (22):
>   drm/i915: Move HAS_PSR definition to platform struct definition
>   drm/i915: Remove .is_mobile field from platform struct
>   drm/i915: Introduce GEN6_FEATURES for device info
>   drm/i915: Move HAS_RUNTIME_PM definition to platform
>   drm/i915: Remove runtime PM for SNB
>   drm/i915: Get rid of HAS_CORE_RING_FREQ
>   drm/i915 Move HAS_CSR definition to platform definition
>   drm/i915: Move HAS_RESOURCE_STREAMER definition to platform definition
>   drm/i915: Move HAS_RC6 definition to platform definition
>   drm/i915: Move HAS_RC6p definition to platform definition
>   drm/i915: Move HAS_DP_MST definition to platform definition
>   drm/i915: Introduce GEN5_FEATURES for device info
>   drm/i915: Move HAS_GMBUS_IRQ definition to platform definition
>   drm/i915: Introduce GEN4_FEATURES for device info
>   drm/i915: Introduce GEN3_FEATURES for device info
>   drm/i915: Introduce GEN2_FEATURES for device info
>   drm/i915: Move HAS_HW_CONTEXTS definition to platform
>   drm/i915: Move HAS_LOGICAL_RING_CONTEXTS definition to platform
>   drm/i915: Move HAS_L3_DPF definition to platform definition
>   drm/i915: Move HAS_GMCH_DISPLAY definition to platform
>   drm/i915: Make HWS_NEEDS_PHYSICAL the exception
>   drm/i915: Move HAS_GUC definition to platform definition
>
>  arch/x86/kernel/early-quirks.c          |   9 +-
>  drivers/gpu/drm/i915/i915_debugfs.c     |   2 +-
>  drivers/gpu/drm/i915/i915_drv.h         |  57 +++----
>  drivers/gpu/drm/i915/i915_gpu_error.c   |   2 +-
>  drivers/gpu/drm/i915/i915_pci.c         | 285 +++++++++++++++++---------------
>  drivers/gpu/drm/i915/intel_ringbuffer.c |  20 +--
>  include/drm/i915_pciids.h               |  38 ++---
>  7 files changed, 209 insertions(+), 204 deletions(-)
>
> --
> 1.9.1
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx



-- 
Rodrigo Vivi
Blog: http://blog.vivi.eng.br
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx