Hi Jacopo, On Tue, Dec 6, 2016 at 5:34 PM, Jacopo Mondi <jacopo@xxxxxxxxxx> wrote: > Add node to test MAX11100 ADC with LED28 connected to the chip input > line. What does the name "LED28" refer to? > The ADC sits on a gpio-spi master. > > Signed-off-by: Jacopo Mondi <jacopo@xxxxxxxxxx> > --- > > Sending as RFC only, not for inclusion. > Used to test MAX11100 ADC driver currently connected to Koelsch GPIO > expander. > Adding LED28 as ADC input line. > > --- > arch/arm/boot/dts/r8a7791-koelsch.dts | 21 +++++++++++++++++++++ > 1 file changed, 21 insertions(+) > > diff --git a/arch/arm/boot/dts/r8a7791-koelsch.dts b/arch/arm/boot/dts/r8a7791-koelsch.dts > index 5405d33..ac8f10d 100644 > --- a/arch/arm/boot/dts/r8a7791-koelsch.dts > +++ b/arch/arm/boot/dts/r8a7791-koelsch.dts > @@ -170,6 +170,27 @@ > gpios = <&gpio2 21 GPIO_ACTIVE_HIGH>; > label = "LED8"; > }; > + led28 { > + gpios = <&gpio5 28 GPIO_ACTIVE_HIGH>; > + label = "LED28"; > + }; > + }; > + > + spi_gpio: spi-gpio { > + compatible = "spi-gpio"; > + #address-cells = <1>; > + #size-cells = <0>; > + gpio-sck = <&gpio5 27 GPIO_ACTIVE_HIGH>; > + gpio-miso = <&gpio5 25 GPIO_ACTIVE_HIGH>; > + num-chipselects = <1>; > + cs-gpios = <&gpio5 26 GPIO_ACTIVE_LOW>; I would keep all gpios together. > + status = "okay"; > + > + spi@0 { adc@0 > + compatible = "maxim,max11100"; > + reg = <0>; > + spi-max-frequency = <2500000>; > + }; > }; Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds