From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f181.google.com (mail-pl1-f181.google.com [209.85.214.181]) (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 ABB9A25A328 for ; Sun, 19 Oct 2025 13:58:59 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.181 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1760882341; cv=none; b=pJX/CfJ73XImYj1G0f4ajKH//XGAe4diumhbHpaFER8he92yKOgy+pBO6TXzL3BUUfnM9jjxZahuj7GfnyPgnFBn226QiDwXfjBxvEbC4aY1mNg7gUa4QmaqJCX0AXVfQWYeX/Cy2IG77RUrWgl/xvtCO2iu1rI8/KDf8E0HCoo= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1760882341; c=relaxed/simple; bh=hRkplG1ra9s4vBIWDyFAgQ3Zlr+bwaWbOINHQ5iTB2c=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=PR59GrU3OGCfHOdZIsgVyHUI08PbMSsAiVc1m6vejSLn81JNYauFIMU06VOEn03O2Rbu0aHLMycvKfSSeY4odj8a46tvJYb6SMTtc91T0QTr9zuTZD3jwkrlSuAT83yr6xmhzwG2kXjVqBgZDsEnLxbZ8ELOU1CNl0/sSXwnGJg= 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=f+ON6Dyk; arc=none smtp.client-ip=209.85.214.181 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="f+ON6Dyk" Received: by mail-pl1-f181.google.com with SMTP id d9443c01a7336-27ee41e074dso41665765ad.1 for ; Sun, 19 Oct 2025 06:58:59 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1760882339; x=1761487139; 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=uIvRt0vzEtUfOdf1MrHqv5pQ80dvz1yAdmfgYjIUz1s=; b=f+ON6DykLdVDTVIG8s0mkIE7AleRp3w0tj6QJvexYX/p7eCDmR7PTuN1uVMSn4xT9B Smb4EdROcd0zwE7kytA17/4airMvtOgF+WCb2TVKdzwC4NRMWS5GKGmV5+vcvAGC3M81 hvd2GD+doDlOlLW+hGzwB/e619Yi6QNCiUkfzuKSEoA9sx4gcGUQK90O8Esyg6JsPI5m 1Y7ylMFwsQibgWVznlyrTOsQ8+qL/xakbSikABsboGzW23jcUMI1soq9gp//eXvHtdEl 04Zqkx40pmuWeMF4RWwrjizo5OfQSFXzbAPp9/haF2YrCsbExxKM4z8NNDr5A0IbfRgO ItIA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1760882339; x=1761487139; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=uIvRt0vzEtUfOdf1MrHqv5pQ80dvz1yAdmfgYjIUz1s=; b=EI0BMOS2MGKy9lSu+wEHTi34WvHiGujvV5gdG/ukqyoevYW9137fSnkowHbvMvitxK vb+TKc2IKhYEr5coD0qGh+LHOu+VhbQhfKO1KaQW1R0NCJ3nCaMbVk25bMw+bz+1niOb 1Pd9rkkIL/6wl6BeY3Y0712h1ie5J0lecl45He3fJ7DAW5W1Ty4wGepg6s17uTCtkm46 57lnsLsKRSVT5hm6PFqxB2i1+ZwNRNVBF2vrfGVj3wCOw3jHOW9zyPyvG+bm3qkcYIMR QTWurYJ2CYk5llgsZQFnnBUBl9HWKJrxWdt9TVwQVFlg3Dtpw92hgLKNUOaPFMaQ2R1D ejaQ== X-Forwarded-Encrypted: i=1; AJvYcCVcZ98laRZ+GcfkoUVhpW8/O6kaboHHniwpjjXaM5ODeJ1AkYej2sNpmYsGs6ShSiMgCSHNSQBDvPU=@vger.kernel.org X-Gm-Message-State: AOJu0Yzjg1fIDmSnBtAc0kw0HnfpkkR+kzVve4P+UN4RN5jQ79FszHw3 GSE/nhjcaLUPc9obtdAxRM7wX/XqyS8snvT5wPiPIBVInN4Gm5jlLQzw X-Gm-Gg: ASbGncsPs53lPRLBrDLGX4FoTc5hebG7xu4mBTMJ32z51+w1Atpuvq984kpxOsUL1mo W0s+FrzgkcTTjLw/FslYhiE1s2FTlkXkVeIAPDoz/dQNMYHzzhtdtPw1Mao9Dai4VTSiRVnqRSh vSTzkkwworYZT6h4wWYd7QVLouHDmOW6GLE4VlWpKFDOXkObAXwrqq+BzE7vczdgW9peTunHqYT z/8bDGfrGYI0j1MZ/TNrgaO0DQGPPM730a3X/cH6ayfpH2VY+9CBaW+HvopcuYrOciUoaUBE09V 24fQtaIZSkr9pnicQPYthnaPkNtJmGjtn/Eedv7tAd32fAuB6/o5nAfe9MdrZ/2u1tHOQ9QKJr4 W5bVCGvYIadGV4mPpznPMbju7cIHWWrVlf/Ztbmc218sYUpj3A2xBc1eDysnvoFhLfG+MN/ulUH CooQZH0Yi4Q3lWd5yW7Zp0m/KzzQ== X-Google-Smtp-Source: AGHT+IGfMFiuCP3pNp6JLr8ZsJfilLCsq2Iy14veN19sT8WT/30N57bEv0uqUBgvns7qDZN+JiYhLQ== X-Received: by 2002:a17:903:32ca:b0:290:b92d:907 with SMTP id d9443c01a7336-290cb376121mr146870135ad.53.1760882338934; Sun, 19 Oct 2025 06:58:58 -0700 (PDT) Received: from [192.168.1.4] ([223.181.116.113]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-292471d5995sm54198275ad.70.2025.10.19.06.58.55 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sun, 19 Oct 2025 06:58:58 -0700 (PDT) Message-ID: Date: Sun, 19 Oct 2025 19:28:49 +0530 Precedence: bulk X-Mailing-List: linux-mmc@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2] dt-bindings: mmc: ti,da830-mmc: convert to DT schema To: Conor Dooley Cc: Ulf Hansson , Rob Herring , Krzysztof Kozlowski , Conor Dooley , linux-mmc@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org References: <20251011-davinci-mmc-v2-1-355da3e25123@gmail.com> <20251012-nickname-morale-e1e21796f1f1@spud> Content-Language: en-US From: Charan Pedumuru In-Reply-To: <20251012-nickname-morale-e1e21796f1f1@spud> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 12-10-2025 19:05, Conor Dooley wrote: > On Sat, Oct 11, 2025 at 08:52:07AM +0000, Charan Pedumuru wrote: >> Convert TI Highspeed MMC host controller binding to YAML format. Define >> 'clocks' and 'interrupts' properties to resolve errors identified by >> 'dt_check' and 'dtb_check'. >> >> Signed-off-by: Charan Pedumuru >> --- >> Changes in v2: >> - Modified the commit message. >> - Removed 'interrupts' from required properties following the old binding. >> - Changed the maintainer for the binding to "Conor Dooley". > > Bro, what? Where did that come from? I know nothing about this device at > all. Find someone from TI to put there. I checked for the compatible driver and found the author but no email address, so can I add Ulf Hansson to the maintainers then? > > Conor. > > >> - Link to v1: https://lore.kernel.org/r/20250523-davinci-mmc-v1-1-ceebd8352d9c@gmail.com >> --- >> .../devicetree/bindings/mmc/davinci_mmc.txt | 32 ------------ >> .../devicetree/bindings/mmc/ti,da830-mmc.yaml | 61 ++++++++++++++++++++++ >> 2 files changed, 61 insertions(+), 32 deletions(-) >> >> diff --git a/Documentation/devicetree/bindings/mmc/davinci_mmc.txt b/Documentation/devicetree/bindings/mmc/davinci_mmc.txt >> deleted file mode 100644 >> index 516fb0143d4c21d1c8e44a8846d55ea5458d7b74..0000000000000000000000000000000000000000 >> --- a/Documentation/devicetree/bindings/mmc/davinci_mmc.txt >> +++ /dev/null >> @@ -1,32 +0,0 @@ >> -* TI Highspeed MMC host controller for DaVinci >> - >> -The Highspeed MMC Host Controller on TI DaVinci family >> -provides an interface for MMC, SD and SDIO types of memory cards. >> - >> -This file documents the properties used by the davinci_mmc driver. >> - >> -Required properties: >> -- compatible: >> - Should be "ti,da830-mmc": for da830, da850, dm365 >> - Should be "ti,dm355-mmc": for dm355, dm644x >> - >> -Optional properties: >> -- bus-width: Number of data lines, can be <1>, <4>, or <8>, default <1> >> -- max-frequency: Maximum operating clock frequency, default 25MHz. >> -- dmas: List of DMA specifiers with the controller specific format >> - as described in the generic DMA client binding. A tx and rx >> - specifier is required. >> -- dma-names: RX and TX DMA request names. These strings correspond >> - 1:1 with the DMA specifiers listed in dmas. >> - >> -Example: >> -mmc0: mmc@1c40000 { >> - compatible = "ti,da830-mmc", >> - reg = <0x40000 0x1000>; >> - interrupts = <16>; >> - bus-width = <4>; >> - max-frequency = <50000000>; >> - dmas = <&edma 16 >> - &edma 17>; >> - dma-names = "rx", "tx"; >> -}; >> diff --git a/Documentation/devicetree/bindings/mmc/ti,da830-mmc.yaml b/Documentation/devicetree/bindings/mmc/ti,da830-mmc.yaml >> new file mode 100644 >> index 0000000000000000000000000000000000000000..1a97c3e447fd10f14bfe0af9e22f9479304f0f26 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/mmc/ti,da830-mmc.yaml >> @@ -0,0 +1,61 @@ >> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) >> +%YAML 1.2 >> +--- >> +$id: http://devicetree.org/schemas/mmc/ti,da830-mmc.yaml# >> +$schema: http://devicetree.org/meta-schemas/core.yaml# >> + >> +title: TI Highspeed MMC host controller for DaVinci >> + >> +description: >> + The Highspeed MMC Host Controller on TI DaVinci family >> + provides an interface for MMC, SD and SDIO types of memory cards. >> + >> +allOf: >> + - $ref: mmc-controller.yaml >> + >> +maintainers: >> + - Conor Dooley >> + >> +properties: >> + compatible: >> + enum: >> + - ti,da830-mmc >> + - ti,dm355-mmc >> + >> + reg: >> + maxItems: 1 >> + >> + clocks: >> + maxItems: 1 >> + >> + interrupts: >> + maxItems: 2 >> + >> + dmas: >> + maxItems: 2 >> + >> + dma-names: >> + items: >> + - const: rx >> + - const: tx >> + >> +required: >> + - compatible >> + - reg >> + >> +unevaluatedProperties: false >> + >> +examples: >> + - | >> + #include >> + mmc@1c40000 { >> + compatible = "ti,da830-mmc"; >> + reg = <0x40000 0x1000>; >> + interrupts = <16 IRQ_TYPE_LEVEL_HIGH>, >> + <17 IRQ_TYPE_LEVEL_HIGH>; >> + bus-width = <4>; >> + max-frequency = <50000000>; >> + dmas = <&edma 16>, <&edma 17>; >> + dma-names = "rx", "tx"; >> + }; >> +... >> >> --- >> base-commit: 7bac2c97af4078d7a627500c9bcdd5b033f97718 >> change-id: 20250523-davinci-mmc-c704440c3dd0 >> >> Best regards, >> -- >> Charan Pedumuru >> -- Best Regards, Charan.