From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752336AbaKQLTx (ORCPT ); Mon, 17 Nov 2014 06:19:53 -0500 Received: from mout.kundenserver.de ([212.227.126.131]:55614 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751236AbaKQLTv (ORCPT ); Mon, 17 Nov 2014 06:19:51 -0500 From: Arnd Bergmann To: linux-arm-kernel@lists.infradead.org Cc: Ankit Jindal , linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, Varka Bhadram , Russell King - ARM Linux , Greg Kroah-Hartman , Anup Patel , Guenter Roeck , patches@apm.com, Rob Herring , "Hans J. Koch" , Tushar Jagad , Kumar Gala , Andreas =?ISO-8859-1?Q?F=E4rber?= Subject: Re: [PATCH v5 5/6] Documentation: dt-bindings: Add binding info for X-Gene QMTM UIO driver Date: Mon, 17 Nov 2014 12:17:16 +0100 Message-ID: <2279061.naADrHDgyn@wuerfel> User-Agent: KMail/4.11.5 (Linux/3.16.0-10-generic; KDE/4.11.5; x86_64; ; ) In-Reply-To: <1416220572-13381-6-git-send-email-ankit.jindal@linaro.org> References: <1416220572-13381-1-git-send-email-ankit.jindal@linaro.org> <1416220572-13381-6-git-send-email-ankit.jindal@linaro.org> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-Provags-ID: V02:K0:Hd+UhnvjslDXOzO1zL1Rtdp8qMXoJXCOquZnEQWF/m1 7DSlILSEa8p/A23o9t0rh+98GTFACOOvkZ6/4HLRCYbFOIpetD eEV435Tsv8axYPczXkSWD8ALD++ayUdrwTHv+i2/NGnrMOwY5t OcqWZnk1+2f4Ic/9dHPPdHFTd8hmZCS1KzzHiG8TA930CwUImm hWNysJ2yGuGw8+oRTNOLfgd4ZoQcBGliGWKl0g6yYyA+d16L/9 jiBzQNGGBm7rlLmimJwmxQYUgaihlZ5kaLTCkzPXQr2r9Xg7bb cNzA27LB4NmP2MAOYT0vV3iC+oNmpCLiMiNPU16CWxgjm0goUL 1Sp4iKxZ6c0lIxjNXKBw= X-UI-Out-Filterresults: notjunk:1; Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Monday 17 November 2014 16:06:11 Ankit Jindal wrote: > + > + qmtm1_uio: qmtm_uio@1f200000 { > + compatible = "apm,xgene-qmtm"; > + status = "disabled"; > + reg = <0x0 0x1f200000 0x0 0x10000>, > + <0x0 0x1b000000 0x0 0x400000>; > + reg-names = "csr", "fabric"; > + qpool-memory = <&qmtm1_uio_qpool>; > + clocks = <&qmtm1clk 0>; > + num-queues = <0x400>; > + devid = <1>; > + }; > + To make my previous review comments clearer: NAK Do not create device nodes that are meant for a specific use case in software and that are not usable for the common case. I don't think it makes any sense to keep on submitting a UIO driver for this until we have a proper network driver that uses this so we can make sure we have a working binding. Arnd From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Subject: Re: [PATCH v5 5/6] Documentation: dt-bindings: Add binding info for X-Gene QMTM UIO driver Date: Mon, 17 Nov 2014 12:17:16 +0100 Message-ID: <2279061.naADrHDgyn@wuerfel> References: <1416220572-13381-1-git-send-email-ankit.jindal@linaro.org> <1416220572-13381-6-git-send-email-ankit.jindal@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Return-path: In-Reply-To: <1416220572-13381-6-git-send-email-ankit.jindal-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org Cc: Ankit Jindal , linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Varka Bhadram , Russell King - ARM Linux , Greg Kroah-Hartman , Anup Patel , Guenter Roeck , patches-qTEPVZfXA3Y@public.gmane.org, Rob Herring , "Hans J. Koch" , Tushar Jagad , Kumar Gala , Andreas =?ISO-8859-1?Q?F=E4rber?= List-Id: devicetree@vger.kernel.org On Monday 17 November 2014 16:06:11 Ankit Jindal wrote: > + > + qmtm1_uio: qmtm_uio@1f200000 { > + compatible = "apm,xgene-qmtm"; > + status = "disabled"; > + reg = <0x0 0x1f200000 0x0 0x10000>, > + <0x0 0x1b000000 0x0 0x400000>; > + reg-names = "csr", "fabric"; > + qpool-memory = <&qmtm1_uio_qpool>; > + clocks = <&qmtm1clk 0>; > + num-queues = <0x400>; > + devid = <1>; > + }; > + To make my previous review comments clearer: NAK Do not create device nodes that are meant for a specific use case in software and that are not usable for the common case. I don't think it makes any sense to keep on submitting a UIO driver for this until we have a proper network driver that uses this so we can make sure we have a working binding. Arnd -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 From: arnd@arndb.de (Arnd Bergmann) Date: Mon, 17 Nov 2014 12:17:16 +0100 Subject: [PATCH v5 5/6] Documentation: dt-bindings: Add binding info for X-Gene QMTM UIO driver In-Reply-To: <1416220572-13381-6-git-send-email-ankit.jindal@linaro.org> References: <1416220572-13381-1-git-send-email-ankit.jindal@linaro.org> <1416220572-13381-6-git-send-email-ankit.jindal@linaro.org> Message-ID: <2279061.naADrHDgyn@wuerfel> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Monday 17 November 2014 16:06:11 Ankit Jindal wrote: > + > + qmtm1_uio: qmtm_uio at 1f200000 { > + compatible = "apm,xgene-qmtm"; > + status = "disabled"; > + reg = <0x0 0x1f200000 0x0 0x10000>, > + <0x0 0x1b000000 0x0 0x400000>; > + reg-names = "csr", "fabric"; > + qpool-memory = <&qmtm1_uio_qpool>; > + clocks = <&qmtm1clk 0>; > + num-queues = <0x400>; > + devid = <1>; > + }; > + To make my previous review comments clearer: NAK Do not create device nodes that are meant for a specific use case in software and that are not usable for the common case. I don't think it makes any sense to keep on submitting a UIO driver for this until we have a proper network driver that uses this so we can make sure we have a working binding. Arnd