[PATCH] staging: ks7010: use little-endian types

2017-06-08 Thread Perry Hooker
This patch fixes a number of sparse warnings of the form: drivers/staging/ks7010/ks_hostif.c:2187:29: warning: incorrect type in assignment (different base types) generated when storing little-endian data in variables that do not have a specified endianness. Signed-off-by: Perry Hooker

Re: [PATCH] staging: wilc1000: use kernel define byte order macros

2017-03-23 Thread Robert Perry Hooker
-0600, Robert Perry Hooker wrote: > > I don't think buff is an ieee80211_hdr struct. I think it's the rx_buffer > > allocated at wilc_wlan.c:1417. > > > > The rx_buffer is going to end up filled with endian data, right? > > regards, > dan carpenter > __

Re: [PATCH] staging: wilc1000: use kernel define byte order macros

2017-03-22 Thread Robert Perry Hooker
I don't think buff is an ieee80211_hdr struct. I think it's the rx_buffer allocated at wilc_wlan.c:1417. Regards, Perry On Wed, 2017-03-22 at 12:24 +0300, Dan Carpenter wrote: > On Tue, Mar 21, 2017 at 03:40:10PM -0600, Robert Perry Hooker wrote: > > Thanks for taking a lo

Re: [PATCH] staging: wilc1000: use kernel define byte order macros

2017-03-21 Thread Robert Perry Hooker
). Is there an issue with calling cpu_to_le16 on a u8 that isn't encountered by implicitly casting a u8 to __le16? Or am I missing something else? Regards, Perry On Tue, 2017-03-21 at 23:19 +0300, Dan Carpenter wrote: > On Tue, Mar 21, 2017 at 01:55:40PM -0600, Perry Hooker wrote: > &g

[PATCH] staging: wilc1000: use kernel define byte order macros

2017-03-21 Thread Perry Hooker
) Signed-off-by: Perry Hooker --- drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 20 +--- 1 file changed, 13 insertions(+), 7 deletions(-) diff --git a/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c b/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c index a37896f..d1c75c7

[PATCH] staging: rtl8192u: use __le16_to_cpu instead of cast

2017-03-12 Thread Perry Hooker
__le16 Signed-off-by: Perry Hooker --- drivers/staging/rtl8192u/r8192U_dm.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U_dm.c b/drivers/staging/rtl8192u/r8192U_dm.c index 9209aad..33d105d 100644 --- a/drivers/staging/rtl8192u/r8192U_dm.c

Re: [PATCH] staging: r8712u: use __le32 type for little-endian data

2017-02-11 Thread Perry Hooker
Thank you all for taking the time to look at this. I'm sorry for filling your inboxes with my mistakes - as you probably guessed, I'm new to kernel development, so I really appreciate the feedback. Perry On Fri, Feb 10, 2017 at 1:55 PM, Larry Finger wrote: > On 02/10/2017 12:23 PM,

Re: [PATCH] staging: r8712u: remove unnecessary le32_to_cpu

2017-02-10 Thread Perry Hooker
-0600, Larry Finger wrote: >>> >>> On 02/10/2017 08:08 AM, Greg KH wrote: >>>> >>>> On Thu, Feb 09, 2017 at 08:51:55PM -0700, Perry Hooker wrote: >>>>> >>>>> This patch fixes the following sparse warning: >>>>> drivers/s

[PATCH] staging: r8712u: use __le32 type for little-endian data

2017-02-10 Thread Perry Hooker
This patch fixes the following sparse warning: drivers/staging/rtl8712/usb_ops_linux.c:212:33: warning: cast to restricted __le32 Signed-off-by: Perry Hooker --- drivers/staging/rtl8712/usb_ops_linux.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/staging

[PATCH] staging: r8712u: remove unnecessary le32_to_cpu

2017-02-09 Thread Perry Hooker
This patch fixes the following sparse warning: drivers/staging/rtl8712/usb_ops_linux.c:212:33: warning: cast to restricted __le32 Signed-off-by: Perry Hooker --- drivers/staging/rtl8712/usb_ops_linux.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8712

[PATCH v3] staging: lustre: libcfs: move assignment out of conditional

2015-07-13 Thread Perry Hooker
Found by checkpatch.pl Signed-off-by: Perry Hooker --- .../staging/lustre/include/linux/libcfs/libcfs_fail.h | 18 ++ 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/drivers/staging/lustre/include/linux/libcfs/libcfs_fail.h b/drivers/staging/lustre/include/linux

[PATCH] staging: lustre: libcfs: move assignment out of conditional

2015-07-12 Thread Perry Hooker
Found by checkpatch.pl Signed-off-by: Perry Hooker Please enter the commit message for your changes. Lines starting --- .../staging/lustre/include/linux/libcfs/libcfs_fail.h | 18 ++ 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/drivers/staging/lustre/include

Re: [HPDD-discuss] [PATCH] staging: lustre: libcfs: move assignment out of if condition

2015-07-12 Thread Perry Hooker
g] on behalf of Perry Hooker [perry.hoo...@gmail.com] Sent: Sunday, July 12, 2015 4:27 PM To: oleg.dro...@intel.com; andreas.dil...@intel.com; gre...@linuxfoundation.org; de...@driverdev.osuosl.org; hpdd-disc...@lists.01.org Cc: Perry Hooker Subject: [HPDD-discuss] [PATCH] staging: lustre: libcfs:

[PATCH] staging: lustre: libcfs: move assignment out of if condition

2015-07-12 Thread Perry Hooker
Found by checkpatch.pl Signed-off-by: Perry Hooker --- drivers/staging/lustre/include/linux/libcfs/libcfs_fail.h | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/include/linux/libcfs/libcfs_fail.h b/drivers/staging/lustre/include/linux/libcfs