Hi, On Fri, Dec 2, 2022 at 7:57 AM Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> wrote: > > Each board should define pin drive/bias for used busses. All boards > using SPI0 (db845c and cheza) already do it, so drop the bias/drive > strength from SoC DTSI. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> > > --- > > Cc: Doug Anderson <dianders@xxxxxxxxxxxx> > > Changes since v2: > 1. New patch. > --- > arch/arm64/boot/dts/qcom/sdm845.dtsi | 2 -- > 1 file changed, 2 deletions(-) Reviewed-by: Douglas Anderson <dianders@xxxxxxxxxxxx> It's hard to review device trees without noticing things that are inefficient / wrong. In case you want any others for your list: a) As written today, `sdm845-db845c.dts` doesn't actually need to override the `pinctrl` of `spi0` since it's the default. So you could remove two extra lines of device tree. b) Officially, db845c actually _should_ be overriding the pinctrl and we should be doing something like what we did for sc7180 where we have different definitions for "cs" and "cs_gpio" for the SPI lines. That's because db845c is actually using "cs-gpio" mode for this SPI part and thus GPIO3 should be marked as being "gpio", not "qup0". It doesn't really matter functionality-wise on Linux because the Qualcomm pinctrl driver will automatically re-mux the pin as a GPIO the moment the SPI core tries to use it as such, but it's a little ugly.
