Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757152Ab2JWN3n (ORCPT ); Tue, 23 Oct 2012 09:29:43 -0400 Received: from opensource.wolfsonmicro.com ([80.75.67.52]:42306 "EHLO opensource.wolfsonmicro.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754639Ab2JWN3m (ORCPT ); Tue, 23 Oct 2012 09:29:42 -0400 Date: Tue, 23 Oct 2012 14:29:39 +0100 From: Dimitris Papastamos To: Ming Lei Cc: linux-kernel@vger.kernel.org, Greg Kroah-Hartman Subject: Re: [PATCH 1/3 v2] firmware: Convert firmware path setup from an array to a list Message-ID: <20121023132939.GA10160@opensource.wolfsonmicro.com> References: <1350996776-17864-1-git-send-email-dp@opensource.wolfsonmicro.com> <1350996776-17864-2-git-send-email-dp@opensource.wolfsonmicro.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.17+20080114 (2008-01-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1416 Lines: 38 On Tue, Oct 23, 2012 at 09:08:50PM +0800, Ming Lei wrote: > On Tue, Oct 23, 2012 at 8:52 PM, Dimitris Papastamos > wrote: > > In preparation to support dynamic listing/updating of firmware > > paths via procfs, this patch converts the firmware path configuration > > from an array to a list. > > I remembered that I have questioned on dynamic listing/updating of > firmware via procfs, looks you don't reply, :-( > > http://marc.info/?l=linux-kernel&m=134988208617179&w=2 > > IMO, it is not a good idea to update firmware path dynamically via proc > because it may be too late to update a dynamic search path via /proc > and drivers need request firmware just after rootfs is mounted. > > kernel parameter should be a good way to pass one customerized > path. > > Thanks, > -- > Ming Lei Hi sorry for being unclear, I mentioned in the old thread that I've not had time to fix this properly (or think about a proper fix), Greg requested a rebase on top of linux-next, however, at least for firmware path listing. Should I amend the commit description so it is not misleading and remove the 'updating' part? Thanks, Dimitris -- 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/