Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1568947 > unrolled thread
| Started by | AdityaXavier <adityaxavier@gmail.com> |
|---|---|
| First post | 2017-01-28 15:50 +0100 |
| Last post | 2017-01-30 18:40 +0100 |
| Articles | 4 — 4 participants |
Back to article view | Back to linux.kernel
[PATCH V2] ARM: dts: BCM5301X: Add missing Netgear R8000 LEDs and Keys AdityaXavier <adityaxavier@gmail.com> - 2017-01-28 15:50 +0100
Re: [PATCH V2] ARM: dts: BCM5301X: Add missing Netgear R8000 LEDs and Keys Rafał Miłecki <zajec5@gmail.com> - 2017-01-28 22:30 +0100
Re: [PATCH V2] ARM: dts: BCM5301X: Add missing Netgear R8000 LEDs and Keys Aditya Xavier <adityaxavier@gmail.com> - 2017-01-30 11:30 +0100
Re: [PATCH V2] ARM: dts: BCM5301X: Add missing Netgear R8000 LEDs and Keys Florian Fainelli <f.fainelli@gmail.com> - 2017-01-30 18:40 +0100
| From | AdityaXavier <adityaxavier@gmail.com> |
|---|---|
| Date | 2017-01-28 15:50 +0100 |
| Subject | [PATCH V2] ARM: dts: BCM5301X: Add missing Netgear R8000 LEDs and Keys |
| Message-ID | <t4Cud-2Gm-19@gated-at.bofh.it> |
From: Aditya Xavier <adityaxavier@gmail.com>
Added two WAN status LEDs and a GPIO Key for Brightness which were missing.
V2: Updated subject, Power LED names, and WAN labels.
Signed-off-by: Aditya Xavier <adityaxavier@gmail.com>
---
arch/arm/boot/dts/bcm4709-netgear-r8000.dts | 22 ++++++++++++++++++++--
1 file changed, 20 insertions(+), 2 deletions(-)
diff --git a/arch/arm/boot/dts/bcm4709-netgear-r8000.dts b/arch/arm/boot/dts/bcm4709-netgear-r8000.dts
index 92f8a72..90d4420 100644
--- a/arch/arm/boot/dts/bcm4709-netgear-r8000.dts
+++ b/arch/arm/boot/dts/bcm4709-netgear-r8000.dts
@@ -27,18 +27,30 @@
leds {
compatible = "gpio-leds";
- power0 {
+ power-white {
label = "bcm53xx:white:power";
gpios = <&chipcommon 2 GPIO_ACTIVE_LOW>;
linux,default-trigger = "default-on";
};
- power1 {
+ power-amber {
label = "bcm53xx:amber:power";
gpios = <&chipcommon 3 GPIO_ACTIVE_LOW>;
linux,default-trigger = "default-off";
};
+ wan-white {
+ label = "bcm53xx:white:wan";
+ gpios = <&chipcommon 8 GPIO_ACTIVE_LOW>;
+ linux,default-trigger = "default-on";
+ };
+
+ wan-amber {
+ label = "bcm53xx:amber:wan";
+ gpios = <&chipcommon 9 GPIO_ACTIVE_HIGH>;
+ linux,default-trigger = "default-off";
+ };
+
5ghz-1 {
label = "bcm53xx:white:5ghz-1";
gpios = <&chipcommon 12 GPIO_ACTIVE_LOW>;
@@ -104,6 +116,12 @@
linux,code = <KEY_RESTART>;
gpios = <&chipcommon 6 GPIO_ACTIVE_LOW>;
};
+
+ brightness {
+ label = "Backlight";
+ linux,code = <KEY_BRIGHTNESS_ZERO>;
+ gpios = <&chipcommon 19 GPIO_ACTIVE_LOW>;
+ };
};
};
--
2.9.3
[toc] | [next] | [standalone]
| From | Rafał Miłecki <zajec5@gmail.com> |
|---|---|
| Date | 2017-01-28 22:30 +0100 |
| Message-ID | <t4IJj-6Fj-5@gated-at.bofh.it> |
| In reply to | #1568947 |
On 28 January 2017 at 15:37, AdityaXavier <adityaxavier@gmail.com> wrote:
> From: Aditya Xavier <adityaxavier@gmail.com>
>
> Added two WAN status LEDs and a GPIO Key for Brightness which were missing.
> V2: Updated subject, Power LED names, and WAN labels.
Changelog (V2 ... part) should go into that /comments/ section (see below).
Florian: can you drop that line when applying this patch? Otherwise it
looks OK to me.
> Signed-off-by: Aditya Xavier <adityaxavier@gmail.com>
Acked-by: Rafał Miłecki <rafal@milecki.pl>
Thanks for the patch!
> ---
Right here, below these 3 dashes is a place where you can add extra
comments (they won't go into log when doing "git am").
> arch/arm/boot/dts/bcm4709-netgear-r8000.dts | 22 ++++++++++++++++++++--
> 1 file changed, 20 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/boot/dts/bcm4709-netgear-r8000.dts b/arch/arm/boot/dts/bcm4709-netgear-r8000.dts
> index 92f8a72..90d4420 100644
> --- a/arch/arm/boot/dts/bcm4709-netgear-r8000.dts
> +++ b/arch/arm/boot/dts/bcm4709-netgear-r8000.dts
> @@ -27,18 +27,30 @@
> leds {
> compatible = "gpio-leds";
>
> - power0 {
> + power-white {
> label = "bcm53xx:white:power";
> gpios = <&chipcommon 2 GPIO_ACTIVE_LOW>;
> linux,default-trigger = "default-on";
> };
>
> - power1 {
> + power-amber {
> label = "bcm53xx:amber:power";
> gpios = <&chipcommon 3 GPIO_ACTIVE_LOW>;
> linux,default-trigger = "default-off";
> };
>
> + wan-white {
> + label = "bcm53xx:white:wan";
> + gpios = <&chipcommon 8 GPIO_ACTIVE_LOW>;
> + linux,default-trigger = "default-on";
> + };
> +
> + wan-amber {
> + label = "bcm53xx:amber:wan";
> + gpios = <&chipcommon 9 GPIO_ACTIVE_HIGH>;
> + linux,default-trigger = "default-off";
> + };
> +
> 5ghz-1 {
> label = "bcm53xx:white:5ghz-1";
> gpios = <&chipcommon 12 GPIO_ACTIVE_LOW>;
> @@ -104,6 +116,12 @@
> linux,code = <KEY_RESTART>;
> gpios = <&chipcommon 6 GPIO_ACTIVE_LOW>;
> };
> +
> + brightness {
> + label = "Backlight";
> + linux,code = <KEY_BRIGHTNESS_ZERO>;
> + gpios = <&chipcommon 19 GPIO_ACTIVE_LOW>;
> + };
> };
> };
>
> --
> 2.9.3
>
--
Rafał
[toc] | [prev] | [next] | [standalone]
| From | Aditya Xavier <adityaxavier@gmail.com> |
|---|---|
| Date | 2017-01-30 11:30 +0100 |
| Subject | Re: [PATCH V2] ARM: dts: BCM5301X: Add missing Netgear R8000 LEDs and Keys |
| Message-ID | <t5hnJ-2NG-31@gated-at.bofh.it> |
| In reply to | #1569012 |
Would you require me to send the revised Patch ?
Or would this do ?
And thanks for guiding me through this process :)
> On 29-Jan-2017, at 2:45 AM, Rafał Miłecki <zajec5@gmail.com> wrote:
>
> On 28 January 2017 at 15:37, AdityaXavier <adityaxavier@gmail.com> wrote:
>> From: Aditya Xavier <adityaxavier@gmail.com>
>>
>> Added two WAN status LEDs and a GPIO Key for Brightness which were missing.
>> V2: Updated subject, Power LED names, and WAN labels.
>
> Changelog (V2 ... part) should go into that /comments/ section (see below).
> Florian: can you drop that line when applying this patch? Otherwise it
> looks OK to me.
>
>
>> Signed-off-by: Aditya Xavier <adityaxavier@gmail.com>
>
> Acked-by: Rafał Miłecki <rafal@milecki.pl>
>
> Thanks for the patch!
>
>
>> ---
>
> Right here, below these 3 dashes is a place where you can add extra
> comments (they won't go into log when doing "git am").
>
>> arch/arm/boot/dts/bcm4709-netgear-r8000.dts | 22 ++++++++++++++++++++--
>> 1 file changed, 20 insertions(+), 2 deletions(-)
>>
>> diff --git a/arch/arm/boot/dts/bcm4709-netgear-r8000.dts b/arch/arm/boot/dts/bcm4709-netgear-r8000.dts
>> index 92f8a72..90d4420 100644
>> --- a/arch/arm/boot/dts/bcm4709-netgear-r8000.dts
>> +++ b/arch/arm/boot/dts/bcm4709-netgear-r8000.dts
>> @@ -27,18 +27,30 @@
>> leds {
>> compatible = "gpio-leds";
>>
>> - power0 {
>> + power-white {
>> label = "bcm53xx:white:power";
>> gpios = <&chipcommon 2 GPIO_ACTIVE_LOW>;
>> linux,default-trigger = "default-on";
>> };
>>
>> - power1 {
>> + power-amber {
>> label = "bcm53xx:amber:power";
>> gpios = <&chipcommon 3 GPIO_ACTIVE_LOW>;
>> linux,default-trigger = "default-off";
>> };
>>
>> + wan-white {
>> + label = "bcm53xx:white:wan";
>> + gpios = <&chipcommon 8 GPIO_ACTIVE_LOW>;
>> + linux,default-trigger = "default-on";
>> + };
>> +
>> + wan-amber {
>> + label = "bcm53xx:amber:wan";
>> + gpios = <&chipcommon 9 GPIO_ACTIVE_HIGH>;
>> + linux,default-trigger = "default-off";
>> + };
>> +
>> 5ghz-1 {
>> label = "bcm53xx:white:5ghz-1";
>> gpios = <&chipcommon 12 GPIO_ACTIVE_LOW>;
>> @@ -104,6 +116,12 @@
>> linux,code = <KEY_RESTART>;
>> gpios = <&chipcommon 6 GPIO_ACTIVE_LOW>;
>> };
>> +
>> + brightness {
>> + label = "Backlight";
>> + linux,code = <KEY_BRIGHTNESS_ZERO>;
>> + gpios = <&chipcommon 19 GPIO_ACTIVE_LOW>;
>> + };
>> };
>> };
>>
>> --
>> 2.9.3
>>
>
>
>
> --
> Rafał
[toc] | [prev] | [next] | [standalone]
| From | Florian Fainelli <f.fainelli@gmail.com> |
|---|---|
| Date | 2017-01-30 18:40 +0100 |
| Subject | Re: [PATCH V2] ARM: dts: BCM5301X: Add missing Netgear R8000 LEDs and Keys |
| Message-ID | <t5o5R-6Mu-45@gated-at.bofh.it> |
| In reply to | #1569558 |
On 01/30/2017 01:08 AM, Aditya Xavier wrote:
> Would you require me to send the revised Patch ?
>
> Or would this do ?
Will take care of it this time.
>
> And thanks for guiding me through this process :)
>
>
>> On 29-Jan-2017, at 2:45 AM, Rafał Miłecki <zajec5@gmail.com> wrote:
>>
>> On 28 January 2017 at 15:37, AdityaXavier <adityaxavier@gmail.com> wrote:
>>> From: Aditya Xavier <adityaxavier@gmail.com>
>>>
>>> Added two WAN status LEDs and a GPIO Key for Brightness which were missing.
>>> V2: Updated subject, Power LED names, and WAN labels.
>>
>> Changelog (V2 ... part) should go into that /comments/ section (see below).
>> Florian: can you drop that line when applying this patch? Otherwise it
>> looks OK to me.
>>
>>
>>> Signed-off-by: Aditya Xavier <adityaxavier@gmail.com>
>>
>> Acked-by: Rafał Miłecki <rafal@milecki.pl>
>>
>> Thanks for the patch!
>>
>>
>>> ---
>>
>> Right here, below these 3 dashes is a place where you can add extra
>> comments (they won't go into log when doing "git am").
>>
>>> arch/arm/boot/dts/bcm4709-netgear-r8000.dts | 22 ++++++++++++++++++++--
>>> 1 file changed, 20 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/arch/arm/boot/dts/bcm4709-netgear-r8000.dts b/arch/arm/boot/dts/bcm4709-netgear-r8000.dts
>>> index 92f8a72..90d4420 100644
>>> --- a/arch/arm/boot/dts/bcm4709-netgear-r8000.dts
>>> +++ b/arch/arm/boot/dts/bcm4709-netgear-r8000.dts
>>> @@ -27,18 +27,30 @@
>>> leds {
>>> compatible = "gpio-leds";
>>>
>>> - power0 {
>>> + power-white {
>>> label = "bcm53xx:white:power";
>>> gpios = <&chipcommon 2 GPIO_ACTIVE_LOW>;
>>> linux,default-trigger = "default-on";
>>> };
>>>
>>> - power1 {
>>> + power-amber {
>>> label = "bcm53xx:amber:power";
>>> gpios = <&chipcommon 3 GPIO_ACTIVE_LOW>;
>>> linux,default-trigger = "default-off";
>>> };
>>>
>>> + wan-white {
>>> + label = "bcm53xx:white:wan";
>>> + gpios = <&chipcommon 8 GPIO_ACTIVE_LOW>;
>>> + linux,default-trigger = "default-on";
>>> + };
>>> +
>>> + wan-amber {
>>> + label = "bcm53xx:amber:wan";
>>> + gpios = <&chipcommon 9 GPIO_ACTIVE_HIGH>;
>>> + linux,default-trigger = "default-off";
>>> + };
>>> +
>>> 5ghz-1 {
>>> label = "bcm53xx:white:5ghz-1";
>>> gpios = <&chipcommon 12 GPIO_ACTIVE_LOW>;
>>> @@ -104,6 +116,12 @@
>>> linux,code = <KEY_RESTART>;
>>> gpios = <&chipcommon 6 GPIO_ACTIVE_LOW>;
>>> };
>>> +
>>> + brightness {
>>> + label = "Backlight";
>>> + linux,code = <KEY_BRIGHTNESS_ZERO>;
>>> + gpios = <&chipcommon 19 GPIO_ACTIVE_LOW>;
>>> + };
>>> };
>>> };
>>>
>>> --
>>> 2.9.3
>>>
>>
>>
>>
>> --
>> Rafał
>
--
Florian
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web