Bug 6718 - Broadcom RFKILL switches are exposed
Summary: Broadcom RFKILL switches are exposed
Status: RESOLVED FIXED
Alias: None
Product: MinnowBoard MAX Firmware
Classification: Hardware Platforms
Component: minnowboard-uefi-firmware (show other bugs)
Version: 2C A1
Hardware: MinnowBoard Max Multiple
: Medium normal
Target Milestone: Production Release
Assignee: Marcel Holtmann
QA Contact:
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2014-09-12 18:06 UTC by Marcel Holtmann
Modified: 2014-11-07 03:44 UTC (History)
6 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: No (bug/feature does not impact docs)


Attachments
test logfile (942 bytes, text/plain)
2014-11-07 03:42 UTC, California Sullivan
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Marcel Holtmann 2014-09-12 18:06:29 UTC
When loading rfkill-gpio driver, the Broadcom RFKILL switches for GPS and Bluetooth are exposed:

[ 7545.939852] rfkill_gpio BCM2E1A:00: BCM2E1A:00 device registered.
[ 7545.945247] rfkill_gpio BCM4752:00: BCM4752:00 device registered.

0: BCM2E1A:00: Bluetooth
	Soft blocked: no
	Hard blocked: no
1: BCM4752:00: GPS
	Soft blocked: no
	Hard blocked: no

Since the board does not have a Broadcom wireless device on it, I have no idea what these are useful for? Are these GPIO PINs actually exposed somehow?

These are platform/80860F0A:00/BCM2E1A:00 and platform/80860F0A:01/BCM4752:00 devices with firmware path \_SB_.URT1.BTH0 and \_SB_.URT2.GPS0 respectively.
Comment 1 David Anders 2014-09-12 18:15:00 UTC
can you provide more information on what device this is, how it is connected to the minnowmax?
Comment 2 Marcel Holtmann 2014-09-12 18:21:02 UTC
These switches are exposed via ACPI. On some Baytrail devices they control the reset and shutdown states of Bluetooth and GPS builtin modules. Since the Minnowboard Max does not have these, it have no idea what the behavior would be. So it might be better if these switches are not exposed via ACPI.
Comment 3 David Anders 2014-09-12 18:30:16 UTC
Tim,

we do not implelment these GPIOs on the minnowmax. they are used to disable wifi/bt/gps via a mPCIe interface. these do not need to be enumerated currently...
Comment 4 Darren Hart 2014-09-12 20:11:26 UTC
When a mini-PCIe Lure is connected, what mechanism do we use to RFKILL?
Comment 5 David Anders 2014-09-12 20:13:25 UTC
we don't currently. we did not implement it on either the seacat or flotsam lures. because this mechanism wasn't ironed out at the time of the design, we did not include it. we can always do a revision of the lures as needed...

Dave
Comment 6 David_Wei 2014-10-09 09:36:21 UTC
Hi marcel,

These devices have just been removed from our open source release. Please verify it and close this bug.
Comment 7 California Sullivan 2014-11-07 03:42:29 UTC
Created attachment 2234 [details]
test logfile

Image: core-image-sato image built with poky dizzy release with kernel configured with rfkill_gpio as a module.

Firmware: 0.73 open source release as found at https://uefidk.com/content/minnowboard-max
Comment 8 California Sullivan 2014-11-07 03:44:59 UTC
Confirmed fixed as per my test a above.

(Sorry for the extra comment - wanted to add an attachment and didn't realize the attachment page was also the comment page)