On Wed, Apr 02, 2025 at 03:36:32PM +0100, Christopher Obbard wrote:
> Add edp_hpd_active pinctrl to the X1E80100 device tree.
Please squash this one with the patch adding the user.
> Signed-off-by: Christopher Obbard <christopher.obb...@linaro.org>
> ---
> arch/arm64/boot/dts/qcom/x1e80100.dtsi | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/x1e80100.dtsi
> b/arch/arm64/boot/dts/qcom/x1e80100.dtsi
> index
> 46b79fce92c90d969e3de48bc88e27915d1592bb..5b1b80c445404fd02e9f6e5dab207610b03ff9c5
> 100644
> --- a/arch/arm64/boot/dts/qcom/x1e80100.dtsi
> +++ b/arch/arm64/boot/dts/qcom/x1e80100.dtsi
> @@ -6389,6 +6389,11 @@ data-pins {
> bias-pull-up;
> };
> };
> +
> + edp_hpd_active: edp-hpd-active-state {
Please keep the nodes sorted by name.
> + pins = "gpio119";
> + function = "edp_hot";
There is no "edp_hot" function on x1e so I wonder how this has been
tested.
As I mentioned in a comment to an earlier revision this pin has been
configured by the firmware as "edp0_hot".
Since there is also an "edp1_hot" pin, this needs to be reflected in the
node name and label.
Pin configurations really do belong in board files, but unfortunately
this was not followed for x1e. You should still include the bias
configuration (somewhere).
> + };
> };
>
> stm@10002000 {
Johan
