Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754865AbbENDEn (ORCPT ); Wed, 13 May 2015 23:04:43 -0400 Received: from helcar.hengli.com.au ([209.40.204.226]:48844 "EHLO helcar.hengli.com.au" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753866AbbENDEm (ORCPT ); Wed, 13 May 2015 23:04:42 -0400 Date: Thu, 14 May 2015 11:04:15 +0800 From: Herbert Xu To: "Luis R. Rodriguez" Cc: ming.lei@canonical.com, rusty@rustcorp.com.au, torvalds@linux-foundation.org, dhowells@redhat.com, seth.forshee@canonical.com, linux-kernel@vger.kernel.org, pebolle@tiscali.nl, linux-wireless@vger.kernel.org, gregkh@linuxfoundation.org, jlee@suse.com, tiwai@suse.de, casey@schaufler-ca.com, keescook@chromium.org, mjg59@srcf.ucam.org, akpm@linux-foundation.org, "Luis R. Rodriguez" , Kyle McMartin , Bruce Allan , Tadeusz Struk , John Griffin Subject: Re: [RFC v2 3/6] crypto: qat - address recursive dependency when fw signing is enabled Message-ID: <20150514030415.GD3853@gondor.apana.org.au> References: <1431541436-17007-1-git-send-email-mcgrof@do-not-panic.com> <1431541436-17007-4-git-send-email-mcgrof@do-not-panic.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1431541436-17007-4-git-send-email-mcgrof@do-not-panic.com> 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 Content-Length: 1219 Lines: 28 On Wed, May 13, 2015 at 11:23:53AM -0700, Luis R. Rodriguez wrote: > From: "Luis R. Rodriguez" > > We're going to add firmware module signing support, but when we do > this we end up with the following recursive dependency. Fix this by > just depending on FW_LOADER, which is typically always enabled > anyway. We do this as a compromise for now as FW_LOADER is only > selectable when EXPERT is enabled and the recursive dependency issue > below is actually real and valid, its proper fix however requires > a bit of work. > > This technically means this change is likely welcomed for other > parts of the kernel but this is really only needed for now here. Nack. Either fix kbuild or make every user of FW_LOADER use a dependency. Singling out a single crypto device makes zero sense. Cheers, -- Email: Herbert Xu Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt -- 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/