From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932965Ab0KLWyS (ORCPT ); Fri, 12 Nov 2010 17:54:18 -0500 Received: from mail-bw0-f46.google.com ([209.85.214.46]:64891 "EHLO mail-bw0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932147Ab0KLWyP convert rfc822-to-8bit (ORCPT ); Fri, 12 Nov 2010 17:54:15 -0500 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type:content-transfer-encoding; b=pkk338G4X2iLVi3j46WMdAlri6DTlmrbz0Y7at7QVMlxzZG7kQzra4IrpR5v9jR0Ks 4+27ozvIgfPUznH3i2m5W7r19/8Bc61B4PBbwbjPTMoa2SqfUkv1pcPj/758iPoYCB++ beJpSab4mahrgW0JylPo0ovPiZ6Uc/fjwIe4s= MIME-Version: 1.0 In-Reply-To: <1289147348-31969-3-git-send-email-alchark@gmail.com> References: <1289147348-31969-1-git-send-email-alchark@gmail.com> <1289147348-31969-3-git-send-email-alchark@gmail.com> Date: Sat, 13 Nov 2010 01:54:14 +0300 Message-ID: Subject: Re: [PATCH 3/6 v2] input: Add support for VIA VT8500 and compatibles in i8042 From: Alexey Charkov To: linux-arm-kernel@lists.infradead.org Cc: vt8500-wm8505-linux-kernel@googlegroups.com, Alexey Charkov , Dmitry Torokhov , Tony Lindgren , Feng Tang , Janusz Krzysztofik , Dmitry Eremin-Solenikov , linux-input@vger.kernel.org, linux-kernel@vger.kernel.org Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 2010/11/7 Alexey Charkov : > VIA and WonderMedia Systems-on-Chip feature a standard i8042-compatible > keyboard and mouse controller. This adds necessary glue to enable use > of the standard driver with these systems. > > Signed-off-by: Alexey Charkov > --- > > Please review and (if appropriate) commit to a relevant git tree for > further integration in 2.6.38. > > Previous version of this code was 'Acked-by: Dmitry Torokhov ' > This one only differs by using runtime-selected IRQ definitions instead > of static compile-time preprocessor macros. > > Relevant register and interrupt definitions are provided by PATCH 1/6 in > this series, so one would need that to make use of this code. > >  drivers/input/serio/Kconfig        |    3 +- >  drivers/input/serio/i8042-vt8500.h |   74 ++++++++++++++++++++++++++++++++++++ >  drivers/input/serio/i8042.h        |    2 + >  3 files changed, 78 insertions(+), 1 deletions(-) >  create mode 100644 drivers/input/serio/i8042-vt8500.h > > diff --git a/drivers/input/serio/Kconfig b/drivers/input/serio/Kconfig > index 6256233..ff799f3 100644 > --- a/drivers/input/serio/Kconfig > +++ b/drivers/input/serio/Kconfig > @@ -21,7 +21,8 @@ if SERIO >  config SERIO_I8042 >        tristate "i8042 PC Keyboard controller" if EMBEDDED || !X86 >        default y > -       depends on !PARISC && (!ARM || ARCH_SHARK || FOOTBRIDGE_HOST) && \ > +       depends on !PARISC && \ > +                 (!ARM || ARCH_SHARK || ARCH_VT8500 || FOOTBRIDGE_HOST) && \ >                   (!SUPERH || SH_CAYMAN) && !M68K && !BLACKFIN >        help >          i8042 is the chip over which the standard AT keyboard and PS/2 > diff --git a/drivers/input/serio/i8042-vt8500.h b/drivers/input/serio/i8042-vt8500.h > new file mode 100644 > index 0000000..4ff9e1c > --- /dev/null > +++ b/drivers/input/serio/i8042-vt8500.h > @@ -0,0 +1,74 @@ > +#ifndef _I8042_VT8500_H > +#define _I8042_VT8500_H > + > +#include > +#include > + > +/* > + * This program is free software; you can redistribute it and/or modify it > + * under the terms of the GNU General Public License version 2 as published by > + * the Free Software Foundation. > + */ > + > +static void __iomem *regbase; > + > +/* > + * Names. > + */ > + > +#define I8042_KBD_PHYS_DESC "vt8500ps2/serio0" > +#define I8042_AUX_PHYS_DESC "vt8500ps2/serio1" > +#define I8042_MUX_PHYS_DESC "vt8500ps2/serio%d" > + > +/* > + * IRQs. > + */ > + > +#define I8042_KBD_IRQ  (wmt_current_irqs->ps2kbd) > +#define I8042_AUX_IRQ  (wmt_current_irqs->ps2mouse) > + > + > +/* > + * Register numbers. > + */ > + > +#define I8042_COMMAND_REG      (regbase + 0x4) > +#define I8042_STATUS_REG       (regbase + 0x4) > +#define I8042_DATA_REG         (regbase + 0x0) > + > +static inline int i8042_read_data(void) > +{ > +       return readl(I8042_DATA_REG); > +} > + > +static inline int i8042_read_status(void) > +{ > +       return readl(I8042_STATUS_REG); > +} > + > +static inline void i8042_write_data(int val) > +{ > +       writel(val, I8042_DATA_REG); > +} > + > +static inline void i8042_write_command(int val) > +{ > +       writel(val, I8042_COMMAND_REG); > +} > + > +static inline int i8042_platform_init(void) > +{ > +       i8042_reset = true; > +       regbase = ioremap(wmt_current_regs->ps2, SZ_1K); > +       if (!regbase) > +               return -ENODEV; > + > +       return 0; > +} > + > +static inline void i8042_platform_exit(void) > +{ > +       iounmap(regbase); > +} > + > +#endif /* _I8042_VT8500_H */ > diff --git a/drivers/input/serio/i8042.h b/drivers/input/serio/i8042.h > index cbc1beb..bdb2aeb 100644 > --- a/drivers/input/serio/i8042.h > +++ b/drivers/input/serio/i8042.h > @@ -16,6 +16,8 @@ > >  #if defined(CONFIG_MACH_JAZZ) >  #include "i8042-jazzio.h" > +#elif defined(CONFIG_ARCH_VT8500) > +#include "i8042-vt8500.h" >  #elif defined(CONFIG_SGI_HAS_I8042) >  #include "i8042-ip22io.h" >  #elif defined(CONFIG_SNI_RM) > -- > 1.7.3.2 > > Any comments about this? Thanks, Alexey