Hi Marcel,
On Wed, Nov 16, 2011 at 10:13:10AM +0900, Marcel Holtmann wrote:
> Hi Andrei,
>
> > > Type can be changed during re-opening device /dev/vhci.
> > >
> > > Signed-off-by: Andrei Emeltchenko <andrei.emeltchenko@xxxxxxxxx>
> > > Acked-by: Marcel Holtmann <marcel@xxxxxxxxxxxx>
> > > ---
> > > drivers/bluetooth/hci_vhci.c | 8 ++++++++
> > > 1 files changed, 8 insertions(+), 0 deletions(-)
> > >
> > > diff --git a/drivers/bluetooth/hci_vhci.c b/drivers/bluetooth/hci_vhci.c
> > > index 2e302a1..2ed6ab1 100644
> > > --- a/drivers/bluetooth/hci_vhci.c
> > > +++ b/drivers/bluetooth/hci_vhci.c
> > > @@ -41,6 +41,8 @@
> > >
> > > #define VERSION "1.3"
> > >
> > > +static bool amp;
> > > +
> > > struct vhci_data {
> > > struct hci_dev *hdev;
> > >
> > > @@ -239,6 +241,9 @@ static int vhci_open(struct inode *inode, struct file *file)
> > > hdev->bus = HCI_VIRTUAL;
> > > hdev->driver_data = data;
> > >
> > > + if (amp)
> > > + hdev->dev_type = HCI_AMP;
> > > +
> >
> > I am thinking about changing lines above to:
> >
> > - if (amp)
> > + if (amp) {
> > hdev->dev_type = HCI_AMP;
> > + hdev->amp_type = AMP_CTRL_BLUETOOTH_ONLY;
> > + }
> >
> > to indicate that that "the AMP Controller is only used by the Bluetooth
> > technology and will not be shared with other non-Bluetooth technologies".
>
> this can not be static information at this point. It will change
> depending on how the WiFi subsystem or WiFi drivers want it to be.
>
> And lets cross the bridge when we get to it. This is too early to figure
> out what to do here.
The code only emulates virtual HCI amp_type. The reason why I added it is
to reply for A2MP Discover Req. If we leave field as 0 it would mean that
that controller is powered down.
This value can be read with "read local amp info", I can implement this
command in hciemu as other option.
Best regards
Andrei Emeltchenko
--
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
