From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753562AbdCPWEs (ORCPT ); Thu, 16 Mar 2017 18:04:48 -0400 Received: from mail-yw0-f193.google.com ([209.85.161.193]:33159 "EHLO mail-yw0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752242AbdCPWEq (ORCPT ); Thu, 16 Mar 2017 18:04:46 -0400 Date: Thu, 16 Mar 2017 17:32:55 -0400 From: Tejun Heo To: Bartlomiej Zolnierkiewicz Cc: Sekhar Nori , Sergei Shtylyov , Kevin Hilman , Arnd Bergmann , Russell King , Dmitry Eremin-Solenikov , linux-ide@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 0/3] ATA/ARM: convert ARM/DaVinci to use libata PATA drivers Message-ID: <20170316213255.GF1480@htj.duckdns.org> References: <1489509414-11491-1-git-send-email-b.zolnierkie@samsung.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1489509414-11491-1-git-send-email-b.zolnierkie@samsung.com> User-Agent: Mutt/1.7.1 (2016-10-04) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Mar 14, 2017 at 05:36:51PM +0100, Bartlomiej Zolnierkiewicz wrote: > Hi, > > This patchset adds Palmchip BK3710 IDE controller driver to > libata and switches ARM/DaVinci to use it (instead of the old > IDE driver). > > Sekhar, please check that it still works after changes, thanks. > > Changes since v1 > (https://www.spinics.net/lists/arm-kernel/msg567442.html): > - addressed review comments from Sergei Shtylyov > - fixed cycle_time unitialized variable issue > > Changes since v0.1 draft patch version > (https://www.spinics.net/lists/arm-kernel/msg566932.html): > - fixed cycle_time build warning > - added platform support fixes from Sekhar > - added defconfig changes from Sekhar > - preserved platform support for the old IDE driver > - split it on 3 patches Looks fine to me from libata side. Once it gets tested, how should the patches be routed? I don't think it'd make sense to route them separately. Thanks. -- tejun