From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757122Ab0KKWV3 (ORCPT ); Thu, 11 Nov 2010 17:21:29 -0500 Received: from rcsinet10.oracle.com ([148.87.113.121]:51932 "EHLO rcsinet10.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756167Ab0KKWV1 (ORCPT ); Thu, 11 Nov 2010 17:21:27 -0500 Message-ID: <4CDC6BA4.3060307@oracle.com> Date: Thu, 11 Nov 2010 14:18:12 -0800 From: Randy Dunlap Organization: Oracle Linux Engineering User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.5) Gecko/20091209 Fedora/3.0-3.fc11 Thunderbird/3.0 MIME-Version: 1.0 To: "R.M. Thomas" CC: Stephen Rothwell , linux-next@vger.kernel.org, LKML , gregkh@suse.de Subject: Re: [PATCH -next] staging/easycap: make module params private/static, fix build References: <20101111125655.15fa7188.sfr@canb.auug.org.au> <20101111104422.28594510.randy.dunlap@oracle.com> <4CDC67FF.3010006@sciolus.org> In-Reply-To: <4CDC67FF.3010006@sciolus.org> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 11/11/10 14:02, R.M. Thomas wrote: > Randy Dunlap wrote: >> The easycap driver has module parameters (bars, gain, & debug) >> with global scope that intrude on the kernel namespace and cause >> build problems. Change the names of them to be driver-specific >> and make 2 of them static. > > I did do test builds of the driver in-tree prior to submitting the > recent set of patches, but did not see any warnings so did not spot > this mistake. There's presumably something wrong with the way I'm > building the kernel. This build problem shows up when this driver is built into the kernel image, not built as a loadable module. Did you only build as a loadable module, maybe? and what CPU architecture did you build for? That could also matter. -- ~Randy *** Remember to use Documentation/SubmitChecklist when testing your code ***