* [v6 1/4] dt-bindings: media: rc: add keymap bindings for MagicBox M16S remote
[not found] <20220726131523.140581-1-zhangn1985@qq.com>
@ 2022-07-26 13:15 ` Zhang Ning
2022-07-26 13:15 ` [v6 2/4] media: rc: add keymap " Zhang Ning
` (2 subsequent siblings)
3 siblings, 0 replies; 8+ messages in thread
From: Zhang Ning @ 2022-07-26 13:15 UTC (permalink / raw)
To: robh+dt, krzysztof.kozlowski+dt, narmstrong, martin.blumenstingl,
sean, linux-amlogic, linux-media, devicetree
Cc: Zhang Ning
MagicBox M16S Tv box shipped with a simple NEC remote.
Signed-off-by: Zhang Ning <zhangn1985@qq.com>
---
Documentation/devicetree/bindings/media/rc.yaml | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/media/rc.yaml b/Documentation/devicetree/bindings/media/rc.yaml
index d4c541c4b164..6491d1bdbac7 100644
--- a/Documentation/devicetree/bindings/media/rc.yaml
+++ b/Documentation/devicetree/bindings/media/rc.yaml
@@ -90,6 +90,7 @@ properties:
- rc-kworld-plus-tv-analog
- rc-leadtek-y04g0051
- rc-lme2510
+ - rc-magicbox
- rc-manli
- rc-mecool-kii-pro
- rc-mecool-kiii-pro
--
2.35.1
_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [v6 2/4] media: rc: add keymap for MagicBox M16S remote
[not found] <20220726131523.140581-1-zhangn1985@qq.com>
2022-07-26 13:15 ` [v6 1/4] dt-bindings: media: rc: add keymap bindings for MagicBox M16S remote Zhang Ning
@ 2022-07-26 13:15 ` Zhang Ning
2022-07-26 13:39 ` Zhang Ning
2022-07-26 13:15 ` [v6 3/4] dt-bindings: arm: amlogic: add MagicBox M16S bindings Zhang Ning
2022-07-26 13:15 ` [v6 4/4] arm64: dts: meson: Add MagicBox M16S support Zhang Ning
3 siblings, 1 reply; 8+ messages in thread
From: Zhang Ning @ 2022-07-26 13:15 UTC (permalink / raw)
To: robh+dt, krzysztof.kozlowski+dt, narmstrong, martin.blumenstingl,
sean, linux-amlogic, linux-media, devicetree
Cc: Zhang Ning
MagicBox M16S Tv box shipped with a simple NEC remote.
Signed-off-by: Zhang Ning <zhangn1985@qq.com>
---
drivers/media/rc/keymaps/Makefile | 1 +
drivers/media/rc/keymaps/rc-magicbox.c | 58 ++++++++++++++++++++++++++
include/media/rc-map.h | 1 +
3 files changed, 60 insertions(+)
create mode 100644 drivers/media/rc/keymaps/rc-magicbox.c
diff --git a/drivers/media/rc/keymaps/Makefile b/drivers/media/rc/keymaps/Makefile
index f513ff5caf4e..02c1c2150f03 100644
--- a/drivers/media/rc/keymaps/Makefile
+++ b/drivers/media/rc/keymaps/Makefile
@@ -71,6 +71,7 @@ obj-$(CONFIG_RC_MAP) += \
rc-kworld-plus-tv-analog.o \
rc-leadtek-y04g0051.o \
rc-lme2510.o \
+ rc-magicbox.o \
rc-manli.o \
rc-mecool-kiii-pro.o \
rc-mecool-kii-pro.o \
diff --git a/drivers/media/rc/keymaps/rc-magicbox.c b/drivers/media/rc/keymaps/rc-magicbox.c
new file mode 100644
index 000000000000..015767429028
--- /dev/null
+++ b/drivers/media/rc/keymaps/rc-magicbox.c
@@ -0,0 +1,58 @@
+// SPDX-License-Identifier: GPL-2.0+
+//
+// Copyright (C) 2022 Zhang Ning <zhangn1985@qq.com>
+
+/*
+ * Keytable for the MagicBox M16S remote control
+ */
+
+#include <media/rc-map.h>
+#include <linux/module.h>
+
+static struct rc_map_table magicbox[] = {
+ { 0x9f57, KEY_POWER },
+
+ /*
+ * This key labeled "M" is used as magic key in vendor OS,
+ * and mapped to KEY_MUTE.
+ */
+ { 0x9f8a, KEY_MUTE },
+
+ { 0x9f43, KEY_UP },
+ { 0x9f0a, KEY_DOWN },
+ { 0x9f06, KEY_LEFT },
+ { 0x9f0e, KEY_RIGHT },
+ { 0x9f02, KEY_OK },
+
+ { 0x9f47, KEY_HOME },
+ { 0x9f4f, KEY_BACK },
+ { 0x9f16, KEY_MENU },
+
+ { 0x9fff, KEY_VOLUMEUP },
+ { 0x9f5d, KEY_VOLUMEDOWN },
+};
+
+static struct rc_map_list magicbox_map = {
+ .map = {
+ .scan = magicbox,
+ .size = ARRAY_SIZE(magicbox),
+ .rc_proto = RC_PROTO_NEC,
+ .name = RC_MAP_MAGICBOX,
+ }
+};
+
+static int __init init_rc_map_magicbox(void)
+{
+ return rc_map_register(&magicbox_map);
+}
+
+static void __exit exit_rc_map_magicbox(void)
+{
+ rc_map_unregister(&magicbox_map);
+}
+
+module_init(init_rc_map_magicbox)
+module_exit(exit_rc_map_magicbox)
+
+MODULE_LICENSE("GPL");
+MODULE_AUTHOR("Zhang Ning <zhangn1985@qq.com>");
diff --git a/include/media/rc-map.h b/include/media/rc-map.h
index 793b54342dff..656217b8e91b 100644
--- a/include/media/rc-map.h
+++ b/include/media/rc-map.h
@@ -277,6 +277,7 @@ struct rc_map *rc_map_get(const char *name);
#define RC_MAP_KWORLD_PLUS_TV_ANALOG "rc-kworld-plus-tv-analog"
#define RC_MAP_LEADTEK_Y04G0051 "rc-leadtek-y04g0051"
#define RC_MAP_LME2510 "rc-lme2510"
+#define RC_MAP_MAGICBOX "rc-magicbox"
#define RC_MAP_MANLI "rc-manli"
#define RC_MAP_MECOOL_KII_PRO "rc-mecool-kii-pro"
#define RC_MAP_MECOOL_KIII_PRO "rc-mecool-kiii-pro"
--
2.35.1
_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [v6 3/4] dt-bindings: arm: amlogic: add MagicBox M16S bindings
[not found] <20220726131523.140581-1-zhangn1985@qq.com>
2022-07-26 13:15 ` [v6 1/4] dt-bindings: media: rc: add keymap bindings for MagicBox M16S remote Zhang Ning
2022-07-26 13:15 ` [v6 2/4] media: rc: add keymap " Zhang Ning
@ 2022-07-26 13:15 ` Zhang Ning
2022-07-26 13:15 ` [v6 4/4] arm64: dts: meson: Add MagicBox M16S support Zhang Ning
3 siblings, 0 replies; 8+ messages in thread
From: Zhang Ning @ 2022-07-26 13:15 UTC (permalink / raw)
To: robh+dt, krzysztof.kozlowski+dt, narmstrong, martin.blumenstingl,
sean, linux-amlogic, linux-media, devicetree
Cc: Zhang Ning
Add the board binding for Magicbox M16S Tv Box
Signed-off-by: Zhang Ning <zhangn1985@qq.com>
---
Documentation/devicetree/bindings/arm/amlogic.yaml | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/arm/amlogic.yaml b/Documentation/devicetree/bindings/arm/amlogic.yaml
index 61a6cabb375b..3eac16a4de68 100644
--- a/Documentation/devicetree/bindings/arm/amlogic.yaml
+++ b/Documentation/devicetree/bindings/arm/amlogic.yaml
@@ -123,6 +123,7 @@ properties:
- khadas,vim2
- kingnovel,r-box-pro
- libretech,aml-s912-pc
+ - magicbox,m16s
- minix,neo-u9h
- nexbox,a1
- tronsmart,vega-s96
--
2.35.1
_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [v6 4/4] arm64: dts: meson: Add MagicBox M16S support
[not found] <20220726131523.140581-1-zhangn1985@qq.com>
` (2 preceding siblings ...)
2022-07-26 13:15 ` [v6 3/4] dt-bindings: arm: amlogic: add MagicBox M16S bindings Zhang Ning
@ 2022-07-26 13:15 ` Zhang Ning
3 siblings, 0 replies; 8+ messages in thread
From: Zhang Ning @ 2022-07-26 13:15 UTC (permalink / raw)
To: robh+dt, krzysztof.kozlowski+dt, narmstrong, martin.blumenstingl,
sean, linux-amlogic, linux-media, devicetree
Cc: Zhang Ning
MagicBox M16S or MagicBox 3Pro is popular Tv box in China.
with spec:
- Amlogic S912-H (S912 with dolby and dts)
- 2G ddr3
- 16G emmc
- Marvell sd8897 BT/wifi module
- 100M ethernet
- IR reciever
- 4K HDMI
- AV out
- Rest hole
- 5V2A power input
- white LED
it's q201_v1 according u-boot log.
and it's almost same as Q201 reference design.
add a simple dts to support this Tv box.
Signed-off-by: Zhang Ning <zhangn1985@qq.com>
---
arch/arm64/boot/dts/amlogic/Makefile | 1 +
.../dts/amlogic/meson-gxm-magicbox-m16s.dts | 39 +++++++++++++++++++
2 files changed, 40 insertions(+)
create mode 100644 arch/arm64/boot/dts/amlogic/meson-gxm-magicbox-m16s.dts
diff --git a/arch/arm64/boot/dts/amlogic/Makefile b/arch/arm64/boot/dts/amlogic/Makefile
index 8773211df50e..e0907fb41829 100644
--- a/arch/arm64/boot/dts/amlogic/Makefile
+++ b/arch/arm64/boot/dts/amlogic/Makefile
@@ -44,6 +44,7 @@ dtb-$(CONFIG_ARCH_MESON) += meson-gxl-s905x-libretech-cc.dtb
dtb-$(CONFIG_ARCH_MESON) += meson-gxl-s905x-nexbox-a95x.dtb
dtb-$(CONFIG_ARCH_MESON) += meson-gxl-s905x-p212.dtb
dtb-$(CONFIG_ARCH_MESON) += meson-gxm-khadas-vim2.dtb
+dtb-$(CONFIG_ARCH_MESON) += meson-gxm-magicbox-m16s.dtb
dtb-$(CONFIG_ARCH_MESON) += meson-gxm-mecool-kiii-pro.dtb
dtb-$(CONFIG_ARCH_MESON) += meson-gxm-minix-neo-u9h.dtb
dtb-$(CONFIG_ARCH_MESON) += meson-gxm-nexbox-a1.dtb
diff --git a/arch/arm64/boot/dts/amlogic/meson-gxm-magicbox-m16s.dts b/arch/arm64/boot/dts/amlogic/meson-gxm-magicbox-m16s.dts
new file mode 100644
index 000000000000..f1805115b177
--- /dev/null
+++ b/arch/arm64/boot/dts/amlogic/meson-gxm-magicbox-m16s.dts
@@ -0,0 +1,39 @@
+// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
+/*
+ * Copyright (c) 2022 Zhang Ning <zhangn1985@qq.com>
+ */
+
+/dts-v1/;
+
+#include "meson-gxm.dtsi"
+#include "meson-gx-p23x-q20x.dtsi"
+#include <dt-bindings/input/input.h>
+
+/ {
+ compatible = "magicbox,m16s", "amlogic,s912", "amlogic,meson-gxm";
+ model = "MagicBox M16S";
+
+ gpio-keys {
+ compatible = "gpio-keys-polled";
+ poll-interval = <100>;
+
+ key-power {
+ label = "power";
+ linux,code = <KEY_POWER>;
+ gpios = <&gpio_ao GPIOAO_2 GPIO_ACTIVE_LOW>;
+ };
+ };
+};
+
+ðmac {
+ phy-mode = "rmii";
+ phy-handle = <&internal_phy>;
+};
+
+&ir {
+ linux,rc-map-name = "rc-magicbox";
+};
+
+&sdio_pwrseq {
+ reset-gpios = <&gpio GPIODV_2 GPIO_ACTIVE_LOW>;
+};
--
2.35.1
_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [v6 2/4] media: rc: add keymap for MagicBox M16S remote
2022-07-26 13:15 ` [v6 2/4] media: rc: add keymap " Zhang Ning
@ 2022-07-26 13:39 ` Zhang Ning
2022-07-26 13:59 ` Sean Young
0 siblings, 1 reply; 8+ messages in thread
From: Zhang Ning @ 2022-07-26 13:39 UTC (permalink / raw)
To: robh+dt, krzysztof.kozlowski+dt, narmstrong, martin.blumenstingl,
sean, linux-amlogic, linux-media, devicetree
On Tue, Jul 26, 2022 at 09:15:21PM +0800, Zhang Ning wrote:
> MagicBox M16S Tv box shipped with a simple NEC remote.
>
> Signed-off-by: Zhang Ning <zhangn1985@qq.com>
> ---
> drivers/media/rc/keymaps/Makefile | 1 +
> drivers/media/rc/keymaps/rc-magicbox.c | 58 ++++++++++++++++++++++++++
> include/media/rc-map.h | 1 +
> 3 files changed, 60 insertions(+)
> create mode 100644 drivers/media/rc/keymaps/rc-magicbox.c
>
> diff --git a/drivers/media/rc/keymaps/Makefile b/drivers/media/rc/keymaps/Makefile
> index f513ff5caf4e..02c1c2150f03 100644
> --- a/drivers/media/rc/keymaps/Makefile
> +++ b/drivers/media/rc/keymaps/Makefile
> @@ -71,6 +71,7 @@ obj-$(CONFIG_RC_MAP) += \
> rc-kworld-plus-tv-analog.o \
> rc-leadtek-y04g0051.o \
> rc-lme2510.o \
> + rc-magicbox.o \
> rc-manli.o \
> rc-mecool-kiii-pro.o \
> rc-mecool-kii-pro.o \
> diff --git a/drivers/media/rc/keymaps/rc-magicbox.c b/drivers/media/rc/keymaps/rc-magicbox.c
> new file mode 100644
> index 000000000000..015767429028
> --- /dev/null
> +++ b/drivers/media/rc/keymaps/rc-magicbox.c
> @@ -0,0 +1,58 @@
> +// SPDX-License-Identifier: GPL-2.0+
> +//
> +// Copyright (C) 2022 Zhang Ning <zhangn1985@qq.com>
> +
> +/*
> + * Keytable for the MagicBox M16S remote control
> + */
> +
> +#include <media/rc-map.h>
> +#include <linux/module.h>
> +
> +static struct rc_map_table magicbox[] = {
> + { 0x9f57, KEY_POWER },
> +
> + /*
> + * This key labeled "M" is used as magic key in vendor OS,
> + * and mapped to KEY_MUTE.
oh no, mixed tab and space. I will fix it next version.
> + */
> + { 0x9f8a, KEY_MUTE },
> +
> + { 0x9f43, KEY_UP },
> + { 0x9f0a, KEY_DOWN },
> + { 0x9f06, KEY_LEFT },
> + { 0x9f0e, KEY_RIGHT },
> + { 0x9f02, KEY_OK },
> +
> + { 0x9f47, KEY_HOME },
> + { 0x9f4f, KEY_BACK },
> + { 0x9f16, KEY_MENU },
> +
> + { 0x9fff, KEY_VOLUMEUP },
> + { 0x9f5d, KEY_VOLUMEDOWN },
> +};
> +
> +static struct rc_map_list magicbox_map = {
> + .map = {
> + .scan = magicbox,
> + .size = ARRAY_SIZE(magicbox),
> + .rc_proto = RC_PROTO_NEC,
> + .name = RC_MAP_MAGICBOX,
> + }
> +};
> +
> +static int __init init_rc_map_magicbox(void)
> +{
> + return rc_map_register(&magicbox_map);
> +}
> +
> +static void __exit exit_rc_map_magicbox(void)
> +{
> + rc_map_unregister(&magicbox_map);
> +}
> +
> +module_init(init_rc_map_magicbox)
> +module_exit(exit_rc_map_magicbox)
> +
> +MODULE_LICENSE("GPL");
> +MODULE_AUTHOR("Zhang Ning <zhangn1985@qq.com>");
> diff --git a/include/media/rc-map.h b/include/media/rc-map.h
> index 793b54342dff..656217b8e91b 100644
> --- a/include/media/rc-map.h
> +++ b/include/media/rc-map.h
> @@ -277,6 +277,7 @@ struct rc_map *rc_map_get(const char *name);
> #define RC_MAP_KWORLD_PLUS_TV_ANALOG "rc-kworld-plus-tv-analog"
> #define RC_MAP_LEADTEK_Y04G0051 "rc-leadtek-y04g0051"
> #define RC_MAP_LME2510 "rc-lme2510"
> +#define RC_MAP_MAGICBOX "rc-magicbox"
> #define RC_MAP_MANLI "rc-manli"
> #define RC_MAP_MECOOL_KII_PRO "rc-mecool-kii-pro"
> #define RC_MAP_MECOOL_KIII_PRO "rc-mecool-kiii-pro"
> --
> 2.35.1
>
_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [v6 2/4] media: rc: add keymap for MagicBox M16S remote
2022-07-26 13:39 ` Zhang Ning
@ 2022-07-26 13:59 ` Sean Young
0 siblings, 0 replies; 8+ messages in thread
From: Sean Young @ 2022-07-26 13:59 UTC (permalink / raw)
To: Zhang Ning
Cc: robh+dt, krzysztof.kozlowski+dt, narmstrong, martin.blumenstingl,
linux-amlogic, linux-media, devicetree
On Tue, Jul 26, 2022 at 09:39:46PM +0800, Zhang Ning wrote:
> On Tue, Jul 26, 2022 at 09:15:21PM +0800, Zhang Ning wrote:
> > MagicBox M16S Tv box shipped with a simple NEC remote.
> >
> > Signed-off-by: Zhang Ning <zhangn1985@qq.com>
> > ---
> > drivers/media/rc/keymaps/Makefile | 1 +
> > drivers/media/rc/keymaps/rc-magicbox.c | 58 ++++++++++++++++++++++++++
> > include/media/rc-map.h | 1 +
> > 3 files changed, 60 insertions(+)
> > create mode 100644 drivers/media/rc/keymaps/rc-magicbox.c
> >
> > diff --git a/drivers/media/rc/keymaps/Makefile b/drivers/media/rc/keymaps/Makefile
> > index f513ff5caf4e..02c1c2150f03 100644
> > --- a/drivers/media/rc/keymaps/Makefile
> > +++ b/drivers/media/rc/keymaps/Makefile
> > @@ -71,6 +71,7 @@ obj-$(CONFIG_RC_MAP) += \
> > rc-kworld-plus-tv-analog.o \
> > rc-leadtek-y04g0051.o \
> > rc-lme2510.o \
> > + rc-magicbox.o \
> > rc-manli.o \
> > rc-mecool-kiii-pro.o \
> > rc-mecool-kii-pro.o \
> > diff --git a/drivers/media/rc/keymaps/rc-magicbox.c b/drivers/media/rc/keymaps/rc-magicbox.c
> > new file mode 100644
> > index 000000000000..015767429028
> > --- /dev/null
> > +++ b/drivers/media/rc/keymaps/rc-magicbox.c
> > @@ -0,0 +1,58 @@
> > +// SPDX-License-Identifier: GPL-2.0+
> > +//
> > +// Copyright (C) 2022 Zhang Ning <zhangn1985@qq.com>
> > +
> > +/*
> > + * Keytable for the MagicBox M16S remote control
> > + */
> > +
> > +#include <media/rc-map.h>
> > +#include <linux/module.h>
> > +
> > +static struct rc_map_table magicbox[] = {
> > + { 0x9f57, KEY_POWER },
> > +
> > + /*
> > + * This key labeled "M" is used as magic key in vendor OS,
> > + * and mapped to KEY_MUTE.
> oh no, mixed tab and space. I will fix it next version.
That's something minor that can be fixed when merging.
Looks good to me, thank you for dealing with all the pedantic reviews.
Signed-off-by: Sean Young <sean@mess.org>
>
> > + */
> > + { 0x9f8a, KEY_MUTE },
> > +
> > + { 0x9f43, KEY_UP },
> > + { 0x9f0a, KEY_DOWN },
> > + { 0x9f06, KEY_LEFT },
> > + { 0x9f0e, KEY_RIGHT },
> > + { 0x9f02, KEY_OK },
> > +
> > + { 0x9f47, KEY_HOME },
> > + { 0x9f4f, KEY_BACK },
> > + { 0x9f16, KEY_MENU },
> > +
> > + { 0x9fff, KEY_VOLUMEUP },
> > + { 0x9f5d, KEY_VOLUMEDOWN },
> > +};
> > +
> > +static struct rc_map_list magicbox_map = {
> > + .map = {
> > + .scan = magicbox,
> > + .size = ARRAY_SIZE(magicbox),
> > + .rc_proto = RC_PROTO_NEC,
> > + .name = RC_MAP_MAGICBOX,
> > + }
> > +};
> > +
> > +static int __init init_rc_map_magicbox(void)
> > +{
> > + return rc_map_register(&magicbox_map);
> > +}
> > +
> > +static void __exit exit_rc_map_magicbox(void)
> > +{
> > + rc_map_unregister(&magicbox_map);
> > +}
> > +
> > +module_init(init_rc_map_magicbox)
> > +module_exit(exit_rc_map_magicbox)
> > +
> > +MODULE_LICENSE("GPL");
> > +MODULE_AUTHOR("Zhang Ning <zhangn1985@qq.com>");
> > diff --git a/include/media/rc-map.h b/include/media/rc-map.h
> > index 793b54342dff..656217b8e91b 100644
> > --- a/include/media/rc-map.h
> > +++ b/include/media/rc-map.h
> > @@ -277,6 +277,7 @@ struct rc_map *rc_map_get(const char *name);
> > #define RC_MAP_KWORLD_PLUS_TV_ANALOG "rc-kworld-plus-tv-analog"
> > #define RC_MAP_LEADTEK_Y04G0051 "rc-leadtek-y04g0051"
> > #define RC_MAP_LME2510 "rc-lme2510"
> > +#define RC_MAP_MAGICBOX "rc-magicbox"
> > #define RC_MAP_MANLI "rc-manli"
> > #define RC_MAP_MECOOL_KII_PRO "rc-mecool-kii-pro"
> > #define RC_MAP_MECOOL_KIII_PRO "rc-mecool-kiii-pro"
> > --
> > 2.35.1
> >
_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [v6 2/4] media: rc: add keymap for MagicBox M16S remote
@ 2022-07-26 13:59 ` Sean Young
0 siblings, 0 replies; 8+ messages in thread
From: Sean Young @ 2022-07-26 13:59 UTC (permalink / raw)
To: Zhang Ning
Cc: robh+dt, krzysztof.kozlowski+dt, narmstrong, martin.blumenstingl,
linux-amlogic, linux-media, devicetree
On Tue, Jul 26, 2022 at 09:39:46PM +0800, Zhang Ning wrote:
> On Tue, Jul 26, 2022 at 09:15:21PM +0800, Zhang Ning wrote:
> > MagicBox M16S Tv box shipped with a simple NEC remote.
> >
> > Signed-off-by: Zhang Ning <zhangn1985@qq.com>
> > ---
> > drivers/media/rc/keymaps/Makefile | 1 +
> > drivers/media/rc/keymaps/rc-magicbox.c | 58 ++++++++++++++++++++++++++
> > include/media/rc-map.h | 1 +
> > 3 files changed, 60 insertions(+)
> > create mode 100644 drivers/media/rc/keymaps/rc-magicbox.c
> >
> > diff --git a/drivers/media/rc/keymaps/Makefile b/drivers/media/rc/keymaps/Makefile
> > index f513ff5caf4e..02c1c2150f03 100644
> > --- a/drivers/media/rc/keymaps/Makefile
> > +++ b/drivers/media/rc/keymaps/Makefile
> > @@ -71,6 +71,7 @@ obj-$(CONFIG_RC_MAP) += \
> > rc-kworld-plus-tv-analog.o \
> > rc-leadtek-y04g0051.o \
> > rc-lme2510.o \
> > + rc-magicbox.o \
> > rc-manli.o \
> > rc-mecool-kiii-pro.o \
> > rc-mecool-kii-pro.o \
> > diff --git a/drivers/media/rc/keymaps/rc-magicbox.c b/drivers/media/rc/keymaps/rc-magicbox.c
> > new file mode 100644
> > index 000000000000..015767429028
> > --- /dev/null
> > +++ b/drivers/media/rc/keymaps/rc-magicbox.c
> > @@ -0,0 +1,58 @@
> > +// SPDX-License-Identifier: GPL-2.0+
> > +//
> > +// Copyright (C) 2022 Zhang Ning <zhangn1985@qq.com>
> > +
> > +/*
> > + * Keytable for the MagicBox M16S remote control
> > + */
> > +
> > +#include <media/rc-map.h>
> > +#include <linux/module.h>
> > +
> > +static struct rc_map_table magicbox[] = {
> > + { 0x9f57, KEY_POWER },
> > +
> > + /*
> > + * This key labeled "M" is used as magic key in vendor OS,
> > + * and mapped to KEY_MUTE.
> oh no, mixed tab and space. I will fix it next version.
That's something minor that can be fixed when merging.
Looks good to me, thank you for dealing with all the pedantic reviews.
Signed-off-by: Sean Young <sean@mess.org>
>
> > + */
> > + { 0x9f8a, KEY_MUTE },
> > +
> > + { 0x9f43, KEY_UP },
> > + { 0x9f0a, KEY_DOWN },
> > + { 0x9f06, KEY_LEFT },
> > + { 0x9f0e, KEY_RIGHT },
> > + { 0x9f02, KEY_OK },
> > +
> > + { 0x9f47, KEY_HOME },
> > + { 0x9f4f, KEY_BACK },
> > + { 0x9f16, KEY_MENU },
> > +
> > + { 0x9fff, KEY_VOLUMEUP },
> > + { 0x9f5d, KEY_VOLUMEDOWN },
> > +};
> > +
> > +static struct rc_map_list magicbox_map = {
> > + .map = {
> > + .scan = magicbox,
> > + .size = ARRAY_SIZE(magicbox),
> > + .rc_proto = RC_PROTO_NEC,
> > + .name = RC_MAP_MAGICBOX,
> > + }
> > +};
> > +
> > +static int __init init_rc_map_magicbox(void)
> > +{
> > + return rc_map_register(&magicbox_map);
> > +}
> > +
> > +static void __exit exit_rc_map_magicbox(void)
> > +{
> > + rc_map_unregister(&magicbox_map);
> > +}
> > +
> > +module_init(init_rc_map_magicbox)
> > +module_exit(exit_rc_map_magicbox)
> > +
> > +MODULE_LICENSE("GPL");
> > +MODULE_AUTHOR("Zhang Ning <zhangn1985@qq.com>");
> > diff --git a/include/media/rc-map.h b/include/media/rc-map.h
> > index 793b54342dff..656217b8e91b 100644
> > --- a/include/media/rc-map.h
> > +++ b/include/media/rc-map.h
> > @@ -277,6 +277,7 @@ struct rc_map *rc_map_get(const char *name);
> > #define RC_MAP_KWORLD_PLUS_TV_ANALOG "rc-kworld-plus-tv-analog"
> > #define RC_MAP_LEADTEK_Y04G0051 "rc-leadtek-y04g0051"
> > #define RC_MAP_LME2510 "rc-lme2510"
> > +#define RC_MAP_MAGICBOX "rc-magicbox"
> > #define RC_MAP_MANLI "rc-manli"
> > #define RC_MAP_MECOOL_KII_PRO "rc-mecool-kii-pro"
> > #define RC_MAP_MECOOL_KIII_PRO "rc-mecool-kiii-pro"
> > --
> > 2.35.1
> >
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [v6 2/4] media: rc: add keymap for MagicBox M16S remote
2022-07-26 13:59 ` Sean Young
(?)
@ 2022-07-26 14:19 ` Zhang Ning
-1 siblings, 0 replies; 8+ messages in thread
From: Zhang Ning @ 2022-07-26 14:19 UTC (permalink / raw)
To: Sean Young
Cc: robh+dt, krzysztof.kozlowski+dt, narmstrong, martin.blumenstingl,
linux-amlogic, linux-media, devicetree
On Tue, Jul 26, 2022 at 02:59:34PM +0100, Sean Young wrote:
> On Tue, Jul 26, 2022 at 09:39:46PM +0800, Zhang Ning wrote:
> > On Tue, Jul 26, 2022 at 09:15:21PM +0800, Zhang Ning wrote:
> > > MagicBox M16S Tv box shipped with a simple NEC remote.
> > >
> > > Signed-off-by: Zhang Ning <zhangn1985@qq.com>
> > > ---
> > > drivers/media/rc/keymaps/Makefile | 1 +
> > > drivers/media/rc/keymaps/rc-magicbox.c | 58 ++++++++++++++++++++++++++
> > > include/media/rc-map.h | 1 +
> > > 3 files changed, 60 insertions(+)
> > > create mode 100644 drivers/media/rc/keymaps/rc-magicbox.c
> > >
> > > diff --git a/drivers/media/rc/keymaps/Makefile b/drivers/media/rc/keymaps/Makefile
> > > index f513ff5caf4e..02c1c2150f03 100644
> > > --- a/drivers/media/rc/keymaps/Makefile
> > > +++ b/drivers/media/rc/keymaps/Makefile
> > > @@ -71,6 +71,7 @@ obj-$(CONFIG_RC_MAP) += \
> > > rc-kworld-plus-tv-analog.o \
> > > rc-leadtek-y04g0051.o \
> > > rc-lme2510.o \
> > > + rc-magicbox.o \
> > > rc-manli.o \
> > > rc-mecool-kiii-pro.o \
> > > rc-mecool-kii-pro.o \
> > > diff --git a/drivers/media/rc/keymaps/rc-magicbox.c b/drivers/media/rc/keymaps/rc-magicbox.c
> > > new file mode 100644
> > > index 000000000000..015767429028
> > > --- /dev/null
> > > +++ b/drivers/media/rc/keymaps/rc-magicbox.c
> > > @@ -0,0 +1,58 @@
> > > +// SPDX-License-Identifier: GPL-2.0+
> > > +//
> > > +// Copyright (C) 2022 Zhang Ning <zhangn1985@qq.com>
> > > +
> > > +/*
> > > + * Keytable for the MagicBox M16S remote control
> > > + */
> > > +
> > > +#include <media/rc-map.h>
> > > +#include <linux/module.h>
> > > +
> > > +static struct rc_map_table magicbox[] = {
> > > + { 0x9f57, KEY_POWER },
> > > +
> > > + /*
> > > + * This key labeled "M" is used as magic key in vendor OS,
> > > + * and mapped to KEY_MUTE.
> > oh no, mixed tab and space. I will fix it next version.
>
> That's something minor that can be fixed when merging.
>
> Looks good to me, thank you for dealing with all the pedantic reviews.
>
> Signed-off-by: Sean Young <sean@mess.org>
I want to send v7, due to checkpatch error.
I will add your Signed-off-by in next version.
thank you.
>
> >
> > > + */
> > > + { 0x9f8a, KEY_MUTE },
> > > +
> > > + { 0x9f43, KEY_UP },
> > > + { 0x9f0a, KEY_DOWN },
> > > + { 0x9f06, KEY_LEFT },
> > > + { 0x9f0e, KEY_RIGHT },
> > > + { 0x9f02, KEY_OK },
> > > +
> > > + { 0x9f47, KEY_HOME },
> > > + { 0x9f4f, KEY_BACK },
> > > + { 0x9f16, KEY_MENU },
> > > +
> > > + { 0x9fff, KEY_VOLUMEUP },
> > > + { 0x9f5d, KEY_VOLUMEDOWN },
> > > +};
> > > +
> > > +static struct rc_map_list magicbox_map = {
> > > + .map = {
> > > + .scan = magicbox,
> > > + .size = ARRAY_SIZE(magicbox),
> > > + .rc_proto = RC_PROTO_NEC,
> > > + .name = RC_MAP_MAGICBOX,
> > > + }
> > > +};
> > > +
> > > +static int __init init_rc_map_magicbox(void)
> > > +{
> > > + return rc_map_register(&magicbox_map);
> > > +}
> > > +
> > > +static void __exit exit_rc_map_magicbox(void)
> > > +{
> > > + rc_map_unregister(&magicbox_map);
> > > +}
> > > +
> > > +module_init(init_rc_map_magicbox)
> > > +module_exit(exit_rc_map_magicbox)
> > > +
> > > +MODULE_LICENSE("GPL");
> > > +MODULE_AUTHOR("Zhang Ning <zhangn1985@qq.com>");
> > > diff --git a/include/media/rc-map.h b/include/media/rc-map.h
> > > index 793b54342dff..656217b8e91b 100644
> > > --- a/include/media/rc-map.h
> > > +++ b/include/media/rc-map.h
> > > @@ -277,6 +277,7 @@ struct rc_map *rc_map_get(const char *name);
> > > #define RC_MAP_KWORLD_PLUS_TV_ANALOG "rc-kworld-plus-tv-analog"
> > > #define RC_MAP_LEADTEK_Y04G0051 "rc-leadtek-y04g0051"
> > > #define RC_MAP_LME2510 "rc-lme2510"
> > > +#define RC_MAP_MAGICBOX "rc-magicbox"
> > > #define RC_MAP_MANLI "rc-manli"
> > > #define RC_MAP_MECOOL_KII_PRO "rc-mecool-kii-pro"
> > > #define RC_MAP_MECOOL_KIII_PRO "rc-mecool-kiii-pro"
> > > --
> > > 2.35.1
> > >
_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2022-07-26 14:21 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20220726131523.140581-1-zhangn1985@qq.com>
2022-07-26 13:15 ` [v6 1/4] dt-bindings: media: rc: add keymap bindings for MagicBox M16S remote Zhang Ning
2022-07-26 13:15 ` [v6 2/4] media: rc: add keymap " Zhang Ning
2022-07-26 13:39 ` Zhang Ning
2022-07-26 13:59 ` Sean Young
2022-07-26 13:59 ` Sean Young
2022-07-26 14:19 ` Zhang Ning
2022-07-26 13:15 ` [v6 3/4] dt-bindings: arm: amlogic: add MagicBox M16S bindings Zhang Ning
2022-07-26 13:15 ` [v6 4/4] arm64: dts: meson: Add MagicBox M16S support Zhang Ning
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.