On Wed, Mar 25, 2015 at 04:18:19PM -0600, Azael Avalos wrote: > > That suggests to me the checks for rfkill, powered, and plugged states should be > > done in the _enable() function itself, rather than at all the call sites. > > > > So... would a better fix be to push these two additional tests into > > toshiba_bluetooth_enable() before it calls AUSB and BTPO and then retain the > > call the _enable() here? > > Sure, I can apply these changes there. > > What I wanted was to isolate what each call does as a preparation for > upcoming patches where I will be purging toshiba_acpi from the BT > RFKill code and add it to toshiba_bluetooth (where it belongs). OK, I'll wait for v3 and pull it in. Thanks Azael. -- Darren Hart Intel Open Source Technology Center -- To unsubscribe from this list: send the line "unsubscribe platform-driver-x86" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html