Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755615Ab2JIN3U (ORCPT ); Tue, 9 Oct 2012 09:29:20 -0400 Received: from acsinet15.oracle.com ([141.146.126.227]:31373 "EHLO acsinet15.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752264Ab2JIN3R (ORCPT ); Tue, 9 Oct 2012 09:29:17 -0400 Date: Tue, 9 Oct 2012 16:28:47 +0300 From: Dan Carpenter To: Ohad Ben-Cohen Cc: Sjur BRENDELAND , Fengguang Wu , "Linus Walleij (linus.walleij@linaro.org)" , "kernel-janitors@vger.kernel.org" , "linux-kernel@vger.kernel.org" Subject: Re: [remoteproc:for-next 6/9] remoteproc_virtio.c:(.text+0x238e7e): undefined reference to `vring_transport_features' Message-ID: <20121009132847.GC4587@mwanda> References: <20121001024544.GA12711@localhost> <81C3A93C17462B4BBD7E272753C1057923BD9B59E3@EXDCVYMBSTM005.EQ1STM.local> 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) X-Source-IP: ucsinet21.oracle.com [156.151.31.93] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1354 Lines: 34 On Tue, Oct 09, 2012 at 01:52:49PM +0200, Ohad Ben-Cohen wrote: > Hi Sjur, > > On Tue, Oct 9, 2012 at 12:30 PM, Sjur BRENDELAND > wrote: > > Sorry for not responding sooner, but I thought this issue was solved with > > your patch "remoteproc: fix (again) the virtio-related build breakage" > > (https://lkml.org/lkml/2012/10/6/85). > > > > I'm not sure I understand why you would want to add a dependency to ARM. > > But if you're uncomfortable by having STE_MODEM_RPROC directly selectable, > > perhaps we could let it be selected by arch specific Kconfig files, e.g. mach-ux500? > > I would just like the Kconfig dependencies to reflect the "real world": > > E.g., if there's no chance the STE modem is going to be used on x86, > then let's not ask x86 folks about it. > > Does limiting the STE modem to certain platform/architectures make > sense ? (if not, that's ok) > Unless there is a good reason why then we shouldn't put arbitrary limits like that. If we leave it in people at least run static analyzers on it and try modprobing it. regards, dan carpenter -- 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/