From mboxrd@z Thu Jan 1 00:00:00 1970 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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 35EBEC00140 for ; Mon, 15 Aug 2022 11:32:41 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-ID:Date:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=QNhOXT2rauTtuvHwlMwTuMDQ+JmHP2cWOB4iJvJvHFY=; b=ltO6k4lqR77emH 7uC5d0xWgu8OD+JjSZPas571UUrEWVt8fNdOvw4MKNQFMIgaelBGL7C1wn5rUH7WmfbtiokUlWM8C lxyoDSYg4SaW55QmyuFPl4ad31MKhuFexYBjJV3u/4Eb3CP1KcUtxpb84xOaViI1/730wWn8Wd0A0 zc5ZY8IWUQLr4KQspGSkon9AKqwFmMZAX6u/3xtjY1/bF9xudRg6nY2C9IJKP6O6HGeNej2D7QYYW owQS6TAI24IHADrXk/QGf9SOHBD8s1VUN83f4qkrwCfBcmi+5i/hxJNGkYfT/sepNEDNYqha1pNp8 xQl9mmVE98cUjoqcpfMw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oNYJY-00FQAw-St; Mon, 15 Aug 2022 11:31:25 +0000 Received: from mx1.tq-group.com ([93.104.207.81]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1oNYJT-00FQ32-LA for linux-arm-kernel@lists.infradead.org; Mon, 15 Aug 2022 11:31:22 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tq-group.com; i=@tq-group.com; q=dns/txt; s=key1; t=1660563079; x=1692099079; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=s8LRGikTNC1M9GDHT268MyAKRQlda5OnSjeu6RwYqHc=; b=COvwFB6qfzFHInWZnjaxABEUb2aZ7sIZesciFQWBjwvMvqBXp/wSLdff ycIfqoDQykemjkLJeaALVnlM8sn6crxoVflq1PD4ycd73efmg9W58aVw+ QUbVHY0S0WxyoGdah5C7QKICFjfaPQvOn6/VSt6lTWDiRSlFwELHPLP2k I63dMERlln42u8BRrrXC/noqi+qTSlKm81DqP87k8xvQkhgyQZ1YJNp4c BJXsc5ZVrjUFeVdWPvKkqqYs3I44Ima5mPIjwvyQzssSOZIjsYO0o1TGp 74FZUn/yKGKgQ4YspjGka03wBuhMbG0NfPm+pvKt0cZYEqMNHKVkC3zCa A==; X-IronPort-AV: E=Sophos;i="5.93,238,1654552800"; d="scan'208";a="25604632" Received: from unknown (HELO tq-pgp-pr1.tq-net.de) ([192.168.6.15]) by mx1-pgp.tq-group.com with ESMTP; 15 Aug 2022 13:31:14 +0200 Received: from mx1.tq-group.com ([192.168.6.7]) by tq-pgp-pr1.tq-net.de (PGP Universal service); Mon, 15 Aug 2022 13:31:14 +0200 X-PGP-Universal: processed; by tq-pgp-pr1.tq-net.de on Mon, 15 Aug 2022 13:31:14 +0200 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tq-group.com; i=@tq-group.com; q=dns/txt; s=key1; t=1660563074; x=1692099074; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=s8LRGikTNC1M9GDHT268MyAKRQlda5OnSjeu6RwYqHc=; b=UHI1XahCvDinNGOdT4MmaDr55GCCAW8UunCYegyJDOirfYQWXM78SQ2u 9T952FZdgC/0YpgjIrEIfJQH6aAIYrd3nNjEPcI7u5B50EkmXhxWfTYsh AEBf//RhfjQFTFO7L6QFxbc0yp70xzsbDRXxscXZRlKOLzBcnyGcIXPtm TuytKS1uaDWHbkF9fD3gHbC3g5Wq/r2MXTENPzotPhPNHnst1TIl0iXMd fxOUH4QuY0Ee3OTXE4tj6wvzypQ12r7+kAKixod7eKXwNHGOdGZwbIWVF u/h33T5YlsU8DBiTrxiXDyf2utMoMSPPoxPRLzNrd0TEmNHpVy7fiuIKr g==; X-IronPort-AV: E=Sophos;i="5.93,238,1654552800"; d="scan'208";a="25604631" Received: from vtuxmail01.tq-net.de ([10.115.0.20]) by mx1.tq-group.com with ESMTP; 15 Aug 2022 13:31:14 +0200 Received: from steina-w.localnet (unknown [10.123.49.11]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) (No client certificate requested) by vtuxmail01.tq-net.de (Postfix) with ESMTPSA id 8FAB6280056; Mon, 15 Aug 2022 13:31:14 +0200 (CEST) From: Alexander Stein To: Rob Herring , Krzysztof Kozlowski Cc: Shawn Guo , Sascha Hauer , Fabio Estevam , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH 1/1] ARM: dts: imx6: delete interrupts property if interrupts-extended is set Date: Mon, 15 Aug 2022 13:31:14 +0200 Message-ID: <1979391.tdWV9SEqCh@steina-w> Organization: TQ-Systems GmbH In-Reply-To: <20220720064158.1413330-1-alexander.stein@ew.tq-group.com> References: <20220720064158.1413330-1-alexander.stein@ew.tq-group.com> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220815_043120_147195_E32FC0EE X-CRM114-Status: GOOD ( 18.08 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hello Rob & Krzysztof, ping. Thanks and best regards Alexander Am Mittwoch, 20. Juli 2022, 08:41:58 CEST schrieb Alexander Stein: > In most cases this is related to fsl,err006687-workaround-present, which > requires a GPIO interrupt next a GIC interrupt. > > This fixes the dtbs_check warning: > imx6dl-mba6a.dtb: ethernet@2188000: More than one condition true in oneOf > schema: {'$filename': 'Documentation/devicetree/bindings/net/fsl,fec.yaml', > [...] > > Signed-off-by: Alexander Stein > --- > Given that all DT add interrupts-extended on the very next line, this > seems the correct thing to do. > > arch/arm/boot/dts/imx6dl-riotboard.dts | 1 + > arch/arm/boot/dts/imx6q-arm2.dts | 1 + > arch/arm/boot/dts/imx6q-evi.dts | 1 + > arch/arm/boot/dts/imx6q-mccmon6.dts | 1 + > arch/arm/boot/dts/imx6qdl-nit6xlite.dtsi | 1 + > arch/arm/boot/dts/imx6qdl-nitrogen6_max.dtsi | 1 + > arch/arm/boot/dts/imx6qdl-nitrogen6_som2.dtsi | 1 + > arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi | 1 + > arch/arm/boot/dts/imx6qdl-sabreauto.dtsi | 1 + > arch/arm/boot/dts/imx6qdl-tqma6a.dtsi | 1 + > arch/arm/boot/dts/imx6qdl-ts7970.dtsi | 1 + > 11 files changed, 11 insertions(+) > > diff --git a/arch/arm/boot/dts/imx6dl-riotboard.dts > b/arch/arm/boot/dts/imx6dl-riotboard.dts index e7d9bfbfd0e4..e7be05f205d3 > 100644 > --- a/arch/arm/boot/dts/imx6dl-riotboard.dts > +++ b/arch/arm/boot/dts/imx6dl-riotboard.dts > @@ -90,6 +90,7 @@ &fec { > pinctrl-0 = <&pinctrl_enet>; > phy-mode = "rgmii-id"; > phy-handle = <&rgmii_phy>; > + /delete-property/ interrupts; > interrupts-extended = <&gpio1 6 IRQ_TYPE_LEVEL_HIGH>, > <&intc 0 119 IRQ_TYPE_LEVEL_HIGH>; > fsl,err006687-workaround-present; > diff --git a/arch/arm/boot/dts/imx6q-arm2.dts > b/arch/arm/boot/dts/imx6q-arm2.dts index 0b40f52268b3..75586299d9ca 100644 > --- a/arch/arm/boot/dts/imx6q-arm2.dts > +++ b/arch/arm/boot/dts/imx6q-arm2.dts > @@ -178,6 +178,7 @@ &fec { > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_enet>; > phy-mode = "rgmii"; > + /delete-property/ interrupts; > interrupts-extended = <&gpio1 6 IRQ_TYPE_LEVEL_HIGH>, > <&intc 0 119 IRQ_TYPE_LEVEL_HIGH>; > fsl,err006687-workaround-present; > diff --git a/arch/arm/boot/dts/imx6q-evi.dts > b/arch/arm/boot/dts/imx6q-evi.dts index c63f371ede8b..78d941fef5df 100644 > --- a/arch/arm/boot/dts/imx6q-evi.dts > +++ b/arch/arm/boot/dts/imx6q-evi.dts > @@ -146,6 +146,7 @@ &fec { > pinctrl-0 = <&pinctrl_enet>; > phy-mode = "rgmii"; > phy-reset-gpios = <&gpio1 25 GPIO_ACTIVE_LOW>; > + /delete-property/ interrupts; > interrupts-extended = <&gpio1 6 IRQ_TYPE_LEVEL_HIGH>, > <&intc 0 119 IRQ_TYPE_LEVEL_HIGH>; > fsl,err006687-workaround-present; > diff --git a/arch/arm/boot/dts/imx6q-mccmon6.dts > b/arch/arm/boot/dts/imx6q-mccmon6.dts index 55692c73943d..64ab01018b71 > 100644 > --- a/arch/arm/boot/dts/imx6q-mccmon6.dts > +++ b/arch/arm/boot/dts/imx6q-mccmon6.dts > @@ -100,6 +100,7 @@ &fec { > pinctrl-0 = <&pinctrl_enet>; > phy-mode = "rgmii"; > phy-reset-gpios = <&gpio1 27 GPIO_ACTIVE_LOW>; > + /delete-property/ interrupts; > interrupts-extended = <&gpio1 6 IRQ_TYPE_LEVEL_HIGH>, > <&intc 0 119 IRQ_TYPE_LEVEL_HIGH>; > status = "okay"; > diff --git a/arch/arm/boot/dts/imx6qdl-nit6xlite.dtsi > b/arch/arm/boot/dts/imx6qdl-nit6xlite.dtsi index 0ad4cb4f1e82..a53a5d0766a5 > 100644 > --- a/arch/arm/boot/dts/imx6qdl-nit6xlite.dtsi > +++ b/arch/arm/boot/dts/imx6qdl-nit6xlite.dtsi > @@ -192,6 +192,7 @@ &fec { > phy-mode = "rgmii"; > phy-handle = <ðphy>; > phy-reset-gpios = <&gpio1 27 GPIO_ACTIVE_LOW>; > + /delete-property/ interrupts; > interrupts-extended = <&gpio1 6 IRQ_TYPE_LEVEL_HIGH>, > <&intc 0 119 IRQ_TYPE_LEVEL_HIGH>; > fsl,err006687-workaround-present; > diff --git a/arch/arm/boot/dts/imx6qdl-nitrogen6_max.dtsi > b/arch/arm/boot/dts/imx6qdl-nitrogen6_max.dtsi index > beaa2dcd436c..57c21a01f126 100644 > --- a/arch/arm/boot/dts/imx6qdl-nitrogen6_max.dtsi > +++ b/arch/arm/boot/dts/imx6qdl-nitrogen6_max.dtsi > @@ -334,6 +334,7 @@ &fec { > phy-mode = "rgmii"; > phy-handle = <ðphy>; > phy-reset-gpios = <&gpio1 27 GPIO_ACTIVE_LOW>; > + /delete-property/ interrupts; > interrupts-extended = <&gpio1 6 IRQ_TYPE_LEVEL_HIGH>, > <&intc 0 119 IRQ_TYPE_LEVEL_HIGH>; > fsl,err006687-workaround-present; > diff --git a/arch/arm/boot/dts/imx6qdl-nitrogen6_som2.dtsi > b/arch/arm/boot/dts/imx6qdl-nitrogen6_som2.dtsi index > ee7e2371f94b..000e9dc97b1a 100644 > --- a/arch/arm/boot/dts/imx6qdl-nitrogen6_som2.dtsi > +++ b/arch/arm/boot/dts/imx6qdl-nitrogen6_som2.dtsi > @@ -263,6 +263,7 @@ &fec { > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_enet>; > phy-mode = "rgmii"; > + /delete-property/ interrupts; > interrupts-extended = <&gpio1 6 IRQ_TYPE_LEVEL_HIGH>, > <&intc 0 119 IRQ_TYPE_LEVEL_HIGH>; > fsl,err006687-workaround-present; > diff --git a/arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi > b/arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi index > 904d5d051d63..731759bdd7f5 100644 > --- a/arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi > +++ b/arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi > @@ -267,6 +267,7 @@ &fec { > phy-mode = "rgmii"; > phy-handle = <ðphy>; > phy-reset-gpios = <&gpio1 27 GPIO_ACTIVE_LOW>; > + /delete-property/ interrupts; > interrupts-extended = <&gpio1 6 IRQ_TYPE_LEVEL_HIGH>, > <&intc 0 119 IRQ_TYPE_LEVEL_HIGH>; > fsl,err006687-workaround-present; > diff --git a/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi > b/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi index 1368a4762037..3dbb460ef102 > 100644 > --- a/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi > +++ b/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi > @@ -295,6 +295,7 @@ &fec { > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_enet>; > phy-mode = "rgmii-id"; > + /delete-property/ interrupts; > interrupts-extended = <&gpio1 6 IRQ_TYPE_LEVEL_HIGH>, > <&intc 0 119 IRQ_TYPE_LEVEL_HIGH>; > fsl,err006687-workaround-present; > diff --git a/arch/arm/boot/dts/imx6qdl-tqma6a.dtsi > b/arch/arm/boot/dts/imx6qdl-tqma6a.dtsi index 7dc3f0005b0f..0a36e1bce375 > 100644 > --- a/arch/arm/boot/dts/imx6qdl-tqma6a.dtsi > +++ b/arch/arm/boot/dts/imx6qdl-tqma6a.dtsi > @@ -7,6 +7,7 @@ > #include > > &fec { > + /delete-property/ interrupts; > interrupts-extended = <&gpio1 6 IRQ_TYPE_LEVEL_HIGH>, > <&intc 0 119 IRQ_TYPE_LEVEL_HIGH>; > fsl,err006687-workaround-present; > diff --git a/arch/arm/boot/dts/imx6qdl-ts7970.dtsi > b/arch/arm/boot/dts/imx6qdl-ts7970.dtsi index d6ba4b2a60f6..c096d25a6f5b > 100644 > --- a/arch/arm/boot/dts/imx6qdl-ts7970.dtsi > +++ b/arch/arm/boot/dts/imx6qdl-ts7970.dtsi > @@ -192,6 +192,7 @@ &fec { > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_enet>; > phy-mode = "rgmii"; > + /delete-property/ interrupts; > interrupts-extended = <&gpio1 6 IRQ_TYPE_LEVEL_HIGH>, > <&intc 0 119 IRQ_TYPE_LEVEL_HIGH>; > fsl,err006687-workaround-present; _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel