The 07/19/2022 07:52, Claudiu Beznea - M18063 wrote: > On 19.07.2022 00:29, Horatiu Vultur wrote: > > The pcb8291 can be rebooted by toggling the GPIO 56. Therefore enable > > this in DT. > > > > Signed-off-by: Horatiu Vultur <horatiu.vultur@xxxxxxxxxxxxx> > > --- > > arch/arm/boot/dts/lan966x-pcb8291.dts | 6 ++++++ > > 1 file changed, 6 insertions(+) > > > > diff --git a/arch/arm/boot/dts/lan966x-pcb8291.dts b/arch/arm/boot/dts/lan966x-pcb8291.dts > > index d56d2054c38d..02fbf7371a8d 100644 > > --- a/arch/arm/boot/dts/lan966x-pcb8291.dts > > +++ b/arch/arm/boot/dts/lan966x-pcb8291.dts > > @@ -16,6 +16,12 @@ chosen { > > aliases { > > serial0 = &usart3; > > }; > > + > > + gpio-restart { > > + compatible = "gpio-restart"; > > + gpios = <&gpio 56 GPIO_ACTIVE_LOW>; > > Could you, please, add also pinctrl-names, pinctrl-0 for this gpio? I am not sure I need to set pinctrl-names and pinctrl-0. Because I don't need to setup any function for GPIO 56 to be able to reset. It is something similar sparx5 [1]. [1] https://elixir.bootlin.com/linux/latest/source/arch/arm64/boot/dts/microchip/sparx5_pcb134_board.dtsi#L10 > > > + priority = <200>; > > + }; > > }; > > > > &gpio { > -- /Horatiu