linux-kernel - Re: [PATCH 1/1] toshiba_acpi: Add support for bluetooth toggling through rfkill

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20080728030437.GC10672@khazad-dum.debian.net>
Date:	Mon, 28 Jul 2008 00:04:37 -0300
From:	Henrique de Moraes Holschuh <hmh@....eng.br>
To:	Philip Langdale <philipl@...rt.org>
Cc:	Ivo van Doorn <ivdoorn@...il.com>,
	LKML <linux-kernel@...r.kernel.org>,
	Matthew Garrett <mjg59@...f.ucam.org>,
	toshiba_acpi@...ebeam.org, linux-wireless@...r.kernel.org
Subject: Re: [PATCH 1/1] toshiba_acpi: Add support for bluetooth toggling
	through rfkill

On Sun, 27 Jul 2008, Philip Langdale wrote:
> Ivo van Doorn wrote:
>>
>> You don't seem to be using rfkill_force_state() which is required to inform the rfkill
>> layer about the state changes.
>
> Hmm? According to rfkill.txt, one can either use force_state() or implement the
> get_state() hook, and I have done the later. If this is not the correct method,
> can you please explain when I should be using force_state?

There is a bunch of rfkill bug fix patches that was not merged in
wireless-testing yet (which is a pity, it would be really good if they could
go into 2.6.27).  One of those patches fixes the docs to make it clear that
rfkill_force_state() is the way to go if you have events of any sort.

The big difference from get_state() is that rfkill_force_state() propagates
state changes as soon as your driver notices them and sends them to rfkill
through rfkill_force_state()... while get_state() can only do that when
someone asks rfkill about the current state.

-- 
  "One disk to rule them all, One disk to find them. One disk to bring
  them all and in the darkness grind them. In the Land of Redmond
  where the shadows lie." -- The Silicon Valley Tarot
  Henrique Holschuh
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/