> -----Original Message-----
> From: Greg Kroah-Hartman <gre...@linuxfoundation.org>
> Sent: Wednesday, January 26, 2022 20:06
> To: Usyskin, Alexander <alexander.usys...@intel.com>
> Cc: Jani Nikula <jani.nik...@linux.intel.com>; Joonas Lahtinen
> <joonas.lahti...@linux.intel.com>; Vivi, Rodrigo <rodrigo.v...@intel.com>;
> David Airlie <airl...@linux.ie>; Daniel Vetter <dan...@ffwll.ch>; Winkler,
> Tomas <tomas.wink...@intel.com>; Lubart, Vitaly <vitaly.lub...@intel.com>;
> intel-gfx@lists.freedesktop.org; linux-ker...@vger.kernel.org
> Subject: Re: [PATCH v2 2/5] mei: add support for graphics system controller
> (gsc) devices
>
> On Wed, Jan 19, 2022 at 05:58:04PM +0200, Alexander Usyskin wrote:
> > From: Tomas Winkler <tomas.wink...@intel.com>
> >
> > GSC is a graphics system controller, based on CSE, it provides
> > a chassis controller for graphics discrete cards, as well as it
> > supports media protection on selected devices.
> >
> > mei_gsc binds to a auxiliary devices exposed by Intel discrete
> > driver i915.
> >
> > Signed-off-by: Alexander Usyskin <alexander.usys...@intel.com>
> > Signed-off-by: Tomas Winkler <tomas.wink...@intel.com>
> > ---
> > drivers/misc/mei/Kconfig | 14 +++
> > drivers/misc/mei/Makefile | 3 +
> > drivers/misc/mei/gsc-me.c | 192
> ++++++++++++++++++++++++++++++++++++++
> > drivers/misc/mei/hw-me.c | 27 +++++-
> > drivers/misc/mei/hw-me.h | 2 +
> > 5 files changed, 236 insertions(+), 2 deletions(-)
> > create mode 100644 drivers/misc/mei/gsc-me.c
> >
> > diff --git a/drivers/misc/mei/Kconfig b/drivers/misc/mei/Kconfig
> > index 0e0bcd0da852..ec119bb98251 100644
> > --- a/drivers/misc/mei/Kconfig
> > +++ b/drivers/misc/mei/Kconfig
> > @@ -46,6 +46,20 @@ config INTEL_MEI_TXE
> > Supported SoCs:
> > Intel Bay Trail
> >
> > +config INTEL_MEI_GSC
> > + tristate "Intel MEI GSC embedded device"
> > + select INTEL_MEI
> > + select INTEL_MEI_ME
>
> Please don't select, why not just depend on?
These are hard dependencies. If user wants to have INTEL_GSC,
user should enable INTEL_MEI and INTEL_MEI_ME anyway.
INTEL_MEI_ME selects INTEL_MEI in this file and it was taken as example.
What is wrong with select? Why to avoid it use?
--
Thanks,
Sasha
>
> thanks,
>
> greg k-h