[linux-sunxi] Re: [PATCH v7] ARM: dts: sun8i: Add leds and switches on Orangepi Plus boards

[Hans de Goede]

Hi,

On 21-03-16 08:59, Maxime Ripard wrote:
> Hi,
>
> On Sun, Mar 20, 2016 at 07:55:02PM +0100, Hans de Goede wrote:
>> From: Krzysztof Adamski <k at japko.eu>
>>
>> OrangePi Plus board has two leds - green ("pwr") and red ("status")
>> and two switches ("sw4" and "sw2"). This patch describes them in
>> devicetree.
>>
>> Signed-off-by: Krzysztof Adamski <k at japko.eu>
>> Signed-off-by: Hans de Goede <hdegoede at redhat.com>
>
> An earlier version has already been sent for 4.6, please base your
> changes on top of this one.
>
> And there's a bunch of things I didn't catch at that time :/

As promised I will send a fixup patch for this, I was wondering
if you can push your current queue, where you've already merged
my other orangepi-plus patches somewhere, so that I can base this
fixup patch on top of it ?

Regards,

Hans


>
>> ---
>> Changes in v6 (hdegoede):
>> -Merge leds and r_leds nodes into one
>> -Fix led labels to be in <board>:<colour>:<role> form
>> -Alphabetically sort the added nodes
>> -Add support for sw2 (this is the small reset-like button)
>> Changes in v7 (hdegoede):
>> -Use PL3 / PL4 in the pinctrl node, not PL03 / PL04
>> ---
>>   arch/arm/boot/dts/sun8i-h3-orangepi-plus.dts | 63 ++++++++++++++++++++++++++++
>>   1 file changed, 63 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/sun8i-h3-orangepi-plus.dts b/arch/arm/boot/dts/sun8i-h3-orangepi-plus.dts
>> index a680b1f..9d8c038 100644
>> --- a/arch/arm/boot/dts/sun8i-h3-orangepi-plus.dts
>> +++ b/arch/arm/boot/dts/sun8i-h3-orangepi-plus.dts
>> @@ -45,6 +45,7 @@
>>   #include "sunxi-common-regulators.dtsi"
>>
>>   #include <dt-bindings/gpio/gpio.h>
>> +#include <dt-bindings/input/input.h>
>>   #include <dt-bindings/pinctrl/sun4i-a10.h>
>>
>>   / {
>> @@ -59,6 +60,43 @@
>>   		stdout-path = "serial0:115200n8";
>>   	};
>>
>> +	gpio_keys {
>> +		compatible = "gpio-keys";
>> +		input-name = "sw4";
>
> This is an undocumented (and apparently unused) property
>
>> +
>> +		pinctrl-names = "default";
>> +		pinctrl-0 = <&sw_r_opc>;
>> +
>> +		sw2 at 0 {
>
> You shouldn't have a unit-address here
>
>> +			label = "sw2";
>> +			linux,code = <BTN_1>;
>> +			gpios = <&r_pio 0 4 GPIO_ACTIVE_LOW>;
>> +		};
>> +
>> +		sw4 at 0 {
>
> Ditto.
>
> Thanks!
> Maxime
>