# Re: \[PATCH\] Correct structures for AMP Assoc Read

- From: Johan Hedberg \<hidden\>
- Date: 2012-01-05 10:40:37 
- List: linux-bluetooth
- Page: https://ratatoskr.run/linux-bluetooth/2012/01/9618467
- Thread: https://ratatoskr.run/linux-bluetooth/2011/12/9617105/t.md

```text
Hi Andrei,

On Thu, Jan 05, 2012, Emeltchenko Andrei wrote:
> On Thu, Dec 15, 2011 at 05:27:59PM +0100, Marcel Holtmann wrote:
> > Hi Andrei,
> > 
> > > Make struct names similar to kernel and remove static array since
> > > fragment length can be other size than HCI_MAX_NAME_LENGTH, which
> > > is for totally different purpose.
> > > ---
> > >  lib/hci.h |   11 +++++------
> > >  1 files changed, 5 insertions(+), 6 deletions(-)
> > > 
> > > diff --git a/lib/hci.h b/lib/hci.h
> > > index 51184ee..48692fa 100644
> > > --- a/lib/hci.h
> > > +++ b/lib/hci.h
> > > @@ -1382,17 +1382,16 @@ typedef struct {
> > >  #define OCF_READ_LOCAL_AMP_ASSOC	0x000A
> > >  typedef struct {
> > >  	uint8_t		handle;
> > > -	uint16_t	length_so_far;
> > > -	uint16_t	assoc_length;
> > > +	uint16_t	len_so_far;
> > > +	uint16_t	max_len;
> > >  } __attribute__ ((packed)) read_local_amp_assoc_cp;
> > > -#define READ_LOCAL_AMP_ASSOC_CP_SIZE 5
> > > +
> > >  typedef struct {
> > >  	uint8_t		status;
> > >  	uint8_t		handle;
> > > -	uint16_t	length;
> > > -	uint8_t		fragment[HCI_MAX_NAME_LENGTH];
> > > +	uint16_t	rem_len;
> > > +	uint8_t		frag[0];
> > >  } __attribute__ ((packed)) read_local_amp_assoc_rp;
> > > -#define READ_LOCAL_AMP_ASSOC_RP_SIZE 252
> > >  
> > >  #define OCF_WRITE_REMOTE_AMP_ASSOC	0x000B
> > >  typedef struct {
> > 
> > I am fine with these changes.
> > 
> > Acked-by: Marcel Holtmann <marcel@holtmann.org>
> 
> Johan can this patch be applied?

Yes, I had marked the email as read by mistake. It has now been pushed
upstream.

Johan
```