lkml.org 
[lkml]   [2022]   [Mar]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [Intel-gfx] [PATCH v11 0/5] Add driver for GSC controller
Date
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@intel.com>
> Sent: Tuesday, March 22, 2022 22:10
> To: Usyskin, Alexander <alexander.usyskin@intel.com>; Greg Kroah-
> Hartman <gregkh@linuxfoundation.org>; Jani Nikula
> <jani.nikula@linux.intel.com>; Joonas Lahtinen
> <joonas.lahtinen@linux.intel.com>; Vivi, Rodrigo <rodrigo.vivi@intel.com>;
> David Airlie <airlied@linux.ie>; Daniel Vetter <daniel@ffwll.ch>; Tvrtko
> Ursulin <tvrtko.ursulin@linux.intel.com>
> Cc: linux-kernel@vger.kernel.org; Winkler, Tomas
> <tomas.winkler@intel.com>; Lubart, Vitaly <vitaly.lubart@intel.com>; intel-
> gfx@lists.freedesktop.org
> Subject: Re: [Intel-gfx] [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
> >

\
 
 \ /
  Last update: 2022-03-28 09:40    [W:0.164 / U:1.056 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site