On Fri, 2017-03-03 at 23:24-0800, Joe Perches wrote:
>
> More stuff the changelog doesn't show :(
>
> On Fri, 2017-03-03 at 22:58 +0200, Ernestas Kulik wrote:
> > This fixes type warnings generated by sparse, replaces instances of
> > ntohs() with be16_to_cpu() and rem
Currently, sparse generates many warnings for the driver. This commit
changes the types of struct fields/function variables to match the
endianness at their assignment.
Signed-off-by: Ernestas Kulik
---
Changes from v1:
* Change the type of the variable being passed to ntohs() instead of casting
On Fri, 2017-03-03 at 22:37-0800, Joe Perches wrote:
>
> struct hostif_hdr.event is declared at uint16_t
> and not as __le16 so I believe this is incorrect
> and actually introduces a sparse error.
Sure, but I change that in the patch. :)
On Sat, 2017-03-04 at 08:28 +0200, Ernestas
On Fri, 2017-03-03 at 17:23-0800, Joe Perches wrote:
> On Fri, 2017-03-03 at 22:58 +0200, Ernestas Kulik wrote:
>
> > This fixes type warnings generated by sparse, replaces instances of
> > ntohs() with be16_to_cpu() and removes unused fields in structs.
>
> There's
This fixes type warnings generated by sparse, replaces instances of
ntohs() with be16_to_cpu() and removes unused fields in structs.
Signed-off-by: Ernestas Kulik
---
drivers/staging/ks7010/ks7010_sdio.c | 12 ++--
drivers/staging/ks7010/ks_hostif.c | 30 +
drivers/staging/ks7010
Using octal permissions instead of symbolic ones is preferred.
Signed-off-by: Ernestas Kulik
---
drivers/staging/lustre/lustre/llite/dir.c | 2 +-
drivers/staging/lustre/lustre/llite/file.c | 2 +-
drivers/staging/lustre/lustre/llite/llite_lib.c | 4 ++--
drivers/staging/lustre
Using octal permissions instead of symbolic ones is preferred.
Signed-off-by: Ernestas Kulik
---
drivers/staging/lustre/lnet/libcfs/module.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/lustre/lnet/libcfs/module.c
b/drivers/staging/lustre/lnet/libcfs