Hi Wolfram, Thanks for your patch!
On Fri, Dec 22, 2017 at 9:13 PM Wolfram Sang <[email protected]> wrote: > SATA needs this GPIO hog, so is probably an overlay? IMHO it can be part of the real board DTS, for the following reasons: 1. SATA is multiplexed with PCIE1, but the SATA connector is on-board, while using PCIE1 requires plugging in an extension board in EXIO Connector A. 2. PCIE1 is the default, for both MD12 and the PCIE-SATA switch, but I assume that is only the case because Salvator-XS supports all of R-Car H3 ES2, M3-W, and M3-N, while M3-W does not have SATA. Note that for the GPIO hog to survive s2ram, the pca9654 driver needs to be enhanced to restore its configuration after system resume. > Signed-off-by: Wolfram Sang <[email protected]> Tested-by: Geert Uytterhoeven <[email protected]> > --- a/arch/arm64/boot/dts/renesas/r8a7795-salvator-xs.dts > +++ b/arch/arm64/boot/dts/renesas/r8a7795-salvator-xs.dts > @@ -101,6 +101,19 @@ > }; > }; > > +&pca9654 { Please insert this node just before &pfc, to preserve sort order. > + pcie_sata_switch { > + gpio-hog; > + gpios = <7 0>; s/0/GPIO_ACTIVE_HIGH/? > + output-low; > + line-name = "PCIE/SATA switch"; Perhaps add a comment saying MD12 must be 1 (SW12-7 must be off)? > + }; > +}; > + > +&sata { > + status = "okay"; > +}; > + > &usb2_phy2 { > pinctrl-0 = <&usb2_pins>; > pinctrl-names = "default"; Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected] 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
