From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail.kernel.org ([198.145.29.99]:42166 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752459AbdHINtp (ORCPT ); Wed, 9 Aug 2017 09:49:45 -0400 Date: Wed, 9 Aug 2017 21:48:18 +0800 From: Shawn Guo To: Sean Young 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 Subject: Re: [PATCH v2 3/3] rc: add zx-irdec remote control driver Message-ID: <20170809134816.GA31819@dragon> References: <1501420993-21977-1-git-send-email-shawnguo@kernel.org> <1501420993-21977-4-git-send-email-shawnguo@kernel.org> <20170809130029.eneyqstlejfkotz2@gofer.mess.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170809130029.eneyqstlejfkotz2@gofer.mess.org> Sender: linux-media-owner@vger.kernel.org List-ID: Hi Sean, 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? Shawn