On Tue, 1 Oct 2024 17:18:30 +0100 Conor Dooley <conor@xxxxxxxxxx> wrote: > On Tue, Oct 01, 2024 at 09:46:16AM +0200, Herve Codina wrote: > > The GE HealthCare PMC Analog to Digital Converter (ADC) is a 16-Channel > > (voltage and current), 16-Bit ADC with an I2C Interface. > > > > Signed-off-by: Herve Codina <herve.codina@xxxxxxxxxxx> > > --- > > .../bindings/iio/adc/gehc,pmc-adc.yaml | 82 +++++++++++++++++++ > > include/dt-bindings/iio/adc/gehc,pmc-adc.h | 10 +++ ... > > + clocks: > > + maxItems: 1 > > + > > + clock-names: > > + items: > > + - const: osc > > Since there's no datasheet for me to look up, why is the clock optional? The component uses an external oscillator (osc) if an external oscillator is connected to its clock pins. Otherwise, it uses an internal reference clock. Best regards, Hervé -- Hervé Codina, Bootlin Embedded Linux and Kernel engineering https://bootlin.com