From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from metis.ext.pengutronix.de ([2001:67c:670:201:290:27ff:fe1d:cc33]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1goE6I-00067A-2J for barebox@lists.infradead.org; Mon, 28 Jan 2019 21:05:51 +0000 Message-ID: <74d4ef0a05b994ae85b83d2206aea431808424aa.camel@pengutronix.de> From: Lucas Stach Date: Mon, 28 Jan 2019 22:05:46 +0100 In-Reply-To: <20190126012926.9236-2-andrew.smirnov@gmail.com> References: <20190126012926.9236-1-andrew.smirnov@gmail.com> <20190126012926.9236-2-andrew.smirnov@gmail.com> Mime-Version: 1.0 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "barebox" Errors-To: barebox-bounces+u.kleine-koenig=pengutronix.de@lists.infradead.org Subject: Re: [PATCH 2/2] ARM: i.MX: Add support for ZII i.MX8MQ based devices To: Andrey Smirnov , barebox@lists.infradead.org Hi Andrey, two very high-level comments that you might choose to ignore, but I want to get them out regardless. Am Freitag, den 25.01.2019, 17:29 -0800 schrieb Andrey Smirnov: > Add support for the following ZII i.MX8MQ based boards: > > - ZII i.MX8MQ RMB3 > - ZII i.MX8MQ Zest > > Most of the basic peripherals are supported by this patch. More > advanced features such as PCIe, display support, etc, are planned to > be added later. > > Signed-off-by: Andrey Smirnov > --- > arch/arm/boards/Makefile | 1 + > arch/arm/boards/zii-imx8mq-dev/Makefile | 2 + > arch/arm/boards/zii-imx8mq-dev/board.c | 24 + > arch/arm/boards/zii-imx8mq-dev/ddr.h | 29 + > arch/arm/boards/zii-imx8mq-dev/ddr_init.c | 225 +++++ > arch/arm/boards/zii-imx8mq-dev/ddrphy_train.c | 935 ++++++++++++++++++ > .../flash-header-zii-imx8mq-dev.imxcfg | 5 + > arch/arm/boards/zii-imx8mq-dev/lowlevel.c | 211 ++++ Why is the directory called -dev? I expect that we will support the final production versions with the same code, so I would drop this suffix. > arch/arm/dts/Makefile | 3 + > arch/arm/dts/imx8mq-zii-ultra-rmb3.dts | 39 + > arch/arm/dts/imx8mq-zii-ultra-zest.dts | 13 + > arch/arm/dts/imx8mq-zii-ultra.dtsi | 453 +++++++++ Somewhat along the same lines, I not really sure if we should use marketing names for the DTs. My experience from other projects is that it's much easier to use internal names like RDU3 if you care about stable names, as marketing people seem to like to rename stuff after the fact. Regards, Lucas > arch/arm/mach-imx/Kconfig | 7 + > images/Makefile.imx | 5 + > 14 files changed, 1952 insertions(+) > create mode 100644 arch/arm/boards/zii-imx8mq-dev/Makefile > create mode 100644 arch/arm/boards/zii-imx8mq-dev/board.c > create mode 100644 arch/arm/boards/zii-imx8mq-dev/ddr.h > create mode 100644 arch/arm/boards/zii-imx8mq-dev/ddr_init.c > create mode 100644 arch/arm/boards/zii-imx8mq-dev/ddrphy_train.c > create mode 100644 arch/arm/boards/zii-imx8mq-dev/flash-header-zii-imx8mq-dev.imxcfg > create mode 100644 arch/arm/boards/zii-imx8mq-dev/lowlevel.c > create mode 100644 arch/arm/dts/imx8mq-zii-ultra-rmb3.dts > create mode 100644 arch/arm/dts/imx8mq-zii-ultra-zest.dts > create mode 100644 arch/arm/dts/imx8mq-zii-ultra.dtsi _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox