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

[Rosen Penev]

On Thu, Nov 6, 2025 at 11:30 PM Xu Yilun <yilun.xu@xxxxxxxxxxxxxxx> 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@xxxxxxxxx>
> > ---
> >  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.
>
> >       help
> >         FPGA manager driver support for the Intel Stratix10 SoC.
> >
> > --
> > 2.51.2
> >
> >