| Message ID | 20240722-miix630-support-v1-0-a6483cfe8674@linaro.org |
|---|---|
| Headers | show |
| Series | arm64: dts: qcom: improve Lenovo Miix 630 support | expand |
On 22.07.2024 1:57 PM, Dmitry Baryshkov wrote: > Add gpio-keys device, responsible for a single button: Volume Up. > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> > --- > .../boot/dts/qcom/msm8998-lenovo-miix-630.dts | 25 ++++++++++++++++++++++ > 1 file changed, 25 insertions(+) > > diff --git a/arch/arm64/boot/dts/qcom/msm8998-lenovo-miix-630.dts b/arch/arm64/boot/dts/qcom/msm8998-lenovo-miix-630.dts > index f585bc8ba2ce..bca309ac0cb8 100644 > --- a/arch/arm64/boot/dts/qcom/msm8998-lenovo-miix-630.dts > +++ b/arch/arm64/boot/dts/qcom/msm8998-lenovo-miix-630.dts > @@ -3,12 +3,27 @@ > > /dts-v1/; > > +#include <dt-bindings/pinctrl/qcom,pmic-gpio.h> > #include "msm8998-clamshell.dtsi" > > / { > model = "Lenovo Miix 630"; > compatible = "lenovo,miix-630", "qcom,msm8998"; > chassis-type = "convertible"; > + > + gpio-keys { > + compatible = "gpio-keys"; > + autorepeat; > + > + pinctrl-names = "default"; > + pinctrl-0 = <&vol_up_pin_a>; property-n property-names > + > + key-vol-up { > + label = "Volume Up"; > + linux,code = <KEY_VOLUMEUP>; > + gpios = <&pm8998_gpios 6 GPIO_ACTIVE_LOW>; No debounce-interval? Konrad
On Tue, 23 Jul 2024 at 02:24, Konrad Dybcio <konrad.dybcio@linaro.org> wrote: > > On 22.07.2024 1:57 PM, Dmitry Baryshkov wrote: > > Add gpio-keys device, responsible for a single button: Volume Up. > > > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> > > --- > > .../boot/dts/qcom/msm8998-lenovo-miix-630.dts | 25 ++++++++++++++++++++++ > > 1 file changed, 25 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/qcom/msm8998-lenovo-miix-630.dts b/arch/arm64/boot/dts/qcom/msm8998-lenovo-miix-630.dts > > index f585bc8ba2ce..bca309ac0cb8 100644 > > --- a/arch/arm64/boot/dts/qcom/msm8998-lenovo-miix-630.dts > > +++ b/arch/arm64/boot/dts/qcom/msm8998-lenovo-miix-630.dts > > @@ -3,12 +3,27 @@ > > > > /dts-v1/; > > > > +#include <dt-bindings/pinctrl/qcom,pmic-gpio.h> > > #include "msm8998-clamshell.dtsi" > > > > / { > > model = "Lenovo Miix 630"; > > compatible = "lenovo,miix-630", "qcom,msm8998"; > > chassis-type = "convertible"; > > + > > + gpio-keys { > > + compatible = "gpio-keys"; > > + autorepeat; > > + > > + pinctrl-names = "default"; > > + pinctrl-0 = <&vol_up_pin_a>; > > property-n > property-names > > > + > > + key-vol-up { > > + label = "Volume Up"; > > + linux,code = <KEY_VOLUMEUP>; > > + gpios = <&pm8998_gpios 6 GPIO_ACTIVE_LOW>; > > No debounce-interval? Strangely enough, no. See existing defines.
On Tue, 23 Jul 2024 at 14:11, Dmitry Baryshkov <dmitry.baryshkov@linaro.org> wrote: > > On Tue, 23 Jul 2024 at 02:24, Konrad Dybcio <konrad.dybcio@linaro.org> wrote: > > > > On 22.07.2024 1:57 PM, Dmitry Baryshkov wrote: > > > Add gpio-keys device, responsible for a single button: Volume Up. > > > > > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> > > > --- > > > .../boot/dts/qcom/msm8998-lenovo-miix-630.dts | 25 ++++++++++++++++++++++ > > > 1 file changed, 25 insertions(+) > > > > > > diff --git a/arch/arm64/boot/dts/qcom/msm8998-lenovo-miix-630.dts b/arch/arm64/boot/dts/qcom/msm8998-lenovo-miix-630.dts > > > index f585bc8ba2ce..bca309ac0cb8 100644 > > > --- a/arch/arm64/boot/dts/qcom/msm8998-lenovo-miix-630.dts > > > +++ b/arch/arm64/boot/dts/qcom/msm8998-lenovo-miix-630.dts > > > + > > > + key-vol-up { > > > + label = "Volume Up"; > > > + linux,code = <KEY_VOLUMEUP>; > > > + gpios = <&pm8998_gpios 6 GPIO_ACTIVE_LOW>; > > > > No debounce-interval? > > Strangely enough, no. See existing defines. After more digging I see it being used. Let's use it too.
Enable some low-hanging fruits growing on the Lenovo Miix 630 laptop, but not being enabled for some reason. Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> --- Dmitry Baryshkov (4): arm64: dts: qcom: msm8998-lenovo-miix-630: enable touchscreen arm64: dts: qcom: msm8998-lenovo-miix-630: enable aDSP and SLPI arm64: dts: qcom: msm8998-lenovo-miix-630: enable VolumeUp button arm64: dts: qcom: msm8998-clamshell: enable resin/VolDown arch/arm64/boot/dts/qcom/msm8998-clamshell.dtsi | 5 ++ .../boot/dts/qcom/msm8998-lenovo-miix-630.dts | 63 ++++++++++++++++++++++ 2 files changed, 68 insertions(+) --- base-commit: 797012914d2d031430268fe512af0ccd7d8e46ef change-id: 20240722-miix630-support-33b3acc3a396 Best regards,