From patchwork Wed Oct 9 21:50:44 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sam Edwards X-Patchwork-Id: 13829310 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 49328CEE358 for ; Wed, 9 Oct 2024 21:59:55 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: MIME-Version:References:In-Reply-To:Message-ID:Date:Subject:Cc:To:From: Reply-To:Content-Type:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=ytdNCpY3ywuE2VnDeLP5mnfyE1FOZ+8OMXSn6PncjqA=; b=l6DWlKo263egynhaRctip3RThW HQCA7OCXjg3RfnPk0Lw1ryQ4nA7Prv3TZ4kcK9GWJawXJH7ufS+VdTfLFOuzzs6DIubp/l1fwFoZD rjqk6NYFmKlSTfwhTjT+yHsZE9lFw+odSmFd8srqz9dnIxTuXLLU9goEp8v6WvohX2bAAG2P3syFT z5Q/H+pwx4bcdznvH37Wv+q5eAbcsRl+WbNJtSrf+0dOXmp59hKIEDgLIci1xK670pqobzqDYd8Oz PNYrx1Ui2Nd996ppQA2g+eWLnPn9H4Q4dt58+K2HyFkl1NODB06OGHKAJSDFXAK1uDhWr52NFkXhd H54MEDfA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1syeif-0000000As9N-0FRQ; Wed, 09 Oct 2024 21:59:45 +0000 Received: from mail-pg1-x536.google.com ([2607:f8b0:4864:20::536]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1syeaM-0000000AqVu-3e1x for linux-arm-kernel@lists.infradead.org; Wed, 09 Oct 2024 21:51:12 +0000 Received: by mail-pg1-x536.google.com with SMTP id 41be03b00d2f7-7e6cbf6cd1dso171437a12.3 for ; Wed, 09 Oct 2024 14:51:10 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1728510670; x=1729115470; darn=lists.infradead.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=ytdNCpY3ywuE2VnDeLP5mnfyE1FOZ+8OMXSn6PncjqA=; b=edNoANB2Ey8MXRRdN/djpe/pfcmQVDhX+PKGLTDIbmF706Gp8/V1UWcwcSW/9CvUEZ ohWjQ3LLMYzN/UESz0PtLBYm3DnibrkbeLrPIHkn/XmAjdoWnkDVAX9peibRYXsyBEaR SgdCE044ItEH6lqDXnAdPJkPAu2Zs82uW/Cmm0tVHjf9E9oNWEmoWTQleGkGwjPM+v9+ /boKClQW19CHgJyhqr996i/Eh78IpeS/3n4DNsIS7NiC6a1GnXi9JAywjjUI1OS80mED PwmW1cJYSsGUtbiLgF6w6Bhu0FS6fUMjxXo1k11Lw/6ul4SwRncomDgR8TNYge5WzJ1F 4NTA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1728510670; x=1729115470; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=ytdNCpY3ywuE2VnDeLP5mnfyE1FOZ+8OMXSn6PncjqA=; b=RcHl8NH4C1Ue9lFtPecqGnWeWVNEbxDNu8/2VZB7OCX2V/LW1wgAymFrz95wVaS2Mp tBORz3HoIiYwibTE1PSmhLna9stiXny0U/4xUouyE8zHA4bXyja9h+2r3D+SCV5GSAXR UTF+U3qv1ZZr3SeuRk9uQTxlkg0Ke36IaRPeZxWrampU811feyPKeP4sCo2ZrGSZktyC MY7Z48r4EeDy17jexMQydN2vc80LVgWQ5X2hVejwc9scheaXIkldPF1o/BbmCBBr4Mxi MKgaFgEdbk3TO22eoXtO0dv2Thk1TgEkI58DJC+0LO0caOLjnOkuE6NCAZIitdVxFLv+ UYXA== X-Forwarded-Encrypted: i=1; AJvYcCUn1Qi/txybvQHkQIfLAc6tdzHDXG+lsKZR939UJ4Sku/5tJIYq1bA4Y1L1U16DPKB5nAKskqRwW6Jpo6mgcXPe@lists.infradead.org X-Gm-Message-State: AOJu0YyANifS8qI3+uBid+MFpJ6dpOw8HJBBYPXQu0yi46aGePAVui5R tGbUJp3Q5LWtXzo66LWNYYS+XjhvCNmoHgTESc/lNFq2ynZEdWzE X-Google-Smtp-Source: AGHT+IGAb8YE10SWaJg6VOPcMZnsWnq/ERhnLIaiMtgIwviert0u7YtGlBsmxO3ymE0702ihsbZdrQ== X-Received: by 2002:a05:6a21:6b0a:b0:1cf:359b:1a3e with SMTP id adf61e73a8af0-1d8ad83cd67mr2252199637.32.1728510670257; Wed, 09 Oct 2024 14:51:10 -0700 (PDT) Received: from luna.turtle.lan ([2601:1c2:c184:dc00::315]) by smtp.gmail.com with ESMTPSA id 41be03b00d2f7-7e9f6833a95sm7651847a12.52.2024.10.09.14.51.09 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 09 Oct 2024 14:51:09 -0700 (PDT) From: Sam Edwards X-Google-Original-From: Sam Edwards To: Florian Fainelli , =?utf-8?b?UmFmYcWCIE1p?= =?utf-8?b?xYJlY2tp?= , William Zhang , Anand Gore , Kursad Oney Cc: Rob Herring , Krzysztof Kozlowski , Conor Dooley , Broadcom internal kernel review list , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Sam Edwards Subject: [PATCH 1/3] dt-bindings: mtd: Relax BCM4908 partition schema Date: Wed, 9 Oct 2024 14:50:44 -0700 Message-ID: <20241009215046.1449389-2-CFSworks@gmail.com> X-Mailer: git-send-email 2.44.2 In-Reply-To: <20241009215046.1449389-1-CFSworks@gmail.com> References: <20241009215046.1449389-1-CFSworks@gmail.com> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20241009_145110_954270_24EB59DA X-CRM114-Status: GOOD ( 13.15 ) 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: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org The BCM4908 partition "parser" is really just a fixed partitions table, with a special partition compatible (`brcm,bcm4908-firmware`) that automatically labels the partition as "firmware" or "backup" depending on what CFE is communicating as the selected active partition. The bcm4908-partitions schema is currently too restrictive, requiring that all child nodes use this special compatible or none at all. This not only contracits what is allowed by the "parser" but also causes warnings for an existing file ("bcm4908-asus-gt-ac5300.dts"). Modify the schema to be strict only for child partitions that use the -firmware compatible. Also update the child name regex to agree with fixed-partitions, so that these differences apply consistently. Signed-off-by: Sam Edwards --- .../mtd/partitions/brcm,bcm4908-partitions.yaml | 17 +++++++++++------ 1 file changed, 11 insertions(+), 6 deletions(-) diff --git a/Documentation/devicetree/bindings/mtd/partitions/brcm,bcm4908-partitions.yaml b/Documentation/devicetree/bindings/mtd/partitions/brcm,bcm4908-partitions.yaml index 94f0742b375c..aed37922a5fc 100644 --- a/Documentation/devicetree/bindings/mtd/partitions/brcm,bcm4908-partitions.yaml +++ b/Documentation/devicetree/bindings/mtd/partitions/brcm,bcm4908-partitions.yaml @@ -30,12 +30,17 @@ properties: enum: [ 1, 2 ] patternProperties: - "^partition@[0-9a-f]+$": - $ref: partition.yaml# - properties: - compatible: - const: brcm,bcm4908-firmware - unevaluatedProperties: false + "^partition(-.+|@[0-9a-f]+)$": + type: object + if: + properties: + compatible: + const: brcm,bcm4908-firmware + then: + $ref: partition.yaml# + properties: + compatible: true + unevaluatedProperties: false required: - "#address-cells"