Re: [PATCH v4 3/4] arm64: dts: qcom: sdm845: align TLMM pin configuration with DT schema

From: Doug Anderson
Date: Thu Dec 22 2022 - 13:54:40 EST


Hi,

On Thu, Dec 22, 2022 at 7:13 AM Krzysztof Kozlowski
<krzysztof.kozlowski@xxxxxxxxxx> wrote:
>
> DT schema expects TLMM pin configuration nodes to be named with
> '-state' suffix and their optional children with '-pins' suffix.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
>
> ---
>
> Cc: Doug Anderson <dianders@xxxxxxxxxxxx>
>
> Tested on Qualcomm RB3. Please kndly test a bit more on other devices.
> This should not have an functional impact.
>
> Changes since v3:
> 1. db845c: drop qup_uart3_default override and use qup_uart3_4pin
> (Doug).
>
> Changes since v2:
> 1. Bring back UART6 4-pin bias/drive strength to DTSI.
>
> Changes since v1:
> 1. Address comments and implement conclusion with Doug. Drop
> default-pins/pinmux/mux/config nodes but instead add subnodes for
> specific pins for UARTs. This should match approach used for SC7180.
>
> v1: https://lore.kernel.org/linux-kernel/1b9dcca8-1abd-99a3-da12-a8763bf77f12@xxxxxxxxxx/
> ---
> arch/arm64/boot/dts/qcom/sdm845-cheza.dtsi | 397 ++++------
> arch/arm64/boot/dts/qcom/sdm845-db845c.dts | 93 +--
> .../arm64/boot/dts/qcom/sdm845-lg-common.dtsi | 60 +-
> arch/arm64/boot/dts/qcom/sdm845-lg-judyln.dts | 2 +-
> arch/arm64/boot/dts/qcom/sdm845-mtp.dts | 77 +-
> .../boot/dts/qcom/sdm845-oneplus-common.dtsi | 101 +--
> .../boot/dts/qcom/sdm845-shift-axolotl.dts | 150 ++--
> .../dts/qcom/sdm845-sony-xperia-tama.dtsi | 6 +-
> .../qcom/sdm845-xiaomi-beryllium-common.dtsi | 12 +-
> .../boot/dts/qcom/sdm845-xiaomi-polaris.dts | 21 +-
> arch/arm64/boot/dts/qcom/sdm845.dtsi | 689 ++++++++----------
> .../boot/dts/qcom/sdm850-lenovo-yoga-c630.dts | 53 +-
> .../boot/dts/qcom/sdm850-samsung-w737.dts | 121 +--
> 13 files changed, 701 insertions(+), 1081 deletions(-)

Reviewed-by: Douglas Anderson <dianders@xxxxxxxxxxxx>