Re: [PATCH 1/2] Process Read Local AMP Info in hciemu

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

 



Hi Marcel,

On Thu, Dec 08, 2011 at 10:11:51AM +0200, Marcel Holtmann wrote:
> Hi Andrei,
> 
> > Add processing of Read Local AMP Info in hciemu, the values returned are
> > used to emulate AMP HCI.
> > ---
> >  test/hciemu.c |   34 ++++++++++++++++++++++++++++++++++
> >  1 files changed, 34 insertions(+), 0 deletions(-)
> > 
> > diff --git a/test/hciemu.c b/test/hciemu.c
> > index 38918f5..1022eb5 100644
> > --- a/test/hciemu.c
> > +++ b/test/hciemu.c
> > @@ -44,6 +44,7 @@
> >  #include <bluetooth/bluetooth.h>
> >  #include <bluetooth/hci.h>
> >  #include <bluetooth/hci_lib.h>
> > +#include <bluetooth/l2cap.h>
> >  
> >  #define VHCI_DEV		"/dev/vhci"
> >  
> > @@ -777,6 +778,35 @@ static void hci_info_param(uint16_t ocf, int plen, uint8_t *data)
> >  		break;
> >  	}
> >  }
> > +static void hci_status_param(uint16_t ocf, int plen, uint8_t *data)
> > +{
> > +	read_local_amp_info_rp ai;
> > +	uint8_t status;
> > +
> > +	const uint16_t ogf = OGF_STATUS_PARAM;
> > +
> > +	switch (ocf) {
> > +		case OCF_READ_LOCAL_AMP_INFO:
> 
> one too many indentation here.

will fix this. BTW: What do you think about current approach that in this
test tool I always reply for this AMP-specific command. Shall we have
command-line parameter for controller type?

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


[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