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

[Chris Morgan]

On Fri, Sep 15, 2023 at 11:49:05AM -0500, Chris Morgan wrote:
> 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

Sorry, one more thing. I've checked for examples of adc-keys naming and
in every instance I see it as adc-keys (or adc-keys0 and adc-keys1 in a
few examples). Is the proper way going forward something like key0-adc
and key1-adc?

Thank you,
Chris.