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 X-Spam-Level: X-Spam-Status: No, score=-11.5 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 152E3C433E8 for ; Tue, 28 Jul 2020 14:31:12 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 0012920663 for ; Tue, 28 Jul 2020 14:31:11 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1730417AbgG1ObK (ORCPT ); Tue, 28 Jul 2020 10:31:10 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:35852 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729410AbgG1ObK (ORCPT ); Tue, 28 Jul 2020 10:31:10 -0400 Received: from metis.ext.pengutronix.de (metis.ext.pengutronix.de [IPv6:2001:67c:670:201:290:27ff:fe1d:cc33]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 59547C061794 for ; Tue, 28 Jul 2020 07:31:10 -0700 (PDT) Received: from pty.hi.pengutronix.de ([2001:67c:670:100:1d::c5]) by metis.ext.pengutronix.de with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1k0QdH-0005d1-7u; Tue, 28 Jul 2020 16:31:07 +0200 Received: from mfe by pty.hi.pengutronix.de with local (Exim 4.89) (envelope-from ) id 1k0QdG-0004du-1g; Tue, 28 Jul 2020 16:31:06 +0200 Date: Tue, 28 Jul 2020 16:31:06 +0200 From: Marco Felsch To: Matthias Schiffer Cc: Shawn Guo , Sascha Hauer , linux-kernel@vger.kernel.org, NXP Linux Team , Pengutronix Kernel Team , Fabio Estevam , linux-arm-kernel@lists.infradead.org Subject: Re: (EXT) Re: [PATCH] arm: dts: imx7: add QSPI Message-ID: <20200728143105.ckc55z6j64e5hiun@pengutronix.de> References: <20200728112814.14158-1-matthias.schiffer@ew.tq-group.com> <20200728135155.aqz3xkcwpgqmvzro@pengutronix.de> <8ea0b3f655b60be02384f22c0aa4dfee9d788d71.camel@ew.tq-group.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <8ea0b3f655b60be02384f22c0aa4dfee9d788d71.camel@ew.tq-group.com> X-Sent-From: Pengutronix Hildesheim X-URL: http://www.pengutronix.de/ X-IRC: #ptxdist @freenode X-Accept-Language: de,en X-Accept-Content-Type: text/plain X-Uptime: 16:22:55 up 256 days, 5:41, 241 users, load average: 0.08, 0.05, 0.07 User-Agent: NeoMutt/20170113 (1.7.2) X-SA-Exim-Connect-IP: 2001:67c:670:100:1d::c5 X-SA-Exim-Mail-From: mfe@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: linux-kernel@vger.kernel.org Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 20-07-28 16:05, Matthias Schiffer wrote: > On Tue, 2020-07-28 at 15:51 +0200, Marco Felsch wrote: > > Hi Matthias, > > > > thanks for the patch. > > > > On 20-07-28 13:28, Matthias Schiffer wrote: > > > In preparation for an update of the TQ-Systems TQMa7x/MBa7x DTS, > > > add the > > > QSPI controller to imx7s.dtsi. > > > > > > Based-on-patch-by: Han Xu > > > Signed-off-by: Matthias Schiffer > > > > > > --- > > > arch/arm/boot/dts/imx7s.dtsi | 13 +++++++++++++ > > > 1 file changed, 13 insertions(+) > > > > > > diff --git a/arch/arm/boot/dts/imx7s.dtsi > > > b/arch/arm/boot/dts/imx7s.dtsi > > > index 1cfaf410aa43..e45683e61593 100644 > > > --- a/arch/arm/boot/dts/imx7s.dtsi > > > +++ b/arch/arm/boot/dts/imx7s.dtsi > > > @@ -1162,6 +1162,19 @@ > > > status = "disabled"; > > > }; > > > > > > + qspi1: spi@30bb0000 { > > > > Are there more controllers and why not using "qspi@30bb0000" as node > > name? > > The vast majority of QSPI controllers use spi@ node names, qspi@ only > appears in a single example in Documentation/devicetree/bindings/, and > in no actual DTS(I) files. IMHO using spi as node name is incorrect because this inherits the assumption to connect 'normal' spi devices to these controllers. But this is absolutly not the case since in most cases the qspi controllers are state-machines optimized for qspi memory devices. So all dts(i) files using spi as node name are incorrect IMHO. But we can't change that due to the backward compability. > There is only one controller. The label "qspi1" is chosen as this has > been in use in the linux-imx vendor kernels for years; IMHO, switching > to "qspi" would just cause unnecessary churn for dependent device > trees. I have no strong opinions on this though. Why? There is no _mainline_ imx7s devicetree using this node currently. I would drop the number since we have only one instance. Regards, Marco > > > + #address-cells = <1>; > > > + #size-cells = <0>; > > > + compatible = "fsl,imx7d-qspi"; > > > + reg = <0x30bb0000 0x10000>, <0x60000000 > > > 0x10000000>; > > > + reg-names = "QuadSPI", "QuadSPI- > > > memory"; > > > > The node should begin with compatible, reg, reg-names properties. > > Pls check the current .dtsi file for examples. > > Thanks, will fix. > > > > > > + interrupts = > > IRQ_TYPE_LEVEL_HIGH>; > > > + clocks = <&clks IMX7D_QSPI_ROOT_CLK>, > > > + <&clks IMX7D_QSPI_ROOT_CLK>; > > > + clock-names = "qspi_en", "qspi"; > > > + status = "disabled"; > > > + }; > > > + > > > sdma: sdma@30bd0000 { > > > compatible = "fsl,imx7d-sdma", > > > "fsl,imx35-sdma"; > > > reg = <0x30bd0000 0x10000>; > > > > Regards, > > Marco > > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | 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 X-Spam-Level: X-Spam-Status: No, score=-11.5 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 128CBC433E4 for ; Tue, 28 Jul 2020 14:32:26 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id D210D20663 for ; Tue, 28 Jul 2020 14:32:25 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="hyOTB9Of" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org D210D20663 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=pengutronix.de Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:Message-ID: Subject:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=vIhuAcCwUAB/0fxr4tRTqzd7u5ZVrqOWQ9YA9x13+5s=; b=hyOTB9OfjQ6OqsdtwDzw+Sk7h 1q6DSgnsBiXEnvYGmKfQXL7EcGwNbW2egjjzlvjmxOrU7iRll+fpU6yiCVkv5HeHBW7rAsWzJp7md mNji0im6IxomaTqJRn4hTn4uUFXFkuvdwkeIUhNwjVnVxkodlHjRuE/LkSgUqRD5fMZ72DCbgTN9B PLBWL5hltf/DN+uWTS9Cg9f8CBXLcydbXa3tPt91imwPP/9OTKSOrNI34Cg2pO4544YWmPIG+RjCd 8hFCCTTNfe1fiQV/18e0rDly7BIM3PIjWHCVjAVW6pnfzByYDYO2JHbGaL7u5jrNbQgyXsj3bMOAG +taEYFSwQ==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1k0QdO-0000tt-GU; Tue, 28 Jul 2020 14:31:14 +0000 Received: from metis.ext.pengutronix.de ([2001:67c:670:201:290:27ff:fe1d:cc33]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1k0QdL-0000sW-IQ for linux-arm-kernel@lists.infradead.org; Tue, 28 Jul 2020 14:31:12 +0000 Received: from pty.hi.pengutronix.de ([2001:67c:670:100:1d::c5]) by metis.ext.pengutronix.de with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1k0QdH-0005d1-7u; Tue, 28 Jul 2020 16:31:07 +0200 Received: from mfe by pty.hi.pengutronix.de with local (Exim 4.89) (envelope-from ) id 1k0QdG-0004du-1g; Tue, 28 Jul 2020 16:31:06 +0200 Date: Tue, 28 Jul 2020 16:31:06 +0200 From: Marco Felsch To: Matthias Schiffer Subject: Re: (EXT) Re: [PATCH] arm: dts: imx7: add QSPI Message-ID: <20200728143105.ckc55z6j64e5hiun@pengutronix.de> References: <20200728112814.14158-1-matthias.schiffer@ew.tq-group.com> <20200728135155.aqz3xkcwpgqmvzro@pengutronix.de> <8ea0b3f655b60be02384f22c0aa4dfee9d788d71.camel@ew.tq-group.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <8ea0b3f655b60be02384f22c0aa4dfee9d788d71.camel@ew.tq-group.com> X-Sent-From: Pengutronix Hildesheim X-URL: http://www.pengutronix.de/ X-IRC: #ptxdist @freenode X-Accept-Language: de,en X-Accept-Content-Type: text/plain X-Uptime: 16:22:55 up 256 days, 5:41, 241 users, load average: 0.08, 0.05, 0.07 User-Agent: NeoMutt/20170113 (1.7.2) X-SA-Exim-Connect-IP: 2001:67c:670:100:1d::c5 X-SA-Exim-Mail-From: mfe@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: linux-arm-kernel@lists.infradead.org X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200728_103111_623764_06BDE7CF X-CRM114-Status: GOOD ( 28.02 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Shawn Guo , Sascha Hauer , linux-kernel@vger.kernel.org, NXP Linux Team , Pengutronix Kernel Team , Fabio Estevam , linux-arm-kernel@lists.infradead.org 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 On 20-07-28 16:05, Matthias Schiffer wrote: > On Tue, 2020-07-28 at 15:51 +0200, Marco Felsch wrote: > > Hi Matthias, > > > > thanks for the patch. > > > > On 20-07-28 13:28, Matthias Schiffer wrote: > > > In preparation for an update of the TQ-Systems TQMa7x/MBa7x DTS, > > > add the > > > QSPI controller to imx7s.dtsi. > > > > > > Based-on-patch-by: Han Xu > > > Signed-off-by: Matthias Schiffer > > > > > > --- > > > arch/arm/boot/dts/imx7s.dtsi | 13 +++++++++++++ > > > 1 file changed, 13 insertions(+) > > > > > > diff --git a/arch/arm/boot/dts/imx7s.dtsi > > > b/arch/arm/boot/dts/imx7s.dtsi > > > index 1cfaf410aa43..e45683e61593 100644 > > > --- a/arch/arm/boot/dts/imx7s.dtsi > > > +++ b/arch/arm/boot/dts/imx7s.dtsi > > > @@ -1162,6 +1162,19 @@ > > > status = "disabled"; > > > }; > > > > > > + qspi1: spi@30bb0000 { > > > > Are there more controllers and why not using "qspi@30bb0000" as node > > name? > > The vast majority of QSPI controllers use spi@ node names, qspi@ only > appears in a single example in Documentation/devicetree/bindings/, and > in no actual DTS(I) files. IMHO using spi as node name is incorrect because this inherits the assumption to connect 'normal' spi devices to these controllers. But this is absolutly not the case since in most cases the qspi controllers are state-machines optimized for qspi memory devices. So all dts(i) files using spi as node name are incorrect IMHO. But we can't change that due to the backward compability. > There is only one controller. The label "qspi1" is chosen as this has > been in use in the linux-imx vendor kernels for years; IMHO, switching > to "qspi" would just cause unnecessary churn for dependent device > trees. I have no strong opinions on this though. Why? There is no _mainline_ imx7s devicetree using this node currently. I would drop the number since we have only one instance. Regards, Marco > > > + #address-cells = <1>; > > > + #size-cells = <0>; > > > + compatible = "fsl,imx7d-qspi"; > > > + reg = <0x30bb0000 0x10000>, <0x60000000 > > > 0x10000000>; > > > + reg-names = "QuadSPI", "QuadSPI- > > > memory"; > > > > The node should begin with compatible, reg, reg-names properties. > > Pls check the current .dtsi file for examples. > > Thanks, will fix. > > > > > > + interrupts = > > IRQ_TYPE_LEVEL_HIGH>; > > > + clocks = <&clks IMX7D_QSPI_ROOT_CLK>, > > > + <&clks IMX7D_QSPI_ROOT_CLK>; > > > + clock-names = "qspi_en", "qspi"; > > > + status = "disabled"; > > > + }; > > > + > > > sdma: sdma@30bd0000 { > > > compatible = "fsl,imx7d-sdma", > > > "fsl,imx35-sdma"; > > > reg = <0x30bd0000 0x10000>; > > > > Regards, > > Marco > > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel