From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sean Young Subject: Re: [PATCH v2 3/3] rc: add zx-irdec remote control driver Date: Wed, 9 Aug 2017 15:42:44 +0100 Message-ID: <20170809144244.6l6k2hllp7fc34pr@gofer.mess.org> References: <1501420993-21977-1-git-send-email-shawnguo@kernel.org> <1501420993-21977-4-git-send-email-shawnguo@kernel.org> <20170809130029.eneyqstlejfkotz2@gofer.mess.org> <20170809134816.GA31819@dragon> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20170809134816.GA31819@dragon> Sender: linux-media-owner@vger.kernel.org To: Shawn Guo Cc: Rob Herring , Baoyou Xie , Xin Zhou , Jun Nie , linux-media@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Shawn Guo List-Id: devicetree@vger.kernel.org Hi Shawn, On Wed, Aug 09, 2017 at 09:48:18PM +0800, Shawn Guo wrote: > On Wed, Aug 09, 2017 at 02:00:29PM +0100, Sean Young wrote: > > On Sun, Jul 30, 2017 at 09:23:13PM +0800, Shawn Guo wrote: > > > From: Shawn Guo > > > > > > It adds the remote control driver and corresponding keymap file for > > > IRDEC block found on ZTE ZX family SoCs. > > > > > > Signed-off-by: Shawn Guo > > > --- > > > drivers/media/rc/Kconfig | 11 ++ > > > drivers/media/rc/Makefile | 1 + > > > drivers/media/rc/keymaps/Makefile | 3 +- > > > drivers/media/rc/keymaps/rc-zx-irdec.c | 79 ++++++++++++++ > > > drivers/media/rc/zx-irdec.c | 183 +++++++++++++++++++++++++++++++++ > > > include/media/rc-map.h | 1 + > > > 6 files changed, 277 insertions(+), 1 deletion(-) > > > create mode 100644 drivers/media/rc/keymaps/rc-zx-irdec.c > > > create mode 100644 drivers/media/rc/zx-irdec.c > > > > We're missing an MAINTAINERS entry for this driver. Could this be added > > please as a separate patch? > > We are using 'ARM/ZTE ARCHITECTURE' entry in MAINTAINERS for ZTE ZX > drivers. I plan to send a separate patch going through arm-soc tree, > updating MAINTAINERS for those new ZTE ZX drivers landed on mainline > during 4.14 merge window. IRDEC will be covered by that patch. Is that > okay? Of course. Thanks, Sean