Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1340571

Re: [PATCH] ARM: dts: add support for gpio buttons for exynos5422-odroidxu3

Show all headers | View raw

2016-02-23 18:17 GMT+09:00 Anand Moon <linux.amoon@gmail.com>:
> Hi Krzysztof,
>
> On 23 February 2016 at 14:03, Krzysztof Kozlowski
>>>>      };
>>>> +
>>>> +    gpio_keys {
>>>> +            compatible = "gpio-keys";
>>>> +            pinctrl-names = "default";
>>>> +            pinctrl-0 = <&gpio_power_key>;
>>>> +
>>>> +            power_key {
>>>> +                    interrupt-parent = <&gpx0>;
>>>> +                    interrupts = <3 IRQ_TYPE_NONE>;
>>>
>>> Hmmm.... why you specify the interrupts?
>>>
>>>> +                    gpios = <&gpx0 3 GPIO_ACTIVE_LOW>;
>>
>> Please, explain it to me. The SW2 key is connected to PWRON of PMIC.
>> However you are adding a GPIO key for external interrupt source 3
>> (XE.INT3)... which comes from PMIC's ONOB.
>>
>> It's interesting.... how does it work? The PMIC will generate ONOB
>> interrupt on PWRON low->high change (when PWRHOLD is high)?
>>
>> Best regards,
>> Krzysztof
>>
>
> I have re-base my changes on HK dts.
> I could not find much details for the schematic diagram.
> I don't know much low level detains on this.

If I understand this correctly: you just took some vendor code,
similar to existing code of Odroid U3, and without full understanding
of the code nor checking with public schematics, you sent it.

Taking vendor stuff is okay but you need to think about it, use it as
an example and deliver proper solution based on that. Not copy-paste.

> How dose this works: This changes will initial the shudown/reboot on Ubuntu.
> I have also tested this similar feature on Odroid U3.

Great :), yes it works because ONOB interrupt from PMIC is generated
on key press. However this is not a strictly speaking key... I don't
really know what to do with this commit and your explanation
(Hardkernel has such code) is not sufficient to convince me.

Best regards,
Krzysztof

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH] ARM: dts: add support for gpio buttons for exynos5422-odroidxu3 Anand Moon <linux.amoon@gmail.com> - 2016-02-23 09:10 +0100
  Re: [PATCH] ARM: dts: add support for gpio buttons for  exynos5422-odroidxu3 Krzysztof Kozlowski <k.kozlowski@samsung.com> - 2016-02-23 09:30 +0100
    Re: [PATCH] ARM: dts: add support for gpio buttons for  exynos5422-odroidxu3 Krzysztof Kozlowski <k.kozlowski@samsung.com> - 2016-02-23 09:40 +0100
      Re: [PATCH] ARM: dts: add support for gpio buttons for  exynos5422-odroidxu3 Krzysztof Kozlowski <k.kozlowski@samsung.com> - 2016-02-23 09:50 +0100
      Re: [PATCH] ARM: dts: add support for gpio buttons for exynos5422-odroidxu3 Anand Moon <linux.amoon@gmail.com> - 2016-02-23 10:20 +0100
        Re: [PATCH] ARM: dts: add support for gpio buttons for exynos5422-odroidxu3 Krzysztof Kozlowski <k.kozlowski@samsung.com> - 2016-02-23 13:10 +0100
          Re: [PATCH] ARM: dts: add support for gpio buttons for exynos5422-odroidxu3 Anand Moon <linux.amoon@gmail.com> - 2016-02-23 15:20 +0100
            Re: [PATCH] ARM: dts: add support for gpio buttons for  exynos5422-odroidxu3 Krzysztof Kozlowski <k.kozlowski@samsung.com> - 2016-02-24 08:30 +0100

csiph-web