From mboxrd@z Thu Jan 1 00:00:00 1970 From: Darren Hart Subject: Re: [PATCH 3/4] net/rfkill: Create "airplane mode" LED trigger Date: Fri, 18 Dec 2015 16:22:12 -0800 Message-ID: <20151219002212.GC7244@malice.jf.intel.com> References: <1450193442-7930-1-git-send-email-jprvita@endlessm.com> <1450193442-7930-4-git-send-email-jprvita@endlessm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: Corentin Chary , platform-driver-x86@vger.kernel.org, acpi4asus-user@lists.sourceforge.net, =?iso-8859-1?Q?Jo=E3o?= Paulo Rechi Vita , linux-wireless@vger.kernel.org, netdev@vger.kernel.org To: =?iso-8859-1?Q?Jo=E3o?= Paulo Rechi Vita , Johannes Berg , "David S. Miller" Return-path: Received: from bombadil.infradead.org ([198.137.202.9]:35142 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752190AbbLSAWR (ORCPT ); Fri, 18 Dec 2015 19:22:17 -0500 Content-Disposition: inline In-Reply-To: <1450193442-7930-4-git-send-email-jprvita@endlessm.com> Sender: netdev-owner@vger.kernel.org List-ID: On Tue, Dec 15, 2015 at 10:30:41AM -0500, Jo=E3o Paulo Rechi Vita wrote= : > For platform drivers to be able to correctly drive the "Airplane Mode= " > indicative LED there needs to be a RFKill LED trigger tied to the glo= bal > state of RFKILL_TYPE_ALL (instead of to a specific RFKill) and that > works in an inverted manner of regular RFKill LED triggers, that is, = the > LED is ON when the state is blocked, and OFF otherwise. >=20 > This commit implements such a trigger, which will be used by the > asus-wrc x86 platform driver. So this will need to go through Johannes and David per get_maintainer.p= l before we can use it in platform drivers. +Johannes +David +wireless +netdev --=20 Darren Hart Intel Open Source Technology Center >=20 > Signed-off-by: Jo=E3o Paulo Rechi Vita > --- > net/rfkill/core.c | 30 ++++++++++++++++++++++++++++++ > 1 file changed, 30 insertions(+) >=20 > diff --git a/net/rfkill/core.c b/net/rfkill/core.c > index b41e9ea..3effc29 100644 > --- a/net/rfkill/core.c > +++ b/net/rfkill/core.c > @@ -124,6 +124,26 @@ static bool rfkill_epo_lock_active; > =20 > =20 > #ifdef CONFIG_RFKILL_LEDS > +static void airplane_mode_led_trigger_activate(struct led_classdev *= led); > + > +static struct led_trigger airplane_mode_led_trigger =3D { > + .name =3D "rfkill-airplane-mode", > + .activate =3D airplane_mode_led_trigger_activate, > +}; > + > +static void airplane_mode_led_trigger_event(void) > +{ > + if (rfkill_global_states[RFKILL_TYPE_ALL].cur & RFKILL_BLOCK_ANY) > + led_trigger_event(&airplane_mode_led_trigger, LED_FULL); > + else > + led_trigger_event(&airplane_mode_led_trigger, LED_OFF); > +} > + > +static void airplane_mode_led_trigger_activate(struct led_classdev *= led) > +{ > + airplane_mode_led_trigger_event(); > +} > + > static void rfkill_led_trigger_event(struct rfkill *rfkill) > { > struct led_trigger *trigger; > @@ -175,6 +195,10 @@ static void rfkill_led_trigger_unregister(struct= rfkill *rfkill) > led_trigger_unregister(&rfkill->led_trigger); > } > #else > +static void airplane_mode_led_trigger_event(void) > +{ > +} > + > static void rfkill_led_trigger_event(struct rfkill *rfkill) > { > } > @@ -346,6 +370,7 @@ static void __rfkill_switch_all(const enum rfkill= _type type, bool blocked) > =20 > for (i =3D 0; i < NUM_RFKILL_TYPES; i++) > rfkill_global_states[i].cur =3D blocked; > + airplane_mode_led_trigger_event(); > } else { > rfkill_global_states[type].cur =3D blocked; > } > @@ -1177,6 +1202,7 @@ static ssize_t rfkill_fop_write(struct file *fi= le, const char __user *buf, > enum rfkill_type i; > for (i =3D 0; i < NUM_RFKILL_TYPES; i++) > rfkill_global_states[i].cur =3D ev.soft; > + airplane_mode_led_trigger_event(); > } else { > rfkill_global_states[ev.type].cur =3D ev.soft; > } > @@ -1293,6 +1319,10 @@ static int __init rfkill_init(void) > } > #endif > =20 > +#ifdef CONFIG_RFKILL_LEDS > + led_trigger_register(&airplane_mode_led_trigger); > +#endif > + > out: > return error; > } > --=20 > 2.5.0 >=20 > -- > To unsubscribe from this list: send the line "unsubscribe platform-dr= iver-x86" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html >=20