From: Zelong Dong <Zelong.Dong@amlogic.com>
To: neil.armstrong@linaro.org, Sean Young <sean@mess.org>,
Mauro Carvalho Chehab <mchehab@kernel.org>,
Rob Herring <robh+dt@kernel.org>,
Jerome Brunet <jbrunet@baylibre.com>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
Conor Dooley <conor+dt@kernel.org>,
Kevin Hilman <khilman@baylibre.com>,
Martin Blumenstingl <martin.blumenstingl@googlemail.com>
Cc: linux-media@vger.kernel.org, linux-amlogic@lists.infradead.org,
devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
Qianggui.Song@amlogic.com, Yonghui.Yu@amlogic.com,
kelvin.zhang@amlogic.com
Subject: Re: [PATCH 1/3] media: rc: meson-ir: support rc driver type RC_DRIVER_SCANCODE
Date: Wed, 6 Sep 2023 15:11:58 +0800 [thread overview]
Message-ID: <646b1955-c89f-a6f5-acd1-a3c68953b0c6@amlogic.com> (raw)
In-Reply-To: <fcdb1fb2-784d-4235-aa47-9e489395c028@linaro.org>
在 2023/8/25 20:19, Neil Armstrong 写道:
> On 25/08/2023 13:53, zelong dong wrote:
>> From: Zelong Dong <zelong.dong@amlogic.com>
>>
>> Meson IR Controller supports hardware decoder in Meson-S4 and later
>> SoC. So far, protocol NEC could be decoded by hardware decoder.
>>
>> Signed-off-by: Zelong Dong <zelong.dong@amlogic.com>
>> ---
>> drivers/media/rc/meson-ir.c | 522 +++++++++++++++++++++++++++++++-----
>> 1 file changed, 451 insertions(+), 71 deletions(-)
>>
>
> <snip>
>
> It looks pretty good now, so far I'm ok with it:
>
> Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org>
>
> I think some test reports would be needed to be sure it doesn't regress
> existing HW.
>
> Thanks,
> Neil
Sorry for my late reply, I spent a long time looking for development boards.
I searched Meson-IR DT Node from all the Amlogic DT, only 4 DTSI as follow:
arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi
arch/arm64/boot/dts/amlogic/meson-axg.dtsi
arch/arm64/boot/dts/amlogic/meson-gx.dtsi
arch/arm/boot/dts/amlogic/meson.dtsi
So I planned to sample the boards to test:
arch/arm64/boot/dts/amlogic/meson-g12a-u200.dts
arch/arm64/boot/dts/amlogic/meson-axg-s400.dts
arch/arm64/boot/dts/amlogic/meson-gxl-s905x-p212.dts
arch/arm/boot/dts/amlogic/meson8b-odroidc1.dts
Unfortunately, meson-gxl-s905x-p212 & meson8b-odroidc1 are too old to be
find.
there are only 2 boards I got, meson-g12a-u200 & meson-axg-s400, and
they worked.
[ 0.943097] IR NEC protocol handler initialized
[ 0.943146] IR RC5(x/sz) protocol handler initialized
[ 0.947029] IR RC6 protocol handler initialized
[ 0.951508] IR RCMM protocol handler initialized
[ 0.956080] IR XMP protocol handler initialized
[ 0.961266] Registered IR keymap rc-empty
[ 0.964674] rc rc0: meson-ir as
/devices/platform/soc/ff800000.bus/ff808000.ir/rc/rc0
[ 0.972437] input: meson-ir as
/devices/platform/soc/ff800000.bus/ff808000.ir/rc/rc0/input0
[ 0.980983] rc rc0: sw decoder init
[ 0.984089] meson-ir ff808000.ir: receiver initialized
# cd /sys/devices/platform/soc/ff800000.bus/ff808000.ir/rc/rc0/
# cat protocols
rc-5 nec rc-6 rc-5-sz xmp rc-mm
# echo nec > protocols
# hexdump /dev/input/event0
0000000 0069 0000 3180 0005 0004 0004 011b 0000
0000010 0069 0000 3180 0005 0000 0000 0000 0000
0000020 0069 0000 fc80 0005 0004 0004 011b 0000
0000030 0069 0000 fc80 0005 0000 0000 0000 0000
0000040 0069 0000 92db 0007 0004 0004 011b 0000
0000050 0069 0000 92db 0007 0000 0000 0000 0000
0000060 0069 0000 388b 0009 0004 0004 011b 0000
0000070 0069 0000 388b 0009 0000 0000 0000 0000
0000080 0069 0000 de72 000a 0004 0004 011b 0000
0000090 0069 0000 de72 000a 0000 0000 0000 0000
next prev parent reply other threads:[~2023-09-06 7:12 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-08-25 11:53 [PATCH 0/3] This patchset adds IR controller driver support for zelong dong
2023-08-25 11:53 ` [PATCH 1/3] media: rc: meson-ir: support rc driver type RC_DRIVER_SCANCODE zelong dong
2023-08-25 12:19 ` Neil Armstrong
2023-09-06 7:11 ` Zelong Dong [this message]
2023-09-06 7:17 ` neil.armstrong
2023-08-29 7:39 ` Sean Young
2023-08-31 12:13 ` Zelong Dong
2023-09-01 8:02 ` Sean Young
2023-09-06 10:37 ` Zelong Dong
2023-09-06 12:57 ` Sean Young
2023-08-25 11:53 ` [PATCH 2/3] dt-bindings: media: Add compatible for Meson-S4 IR Controller zelong dong
2023-08-25 12:03 ` Krzysztof Kozlowski
2023-08-25 11:53 ` [PATCH 3/3] arm64: dts: meson: add IR controller for Meson-S4 SoC zelong dong
2023-08-25 12:16 ` Neil Armstrong
2023-09-11 9:46 ` (subset) [PATCH 0/3] This patchset adds IR controller driver support for Neil Armstrong
2023-09-11 10:30 ` Sean Young
2023-09-11 10:35 ` Neil Armstrong
2023-09-11 10:40 ` Krzysztof Kozlowski
-- strict thread matches above, loose matches on Subject: below --
2023-03-02 6:33 [PATCH 0/3] media: rc: meson-s4: support RC_DRIVER_SCANCODE driver zelong dong
2023-03-02 6:34 ` [PATCH 1/3] media: rc: meson-ir: support rc driver type RC_DRIVER_SCANCODE zelong dong
2023-03-02 9:27 ` Neil Armstrong
2023-03-03 13:37 ` Dmitry Rokosov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=646b1955-c89f-a6f5-acd1-a3c68953b0c6@amlogic.com \
--to=zelong.dong@amlogic.com \
--cc=Qianggui.Song@amlogic.com \
--cc=Yonghui.Yu@amlogic.com \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=jbrunet@baylibre.com \
--cc=kelvin.zhang@amlogic.com \
--cc=khilman@baylibre.com \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=linux-amlogic@lists.infradead.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-media@vger.kernel.org \
--cc=martin.blumenstingl@googlemail.com \
--cc=mchehab@kernel.org \
--cc=neil.armstrong@linaro.org \
--cc=robh+dt@kernel.org \
--cc=sean@mess.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).