From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pf1-f175.google.com (mail-pf1-f175.google.com [209.85.210.175]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id C6FDF26E6FA for ; Fri, 13 Mar 2026 06:18:10 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.175 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773382692; cv=none; b=hKda79ai2DbwJW0DrcpelRk0Qs4CA7WVxSpt9pAzZ3ULSSsLdKmzRDOe+siV+ZN25QWQePXuMSxP+CujI5XRMS51BBEgxUaBU7k2QC+88dE+iMjwX0v0fih09om1VIyWnRUn+ao/vS0BI9Gzv+1UwcGvnzk6WwKRPktwc10itZw= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773382692; c=relaxed/simple; bh=6ZWQY4ymrBGJ8hYNeyFqtRtQF7QKPc3nK2vfrQJD4sY=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=ZsjA5vDFmJG3rHNKh7PgmpVEMvQWoIyhgHB9gtLweat4DI1b3TE27AU67k5Bp7uy8XLlEIYjVS3zTA7wyZ+ght2ZUB7rNbU4LavKFfQKQVTg5m+KOGOF8m94g2O9hzMuCUQPY6Nj2MKxz07OkOeOoFMp3K0ukOUDEZFU8G6CuMY= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=acmQIefg; arc=none smtp.client-ip=209.85.210.175 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="acmQIefg" Received: by mail-pf1-f175.google.com with SMTP id d2e1a72fcca58-829b2019b39so1058904b3a.3 for ; Thu, 12 Mar 2026 23:18:10 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1773382690; x=1773987490; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=zmh5HzaCjwFFl1hth5ZY2QU6iyHPHYjmE5iTWZ5JFAE=; b=acmQIefgK0C9TPYg911+7dfh/rixy9ccWBEjNEp0FJAUt9RzIdnHtVBKo2wmrAHqgu EenbCbHzDbluas5av8mr5oLD0u+dP3aPiBO+YqNTViZRBNtUuqLAYeLa11TEspnqE6i6 mS+vNRY+wG/Mr/nKozjcQ5+riNnBmJhfvJ6EhA3dYGDkLS6eaL8Uf0mxxHgiL/XmjY3w zDfdGQT4E+3+Em2ueqITCmezr6LKqEu83RpFAGKMuquQAyItUXq7Z2N1+vEe7oZsThB8 0QxW8jetROf8vh+hgQLHUVgB+PTab8jPFc5MTUo6qId+/J985gXkqOQCMZzPC2Zjng3h 1Wjw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1773382690; x=1773987490; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=zmh5HzaCjwFFl1hth5ZY2QU6iyHPHYjmE5iTWZ5JFAE=; b=lLjeC7XGcIcupLTCZsDy0mIa7alnRVU9TkaXAiybmeH/wTHQNtgJV9otsXlYNRLq86 XHKoWEq+OoAtFMZhd6HlMtTJ039ZL40elIP/oFmy2CqIINbMdPg8WKLbGHEQGu1ZFZYo m2RUDkNjpBflYuZWJlfs34R9TqvtWJigmBqGAWLu9H55kg/ZVtiI7k+WhrGnO4eV+hgL KyiJO4S3v0uQ8F38HknYGfhSxgQFX2Lhi5hB8oMxZ4/QscV8vKpxkdfJnidGKVjIi37M ITrKSG/PBKXwauq3Lf00c9LgRT/I4wq/zXTcnyuRpOYEjNRZrOm6SKIK36BvZ84x/+vW GEDw== X-Forwarded-Encrypted: i=1; AJvYcCXuaFa+XLzcUk+gwnSSXzT5vfXNJDsrjVs/7w81bSCv61DyoSeNXHI6cqe3WMXHHGRsP9sWRWR35kA0hIU=@vger.kernel.org X-Gm-Message-State: AOJu0YxkUlPzVtB4I0KOqh/SxcphfQ+sGZM0gBke3MN02JU1jBlSu21K lsJCX/N0KCsrhYIJGnz9xJjxm/zzP2iVUcXj2n6Rx43BCjCET6pxz5OS X-Gm-Gg: ATEYQzwuPb3aZxqJlVguQnZRHdd8LDF7E2yOYSf9+4zBWtg2FC1/B5jeKOApp8AxZpD /aktP3IG4GdmZeA3YUiAJGPjkWLUK2MBt4u/qy4U5r1J4+LaPGorr4bREHVWtjqHNqZDz0UXQEk xb4gspMm1hNzbtHTLWOFVDsaPS/i6/IMIisZ50e7joLQK0TOPq/sGdS+y0OaL9WxTr5rXcasDj6 dDiZNdaZGWN+5WKmfKwGOMQBBEd9lcz0V3XULAg6cwimlURIp8Fyc6TksgiHA0dlecxlMOaiQtR wQ3p3Ev6K8hO2+vj2j2ejc7AFRivXokzmMbTaJ7hUIAxPm2oqhxsPHx7JGuMm2tyNeZDK9YDHDA pCC4PFSOM1z/v5naK7ISoNFt2w4VxuJxmJfQ/bQR/dWIOl56PsO+pXAZSEA9xUiO9fg4mS95qUu StXn3KuyF3p3vMqdyjR2Ala8FRrWjiD/YsL8CcPMghwAQ4OdQ6I2WC9v3MSGZoqPVEKGbYV/GdC LFviUhG4+nMQU3BJ7M= X-Received: by 2002:a05:6a21:618e:b0:398:8dbb:d2b with SMTP id adf61e73a8af0-398ecdd56bamr1753374637.58.1773382690097; Thu, 12 Mar 2026 23:18:10 -0700 (PDT) Received: from ?IPV6:2401:4900:892e:1f2a:440a:1550:d03:380c? ([2401:4900:892e:1f2a:440a:1550:d03:380c]) by smtp.gmail.com with ESMTPSA id 41be03b00d2f7-c73ebb76f04sm915634a12.26.2026.03.12.23.18.02 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 12 Mar 2026 23:18:09 -0700 (PDT) Message-ID: Date: Fri, 13 Mar 2026 11:48:00 +0530 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH] dt-bindings: mtd: hisilicon,504-nfc: convert to DT schema To: Rob Herring Cc: Miquel Raynal , Richard Weinberger , Vignesh Raghavendra , Krzysztof Kozlowski , Conor Dooley , linux-mtd@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org References: <20260307-hisi504-1-v1-1-8bf9a186faf2@gmail.com> <20260312152851.GA3190071-robh@kernel.org> Content-Language: en-US From: Akhila YS In-Reply-To: <20260312152851.GA3190071-robh@kernel.org> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 12-03-2026 20:58, Rob Herring wrote: > On Sat, Mar 07, 2026 at 04:43:56PM +0000, Akhila YS wrote: >> Convert Hisilicon Hip04 Soc NAND controller DT binding to YAML format. >> >> Signed-off-by: Akhila YS >> --- >> .../devicetree/bindings/mtd/hisilicon,504-nfc.yaml | 90 ++++++++++++++++++++++ >> 1 file changed, 90 insertions(+) > Convert implies the removal of the .txt binding. Where is that? Could be > missing? If so, say that. Hi rob, yeah the removed .txt binding is not added in this commit, i will add it and send that in a v2. >> diff --git a/Documentation/devicetree/bindings/mtd/hisilicon,504-nfc.yaml b/Documentation/devicetree/bindings/mtd/hisilicon,504-nfc.yaml >> new file mode 100644 >> index 000000000000..805ef0af0e04 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/mtd/hisilicon,504-nfc.yaml >> @@ -0,0 +1,90 @@ >> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) >> +%YAML 1.2 >> +--- >> +$id: http://devicetree.org/schemas/mtd/hisilicon,504-nfc.yaml# >> +$schema: http://devicetree.org/meta-schemas/core.yaml# >> + >> +title: HiSilicon Hip04 NAND Flash Controller >> + >> +maintainers: >> + - Miquel Raynal >> + - Richard Weinberger > This should be someone with the h/w. I would use the HiSilicon > maintainer from MAINTAINERS. Okay, sure. > >> + >> +description: >> + The HiSilicon 504 NFC is a NAND flash memory controller used in the >> + Hip04 SoC. It supports hardware ECC for NAND devices and provides >> + register and buffer regions for NAND operations. >> + >> +allOf: >> + - $ref: nand-controller.yaml# >> + >> +properties: >> + compatible: >> + const: hisilicon,504-nfc >> + >> + reg: >> + minItems: 2 >> + maxItems: 2 > Need to define what each entry is. Okay. > >> + >> + interrupts: >> + maxItems: 1 >> + >> + nand-bus-width: >> + enum: [8, 16] >> + >> + nand-ecc-mode: >> + enum: >> + - none >> + - hw >> + >> + nand-ecc-strength: >> + const: 16 >> + >> + nand-ecc-step-size: >> + const: 1024 >> + >> + "#address-cells": >> + const: 1 >> + >> + "#size-cells": >> + const: 0 > These can be dropped as the child (partitions) doesn't have an address. Okay. > >> + >> + partitions: >> + $ref: /schemas/mtd/partitions/fixed-partitions.yaml# >> + >> +required: >> + - compatible >> + - reg >> + - interrupts >> + - nand-bus-width >> + - nand-ecc-mode >> + - "#address-cells" >> + - "#size-cells" >> + >> +unevaluatedProperties: false >> + >> +examples: >> + - | >> + nand-controller@4020000 { >> + compatible = "hisilicon,504-nfc"; >> + reg = <0x04020000 0x10000>, <0x05000000 0x1000>; >> + interrupts = <0 379 4>; >> + nand-bus-width = <8>; >> + nand-ecc-mode = "hw"; >> + nand-ecc-strength = <16>; >> + nand-ecc-step-size = <1024>; >> + #address-cells = <1>; >> + #size-cells = <0>; >> + >> + partitions { >> + compatible = "fixed-partitions"; >> + #address-cells = <1>; >> + #size-cells = <1>; >> + >> + partition@0 { >> + label = "nand_text"; >> + reg = <0x0 0x400000>; >> + }; >> + }; >> + }; >> +... >> >> --- >> base-commit: d4906ae14a5f136ceb671bb14cedbf13fa560da6 >> change-id: 20260306-hisi504-1-d7aa09c70d93 >> >> Best regards, >> -- >> Akhila YS >> -- Best Regards, Akhila.