Hi,
> -----Original Message-----
> From: Bjørn Mork [mailto:[email protected]]
> Sent: Thursday, March 12, 2015 6:38 PM
> To: Avinash Patil
> Cc: [email protected]; Amitkumar Karwar; Cathy Luo; Zhaoyang Liu;
> Chin-Ran Lo; Plus Chen; Shengzhen Li; Nishant Sarmukadam
> Subject: Re: [PATCH 3/9] mwifiex: enhance SD8897 MP aggregation limits
>
> Avinash Patil <[email protected]> writes:
>
> > diff --git a/drivers/net/wireless/mwifiex/sdio.h
> > b/drivers/net/wireless/mwifiex/sdio.h
> > index c636944..5a23bd0 100644
> > --- a/drivers/net/wireless/mwifiex/sdio.h
> > +++ b/drivers/net/wireless/mwifiex/sdio.h
> > @@ -67,6 +67,7 @@
> >
> > #define MWIFIEX_MP_AGGR_BUF_SIZE_16K (16384)
> > #define MWIFIEX_MP_AGGR_BUF_SIZE_32K (32768)
> > +#define MWIFIEX_MP_AGGR_BUF_SIZE_64K (65280)
>
> The first two macros are clearly matching their 16K and 32K names, but the 64K
> one is not. Care to add a comment explaining where the missing
> 256 bytes went?
We reserve first block of 256 bytes for DMA alignment.
I will add a comment to definition; also rename define since it is not exactly
64K.
> Bjørn (always curious about surprising numbers :-)
-Avinash
N�����r��y����b�X��ǧv�^�){.n�+����{��*ޕ�,�{ay� ʇڙ�,j��f���h���z� �w���
���j:+v���w�j�m��������zZ+�����ݢj"��!�i
