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 C62E5C77B7A for ; Tue, 6 Jun 2023 11:14:52 +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:References:Cc:To:From: Subject:MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=1/ycahwJvVY8yz8Fe3YUJ+kvBi0J1zVXfSmM9ToTzS8=; b=bvTkXkcScWKjVp 2Vgc7w7z7FNRov5gs47VHrv9bWnGAAuRldckgqeyKSRM7xovE/kBN2R9r31mWfggLWYJGc57o/R3k EHjcTeczDTjQlffp5zkz3eQkvE0yY2lYwjexvw2mj2JfHCGOWyyYcJYzjWjeqr5uBDjU5+W66jN2T g+my62sORZifzHvZKN0rTG+QzXjPWsUhCDNTkZj+kD5hm1+7Uk23sD20hZzc9/QP+hqPLm5UL8T1c XgUS+VcLlj2ZNeC0kGX1Ykg03aoSvouwYJL+AkhLx46X6KMRxli+NHO8vHEZ5ncoRSwU31R7VuOAF CTgGBqXsh/gcpMFmRn6w==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1q6Udz-001PaG-1y; Tue, 06 Jun 2023 11:14:31 +0000 Received: from mail-ej1-x62f.google.com ([2a00:1450:4864:20::62f]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1q6Udw-001PXh-2K for linux-arm-kernel@lists.infradead.org; Tue, 06 Jun 2023 11:14:30 +0000 Received: by mail-ej1-x62f.google.com with SMTP id a640c23a62f3a-977cf86aae5so394242966b.0 for ; Tue, 06 Jun 2023 04:14:23 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1686050062; x=1688642062; h=content-transfer-encoding:in-reply-to:references:cc:to:from :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=lZSmA8DY0eTLKSTs9zFUI42k53ST/DbB3g3jLk/VVH4=; b=rCaOS62Yrkxmzf8yxhctHVMGqbGEnvBbPAPUum2pPYYLMqQ16bKL7AcLgocA1/0Lec jcBSGmJ/YMduP30NT0m33MVzOgohuUQxM22FbiVRT5+KBSqpeZb01zlIiD3amv5cewKF HNo+uOfiN1lOJjmKumu1r2ABFRFtTazGXI2OuBTMl+vMoUPy18+pcQaQqlJIqc21no4D /mdFcxLbSIpqfM5gL7K9RQn+KNyU5KP6wBOdpgTwtAbx+P1IETZi341c4Vortt5X5Y5O mcNJu2XF/X1JpLT+dzDDJ997Db+0z1jgnPOQMrsTh1CVNRugZGQf4L7oxpcDky4jH8iI 3jtw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1686050062; x=1688642062; h=content-transfer-encoding:in-reply-to:references:cc:to:from :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=lZSmA8DY0eTLKSTs9zFUI42k53ST/DbB3g3jLk/VVH4=; b=juqfjj36mmfDANZiF2HpCzAWe8GBQ/GG3JgpVvXSDDXkJ0g/aoUXpx2At6eSBkw+86 1QzX/TaCuHpZrguRFVOeQFM3kLRWJ2mcr6p09GGpn5S1+GEr3QIO/RRXYyl2wAuX2LMl hH3P1ESqKAlBVDu6JxzvJJ84yhR3jXUsgfKeWi3xyPUhcatfgXMhScfYQbwoyBW4kvBr 12io5f0CFBRlowuywXhOKK0o6i+M0JUHjmpwJA/cpsc3ISVr+No/DWAizHJHBy2u20DY Lt7VKM8MYn/syb9ivPVL8Bkl7Bt09Dtt8tfD+r+xe4XUWFdrNZoqMc9iCTfL5ye1eu0c pdJA== X-Gm-Message-State: AC+VfDwyddOUOCRpFu+23O2/qB4AcH+RajO5cUI2qlxICgIet/CRunLf YOmj1ajU1NLm5RYYNZu4Ll3T7g== X-Google-Smtp-Source: ACHHUZ7Te2mWiSqGZNgHvL+5bOOq01EPLyHtb7WGfPbOM1p4qruYTL10yBMBpLJ3aF5t8VoUPsjUQQ== X-Received: by 2002:a17:907:3f8f:b0:969:f54c:dee2 with SMTP id hr15-20020a1709073f8f00b00969f54cdee2mr2171446ejc.26.1686050062111; Tue, 06 Jun 2023 04:14:22 -0700 (PDT) Received: from [192.168.1.20] ([178.197.219.26]) by smtp.gmail.com with ESMTPSA id o18-20020a170906601200b0096f71ace804sm5463236ejj.99.2023.06.06.04.14.20 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 06 Jun 2023 04:14:21 -0700 (PDT) Message-ID: <49a33caf-e310-fcfb-7786-6731dcc747f5@linaro.org> Date: Tue, 6 Jun 2023 13:14:19 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.11.2 Subject: Re: [PATCH v8 3/3] dt-bindings: mtd: marvell-nand: Convert to YAML DT scheme Content-Language: en-US From: Krzysztof Kozlowski To: Miquel Raynal Cc: Chris Packham , "richard@nod.at" , "vigneshr@ti.com" , "robh+dt@kernel.org" , "krzysztof.kozlowski+dt@linaro.org" , "conor+dt@kernel.org" , "andrew@lunn.ch" , "gregory.clement@bootlin.com" , "sebastian.hesselbarth@gmail.com" , "conor@kernel.org" , "linux-mtd@lists.infradead.org" , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "enachman@marvell.com" , Vadym Kochan References: <20230531234923.2307013-1-chris.packham@alliedtelesis.co.nz> <20230531234923.2307013-4-chris.packham@alliedtelesis.co.nz> <785368df-1881-e62e-6172-d902cee814a8@alliedtelesis.co.nz> <4ea0b16e-0cec-00db-c598-e0364a7edef8@alliedtelesis.co.nz> <9fc57052-5049-ed50-ca95-cfd1d0420dd9@alliedtelesis.co.nz> <20230606094855.1ab005eb@xps-13> <845924ba-d9bf-d0ec-e1f2-f721366f43c0@linaro.org> <20230606122812.411b223a@xps-13> <20230606125724.126a4685@xps-13> <20230606130719.5350174c@xps-13> <037f5455-35c1-e13d-814c-16317a2a15f3@linaro.org> In-Reply-To: <037f5455-35c1-e13d-814c-16317a2a15f3@linaro.org> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230606_041428_763971_79083675 X-CRM114-Status: GOOD ( 19.77 ) 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 06/06/2023 13:11, Krzysztof Kozlowski wrote: >>> If yes, please ignore the series sent aside, I will work on it again >>> and send a v2. >> >> Actually I already see a problem, let's the ingenic,nand.yaml example. >> The goal, IIUC, is to do: >> >> patternProperties: >> "^nand@[a-f0-9]$": >> type: object >> + $ref: nand-chip.yaml >> properties: >> >> ... >> >> + unevaluatedProperties: false >> >> The example in this file uses a property, nand-on-flash-bbt, which is >> described inside nand-controller.yaml instead of nand-chip.yaml. >> Indeed, the former actually describes many properties which are a bit >> more controller related than chip related. With the above description, >> the example fails because nand-on-flash-bbt is not allowed (it is not >> listed in nand-chip.yaml). >> >> How would you proceed in this case? >> >> Maybe I could move all the NAND chip properties which are somehow >> related to NAND controllers (and defined in nand-controller.yaml) in a >> dedicated file and reference it from nand-chip.yaml? Any other idea is >> welcome. > > Yes, this would work and seems reasonable. Actually, since nand-chip is used by both SPI and NAND, then I think better would be to create separate file - nand-only-chip.yaml (name to be discussed): nand-controller.yaml: "^nand@[a-f0-9]$": $ref: nand-only-chip.yaml nand-only-chip.yaml: $ref: nand-chip.yaml all nand-controller-chip properties follow > Other way could be to add > unevaluatedProperties:false on this level (so after ref:nand-chip.yaml) > in nand-controller.yaml. This however would not allow any new properties > to be defined in device bindings. Best regards, Krzysztof _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel