Re: [PATCH] arm64: dts: qcom: sc7180-trogdor: Simply SAR sensor enabling

[Doug Anderson]

Hi,

On Fri, Mar 25, 2022 at 2:16 PM Stephen Boyd <swboyd@xxxxxxxxxxxx> wrote:
>
> The SAR node, ap_sar_sensor, needs to be enabled in addition to the i2c
> bus it resides on. Let's simplify this by leaving the sensor node
> enabled by default while leaving the i2c bus disabled by default. On
> boards that use the sensor, we already enable the i2c bus so we can
> simply remove the extra bit that enables the sar sensor node. This saves
> some lines but is otherwise a non-functional change.
>
> Cc: Douglas Anderson <dianders@xxxxxxxxxxxx>
> Signed-off-by: Stephen Boyd <swboyd@xxxxxxxxxxxx>
> ---
>  arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r1-lte.dts | 4 ----
>  arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r3-lte.dts | 4 ----
>  arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r9-lte.dts | 4 ----
>  arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi             | 1 -
>  4 files changed, 13 deletions(-)

Makes sense to me and even saves a byte or two.

Reviewed-by: Douglas Anderson <dianders@xxxxxxxxxxxx>

-Doug