On Tue, 6 Apr 2021 at 18:10, Guenter Roeck <linux@xxxxxxxxxxxx> wrote: > > On Thu, Mar 11, 2021 at 04:25:38PM +0100, Krzysztof Kozlowski wrote: > > Agilex, N5X and Stratix 10 share all quite similar arm64 hard cores and > > SoC-part. Up to a point that N5X uses the same DTSI as Agilex. From > > the Linux kernel point of view these are flavors of the same > > architecture so there is no need for three top-level arm64 > > architectures. Simplify this by merging all three architectures into > > ARCH_INTEL_SOCFPGA and dropping the other ARCH* arm64 Kconfig entries. > > > > The side effect is that the INTEL_STRATIX10_SERVICE will now be > > available for both 32-bit and 64-bit Intel SoCFPGA, even though it is > > used only for 64-bit. > > Did you try to compile, say, arm:allmodconfig with this patch applied ? > Because for me that results in: > > In file included from <command-line>: > drivers/firmware/stratix10-rsu.c: In function 'rsu_status_callback': > include/linux/compiler_types.h:320:38: error: > call to '__compiletime_assert_177' declared with attribute error: > FIELD_GET: type of reg too small for mask > > and lots of similar errors. Thanks for the report. I fixed that already with: https://lore.kernel.org/lkml/20210321184650.10926-1-krzysztof.kozlowski@xxxxxxxxxxxxx/ (and https://lore.kernel.org/lkml/20210404124609.122377-1-dinguyen@xxxxxxxxxx/ ) but for some reason it did not go to the same tree. Best regards, Krzysztof
