From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-wm1-x344.google.com ([2a00:1450:4864:20::344]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1giPHi-0000K0-7a for barebox@lists.infradead.org; Sat, 12 Jan 2019 19:49:35 +0000 Received: by mail-wm1-x344.google.com with SMTP id m22so5358564wml.3 for ; Sat, 12 Jan 2019 11:49:32 -0800 (PST) MIME-Version: 1.0 References: <20190112055524.7733-1-andrew.smirnov@gmail.com> <20190112104822.GB14273@ravnborg.org> In-Reply-To: <20190112104822.GB14273@ravnborg.org> From: Andrey Smirnov Date: Sat, 12 Jan 2019 11:49:19 -0800 Message-ID: List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "barebox" Errors-To: barebox-bounces+u.kleine-koenig=pengutronix.de@lists.infradead.org Subject: Re: [PATCH v2 00/18] PCIe support for i.MX7 To: Sam Ravnborg Cc: Barebox List On Sat, Jan 12, 2019 at 2:48 AM Sam Ravnborg wrote: > > Hi Andrey. > > > Feedback is welcome! > > I have looked through the patches - and nothing jumped at me. > A few places could use a newline but not worth a re-spin. > > I personal dislike filenames like "helpers.c" but thats pure bikeshedding. > One of my goals was to keep things as close to how they are in Linux as possible. That's how that file is called there, so you might have to take it with Mark Brown if you want to change that. > Looks good with the simple regulator support, which most likely will prove > useful later also for other drivers. > I hope we do not need deferred probe much... > > With deferred probe ported from the kernel, I wonder > how far we are from porting the devm_* functionlity too. > Not the I think this will share code with deferred probe, > just that this is another nice kernel idiom we could > benefit from in barebox. > Just to clarify, deferred probe was already a part of Barebox for a while, the patches in this series only change the fact that Barebox core now has a chance to "give up" on deferred device and declare it as non-existent. As for devm_* functions, not far at all. I don't think it would be difficult to port and I've been meaning to do that for a while, just never had an opportunity to do it so far. Thanks, Andrey Smirnov _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox