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 AC116C5479D for ; Mon, 9 Jan 2023 23:17:27 +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:MIME-Version:References:In-Reply-To: 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=/7gSsj6senUoSmA3W71wrDJ9FYFpX7FeO9WrLgx28QM=; b=nYxwoUeBnf/UqE 3YMJQ1p4SsTdYW+6iU5ZG69ls0bsaUkOm+IPyKFuBp7fkCYFFa1QfGt8Jtutx0epYnGwEOhKVRNb6 6RoWL3Jo7BSjzTsOCmLYl8FJg+3ymxp20+UqDVmfc1W+8XrXaxD1rRbGejTrXBZ6Zxy+w39gpSSvI ARA74K9m8p3JWCvLb2wsy/CZ0SfHh9sojHv2O0pJohleeBkMFmxRqNGxfILIH83gEvgFBnzqJkjBr HJqjvTIFAn5LGChClSlbJ5i7H/wjO7Bz/mRl5KdhyB9XBppB1mZHpQsM5JY+GuoDP0C1wGuuhuCo3 lgHriS7pfIlr1m6GzsTg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1pF1NM-004aeR-Gy; Mon, 09 Jan 2023 23:16:20 +0000 Received: from mail.andi.de1.cc ([2a01:238:4321:8900:456f:ecd6:43e:202c]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1pF1NJ-004acs-N1 for linux-arm-kernel@lists.infradead.org; Mon, 09 Jan 2023 23:16:19 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=kemnade.info; s=20220719; h=Content-Transfer-Encoding:Content-Type: MIME-Version:References:In-Reply-To:Message-ID:Subject:Cc:To:From:Date:Sender :Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id:List-Help: List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=VZOuenxf0fran8WbzZ3dqItE5pbiY4GG647n81NLsis=; b=Ei763lhn0X+krVNSqLA/I9N+31 wqxYC6nO3n9ZCZ4yR+JUCqpgZkqyV3KpzQgnBxn2aOxfP5pcrIkE1MmovutWQU5w8ke0k4m10k70y P81NU532ttThN5vd12qxh6zXOdRNYqa5fFE1yp/Gh6oEjEZzEKvFIiEWyd/przV5psOZ6smk1MpUe payClQlsdQNtOuBh7AGtCAybWrnbAvI27C82K2144GYLbX8MkrHv48pnc3AjNjplnwQtEt5Yzg776 IRIllf/+A6bqI82clUzOG76+t43bYP/kheGyUIfBjJHMbAOuErHDnwn8SyMym0Irfx5OE7exBhXwW 8xyw1MaQ==; Received: from p200300ccff06a2001a3da2fffebfd33a.dip0.t-ipconnect.de ([2003:cc:ff06:a200:1a3d:a2ff:febf:d33a] helo=aktux) by mail.andi.de1.cc with esmtpsa (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.89) (envelope-from ) id 1pF1N0-0003Zo-9l; Tue, 10 Jan 2023 00:15:58 +0100 Date: Tue, 10 Jan 2023 00:15:57 +0100 From: Andreas Kemnade To: Rob Herring Cc: Krzysztof Kozlowski , ulf.hansson@linaro.org, krzysztof.kozlowski+dt@linaro.org, shawnguo@kernel.org, s.hauer@pengutronix.de, kernel@pengutronix.de, festevam@gmail.com, linux-imx@nxp.com, linux-mmc@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] dt-bindings: mmc: fsl-imx-esdhc: allow more compatible combinations Message-ID: <20230110001557.204dd5da@aktux> In-Reply-To: <20230108224633.GA353691-robh@kernel.org> References: <20230106203358.14878660@aktux> <967cc7b7-f0bb-de37-52b9-7bfab05eadd7@linaro.org> <20230107144336.2ecff4f9@aktux> <123d1a56-8134-dc75-8b2a-b3836e727d4a@linaro.org> <20230107150740.0ba34aa1@aktux> <0ab84fb8-6173-54e0-abad-a0e0e4ba82e7@linaro.org> <20230107160105.66df4136@aktux> <20230107165457.30f4dddf@aktux> <20230108224633.GA353691-robh@kernel.org> X-Mailer: Claws Mail 3.17.8 (GTK+ 2.24.33; x86_64-pc-linux-gnu) MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230109_151617_942878_57404326 X-CRM114-Status: GOOD ( 21.09 ) 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="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 Hi Rob, On Sun, 8 Jan 2023 16:46:33 -0600 Rob Herring wrote: > A is either compatible with B or it isn't. I think English and many natural languages allow to use the adjective in a non-black and white way to express things. So there can be things more compatible than others with certain levels of "gray" in between. But "compatible" in an artifical language like the devicetree can of course be more restrictive, so there needs to be a certian level of gray where the limit needs to be defined.The definition you give below. Thanks for that. > You can look at that from > the h/w perspective and client/OS perspective. From the h/w side, is the > h/w interface the same or only has additions which can be ignored? On > the client side, the question is whether a client that only understands > B could use A's h/w without change. Looking at the match data is a > good indicator of that for Linux. It seems that from a Linux client perspective that is a no in different cases, so B is not compatible. > It's also possible the answer is > different for different clients, but we only need 1 client that could > benefit from compatibility. > On U-Boot side things seem to look different, since high speed modes are not enabled at all and pm is not done that much, so no quirks needed for that. Looking at recent mainline u-boot. { .compatible = "fsl,imx51-esdhc", }, { .compatible = "fsl,imx53-esdhc", }, { .compatible = "fsl,imx6ul-usdhc", }, { .compatible = "fsl,imx6sx-usdhc", }, { .compatible = "fsl,imx6sl-usdhc", }, { .compatible = "fsl,imx6q-usdhc", }, So U-Boot will benefit from that additional compatible for fsl,imx6[su]ll-usdhc. The first list of compatibles in U-Boot commit (96f0407b00f) was: { .compatible = "fsl,imx6ul-usdhc", }, { .compatible = "fsl,imx6sx-usdhc", }, { .compatible = "fsl,imx6sl-usdhc", }, { .compatible = "fsl,imx6q-usdhc", }, { .compatible = "fsl,imx7d-usdhc", }, So replacing imx5X fallback compatibles with imx6something might be helpful for that old U-Boot. But I cannot fully jugde that, so I will not touch. Well, I could also delete entries of this list and push a bunch of U-Boot forks somewhere, so creating a large number of different clients, which would then justify a long list of compatibles ;-) But what initially worried me would be that there could be a client out there knowing only "B" and using all features but missing the in that case needed quirks for "A". Then adding "B" would cause harm. But apparently that is nothing to worry about. I will send a reduced patch with just the things which are 100% clear to me. Regards, Andreas _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel