Thread (32 messages) 32 messages, 8 authors, 2021-04-06

Re: [PATCH v3 08/15] arm64: socfpga: merge Agilex and N5X into ARCH_INTEL_SOCFPGA

From: Krzysztof Kozlowski <hidden>
Date: 2021-04-06 16:15:39
Also in: dmaengine, linux-clk, linux-devicetree, linux-edac, linux-fpga, linux-i2c, lkml, netdev

On Tue, 6 Apr 2021 at 18:10, Guenter Roeck [off-list ref] wrote:
On Thu, Mar 11, 2021 at 04:25:38PM +0100, Krzysztof Kozlowski wrote:
quoted
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@canonical.com/ (local)
(and https://lore.kernel.org/lkml/20210404124609.122377-1-dinguyen@kernel.org/ (local) )
but for some reason it did not go to the same tree.

Best regards,
Krzysztof

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help