From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ben Hutchings Subject: Re: [PATCH v2] drivers: net: Fix dependency for EEPROM_93CX6 Date: Thu, 5 Jan 2012 17:03:27 +0000 Message-ID: <1325783007.3764.19.camel@bwh-desktop> References: <1325770673-17309-1-git-send-email-festevam@gmail.com> <4F05CBA9.1070602@lwfinger.net> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Cc: Fabio Estevam , , , , , , Fabio Estevam To: Larry Finger Return-path: In-Reply-To: <4F05CBA9.1070602-tQ5ms3gMjBLk1uMJSBkQmQ@public.gmane.org> Sender: linux-wireless-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: netdev.vger.kernel.org On Thu, 2012-01-05 at 10:11 -0600, Larry Finger wrote: > On 01/05/2012 07:37 AM, Fabio Estevam wrote: > > Fix the following build warning: > > > > warning: (KS8851&& AX88796_93CX6&& RTL8180&& RTL8187&& ADM8211&& RT2400PCI&& RT2500PCI&& RT61PCI&& RT2800PCI&& R8187SE) selects EEPROM_93CX6 which has unmet direct dependencies (MISC_DEVICES) > > > > Signed-off-by: Fabio Estevam > > --- > > Changes since v1: > > - Place MISC_DEVICES dependency into the 'depends on' line > > Is this the right way to fix this? Whenever I get this kind of build warning, I > usually attribute it to a problem with my local configuration and fix my copy of > .config, not modify the build system. With this change, it seems to me that a > lot of devices will suddenly disappear from the build with little explanation. I entirely agree. > I don't feel confident enough to NACK the patch, but I would like an expert to > comment. I'm not an expert but am prepared to be opinionated! > I have noticed that the defconfigs for various architectures are split between > turning MISC_DEVICES on or off. That seems like a bug, since MISC_DEVICES doesn't by itself select any code. (It's also not a meaningful category and maybe ought not to be an option at all.) Ben. -- Ben Hutchings, Staff Engineer, Solarflare Not speaking for my employer; that's the marketing department's job. They asked us to note that Solarflare product names are trademarked. -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html