On 1/10/2019 11:37 PM, Thierry Reding wrote:
> On Wed, Jan 09, 2019 at 09:24:18AM +0800, Mark Zhang wrote:
>> Add gpio-keys nodes for the power button.
>>
>> Signed-off-by: Mark Zhang <markz@xxxxxxxxxx>
>> ---
>> arch/arm64/boot/dts/nvidia/tegra210-p2894.dtsi | 14 ++++++++++++++
>> 1 file changed, 14 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/nvidia/tegra210-p2894.dtsi b/arch/arm64/boot/dts/nvidia/tegra210-p2894.dtsi
>> index ac52286ab9ab..b21902a98f11 100644
>> --- a/arch/arm64/boot/dts/nvidia/tegra210-p2894.dtsi
>> +++ b/arch/arm64/boot/dts/nvidia/tegra210-p2894.dtsi
>> @@ -1,5 +1,6 @@
>> // SPDX-License-Identifier: GPL-2.0
>>
>> +#include <dt-bindings/input/input.h>
>> #include "tegra210.dtsi"
>>
>> / {
>> @@ -52,6 +53,19 @@
>> };
>> };
>>
>> + gpio-keys {
>> + compatible = "gpio-keys";
>> + status = "okay";
>> +
>> + power {
>> + label = "Power";
>> + gpios = <&gpio TEGRA_GPIO(X, 7) GPIO_ACTIVE_LOW>;
>> + linux,code = <KEY_POWER>;
>> + wakeup-source;
>
> I think if you have wakeup-source you also need wakeup-event-action, see
> Documentation/devicetree/bindings/input/gpio-keys.txt.
Done. Thanks for pointing out.
Mark
>
> Thierry
>
