From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pg1-f180.google.com (mail-pg1-f180.google.com [209.85.215.180]) (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 0F8E1242D9E for ; Mon, 20 Oct 2025 15:44:28 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.215.180 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1760975070; cv=none; b=ZdQtYx+oFCVaLIzp5V659g88Gssn4Hzv/ossrHyHO3vmPlcL8bdCFilYdmjXkJ8Tib8j5IJhccvRMnFC3XvI2CLexoElgPRRylTHhhAXXkxeqVdCRFm7QiO8vzeD1EfnKvD2FygwIb6yoyvVQF0vc+D4b7n9POnAhj0Umggw+94= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1760975070; c=relaxed/simple; bh=vVCAMQZ9VZYVBMCVBw/5S+OiKMRfl7we43CExJvn3Ng=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=ifF+CaiZqaOM0C7GJ0XKoIgnoWvRncLx52u4YrIg80Jo+vk+fdTJGzOVrsUTvg09XKHxY3JNy9rQ/fG/gRg10KwK97X/eJ/Xw0PThoaDqeIY/KOZjKdYMMYtmeOeH2zxK/dVq2HMeXe1W8Z4tH8WhwZ22uzQHutnKNv4zQzfNag= 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=gKXboRap; arc=none smtp.client-ip=209.85.215.180 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="gKXboRap" Received: by mail-pg1-f180.google.com with SMTP id 41be03b00d2f7-b57d93ae3b0so2941155a12.1 for ; Mon, 20 Oct 2025 08:44:28 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1760975068; x=1761579868; 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=1Y1CSxzIR0fv7maZ+ANOh4LIYK2ZvUFcV3YgMDZQcxA=; b=gKXboRapbe59G+3tNO4Fjvu0tkJ1ijWWxnooZngZENQIwIMREly+jWh0SNmrahLzY+ uZdqEwPMzKPUCwMvkG6o1EZC4OVPCNlPBypSsbOu1sQm0Pn/Uycb+PwxRE2Z9MrPFPa7 AgAfABDRWyYuXAhQMOdfq9LCWJErp/qocXJZesiniNaKLzwu9NDiJsbJ4s2yp21kJL2U w75sBffdJ/Cf3HO9Y6TN1tQssRqKsEsB1FrhYtFJZvslAdxx1MXwMqcC7zKsRbi0Z+SM DSUdxsSiH8O6AgCR2VC51POnJzW7bw02NyXtPTfbBq0XXHfpvVp/vQEFICoP4YNc1qP6 9VwA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1760975068; x=1761579868; 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=1Y1CSxzIR0fv7maZ+ANOh4LIYK2ZvUFcV3YgMDZQcxA=; b=kB9hRUgPFAmRDWJNDVlvgyz1TdpHpWYTVOvpBlqei+AA97XWUDRq3zHzimMqnrb6Uh D9a51oA4dSsc2BOBOZ71UnzOBCGWw+cG6Lqk9FMZOC7FjE4ADe4nbWVkF+ZdoR5Ic7vL td2drLjTBAAfnfDFEp1FrZuBLwUEKumIakHGT9jiSlvQ3HrBzA43xBC+6n1mIpOjn4P9 aHuPS1Ocb4nkdj2j4nC2mxTIlP0Nil2f7hx/aRIKoBm+H2lH2XIgzfBNJSPW2D+dLvVD flvmEly9CTKfzpD1auEu2UdLdqUDEhLlvXIj5cHYiWRqgPFwniXubpE7FiQCYMdzpakB Wsiw== X-Forwarded-Encrypted: i=1; AJvYcCVuEIxxKCGVnd4OSQgvtJgf4GPErDDeJbLO1uuPUP5OPMr3V3b2/ljZekfZsxcR2fO/A5xZ0YzPIQBqbdM=@vger.kernel.org X-Gm-Message-State: AOJu0YwuNHmg3i1Om9lLW5BVnq4VDt2I7C/0BXilVMwh4AgQRoRNsQi9 y2ePkRzdZjHmU+TSeTO34d/EisZW9FvHbScpdAwSOqfWUvi7xjy0UhA1 X-Gm-Gg: ASbGnctC0kd2LvlAZzQQsutrrQscsBx8qIAxd89gFO/LVDKFH0Og2AY0SYQ1h7qoNFC 2zIX0n4d2xhaB5ZwBQzYhDC+j/wKAQU4hZ9OeWC6yo/ldaSmkt/nXE9fId9EmICQem2xYKvK4Zy ljFK8y1kJu7GJ0GAhpDE2WHgTRLP6JnZbXCyx1TZpfSpKCjUH58TsvTpVPYzDct8zoV7S7QNQEX UROmaXGhryRaqR0qDJwzXZVOS6V8zkSPKRnfaKBWCxCiiv83yRqyIiqNQl4zZQsv+e9Y3cfwfK2 Aq2SoK24GFU/q5v46WXYCHVAx6gaHIR6SwiWChTioXcyGuMR9pqTE3RUK/yNfhlWEN24VxZZcjA awaG+xOToSBAYmXtheIqRwlgRamQkizdss+b9O9VJJzQJ5IUFfg9rM1aYGPoAJa9uTssLKR3GEr o0giRTk1h0BckimtgFoylBMdxfd7vpuywxnxzO X-Google-Smtp-Source: AGHT+IEMOFE9N0vOyy2Knz1EYUYNFp/diS2UfD4ZLPam7/RwKCkbB21p+DSbZLLCJqeti9P1f9KOzQ== X-Received: by 2002:a17:902:db0b:b0:290:b53b:7459 with SMTP id d9443c01a7336-290cb65dc2amr160144995ad.56.1760975068096; Mon, 20 Oct 2025 08:44:28 -0700 (PDT) Received: from [192.168.1.4] ([223.181.116.113]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-292472197desm83363565ad.115.2025.10.20.08.44.23 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 20 Oct 2025 08:44:27 -0700 (PDT) Message-ID: <46aa84cd-63cc-4263-9061-021fa3205b87@gmail.com> Date: Mon, 20 Oct 2025 21:14:19 +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 v4 0/2] dt-bindings: mmc: ti,omap2430-sdhci: Add json schema for the text binding To: Rob Herring Cc: Ulf Hansson , Krzysztof Kozlowski , Conor Dooley , Paul Barker , Marc Murphy , Tony Lindgren , Kishon Vijay Abraham I , linux-mmc@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-omap@vger.kernel.org References: <20251019-ti-sdhci-omap-v4-0-fdc63aaeb083@gmail.com> <20251020142710.GA576827-robh@kernel.org> Content-Language: en-US From: Charan Pedumuru In-Reply-To: <20251020142710.GA576827-robh@kernel.org> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 20-10-2025 19:57, Rob Herring wrote: > On Sun, Oct 19, 2025 at 01:04:36PM +0000, Charan Pedumuru wrote: >> Create a YAML binding for ti,omap2430-sdhci and fix vmmc-supply >> property typo for a DTS file. >> >> Signed-off-by: Charan Pedumuru >> --- >> Note: The property "ti,needs-special-reset" was not removed from DTS cause it will >> disrupt the compilation for other compatibles as the node &mmc is used for all >> compatibles for some DTS files. > > I don't understand. AFIACT, "ti,needs-special-reset" is only used for > the hsmmc driver/binding. But this series for for the sdhci > driver/binding. So shouldn't the property be removed from sdhci nodes > (and the binding), but kept for hsmmc nodes? Yes we can remove that property from sdhci , but &mmc node in DTS is common for all mmc drivers and this "ti,needs-special-reset" property is defined there for one board, so even when I remove it from DTSI for sdhci nodes, the DTS file still contains this property in &mmc node which is also common for other mmc drivers, so even if we remove that property for sdhci node, we still need to define it in YAML to resolve dtb_check. The issue here is not removing the property from sdhci node in DTSI file, but to remove it from &mmc node from a DTS file which is common to all mmc drivers. Here is the DTS node (ti/omap/am5729-beagleboneai.dts) which contain that property and is common for all mmc drivers. &mmc2 { status = "okay"; vmmc-supply = <&vdd_1v8>; vqmmc-supply = <&vdd_1v8>; bus-width = <8>; ti,non-removable; non-removable; mmc-pwrseq = <&emmc_pwrseq>; ti,needs-special-reset; dmas = <&sdma_xbar 47>, <&sdma_xbar 48>; dma-names = "tx", "rx"; }; > > Rob -- Best Regards, Charan.