| Message ID | 20230705172759.1610753-5-gatien.chevallier@foss.st.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series | Introduce STM32 Firewall framework | expand |
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@foss.st.com> >> --- >> >> 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
Gatien Chevallier <gatien.chevallier@foss.st.com> writes: > 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@foss.st.com> > --- > > 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 > + I beliewe feature-domains is generic binding. This means that maxItems can be implementation dependend. I would rather drop maxItems so the following format will be possible: feature-domains = <&etzpc 1>, <&etzpc 2>, <&some_other_domain 1 2 3 4> feature-domain-names = "firewall 1", "firewall 2", "other_domain" Also I beliewe driver will handle feature-domain-names property so it will parse feature-domains only related to the firewall. > required: > - compatible > - reg > diff --git a/Documentation/devicetree/bindings/dma/st,stm32-dma.yaml b/Documentation/devicetree/bindings/dma/st,stm32-dma.yaml > index 329847ef096a..2236ac95574b 100644 > --- a/Documentation/devicetree/bindings/dma/st,stm32-dma.yaml > +++ b/Documentation/devicetree/bindings/dma/st,stm32-dma.yaml > @@ -82,6 +82,10 @@ properties: > description: if defined, it indicates that the controller > supports memory-to-memory transfer > > + feature-domains: > + minItems: 1 > + maxItems: 3 > + > required: > - compatible > - reg > diff --git a/Documentation/devicetree/bindings/dma/st,stm32-dmamux.yaml b/Documentation/devicetree/bindings/dma/st,stm32-dmamux.yaml > index e722fbcd8a5f..47ae890f5bd9 100644 > --- a/Documentation/devicetree/bindings/dma/st,stm32-dmamux.yaml > +++ b/Documentation/devicetree/bindings/dma/st,stm32-dmamux.yaml > @@ -28,6 +28,10 @@ properties: > resets: > maxItems: 1 > > + feature-domains: > + minItems: 1 > + maxItems: 3 > + > required: > - compatible > - reg > diff --git a/Documentation/devicetree/bindings/i2c/st,stm32-i2c.yaml b/Documentation/devicetree/bindings/i2c/st,stm32-i2c.yaml > index 94b75d9f66cd..326a96741f50 100644 > --- a/Documentation/devicetree/bindings/i2c/st,stm32-i2c.yaml > +++ b/Documentation/devicetree/bindings/i2c/st,stm32-i2c.yaml > @@ -99,6 +99,10 @@ properties: > > wakeup-source: true > > + feature-domains: > + minItems: 1 > + maxItems: 3 > + > required: > - compatible > - reg > diff --git a/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml b/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml > index 995cbf8cefc6..3eb20d67f0fc 100644 > --- a/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml > +++ b/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml > @@ -93,6 +93,10 @@ properties: > '#size-cells': > const: 0 > > + feature-domains: > + minItems: 1 > + maxItems: 3 > + > allOf: > - if: > properties: > diff --git a/Documentation/devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml b/Documentation/devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml > index 1970503389aa..bc34ae172417 100644 > --- a/Documentation/devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml > +++ b/Documentation/devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml > @@ -59,6 +59,10 @@ properties: > If not, SPI CLKOUT frequency will not be accurate. > maximum: 20000000 > > + feature-domains: > + minItems: 1 > + maxItems: 3 > + > required: > - compatible > - reg > diff --git a/Documentation/devicetree/bindings/iio/dac/st,stm32-dac.yaml b/Documentation/devicetree/bindings/iio/dac/st,stm32-dac.yaml > index 04045b932bd2..90d35a2a6504 100644 > --- a/Documentation/devicetree/bindings/iio/dac/st,stm32-dac.yaml > +++ b/Documentation/devicetree/bindings/iio/dac/st,stm32-dac.yaml > @@ -45,6 +45,10 @@ properties: > '#size-cells': > const: 0 > > + feature-domains: > + minItems: 1 > + maxItems: 3 > + > additionalProperties: false > > required: > diff --git a/Documentation/devicetree/bindings/media/cec/st,stm32-cec.yaml b/Documentation/devicetree/bindings/media/cec/st,stm32-cec.yaml > index 2314a9a14650..f88e3c0e6175 100644 > --- a/Documentation/devicetree/bindings/media/cec/st,stm32-cec.yaml > +++ b/Documentation/devicetree/bindings/media/cec/st,stm32-cec.yaml > @@ -29,6 +29,10 @@ properties: > - const: cec > - const: hdmi-cec > > + feature-domains: > + minItems: 1 > + maxItems: 3 > + > required: > - compatible > - reg > diff --git a/Documentation/devicetree/bindings/media/st,stm32-dcmi.yaml b/Documentation/devicetree/bindings/media/st,stm32-dcmi.yaml > index 6b3e413cedb2..4fa1d14910df 100644 > --- a/Documentation/devicetree/bindings/media/st,stm32-dcmi.yaml > +++ b/Documentation/devicetree/bindings/media/st,stm32-dcmi.yaml > @@ -36,6 +36,10 @@ properties: > resets: > maxItems: 1 > > + feature-domains: > + minItems: 1 > + maxItems: 3 > + > port: > $ref: /schemas/graph.yaml#/$defs/port-base > unevaluatedProperties: false > diff --git a/Documentation/devicetree/bindings/memory-controllers/st,stm32-fmc2-ebi.yaml b/Documentation/devicetree/bindings/memory-controllers/st,stm32-fmc2-ebi.yaml > index 14f1833d37c9..63b3d012147b 100644 > --- a/Documentation/devicetree/bindings/memory-controllers/st,stm32-fmc2-ebi.yaml > +++ b/Documentation/devicetree/bindings/memory-controllers/st,stm32-fmc2-ebi.yaml > @@ -45,6 +45,10 @@ properties: > Reflects the memory layout with four integer values per bank. Format: > <bank-number> 0 <address of the bank> <size> > > + feature-domains: > + minItems: 1 > + maxItems: 3 > + > patternProperties: > "^.*@[0-4],[a-f0-9]+$": > additionalProperties: true > diff --git a/Documentation/devicetree/bindings/mfd/st,stm32-lptimer.yaml b/Documentation/devicetree/bindings/mfd/st,stm32-lptimer.yaml > index 27329c5dc38e..59d770544950 100644 > --- a/Documentation/devicetree/bindings/mfd/st,stm32-lptimer.yaml > +++ b/Documentation/devicetree/bindings/mfd/st,stm32-lptimer.yaml > @@ -44,6 +44,10 @@ properties: > > wakeup-source: true > > + feature-domains: > + minItems: 1 > + maxItems: 3 > + > pwm: > type: object > additionalProperties: false > diff --git a/Documentation/devicetree/bindings/mfd/st,stm32-timers.yaml b/Documentation/devicetree/bindings/mfd/st,stm32-timers.yaml > index f84e09a5743b..d5ad097e94ec 100644 > --- a/Documentation/devicetree/bindings/mfd/st,stm32-timers.yaml > +++ b/Documentation/devicetree/bindings/mfd/st,stm32-timers.yaml > @@ -67,6 +67,11 @@ properties: > "#size-cells": > const: 0 > > + feature-domains: > + minItems: 1 > + maxItems: 3 > + > + > pwm: > type: object > additionalProperties: false > diff --git a/Documentation/devicetree/bindings/mmc/arm,pl18x.yaml b/Documentation/devicetree/bindings/mmc/arm,pl18x.yaml > index 2459a55ed540..6ebedee65153 100644 > --- a/Documentation/devicetree/bindings/mmc/arm,pl18x.yaml > +++ b/Documentation/devicetree/bindings/mmc/arm,pl18x.yaml > @@ -79,6 +79,10 @@ properties: > - const: rx > - const: tx > > + feature-domains: > + minItems: 1 > + maxItems: 3 > + > power-domains: true > > resets: > diff --git a/Documentation/devicetree/bindings/net/stm32-dwmac.yaml b/Documentation/devicetree/bindings/net/stm32-dwmac.yaml > index fc8c96b08d7d..0e408dc85c13 100644 > --- a/Documentation/devicetree/bindings/net/stm32-dwmac.yaml > +++ b/Documentation/devicetree/bindings/net/stm32-dwmac.yaml > @@ -93,6 +93,10 @@ properties: > select RCC clock instead of ETH_REF_CLK. > type: boolean > > + feature-domains: > + minItems: 1 > + maxItems: 3 > + > required: > - compatible > - clocks > diff --git a/Documentation/devicetree/bindings/phy/phy-stm32-usbphyc.yaml b/Documentation/devicetree/bindings/phy/phy-stm32-usbphyc.yaml > index 24a3dbde223b..b9ac20c8bf05 100644 > --- a/Documentation/devicetree/bindings/phy/phy-stm32-usbphyc.yaml > +++ b/Documentation/devicetree/bindings/phy/phy-stm32-usbphyc.yaml > @@ -55,6 +55,10 @@ properties: > description: number of clock cells for ck_usbo_48m consumer > const: 0 > > + feature-domains: > + minItems: 1 > + maxItems: 3 > + > # Required child nodes: > > patternProperties: > diff --git a/Documentation/devicetree/bindings/regulator/st,stm32-vrefbuf.yaml b/Documentation/devicetree/bindings/regulator/st,stm32-vrefbuf.yaml > index 05f4ad2c7d3a..02cefe4ef42b 100644 > --- a/Documentation/devicetree/bindings/regulator/st,stm32-vrefbuf.yaml > +++ b/Documentation/devicetree/bindings/regulator/st,stm32-vrefbuf.yaml > @@ -30,6 +30,10 @@ properties: > vdda-supply: > description: phandle to the vdda input analog voltage. > > + feature-domains: > + minItems: 1 > + maxItems: 3 > + > required: > - compatible > - reg > diff --git a/Documentation/devicetree/bindings/rng/st,stm32-rng.yaml b/Documentation/devicetree/bindings/rng/st,stm32-rng.yaml > index 187b172d0cca..79eb5f5bd252 100644 > --- a/Documentation/devicetree/bindings/rng/st,stm32-rng.yaml > +++ b/Documentation/devicetree/bindings/rng/st,stm32-rng.yaml > @@ -30,6 +30,10 @@ properties: > type: boolean > description: If set enable the clock detection management > > + feature-domains: > + minItems: 1 > + maxItems: 3 > + > required: > - compatible > - reg > diff --git a/Documentation/devicetree/bindings/serial/st,stm32-uart.yaml b/Documentation/devicetree/bindings/serial/st,stm32-uart.yaml > index 1df8ffe95fc6..893978e7170f 100644 > --- a/Documentation/devicetree/bindings/serial/st,stm32-uart.yaml > +++ b/Documentation/devicetree/bindings/serial/st,stm32-uart.yaml > @@ -70,6 +70,10 @@ properties: > enum: [1, 2, 4, 8, 12, 14, 16] > default: 8 > > + feature-domains: > + minItems: 1 > + maxItems: 3 > + > allOf: > - $ref: rs485.yaml# > - $ref: serial.yaml# > diff --git a/Documentation/devicetree/bindings/sound/st,stm32-i2s.yaml b/Documentation/devicetree/bindings/sound/st,stm32-i2s.yaml > index b9111d375b93..64c5898e51f8 100644 > --- a/Documentation/devicetree/bindings/sound/st,stm32-i2s.yaml > +++ b/Documentation/devicetree/bindings/sound/st,stm32-i2s.yaml > @@ -65,6 +65,10 @@ properties: > $ref: audio-graph-port.yaml# > unevaluatedProperties: false > > + feature-domains: > + minItems: 1 > + maxItems: 3 > + > required: > - compatible > - "#sound-dai-cells" > diff --git a/Documentation/devicetree/bindings/sound/st,stm32-sai.yaml b/Documentation/devicetree/bindings/sound/st,stm32-sai.yaml > index 56d206f97a96..9bc08b7645dc 100644 > --- a/Documentation/devicetree/bindings/sound/st,stm32-sai.yaml > +++ b/Documentation/devicetree/bindings/sound/st,stm32-sai.yaml > @@ -48,6 +48,10 @@ properties: > clock-names: > maxItems: 3 > > + feature-domains: > + minItems: 1 > + maxItems: 3 > + > required: > - compatible > - reg > diff --git a/Documentation/devicetree/bindings/sound/st,stm32-spdifrx.yaml b/Documentation/devicetree/bindings/sound/st,stm32-spdifrx.yaml > index bc48151b9adb..f00e5db9ee3b 100644 > --- a/Documentation/devicetree/bindings/sound/st,stm32-spdifrx.yaml > +++ b/Documentation/devicetree/bindings/sound/st,stm32-spdifrx.yaml > @@ -50,6 +50,10 @@ properties: > resets: > maxItems: 1 > > + feature-domains: > + minItems: 1 > + maxItems: 3 > + > required: > - compatible > - "#sound-dai-cells" > diff --git a/Documentation/devicetree/bindings/spi/st,stm32-qspi.yaml b/Documentation/devicetree/bindings/spi/st,stm32-qspi.yaml > index 8bba965a9ae6..2ac136802467 100644 > --- a/Documentation/devicetree/bindings/spi/st,stm32-qspi.yaml > +++ b/Documentation/devicetree/bindings/spi/st,stm32-qspi.yaml > @@ -46,6 +46,10 @@ properties: > - const: tx > - const: rx > > + feature-domains: > + minItems: 1 > + maxItems: 3 > + > required: > - compatible > - reg > diff --git a/Documentation/devicetree/bindings/spi/st,stm32-spi.yaml b/Documentation/devicetree/bindings/spi/st,stm32-spi.yaml > index 9ca1a843c820..725c26daabe4 100644 > --- a/Documentation/devicetree/bindings/spi/st,stm32-spi.yaml > +++ b/Documentation/devicetree/bindings/spi/st,stm32-spi.yaml > @@ -59,6 +59,10 @@ properties: > - const: rx > - const: tx > > + feature-domains: > + minItems: 1 > + maxItems: 3 > + > patternProperties: > "^[a-zA-Z][a-zA-Z0-9,+\\-._]{0,63}@[0-9a-f]+$": > type: object > diff --git a/Documentation/devicetree/bindings/usb/dwc2.yaml b/Documentation/devicetree/bindings/usb/dwc2.yaml > index d3506090f8b1..c372caf154fc 100644 > --- a/Documentation/devicetree/bindings/usb/dwc2.yaml > +++ b/Documentation/devicetree/bindings/usb/dwc2.yaml > @@ -171,6 +171,10 @@ properties: > > tpl-support: true > > + feature-domains: > + minItems: 1 > + maxItems: 3 > + > dependencies: > port: [ usb-role-switch ] > role-switch-default-mode: [ usb-role-switch ]
On Fri, Jul 07, 2023 at 02:28:28PM +0200, Gatien CHEVALLIER wrote: > 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@foss.st.com> > > > --- > > > > > > 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. Humm, that should limit the maximum entries to 2, so 3 would never work (if RIFSC is the parent). > I picked 3 as a maxItems for our ST needs, I'll give it some more thought > when coming back with something clearer. I'd expect you have 1 entry for register bus and 1 entry for DMA bus if there is one. It's block specific for how many entries, so the RIFSC schema should not be setting that. You could possibly say that 'feature-domains' is required for all the child nodes though. Rob
On 7/7/23 16:07, Oleksii Moisieiev wrote: > > Gatien Chevallier <gatien.chevallier@foss.st.com> writes: > >> 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@foss.st.com> >> --- >> >> 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 >> + > > I beliewe feature-domains is generic binding. This means that maxItems > can be implementation dependend. I would rather drop maxItems so the > following format will be possible: > > feature-domains = <&etzpc 1>, <&etzpc 2>, <&some_other_domain 1 2 3 4> > feature-domain-names = "firewall 1", "firewall 2", "other_domain" > I'd prefer to drop the maxItems as well. I've been told at one point in the first series to choose a number for this maybe picking a high but reasonnable number is preferrable. Based on How to Get Your DT Schema Bindings Accepted in Less than 10 Iterations pdf published by Krzysztof, I see that examples use minItems/maxItems. But I can't find if it's mandatory > Also I beliewe driver will handle feature-domain-names property so it > will parse feature-domains only related to the firewall. > Yep, in case of multiple feature-domains, it could be nice. >> required: >> - compatible >> - reg >> diff --git a/Documentation/devicetree/bindings/dma/st,stm32-dma.yaml b/Documentation/devicetree/bindings/dma/st,stm32-dma.yaml >> index 329847ef096a..2236ac95574b 100644 >> --- a/Documentation/devicetree/bindings/dma/st,stm32-dma.yaml >> +++ b/Documentation/devicetree/bindings/dma/st,stm32-dma.yaml >> @@ -82,6 +82,10 @@ properties: >> description: if defined, it indicates that the controller >> supports memory-to-memory transfer >> >> + feature-domains: >> + minItems: 1 >> + maxItems: 3 >> + >> required: >> - compatible >> - reg >> diff --git a/Documentation/devicetree/bindings/dma/st,stm32-dmamux.yaml b/Documentation/devicetree/bindings/dma/st,stm32-dmamux.yaml >> index e722fbcd8a5f..47ae890f5bd9 100644 >> --- a/Documentation/devicetree/bindings/dma/st,stm32-dmamux.yaml >> +++ b/Documentation/devicetree/bindings/dma/st,stm32-dmamux.yaml >> @@ -28,6 +28,10 @@ properties: >> resets: >> maxItems: 1 >> >> + feature-domains: >> + minItems: 1 >> + maxItems: 3 >> + >> required: >> - compatible >> - reg >> diff --git a/Documentation/devicetree/bindings/i2c/st,stm32-i2c.yaml b/Documentation/devicetree/bindings/i2c/st,stm32-i2c.yaml >> index 94b75d9f66cd..326a96741f50 100644 >> --- a/Documentation/devicetree/bindings/i2c/st,stm32-i2c.yaml >> +++ b/Documentation/devicetree/bindings/i2c/st,stm32-i2c.yaml >> @@ -99,6 +99,10 @@ properties: >> >> wakeup-source: true >> >> + feature-domains: >> + minItems: 1 >> + maxItems: 3 >> + >> required: >> - compatible >> - reg >> diff --git a/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml b/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml >> index 995cbf8cefc6..3eb20d67f0fc 100644 >> --- a/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml >> +++ b/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml >> @@ -93,6 +93,10 @@ properties: >> '#size-cells': >> const: 0 >> >> + feature-domains: >> + minItems: 1 >> + maxItems: 3 >> + >> allOf: >> - if: >> properties: >> diff --git a/Documentation/devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml b/Documentation/devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml >> index 1970503389aa..bc34ae172417 100644 >> --- a/Documentation/devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml >> +++ b/Documentation/devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml >> @@ -59,6 +59,10 @@ properties: >> If not, SPI CLKOUT frequency will not be accurate. >> maximum: 20000000 >> >> + feature-domains: >> + minItems: 1 >> + maxItems: 3 >> + >> required: >> - compatible >> - reg >> diff --git a/Documentation/devicetree/bindings/iio/dac/st,stm32-dac.yaml b/Documentation/devicetree/bindings/iio/dac/st,stm32-dac.yaml >> index 04045b932bd2..90d35a2a6504 100644 >> --- a/Documentation/devicetree/bindings/iio/dac/st,stm32-dac.yaml >> +++ b/Documentation/devicetree/bindings/iio/dac/st,stm32-dac.yaml >> @@ -45,6 +45,10 @@ properties: >> '#size-cells': >> const: 0 >> >> + feature-domains: >> + minItems: 1 >> + maxItems: 3 >> + >> additionalProperties: false >> >> required: >> diff --git a/Documentation/devicetree/bindings/media/cec/st,stm32-cec.yaml b/Documentation/devicetree/bindings/media/cec/st,stm32-cec.yaml >> index 2314a9a14650..f88e3c0e6175 100644 >> --- a/Documentation/devicetree/bindings/media/cec/st,stm32-cec.yaml >> +++ b/Documentation/devicetree/bindings/media/cec/st,stm32-cec.yaml >> @@ -29,6 +29,10 @@ properties: >> - const: cec >> - const: hdmi-cec >> >> + feature-domains: >> + minItems: 1 >> + maxItems: 3 >> + >> required: >> - compatible >> - reg >> diff --git a/Documentation/devicetree/bindings/media/st,stm32-dcmi.yaml b/Documentation/devicetree/bindings/media/st,stm32-dcmi.yaml >> index 6b3e413cedb2..4fa1d14910df 100644 >> --- a/Documentation/devicetree/bindings/media/st,stm32-dcmi.yaml >> +++ b/Documentation/devicetree/bindings/media/st,stm32-dcmi.yaml >> @@ -36,6 +36,10 @@ properties: >> resets: >> maxItems: 1 >> >> + feature-domains: >> + minItems: 1 >> + maxItems: 3 >> + >> port: >> $ref: /schemas/graph.yaml#/$defs/port-base >> unevaluatedProperties: false >> diff --git a/Documentation/devicetree/bindings/memory-controllers/st,stm32-fmc2-ebi.yaml b/Documentation/devicetree/bindings/memory-controllers/st,stm32-fmc2-ebi.yaml >> index 14f1833d37c9..63b3d012147b 100644 >> --- a/Documentation/devicetree/bindings/memory-controllers/st,stm32-fmc2-ebi.yaml >> +++ b/Documentation/devicetree/bindings/memory-controllers/st,stm32-fmc2-ebi.yaml >> @@ -45,6 +45,10 @@ properties: >> Reflects the memory layout with four integer values per bank. Format: >> <bank-number> 0 <address of the bank> <size> >> >> + feature-domains: >> + minItems: 1 >> + maxItems: 3 >> + >> patternProperties: >> "^.*@[0-4],[a-f0-9]+$": >> additionalProperties: true >> diff --git a/Documentation/devicetree/bindings/mfd/st,stm32-lptimer.yaml b/Documentation/devicetree/bindings/mfd/st,stm32-lptimer.yaml >> index 27329c5dc38e..59d770544950 100644 >> --- a/Documentation/devicetree/bindings/mfd/st,stm32-lptimer.yaml >> +++ b/Documentation/devicetree/bindings/mfd/st,stm32-lptimer.yaml >> @@ -44,6 +44,10 @@ properties: >> >> wakeup-source: true >> >> + feature-domains: >> + minItems: 1 >> + maxItems: 3 >> + >> pwm: >> type: object >> additionalProperties: false >> diff --git a/Documentation/devicetree/bindings/mfd/st,stm32-timers.yaml b/Documentation/devicetree/bindings/mfd/st,stm32-timers.yaml >> index f84e09a5743b..d5ad097e94ec 100644 >> --- a/Documentation/devicetree/bindings/mfd/st,stm32-timers.yaml >> +++ b/Documentation/devicetree/bindings/mfd/st,stm32-timers.yaml >> @@ -67,6 +67,11 @@ properties: >> "#size-cells": >> const: 0 >> >> + feature-domains: >> + minItems: 1 >> + maxItems: 3 >> + >> + >> pwm: >> type: object >> additionalProperties: false >> diff --git a/Documentation/devicetree/bindings/mmc/arm,pl18x.yaml b/Documentation/devicetree/bindings/mmc/arm,pl18x.yaml >> index 2459a55ed540..6ebedee65153 100644 >> --- a/Documentation/devicetree/bindings/mmc/arm,pl18x.yaml >> +++ b/Documentation/devicetree/bindings/mmc/arm,pl18x.yaml >> @@ -79,6 +79,10 @@ properties: >> - const: rx >> - const: tx >> >> + feature-domains: >> + minItems: 1 >> + maxItems: 3 >> + >> power-domains: true >> >> resets: >> diff --git a/Documentation/devicetree/bindings/net/stm32-dwmac.yaml b/Documentation/devicetree/bindings/net/stm32-dwmac.yaml >> index fc8c96b08d7d..0e408dc85c13 100644 >> --- a/Documentation/devicetree/bindings/net/stm32-dwmac.yaml >> +++ b/Documentation/devicetree/bindings/net/stm32-dwmac.yaml >> @@ -93,6 +93,10 @@ properties: >> select RCC clock instead of ETH_REF_CLK. >> type: boolean >> >> + feature-domains: >> + minItems: 1 >> + maxItems: 3 >> + >> required: >> - compatible >> - clocks >> diff --git a/Documentation/devicetree/bindings/phy/phy-stm32-usbphyc.yaml b/Documentation/devicetree/bindings/phy/phy-stm32-usbphyc.yaml >> index 24a3dbde223b..b9ac20c8bf05 100644 >> --- a/Documentation/devicetree/bindings/phy/phy-stm32-usbphyc.yaml >> +++ b/Documentation/devicetree/bindings/phy/phy-stm32-usbphyc.yaml >> @@ -55,6 +55,10 @@ properties: >> description: number of clock cells for ck_usbo_48m consumer >> const: 0 >> >> + feature-domains: >> + minItems: 1 >> + maxItems: 3 >> + >> # Required child nodes: >> >> patternProperties: >> diff --git a/Documentation/devicetree/bindings/regulator/st,stm32-vrefbuf.yaml b/Documentation/devicetree/bindings/regulator/st,stm32-vrefbuf.yaml >> index 05f4ad2c7d3a..02cefe4ef42b 100644 >> --- a/Documentation/devicetree/bindings/regulator/st,stm32-vrefbuf.yaml >> +++ b/Documentation/devicetree/bindings/regulator/st,stm32-vrefbuf.yaml >> @@ -30,6 +30,10 @@ properties: >> vdda-supply: >> description: phandle to the vdda input analog voltage. >> >> + feature-domains: >> + minItems: 1 >> + maxItems: 3 >> + >> required: >> - compatible >> - reg >> diff --git a/Documentation/devicetree/bindings/rng/st,stm32-rng.yaml b/Documentation/devicetree/bindings/rng/st,stm32-rng.yaml >> index 187b172d0cca..79eb5f5bd252 100644 >> --- a/Documentation/devicetree/bindings/rng/st,stm32-rng.yaml >> +++ b/Documentation/devicetree/bindings/rng/st,stm32-rng.yaml >> @@ -30,6 +30,10 @@ properties: >> type: boolean >> description: If set enable the clock detection management >> >> + feature-domains: >> + minItems: 1 >> + maxItems: 3 >> + >> required: >> - compatible >> - reg >> diff --git a/Documentation/devicetree/bindings/serial/st,stm32-uart.yaml b/Documentation/devicetree/bindings/serial/st,stm32-uart.yaml >> index 1df8ffe95fc6..893978e7170f 100644 >> --- a/Documentation/devicetree/bindings/serial/st,stm32-uart.yaml >> +++ b/Documentation/devicetree/bindings/serial/st,stm32-uart.yaml >> @@ -70,6 +70,10 @@ properties: >> enum: [1, 2, 4, 8, 12, 14, 16] >> default: 8 >> >> + feature-domains: >> + minItems: 1 >> + maxItems: 3 >> + >> allOf: >> - $ref: rs485.yaml# >> - $ref: serial.yaml# >> diff --git a/Documentation/devicetree/bindings/sound/st,stm32-i2s.yaml b/Documentation/devicetree/bindings/sound/st,stm32-i2s.yaml >> index b9111d375b93..64c5898e51f8 100644 >> --- a/Documentation/devicetree/bindings/sound/st,stm32-i2s.yaml >> +++ b/Documentation/devicetree/bindings/sound/st,stm32-i2s.yaml >> @@ -65,6 +65,10 @@ properties: >> $ref: audio-graph-port.yaml# >> unevaluatedProperties: false >> >> + feature-domains: >> + minItems: 1 >> + maxItems: 3 >> + >> required: >> - compatible >> - "#sound-dai-cells" >> diff --git a/Documentation/devicetree/bindings/sound/st,stm32-sai.yaml b/Documentation/devicetree/bindings/sound/st,stm32-sai.yaml >> index 56d206f97a96..9bc08b7645dc 100644 >> --- a/Documentation/devicetree/bindings/sound/st,stm32-sai.yaml >> +++ b/Documentation/devicetree/bindings/sound/st,stm32-sai.yaml >> @@ -48,6 +48,10 @@ properties: >> clock-names: >> maxItems: 3 >> >> + feature-domains: >> + minItems: 1 >> + maxItems: 3 >> + >> required: >> - compatible >> - reg >> diff --git a/Documentation/devicetree/bindings/sound/st,stm32-spdifrx.yaml b/Documentation/devicetree/bindings/sound/st,stm32-spdifrx.yaml >> index bc48151b9adb..f00e5db9ee3b 100644 >> --- a/Documentation/devicetree/bindings/sound/st,stm32-spdifrx.yaml >> +++ b/Documentation/devicetree/bindings/sound/st,stm32-spdifrx.yaml >> @@ -50,6 +50,10 @@ properties: >> resets: >> maxItems: 1 >> >> + feature-domains: >> + minItems: 1 >> + maxItems: 3 >> + >> required: >> - compatible >> - "#sound-dai-cells" >> diff --git a/Documentation/devicetree/bindings/spi/st,stm32-qspi.yaml b/Documentation/devicetree/bindings/spi/st,stm32-qspi.yaml >> index 8bba965a9ae6..2ac136802467 100644 >> --- a/Documentation/devicetree/bindings/spi/st,stm32-qspi.yaml >> +++ b/Documentation/devicetree/bindings/spi/st,stm32-qspi.yaml >> @@ -46,6 +46,10 @@ properties: >> - const: tx >> - const: rx >> >> + feature-domains: >> + minItems: 1 >> + maxItems: 3 >> + >> required: >> - compatible >> - reg >> diff --git a/Documentation/devicetree/bindings/spi/st,stm32-spi.yaml b/Documentation/devicetree/bindings/spi/st,stm32-spi.yaml >> index 9ca1a843c820..725c26daabe4 100644 >> --- a/Documentation/devicetree/bindings/spi/st,stm32-spi.yaml >> +++ b/Documentation/devicetree/bindings/spi/st,stm32-spi.yaml >> @@ -59,6 +59,10 @@ properties: >> - const: rx >> - const: tx >> >> + feature-domains: >> + minItems: 1 >> + maxItems: 3 >> + >> patternProperties: >> "^[a-zA-Z][a-zA-Z0-9,+\\-._]{0,63}@[0-9a-f]+$": >> type: object >> diff --git a/Documentation/devicetree/bindings/usb/dwc2.yaml b/Documentation/devicetree/bindings/usb/dwc2.yaml >> index d3506090f8b1..c372caf154fc 100644 >> --- a/Documentation/devicetree/bindings/usb/dwc2.yaml >> +++ b/Documentation/devicetree/bindings/usb/dwc2.yaml >> @@ -171,6 +171,10 @@ properties: >> >> tpl-support: true >> >> + feature-domains: >> + minItems: 1 >> + maxItems: 3 >> + >> dependencies: >> port: [ usb-role-switch ] >> role-switch-default-mode: [ usb-role-switch ] > >
On Fri, Jul 07, 2023 at 02:07:18PM +0000, Oleksii Moisieiev wrote: > > Gatien Chevallier <gatien.chevallier@foss.st.com> writes: > > > 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@foss.st.com> > > --- > > > > 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 > > + > > I beliewe feature-domains is generic binding. This means that maxItems > can be implementation dependend. I would rather drop maxItems so the > following format will be possible: > > feature-domains = <&etzpc 1>, <&etzpc 2>, <&some_other_domain 1 2 3 4> > feature-domain-names = "firewall 1", "firewall 2", "other_domain" The above already allows this (not -names, but the number of entries). > > Also I beliewe driver will handle feature-domain-names property so it > will parse feature-domains only related to the firewall. Now I'm curious. What's an example that's not a firewall? (Note I'm still not happy with the naming of 'feature' as anything is a feature, but that's the least of the issues really.) Rob
Hi Rob, Rob Herring <robh@kernel.org> writes: > On Fri, Jul 07, 2023 at 02:07:18PM +0000, Oleksii Moisieiev wrote: >> >> Gatien Chevallier <gatien.chevallier@foss.st.com> writes: >> >> > 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@foss.st.com> >> > --- >> > >> > 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 >> > + >> >> I beliewe feature-domains is generic binding. This means that maxItems >> can be implementation dependend. I would rather drop maxItems so the >> following format will be possible: >> >> feature-domains = <&etzpc 1>, <&etzpc 2>, <&some_other_domain 1 2 3 4> >> feature-domain-names = "firewall 1", "firewall 2", "other_domain" > > The above already allows this (not -names, but the number of entries). >> >> Also I beliewe driver will handle feature-domain-names property so it >> will parse feature-domains only related to the firewall. > > Now I'm curious. What's an example that's not a firewall? > > (Note I'm still not happy with the naming of 'feature' as anything is a > feature, but that's the least of the issues really.) > The alternative usages of feature-domains was originally proposed by me here: https://lore.kernel.org/lkml/c869d2751125181a55bc8a88c96e3a892b42f37a.1668070216.git.oleksii_moisieiev@epam.com/ Also I remember Peng Fan also was interested in those bindings. I think the use-case when one node is protected by firewall and also is controlled by scmi feature-domain-controller (As was proposed in my patch series) may take place. As for the naming maybe you have some thoughts about better name?
On Fri, Jul 7, 2023 at 10:10 AM Oleksii Moisieiev <Oleksii_Moisieiev@epam.com> wrote: > > > Hi Rob, > > Rob Herring <robh@kernel.org> writes: > > > On Fri, Jul 07, 2023 at 02:07:18PM +0000, Oleksii Moisieiev wrote: > >> > >> Gatien Chevallier <gatien.chevallier@foss.st.com> writes: > >> > >> > 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@foss.st.com> > >> > --- > >> > > >> > 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 > >> > + > >> > >> I beliewe feature-domains is generic binding. This means that maxItems > >> can be implementation dependend. I would rather drop maxItems so the > >> following format will be possible: > >> > >> feature-domains = <&etzpc 1>, <&etzpc 2>, <&some_other_domain 1 2 3 4> > >> feature-domain-names = "firewall 1", "firewall 2", "other_domain" > > > > The above already allows this (not -names, but the number of entries). > >> > >> Also I beliewe driver will handle feature-domain-names property so it > >> will parse feature-domains only related to the firewall. > > > > Now I'm curious. What's an example that's not a firewall? > > > > (Note I'm still not happy with the naming of 'feature' as anything is a > > feature, but that's the least of the issues really.) > > > > The alternative usages of feature-domains was originally proposed by me > here: > https://lore.kernel.org/lkml/c869d2751125181a55bc8a88c96e3a892b42f37a.1668070216.git.oleksii_moisieiev@epam.com/ > > Also I remember Peng Fan also was interested in those bindings. It helps to Cc people when you talk about them. If the parties interested in this want to see progress on this, you all must work together and show this is a solution for multiple platforms. > I think the use-case when one node is protected by firewall and also is > controlled by scmi feature-domain-controller (As was proposed in my > patch series) may take place. But isn't the SCMI device protection interface the same thing? Some interface to say "can I access this device?" and/or control access to it. The other possible use I'm aware of is system partitioning. OpenAMP or similar where an SoC is partitioned into multiple OS instances and peripherals are assigned to different partitions. > As for the naming maybe you have some thoughts about better name? If I did, I would have. Something with 'access' in it is as far as I've gotten. Rob
Hi Rob, Rob Herring <robh@kernel.org> writes: > On Fri, Jul 7, 2023 at 10:10 AM Oleksii Moisieiev > <Oleksii_Moisieiev@epam.com> wrote: >> >> >> Hi Rob, >> >> Rob Herring <robh@kernel.org> writes: >> >> > On Fri, Jul 07, 2023 at 02:07:18PM +0000, Oleksii Moisieiev wrote: >> >> >> >> Gatien Chevallier <gatien.chevallier@foss.st.com> writes: >> >> >> >> > 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@foss.st.com> >> >> > --- >> >> > >> >> > 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 >> >> > + >> >> >> >> I beliewe feature-domains is generic binding. This means that maxItems >> >> can be implementation dependend. I would rather drop maxItems so the >> >> following format will be possible: >> >> >> >> feature-domains = <&etzpc 1>, <&etzpc 2>, <&some_other_domain 1 2 3 4> >> >> feature-domain-names = "firewall 1", "firewall 2", "other_domain" >> > >> > The above already allows this (not -names, but the number of entries). >> >> >> >> Also I beliewe driver will handle feature-domain-names property so it >> >> will parse feature-domains only related to the firewall. >> > >> > Now I'm curious. What's an example that's not a firewall? >> > >> > (Note I'm still not happy with the naming of 'feature' as anything is a >> > feature, but that's the least of the issues really.) >> > >> >> The alternative usages of feature-domains was originally proposed by me >> here: >> https://urldefense.com/v3/__https://lore.kernel.org/lkml/c869d2751125181a55bc8a88c96e3a892b42f37a.1668070216.git.oleksii_moisieiev@epam.com/__;!!GF_29dbcQIUBPA!zHa-1LhJFYFC4vvB1OiKWJ8-NXvcBXNhsZuVi-Xnb4L6bNUC4yp8tSyIUMcKjyrEA5_UmIqNwUwCvMxNng$ [lore[.]kernel[.]org] >> >> Also I remember Peng Fan also was interested in those bindings. > > It helps to Cc people when you talk about them. > > If the parties interested in this want to see progress on this, you > all must work together and show this is a solution for multiple > platforms. > Sorry. Missed that. >> I think the use-case when one node is protected by firewall and also is >> controlled by scmi feature-domain-controller (As was proposed in my >> patch series) may take place. > > But isn't the SCMI device protection interface the same thing? Some > interface to say "can I access this device?" and/or control access to > it. My patch series introduce using of the feature-domain-controller to control access for the devices across the VMs. In our case we have virtualized system, controlled by Xen hypervisor with some amount of the VMs called Domains. Each Domain is the complete OS and HW is separated between them. We need SCMI to control power/clocks/resets etc from each Domain. feature-domain-controller in our case is SCMI node and each device will receive unique ID which match the ID from SCP firmware to set the device permissions so the SCP will know whether to give access to the resource for the Domain or not. Those feature-domain-controller node and links from devices will be processed by Xen during Domain creation and the DT for the Domain will be generated without those bindings. So probably Firewall will not see the scmi related links in the device nodes, but I think that the case when the device node is linked to more then one feature-domain-controller is possible because bindings are generic. > > The other possible use I'm aware of is system partitioning. OpenAMP or > similar where an SoC is partitioned into multiple OS instances and > peripherals are assigned to different partitions. > i.MX boards are also have system partitioning concept in SCFW. >> As for the naming maybe you have some thoughts about better name? > > If I did, I would have. Something with 'access' in it is as far as I've gotten. > What do you think about access-controller or feature-access-controller? Maybe it will fit better here if all potential use cases for this bingins is to say if the device can be accessed.
On 7/7/23 17:20, Rob Herring wrote: > On Fri, Jul 07, 2023 at 02:28:28PM +0200, Gatien CHEVALLIER wrote: >> 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@foss.st.com> >>>> --- >>>> >>>> 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. > > Humm, that should limit the maximum entries to 2, so 3 would never work > (if RIFSC is the parent). > >> I picked 3 as a maxItems for our ST needs, I'll give it some more thought >> when coming back with something clearer. > > I'd expect you have 1 entry for register bus and 1 entry for DMA bus if > there is one. It's block specific for how many entries, so the RIFSC > schema should not be setting that. You could possibly say that > 'feature-domains' is required for all the child nodes though. Ok, I will change to not specifying the number of entries in the RIFSC YAML file for V2. > > Rob Some hardware blocks may have a firewall ID for their device part and another ID for their master part as well. In the end, the number of entries could very well vary between different platforms. And the YAML files are common to these platforms. This property could be used for "extra" arguments as well, that are not firewall IDs. What do you suggest between picking a high maxItems value that would (hopefully) cover all cases and not specifying maxItems at all? Or maybe another property dedicated to such arguments? Best regards, Gatien
On Mon, Jul 10, 2023 at 10:22:10AM +0200, Gatien CHEVALLIER wrote: > > > On 7/7/23 17:20, Rob Herring wrote: > > On Fri, Jul 07, 2023 at 02:28:28PM +0200, Gatien CHEVALLIER wrote: > > > 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@foss.st.com> > > > > > --- > > > > > > > > > > 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. > > > > Humm, that should limit the maximum entries to 2, so 3 would never work > > (if RIFSC is the parent). > > > > > I picked 3 as a maxItems for our ST needs, I'll give it some more thought > > > when coming back with something clearer. > > > > I'd expect you have 1 entry for register bus and 1 entry for DMA bus if > > there is one. It's block specific for how many entries, so the RIFSC > > schema should not be setting that. You could possibly say that > > 'feature-domains' is required for all the child nodes though. > > Ok, I will change to not specifying the number of entries in the > RIFSC YAML file for V2. > > > > > Rob > Some hardware blocks may have a firewall ID for their device part and > another ID for their master part as well. In the end, the number of > entries could very well vary between different platforms. And the YAML > files are common to these platforms. A given device has a fixed number of buses. Usually 1 or 2. That does *not* vary by platform (unless the device is modified). You could have the same firewall controller and id for multiple buses, but that should not change the number of entries for the device. Now maybe a bus has no firewall on some platforms. In that case, you should make the optional firewall entry the last one, have a null phandle (0 or -1), or use -names to distinguish the entries. > This property could be used for "extra" arguments as well, that are not > firewall IDs. The arg cells are dictated by the provider and opaque to the client. > What do you suggest between picking a high maxItems value that would > (hopefully) cover all cases and not specifying maxItems at all? Or maybe > another property dedicated to such arguments? You should not specify maxItems in the firewall controller binding. Rob
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 + required: - compatible - reg diff --git a/Documentation/devicetree/bindings/dma/st,stm32-dma.yaml b/Documentation/devicetree/bindings/dma/st,stm32-dma.yaml index 329847ef096a..2236ac95574b 100644 --- a/Documentation/devicetree/bindings/dma/st,stm32-dma.yaml +++ b/Documentation/devicetree/bindings/dma/st,stm32-dma.yaml @@ -82,6 +82,10 @@ properties: description: if defined, it indicates that the controller supports memory-to-memory transfer + feature-domains: + minItems: 1 + maxItems: 3 + required: - compatible - reg diff --git a/Documentation/devicetree/bindings/dma/st,stm32-dmamux.yaml b/Documentation/devicetree/bindings/dma/st,stm32-dmamux.yaml index e722fbcd8a5f..47ae890f5bd9 100644 --- a/Documentation/devicetree/bindings/dma/st,stm32-dmamux.yaml +++ b/Documentation/devicetree/bindings/dma/st,stm32-dmamux.yaml @@ -28,6 +28,10 @@ properties: resets: maxItems: 1 + feature-domains: + minItems: 1 + maxItems: 3 + required: - compatible - reg diff --git a/Documentation/devicetree/bindings/i2c/st,stm32-i2c.yaml b/Documentation/devicetree/bindings/i2c/st,stm32-i2c.yaml index 94b75d9f66cd..326a96741f50 100644 --- a/Documentation/devicetree/bindings/i2c/st,stm32-i2c.yaml +++ b/Documentation/devicetree/bindings/i2c/st,stm32-i2c.yaml @@ -99,6 +99,10 @@ properties: wakeup-source: true + feature-domains: + minItems: 1 + maxItems: 3 + required: - compatible - reg diff --git a/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml b/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml index 995cbf8cefc6..3eb20d67f0fc 100644 --- a/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml +++ b/Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml @@ -93,6 +93,10 @@ properties: '#size-cells': const: 0 + feature-domains: + minItems: 1 + maxItems: 3 + allOf: - if: properties: diff --git a/Documentation/devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml b/Documentation/devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml index 1970503389aa..bc34ae172417 100644 --- a/Documentation/devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml +++ b/Documentation/devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml @@ -59,6 +59,10 @@ properties: If not, SPI CLKOUT frequency will not be accurate. maximum: 20000000 + feature-domains: + minItems: 1 + maxItems: 3 + required: - compatible - reg diff --git a/Documentation/devicetree/bindings/iio/dac/st,stm32-dac.yaml b/Documentation/devicetree/bindings/iio/dac/st,stm32-dac.yaml index 04045b932bd2..90d35a2a6504 100644 --- a/Documentation/devicetree/bindings/iio/dac/st,stm32-dac.yaml +++ b/Documentation/devicetree/bindings/iio/dac/st,stm32-dac.yaml @@ -45,6 +45,10 @@ properties: '#size-cells': const: 0 + feature-domains: + minItems: 1 + maxItems: 3 + additionalProperties: false required: diff --git a/Documentation/devicetree/bindings/media/cec/st,stm32-cec.yaml b/Documentation/devicetree/bindings/media/cec/st,stm32-cec.yaml index 2314a9a14650..f88e3c0e6175 100644 --- a/Documentation/devicetree/bindings/media/cec/st,stm32-cec.yaml +++ b/Documentation/devicetree/bindings/media/cec/st,stm32-cec.yaml @@ -29,6 +29,10 @@ properties: - const: cec - const: hdmi-cec + feature-domains: + minItems: 1 + maxItems: 3 + required: - compatible - reg diff --git a/Documentation/devicetree/bindings/media/st,stm32-dcmi.yaml b/Documentation/devicetree/bindings/media/st,stm32-dcmi.yaml index 6b3e413cedb2..4fa1d14910df 100644 --- a/Documentation/devicetree/bindings/media/st,stm32-dcmi.yaml +++ b/Documentation/devicetree/bindings/media/st,stm32-dcmi.yaml @@ -36,6 +36,10 @@ properties: resets: maxItems: 1 + feature-domains: + minItems: 1 + maxItems: 3 + port: $ref: /schemas/graph.yaml#/$defs/port-base unevaluatedProperties: false diff --git a/Documentation/devicetree/bindings/memory-controllers/st,stm32-fmc2-ebi.yaml b/Documentation/devicetree/bindings/memory-controllers/st,stm32-fmc2-ebi.yaml index 14f1833d37c9..63b3d012147b 100644 --- a/Documentation/devicetree/bindings/memory-controllers/st,stm32-fmc2-ebi.yaml +++ b/Documentation/devicetree/bindings/memory-controllers/st,stm32-fmc2-ebi.yaml @@ -45,6 +45,10 @@ properties: Reflects the memory layout with four integer values per bank. Format: <bank-number> 0 <address of the bank> <size> + feature-domains: + minItems: 1 + maxItems: 3 + patternProperties: "^.*@[0-4],[a-f0-9]+$": additionalProperties: true diff --git a/Documentation/devicetree/bindings/mfd/st,stm32-lptimer.yaml b/Documentation/devicetree/bindings/mfd/st,stm32-lptimer.yaml index 27329c5dc38e..59d770544950 100644 --- a/Documentation/devicetree/bindings/mfd/st,stm32-lptimer.yaml +++ b/Documentation/devicetree/bindings/mfd/st,stm32-lptimer.yaml @@ -44,6 +44,10 @@ properties: wakeup-source: true + feature-domains: + minItems: 1 + maxItems: 3 + pwm: type: object additionalProperties: false diff --git a/Documentation/devicetree/bindings/mfd/st,stm32-timers.yaml b/Documentation/devicetree/bindings/mfd/st,stm32-timers.yaml index f84e09a5743b..d5ad097e94ec 100644 --- a/Documentation/devicetree/bindings/mfd/st,stm32-timers.yaml +++ b/Documentation/devicetree/bindings/mfd/st,stm32-timers.yaml @@ -67,6 +67,11 @@ properties: "#size-cells": const: 0 + feature-domains: + minItems: 1 + maxItems: 3 + + pwm: type: object additionalProperties: false diff --git a/Documentation/devicetree/bindings/mmc/arm,pl18x.yaml b/Documentation/devicetree/bindings/mmc/arm,pl18x.yaml index 2459a55ed540..6ebedee65153 100644 --- a/Documentation/devicetree/bindings/mmc/arm,pl18x.yaml +++ b/Documentation/devicetree/bindings/mmc/arm,pl18x.yaml @@ -79,6 +79,10 @@ properties: - const: rx - const: tx + feature-domains: + minItems: 1 + maxItems: 3 + power-domains: true resets: diff --git a/Documentation/devicetree/bindings/net/stm32-dwmac.yaml b/Documentation/devicetree/bindings/net/stm32-dwmac.yaml index fc8c96b08d7d..0e408dc85c13 100644 --- a/Documentation/devicetree/bindings/net/stm32-dwmac.yaml +++ b/Documentation/devicetree/bindings/net/stm32-dwmac.yaml @@ -93,6 +93,10 @@ properties: select RCC clock instead of ETH_REF_CLK. type: boolean + feature-domains: + minItems: 1 + maxItems: 3 + required: - compatible - clocks diff --git a/Documentation/devicetree/bindings/phy/phy-stm32-usbphyc.yaml b/Documentation/devicetree/bindings/phy/phy-stm32-usbphyc.yaml index 24a3dbde223b..b9ac20c8bf05 100644 --- a/Documentation/devicetree/bindings/phy/phy-stm32-usbphyc.yaml +++ b/Documentation/devicetree/bindings/phy/phy-stm32-usbphyc.yaml @@ -55,6 +55,10 @@ properties: description: number of clock cells for ck_usbo_48m consumer const: 0 + feature-domains: + minItems: 1 + maxItems: 3 + # Required child nodes: patternProperties: diff --git a/Documentation/devicetree/bindings/regulator/st,stm32-vrefbuf.yaml b/Documentation/devicetree/bindings/regulator/st,stm32-vrefbuf.yaml index 05f4ad2c7d3a..02cefe4ef42b 100644 --- a/Documentation/devicetree/bindings/regulator/st,stm32-vrefbuf.yaml +++ b/Documentation/devicetree/bindings/regulator/st,stm32-vrefbuf.yaml @@ -30,6 +30,10 @@ properties: vdda-supply: description: phandle to the vdda input analog voltage. + feature-domains: + minItems: 1 + maxItems: 3 + required: - compatible - reg diff --git a/Documentation/devicetree/bindings/rng/st,stm32-rng.yaml b/Documentation/devicetree/bindings/rng/st,stm32-rng.yaml index 187b172d0cca..79eb5f5bd252 100644 --- a/Documentation/devicetree/bindings/rng/st,stm32-rng.yaml +++ b/Documentation/devicetree/bindings/rng/st,stm32-rng.yaml @@ -30,6 +30,10 @@ properties: type: boolean description: If set enable the clock detection management + feature-domains: + minItems: 1 + maxItems: 3 + required: - compatible - reg diff --git a/Documentation/devicetree/bindings/serial/st,stm32-uart.yaml b/Documentation/devicetree/bindings/serial/st,stm32-uart.yaml index 1df8ffe95fc6..893978e7170f 100644 --- a/Documentation/devicetree/bindings/serial/st,stm32-uart.yaml +++ b/Documentation/devicetree/bindings/serial/st,stm32-uart.yaml @@ -70,6 +70,10 @@ properties: enum: [1, 2, 4, 8, 12, 14, 16] default: 8 + feature-domains: + minItems: 1 + maxItems: 3 + allOf: - $ref: rs485.yaml# - $ref: serial.yaml# diff --git a/Documentation/devicetree/bindings/sound/st,stm32-i2s.yaml b/Documentation/devicetree/bindings/sound/st,stm32-i2s.yaml index b9111d375b93..64c5898e51f8 100644 --- a/Documentation/devicetree/bindings/sound/st,stm32-i2s.yaml +++ b/Documentation/devicetree/bindings/sound/st,stm32-i2s.yaml @@ -65,6 +65,10 @@ properties: $ref: audio-graph-port.yaml# unevaluatedProperties: false + feature-domains: + minItems: 1 + maxItems: 3 + required: - compatible - "#sound-dai-cells" diff --git a/Documentation/devicetree/bindings/sound/st,stm32-sai.yaml b/Documentation/devicetree/bindings/sound/st,stm32-sai.yaml index 56d206f97a96..9bc08b7645dc 100644 --- a/Documentation/devicetree/bindings/sound/st,stm32-sai.yaml +++ b/Documentation/devicetree/bindings/sound/st,stm32-sai.yaml @@ -48,6 +48,10 @@ properties: clock-names: maxItems: 3 + feature-domains: + minItems: 1 + maxItems: 3 + required: - compatible - reg diff --git a/Documentation/devicetree/bindings/sound/st,stm32-spdifrx.yaml b/Documentation/devicetree/bindings/sound/st,stm32-spdifrx.yaml index bc48151b9adb..f00e5db9ee3b 100644 --- a/Documentation/devicetree/bindings/sound/st,stm32-spdifrx.yaml +++ b/Documentation/devicetree/bindings/sound/st,stm32-spdifrx.yaml @@ -50,6 +50,10 @@ properties: resets: maxItems: 1 + feature-domains: + minItems: 1 + maxItems: 3 + required: - compatible - "#sound-dai-cells" diff --git a/Documentation/devicetree/bindings/spi/st,stm32-qspi.yaml b/Documentation/devicetree/bindings/spi/st,stm32-qspi.yaml index 8bba965a9ae6..2ac136802467 100644 --- a/Documentation/devicetree/bindings/spi/st,stm32-qspi.yaml +++ b/Documentation/devicetree/bindings/spi/st,stm32-qspi.yaml @@ -46,6 +46,10 @@ properties: - const: tx - const: rx + feature-domains: + minItems: 1 + maxItems: 3 + required: - compatible - reg diff --git a/Documentation/devicetree/bindings/spi/st,stm32-spi.yaml b/Documentation/devicetree/bindings/spi/st,stm32-spi.yaml index 9ca1a843c820..725c26daabe4 100644 --- a/Documentation/devicetree/bindings/spi/st,stm32-spi.yaml +++ b/Documentation/devicetree/bindings/spi/st,stm32-spi.yaml @@ -59,6 +59,10 @@ properties: - const: rx - const: tx + feature-domains: + minItems: 1 + maxItems: 3 + patternProperties: "^[a-zA-Z][a-zA-Z0-9,+\\-._]{0,63}@[0-9a-f]+$": type: object diff --git a/Documentation/devicetree/bindings/usb/dwc2.yaml b/Documentation/devicetree/bindings/usb/dwc2.yaml index d3506090f8b1..c372caf154fc 100644 --- a/Documentation/devicetree/bindings/usb/dwc2.yaml +++ b/Documentation/devicetree/bindings/usb/dwc2.yaml @@ -171,6 +171,10 @@ properties: tpl-support: true + feature-domains: + minItems: 1 + maxItems: 3 + dependencies: port: [ usb-role-switch ] role-switch-default-mode: [ usb-role-switch ]
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@foss.st.com> --- 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(+)