Re: [PATCH v2 0/3] media: rc: sunxi-cir: support the A523/H728/T527 IR receiver

Andre Przywara <[email protected]>
Newsgroups dev.linux.lists.linux-sunxi,org.infradead.lists.linux-arm-kernel,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-media
Organization Arm Ltd.
Message-ID <[email protected]>
On Wed, 22 Jul 2026 16:17:14 -0400
Justin Suess <[email protected]> wrote:

Hi Justin,

> Sorry for the delay,

no worries, thanks for the update.

> This is v2 of the series that adds support for the CIR
> receiver found in the Allwinner A523/T527/H728 family (sun55i).
> 
> Unlike the previous version of the series, this version has no driver changes,
> and just uses a fallback compatible and enables the receiver.
> 
> Tested again on my X96Q Pro+ (H728) TV box with the Google TV remote that
> came in the box (using NEC keymap). Was able to press buttons and
> get readings from lirc / ir-keytable again.

this all looks good to me. As mentioned on the individual patches, I
tested this on two boards here. I sent a patch [1] to enable IR on the
Avaota board, which goes on top of this series.

[1]
https://lore.kernel.org/linux-sunxi/[email protected]/T/#u
 
> The series is (re)based on tag v7.2-rc4 on mainline.

Please base your submissions on -rc1, that simplifies merging/rebasing
for the maintainers, because then all new patches have the same base.
Not a particular problem for this series, since everything still applies
cleanly, but keep that in mind for future submissions.

Cheers,
Andre

> 
> Justin Suess (3):
>   media: dt-bindings: allwinner,sun4i-a10-ir: add A523 compatible
>   arm64: dts: allwinner: a523: add IR receiver nodes
>   arm64: dts: allwinner: a523: enable IR receiver on the X96Q Pro+
> 
>  .../media/allwinner,sun4i-a10-ir.yaml         |  8 ++++--
>  .../arm64/boot/dts/allwinner/sun55i-a523.dtsi | 28 +++++++++++++++++++
>  .../dts/allwinner/sun55i-h728-x96qpro+.dts    |  6 ++++
>  3 files changed, 39 insertions(+), 3 deletions(-)
> 
> 
> base-commit: 1590cf0329716306e948a8fc29f1d3ee87d3989f
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.