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 BA4BCC433F5 for ; Wed, 6 Apr 2022 13:03:18 +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-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc: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=fZXfCtSf8C3zvGH9H8y+nruzHNVveVAqHHw9Rs7Yipw=; b=I0vuYr41sUWMxO +ysxDOxTHzcgp50XoqKuvmCR39SurVNQptz3qNxe4lkc/kgTClMkMvWry0P7RNTOGMEBzl3iu4xJM kBi2BXjlNR9WWFlbPlUhWvwEyAwVoSt3imA1uOUQzCXhGzGHlG38ZpQY7PPnD3sHt18UeK9qktm98 2nfGVVl3XMddUJkkPg6Ou3K3jpYDOLXNN1JnTm+pOmpBLbjKGlaVqEunxY5VrlE2S7EmhkHRmqZon ESWPRA4euPmU3nA/uCx6+/J9KupUE9DJKq8exHOHWwhdps7P9HECG7oQVN+tjlr54eccQ2AjZw21J Dcu2X7Z12z8U2O7mi/Eg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nc5Id-006Adl-80; Wed, 06 Apr 2022 13:02:15 +0000 Received: from dfw.source.kernel.org ([139.178.84.217]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nc5IZ-006AcU-Fa for linux-arm-kernel@lists.infradead.org; Wed, 06 Apr 2022 13:02:13 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 034BE61BB8; Wed, 6 Apr 2022 13:02:11 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id ED27FC385A1; Wed, 6 Apr 2022 13:02:08 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1649250130; bh=yoiRnSmms+ZvT7T7Nf5h8LP9zFWwK0yIswbzBY0vQbc=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=QrBabtYB/6p3wn5/wntsOggsMmniqRabqqJeGdu3ch5IAqC+HAulCjSFWV7pwVnJn Co4U7yYcusAd0jx2I5y1M9FdffHPt4llibJsosvf1MrlXngIMN7w+5gmutAW6G0PpH FJPEdegoRKR4JSC+lnYAfjKk9P4JcoIoZb8EBjcww15sX8KE5G8S4v2jl7bz0O/wbi zclNYXRAylDuH/NToeIzQ+RCUOoDrvIfaA8/kDR1e0lBg3jJ2a/ysuNthUyCxKwhMv cfYQrhHXDIE0uaitCxShCV8q+DAoevHvFjorlB2jeZHVlsd353raqwSmIPeEYNR8/n rLoRi6HBF2yGg== Date: Wed, 6 Apr 2022 21:02:04 +0800 From: Shawn Guo To: Alexander Stein Cc: Rob Herring , Krzysztof Kozlowski , Sascha Hauer , Fabio Estevam , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v3 0/5] Support for TQMa6ULx & TQMa6ULxL modules Message-ID: <20220406130204.GP129381@dragon> References: <20220222070945.563672-1-alexander.stein@ew.tq-group.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20220222070945.563672-1-alexander.stein@ew.tq-group.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220406_060211_637768_059FDB22 X-CRM114-Status: GOOD ( 31.25 ) 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-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Tue, Feb 22, 2022 at 08:09:40AM +0100, Alexander Stein wrote: > Hi everyone, > = > thanks for your feedback on v2. > = > Changes in v3: > * Reviewed-by: Krzysztof Kozlowski to PATCH 1 > * Fixed gpio-keys DT node names in PATCH 2 > = > Changes in v2: > * Acked-by: Rob Herring to PATCH 1 > * Removed 'status' property for non-disabled devices in PATCH 2 > * Removed superfluous blank lines in PATCH 2 > * Removed doubled status =3D "disabled" in PATCH 2 > * Moved 'chosen' node up in PATCH 2 > * Removed unit-address for GPIO buttons in PATCH 2 > * Removed internal (und unsupported) SPI device node in PATCH 2 > * Removed QSPI flash partitions in PATCH 2 > * Rename audio-codec node name in PATCH 2 > = > Below is the summary from v1 which has not changed. > = > This patch series adds support for the TQ-Systems TQMa6ULx & TQMa6ULxL mo= dules. > Admittedly the name can be confusing, so I'll explain in more detail. > = > TQMa6ULx [1] is the module series using i.MX6UL. The 'x' is a placeholder= for > different variants which have a different =B5C. TQMa6UL1 is using MCIMX6G= 1 which > has some peripheries disabled, more about it below. TQMa6UL2 (and TQMa6UL= 3) > has a full feature set and are treated identical on Linux side. > = > TQMa6ULxL [2] is the LGA variant of TQMa6ULx with a differnt module PCB, = but > still common parts. There is no variant using MCIMX6G1. > = > TQMa6ULLx [1] is identical to TQMa6ULx, but using an i.MX6ULL instead. > There is no TQMa6UL1L. > = > TQMa6ULLxL [2] is identical to TQMa6ULxL, but using an i.MX6ULL instead. > There is no TQMa6ULL1L. > = > There are also 2 different mainboards namely MBAa6ULx & MBa6ULxL ('x' her= e is > not a placeholder, but part of the name). MBAa6ULx is for non-LGA variants > TQMa6ULx & TQMa6ULLx, while MBa6ULxL is used for TQMa6ULxL & TQMa6ULLxL. > It is also possible to mount a TQMa6ULxL to MBa6ULx using an LGA adapter > providing the sockets. > = > TQMa6UL1 uses MCIMX6G1 which has FEC2, CAN2, CSI and LCDIF disabled by fu= ses. > This wouldn't be a problem at all, iff fec would support EPROBE_DEFER for > Ethernet PHYs on a different MDIO bus. Both PHYs are connected to the sam= e MDIO > bus. This is not a proble, but as FEC2 has the lower base address than FE= C1 > it is probed first. For this reason the PHYs have to be attached to FEC2 = or > things wont work. But exactly this breaks on TQMa6UL1 which has no FEC2 a= t all. > There is a suggestion for fec driver at [3] but without solution (yet). > This is the single reason there is a 'tq,imx6ul-tqma6ul1' compatible. > In order for things to work, the PHYs have to be deleted from fec2 & mdio > subnode and added to fec1 again. > = > Regardings the file structure there is a .dtsi file for > * common common parts on all 4 module variants (LGA & non-LGA, imx6ul & i= mx6ull): > imx6ul-tqma6ul-common.dtsi > * common to non-LGA variants only: imx6ul-tqma6ulx-common.dtsi > * common to LGA variants only: imx6ul-tqma6ulxl-common.dtsi > = > This is valid for both TQMa6ULx and TQMa6ULLx. To not introduce another f= ile > name separation, 'imx6ul-' is the prefix for common parts for imx6ul as w= ell > as imx6ull. > = > Best regards, > Alexander > = > [1] https://www.tq-group.com/de/produkte/tq-embedded/arm-architektur/tqma= 6ulx/ > [2] https://www.tq-group.com/de/produkte/tq-embedded/arm-architektur/tqma= 6ulxl/ > [3] https://lkml.org/lkml/2021/10/14/430 > = > Alexander Stein (4): > ARM: dts: imx6ul: add TQ-Systems MBa6ULx device trees > ARM: dts: imx6ul: add TQ-Systems MBa6ULxL device trees > ARM: dts: imx6ull: add TQ-Systems MBa6ULLx device trees > ARM: dts: imx6ull: add TQ-Systems MBa6ULLxL device trees > = > Matthias Schiffer (1): > dt-bindings: arm: fsl: add TQ Systems boards based on i.MX6UL(L) Applied all, thanks! _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel