From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752614AbbIPJqv (ORCPT ); Wed, 16 Sep 2015 05:46:51 -0400 Received: from smtp08.smtpout.orange.fr ([80.12.242.130]:57997 "EHLO smtp.smtpout.orange.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752087AbbIPJqt (ORCPT ); Wed, 16 Sep 2015 05:46:49 -0400 X-ME-Helo: belgarion X-ME-Auth: amFyem1pay5yb2JlcnRAb3JhbmdlLmZy X-ME-Date: Wed, 16 Sep 2015 11:46:48 +0200 X-ME-IP: 90.16.209.104 From: Robert Jarzmik To: David Miller , Russell King - ARM Linux Cc: nico@fluxnic.net, arnd@arndb.de, netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] net: smc91x: convert pxa dma to dmaengine References: <1441913164-3785-1-git-send-email-robert.jarzmik@free.fr> <20150915.162414.1153837609419205002.davem@davemloft.net> X-URL: http://belgarath.falguerolles.org/ Date: Wed, 16 Sep 2015 11:41:54 +0200 In-Reply-To: <20150915.162414.1153837609419205002.davem@davemloft.net> (David Miller's message of "Tue, 15 Sep 2015 16:24:14 -0700 (PDT)") Message-ID: <87a8smsntp.fsf@belgarion.home> User-Agent: Gnus/5.130008 (Ma Gnus v0.8) Emacs/24.4 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org David Miller writes: > From: Robert Jarzmik > Date: Thu, 10 Sep 2015 21:26:04 +0200 > >> Convert the dma transfers to be dmaengine based, now pxa has a dmaengine >> slave driver. This makes this driver a bit more PXA agnostic. >> >> The driver was tested on pxa27x (mainstone) and pxa310 (zylonite), >> ie. only pxa platforms. >> >> Signed-off-by: Robert Jarzmik >> Cc: Russell King >> Cc: Arnd Bergmann >> --- >> This has potential to break other platform such as Neponset, Idp, >> halibut and qsd8x50, so I added Russell and Arnd as they were discussing >> smc91x support last February. > > Is someone testing whether such platforms break or not? I'm waiting for > that before I consider applying this patch. My understanding is that Russell is the only one left testing them, or at least he was the only one complaining about a breakage lately on neponset. I can wait several weeks for Russell to have a bit of time to try : I know it will compile correctly at least for neponset, and I know almost all the code is under #ifdef CONFIG_ARCH_PXA. And still I would feel far more comfortable if it was tested, just as you. Cheers. -- Robert