lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Thu, 16 Sep 2021 23:07:05 -0700 From: Stephen Boyd <swboyd@...omium.org> To: skakit@...eaurora.org Cc: Andy Gross <agross@...nel.org>, Bjorn Andersson <bjorn.andersson@...aro.org>, Linus Walleij <linus.walleij@...aro.org>, Rob Herring <robh+dt@...nel.org>, David Collins <collinsd@...eaurora.org>, kgunda@...eaurora.org, linux-gpio@...r.kernel.org, linux-kernel@...r.kernel.org, devicetree@...r.kernel.org, linux-arm-msm@...r.kernel.org Subject: Re: [PATCH V5 2/2] arm64: dts: sc7280: Add volume up support for sc7280-idp Quoting skakit@...eaurora.org (2021-09-16 21:08:11) > On 2021-09-17 00:32, Stephen Boyd wrote: > > Quoting Satya Priya (2021-09-16 06:21:38) > >> diff --git a/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi > >> b/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi > >> index 371a2a9..cbbb0ee 100644 > >> --- a/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi > >> +++ b/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi > >> @@ -12,6 +12,26 @@ > >> #include "pm8350c.dtsi" > >> #include "pmk8350.dtsi" > >> > >> +/ { > >> + gpio-keys { > >> + compatible = "gpio-keys"; > >> + label = "gpio-keys"; > >> + > >> + pinctrl-names = "default"; > >> + pinctrl-0 = <&key_vol_up_default>; > >> + > >> + volume-up { > >> + label = "volume_up"; > >> + gpios = <&pm7325_gpios 6 GPIO_ACTIVE_LOW>; > >> + linux,input-type = <1>; > >> + linux,code = <KEY_VOLUMEUP>; > > > > Is there an include for this define? Looks like > > <dt-bindings/input/input.h> should be added as well? Did you try > > compiling? > > > > Yeah, it needs <dt-bindings/input/linux-event-codes.h> to be included. > This header was included in pmk8350.dtsi which is added in this file. Alright, got it. If you can include this header in this dts file it would be better so we don't rely on implicit includes from other files to get macros. With that change you can add my Reviewed-by: Stephen Boyd <swboyd@...omium.org>
Powered by blists - more mailing lists