Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Fri, 19 Jan 2018 09:32:12 -0800
From:   Dmitry Torokhov <dmitry.torokhov@gmail.com>
To:     Hans-Christian Noren Egtvedt <egtvedt@samfundet.no>
Cc:     Corentin Labbe <clabbe.montjoie@gmail.com>,
        alexandre.belloni@free-electrons.com, nicolas.ferre@microchip.com,
        linux-arm-kernel@lists.infradead.org, linux-input@vger.kernel.org,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH] input: touchscreen: remove atmel-wm97xx
Message-ID: <20180119173212.jpztigqxjti5taq6@dtor-ws>
References: <20180119081745.17557-1-clabbe.montjoie@gmail.com>
 <20180119105058.bjh2gatrmga5yrni@samfundet.no>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20180119105058.bjh2gatrmga5yrni@samfundet.no>
User-Agent: NeoMutt/20170609 (1.8.3)
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Fri, Jan 19, 2018 at 11:50:58AM +0100, Hans-Christian Noren Egtvedt wrote:
> Around Fri 19 Jan 2018 09:17:45 +0100 or thereabout, Corentin Labbe wrote:
> > Since AVR32 arch is gone, atmel-wm97xx driver is useless.
> > Remove it.
> 
> In theory it could have been rewritten to work with AT91 devices, but I would
> recommend starting from scratch. This driver is from the platform data era,
> and IIRC a bit hard coded to work on AVR32 hardware variant of the AC97C
> peripheral.
> 
> > Signed-off-by: Corentin Labbe <clabbe.montjoie@gmail.com>
> >
> 
> Acked-by: Hans-Christian Noren Egtvedt <egtvedt@samfundet.no>

Applied, thank you.

> 
> > ---
> >  drivers/input/touchscreen/Kconfig        |  15 --
> >  drivers/input/touchscreen/Makefile       |   1 -
> >  drivers/input/touchscreen/atmel-wm97xx.c | 436 -------------------------------
> >  3 files changed, 452 deletions(-)
> >  delete mode 100644 drivers/input/touchscreen/atmel-wm97xx.c
> > 
> > diff --git a/drivers/input/touchscreen/Kconfig b/drivers/input/touchscreen/Kconfig
> > index 38a226f9fcbd..4f15496fec8b 100644
> > --- a/drivers/input/touchscreen/Kconfig
> > +++ b/drivers/input/touchscreen/Kconfig
> > @@ -795,21 +795,6 @@ config TOUCHSCREEN_WM9713
> >  	  Say Y here to enable support for the Wolfson Microelectronics
> >  	  WM9713 touchscreen controller.
> >  
> > -config TOUCHSCREEN_WM97XX_ATMEL
> > -	tristate "WM97xx Atmel accelerated touch"
> > -	depends on TOUCHSCREEN_WM97XX && AVR32
> > -	help
> > -	  Say Y here for support for streaming mode with WM97xx touchscreens
> > -	  on Atmel AT91 or AVR32 systems with an AC97C module.
> > -
> > -	  Be aware that this will use channel B in the controller for
> > -	  streaming data, this must not conflict with other AC97C drivers.
> > -
> > -	  If unsure, say N.
> > -
> > -	  To compile this driver as a module, choose M here: the module will
> > -	  be called atmel-wm97xx.
> > -
> >  config TOUCHSCREEN_WM97XX_MAINSTONE
> >  	tristate "WM97xx Mainstone/Palm accelerated touch"
> >  	depends on TOUCHSCREEN_WM97XX && ARCH_PXA
> > diff --git a/drivers/input/touchscreen/Makefile b/drivers/input/touchscreen/Makefile
> > index d2a2b3b7af27..dddae7973436 100644
> > --- a/drivers/input/touchscreen/Makefile
> > +++ b/drivers/input/touchscreen/Makefile
> > @@ -97,7 +97,6 @@ obj-$(CONFIG_TOUCHSCREEN_WM97XX)	+= wm97xx-ts.o
> >  wm97xx-ts-$(CONFIG_TOUCHSCREEN_WM9705)	+= wm9705.o
> >  wm97xx-ts-$(CONFIG_TOUCHSCREEN_WM9712)	+= wm9712.o
> >  wm97xx-ts-$(CONFIG_TOUCHSCREEN_WM9713)	+= wm9713.o
> > -obj-$(CONFIG_TOUCHSCREEN_WM97XX_ATMEL)	+= atmel-wm97xx.o
> >  obj-$(CONFIG_TOUCHSCREEN_WM97XX_MAINSTONE)	+= mainstone-wm97xx.o
> >  obj-$(CONFIG_TOUCHSCREEN_WM97XX_ZYLONITE)	+= zylonite-wm97xx.o
> >  obj-$(CONFIG_TOUCHSCREEN_W90X900)	+= w90p910_ts.o
> 
> <snipp diff>
> 
> -- 
> mvh
> Hans-Christian Noren Egtvedt

-- 
Dmitry