From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1422664AbcFMHTz (ORCPT ); Mon, 13 Jun 2016 03:19:55 -0400 Received: from bastet.se.axis.com ([195.60.68.11]:42058 "EHLO bastet.se.axis.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1030185AbcFMHTx (ORCPT ); Mon, 13 Jun 2016 03:19:53 -0400 Date: Mon, 13 Jun 2016 09:19:44 +0200 From: Ricard Wanderlof X-X-Sender: ricardw@lnxricardw1.se.axis.com To: Boris Brezillon CC: Mychaela Falconia , , Tony Lindgren , , Oleksij Rempel , Linux mtd , Benoit Cousson , Brian Norris , David Woodhouse Subject: Re: [PATCH 3/4] mtd: nand: Add support for Evatronix NANDFLASH-CTRL In-Reply-To: <20160610180713.6f09f802@bbrezillon> Message-ID: References: <20160603165909.09f27ee0@bbrezillon> <20160609200128.5b7a6e29@bbrezillon> <20160609222307.5db4e67f@bbrezillon> <20160610180713.6f09f802@bbrezillon> User-Agent: Alpine 2.02 (DEB 1266 2009-07-14) MIME-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-15" Content-Transfer-Encoding: 8BIT X-Originating-IP: [10.0.5.107] X-ClientProxiedBy: XBOX02.axis.com (10.0.5.16) To XBOX03.axis.com (10.0.5.17) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 10 Jun 2016, Boris Brezillon wrote: > > Something that I am mildly miffed about is that I've posted this driver > > twice before on the mtd list (although, admittedly, not directly addressed > > to any maintainer), first as an RFC and later as a complete patch, without > > a single response. (Apparently Boris you did respond with comments on > > Oleksij's driver though which I must have missed). Although an RFC might > > not have initiated a detailed review, it would have been a large advantage > > (and wasted less time for all) if the point of using 'wrong' driver > > interface had been brought up and consequently discussed earlier. Yes I > > know, everyone is busy, it's easy to miss things, each and every driver > > can't be reviewed in detail, etc. > > Yes, I know. I was not maintainer at that time, and was only reviewing > a few patches from time to time. > Now that I am, I try to answer as fast as possible. Note that the set > of requirements might have change with me, so even if someone add > agreed on you first submission but not taken the patches, I would have > required the same set of changes ;). :-) Well, I'm not questioning that changes are required, and I'm happy to implement them once I've come to a clear understanding of what is needed, I just would have liked to known about them earlier, especially the major concepts. I know there was probably no one interested enough at the time, everyone is busy, not every idea presented can be reviewed in detail etc etc. And it's not a big issue, just a minor annoyance. /Ricard -- Ricard Wolf Wanderlöf ricardw(at)axis.com Axis Communications AB, Lund, Sweden www.axis.com Phone +46 46 272 2016 Fax +46 46 13 61 30 From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ricard Wanderlof Subject: Re: [PATCH 3/4] mtd: nand: Add support for Evatronix NANDFLASH-CTRL Date: Mon, 13 Jun 2016 09:19:44 +0200 Message-ID: References: <20160603165909.09f27ee0@bbrezillon> <20160609200128.5b7a6e29@bbrezillon> <20160609222307.5db4e67f@bbrezillon> <20160610180713.6f09f802@bbrezillon> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: In-Reply-To: <20160610180713.6f09f802@bbrezillon> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Boris Brezillon Cc: Mychaela Falconia , devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Tony Lindgren , linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Oleksij Rempel , Linux mtd , Benoit Cousson , Brian Norris , David Woodhouse List-Id: devicetree@vger.kernel.org On Fri, 10 Jun 2016, Boris Brezillon wrote: > > Something that I am mildly miffed about is that I've posted this dr= iver=20 > > twice before on the mtd list (although, admittedly, not directly ad= dressed=20 > > to any maintainer), first as an RFC and later as a complete patch, = without=20 > > a single response. (Apparently Boris you did respond with comments = on=20 > > Oleksij's driver though which I must have missed). Although an RFC = might=20 > > not have initiated a detailed review, it would have been a large ad= vantage=20 > > (and wasted less time for all) if the point of using 'wrong' driver= =20 > > interface had been brought up and consequently discussed earlier. Y= es I=20 > > know, everyone is busy, it's easy to miss things, each and every dr= iver=20 > > can't be reviewed in detail, etc. >=20 > Yes, I know. I was not maintainer at that time, and was only reviewin= g > a few patches from time to time. > Now that I am, I try to answer as fast as possible. Note that the set > of requirements might have change with me, so even if someone add > agreed on you first submission but not taken the patches, I would hav= e > required the same set of changes ;). :-) Well, I'm not questioning that changes are required, and I'm happy to=20 implement them once I've come to a clear understanding of what is neede= d,=20 I just would have liked to known about them earlier, especially the maj= or=20 concepts. I know there was probably no one interested enough at the time, everyon= e=20 is busy, not every idea presented can be reviewed in detail etc etc. An= d=20 it's not a big issue, just a minor annoyance. /Ricard --=20 Ricard Wolf Wanderl=F6f ricardw(at)axis.com Axis Communications AB, Lund, Sweden www.axis.com Phone +46 46 272 2016 Fax +46 46 13 61 30 -- To unsubscribe from this list: send the line "unsubscribe devicetree" i= n the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html