[PATCH 3/3] arm64: dts: rockchip: Add saradc node to rk3588s-indiedroid-nova

[Chris Morgan]

On Tue, Sep 12, 2023 at 10:31:19AM +0200, Krzysztof Kozlowski wrote:
> On 08/09/2023 18:41, Chris Morgan wrote:
> > From: Chris Morgan <macromorgan at hotmail.com>
> > 
> > Add ADC support for the Indiedroid Nova, as well as the two ADC buttons
> > found on the device. The buttons are documented as "boot" and
> > "recovery". The boot button is used by the bootloader to boot into USB
> > recovery mode. The recovery button use is currently unknown.
> > 
> > Signed-off-by: Chris Morgan <macromorgan at hotmail.com>
> > ---
> >  .../dts/rockchip/rk3588s-indiedroid-nova.dts  | 34 +++++++++++++++++++
> >  1 file changed, 34 insertions(+)
> > 
> > diff --git a/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts b/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts
> > index 1e2336d3065b..4c2d662d9c97 100644
> > --- a/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts
> > +++ b/arch/arm64/boot/dts/rockchip/rk3588s-indiedroid-nova.dts
> > @@ -3,6 +3,7 @@
> >  /dts-v1/;
> >  
> >  #include <dt-bindings/gpio/gpio.h>
> > +#include <dt-bindings/input/linux-event-codes.h>
> >  #include <dt-bindings/pinctrl/rockchip.h>
> >  #include <dt-bindings/usb/pd.h>
> >  #include "rk3588s.dtsi"
> > @@ -11,6 +12,34 @@ / {
> >  	model = "Indiedroid Nova";
> >  	compatible = "indiedroid,nova", "rockchip,rk3588s";
> >  
> > +	adc_keys0 {
> 
> No underscores in node names.

My apologies, I should have known better...

> 
> Suffixes usually are after -
> 

Thank you.

> Best regards,
> Krzysztof
> 

I'll correct this and resend.

Chris