Re: [PATCH 2/2] fpga: stratix10-soc: add COMPILE_TEST support

[Xu Yilun <yilun.xu@linux.intel.com>]

On Fri, Nov 07, 2025 at 11:28:03AM -0800, Rosen Penev wrote:
> On Thu, Nov 6, 2025 at 11:30 PM Xu Yilun <yilun.xu@linux.intel.com> wrote:
> >
> > On Thu, Nov 06, 2025 at 10:59:38AM -0800, Rosen Penev wrote:
> > > Allow the buildbots to find compilation issues.
> > >
> > > Signed-off-by: Rosen Penev <rosenp@gmail.com>
> > > ---
> > >  drivers/fpga/Kconfig | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/drivers/fpga/Kconfig b/drivers/fpga/Kconfig
> > > index 311313f3b282..f2e2776acdd5 100644
> > > --- a/drivers/fpga/Kconfig
> > > +++ b/drivers/fpga/Kconfig
> > > @@ -60,7 +60,7 @@ config FPGA_MGR_ZYNQ_FPGA
> > >
> > >  config FPGA_MGR_STRATIX10_SOC
> > >       tristate "Intel Stratix10 SoC FPGA Manager"
> > > -     depends on (ARCH_INTEL_SOCFPGA && INTEL_STRATIX10_SERVICE)
> > > +     depends on (ARCH_INTEL_SOCFPGA && INTEL_STRATIX10_SERVICE) || COMPILE_TEST
> >
> > I don't think it works without INTEL_STRATIX10_SERVICE, maybe:
> >
> >         depends on ARCH_INTEL_SOCFPGA || COMPILE_TEST
> >         depends on INTEL_STRATIX10_SERVICE
> >
> > But INTEL_STRATIX10_SERVICE depends on HAVE_ARM_SMCCC, and they all
> > require arch configurations...
> Yeah I don't think INTEL_STRATIX10_SERVICE is needed for COMPILE_TEST.

Have you actually passed compilation without INTEL_STRATIX10_SERVICE?
I can't image how it works without stratix10_svc_xx kAPI definitions.

> >
> > >       help
> > >         FPGA manager driver support for the Intel Stratix10 SoC.
> > >
> > > --
> > > 2.51.2
> > >
> > >