Re: [Intel-gfx] [PATCH v11 0/5] Add driver for GSC controller

["Ceraolo Spurio, Daniele" <daniele.ceraolospurio@intel.com>]

Can you re-send this series with an added patch to force 
CONFIG_INTEL_MEI_GSC to be selected for CI? we don't need to review or 
merge that additional patch, but I want to make sure we get CI results 
with the config turned on before we merge this series. I'm also going to 
ping the CI team to see if we can turn it on by default for CI builds.

Daniele

On 3/15/2022 6:11 AM, Alexander Usyskin wrote:
> GSC is a graphics system controller, it provides
> a chassis controller for graphics discrete cards.
>
> There are two MEI interfaces in GSC: HECI1 and HECI2.
>
> This series includes instantiation of the auxiliary devices for HECI2
> and mei-gsc auxiliary device driver that binds to the auxiliary device.
>
> The prinicpal user of this interface is the
> Intel Graphics System Controller Firmware Update Library (IGSC FU)
> (https://github.com/intel/igsc)
>
> In v2 the platform device was replaced by the auxiliary device.
> v3 is the rebase over drm-tip to make public CI running.
> In v4 the not needed debug prints and empty line were removed,
>        'select' were replaced by 'depends on' in MEI Kconfig,
>        the new include file now listed in the MAINTATINERS file.
> V5, rebase and add Greg KH Reviewed-by
> V6, rebase and drop redundant assignments found by the kernel test
> robot.
> V7, add Greg KH Reviewed-by to the individual patches
> V8, address Tvrtko comments for i915
> V9, rebase and address more Tvrtko comments, use drm error printing
> V10, rebase
> V11, address Rodrigo comments about code style,
>       set missed mask in the interrupt config,
>       add explicit devm_irq_free to error and remove flows
>
> Tomas, please look at the devm_irq_free part.
>
> Alexander Usyskin (2):
>    mei: gsc: setup char driver alive in spite of firmware handshake
>      failure
>    mei: gsc: retrieve the firmware version
>
> Tomas Winkler (3):
>    drm/i915/gsc: add gsc as a mei auxiliary device
>    mei: add support for graphics system controller (gsc) devices
>    mei: gsc: add runtime pm handlers
>
>   MAINTAINERS                              |   1 +
>   drivers/gpu/drm/i915/Kconfig             |   1 +
>   drivers/gpu/drm/i915/Makefile            |   3 +
>   drivers/gpu/drm/i915/gt/intel_gsc.c      | 204 ++++++++++++++++++
>   drivers/gpu/drm/i915/gt/intel_gsc.h      |  37 ++++
>   drivers/gpu/drm/i915/gt/intel_gt.c       |   3 +
>   drivers/gpu/drm/i915/gt/intel_gt.h       |   5 +
>   drivers/gpu/drm/i915/gt/intel_gt_irq.c   |  13 ++
>   drivers/gpu/drm/i915/gt/intel_gt_regs.h  |   1 +
>   drivers/gpu/drm/i915/gt/intel_gt_types.h |   2 +
>   drivers/gpu/drm/i915/i915_drv.h          |   8 +
>   drivers/gpu/drm/i915/i915_pci.c          |   3 +-
>   drivers/gpu/drm/i915/i915_reg.h          |   2 +
>   drivers/gpu/drm/i915/intel_device_info.h |   2 +
>   drivers/misc/mei/Kconfig                 |  14 ++
>   drivers/misc/mei/Makefile                |   3 +
>   drivers/misc/mei/bus-fixup.c             |  25 +++
>   drivers/misc/mei/gsc-me.c                | 259 +++++++++++++++++++++++
>   drivers/misc/mei/hw-me.c                 |  29 ++-
>   drivers/misc/mei/hw-me.h                 |   2 +
>   include/linux/mei_aux.h                  |  19 ++
>   21 files changed, 633 insertions(+), 3 deletions(-)
>   create mode 100644 drivers/gpu/drm/i915/gt/intel_gsc.c
>   create mode 100644 drivers/gpu/drm/i915/gt/intel_gsc.h
>   create mode 100644 drivers/misc/mei/gsc-me.c
>   create mode 100644 include/linux/mei_aux.h
>