From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from nbd.name ([88.198.39.176]:48212 "EHLO ds10.nbd.name" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750984Ab0HBKK6 (ORCPT ); Mon, 2 Aug 2010 06:10:58 -0400 Message-ID: <4C5699AE.6090103@openwrt.org> Date: Mon, 02 Aug 2010 12:10:54 +0200 From: Felix Fietkau MIME-Version: 1.0 To: "Luis R. Rodriguez" CC: Bruno Randolf , johannes@sipsolutions.net, linville@tuxdriver.com, linux-wireless@vger.kernel.org Subject: Re: [PATCH v5] cfg80211: Add nl80211 antenna configuration References: <20100729035820.5930.29864.stgit@tt-desk> <201007291812.28639.br1@einfach.org> <201008021313.54850.br1@einfach.org> <4C5688F7.1070200@openwrt.org> <4C568D23.2000203@openwrt.org> <4C5690C7.2080104@openwrt.org> In-Reply-To: Content-Type: text/plain; charset=UTF-8 Sender: linux-wireless-owner@vger.kernel.org List-ID: On 2010-08-02 11:52 AM, Luis R. Rodriguez wrote: > On Mon, Aug 2, 2010 at 2:32 AM, Felix Fietkau wrote: >> On 2010-08-02 11:23 AM, Luis R. Rodriguez wrote: >>> Sure, I just did not see any code for this in these patches. My point >>> about the hw config vs fake/mod'd is if we'd expose the mod'd config >>> changes to userspace or if we'd keep them internal to cfg80211. How >>> would this be dealt with? >> Right now, cfg80211 doesn't know enough to handle this stuff on its own, >> so let's handle it in the driver completely on the first iteration. The >> patches do not need any changes for this right now. > > I'd prefer that code to be written rather then let this be defined as > API now and let drivers deal with this differently. But that's me, I'm > not the maintainer, I just will not deal with bug reports dealing with > this and I'll assign them to you guys if this gets through. Still > think its crap and should just go through debugfs until all the code > mentioned does exist. Sorry, but WTF? There's two parts to this: API visible to user space, and the internal API for handling changes. So you're suggesting to reject the user space API, because of missing parts in the internal API (which we can change any time) that will only be used for drivers that this series doesn't even contain any code for?? Am I confused here, or does this seem rather strange? - Felix