Re: [PATCH 04/10] dt-bindings: treewide: add feature-domains description in binding files

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello Rob,

On 7/6/23 16:51, Rob Herring wrote:
On Wed, Jul 05, 2023 at 07:27:53PM +0200, Gatien Chevallier wrote:
feature-domains is an optional property that allows a peripheral to
refer to one or more feature domain controller(s).

Description of this property is added to all peripheral binding files of
the peripheral under the STM32 firewall controllers. It allows an accurate
representation of the hardware, where various peripherals are connected
to this firewall bus. The firewall can then check the peripheral accesses
before allowing it to probe.

Signed-off-by: Gatien Chevallier <gatien.chevallier@xxxxxxxxxxx>
---

Disclaimer: Some error with dtbs_check will be observed as I've
considered the property to be generic, as Rob asked

  Documentation/devicetree/bindings/crypto/st,stm32-hash.yaml  | 4 ++++
  Documentation/devicetree/bindings/dma/st,stm32-dma.yaml      | 4 ++++
  Documentation/devicetree/bindings/dma/st,stm32-dmamux.yaml   | 4 ++++
  Documentation/devicetree/bindings/i2c/st,stm32-i2c.yaml      | 4 ++++
  Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml  | 4 ++++
  .../devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml      | 4 ++++
  Documentation/devicetree/bindings/iio/dac/st,stm32-dac.yaml  | 4 ++++
  .../devicetree/bindings/media/cec/st,stm32-cec.yaml          | 4 ++++
  Documentation/devicetree/bindings/media/st,stm32-dcmi.yaml   | 4 ++++
  .../bindings/memory-controllers/st,stm32-fmc2-ebi.yaml       | 4 ++++
  Documentation/devicetree/bindings/mfd/st,stm32-lptimer.yaml  | 4 ++++
  Documentation/devicetree/bindings/mfd/st,stm32-timers.yaml   | 5 +++++
  Documentation/devicetree/bindings/mmc/arm,pl18x.yaml         | 4 ++++
  Documentation/devicetree/bindings/net/stm32-dwmac.yaml       | 4 ++++
  Documentation/devicetree/bindings/phy/phy-stm32-usbphyc.yaml | 4 ++++
  .../devicetree/bindings/regulator/st,stm32-vrefbuf.yaml      | 4 ++++
  Documentation/devicetree/bindings/rng/st,stm32-rng.yaml      | 4 ++++
  Documentation/devicetree/bindings/serial/st,stm32-uart.yaml  | 4 ++++
  Documentation/devicetree/bindings/sound/st,stm32-i2s.yaml    | 4 ++++
  Documentation/devicetree/bindings/sound/st,stm32-sai.yaml    | 4 ++++
  .../devicetree/bindings/sound/st,stm32-spdifrx.yaml          | 4 ++++
  Documentation/devicetree/bindings/spi/st,stm32-qspi.yaml     | 4 ++++
  Documentation/devicetree/bindings/spi/st,stm32-spi.yaml      | 4 ++++
  Documentation/devicetree/bindings/usb/dwc2.yaml              | 4 ++++
  24 files changed, 97 insertions(+)

diff --git a/Documentation/devicetree/bindings/crypto/st,stm32-hash.yaml b/Documentation/devicetree/bindings/crypto/st,stm32-hash.yaml
index b767ec72a999..daf8dcaef627 100644
--- a/Documentation/devicetree/bindings/crypto/st,stm32-hash.yaml
+++ b/Documentation/devicetree/bindings/crypto/st,stm32-hash.yaml
@@ -50,6 +50,10 @@ properties:
    power-domains:
      maxItems: 1
+ feature-domains:
+    minItems: 1
+    maxItems: 3

What are the 3 entries?

Rob

I thought I was benefiting from the description of the pattern-property in the RIFSC YAML file. But yes anyway, it seems like it needs some description here as the dependency does not appear in this file.

I picked 3 as a maxItems for our ST needs, I'll give it some more thought when coming back with something clearer.

I will change that in V2, thank you for pointing that out.

Best regards,
Gatien



[Index of Archives]     [Linux Kernel]     [Linux ARM (vger)]     [Linux ARM MSM]     [Linux Omap]     [Linux Arm]     [Linux Tegra]     [Fedora ARM]     [Linux for Samsung SOC]     [eCos]     [Linux PCI]     [Linux Fastboot]     [Gcc Help]     [Git]     [DCCP]     [IETF Announce]     [Security]     [Linux MIPS]     [Yosemite Campsites]

  Powered by Linux