Re: [PATCH] staging: rtl8723au: Fix sparse warning: cast to restricted __le16

2015-08-07 Thread Johannes Postma
On 06/08/15 at 08:21am, Jes Sorensen wrote: Looks OK to me. Probably worth changing the *(usPtr + index) to be usPtr[index] as well to make the code easier to read. Jes Thank you for reviewing. I will make a seperate patch for that. I will send it after this one is accepted. Or should

Re: [PATCH] staging: rtl8723au: Fix sparse warning: cast to restricted __le16

2015-08-07 Thread Jes Sorensen
Johannes Postma jgmpos...@gmail.com writes: On 06/08/15 at 08:21am, Jes Sorensen wrote: Looks OK to me. Probably worth changing the *(usPtr + index) to be usPtr[index] as well to make the code easier to read. Jes Thank you for reviewing. I will make a seperate patch for that. I will

Re: [PATCH] staging: rtl8723au: Fix sparse warning: cast to restricted __le16

2015-08-07 Thread Johannes Postma
On 07/08/15 at 08:26am, Jes Sorensen wrote: Johannes Postma jgmpos...@gmail.com writes: On 06/08/15 at 08:21am, Jes Sorensen wrote: Looks OK to me. Probably worth changing the *(usPtr + index) to be usPtr[index] as well to make the code easier to read. Jes Thank you for

[PATCH] staging: rtl8723au: Fix sparse warning: cast to restricted __le16

2015-08-06 Thread Johannes Postma
usPtr is used as __le16 *, but was defined as u16 *. This was reported by sparse as: drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c:1850:29: warning: cast to restricted __le16 This patch fixes the type of usPtr. Signed-off-by: Johannes Postma jgmpos...@gmail.com ---

Re: [PATCH] staging: rtl8723au: Fix sparse warning: cast to restricted __le16

2015-08-06 Thread Jes Sorensen
Johannes Postma jgmpos...@gmail.com writes: usPtr is used as __le16 *, but was defined as u16 *. This was reported by sparse as: drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c:1850:29: warning: cast to restricted __le16 This patch fixes the type of usPtr. Signed-off-by: Johannes Postma

[PATCH] staging: rtl8723au: Fix sparse warning cast to restricted __le16

2015-03-11 Thread Marcus Folkesson
This patch fixes the following sparse warnings: CHECK drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c:265:37: warning: cast to restricted __le16 drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c:267:39: warning: cast to restricted

Re: [PATCH] staging: rtl8723au: Fix sparse warning cast to restricted __le16

2015-03-11 Thread Jes Sorensen
Marcus Folkesson marcus.folkes...@gmail.com writes: This patch fixes the following sparse warnings: CHECK drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c:265:37: warning: cast to restricted __le16