2008-10-23 20:17:20

by Mario Limonciello

[permalink] [raw]
Subject: [PATCH] first udev rules for running hid2hci

Hi:

Attached is a patch that adds another set of udev rules that will run
hid2hci, at least for the Dell BT adapters. Other adapters can get
added in too, but I think some more attributes will be needed for them.
I'm doing this as a separate set of udev rules than the PCMCIA rules so
that these can be installed by default whereas the PCMCIA ones are
normally placed in a different binary package for distros.

Regards
--
Mario Limonciello
*Dell | Linux Engineering*
[email protected]


Attachments:
hid2hci_udev.patch (8.06 kB)
signature.asc (260.00 B)
OpenPGP digital signature
Download all attachments

2008-10-23 21:05:20

by Mario Limonciello

[permalink] [raw]
Subject: Re: [PATCH] first udev rules for running hid2hci

Marcel Holtmann wrote:
> Hi Mario,
>
>
>
> I like to have this all in one and that the PCMCIA ones are in a
> different binary package is a distro problem. They don't have to be
> since we don't have the dependency of pcmcia-cs anymore. That is ancient
> 2.4 stuff.
>
> If you are doing this, then also please modify hid2hci so it can take a
> parameter to select the device you run it for. Currently it would switch
> all of them. Even if you trigger it for only one.
>
> And I was actually more thinking to convert hid2hci into a bluetoothd
> plugin so the core daemon has control over it. Think about it.
>
> Regards
>
> Marcel
>
>
>
Hi Marcel:

That's about the response that I was expecting/thinking. I'll look into
modifying hid2hci to take parameters.

Regards

--
Mario Limonciello
*Dell | Linux Engineering*
[email protected]


Attachments:
signature.asc (260.00 B)
OpenPGP digital signature

2008-10-23 20:58:12

by Marcel Holtmann

[permalink] [raw]
Subject: Re: [PATCH] first udev rules for running hid2hci

Hi Mario,

> Attached is a patch that adds another set of udev rules that will run
> hid2hci, at least for the Dell BT adapters. Other adapters can get
> added in too, but I think some more attributes will be needed for them.
> I'm doing this as a separate set of udev rules than the PCMCIA rules so
> that these can be installed by default whereas the PCMCIA ones are
> normally placed in a different binary package for distros.

I like to have this all in one and that the PCMCIA ones are in a
different binary package is a distro problem. They don't have to be
since we don't have the dependency of pcmcia-cs anymore. That is ancient
2.4 stuff.

If you are doing this, then also please modify hid2hci so it can take a
parameter to select the device you run it for. Currently it would switch
all of them. Even if you trigger it for only one.

And I was actually more thinking to convert hid2hci into a bluetoothd
plugin so the core daemon has control over it. Think about it.

Regards

Marcel