All of lore.kernel.org
 help / color / mirror / Atom feed
From: Kevin Hilman <khilman@baylibre.com>
To: Mohammad Rasim <mohammad.rasim96@gmail.com>,
	linux-media@vger.kernel.org, Sean Young <sean@mess.org>,
	Mauro Carvalho Chehab <mchehab@kernel.org>,
	linux-amlogic@lists.infradead.org
Cc: Mohammad Rasim <mohammad.rasim96@gmail.com>
Subject: Re: [PATCH v5 0/3] add videostrong kii pro keymap
Date: Sat, 29 Feb 2020 17:40:35 +0100	[thread overview]
Message-ID: <7h36atbb3w.fsf@baylibre.com> (raw)
In-Reply-To: <20200214085802.28742-1-mohammad.rasim96@gmail.com>

Hi Mohammad,

Mohammad Rasim <mohammad.rasim96@gmail.com> writes:

> Videostrong kii pro comes with a NEC remote control, this adds support
> for this remote and also select it from the device dts

With the bindings reviewed/ack'd I'll queue the DT patch (PATCH 3/3) via
the amlogic tree, but the binding and driver patches need to go via the
media subsystem.

Thanks for the patches,

Kevin

> Changes since v1:
> - fix styling issues
>
> Changes since v2:
> - use KEY_VENDOR for mouse key
> - use KEY_PVR instead of KEY_RECORD
> - use KEY_APPSELECT for the launcher
> - use KEY_TV for the tv app key
>
> Changes since v3:
> - add a patch to use the keymap by the device's dts
>
> Changes since v4:
> - add a patch to update the rc bindings
>
> Mohammad Rasim (3):
>   media: rc: add keymap for Videostrong KII Pro
>   dt-bindings: media: add new kii pro key map
>   arm64: dts: amlogic: add rc-videostrong-kii-pro keymap
>
>  .../devicetree/bindings/media/rc.yaml         |  1 +
>  .../boot/dts/amlogic/meson-gxbb-kii-pro.dts   |  4 +
>  drivers/media/rc/keymaps/Makefile             |  1 +
>  .../media/rc/keymaps/rc-videostrong-kii-pro.c | 83 +++++++++++++++++++
>  include/media/rc-map.h                        |  1 +
>  5 files changed, 90 insertions(+)
>  create mode 100644 drivers/media/rc/keymaps/rc-videostrong-kii-pro.c
>
> --
> 2.25.0

_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic

WARNING: multiple messages have this Message-ID (diff)
From: Kevin Hilman <khilman@baylibre.com>
To: Mohammad Rasim <mohammad.rasim96@gmail.com>,
	linux-media@vger.kernel.org, Sean Young <sean@mess.org>,
	Mauro Carvalho Chehab <mchehab@kernel.org>,
	linux-amlogic@lists.infradead.org
Cc: Mohammad Rasim <mohammad.rasim96@gmail.com>
Subject: Re: [PATCH v5 0/3] add videostrong kii pro keymap
Date: Sat, 29 Feb 2020 17:40:35 +0100	[thread overview]
Message-ID: <7h36atbb3w.fsf@baylibre.com> (raw)
In-Reply-To: <20200214085802.28742-1-mohammad.rasim96@gmail.com>

Hi Mohammad,

Mohammad Rasim <mohammad.rasim96@gmail.com> writes:

> Videostrong kii pro comes with a NEC remote control, this adds support
> for this remote and also select it from the device dts

With the bindings reviewed/ack'd I'll queue the DT patch (PATCH 3/3) via
the amlogic tree, but the binding and driver patches need to go via the
media subsystem.

Thanks for the patches,

Kevin

> Changes since v1:
> - fix styling issues
>
> Changes since v2:
> - use KEY_VENDOR for mouse key
> - use KEY_PVR instead of KEY_RECORD
> - use KEY_APPSELECT for the launcher
> - use KEY_TV for the tv app key
>
> Changes since v3:
> - add a patch to use the keymap by the device's dts
>
> Changes since v4:
> - add a patch to update the rc bindings
>
> Mohammad Rasim (3):
>   media: rc: add keymap for Videostrong KII Pro
>   dt-bindings: media: add new kii pro key map
>   arm64: dts: amlogic: add rc-videostrong-kii-pro keymap
>
>  .../devicetree/bindings/media/rc.yaml         |  1 +
>  .../boot/dts/amlogic/meson-gxbb-kii-pro.dts   |  4 +
>  drivers/media/rc/keymaps/Makefile             |  1 +
>  .../media/rc/keymaps/rc-videostrong-kii-pro.c | 83 +++++++++++++++++++
>  include/media/rc-map.h                        |  1 +
>  5 files changed, 90 insertions(+)
>  create mode 100644 drivers/media/rc/keymaps/rc-videostrong-kii-pro.c
>
> --
> 2.25.0

  parent reply	other threads:[~2020-02-29 16:40 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-02-14  8:57 [PATCH v5 0/3] add videostrong kii pro keymap Mohammad Rasim
2020-02-14  8:57 ` Mohammad Rasim
2020-02-14  8:58 ` [PATCH v5 1/3] media: rc: add keymap for Videostrong KII Pro Mohammad Rasim
2020-02-14  8:58   ` Mohammad Rasim
2020-02-19 14:11   ` Neil Armstrong
2020-02-19 14:11     ` Neil Armstrong
2020-02-14  8:58 ` [PATCH v5 2/3] dt-bindings: media: add new kii pro key map Mohammad Rasim
2020-02-14  8:58   ` Mohammad Rasim
2020-02-18 21:04   ` Rob Herring
2020-02-18 21:04     ` Rob Herring
2020-02-14  8:58 ` [PATCH v5 3/3] arm64: dts: amlogic: add rc-videostrong-kii-pro keymap Mohammad Rasim
2020-02-14  8:58   ` Mohammad Rasim
2020-02-19 14:11   ` Neil Armstrong
2020-02-19 14:11     ` Neil Armstrong
2020-02-29 16:40 ` Kevin Hilman [this message]
2020-02-29 16:40   ` [PATCH v5 0/3] add videostrong kii pro keymap Kevin Hilman

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=7h36atbb3w.fsf@baylibre.com \
    --to=khilman@baylibre.com \
    --cc=linux-amlogic@lists.infradead.org \
    --cc=linux-media@vger.kernel.org \
    --cc=mchehab@kernel.org \
    --cc=mohammad.rasim96@gmail.com \
    --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 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.