RE: [PATCH] issei: INTEL_SSEI should depend on X86

[Usyskin, Alexander]

> Subject: RE: [PATCH] issei: INTEL_SSEI should depend on X86
> 
> > Subject: Re: [PATCH] issei: INTEL_SSEI should depend on X86
> >
> > Hi David,
> >
> > On Thu, 27 Aug 2026 at 10:35, David Laight <david.laight.linux@xxxxxxxxx>
> > wrote:
> > > On Wed, 26 Aug 2026 16:03:11 +0200
> > > Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> wrote:
> > > > On Wed, 26 Aug 2026 at 11:08, David Laight
> > <david.laight.linux@xxxxxxxxx> wrote:
> > > > > On Thu,  6 Aug 2026 11:46:55 +0200
> > > > > Geert Uytterhoeven <geert+renesas@xxxxxxxxx> wrote:
> > > > > > The Intel Silicon Security Engine is only present on systems with Intel
> > > > > > Lunar Lake CPUs.  Hence add a dependency on X86, to prevent asking
> > the
> > > > > > user about this subsystem when configuring a kernel for a non-x86
> > > > > > architecture.
> > > > >
> > > > > Should it be X86_64 ?
> > > >
> > > > I don't know. Can you still run 32-bit kernels on these CPUs?
> > >
> > > You should be able to, but there is no need to support new features.
> >
> > FTR, INTEL_SSEI_HW_HECI (which depends on INTEL_SSEI) depends
> > on X86, too.
> >
> 
> INTEL_SSEI_HW_HECI is related to specific HW, which is x86, in theory
> INTEL_SSEI is HW agnostic.
> But in current state having only one HW implementation it makes sense also
> limit
> INTEL_SSEI to X86.
> Not sure what benefit we'll have from X86_64 thought...
> 
> - -
> Thanks,
> Sasha
> 

Look like it is stalled and I prefer this patch in the original form, frankly.

LGTM
Acked-by: Alexander Usyskin <alexander.usyskin@xxxxxxxxx>

> 
> > > > > > --- a/drivers/misc/issei/Kconfig
> > > > > > +++ b/drivers/misc/issei/Kconfig
> > > > > > @@ -2,6 +2,7 @@
> > > > > >  # Copyright (C) 2023-2026 Intel Corporation
> > > > > >  config INTEL_SSEI
> > > > > >       tristate "Intel Silicon Security Engine Interface"
> > > > > > +     depends on X86 || COMPILE_TEST
> > > > > >       help
> > > > > >         The ISSEI (Intel Silicon Security Engine Interface)
> > > > > >         subsystem provides a communication channel between the host
> > and the
> >
> > Gr{oetje,eeting}s,
> >
> >                         Geert
> >
> > --
> > Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-
> > m68k.org
> >
> > In personal conversations with technical people, I call myself a hacker. But
> > when I'm talking to journalists I just say "programmer" or something like
> that.
> >                                 -- Linus Torvalds