devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] dt-bindings: mmc: arasan,sdhci: Make interrupt optional
@ 2025-06-04  8:51 Michal Simek
  2025-06-05 13:03 ` Rob Herring
  0 siblings, 1 reply; 3+ messages in thread
From: Michal Simek @ 2025-06-04  8:51 UTC (permalink / raw)
  To: linux-kernel, monstr, michal.simek, git
  Cc: Adrian Hunter, Conor Dooley, Krzysztof Kozlowski, Rob Herring,
	Ulf Hansson,
	open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
	open list:MULTIMEDIA CARD (MMC), SECURE DIGITAL (SD) AND...

Bootloader (as U-Boot) is not using interrupt which doesn't need to be even
described in DT that's why make interrupt optional property.

Signed-off-by: Michal Simek <michal.simek@amd.com>
---

For example mini configurations used for programming.
https://source.denx.de/u-boot/u-boot/-/blob/master/arch/arm/dts/zynqmp-mini-emmc0.dts
---
 Documentation/devicetree/bindings/mmc/arasan,sdhci.yaml | 1 -
 1 file changed, 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/mmc/arasan,sdhci.yaml b/Documentation/devicetree/bindings/mmc/arasan,sdhci.yaml
index 8e79de97b242..0472225e6e58 100644
--- a/Documentation/devicetree/bindings/mmc/arasan,sdhci.yaml
+++ b/Documentation/devicetree/bindings/mmc/arasan,sdhci.yaml
@@ -192,7 +192,6 @@ dependencies:
 required:
   - compatible
   - reg
-  - interrupts
   - clocks
   - clock-names
 
-- 
2.43.0


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] dt-bindings: mmc: arasan,sdhci: Make interrupt optional
  2025-06-04  8:51 [PATCH] dt-bindings: mmc: arasan,sdhci: Make interrupt optional Michal Simek
@ 2025-06-05 13:03 ` Rob Herring
  2025-06-05 13:06   ` Michal Simek
  0 siblings, 1 reply; 3+ messages in thread
From: Rob Herring @ 2025-06-05 13:03 UTC (permalink / raw)
  To: Michal Simek
  Cc: linux-kernel, monstr, michal.simek, git, Adrian Hunter,
	Conor Dooley, Krzysztof Kozlowski, Ulf Hansson,
	open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
	open list:MULTIMEDIA CARD (MMC), SECURE DIGITAL (SD) AND...

On Wed, Jun 04, 2025 at 10:51:54AM +0200, Michal Simek wrote:
> Bootloader (as U-Boot) is not using interrupt which doesn't need to be even
> described in DT that's why make interrupt optional property.

That's true for 'interrupts' everywhere. It could also be true for 
clocks, resets, etc. If the h/w always has the resource, it should be 
required in DT.

Rob

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] dt-bindings: mmc: arasan,sdhci: Make interrupt optional
  2025-06-05 13:03 ` Rob Herring
@ 2025-06-05 13:06   ` Michal Simek
  0 siblings, 0 replies; 3+ messages in thread
From: Michal Simek @ 2025-06-05 13:06 UTC (permalink / raw)
  To: Rob Herring
  Cc: linux-kernel, monstr, michal.simek, git, Adrian Hunter,
	Conor Dooley, Krzysztof Kozlowski, Ulf Hansson,
	open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
	open list:MULTIMEDIA CARD (MMC), SECURE DIGITAL (SD) AND...



On 6/5/25 15:03, Rob Herring wrote:
> On Wed, Jun 04, 2025 at 10:51:54AM +0200, Michal Simek wrote:
>> Bootloader (as U-Boot) is not using interrupt which doesn't need to be even
>> described in DT that's why make interrupt optional property.
> 
> That's true for 'interrupts' everywhere. It could also be true for
> clocks, resets, etc. If the h/w always has the resource, it should be
> required in DT.

ok. fine.

Thanks,
Michal

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2025-06-05 13:06 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-06-04  8:51 [PATCH] dt-bindings: mmc: arasan,sdhci: Make interrupt optional Michal Simek
2025-06-05 13:03 ` Rob Herring
2025-06-05 13:06   ` Michal Simek

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).