Hi As I understand the config patch should go to https://gitlab.freedesktop.org/gfx-ci/i915-infra kconfig/debug, branch "master". Thus, this series does not need this change. Am I right? -- Thanks, Sasha > -----Original Message----- > From: Ceraolo Spurio, Daniele <daniele.ceraolospurio@xxxxxxxxx> > Sent: Tuesday, March 22, 2022 22:10 > To: Usyskin, Alexander <alexander.usyskin@xxxxxxxxx>; Greg Kroah- > Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>; Jani Nikula > <jani.nikula@xxxxxxxxxxxxxxx>; Joonas Lahtinen > <joonas.lahtinen@xxxxxxxxxxxxxxx>; Vivi, Rodrigo <rodrigo.vivi@xxxxxxxxx>; > David Airlie <airlied@xxxxxxxx>; Daniel Vetter <daniel@xxxxxxxx>; Tvrtko > Ursulin <tvrtko.ursulin@xxxxxxxxxxxxxxx> > Cc: linux-kernel@xxxxxxxxxxxxxxx; Winkler, Tomas > <tomas.winkler@xxxxxxxxx>; Lubart, Vitaly <vitaly.lubart@xxxxxxxxx>; intel- > gfx@xxxxxxxxxxxxxxxxxxxxx > Subject: Re: [PATCH v11 0/5] Add driver for GSC controller > > 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 > >
