From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757583Ab2JXFmE (ORCPT ); Wed, 24 Oct 2012 01:42:04 -0400 Received: from mail-pa0-f46.google.com ([209.85.220.46]:59920 "EHLO mail-pa0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750863Ab2JXFmC (ORCPT ); Wed, 24 Oct 2012 01:42:02 -0400 Date: Tue, 23 Oct 2012 22:42:36 -0700 From: Greg Kroah-Hartman To: Ming Lei Cc: linux-kernel@vger.kernel.org, Linus Torvalds Subject: Re: [PATCH 2/2] firmware loader: introduce kernel parameter to customize fw search path Message-ID: <20121024054236.GA14826@kroah.com> References: <1351046993-2220-1-git-send-email-ming.lei@canonical.com> <20121024031845.GA11255@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Oct 24, 2012 at 11:31:16AM +0800, Ming Lei wrote: > On Wed, Oct 24, 2012 at 11:18 AM, Greg Kroah-Hartman > wrote: > > > > Ick, no, not another kernel boot parameter. I would prefer the /proc/ > > file solution instead, just like /proc/sys/kernel/hotplug is. > > OK, got it, but we could use the module parameter(firmware_class.path > via kernel command) to customize the search path too, couldn't we? Yes, but why? Let's not overdesign this, get the basics working and then, if someone really needs it, we can add it. thanks, greg k-h