diff mbox series

arm64: dts: rockchip: Fix IR on Beelink A1

Message ID 44269c08e2a5d75b03ded87d2eb11621762d8249.1577636223.git.robin.murphy@arm.com (mailing list archive)
State New, archived
Headers show
Series arm64: dts: rockchip: Fix IR on Beelink A1 | expand

Commit Message

Robin Murphy Dec. 29, 2019, 5:48 p.m. UTC
Apparently I wasn't paying enough attention... And nor is the lazy
test of `cat /dev/lirc0` sufficiently blunder-proof. Oh well, with
the correct polarity, let's also hook up a keymap now that one for
the standard Beelink remote has handily appeared.

Signed-off-by: Robin Murphy <robin.murphy@arm.com>
---
 arch/arm64/boot/dts/rockchip/rk3328-a1.dts | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Heiko Stuebner Dec. 31, 2019, 11:50 a.m. UTC | #1
Am Sonntag, 29. Dezember 2019, 18:48:05 CET schrieb Robin Murphy:
> Apparently I wasn't paying enough attention... And nor is the lazy
> test of `cat /dev/lirc0` sufficiently blunder-proof. Oh well, with
> the correct polarity, let's also hook up a keymap now that one for
> the standard Beelink remote has handily appeared.
> 
> Signed-off-by: Robin Murphy <robin.murphy@arm.com>
> ---

applied for 5.6

Thanks
Heiko
Robin Murphy Jan. 1, 2020, 1:01 p.m. UTC | #2
On 2019-12-31 11:50 am, Heiko Stuebner wrote:
> Am Sonntag, 29. Dezember 2019, 18:48:05 CET schrieb Robin Murphy:
>> Apparently I wasn't paying enough attention... And nor is the lazy
>> test of `cat /dev/lirc0` sufficiently blunder-proof. Oh well, with
>> the correct polarity, let's also hook up a keymap now that one for
>> the standard Beelink remote has handily appeared.
>>
>> Signed-off-by: Robin Murphy <robin.murphy@arm.com>
>> ---
> 
> applied for 5.6

Thanks Heiko - although if you have any other fixes to send for 5.5 do 
you think this might be able to sneak in too? Someone mentioned on IRC 
that unlike mine, their box wouldn't even generate interrupts with the 
polarity backwards.

Cheers,
Robin.
Heiko Stuebner Jan. 6, 2020, 2:23 p.m. UTC | #3
Hi Robin,

Am Mittwoch, 1. Januar 2020, 14:01:32 CET schrieb Robin Murphy:
> On 2019-12-31 11:50 am, Heiko Stuebner wrote:
> > Am Sonntag, 29. Dezember 2019, 18:48:05 CET schrieb Robin Murphy:
> >> Apparently I wasn't paying enough attention... And nor is the lazy
> >> test of `cat /dev/lirc0` sufficiently blunder-proof. Oh well, with
> >> the correct polarity, let's also hook up a keymap now that one for
> >> the standard Beelink remote has handily appeared.
> >>
> >> Signed-off-by: Robin Murphy <robin.murphy@arm.com>
> >> ---
> > 
> > applied for 5.6
> 
> Thanks Heiko - although if you have any other fixes to send for 5.5 do 
> you think this might be able to sneak in too? Someone mentioned on IRC 
> that unlike mine, their box wouldn't even generate interrupts with the 
> polarity backwards.

I've moved the patch over to a shiny new fixes branch for 5.5 now :-)

Heiko
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/rockchip/rk3328-a1.dts b/arch/arm64/boot/dts/rockchip/rk3328-a1.dts
index 76b49f573101..16f1656d5203 100644
--- a/arch/arm64/boot/dts/rockchip/rk3328-a1.dts
+++ b/arch/arm64/boot/dts/rockchip/rk3328-a1.dts
@@ -49,7 +49,8 @@ 
 
 	ir-receiver {
 		compatible = "gpio-ir-receiver";
-		gpios = <&gpio2 RK_PA2 GPIO_ACTIVE_HIGH>;
+		gpios = <&gpio2 RK_PA2 GPIO_ACTIVE_LOW>;
+		linux,rc-map-name = "rc-beelink-gs1";
 	};
 };