From nobody Wed Dec 17 19:07:50 2025 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id A4337C77B72 for ; Mon, 17 Apr 2023 21:40:51 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230408AbjDQVku (ORCPT ); Mon, 17 Apr 2023 17:40:50 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:40566 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230218AbjDQVkm (ORCPT ); Mon, 17 Apr 2023 17:40:42 -0400 Received: from madras.collabora.co.uk (madras.collabora.co.uk [IPv6:2a00:1098:0:82:1000:25:2eeb:e5ab]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 57D374C16; Mon, 17 Apr 2023 14:40:40 -0700 (PDT) Received: from jupiter.universe (dyndsl-091-248-191-155.ewe-ip-backbone.de [91.248.191.155]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: sre) by madras.collabora.co.uk (Postfix) with ESMTPSA id EF2366603236; Mon, 17 Apr 2023 22:40:38 +0100 (BST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1681767639; bh=UynN8wwAWCU0QowIo5m3jRjJPnlQ7WEXJDNgTSemnqQ=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=ooOoMWaFWTQJowHowA8rAID5lEq4k5d/D4+FczpMSaf6AQfCKdSSmQ+z3DKGJTHbj tglAqcBknXSCpS6w7BKaWbZ3BGTZAKGtK+rSt6jMkaZ/JJK1mgRUhFV/vLErdFMY4a UnRYCM2mbbPppbo1Fs9orS6pRMIK/3p9JrCNSktEE5jZwx3ur/AmOgDCI+2dmVB/y4 uxvi70f9CjuC6JdzAGaeS3wFfQQ+6YLjD2qX9I8A/kvxq73dAM1YDNvokL8zeZ6UtJ EYFBIuj5xUrJIWl2NhXLvYL4KSvj6rFJAXhSC66S1UL5T3/aUyWaaMnjcJzbjnN6uy 4mLYHO13CxHFg== Received: by jupiter.universe (Postfix, from userid 1000) id 014954807E3; Mon, 17 Apr 2023 23:40:36 +0200 (CEST) From: Sebastian Reichel To: Marc Zyngier , Heiko Stuebner Cc: Rob Herring , Krzysztof Kozlowski , Thomas Gleixner , Peng Fan , Robin Murphy , Peter Geis , XiaoDong Huang , Kever Yang , linux-rockchip@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Sebastian Reichel , kernel@collabora.com Subject: [PATCH v3 1/2] irqchip/gic-v3: Add Rockchip 3588001 errata workaround Date: Mon, 17 Apr 2023 23:40:34 +0200 Message-Id: <20230417214035.101190-2-sebastian.reichel@collabora.com> X-Mailer: git-send-email 2.39.2 In-Reply-To: <20230417214035.101190-1-sebastian.reichel@collabora.com> References: <20230417214035.101190-1-sebastian.reichel@collabora.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" Rockchip RK3588/RK3588s GIC600 integration does not support the sharability feature. Rockchip assigned Errata ID #3588001 for this issue. Note, that the 0x0201743b ID is not Rockchip specific and thus there is an extra of_machine_is_compatible() check. The flag are named BROKEN_SHAREABILITY to be vendor agnostic, since apparently similar integration design errors exist in other platforms and they can reuse the same flag. Co-developed-by: XiaoDong Huang Signed-off-by: XiaoDong Huang Co-developed-by: Kever Yang Signed-off-by: Kever Yang Co-developed-by: Lucas Tanure Signed-off-by: Lucas Tanure Signed-off-by: Sebastian Reichel Reviewed-by: AngeloGioacchino Del Regno --- Documentation/arm64/silicon-errata.rst | 3 +++ arch/arm64/Kconfig | 10 ++++++++ drivers/irqchip/irq-gic-v3-its.c | 35 ++++++++++++++++++++++++++ 3 files changed, 48 insertions(+) diff --git a/Documentation/arm64/silicon-errata.rst b/Documentation/arm64/s= ilicon-errata.rst index ec5f889d7681..46d06ed3e4f4 100644 --- a/Documentation/arm64/silicon-errata.rst +++ b/Documentation/arm64/silicon-errata.rst @@ -205,6 +205,9 @@ stable kernels. +----------------+-----------------+-----------------+--------------------= ---------+ | Qualcomm Tech. | Kryo4xx Gold | N/A | ARM64_ERRATUM_12868= 07 | +----------------+-----------------+-----------------+--------------------= ---------+ ++----------------+-----------------+-----------------+--------------------= ---------+ +| Rockchip | RK3588 | #3588001 | ROCKCHIP_ERRATUM_35= 88001 | ++----------------+-----------------+-----------------+--------------------= ---------+ =20 +----------------+-----------------+-----------------+--------------------= ---------+ | Fujitsu | A64FX | E#010001 | FUJITSU_ERRATUM_010= 001 | diff --git a/arch/arm64/Kconfig b/arch/arm64/Kconfig index 1023e896d46b..1bc43faef10a 100644 --- a/arch/arm64/Kconfig +++ b/arch/arm64/Kconfig @@ -1150,6 +1150,16 @@ config NVIDIA_CARMEL_CNP_ERRATUM =20 If unsure, say Y. =20 +config ROCKCHIP_ERRATUM_3588001 + bool "Rockchip 3588001: GIC600 can not support shareability attributes" + default y + help + The Rockchip RK3588 GIC600 SoC integration does not support ACE/ACE-lit= e. + This means, that the GIC600 may not use the GIC600 sharability feature + even though it is supported by the IP itself. + + If unsure, say Y. + config SOCIONEXT_SYNQUACER_PREITS bool "Socionext Synquacer: Workaround for GICv3 pre-ITS" default y diff --git a/drivers/irqchip/irq-gic-v3-its.c b/drivers/irqchip/irq-gic-v3-= its.c index 586271b8aa39..5b7aa48dde25 100644 --- a/drivers/irqchip/irq-gic-v3-its.c +++ b/drivers/irqchip/irq-gic-v3-its.c @@ -42,9 +42,11 @@ #define ITS_FLAGS_CMDQ_NEEDS_FLUSHING (1ULL << 0) #define ITS_FLAGS_WORKAROUND_CAVIUM_22375 (1ULL << 1) #define ITS_FLAGS_WORKAROUND_CAVIUM_23144 (1ULL << 2) +#define ITS_FLAGS_BROKEN_SHAREABILITY (1ULL << 3) =20 #define RDIST_FLAGS_PROPBASE_NEEDS_FLUSHING (1 << 0) #define RDIST_FLAGS_RD_TABLES_PREALLOCATED (1 << 1) +#define RDIST_FLAGS_BROKEN_SHAREABILITY (1 << 2) =20 #define RD_LOCAL_LPI_ENABLED BIT(0) #define RD_LOCAL_PENDTABLE_PREALLOCATED BIT(1) @@ -2359,6 +2361,9 @@ static int its_setup_baser(struct its_node *its, stru= ct its_baser *baser, its_write_baser(its, baser, val); tmp =3D baser->val; =20 + if (its->flags & ITS_FLAGS_BROKEN_SHAREABILITY) + tmp &=3D ~GITS_BASER_SHAREABILITY_MASK; + if ((val ^ tmp) & GITS_BASER_SHAREABILITY_MASK) { /* * Shareability didn't stick. Just use @@ -3096,6 +3101,9 @@ static void its_cpu_init_lpis(void) gicr_write_propbaser(val, rbase + GICR_PROPBASER); tmp =3D gicr_read_propbaser(rbase + GICR_PROPBASER); =20 + if (gic_rdists->flags & RDIST_FLAGS_BROKEN_SHAREABILITY) + tmp &=3D ~GICR_PROPBASER_SHAREABILITY_MASK; + if ((tmp ^ val) & GICR_PROPBASER_SHAREABILITY_MASK) { if (!(tmp & GICR_PROPBASER_SHAREABILITY_MASK)) { /* @@ -3120,6 +3128,9 @@ static void its_cpu_init_lpis(void) gicr_write_pendbaser(val, rbase + GICR_PENDBASER); tmp =3D gicr_read_pendbaser(rbase + GICR_PENDBASER); =20 + if (gic_rdists->flags & RDIST_FLAGS_BROKEN_SHAREABILITY) + tmp &=3D ~GICR_PENDBASER_SHAREABILITY_MASK; + if (!(tmp & GICR_PENDBASER_SHAREABILITY_MASK)) { /* * The HW reports non-shareable, we must remove the @@ -4710,6 +4721,19 @@ static bool __maybe_unused its_enable_quirk_hip07_16= 1600802(void *data) return true; } =20 +static bool __maybe_unused its_enable_rk3588001(void *data) +{ + struct its_node *its =3D data; + + if (!of_machine_is_compatible("rockchip,rk3588")) + return false; + + its->flags |=3D ITS_FLAGS_BROKEN_SHAREABILITY; + gic_rdists->flags |=3D RDIST_FLAGS_BROKEN_SHAREABILITY; + + return true; +} + static const struct gic_quirk its_quirks[] =3D { #ifdef CONFIG_CAVIUM_ERRATUM_22375 { @@ -4755,6 +4779,14 @@ static const struct gic_quirk its_quirks[] =3D { .mask =3D 0xffffffff, .init =3D its_enable_quirk_hip07_161600802, }, +#endif +#ifdef CONFIG_ROCKCHIP_ERRATUM_3588001 + { + .desc =3D "ITS: Rockchip erratum RK3588001", + .iidr =3D 0x0201743b, + .mask =3D 0xffffffff, + .init =3D its_enable_rk3588001, + }, #endif { } @@ -5096,6 +5128,9 @@ static int __init its_probe_one(struct resource *res, gits_write_cbaser(baser, its->base + GITS_CBASER); tmp =3D gits_read_cbaser(its->base + GITS_CBASER); =20 + if (its->flags & ITS_FLAGS_BROKEN_SHAREABILITY) + tmp &=3D ~GITS_CBASER_SHAREABILITY_MASK; + if ((tmp ^ baser) & GITS_CBASER_SHAREABILITY_MASK) { if (!(tmp & GITS_CBASER_SHAREABILITY_MASK)) { /* --=20 2.39.2 From nobody Wed Dec 17 19:07:50 2025 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id B077DC77B76 for ; Mon, 17 Apr 2023 21:40:48 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230307AbjDQVkr (ORCPT ); Mon, 17 Apr 2023 17:40:47 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:40548 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229614AbjDQVkl (ORCPT ); Mon, 17 Apr 2023 17:40:41 -0400 Received: from madras.collabora.co.uk (madras.collabora.co.uk [IPv6:2a00:1098:0:82:1000:25:2eeb:e5ab]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 73E2F4C28; Mon, 17 Apr 2023 14:40:40 -0700 (PDT) Received: from jupiter.universe (dyndsl-091-248-191-155.ewe-ip-backbone.de [91.248.191.155]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits)) (No client certificate requested) (Authenticated sender: sre) by madras.collabora.co.uk (Postfix) with ESMTPSA id 06E126603239; Mon, 17 Apr 2023 22:40:39 +0100 (BST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1681767639; bh=JaiGX/zKLMMzlVpGom/T53NzObIEnY5d4OtYAaV22XU=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=JBC4My2jOefTtSE40zQd5Zg2VO3ms+tO9845Oz4F7aCh3uJ7OxO/LgWe/WlbVz7te pG7XgSmDV83dQklr8xLw0oLlZtbeLZSJW5PJhHzBxnvd8CppU6qtSKf3XxJXbQF0aK K45SpXQe5879AxpkTVfBAukjduR09sIJhKplpLmN/XkQpxiKY7CM2xIRbo+JN4CNow Q1ktK0PrThXvY9P2DmKwYI8MYJ5VJafqG2pVKKchdCUlHy2Q/vZFYSPfk80/X4pnFC VFXF7TgJjasFNTWudpFPI9JQjcu2v3TMp4/QW8YLQgfP8osQ7IT17BihdVC0jtMTTE GE3BO0lkHXR+w== Received: by jupiter.universe (Postfix, from userid 1000) id 039E94807EF; Mon, 17 Apr 2023 23:40:36 +0200 (CEST) From: Sebastian Reichel To: Marc Zyngier , Heiko Stuebner Cc: Rob Herring , Krzysztof Kozlowski , Thomas Gleixner , Peng Fan , Robin Murphy , Peter Geis , XiaoDong Huang , Kever Yang , linux-rockchip@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Sebastian Reichel , kernel@collabora.com Subject: [PATCH v3 2/2] arm64: dts: rockchip: rk3588: add GIC ITS support Date: Mon, 17 Apr 2023 23:40:35 +0200 Message-Id: <20230417214035.101190-3-sebastian.reichel@collabora.com> X-Mailer: git-send-email 2.39.2 In-Reply-To: <20230417214035.101190-1-sebastian.reichel@collabora.com> References: <20230417214035.101190-1-sebastian.reichel@collabora.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" Add the two Interrupt Translation Service (ITS) IPs that are part of the GIC-600. They are mainly required for PCIe Message Signalled Interrupts (MSI). Co-developed-by: Kever Yang Signed-off-by: Kever Yang Signed-off-by: Sebastian Reichel --- arch/arm64/boot/dts/rockchip/rk3588s.dtsi | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) diff --git a/arch/arm64/boot/dts/rockchip/rk3588s.dtsi b/arch/arm64/boot/dt= s/rockchip/rk3588s.dtsi index 2124c654f665..62204b96b0b4 100644 --- a/arch/arm64/boot/dts/rockchip/rk3588s.dtsi +++ b/arch/arm64/boot/dts/rockchip/rk3588s.dtsi @@ -1741,7 +1741,24 @@ gic: interrupt-controller@fe600000 { mbi-alias =3D <0x0 0xfe610000>; mbi-ranges =3D <424 56>; msi-controller; + ranges; + #address-cells =3D <2>; #interrupt-cells =3D <4>; + #size-cells =3D <2>; + + its0: msi-controller@fe640000 { + compatible =3D "arm,gic-v3-its"; + msi-controller; + #msi-cells =3D <1>; + reg =3D <0x0 0xfe640000 0x0 0x20000>; + }; + + its1: msi-controller@fe660000 { + compatible =3D "arm,gic-v3-its"; + msi-controller; + #msi-cells =3D <1>; + reg =3D <0x0 0xfe660000 0x0 0x20000>; + }; =20 ppi-partitions { ppi_partition0: interrupt-partition-0 { --=20 2.39.2