On 19/12/2024 16:49, Anthony Ruhier via B4 Relay wrote:
> From: Anthony Ruhier <aruhier@xxxxxxxxxxx>
>
> Add the lid switch for the Lenovo Yoga Slim 7x.
>
> Other x1e80100 laptops use the GPIO pin 92 only, however on the Yoga
> Slim 7x this pin seems to be bridged with the pin 71. By default, the
> pin 71 is set as output-high, which blocks any event on pin 92.
>
> This patch sets the pin 71 as output-disable and sets the LID switch on
> pin 92. This is aligned with how they're configured on Windows:
> GPIO 71 | 0xf147000 | in | func0 | hi | pull up | 16 mA
> GPIO 92 | 0xf15c000 | in | func0 | lo | no pull | 2 mA
>
> Signed-off-by: Anthony Ruhier <aruhier@xxxxxxxxxxx>
> ---
> Changes in v2:
> - Fixes patch format.
>
> Tested-by: Maya Matuszczyk <maccraft123mc@xxxxxxxxx>
Something is not right here. You cannot add tags in changelog section.
Please use b4 to collect the tags.
> ---
> .../boot/dts/qcom/x1e80100-lenovo-yoga-slim7x.dts | 38 ++++++++++++++++++++++
> 1 file changed, 38 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/x1e80100-lenovo-yoga-slim7x.dts b/arch/arm64/boot/dts/qcom/x1e80100-lenovo-yoga-slim7x.dts
> index ca5a808f2c7df66a861a933df407fd4bdaea3fe1..311202aa9015a30f1d70108bb214d427f811dc3b 100644
> --- a/arch/arm64/boot/dts/qcom/x1e80100-lenovo-yoga-slim7x.dts
> +++ b/arch/arm64/boot/dts/qcom/x1e80100-lenovo-yoga-slim7x.dts
> @@ -6,6 +6,7 @@
> /dts-v1/;
>
> #include <dt-bindings/gpio/gpio.h>
> +#include <dt-bindings/input/gpio-keys.h>
> #include <dt-bindings/regulator/qcom,rpmh-regulator.h>
>
> #include "x1e80100.dtsi"
> @@ -19,6 +20,21 @@ aliases {
> serial0 = &uart21;
> };
>
> + gpio-keys {
Oddly placed, 'g' goes after 'c'. Please keep proper order (see DTS
coding style).
=
Best regards,
Krzysztof
