Re: [PATCH 2/2] arm64: dts: qcom: sc8180x-flex-5g: remove superfluous "input-enable"

From: Konrad Dybcio
Date: Mon Jun 19 2023 - 09:19:23 EST


On 17.06.2023 13:21, Krzysztof Kozlowski wrote:
> Pin configuration property "input-enable" was used with the intention to
> disable the output, but this is done by default by Linux drivers. Since
> commit c4a48b0df8bf ("dt-bindings: pinctrl: qcom: tlmm should use
> output-disable, not input-enable") the property is not accepted anymore.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@xxxxxxxxxx>

Konrad
> arch/arm64/boot/dts/qcom/sc8180x-lenovo-flex-5g.dts | 3 ---
> 1 file changed, 3 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/sc8180x-lenovo-flex-5g.dts b/arch/arm64/boot/dts/qcom/sc8180x-lenovo-flex-5g.dts
> index fe3b366e1435..b12e0a61b3c0 100644
> --- a/arch/arm64/boot/dts/qcom/sc8180x-lenovo-flex-5g.dts
> +++ b/arch/arm64/boot/dts/qcom/sc8180x-lenovo-flex-5g.dts
> @@ -497,7 +497,6 @@ hall_int_active_state: hall-int-active-state {
> pins = "gpio121";
> function = "gpio";
>
> - input-enable;
> bias-disable;
> };
>
> @@ -513,7 +512,6 @@ i2c1_hid_active: i2c1-hid-active-state {
> pins = "gpio122";
> function = "gpio";
>
> - input-enable;
> bias-pull-up;
> drive-strength = <2>;
> };
> @@ -530,7 +528,6 @@ i2c7_hid_active: i2c7-hid-active-state {
> pins = "gpio37", "gpio24";
> function = "gpio";
>
> - input-enable;
> bias-pull-up;
> drive-strength = <2>;
> };