Re: [Outreachy kernel] [PATCH 1/6] staging: wlan-ng: Fix sparse warning of restricted __le16

2017-03-02 Thread Alison Schofield
On Thu, Mar 02, 2017 at 12:14:40PM +0100, Julia Lawall wrote: > > > On Thu, 2 Mar 2017, SIMRAN SINGHAL wrote: > > > On Thu, Mar 2, 2017 at 3:20 PM, Julia Lawall wrote: > > > > > > > > > On Thu, 2 Mar 2017, Julia Lawall wrote: > > > > > >> > > >> > > >> On Thu, 2 Mar 2017,

Re: [Outreachy kernel] [PATCH 1/6] staging: wlan-ng: Fix sparse warning of restricted __le16

2017-03-02 Thread Alison Schofield
On Thu, Mar 02, 2017 at 12:14:40PM +0100, Julia Lawall wrote: > > > On Thu, 2 Mar 2017, SIMRAN SINGHAL wrote: > > > On Thu, Mar 2, 2017 at 3:20 PM, Julia Lawall wrote: > > > > > > > > > On Thu, 2 Mar 2017, Julia Lawall wrote: > > > > > >> > > >> > > >> On Thu, 2 Mar 2017, simran singhal wrote:

Re: [Outreachy kernel] [PATCH 1/6] staging: wlan-ng: Fix sparse warning of restricted __le16

2017-03-02 Thread Julia Lawall
On Thu, 2 Mar 2017, simran singhal wrote: > This patch fixes the following sparse warning: > warning: cast to restricted __le16 You commit message should not say just fix X. What have you done to carry out the fix and why is this the correct approach? julia > > Signed-off-by: simran singhal

Re: [Outreachy kernel] [PATCH 1/6] staging: wlan-ng: Fix sparse warning of restricted __le16

2017-03-02 Thread Julia Lawall
On Thu, 2 Mar 2017, simran singhal wrote: > This patch fixes the following sparse warning: > warning: cast to restricted __le16 You commit message should not say just fix X. What have you done to carry out the fix and why is this the correct approach? julia > > Signed-off-by: simran singhal

Re: [Outreachy kernel] [PATCH 1/6] staging: wlan-ng: Fix sparse warning of restricted __le16

2017-03-02 Thread SIMRAN SINGHAL
On Thu, Mar 2, 2017 at 3:20 PM, Julia Lawall wrote: > > > On Thu, 2 Mar 2017, Julia Lawall wrote: > >> >> >> On Thu, 2 Mar 2017, simran singhal wrote: >> >> > This patch fixes the following sparse warning: >> > warning: cast to restricted __le16 >> >> You commit message

Re: [Outreachy kernel] [PATCH 1/6] staging: wlan-ng: Fix sparse warning of restricted __le16

2017-03-02 Thread SIMRAN SINGHAL
On Thu, Mar 2, 2017 at 3:20 PM, Julia Lawall wrote: > > > On Thu, 2 Mar 2017, Julia Lawall wrote: > >> >> >> On Thu, 2 Mar 2017, simran singhal wrote: >> >> > This patch fixes the following sparse warning: >> > warning: cast to restricted __le16 >> >> You commit message should not say just fix X.

Re: [Outreachy kernel] [PATCH 1/6] staging: wlan-ng: Fix sparse warning of restricted __le16

2017-03-02 Thread Julia Lawall
On Thu, 2 Mar 2017, SIMRAN SINGHAL wrote: > On Thu, Mar 2, 2017 at 3:20 PM, Julia Lawall wrote: > > > > > > On Thu, 2 Mar 2017, Julia Lawall wrote: > > > >> > >> > >> On Thu, 2 Mar 2017, simran singhal wrote: > >> > >> > This patch fixes the following sparse warning: > >>

Re: [Outreachy kernel] [PATCH 1/6] staging: wlan-ng: Fix sparse warning of restricted __le16

2017-03-02 Thread Julia Lawall
On Thu, 2 Mar 2017, SIMRAN SINGHAL wrote: > On Thu, Mar 2, 2017 at 3:20 PM, Julia Lawall wrote: > > > > > > On Thu, 2 Mar 2017, Julia Lawall wrote: > > > >> > >> > >> On Thu, 2 Mar 2017, simran singhal wrote: > >> > >> > This patch fixes the following sparse warning: > >> > warning: cast to

Re: [Outreachy kernel] [PATCH 1/6] staging: wlan-ng: Fix sparse warning of restricted __le16

2017-03-02 Thread Julia Lawall
On Thu, 2 Mar 2017, Julia Lawall wrote: > > > On Thu, 2 Mar 2017, simran singhal wrote: > > > This patch fixes the following sparse warning: > > warning: cast to restricted __le16 > > You commit message should not say just fix X. What have you done to carry > out the fix and why is this the

Re: [Outreachy kernel] [PATCH 1/6] staging: wlan-ng: Fix sparse warning of restricted __le16

2017-03-02 Thread Julia Lawall
On Thu, 2 Mar 2017, Julia Lawall wrote: > > > On Thu, 2 Mar 2017, simran singhal wrote: > > > This patch fixes the following sparse warning: > > warning: cast to restricted __le16 > > You commit message should not say just fix X. What have you done to carry > out the fix and why is this the