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 F29AFC43334 for ; Thu, 7 Jul 2022 16:49: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: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=qlFS9c5MjGj+F9R8/nGGjfgqyXaDHA8AUBqLtiUvwyg=; b=c1ZmLXWwp7WLlI T66AYr7Atj1zXbUsr9Jmax5OmSIrlVDs7WkQ09g5CZ3TVBXDwiti0h2B7iSDjZ8Cj4Yp0y9gnp6s+ cqJN/h9Tlrka2dNBx45PP2O6DgFHzlNf+nWbH5xDI43XfytMA7/IpgPQqESUu5oZ96kxZFEoXktLr SgJS9XZxuHBfJYPXCUQmzzNGG0E9FTJvcfHLZjNEA5DVVj1xJLZjukpSgwdVz4e64CQr39SGOnS3h d3OhY20NZ61kWYxNPf+vlbMXayb9xde0+D9ZvdjPuMPePJD1bhwkKX9IaEYvHKlYP+/ugBW/rkKQw PFWhaKtW8LAFo3y4tiBg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1o9Ufr-00H4CF-Aj; Thu, 07 Jul 2022 16:48:19 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1o9Ufh-00H49o-43 for linux-arm-kernel@lists.infradead.org; Thu, 07 Jul 2022 16:48:16 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 9D9491063; Thu, 7 Jul 2022 09:48:06 -0700 (PDT) Received: from donnerap.cambridge.arm.com (usa-sjc-imap-foss1.foss.arm.com [10.121.207.14]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id E99803F792; Thu, 7 Jul 2022 09:48:04 -0700 (PDT) Date: Thu, 7 Jul 2022 17:48:01 +0100 From: Andre Przywara To: Michal Suchanek Cc: linux-sunxi@lists.linux.dev, Icenowy Zheng , Rob Herring , Krzysztof Kozlowski , Chen-Yu Tsai , Jernej Skrabec , Samuel Holland , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] sunxi: dts: Fix SPI NOR campatible on Orange Pi Zero Message-ID: <20220707174801.36e010da@donnerap.cambridge.arm.com> In-Reply-To: <20220707163855.18838-1-msuchanek@suse.de> References: <20220707163855.18838-1-msuchanek@suse.de> Organization: ARM X-Mailer: Claws Mail 3.18.0 (GTK+ 2.24.32; aarch64-unknown-linux-gnu) MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220707_094809_269111_762186C5 X-CRM114-Status: GOOD ( 18.48 ) 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 On Thu, 7 Jul 2022 18:38:55 +0200 Michal Suchanek wrote: Hi, > Without "jedec,spi-nor" compatible the flash memory cannot be probed by > u-boot. > > Signed-off-by: Michal Suchanek > --- > arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts > index f19ed981da9d..d114bbc5f441 100644 > --- a/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts > +++ b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts > @@ -169,7 +169,7 @@ &spi0 { > flash@0 { > #address-cells = <1>; > #size-cells = <1>; > - compatible = "mxicy,mx25l1606e", "winbond,w25q128"; > + compatible = "mxicy,mx25l1606e", "winbond,w25q128", "jedec,spi-nor"; Naming three compatible strings violates the binding: arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dtb: flash@0: compatible: 'oneOf' conditional failed, one must be fixed: ['mxicy,mx25l1606e', 'winbond,w25q128', 'jedec,spi-nor'] is too long 'mxicy,mx25l1606e' is not one of ['issi,is25lp016d', 'micron,mt25qu02g', 'mxicy,mx25r1635f', 'mxicy,mx25u6435f', 'mxicy,mx25v8035f', 'spansion,s25sl12801', 'spansion,s25fs512s'] 'jedec,spi-nor' was expected From schema: Documentation/devicetree/bindings/mtd/jedec,spi-nor.yaml Not entirely sure why it didn't scream before, actually, because to me it looks like ending in jedec,spi-nor is mandatory. Anyway, IIUC the Macronix chip was the one shipped with the (later) boards, so we should use just that and "jedec,spi-nor". The actual vendor would be detected at runtime anyway. Cheers, Andre > reg = <0>; > spi-max-frequency = <40000000>; > }; _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel