Re: [PATCH] Bluetooth: Fix permission of enable_le param

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On Tue, Oct 11, 2011 at 3:33 PM, Claudio Takahasi
<claudio.takahasi@xxxxxxxxxxxxx> wrote:
> On Tue, Oct 11, 2011 at 3:58 PM, Gustavo F. Padovan
> <padovan@xxxxxxxxxxxxxx> wrote:
>> With 0444 it is impossible to change the param, changing it to 0644.
>
> If you allow changing the "enable_le" parameter dynamically it is also
> necessary check the controller features and update the LE host
> supported bit(if necessary).
> I am afraid that allowing to change it dynamically will break the
> consistency: what is gonna happen with the active operations(discovery
> & connections)?
>
> In our backlog we were planning to check LE host supported bit to
> allow/block LE connections or any other feature related to LE.

For me, it makes more sense to have bluez set LE host supported bit on
start, and have no "enable_le" module parameter at all.

I think just allowing to change enable_le after the module has been
loaded will not work as expected.

Regards,
-- 
Anderson Lizardo
Instituto Nokia de Tecnologia - INdT
Manaus - Brazil
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux