Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753064Ab3IVXNI (ORCPT ); Sun, 22 Sep 2013 19:13:08 -0400 Received: from merlin.infradead.org ([205.233.59.134]:38130 "EHLO merlin.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752716Ab3IVXNF (ORCPT ); Sun, 22 Sep 2013 19:13:05 -0400 Date: Sun, 22 Sep 2013 17:12:59 -0600 From: Jens Axboe To: Olof Johansson Cc: Andrew Morton , Jeff Moyer , OS Engineering , "linux-kernel@vger.kernel.org" , Akhil Bhansali , Ramprasad Chinthekindi , Amit Phansalkar Subject: Re: [PATCH] block: Device driver for sTec's PCIe Kronos Card. Message-ID: <20130922231259.GG31014@kernel.dk> References: <26D762E250385C4D8E9D6EC3C8E47DC11E6FCF20@sambx4.stec-inc.ad> <26D762E250385C4D8E9D6EC3C8E47DC11E6FE729@sambx4.stec-inc.ad> <5228913A.50807@kernel.dk> <5238B9A7.6040002@kernel.dk> <20130920153128.63244ec2658b7ad3aee0e2e4@linux-foundation.org> <20130921200132.GF2312@kernel.dk> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1663 Lines: 38 On Sun, Sep 22 2013, Olof Johansson wrote: > On Sat, Sep 21, 2013 at 1:01 PM, Jens Axboe wrote: > > On Fri, Sep 20 2013, Andrew Morton wrote: > >> On Tue, 17 Sep 2013 14:20:55 -0600 Jens Axboe wrote: > >> > >> > > So, it looks like this driver needs a bunch of work before it's ready > >> > > to go in. Or, maybe it's better to submit it with a TODO list for the > >> > > staging tree instead? > >> > > >> > Not disagreeing with you, it definitely needs a bit of cleaning. And so > >> > far stec has not been very responsive in fixing those issues. > >> > >> Geeze. Here's what I came up with, mainly to make i386 compile: > > > > Thanks Andrew. It's not the fixes themselves that are tricky, I would > > just have liked a little more proof that the submission wasn't just a > > dump'n run. > > Well, that's what much of the staging process is about, since it needs > to come with a todo list (and that list needs to be taken care of > before the driver graduates). Honestly, this looks like a prime > candidate for that. But it's your tree and your call. > > Thanks for applying the patch from Andrew, that resolves most of it > for me right now. I'm sort-of on the fence. It's minor stuff, but it does need cleaning up. So not necessarily something that will prevent it from going into 3.12, it's more the continued support and development I'm worried about. -- Jens Axboe -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/