From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-oi0-f66.google.com ([209.85.218.66]:33718 "EHLO mail-oi0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751256AbdGQE7I (ORCPT ); Mon, 17 Jul 2017 00:59:08 -0400 MIME-Version: 1.0 In-Reply-To: References: <20170707140941.122115-1-russell.joyce@york.ac.uk> <60de1301-b4a3-95a7-117f-a953576655d9@broadcom.com> From: =?UTF-8?B?UmFmYcWCIE1pxYJlY2tp?= Date: Mon, 17 Jul 2017 06:59:07 +0200 Message-ID: (sfid-20170717_065924_990949_B6BD574D) Subject: Re: [PATCH] brcmfmac: added LED triggers for transmit/receive To: Russell Joyce Cc: Arend van Spriel , Alan Millard , Franky Lin , Hante Meuleman , Chi-Hsien Lin , Wright Feng , Kalle Valo , Marc Kleine-Budde , Marcel Holtmann , "Michael S. Tsirkin" , Pieter-Paul Giesberts , =?UTF-8?B?UmFmYcWCIE1pxYJlY2tp?= , "mhiramat@kernel.org" , James Hughes , Tobias Klauser , Linux Kernel Mailing List , "linux-wireless@vger.kernel.org" , "open list:BROADCOM BRCM80211 IEEE802.11n WIRELESS DRIVER" , brcm80211-dev-list@cypress.com, Network Development Content-Type: text/plain; charset="UTF-8" Sender: linux-wireless-owner@vger.kernel.org List-ID: On 11 July 2017 at 17:01, Russell Joyce wrote: > Thanks for your comments. > >> What I think Rafa=C5=82 is saying is that it would be better to have thi= s >> code in cfg80211 so other drivers including mac80211 could use it. > > > While I agree that moving all wireless LED triggers to cfg80211 would be = an > ideal situation, it seems a bit out of scope for what I was trying to ach= ieve. > This would probably also require removing the mac80211 LED triggers (and = any > other similar triggers that might be created by specific wireless drivers= not > using mac80211), in order to consolidate them in one place. > > Besides this, I'm not sure where exactly in cfg80211 this functionality w= ould > go (I assume it was originally put in mac80211 instead for a reason?), > although I'm certainly no expert in this area of the kernel. I don't expect you to rewrite all mac80211 drivers at this point. Just focus on the generic cfg80211 helper and use it in brcmfmac. Other cfg80211 drivers and mac80211 may follow in the future. I'm not sure what's the best place in cfg80211 for this. Try something, or try to get some comments from cfg80211 guys. >> Indeed. However, the LED subsystem could/should(?) take care of mapping >> "rx" and "tx" triggers to the same LED. > > In terms of the LED triggers, the only alternative I can see is to create= a > single complex trigger that exposes "rx" and "tx" parameters that can be > individually enabled or disabled. This would reduce the number of trigger= s from > three to one, but also makes things slightly more awkward for the user, a= nd > deviates from the convention set by mac80211. Ideally we should have "rx" and "tx". LED subsystem should allow assigning *both* (at the same time) to the LED. I'll try to discuss with with LED guys this week. Sorry, I was busy last week. --=20 Rafa=C5=82