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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 051D5C433F5 for ; Sat, 18 Dec 2021 15:53:48 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233523AbhLRPxp (ORCPT ); Sat, 18 Dec 2021 10:53:45 -0500 Received: from esa.microchip.iphmx.com ([68.232.153.233]:40514 "EHLO esa.microchip.iphmx.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232268AbhLRPxp (ORCPT ); Sat, 18 Dec 2021 10:53:45 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=microchip.com; i=@microchip.com; q=dns/txt; s=mchp; t=1639842825; x=1671378825; h=subject:to:cc:references:from:message-id:date: mime-version:in-reply-to:content-transfer-encoding; bh=rmaT00LHUtluk/HN/eeLu1zAv9UaHnpvB4QFRhkBa1k=; b=h89/7MGu6PPryFygOAHWVVYNyy2uRbliEh9t6xumsCG+/W2qgNEOcvye NyamSGL5+Ed8e/IHxbkFZzcB2RESR9/udsLAS7THPfd4WHEjf93U6E5En m5XaiA0LMvGBilg5bE0F0x/333a5Ny+dJmDarnmr0Vfgrv5zUTx8ziQ5j +iRnzSpJ5Ayzn0srN/VQFkeXcM3g2z4D1/PZ0znD80hXKFHUmMLP49h6t Y5IjSNUkACSK/DG9Kkpiyqz+FTS7F8gjhfkwwis7/J6mlir1HzT2POZ9y l7jgBZwSWeQwf5JWjizquNJWVIsotp0X59CM1QUpzjVYbWxgmIllmWpzx g==; IronPort-SDR: vFkae82iloPXziCbPTryS0XrPDIKbHNH7Raa2Xb47W6AMYO4Wm+79btpA6vRBq/806sRu/PBrQ DF+G/+w3UQnPknhn8XdKztFb6KX63bl95tEXnD9eLlvzRi+Fz2DIeve4+H85hBfl9paoVp56t7 jWkRfCKUlKD4/PPTiLJPdl1Q669GP+9RvB97iCzz2vjtH+mwyqXh+A7dbRvxtoVn4l20FGDmEB QJsAEB7VrtDmbbeqLEOE5jan6kldkVMOXI/oKGKggIEallWlmyzE0uMpG+3WrR2yczVMTzcIQe /SqECfJn+q9kdy+9a/KcTy/+ X-IronPort-AV: E=Sophos;i="5.88,216,1635231600"; d="scan'208";a="155977096" Received: from smtpout.microchip.com (HELO email.microchip.com) ([198.175.253.82]) by esa1.microchip.iphmx.com with ESMTP/TLS/AES256-SHA256; 18 Dec 2021 08:53:43 -0700 Received: from chn-vm-ex03.mchp-main.com (10.10.85.151) by chn-vm-ex03.mchp-main.com (10.10.85.151) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.17; Sat, 18 Dec 2021 08:53:42 -0700 Received: from [10.171.246.7] (10.10.115.15) by chn-vm-ex03.mchp-main.com (10.10.85.151) with Microsoft SMTP Server id 15.1.2375.17 via Frontend Transport; Sat, 18 Dec 2021 08:53:38 -0700 Subject: Re: [PATCH] ARM: dts: Remove "spidev" nodes To: Rob Herring , , Alexandre Belloni , Ludovic Desroches , Shawn Guo , "Sascha Hauer" , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , =?UTF-8?Q?Beno=c3=aet_Cousson?= , Tony Lindgren , Andy Gross , "Bjorn Andersson" , Viresh Kumar , Shiraz Hashim CC: Mark Brown , , , , , References: <20211217221232.3664417-1-robh@kernel.org> From: Nicolas Ferre Organization: microchip Message-ID: <748816cb-e511-8649-29fa-a55ab581246b@microchip.com> Date: Sat, 18 Dec 2021 16:53:38 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.11.0 MIME-Version: 1.0 In-Reply-To: <20211217221232.3664417-1-robh@kernel.org> Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-arm-msm@vger.kernel.org On 17/12/2021 at 23:12, Rob Herring wrote: > "spidev" is not a real device, but a Linux implementation detail. It has > never been documented either. The kernel has WARNed on the use of it for > over 6 years. Time to remove its usage from the tree. > > Cc: Mark Brown > Signed-off-by: Rob Herring > --- > Arnd, Olof, Can you please apply this directly. > > arch/arm/boot/dts/at91-q5xr5.dts | 18 ---------- > arch/arm/boot/dts/at91-wb50n.dts | 6 ---- For at91: Acked-by: Nicolas Ferre Sorry for having re-introduced this use recently. Best regards, Nicolas > arch/arm/boot/dts/imx28-tx28.dts | 36 ------------------- > arch/arm/boot/dts/imx53-tx53.dtsi | 11 ------ > arch/arm/boot/dts/imx6qdl-tx6.dtsi | 12 ------- > .../boot/dts/imx6ul-imx6ull-opos6uldev.dtsi | 12 ------- > arch/arm/boot/dts/imx6ul-prti6g.dts | 6 ---- > arch/arm/boot/dts/imx6ul-tx6ul.dtsi | 24 ------------- > arch/arm/boot/dts/omap3-tao3530.dtsi | 14 -------- > .../dts/qcom-mdm9615-wp8548-mangoh-green.dts | 2 +- > arch/arm/boot/dts/spear1310-evb.dts | 16 --------- > arch/arm/boot/dts/spear1340-evb.dts | 16 --------- > 12 files changed, 1 insertion(+), 172 deletions(-) > > diff --git a/arch/arm/boot/dts/at91-q5xr5.dts b/arch/arm/boot/dts/at91-q5xr5.dts > index 5827383b181b..47a00062f01f 100644 > --- a/arch/arm/boot/dts/at91-q5xr5.dts > +++ b/arch/arm/boot/dts/at91-q5xr5.dts > @@ -147,12 +147,6 @@ uboot@8000 { > reg = <0x8000 0x3E000>; > }; > }; > - > - spidev@1 { > - compatible = "spidev"; > - spi-max-frequency = <2000000>; > - reg = <1>; > - }; > }; > > &spi1 { > @@ -160,18 +154,6 @@ &spi1 { > pinctrl-0 = <&pinctrl_spi1 &pinctrl_spi1_npcs0 &pinctrl_spi1_npcs1>; > cs-gpios = <&pioB 3 GPIO_ACTIVE_HIGH>, <&pioC 5 GPIO_ACTIVE_LOW>, <0>, <0>; > status = "okay"; > - > - spidev@0 { > - compatible = "spidev"; > - spi-max-frequency = <2000000>; > - reg = <0>; > - }; > - > - spidev@1 { > - compatible = "spidev"; > - spi-max-frequency = <2000000>; > - reg = <1>; > - }; > }; > > &usart0 { > diff --git a/arch/arm/boot/dts/at91-wb50n.dts b/arch/arm/boot/dts/at91-wb50n.dts > index a5e45bb95c04..89f0f717f7ed 100644 > --- a/arch/arm/boot/dts/at91-wb50n.dts > +++ b/arch/arm/boot/dts/at91-wb50n.dts > @@ -90,12 +90,6 @@ &i2c0 { > > &spi1 { > status = "okay"; > - > - spidev@0 { > - compatible = "spidev"; > - reg = <0>; > - spi-max-frequency = <8000000>; > - }; > }; > > &usb0 { > diff --git a/arch/arm/boot/dts/imx28-tx28.dts b/arch/arm/boot/dts/imx28-tx28.dts > index 164254c28f8e..9e5651c7ea6b 100644 > --- a/arch/arm/boot/dts/imx28-tx28.dts > +++ b/arch/arm/boot/dts/imx28-tx28.dts > @@ -239,24 +239,6 @@ &gpio3 9 GPIO_ACTIVE_LOW > >; > /* enable this and disable ssp3 below, if you need full duplex SPI transfer */ > status = "disabled"; > - > - spi@0 { > - compatible = "spidev"; > - reg = <0>; > - spi-max-frequency = <57600000>; > - }; > - > - spi@1 { > - compatible = "spidev"; > - reg = <1>; > - spi-max-frequency = <57600000>; > - }; > - > - spi@2 { > - compatible = "spidev"; > - reg = <2>; > - spi-max-frequency = <57600000>; > - }; > }; > }; > > @@ -700,24 +682,6 @@ &ssp3 { > pinctrl-0 = <&spi3_pins_a>; > clock-frequency = <57600000>; > status = "okay"; > - > - spi@0 { > - compatible = "spidev"; > - reg = <0>; > - spi-max-frequency = <57600000>; > - }; > - > - spi@1 { > - compatible = "spidev"; > - reg = <1>; > - spi-max-frequency = <57600000>; > - }; > - > - spi@2 { > - compatible = "spidev"; > - reg = <2>; > - spi-max-frequency = <57600000>; > - }; > }; > > &usb0 { > diff --git a/arch/arm/boot/dts/imx53-tx53.dtsi b/arch/arm/boot/dts/imx53-tx53.dtsi > index 7c9730f3f820..81c2726a328a 100644 > --- a/arch/arm/boot/dts/imx53-tx53.dtsi > +++ b/arch/arm/boot/dts/imx53-tx53.dtsi > @@ -192,17 +192,6 @@ &gpio2 30 GPIO_ACTIVE_HIGH > &gpio3 19 GPIO_ACTIVE_HIGH > >; > > - spidev0: spi@0 { > - compatible = "spidev"; > - reg = <0>; > - spi-max-frequency = <54000000>; > - }; > - > - spidev1: spi@1 { > - compatible = "spidev"; > - reg = <1>; > - spi-max-frequency = <54000000>; > - }; > }; > > &esdhc1 { > diff --git a/arch/arm/boot/dts/imx6qdl-tx6.dtsi b/arch/arm/boot/dts/imx6qdl-tx6.dtsi > index 362e65ccaa78..bcc5bbcce769 100644 > --- a/arch/arm/boot/dts/imx6qdl-tx6.dtsi > +++ b/arch/arm/boot/dts/imx6qdl-tx6.dtsi > @@ -259,18 +259,6 @@ &gpio2 30 GPIO_ACTIVE_HIGH > &gpio3 19 GPIO_ACTIVE_HIGH > >; > status = "disabled"; > - > - spidev0: spi@0 { > - compatible = "spidev"; > - reg = <0>; > - spi-max-frequency = <54000000>; > - }; > - > - spidev1: spi@1 { > - compatible = "spidev"; > - reg = <1>; > - spi-max-frequency = <54000000>; > - }; > }; > > &fec { > diff --git a/arch/arm/boot/dts/imx6ul-imx6ull-opos6uldev.dtsi b/arch/arm/boot/dts/imx6ul-imx6ull-opos6uldev.dtsi > index 935a77d717a6..18cac19aa9b0 100644 > --- a/arch/arm/boot/dts/imx6ul-imx6ull-opos6uldev.dtsi > +++ b/arch/arm/boot/dts/imx6ul-imx6ull-opos6uldev.dtsi > @@ -114,18 +114,6 @@ &ecspi4 { > pinctrl-0 = <&pinctrl_ecspi4>; > cs-gpios = <&gpio4 9 GPIO_ACTIVE_LOW>, <&gpio4 3 GPIO_ACTIVE_LOW>; > status = "okay"; > - > - spidev0: spi@0 { > - compatible = "spidev"; > - reg = <0>; > - spi-max-frequency = <5000000>; > - }; > - > - spidev1: spi@1 { > - compatible = "spidev"; > - reg = <1>; > - spi-max-frequency = <5000000>; > - }; > }; > > &i2c1 { > diff --git a/arch/arm/boot/dts/imx6ul-prti6g.dts b/arch/arm/boot/dts/imx6ul-prti6g.dts > index d62015701d0a..c18390f238e1 100644 > --- a/arch/arm/boot/dts/imx6ul-prti6g.dts > +++ b/arch/arm/boot/dts/imx6ul-prti6g.dts > @@ -78,12 +78,6 @@ &ecspi2 { > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_ecspi2>; > status = "okay"; > - > - spi@0 { > - compatible = "spidev"; > - reg = <0>; > - spi-max-frequency = <1000000>; > - }; > }; > > &fec1 { > diff --git a/arch/arm/boot/dts/imx6ul-tx6ul.dtsi b/arch/arm/boot/dts/imx6ul-tx6ul.dtsi > index 938a32ced88d..c485d058e079 100644 > --- a/arch/arm/boot/dts/imx6ul-tx6ul.dtsi > +++ b/arch/arm/boot/dts/imx6ul-tx6ul.dtsi > @@ -227,18 +227,6 @@ &gpio1 29 GPIO_ACTIVE_HIGH > &gpio1 10 GPIO_ACTIVE_HIGH > >; > status = "disabled"; > - > - spi@0 { > - compatible = "spidev"; > - reg = <0>; > - spi-max-frequency = <660000>; > - }; > - > - spi@1 { > - compatible = "spidev"; > - reg = <1>; > - spi-max-frequency = <660000>; > - }; > }; > > sound { > @@ -290,18 +278,6 @@ &gpio1 29 GPIO_ACTIVE_HIGH > &gpio1 10 GPIO_ACTIVE_HIGH > >; > status = "disabled"; > - > - spidev0: spi@0 { > - compatible = "spidev"; > - reg = <0>; > - spi-max-frequency = <60000000>; > - }; > - > - spidev1: spi@1 { > - compatible = "spidev"; > - reg = <1>; > - spi-max-frequency = <60000000>; > - }; > }; > > &fec1 { > diff --git a/arch/arm/boot/dts/omap3-tao3530.dtsi b/arch/arm/boot/dts/omap3-tao3530.dtsi > index 580bfa1931c8..7f440d11f7e7 100644 > --- a/arch/arm/boot/dts/omap3-tao3530.dtsi > +++ b/arch/arm/boot/dts/omap3-tao3530.dtsi > @@ -197,25 +197,11 @@ &i2c3 { > &mcspi1 { > pinctrl-names = "default"; > pinctrl-0 = <&mcspi1_pins>; > - > - spidev@0 { > - compatible = "spidev"; > - spi-max-frequency = <48000000>; > - reg = <0>; > - spi-cpha; > - }; > }; > > &mcspi3 { > pinctrl-names = "default"; > pinctrl-0 = <&mcspi3_pins>; > - > - spidev@0 { > - compatible = "spidev"; > - spi-max-frequency = <48000000>; > - reg = <0>; > - spi-cpha; > - }; > }; > > #include "twl4030.dtsi" > diff --git a/arch/arm/boot/dts/qcom-mdm9615-wp8548-mangoh-green.dts b/arch/arm/boot/dts/qcom-mdm9615-wp8548-mangoh-green.dts > index 942e3a2cac35..0827de5426c1 100644 > --- a/arch/arm/boot/dts/qcom-mdm9615-wp8548-mangoh-green.dts > +++ b/arch/arm/boot/dts/qcom-mdm9615-wp8548-mangoh-green.dts > @@ -103,7 +103,7 @@ pins { > > &gsbi3_spi { > spi@0 { > - compatible = "swir,mangoh-iotport-spi", "spidev"; > + compatible = "swir,mangoh-iotport-spi"; > spi-max-frequency = <24000000>; > reg = <0>; > }; > diff --git a/arch/arm/boot/dts/spear1310-evb.dts b/arch/arm/boot/dts/spear1310-evb.dts > index 8fcb6be6e7c7..4cbadcb41084 100644 > --- a/arch/arm/boot/dts/spear1310-evb.dts > +++ b/arch/arm/boot/dts/spear1310-evb.dts > @@ -395,22 +395,6 @@ m25p80@1 { > pl022,wait-state = <0>; > pl022,duplex = <0>; > }; > - > - spidev@2 { > - compatible = "spidev"; > - reg = <2>; > - spi-max-frequency = <25000000>; > - spi-cpha; > - pl022,hierarchy = <0>; > - pl022,interface = <0>; > - pl022,slave-tx-disable; > - pl022,com-mode = <0x2>; > - pl022,rx-level-trig = <0>; > - pl022,tx-level-trig = <0>; > - pl022,ctrl-len = <0x11>; > - pl022,wait-state = <0>; > - pl022,duplex = <0>; > - }; > }; > > wdt@ec800620 { > diff --git a/arch/arm/boot/dts/spear1340-evb.dts b/arch/arm/boot/dts/spear1340-evb.dts > index f70ff56d4542..fd194ebeedc9 100644 > --- a/arch/arm/boot/dts/spear1340-evb.dts > +++ b/arch/arm/boot/dts/spear1340-evb.dts > @@ -489,22 +489,6 @@ stmpe_touchscreen { > ts,i-drive = <1>; > }; > }; > - > - spidev@2 { > - compatible = "spidev"; > - reg = <2>; > - spi-max-frequency = <25000000>; > - spi-cpha; > - pl022,hierarchy = <0>; > - pl022,interface = <0>; > - pl022,slave-tx-disable; > - pl022,com-mode = <0x2>; > - pl022,rx-level-trig = <0>; > - pl022,tx-level-trig = <0>; > - pl022,ctrl-len = <0x11>; > - pl022,wait-state = <0>; > - pl022,duplex = <0>; > - }; > }; > > timer@ec800600 { > -- > 2.32.0 > -- Nicolas Ferre 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 649B0C433EF for ; Sat, 18 Dec 2021 15:55:44 +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-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:Message-ID:From: References:CC:To:Subject:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=HITuxehdCD6ykWezdvzbi0EQqDBA55fhgBPXKEV3JIc=; b=uWKLVObdpllomMePD4/CywoqEi yxcIsI/cnCmeJQOlysMgnmAO3VORrS9oZFNWFp/Pe9AfEyYSUBtNXVbVAqM+zIyPpF7yFn6zv47NE VXgI9ixlj6hfYIkW5Bgv9QmbzejF6Ywrr3P/YHgevcT3jZlMw6HTWXJcRQWxEQPVHvfCIpVuBg7gZ 7oIWTxufC1Pghi0MheONCjvwj6N47uwQfu5yvIi6pPyvbh5b9d6Ft5/2YLYFhuOnkZIBlurJeXcR/ 6Y4J2mGbzFbM0Uw7yjomNrhPxjndc53VVT/EXL9eDPktNISxTnQPAxIN74/ovZs2M/BX452LMtiG3 Xmy4WRNA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1myc1u-00EB3R-HL; Sat, 18 Dec 2021 15:53:50 +0000 Received: from esa.microchip.iphmx.com ([68.232.153.233]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1myc1q-00EB2K-Bi for linux-arm-kernel@lists.infradead.org; Sat, 18 Dec 2021 15:53:48 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=microchip.com; i=@microchip.com; q=dns/txt; s=mchp; t=1639842826; x=1671378826; h=subject:to:cc:references:from:message-id:date: mime-version:in-reply-to:content-transfer-encoding; bh=rmaT00LHUtluk/HN/eeLu1zAv9UaHnpvB4QFRhkBa1k=; b=gS4xpkVqs2gI44jAHWd/rYY7A0GQnkZh52NfkxBcyJgJFmozeYDjtCeT jqY60zhGiDQlQDD/f6iES6Mm0epXqNZBJivOZWuEzBAA3mxlzohjcPM3k iVuf9CATIg5KuPCWXW7BlIIEDviw9ixUPl5C2w9ee54i59ebOu9/nhhio 4SUrH3WxPnq4ZW7gXHTKKT/JzviLOP8Fwgk+7K3HGVwK3wpdnEHysBuOS dulN5KuU19IvkVRjwLO7JthqrtH6s/TBHQeTf+/3uJSPW3rmIK/e4tIjg bGgAnGYrVkoPIYS4lJuTz/lbrx+KjyPu4WulyJnOF+pUaMTNNoreZWAVG g==; IronPort-SDR: vFkae82iloPXziCbPTryS0XrPDIKbHNH7Raa2Xb47W6AMYO4Wm+79btpA6vRBq/806sRu/PBrQ DF+G/+w3UQnPknhn8XdKztFb6KX63bl95tEXnD9eLlvzRi+Fz2DIeve4+H85hBfl9paoVp56t7 jWkRfCKUlKD4/PPTiLJPdl1Q669GP+9RvB97iCzz2vjtH+mwyqXh+A7dbRvxtoVn4l20FGDmEB QJsAEB7VrtDmbbeqLEOE5jan6kldkVMOXI/oKGKggIEallWlmyzE0uMpG+3WrR2yczVMTzcIQe /SqECfJn+q9kdy+9a/KcTy/+ X-IronPort-AV: E=Sophos;i="5.88,216,1635231600"; d="scan'208";a="155977096" Received: from smtpout.microchip.com (HELO email.microchip.com) ([198.175.253.82]) by esa1.microchip.iphmx.com with ESMTP/TLS/AES256-SHA256; 18 Dec 2021 08:53:43 -0700 Received: from chn-vm-ex03.mchp-main.com (10.10.85.151) by chn-vm-ex03.mchp-main.com (10.10.85.151) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.17; Sat, 18 Dec 2021 08:53:42 -0700 Received: from [10.171.246.7] (10.10.115.15) by chn-vm-ex03.mchp-main.com (10.10.85.151) with Microsoft SMTP Server id 15.1.2375.17 via Frontend Transport; Sat, 18 Dec 2021 08:53:38 -0700 Subject: Re: [PATCH] ARM: dts: Remove "spidev" nodes To: Rob Herring , , Alexandre Belloni , Ludovic Desroches , Shawn Guo , "Sascha Hauer" , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , =?UTF-8?Q?Beno=c3=aet_Cousson?= , Tony Lindgren , Andy Gross , "Bjorn Andersson" , Viresh Kumar , Shiraz Hashim CC: Mark Brown , , , , , References: <20211217221232.3664417-1-robh@kernel.org> From: Nicolas Ferre Organization: microchip Message-ID: <748816cb-e511-8649-29fa-a55ab581246b@microchip.com> Date: Sat, 18 Dec 2021 16:53:38 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.11.0 MIME-Version: 1.0 In-Reply-To: <20211217221232.3664417-1-robh@kernel.org> Content-Language: en-US X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20211218_075346_494103_EBCA15DF X-CRM114-Status: GOOD ( 19.16 ) 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-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 17/12/2021 at 23:12, Rob Herring wrote: > "spidev" is not a real device, but a Linux implementation detail. It has > never been documented either. The kernel has WARNed on the use of it for > over 6 years. Time to remove its usage from the tree. > > Cc: Mark Brown > Signed-off-by: Rob Herring > --- > Arnd, Olof, Can you please apply this directly. > > arch/arm/boot/dts/at91-q5xr5.dts | 18 ---------- > arch/arm/boot/dts/at91-wb50n.dts | 6 ---- For at91: Acked-by: Nicolas Ferre Sorry for having re-introduced this use recently. Best regards, Nicolas > arch/arm/boot/dts/imx28-tx28.dts | 36 ------------------- > arch/arm/boot/dts/imx53-tx53.dtsi | 11 ------ > arch/arm/boot/dts/imx6qdl-tx6.dtsi | 12 ------- > .../boot/dts/imx6ul-imx6ull-opos6uldev.dtsi | 12 ------- > arch/arm/boot/dts/imx6ul-prti6g.dts | 6 ---- > arch/arm/boot/dts/imx6ul-tx6ul.dtsi | 24 ------------- > arch/arm/boot/dts/omap3-tao3530.dtsi | 14 -------- > .../dts/qcom-mdm9615-wp8548-mangoh-green.dts | 2 +- > arch/arm/boot/dts/spear1310-evb.dts | 16 --------- > arch/arm/boot/dts/spear1340-evb.dts | 16 --------- > 12 files changed, 1 insertion(+), 172 deletions(-) > > diff --git a/arch/arm/boot/dts/at91-q5xr5.dts b/arch/arm/boot/dts/at91-q5xr5.dts > index 5827383b181b..47a00062f01f 100644 > --- a/arch/arm/boot/dts/at91-q5xr5.dts > +++ b/arch/arm/boot/dts/at91-q5xr5.dts > @@ -147,12 +147,6 @@ uboot@8000 { > reg = <0x8000 0x3E000>; > }; > }; > - > - spidev@1 { > - compatible = "spidev"; > - spi-max-frequency = <2000000>; > - reg = <1>; > - }; > }; > > &spi1 { > @@ -160,18 +154,6 @@ &spi1 { > pinctrl-0 = <&pinctrl_spi1 &pinctrl_spi1_npcs0 &pinctrl_spi1_npcs1>; > cs-gpios = <&pioB 3 GPIO_ACTIVE_HIGH>, <&pioC 5 GPIO_ACTIVE_LOW>, <0>, <0>; > status = "okay"; > - > - spidev@0 { > - compatible = "spidev"; > - spi-max-frequency = <2000000>; > - reg = <0>; > - }; > - > - spidev@1 { > - compatible = "spidev"; > - spi-max-frequency = <2000000>; > - reg = <1>; > - }; > }; > > &usart0 { > diff --git a/arch/arm/boot/dts/at91-wb50n.dts b/arch/arm/boot/dts/at91-wb50n.dts > index a5e45bb95c04..89f0f717f7ed 100644 > --- a/arch/arm/boot/dts/at91-wb50n.dts > +++ b/arch/arm/boot/dts/at91-wb50n.dts > @@ -90,12 +90,6 @@ &i2c0 { > > &spi1 { > status = "okay"; > - > - spidev@0 { > - compatible = "spidev"; > - reg = <0>; > - spi-max-frequency = <8000000>; > - }; > }; > > &usb0 { > diff --git a/arch/arm/boot/dts/imx28-tx28.dts b/arch/arm/boot/dts/imx28-tx28.dts > index 164254c28f8e..9e5651c7ea6b 100644 > --- a/arch/arm/boot/dts/imx28-tx28.dts > +++ b/arch/arm/boot/dts/imx28-tx28.dts > @@ -239,24 +239,6 @@ &gpio3 9 GPIO_ACTIVE_LOW > >; > /* enable this and disable ssp3 below, if you need full duplex SPI transfer */ > status = "disabled"; > - > - spi@0 { > - compatible = "spidev"; > - reg = <0>; > - spi-max-frequency = <57600000>; > - }; > - > - spi@1 { > - compatible = "spidev"; > - reg = <1>; > - spi-max-frequency = <57600000>; > - }; > - > - spi@2 { > - compatible = "spidev"; > - reg = <2>; > - spi-max-frequency = <57600000>; > - }; > }; > }; > > @@ -700,24 +682,6 @@ &ssp3 { > pinctrl-0 = <&spi3_pins_a>; > clock-frequency = <57600000>; > status = "okay"; > - > - spi@0 { > - compatible = "spidev"; > - reg = <0>; > - spi-max-frequency = <57600000>; > - }; > - > - spi@1 { > - compatible = "spidev"; > - reg = <1>; > - spi-max-frequency = <57600000>; > - }; > - > - spi@2 { > - compatible = "spidev"; > - reg = <2>; > - spi-max-frequency = <57600000>; > - }; > }; > > &usb0 { > diff --git a/arch/arm/boot/dts/imx53-tx53.dtsi b/arch/arm/boot/dts/imx53-tx53.dtsi > index 7c9730f3f820..81c2726a328a 100644 > --- a/arch/arm/boot/dts/imx53-tx53.dtsi > +++ b/arch/arm/boot/dts/imx53-tx53.dtsi > @@ -192,17 +192,6 @@ &gpio2 30 GPIO_ACTIVE_HIGH > &gpio3 19 GPIO_ACTIVE_HIGH > >; > > - spidev0: spi@0 { > - compatible = "spidev"; > - reg = <0>; > - spi-max-frequency = <54000000>; > - }; > - > - spidev1: spi@1 { > - compatible = "spidev"; > - reg = <1>; > - spi-max-frequency = <54000000>; > - }; > }; > > &esdhc1 { > diff --git a/arch/arm/boot/dts/imx6qdl-tx6.dtsi b/arch/arm/boot/dts/imx6qdl-tx6.dtsi > index 362e65ccaa78..bcc5bbcce769 100644 > --- a/arch/arm/boot/dts/imx6qdl-tx6.dtsi > +++ b/arch/arm/boot/dts/imx6qdl-tx6.dtsi > @@ -259,18 +259,6 @@ &gpio2 30 GPIO_ACTIVE_HIGH > &gpio3 19 GPIO_ACTIVE_HIGH > >; > status = "disabled"; > - > - spidev0: spi@0 { > - compatible = "spidev"; > - reg = <0>; > - spi-max-frequency = <54000000>; > - }; > - > - spidev1: spi@1 { > - compatible = "spidev"; > - reg = <1>; > - spi-max-frequency = <54000000>; > - }; > }; > > &fec { > diff --git a/arch/arm/boot/dts/imx6ul-imx6ull-opos6uldev.dtsi b/arch/arm/boot/dts/imx6ul-imx6ull-opos6uldev.dtsi > index 935a77d717a6..18cac19aa9b0 100644 > --- a/arch/arm/boot/dts/imx6ul-imx6ull-opos6uldev.dtsi > +++ b/arch/arm/boot/dts/imx6ul-imx6ull-opos6uldev.dtsi > @@ -114,18 +114,6 @@ &ecspi4 { > pinctrl-0 = <&pinctrl_ecspi4>; > cs-gpios = <&gpio4 9 GPIO_ACTIVE_LOW>, <&gpio4 3 GPIO_ACTIVE_LOW>; > status = "okay"; > - > - spidev0: spi@0 { > - compatible = "spidev"; > - reg = <0>; > - spi-max-frequency = <5000000>; > - }; > - > - spidev1: spi@1 { > - compatible = "spidev"; > - reg = <1>; > - spi-max-frequency = <5000000>; > - }; > }; > > &i2c1 { > diff --git a/arch/arm/boot/dts/imx6ul-prti6g.dts b/arch/arm/boot/dts/imx6ul-prti6g.dts > index d62015701d0a..c18390f238e1 100644 > --- a/arch/arm/boot/dts/imx6ul-prti6g.dts > +++ b/arch/arm/boot/dts/imx6ul-prti6g.dts > @@ -78,12 +78,6 @@ &ecspi2 { > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_ecspi2>; > status = "okay"; > - > - spi@0 { > - compatible = "spidev"; > - reg = <0>; > - spi-max-frequency = <1000000>; > - }; > }; > > &fec1 { > diff --git a/arch/arm/boot/dts/imx6ul-tx6ul.dtsi b/arch/arm/boot/dts/imx6ul-tx6ul.dtsi > index 938a32ced88d..c485d058e079 100644 > --- a/arch/arm/boot/dts/imx6ul-tx6ul.dtsi > +++ b/arch/arm/boot/dts/imx6ul-tx6ul.dtsi > @@ -227,18 +227,6 @@ &gpio1 29 GPIO_ACTIVE_HIGH > &gpio1 10 GPIO_ACTIVE_HIGH > >; > status = "disabled"; > - > - spi@0 { > - compatible = "spidev"; > - reg = <0>; > - spi-max-frequency = <660000>; > - }; > - > - spi@1 { > - compatible = "spidev"; > - reg = <1>; > - spi-max-frequency = <660000>; > - }; > }; > > sound { > @@ -290,18 +278,6 @@ &gpio1 29 GPIO_ACTIVE_HIGH > &gpio1 10 GPIO_ACTIVE_HIGH > >; > status = "disabled"; > - > - spidev0: spi@0 { > - compatible = "spidev"; > - reg = <0>; > - spi-max-frequency = <60000000>; > - }; > - > - spidev1: spi@1 { > - compatible = "spidev"; > - reg = <1>; > - spi-max-frequency = <60000000>; > - }; > }; > > &fec1 { > diff --git a/arch/arm/boot/dts/omap3-tao3530.dtsi b/arch/arm/boot/dts/omap3-tao3530.dtsi > index 580bfa1931c8..7f440d11f7e7 100644 > --- a/arch/arm/boot/dts/omap3-tao3530.dtsi > +++ b/arch/arm/boot/dts/omap3-tao3530.dtsi > @@ -197,25 +197,11 @@ &i2c3 { > &mcspi1 { > pinctrl-names = "default"; > pinctrl-0 = <&mcspi1_pins>; > - > - spidev@0 { > - compatible = "spidev"; > - spi-max-frequency = <48000000>; > - reg = <0>; > - spi-cpha; > - }; > }; > > &mcspi3 { > pinctrl-names = "default"; > pinctrl-0 = <&mcspi3_pins>; > - > - spidev@0 { > - compatible = "spidev"; > - spi-max-frequency = <48000000>; > - reg = <0>; > - spi-cpha; > - }; > }; > > #include "twl4030.dtsi" > diff --git a/arch/arm/boot/dts/qcom-mdm9615-wp8548-mangoh-green.dts b/arch/arm/boot/dts/qcom-mdm9615-wp8548-mangoh-green.dts > index 942e3a2cac35..0827de5426c1 100644 > --- a/arch/arm/boot/dts/qcom-mdm9615-wp8548-mangoh-green.dts > +++ b/arch/arm/boot/dts/qcom-mdm9615-wp8548-mangoh-green.dts > @@ -103,7 +103,7 @@ pins { > > &gsbi3_spi { > spi@0 { > - compatible = "swir,mangoh-iotport-spi", "spidev"; > + compatible = "swir,mangoh-iotport-spi"; > spi-max-frequency = <24000000>; > reg = <0>; > }; > diff --git a/arch/arm/boot/dts/spear1310-evb.dts b/arch/arm/boot/dts/spear1310-evb.dts > index 8fcb6be6e7c7..4cbadcb41084 100644 > --- a/arch/arm/boot/dts/spear1310-evb.dts > +++ b/arch/arm/boot/dts/spear1310-evb.dts > @@ -395,22 +395,6 @@ m25p80@1 { > pl022,wait-state = <0>; > pl022,duplex = <0>; > }; > - > - spidev@2 { > - compatible = "spidev"; > - reg = <2>; > - spi-max-frequency = <25000000>; > - spi-cpha; > - pl022,hierarchy = <0>; > - pl022,interface = <0>; > - pl022,slave-tx-disable; > - pl022,com-mode = <0x2>; > - pl022,rx-level-trig = <0>; > - pl022,tx-level-trig = <0>; > - pl022,ctrl-len = <0x11>; > - pl022,wait-state = <0>; > - pl022,duplex = <0>; > - }; > }; > > wdt@ec800620 { > diff --git a/arch/arm/boot/dts/spear1340-evb.dts b/arch/arm/boot/dts/spear1340-evb.dts > index f70ff56d4542..fd194ebeedc9 100644 > --- a/arch/arm/boot/dts/spear1340-evb.dts > +++ b/arch/arm/boot/dts/spear1340-evb.dts > @@ -489,22 +489,6 @@ stmpe_touchscreen { > ts,i-drive = <1>; > }; > }; > - > - spidev@2 { > - compatible = "spidev"; > - reg = <2>; > - spi-max-frequency = <25000000>; > - spi-cpha; > - pl022,hierarchy = <0>; > - pl022,interface = <0>; > - pl022,slave-tx-disable; > - pl022,com-mode = <0x2>; > - pl022,rx-level-trig = <0>; > - pl022,tx-level-trig = <0>; > - pl022,ctrl-len = <0x11>; > - pl022,wait-state = <0>; > - pl022,duplex = <0>; > - }; > }; > > timer@ec800600 { > -- > 2.32.0 > -- Nicolas Ferre _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel