enum: [100, 400] default: 100 â??â??instead of declaring its range. What do you think? In patches v2 I used if (dev->bus_capacitance_pf == 400) {... } else {...} and other statements in the driver code to indicate that the capacitance can only be 400pf or not. Maybe this is a metaphor. I'm sorry that I wasn't more explicit about the constraints. > > + > > + snps,clk-freq-optimized: > > + description: > > > + This property indicates whether the hardware input clock frequency > is > > + reduced by reducing the internal latency. This value is used to > compute > > + the tHIGH and tLOW periods for high speed mode. > > + type: boolean > > + > > unevaluatedProperties: false > > > > required: > > @@ -146,4 +161,13 @@ examples: > > interrupts = <8>; > > clocks = <&ahb_clk>; > > }; > > + - | > > + i2c@c5000000 { > > + compatible = "snps,designware-i2c"; > > Extend EXISTING example. Not add new example. Should I insert these two properties into one or all existing examples? Thanks & Regards Michael