public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [REGRESSION] dell-laptop: Caused by "Remove rfkill code" commit
@ 2012-08-23 12:53 Sebastian Pöhn
  2012-08-23 13:14 ` Matthew Garrett
  0 siblings, 1 reply; 4+ messages in thread
From: Sebastian Pöhn @ 2012-08-23 12:53 UTC (permalink / raw)
  To: linux-kernel; +Cc: mjg

dell-laptop: Remove rfkill code
a6c2390cd6d2083d27a2359658e08f2d3df375ac

This patch causes some trouble with my dell laptop (Vostro 3300). It produces some annoying behaviour with the Wifi kill-switch.

As rfkill can not get the state of the physical kill-switch, both state may differ.

physical switch is OFF
rfkill state is ON
=> Laptop hangs on boot (maybe because a physical ascent device is used by network subsystem)

physical switch is OFF
rfkill state is OFF
=> Activate interface in Gnome Connection Manager => System hang

test setup:
Dell Vostro 3300
Broadcom 4313 Wifi
Kernel 3.5.[0-2] & 3.6
Fedora 17 

---

Again this is an example for patches accepted in mainline without being tested on supported hardware :-(


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [REGRESSION] dell-laptop: Caused by "Remove rfkill code" commit
  2012-08-23 12:53 [REGRESSION] dell-laptop: Caused by "Remove rfkill code" commit Sebastian Pöhn
@ 2012-08-23 13:14 ` Matthew Garrett
  2012-08-23 13:46   ` Sebastian Poehn
  0 siblings, 1 reply; 4+ messages in thread
From: Matthew Garrett @ 2012-08-23 13:14 UTC (permalink / raw)
  To: Sebastian Pöhn; +Cc: linux-kernel

On Thu, Aug 23, 2012 at 02:53:21PM +0200, Sebastian Pöhn wrote:

> As rfkill can not get the state of the physical kill-switch, both state may differ.

b43 should know the state of the physical rfkill switch even if there's 
no platform code to support it.

-- 
Matthew Garrett | mjg59@srcf.ucam.org

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [REGRESSION] dell-laptop: Caused by "Remove rfkill code" commit
  2012-08-23 13:14 ` Matthew Garrett
@ 2012-08-23 13:46   ` Sebastian Poehn
  2012-08-23 13:48     ` Matthew Garrett
  0 siblings, 1 reply; 4+ messages in thread
From: Sebastian Poehn @ 2012-08-23 13:46 UTC (permalink / raw)
  To: Matthew Garrett; +Cc: linux-kernel

On Thu, 2012-08-23 at 14:14 +0100, Matthew Garrett wrote:
> On Thu, Aug 23, 2012 at 02:53:21PM +0200, Sebastian Pöhn wrote:
> 
> > As rfkill can not get the state of the physical kill-switch, both state may differ.
> 
> b43 should know the state of the physical rfkill switch even if there's 
> no platform code to support it.
> 
BCM 4313 is _only_ supported by BCMs proprietar driver and brcmsmac; and
the later one I use.

Further there should be problems with other Wifis, like Intel?



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [REGRESSION] dell-laptop: Caused by "Remove rfkill code" commit
  2012-08-23 13:46   ` Sebastian Poehn
@ 2012-08-23 13:48     ` Matthew Garrett
  0 siblings, 0 replies; 4+ messages in thread
From: Matthew Garrett @ 2012-08-23 13:48 UTC (permalink / raw)
  To: Sebastian Poehn; +Cc: linux-kernel

On Thu, Aug 23, 2012 at 03:46:43PM +0200, Sebastian Poehn wrote:
> On Thu, 2012-08-23 at 14:14 +0100, Matthew Garrett wrote:
> > On Thu, Aug 23, 2012 at 02:53:21PM +0200, Sebastian Pöhn wrote:
> > 
> > > As rfkill can not get the state of the physical kill-switch, both state may differ.
> > 
> > b43 should know the state of the physical rfkill switch even if there's 
> > no platform code to support it.
> > 
> BCM 4313 is _only_ supported by BCMs proprietar driver and brcmsmac; and
> the later one I use.

Ok, well definitely seems like a bug in brcmsmac.

> Further there should be problems with other Wifis, like Intel?

Why?

-- 
Matthew Garrett | mjg59@srcf.ucam.org

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2012-08-23 13:48 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-08-23 12:53 [REGRESSION] dell-laptop: Caused by "Remove rfkill code" commit Sebastian Pöhn
2012-08-23 13:14 ` Matthew Garrett
2012-08-23 13:46   ` Sebastian Poehn
2012-08-23 13:48     ` Matthew Garrett

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox