Received: by 2002:a25:7ec1:0:0:0:0:0 with SMTP id z184csp560079ybc; Fri, 22 Nov 2019 09:24:42 -0800 (PST) X-Google-Smtp-Source: APXvYqwuiGsz8R/Fc/L4xruNG8N3IiTPg06YeV9tKb3igdyyXSXhmXAoo4LBOS4U0eLhgXrP4bvu X-Received: by 2002:a17:906:fcdb:: with SMTP id qx27mr23215396ejb.255.1574443481965; Fri, 22 Nov 2019 09:24:41 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1574443481; cv=none; d=google.com; s=arc-20160816; b=qyg9LJd7xZtgdQMKmDjQmzni92DS2oYQZ8tm9zZ+IjTqex2+FY4ccmexmo+wMylngp id9FEFzv2Jytqxt92XioHjC33MsWDrs7R2khhT7VLD/elKgGeRoH9dtkfVVkgNlDu3ot d7RNs/PqqpXBRledrCxvh6bvFoUqv1w5c7XXrTA5USkd7q7Pa+SBCRZBKKvjrmiifOuy tv1PGAuVs268Ig3y29SNJ3IDKpLJF+aUnCg5F4Au7wgvjxX3lyumbhQde3mzuM3rdw2b iiiEHaZegb+RobTyMCpNFrVbZWZrMXQiUAz8fnBzuoVXRP4jpzJ/9ObSysgEamMXvXcy 5QrQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding :content-language:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject; bh=c0i5Ln0JPH9846DttaY9Nj73cX/KD39/VrAsMzTXVOQ=; b=HhSN1IxN0+cG6NtYzcEFmA6cZICFXuprw0g9g0ZIq69rrfDLYbs4QEHs72+vMWZ1UV DwcRq4mklV4awe0smcevSMTbxYMQ8mpnUkeTSYl9f6iQ76MiqX3eOlqeGR4H2GPjbNDi Ng0D1Ya8H57M+FpvZL4zhF2Dsj6//SaAnQBi86lyhzldISoHH5mnzqd5VXQIAEdhBqgO S5u4ILSAuRNh5Y3pO8JaL6dHRSBq+2Hh7jbbkeyIqWyfKNrrtItHzALPub7WwdThl0cS L8GAO82M2Pt9eSZZ66CLBMSeDRgvRDuZvsGwVSb0JrfpFSHJ8j/nPz4vbNh2WcJDOpzq V8uw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id v24si2325584edm.399.2019.11.22.09.24.17; Fri, 22 Nov 2019 09:24:41 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727425AbfKVRWt (ORCPT + 99 others); Fri, 22 Nov 2019 12:22:49 -0500 Received: from foss.arm.com ([217.140.110.172]:50202 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726046AbfKVRWs (ORCPT ); Fri, 22 Nov 2019 12:22:48 -0500 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 901CFDA7; Fri, 22 Nov 2019 09:22:46 -0800 (PST) Received: from [10.1.196.37] (e121345-lin.cambridge.arm.com [10.1.196.37]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id C19063F6C4; Fri, 22 Nov 2019 09:22:44 -0800 (PST) Subject: Re: [PATCH 3/3] ARM: dts: at91: Remove the USB EP child node To: Gregory CLEMENT , Cristian Birsan , Felipe Balbi , Greg Kroah-Hartman , Alan Stern , linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org Cc: devicetree@vger.kernel.org, Alexandre Belloni , Ludovic Desroches , Rob Herring , Thomas Petazzoni , linux-arm-kernel@lists.infradead.org References: <20191107153128.11038-1-gregory.clement@bootlin.com> <20191107153128.11038-4-gregory.clement@bootlin.com> From: Robin Murphy Message-ID: Date: Fri, 22 Nov 2019 17:22:43 +0000 User-Agent: Mozilla/5.0 (X11; Linux aarch64; rv:60.0) Gecko/20100101 Thunderbird/60.9.0 MIME-Version: 1.0 In-Reply-To: <20191107153128.11038-4-gregory.clement@bootlin.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-GB Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 07/11/2019 3:31 pm, Gregory CLEMENT wrote: > The endpoint configuration used to be stored in the device tree, > however the configuration depend on the "version" of the controller > itself. > > Then the EP child node are useless and describe as deprecated in the > documentation binding: remove all the nodes from the SoC device tree > file. Just as a drive-by comment, it's presumably worth getting rid of the #address-cells and #size-cells properties too (here and in the binding example). Robin. > Signed-off-by: Gregory CLEMENT > --- > arch/arm/boot/dts/at91sam9g45.dtsi | 52 ------------- > arch/arm/boot/dts/at91sam9rl.dtsi | 52 ------------- > arch/arm/boot/dts/at91sam9x5.dtsi | 52 ------------- > arch/arm/boot/dts/sama5d2.dtsi | 118 ----------------------------- > arch/arm/boot/dts/sama5d3.dtsi | 105 ------------------------- > arch/arm/boot/dts/sama5d4.dtsi | 118 ----------------------------- > 6 files changed, 497 deletions(-) > > diff --git a/arch/arm/boot/dts/at91sam9g45.dtsi b/arch/arm/boot/dts/at91sam9g45.dtsi > index 691c95ea6175..63bfe546cd8d 100644 > --- a/arch/arm/boot/dts/at91sam9g45.dtsi > +++ b/arch/arm/boot/dts/at91sam9g45.dtsi > @@ -1204,58 +1204,6 @@ > clocks = <&udphs_clk>, <&utmi>; > clock-names = "pclk", "hclk"; > status = "disabled"; > - > - ep@0 { > - reg = <0>; > - atmel,fifo-size = <64>; > - atmel,nb-banks = <1>; > - }; > - > - ep@1 { > - reg = <1>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-dma; > - atmel,can-isoc; > - }; > - > - ep@2 { > - reg = <2>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-dma; > - atmel,can-isoc; > - }; > - > - ep@3 { > - reg = <3>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <3>; > - atmel,can-dma; > - }; > - > - ep@4 { > - reg = <4>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <3>; > - atmel,can-dma; > - }; > - > - ep@5 { > - reg = <5>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <3>; > - atmel,can-dma; > - atmel,can-isoc; > - }; > - > - ep@6 { > - reg = <6>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <3>; > - atmel,can-dma; > - atmel,can-isoc; > - }; > }; > > clk32k: sckc@fffffd50 { > diff --git a/arch/arm/boot/dts/at91sam9rl.dtsi b/arch/arm/boot/dts/at91sam9rl.dtsi > index 8643b7151565..e118bacb7d7c 100644 > --- a/arch/arm/boot/dts/at91sam9rl.dtsi > +++ b/arch/arm/boot/dts/at91sam9rl.dtsi > @@ -308,58 +308,6 @@ > clocks = <&pmc PMC_TYPE_PERIPHERAL 22>, <&pmc PMC_TYPE_CORE PMC_UTMI>; > clock-names = "pclk", "hclk"; > status = "disabled"; > - > - ep@0 { > - reg = <0>; > - atmel,fifo-size = <64>; > - atmel,nb-banks = <1>; > - }; > - > - ep@1 { > - reg = <1>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-dma; > - atmel,can-isoc; > - }; > - > - ep@2 { > - reg = <2>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-dma; > - atmel,can-isoc; > - }; > - > - ep@3 { > - reg = <3>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <3>; > - atmel,can-dma; > - }; > - > - ep@4 { > - reg = <4>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <3>; > - atmel,can-dma; > - }; > - > - ep@5 { > - reg = <5>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <3>; > - atmel,can-dma; > - atmel,can-isoc; > - }; > - > - ep@6 { > - reg = <6>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <3>; > - atmel,can-dma; > - atmel,can-isoc; > - }; > }; > > dma0: dma-controller@ffffe600 { > diff --git a/arch/arm/boot/dts/at91sam9x5.dtsi b/arch/arm/boot/dts/at91sam9x5.dtsi > index 7c2eb93f8cac..685a1b9f3ae5 100644 > --- a/arch/arm/boot/dts/at91sam9x5.dtsi > +++ b/arch/arm/boot/dts/at91sam9x5.dtsi > @@ -876,58 +876,6 @@ > clocks = <&pmc PMC_TYPE_CORE PMC_UTMI>, <&pmc PMC_TYPE_PERIPHERAL 23>; > clock-names = "hclk", "pclk"; > status = "disabled"; > - > - ep@0 { > - reg = <0>; > - atmel,fifo-size = <64>; > - atmel,nb-banks = <1>; > - }; > - > - ep@1 { > - reg = <1>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-dma; > - atmel,can-isoc; > - }; > - > - ep@2 { > - reg = <2>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-dma; > - atmel,can-isoc; > - }; > - > - ep@3 { > - reg = <3>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <3>; > - atmel,can-dma; > - }; > - > - ep@4 { > - reg = <4>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <3>; > - atmel,can-dma; > - }; > - > - ep@5 { > - reg = <5>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <3>; > - atmel,can-dma; > - atmel,can-isoc; > - }; > - > - ep@6 { > - reg = <6>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <3>; > - atmel,can-dma; > - atmel,can-isoc; > - }; > }; > > watchdog: watchdog@fffffe40 { > diff --git a/arch/arm/boot/dts/sama5d2.dtsi b/arch/arm/boot/dts/sama5d2.dtsi > index 2e2c1a7b1d1d..daafcffbe033 100644 > --- a/arch/arm/boot/dts/sama5d2.dtsi > +++ b/arch/arm/boot/dts/sama5d2.dtsi > @@ -122,124 +122,6 @@ > clocks = <&pmc PMC_TYPE_PERIPHERAL 42>, <&pmc PMC_TYPE_CORE PMC_UTMI>; > clock-names = "pclk", "hclk"; > status = "disabled"; > - > - ep@0 { > - reg = <0>; > - atmel,fifo-size = <64>; > - atmel,nb-banks = <1>; > - }; > - > - ep@1 { > - reg = <1>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <3>; > - atmel,can-dma; > - atmel,can-isoc; > - }; > - > - ep@2 { > - reg = <2>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <3>; > - atmel,can-dma; > - atmel,can-isoc; > - }; > - > - ep@3 { > - reg = <3>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-dma; > - atmel,can-isoc; > - }; > - > - ep@4 { > - reg = <4>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-dma; > - atmel,can-isoc; > - }; > - > - ep@5 { > - reg = <5>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-dma; > - atmel,can-isoc; > - }; > - > - ep@6 { > - reg = <6>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-dma; > - atmel,can-isoc; > - }; > - > - ep@7 { > - reg = <7>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-dma; > - atmel,can-isoc; > - }; > - > - ep@8 { > - reg = <8>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-isoc; > - }; > - > - ep@9 { > - reg = <9>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-isoc; > - }; > - > - ep@10 { > - reg = <10>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-isoc; > - }; > - > - ep@11 { > - reg = <11>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-isoc; > - }; > - > - ep@12 { > - reg = <12>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-isoc; > - }; > - > - ep@13 { > - reg = <13>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-isoc; > - }; > - > - ep@14 { > - reg = <14>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-isoc; > - }; > - > - ep@15 { > - reg = <15>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-isoc; > - }; > }; > > usb1: ohci@400000 { > diff --git a/arch/arm/boot/dts/sama5d3.dtsi b/arch/arm/boot/dts/sama5d3.dtsi > index f770aace0efd..dfd095f33f95 100644 > --- a/arch/arm/boot/dts/sama5d3.dtsi > +++ b/arch/arm/boot/dts/sama5d3.dtsi > @@ -1402,111 +1402,6 @@ > clocks = <&udphs_clk>, <&utmi>; > clock-names = "pclk", "hclk"; > status = "disabled"; > - > - ep@0 { > - reg = <0>; > - atmel,fifo-size = <64>; > - atmel,nb-banks = <1>; > - }; > - > - ep@1 { > - reg = <1>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <3>; > - atmel,can-dma; > - atmel,can-isoc; > - }; > - > - ep@2 { > - reg = <2>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <3>; > - atmel,can-dma; > - atmel,can-isoc; > - }; > - > - ep@3 { > - reg = <3>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-dma; > - }; > - > - ep@4 { > - reg = <4>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-dma; > - }; > - > - ep@5 { > - reg = <5>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-dma; > - }; > - > - ep@6 { > - reg = <6>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-dma; > - }; > - > - ep@7 { > - reg = <7>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-dma; > - }; > - > - ep@8 { > - reg = <8>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - }; > - > - ep@9 { > - reg = <9>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - }; > - > - ep@10 { > - reg = <10>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - }; > - > - ep@11 { > - reg = <11>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - }; > - > - ep@12 { > - reg = <12>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - }; > - > - ep@13 { > - reg = <13>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - }; > - > - ep@14 { > - reg = <14>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - }; > - > - ep@15 { > - reg = <15>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - }; > }; > > usb1: ohci@600000 { > diff --git a/arch/arm/boot/dts/sama5d4.dtsi b/arch/arm/boot/dts/sama5d4.dtsi > index 6ab27a7b388d..0ece6b22d287 100644 > --- a/arch/arm/boot/dts/sama5d4.dtsi > +++ b/arch/arm/boot/dts/sama5d4.dtsi > @@ -105,124 +105,6 @@ > clocks = <&pmc PMC_TYPE_PERIPHERAL 47>, <&pmc PMC_TYPE_CORE PMC_UTMI>; > clock-names = "pclk", "hclk"; > status = "disabled"; > - > - ep@0 { > - reg = <0>; > - atmel,fifo-size = <64>; > - atmel,nb-banks = <1>; > - }; > - > - ep@1 { > - reg = <1>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <3>; > - atmel,can-dma; > - atmel,can-isoc; > - }; > - > - ep@2 { > - reg = <2>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <3>; > - atmel,can-dma; > - atmel,can-isoc; > - }; > - > - ep@3 { > - reg = <3>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-dma; > - atmel,can-isoc; > - }; > - > - ep@4 { > - reg = <4>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-dma; > - atmel,can-isoc; > - }; > - > - ep@5 { > - reg = <5>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-dma; > - atmel,can-isoc; > - }; > - > - ep@6 { > - reg = <6>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-dma; > - atmel,can-isoc; > - }; > - > - ep@7 { > - reg = <7>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-dma; > - atmel,can-isoc; > - }; > - > - ep@8 { > - reg = <8>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-isoc; > - }; > - > - ep@9 { > - reg = <9>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-isoc; > - }; > - > - ep@10 { > - reg = <10>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-isoc; > - }; > - > - ep@11 { > - reg = <11>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-isoc; > - }; > - > - ep@12 { > - reg = <12>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-isoc; > - }; > - > - ep@13 { > - reg = <13>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-isoc; > - }; > - > - ep@14 { > - reg = <14>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-isoc; > - }; > - > - ep@15 { > - reg = <15>; > - atmel,fifo-size = <1024>; > - atmel,nb-banks = <2>; > - atmel,can-isoc; > - }; > }; > > usb1: ohci@500000 { >