[RFC 00/13] drivers: replace long unsigned int by unsigned long

Trivial patchset to use standard unsigned long in drivers/ instead of long unsigned int I'm just sending the summary; if you think it's worth being applied, I'll send this patchset and smaller ones for other trees. Regards, Fabian Fabian Frederick (13): qla4xxx: replace long

[PATCH 1/1 linux-next] scsi: remove unused label in sg_scsi_ioctl()

label 'out' is unused since commit 374f8fdea4aa ("scsi: Fix error handling in SCSI_IOCTL_SEND_COMMAND") Signed-off-by: Fabian Frederick --- block/scsi_ioctl.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/block/scsi_ioctl.c b/block/scsi_ioctl.c index cc7927a..d1ae46

Re: [PATCH 1/1 net-next] Bluetooth: fix shadow warning in hci_disconnect()

> On 23 October 2014 at 18:29 Marcel Holtmann wrote: > > > Hi Fabian, > > > use cpr for hci_cp_read_clock_offset instead of cp > > (already defined above). > > > > Signed-off-by: Fabian Frederick > > --- > > net/bluetooth/hci_conn.c | 6 +++-

[PATCH 1/1 linux-next] bsg: fix shadow warning in bsg_ioctl()

directly call scsi_cmd_ioctl() with (void __user *)arg instead of uarg redeclaration. Signed-off-by: Fabian Frederick --- block/bsg.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/block/bsg.c b/block/bsg.c index 276e869..c2f90bd 100644 --- a/block/bsg.c +++ b/block

[PATCH 1/1 linux-next] block: remove struct partition_meta_info from rescan_partitions

&state->parts[p].info is directly passed to add_partition(). local info is initialized but unused. Signed-off-by: Fabian Frederick --- block/partition-generic.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/block/partition-generic.c b/block/partition-generic.c index 0d9e5f9..

[PATCH 1/1 net-next] Bluetooth: fix shadow warning in hci_disconnect()

use cpr for hci_cp_read_clock_offset instead of cp (already defined above). Signed-off-by: Fabian Frederick --- net/bluetooth/hci_conn.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/net/bluetooth/hci_conn.c b/net/bluetooth/hci_conn.c index b9517bd..6151e09 100644

[PATCH 1/1 net-next] 9P: remove unused variable in p9_fd_create()

p is initialized but unused. Signed-off-by: Fabian Frederick --- net/9p/trans_fd.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/net/9p/trans_fd.c b/net/9p/trans_fd.c index 80d08f6..c73b894 100644 --- a/net/9p/trans_fd.c +++ b/net/9p/trans_fd.c @@ -1013,7 +1013,6 @@ p9_fd_create(struct

[PATCH 1/1 net-next] libceph: remove unused variable in handle_reply()

osdmap_epoch is redundant with reassert_epoch and unused. Signed-off-by: Fabian Frederick --- net/ceph/osd_client.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/net/ceph/osd_client.c b/net/ceph/osd_client.c index f3fc54e..432bd75 100644 --- a/net/ceph/osd_client.c +++ b/net/ceph

[PATCH 1/1 net-next] net: llc: include linux/errno.h instead of asm/errno.h

Signed-off-by: Fabian Frederick --- net/llc/llc_if.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/llc/llc_if.c b/net/llc/llc_if.c index 25c31c0..6daf391 100644 --- a/net/llc/llc_if.c +++ b/net/llc/llc_if.c @@ -15,7 +15,7 @@ #include #include #include -#include

[PATCH 1/1 net-next] lapb: move EXPORT_SYMBOL after functions.

See Documentation/CodingStyle Chapter 6 Signed-off-by: Fabian Frederick --- net/lapb/lapb_iface.c | 17 - 1 file changed, 8 insertions(+), 9 deletions(-) diff --git a/net/lapb/lapb_iface.c b/net/lapb/lapb_iface.c index 3cdaa04..fc60d9d 100644 --- a/net/lapb/lapb_iface.c +++ b

Re: [PATCH 1/1 net-next] mac80211: remove unnecessary null test before debugfs_remove

> On 21 October 2014 at 21:06 Johannes Berg wrote: > > > On Tue, 2014-10-21 at 18:20 +0200, Fabian Frederick wrote: > > Fix checkpatch warnings: > > > >     WARNING: debugfs_remove(NULL) is safe this check is probably not > >required > > I'll ap

[PATCH 1/1 linux-next] ceph: remove unused ceph_get_direct_page_vector

This function is not used since commit 64c3131161c1 ("ceph_sync_direct_write: stop poking into iov_iter guts") Signed-off-by: Fabian Frederick --- include/linux/ceph/libceph.h | 3 --- net/ceph/pagevec.c | 35 --- 2 files changed, 38 deletion

[PATCH 1/1 linux-next] fs/affs/file.c: adding support to O_DIRECT

Based on ext2_direct_IO Cc: Al Viro Cc: Andrew Morton Signed-off-by: Fabian Frederick --- fs/affs/file.c | 18 ++ 1 file changed, 18 insertions(+) diff --git a/fs/affs/file.c b/fs/affs/file.c index 8e51085..05005bd 100644 --- a/fs/affs/file.c +++ b/fs/affs/file.c @@ -12,6

[PATCH 1/1 net-next] mac80211: remove unnecessary null test before debugfs_remove

Fix checkpatch warnings: WARNING: debugfs_remove(NULL) is safe this check is probably not required Signed-off-by: Fabian Frederick --- net/mac80211/debugfs_key.c | 13 - 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/net/mac80211/debugfs_key.c b/net/mac80211

[tip:x86/cleanups] x86, msr: Use seek definitions instead of hard-coded values

Commit-ID: 03452d27c6cd9cebb59a6bb0fb6bd8557916c263 Gitweb: http://git.kernel.org/tip/03452d27c6cd9cebb59a6bb0fb6bd8557916c263 Author: Fabian Frederick AuthorDate: Fri, 17 Oct 2014 22:01:59 +0200 Committer: H. Peter Anvin CommitDate: Fri, 17 Oct 2014 13:40:55 -0700 x86, msr: Use seek

[tip:x86/cleanups] x86, msr: Convert printk to pr_foo()

Commit-ID: 951a18c6fee5d2f6f5d7b5118776bf787bf9c351 Gitweb: http://git.kernel.org/tip/951a18c6fee5d2f6f5d7b5118776bf787bf9c351 Author: Fabian Frederick AuthorDate: Fri, 17 Oct 2014 22:01:50 +0200 Committer: H. Peter Anvin CommitDate: Fri, 17 Oct 2014 13:40:54 -0700 x86, msr: Convert

[tip:x86/cleanups] x86, msr: Use PTR_ERR_OR_ZERO

Commit-ID: cba0fdbcff8b54c206c8e18898a55456959ed51a Gitweb: http://git.kernel.org/tip/cba0fdbcff8b54c206c8e18898a55456959ed51a Author: Fabian Frederick AuthorDate: Fri, 17 Oct 2014 22:01:38 +0200 Committer: H. Peter Anvin CommitDate: Fri, 17 Oct 2014 13:40:53 -0700 x86, msr: Use

[tip:x86/cleanups] x86/simplefb: Use PTR_ERR_OR_ZERO

Commit-ID: 76ef0db72f8b74bb92e24775d9d7f5ab26352bec Gitweb: http://git.kernel.org/tip/76ef0db72f8b74bb92e24775d9d7f5ab26352bec Author: Fabian Frederick AuthorDate: Fri, 17 Oct 2014 22:01:05 +0200 Committer: H. Peter Anvin CommitDate: Fri, 17 Oct 2014 13:40:52 -0700 x86/simplefb: Use

[tip:x86/cleanups] x86/sysfb: Use PTR_ERR_OR_ZERO

Commit-ID: e8d95ce9705c6683f57dc146b8c726c4d2242a52 Gitweb: http://git.kernel.org/tip/e8d95ce9705c6683f57dc146b8c726c4d2242a52 Author: Fabian Frederick AuthorDate: Fri, 17 Oct 2014 22:00:53 +0200 Committer: H. Peter Anvin CommitDate: Fri, 17 Oct 2014 13:40:52 -0700 x86/sysfb: Use

[tip:x86/cleanups] x86, cpuid: Use PTR_ERR_OR_ZERO

Commit-ID: cbda45a2d4798912637b5b792c1c7a97c8d8b080 Gitweb: http://git.kernel.org/tip/cbda45a2d4798912637b5b792c1c7a97c8d8b080 Author: Fabian Frederick AuthorDate: Fri, 17 Oct 2014 22:01:17 +0200 Committer: H. Peter Anvin CommitDate: Fri, 17 Oct 2014 13:40:51 -0700 x86, cpuid: Use

[PATCH 4/4 linux-next] x86, msr: use seek definitions instead of hard-coded values

Replace 0/1 by SEEK_SET/SEEK_CUR. Signed-off-by: Fabian Frederick --- arch/x86/kernel/msr.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/x86/kernel/msr.c b/arch/x86/kernel/msr.c index 1c66cd0..1ed1190 100644 --- a/arch/x86/kernel/msr.c +++ b/arch/x86/kernel/msr.c

[PATCH 3/4 linux-next] x86, msr: convert printk to pr_foo()

Also define pr_fmt. Signed-off-by: Fabian Frederick --- arch/x86/kernel/msr.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/arch/x86/kernel/msr.c b/arch/x86/kernel/msr.c index 5b14dac..1c66cd0 100644 --- a/arch/x86/kernel/msr.c +++ b/arch/x86/kernel/msr.c @@ -22,6

[PATCH 1/1 linux-next] x86, cpuid: use PTR_ERR_OR_ZERO

replace IS_ERR/PTR_ERR Signed-off-by: Fabian Frederick --- arch/x86/kernel/cpuid.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/x86/kernel/cpuid.c b/arch/x86/kernel/cpuid.c index 3225ae6c..83741a7 100644 --- a/arch/x86/kernel/cpuid.c +++ b/arch/x86/kernel/cpuid.c

[PATCH 1/4 linux-next] x86, msr: define msr chunksize

define MSR_CHUNK_SIZE instead of hard-coded value (8). Signed-off-by: Fabian Frederick --- arch/x86/include/asm/msr.h | 2 ++ arch/x86/kernel/msr.c | 18 +- 2 files changed, 11 insertions(+), 9 deletions(-) diff --git a/arch/x86/include/asm/msr.h b/arch/x86/include/asm

[PATCH 2/4 linux-next] x86, msr: use PTR_ERR_OR_ZERO

replace IS_ERR/PTR_ERR Signed-off-by: Fabian Frederick --- arch/x86/kernel/msr.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/x86/kernel/msr.c b/arch/x86/kernel/msr.c index f1f3ed0..5b14dac 100644 --- a/arch/x86/kernel/msr.c +++ b/arch/x86/kernel/msr.c @@ -206,7

[PATCH 1/1 linux-next] x86: sysfb: use PTR_ERR_OR_ZERO

replace IS_ERR/PTR_ERR Signed-off-by: Fabian Frederick --- arch/x86/kernel/sysfb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/x86/kernel/sysfb.c b/arch/x86/kernel/sysfb.c index 193ec2c..160386e 100644 --- a/arch/x86/kernel/sysfb.c +++ b/arch/x86/kernel/sysfb.c

[PATCH 1/1 linux-next] x86/simplefb: use PTR_ERR_OR_ZERO

replace IS_ERR/PTR_ERR Signed-off-by: Fabian Frederick --- arch/x86/kernel/sysfb_simplefb.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/arch/x86/kernel/sysfb_simplefb.c b/arch/x86/kernel/sysfb_simplefb.c index 86179d4..764a29f 100644 --- a/arch/x86/kernel

[PATCH 1/1 net-next] netrom: use linux/uaccess.h

replace asm/uaccess.h by linux/uaccess.h Signed-off-by: Fabian Frederick --- net/netrom/af_netrom.c | 2 +- net/netrom/nr_dev.c| 2 +- net/netrom/nr_in.c | 2 +- net/netrom/nr_out.c| 2 +- net/netrom/nr_route.c | 2 +- net/netrom/nr_subr.c | 2 +- net/netrom/nr_timer.c | 2 +- 7

Re: [PATCH 1/1 linux-next] zd1201: replace kmalloc/memset by kzalloc

> On 14 October 2014 at 20:08 Bjørn Mork wrote: > > > Joe Perches writes: > > > And does this really need to be alloc'd? > > Yes, it does. It is used as a transfer_buffer in usb_fill_bulk_urb() and > must be "suitable for DMA".  From include/linux/usb.h: > > /** >  * struct urb - USB Request Bl

Re: [PATCH 1/1 linux-next] fs/jffs2/nodelist.h: remove unnecessary __constant_ prefix

> On 14 October 2014 at 18:52 Joe Perches wrote: > > > On Tue, 2014-10-14 at 18:44 +0200, Fabian Frederick wrote: > > See commit fbdb8138cf0c > > ("checkpatch: warn on uses of __constant_ functions") > > Perhaps it'd be better to use the same > __

[PATCH 1/1 net-next] openvswitch: kerneldoc warning fix

s/sock/gs Signed-off-by: Fabian Frederick --- net/openvswitch/vport-geneve.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/openvswitch/vport-geneve.c b/net/openvswitch/vport-geneve.c index 910b3ef..106a9d8 100644 --- a/net/openvswitch/vport-geneve.c +++ b/net

[PATCH 1/1 net-next] NFC: netlink: use sizeof(*iter) instead of class_dev_iter

Also fixes kzalloc on struct se_io_ctx See Documentation/CodingStyle Chapter 14 Signed-off-by: Fabian Frederick --- net/nfc/netlink.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/net/nfc/netlink.c b/net/nfc/netlink.c index 43cb1c1..b4ba68e 100644 --- a/net/nfc

[PATCH 1/1 net-next] NFC: llcp: use sizeof(*sdres) instead of nfc_llcp_sdp_tlv

See Documentation/CodingStyle Chapter 14 Signed-off-by: Fabian Frederick --- net/nfc/llcp_commands.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/net/nfc/llcp_commands.c b/net/nfc/llcp_commands.c index a3ad69a..a450075 100644 --- a/net/nfc/llcp_commands.c +++ b/net

[PATCH 1/1 net-next] openvswitch: use vport instead of p

All functions used struct vport *vport except ovs_vport_find_upcall_portid. This fixes 1 kerneldoc warning Signed-off-by: Fabian Frederick --- net/openvswitch/vport.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/net/openvswitch/vport.c b/net/openvswitch/vport.c index

[PATCH 1/1 net-next] NFC: digital: use sizeof(*target) instead of nfc_target

See Documentation/CodingStyle Chapter 14 Signed-off-by: Fabian Frederick --- net/nfc/digital_technology.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/net/nfc/digital_technology.c b/net/nfc/digital_technology.c index fb58ed2d..5e94b6f 100644 --- a/net/nfc

[PATCH 1/1 net-next] NFC: llcp: remove bool comparison on device

Fixes coccinelle warning: net/nfc/llcp_core.c:132:5-11: WARNING: Comparison to bool Signed-off-by: Fabian Frederick --- net/nfc/llcp_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/nfc/llcp_core.c b/net/nfc/llcp_core.c index 51e7887..978399d 100644 --- a/net/nfc

[PATCH 1/1 net-next] NFC: digital: remove redundant memory message

Let MM subsystem display out of memory messages. Signed-off-by: Fabian Frederick --- net/nfc/digital_core.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/net/nfc/digital_core.c b/net/nfc/digital_core.c index 009bcf3..41ba70c 100644 --- a/net/nfc/digital_core.c +++ b/net

[PATCH 1/1 net-next] HCI: remove comparisons on shdlc boolean members.

/llc_shdlc.c:333:6-16: WARNING: Comparison to bool net/nfc/hci/llc_shdlc.c:431:31-43: WARNING: Comparison to bool Signed-off-by: Fabian Frederick --- net/nfc/hci/llc_shdlc.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/net/nfc/hci/llc_shdlc.c b/net/nfc/hci/llc_shdlc.c

Re: [RFC 1/1 linux-next] btrfs: don't opencode zero_user_segment

> On 14 October 2014 at 21:15 Zach Brown wrote: > > > On Tue, Oct 14, 2014 at 07:46:14PM +0200, Fabian Frederick wrote: > > use function defined in include/linux/highmem.h > > Note that this reverts 2 last function call order > > And adds a BUG_ON(PAGE_CACHE_SIZE &

[RFC 1/1 linux-next] btrfs: don't opencode zero_user_segment

use function defined in include/linux/highmem.h Note that this reverts 2 last function call order Signed-off-by: Fabian Frederick --- fs/btrfs/scrub.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/fs/btrfs/scrub.c b/fs/btrfs/scrub.c index efa0831..fa063cd 100644

[PATCH V2 2/2 net-next] caif_usb: use target structure member in memset

parent cfusbl was used instead of first structure member 'layer' Suggested-by: Joe Perches Signed-off-by: Fabian Frederick --- net/caif/caif_usb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/caif/caif_usb.c b/net/caif/caif_usb.c index 0e487b0..5cd44f0 10

[PATCH V2 1/2 net-next] caif_usb: remove redundant memory message

Let MM subsystem display out of memory messages. Signed-off-by: Fabian Frederick --- V2: add second patch with memset fix net/caif/caif_usb.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/net/caif/caif_usb.c b/net/caif/caif_usb.c index ba02db0..0e487b0 100644 --- a

[PATCH 1/1 linux-next] fs/jffs2/nodelist.h: remove unnecessary __constant_ prefix

See commit fbdb8138cf0c ("checkpatch: warn on uses of __constant_ functions") Cc: Joe Perches Cc: Andrew Morton Cc: David Woodhouse Cc: linux-...@lists.infradead.org Signed-off-by: Fabian Frederick --- fs/jffs2/nodelist.h | 8 1 file changed, 4 insertions(+), 4 deletion

[PATCH 1/1 linux-next] zd1201: replace kmalloc/memset by kzalloc

Signed-off-by: Fabian Frederick --- drivers/net/wireless/zd1201.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/net/wireless/zd1201.c b/drivers/net/wireless/zd1201.c index 6f5c793..4630b26 100644 --- a/drivers/net/wireless/zd1201.c +++ b/drivers/net/wireless

[PATCH V2 linux-next] fs/affs/amigaffs.c: use va_format instead of buffer/vnsprintf

-Remove ErrorBuffer and use %pV -Add __printf to enable argument mistmatch warnings Original-patch-by: Joe Perches Cc: Joe Perches Cc: Andrew Morton Signed-off-by: Fabian Frederick --- V2: add __printf (suggested by Joe Perches) fs/affs/affs.h | 2 ++ fs/affs/amigaffs.c | 28

[PATCH 1/1 linux-next] fs/affs/file.c: forward declaration clean-up

-Move file_operations to avoid forward declarations. -Remove unused declarations. Cc: Andrew Morton Signed-off-by: Fabian Frederick --- fs/affs/file.c | 38 -- 1 file changed, 16 insertions(+), 22 deletions(-) diff --git a/fs/affs/file.c b/fs/affs/file.c

[PATCH 1/1 linux-next] fs/affs/amigaffs.c: use va_format instead of buffer/vnsprintf

Remove ErrorBuffer and use %pV Cc: Joe Perches Cc: Andrew Morton Signed-off-by: Fabian Frederick --- fs/affs/amigaffs.c | 28 +--- 1 file changed, 13 insertions(+), 15 deletions(-) diff --git a/fs/affs/amigaffs.c b/fs/affs/amigaffs.c index abc8539..9643bdd 100644

[PATCH 1/1 net-next] caif_usb: remove redundant memory message

Let MM subsystem display out of memory messages. Signed-off-by: Fabian Frederick --- net/caif/caif_usb.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/net/caif/caif_usb.c b/net/caif/caif_usb.c index ba02db0..0e487b0 100644 --- a/net/caif/caif_usb.c +++ b/net/caif

[PATCH 1/1 net-next] caif: replace kmalloc/memset 0 by kzalloc

Also add blank line after declaration Signed-off-by: Fabian Frederick --- net/caif/cfmuxl.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/net/caif/cfmuxl.c b/net/caif/cfmuxl.c index 8c5d638..510aa5a 100644 --- a/net/caif/cfmuxl.c +++ b/net/caif/cfmuxl.c @@ -47,10

[PATCH 1/1 linux-next] scripts/checkpatch.pl: fix k[mz]alloc with multiplies check

checkpatch doesn't seem to detect any case since Commit e367455a9f25 ("checkpatch: emit fewer kmalloc_array/kcalloc conversion warnings") Cc: Joe Perches Cc: Andrew Morton Signed-off-by: Fabian Frederick --- scripts/checkpatch.pl | 4 ++-- 1 file changed, 2 insertions(

Re: [PATCH 1/1 linux-next] ceph: fix bool assignments

> On 10 October 2014 at 10:24 Ilya Dryomov wrote: > > > On Fri, Oct 10, 2014 at 4:41 AM, Dan Mick wrote: > > > > > > On 10/09/2014 02:16 PM, Fabian Frederick wrote: > >> Fix some coccinelle warnings: > >> fs/ceph/caps.c:2400:6-10: WARNING: Assign

[PATCH 1/1 linux-next] scripts/coccinelle/misc/boolinit.cocci: fix assignment warnings

Replace "Assignment of bool to 0/1" by "Assignment of 0/1 to bool" Suggested-by: Dan Mick Signed-off-by: Fabian Frederick --- scripts/coccinelle/misc/boolinit.cocci | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/scripts/coccinelle/misc/booli

[PATCH 1/1 linux-next] ceph: fix bool assignments

:3-7: WARNING: Assignment of bool to 0/1 fs/ceph/caps.c:2549:3-12: WARNING: Assignment of bool to 0/1 fs/ceph/caps.c:2575:2-6: WARNING: Assignment of bool to 0/1 fs/ceph/caps.c:2589:3-7: WARNING: Assignment of bool to 0/1 Signed-off-by: Fabian Frederick --- fs/ceph/caps.c | 26

[PATCH 1/1 linux-next] GFS2: directly return gfs2_dir_check()

No need to store gfs2_dir_check result and test it before returning. Signed-off-by: Fabian Frederick --- fs/gfs2/inode.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/fs/gfs2/inode.c b/fs/gfs2/inode.c index c4ed823..b41b5c7 100644 --- a/fs/gfs2/inode.c +++ b/fs/gfs2

[PATCH 1/1 net-next] mac80211: directly return ieee80211_vif_use_reserved_context()

No need to store ieee80211_vif_use_reserved_context result and test it before returning. Signed-off-by: Fabian Frederick --- net/mac80211/cfg.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/net/mac80211/cfg.c b/net/mac80211/cfg.c index fb6a150..c76b457 100644 --- a

[PATCH 1/1 linux-next] ceph: return error code directly.

Testing err before returning value is not needed. Signed-off-by: Fabian Frederick --- fs/ceph/ioctl.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/fs/ceph/ioctl.c b/fs/ceph/ioctl.c index a822a6e..b2d777a 100644 --- a/fs/ceph/ioctl.c +++ b/fs/ceph/ioctl.c @@ -54,10

[RFC 1/1 linux-next] fs/affs: only call remove_header on empty directory

Signed-off-by: Fabian Frederick --- fs/affs/affs.h | 1 + fs/affs/amigaffs.c | 18 +- fs/affs/namei.c| 11 +-- 3 files changed, 11 insertions(+), 19 deletions(-) diff --git a/fs/affs/affs.h b/fs/affs/affs.h index 9bca881..3a7f287 100644 --- a/fs/affs/affs.h +++ b

[PATCH 1/1 net-next] netlabel: directly return netlbl_unlabel_genl_init()

No need to store netlbl_unlabel_genl_init result and test it before returning. Signed-off-by: Fabian Frederick --- net/netlabel/netlabel_user.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/net/netlabel/netlabel_user.c b/net/netlabel/netlabel_user.c index 1e779bb

Re: [PATCH 1/1 net-next] af_unix: remove NULL assignment on static

> On 07 October 2014 at 22:54 Hannes Frederic Sowa > wrote: > > > On Di, 2014-10-07 at 22:49 +0200, Fabian Frederick wrote: > > > > > On 07 October 2014 at 22:33 Guenter Roeck wrote: > > > > > > > > > On Tue, Oct 07, 2014 at 04:18:32PM

[PATCH V2 net-next] af_unix: remove 0 assignment on static

static values are automatically initialized to 0 Signed-off-by: Fabian Frederick --- V2: replace NULL by 0 in description net/unix/garbage.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/unix/garbage.c b/net/unix/garbage.c index 9bc73f8..99f7012 100644 --- a/net/unix

Re: [PATCH 1/1 net-next] af_unix: remove NULL assignment on static

> On 07 October 2014 at 22:33 Guenter Roeck wrote: > > > On Tue, Oct 07, 2014 at 04:18:32PM -0400, David Miller wrote: > > From: Fabian Frederick > > Date: Tue,  7 Oct 2014 22:16:36 +0200 > > > > > static values are automatically initialized to NULL >

[PATCH 1/1 net-next] netlabel: kernel-doc warning fix

no secid argument in netlbl_cfg_unlbl_static_del Signed-off-by: Fabian Frederick --- net/netlabel/netlabel_kapi.c | 1 - 1 file changed, 1 deletion(-) diff --git a/net/netlabel/netlabel_kapi.c b/net/netlabel/netlabel_kapi.c index 0b4692d..a845cd4 100644 --- a/net/netlabel/netlabel_kapi.c +++ b

[PATCH 1/1 net-next] net: rfkill: kernel-doc warning fixes

s/state/blocked Signed-off-by: Fabian Frederick --- net/rfkill/core.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/net/rfkill/core.c b/net/rfkill/core.c index b3b16c0..fa7cd79 100644 --- a/net/rfkill/core.c +++ b/net/rfkill/core.c @@ -329,7 +329,7 @@ static atomic_t

[PATCH 1/1 net-next] af_unix: remove NULL assignment on static

static values are automatically initialized to NULL Signed-off-by: Fabian Frederick --- net/unix/garbage.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/unix/garbage.c b/net/unix/garbage.c index 9bc73f8..99f7012 100644 --- a/net/unix/garbage.c +++ b/net/unix/garbage.c

[PATCH 1/1 net-next] wimax: convert printk to pr_foo()

Use current logging functions and add module name prefix. Signed-off-by: Fabian Frederick --- Compiled but untested. net/wimax/id-table.c | 2 +- net/wimax/op-msg.c | 9 - net/wimax/op-reset.c | 3 +-- net/wimax/op-rfkill.c | 3 +-- net/wimax/op-state-get.c

Re: [PATCH 0/7 linux-next] drivers: remove deprecated IRQF_DISABLED

> On 06 October 2014 at 20:46 Borislav Petkov wrote: > > > On Mon, Oct 06, 2014 at 08:33:00PM +0200, Fabian Frederick wrote: > >    You're right, I guess we can forget this patchset. > > I didn't see it was already done. (nothing in linux-next yet). > >

Re: [PATCH 0/7 linux-next] drivers: remove deprecated IRQF_DISABLED

> On 06 October 2014 at 19:40 Borislav Petkov wrote: > > > On Mon, Oct 06, 2014 at 05:35:47PM +0200, Fabian Frederick wrote: > > This small patchset removes IRQF_DISABLED from drivers branch. > > > > See include/linux/interrupt.h: > > "This f

[PATCH V2 linux-next] net: fix rcu access on phonet_routes

:391:17: error: incompatible types in comparison expression (different address spaces) Suggested-by: Eric Dumazet Signed-off-by: Fabian Frederick --- V2: use rcu_access_pointer instead of rcu_dereference out of rcu_read_lock context (suggested by Eric Dumazet). net/phonet/pn_dev.c | 6

[PATCH 2/7 linux-next] ppc4xx_edac: remove deprecated IRQF_DISABLED

See include/linux/interrupt.h: "This flag is a NOOP and scheduled to be removed" Signed-off-by: Fabian Frederick --- drivers/edac/ppc4xx_edac.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/edac/ppc4xx_edac.c b/drivers/edac/ppc4xx_edac.c index 0f04d5

[PATCH 3/7 linux-next] tw68: remove deprecated IRQF_DISABLED

See include/linux/interrupt.h: "This flag is a NOOP and scheduled to be removed" Signed-off-by: Fabian Frederick --- drivers/media/pci/tw68/tw68-core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/pci/tw68/tw68-core.c b/drivers/media/pci/tw68/t

[PATCH 4/7 linux-next] cpqarray: remove deprecated IRQF_DISABLED

See include/linux/interrupt.h: "This flag is a NOOP and scheduled to be removed" Signed-off-by: Fabian Frederick --- drivers/block/cpqarray.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/block/cpqarray.c b/drivers/block/cpqarray.c index 2b94403..3707

[PATCH 5/7 linux-next] HSI: remove deprecated IRQF_DISABLED

See include/linux/interrupt.h: "This flag is a NOOP and scheduled to be removed" Signed-off-by: Fabian Frederick --- drivers/hsi/clients/nokia-modem.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/hsi/clients/nokia-modem.c b/drivers/hsi/clients/nok

[PATCH 7/7 linux-next] bus: omap_l3_smx: remove deprecated IRQF_DISABLED

See include/linux/interrupt.h: "This flag is a NOOP and scheduled to be removed" Signed-off-by: Fabian Frederick --- drivers/bus/omap_l3_smx.c | 10 -- 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/drivers/bus/omap_l3_smx.c b/drivers/bus/omap_l3_smx.c ind

[PATCH 6/7 linux-next] bus: omap_l3_noc: remove deprecated IRQF_DISABLED

See include/linux/interrupt.h: "This flag is a NOOP and scheduled to be removed" Signed-off-by: Fabian Frederick --- drivers/bus/omap_l3_noc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/bus/omap_l3_noc.c b/drivers/bus/omap_l3_noc.c index 531ae5

[PATCH 1/7 linux-next] mv64x60_edac: remove deprecated IRQF_DISABLED

See include/linux/interrupt.h: Signed-off-by: Fabian Frederick --- drivers/edac/mv64x60_edac.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/edac/mv64x60_edac.c b/drivers/edac/mv64x60_edac.c index 542fad7..6366e88 100644 --- a/drivers/edac/mv64x60_edac.c

[PATCH 0/7 linux-next] drivers: remove deprecated IRQF_DISABLED

This small patchset removes IRQF_DISABLED from drivers branch. See include/linux/interrupt.h: "This flag is a NOOP and scheduled to be removed" Note: (cross)compiled but untested. Fabian Frederick (7): mv64x60_edac: remove deprecated IRQF_DISABLED ppc4xx_edac: remove

[PATCH 13/13 linux-next] tty: ar933x_uart: use container_of to resolve ar933x_uart_port from uart_port

Use container_of instead of casting first structure member. Signed-off-by: Fabian Frederick --- drivers/tty/serial/ar933x_uart.c | 30 -- 1 file changed, 20 insertions(+), 10 deletions(-) diff --git a/drivers/tty/serial/ar933x_uart.c b/drivers/tty/serial

[PATCH 12/13 linux-next] serial: use container_of to resolve uart_sunzilog_port from uart_port

Use container_of instead of casting first structure member. Signed-off-by: Fabian Frederick --- drivers/tty/serial/sunzilog.c | 24 1 file changed, 16 insertions(+), 8 deletions(-) diff --git a/drivers/tty/serial/sunzilog.c b/drivers/tty/serial/sunzilog.c index 02df394

[PATCH 11/13 linux-next] serial: pnx8xxx: use container_of to resolve pnx8xxx_port from uart_port

Use container_of instead of casting first structure member. Signed-off-by: Fabian Frederick --- drivers/tty/serial/pnx8xxx_uart.c | 48 ++- 1 file changed, 32 insertions(+), 16 deletions(-) diff --git a/drivers/tty/serial/pnx8xxx_uart.c b/drivers/tty/serial

[PATCH 10/13 linux-next] serial: amba-pl010: use container_of to resolve uart_amba_port from uart_port

Use container_of instead of casting first structure member. Signed-off-by: Fabian Frederick --- drivers/tty/serial/amba-pl010.c | 36 1 file changed, 24 insertions(+), 12 deletions(-) diff --git a/drivers/tty/serial/amba-pl010.c b/drivers/tty/serial/amba

[PATCH 09/13 linux-next] serial: sunsab: use container_of to resolve uart_sunsu_port from uart_port

Use container_of instead of casting first structure member. Signed-off-by: Fabian Frederick --- drivers/tty/serial/sunsab.c | 36 1 file changed, 24 insertions(+), 12 deletions(-) diff --git a/drivers/tty/serial/sunsab.c b/drivers/tty/serial/sunsab.c index

[PATCH 08/13 linux-next] serial: sunsu: use container_of to resolve uart_sunsu_port from uart_port

Use container_of instead of casting first structure member. Signed-off-by: Fabian Frederick --- drivers/tty/serial/sunsu.c | 39 ++- 1 file changed, 26 insertions(+), 13 deletions(-) diff --git a/drivers/tty/serial/sunsu.c b/drivers/tty/serial/sunsu.c index

[PATCH 07/13 linux-next] tty: use container_of to resolve uart_pmac_port from uart_port

Use container_of instead of casting first structure member. Signed-off-by: Fabian Frederick --- drivers/tty/serial/pmac_zilog.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/drivers/tty/serial/pmac_zilog.c b/drivers/tty/serial/pmac_zilog.c index abbfedb..4aca322

[PATCH 06/13 linux-next] TTY: jsm: use container_of to resolve jsm_channel from uart_port

Use container_of instead of casting first structure member. Signed-off-by: Fabian Frederick --- drivers/tty/serial/jsm/jsm_tty.c | 30 -- 1 file changed, 20 insertions(+), 10 deletions(-) diff --git a/drivers/tty/serial/jsm/jsm_tty.c b/drivers/tty/serial/jsm

[PATCH 05/13 linux-next] serial: cpm_uart: use container_of to resolve uart_cpm_port from uart_port

Use container_of instead of casting first structure member. Signed-off-by: Fabian Frederick --- drivers/tty/serial/cpm_uart/cpm_uart_core.c | 48 +++-- 1 file changed, 32 insertions(+), 16 deletions(-) diff --git a/drivers/tty/serial/cpm_uart/cpm_uart_core.c b/drivers

[PATCH 04/13 linux-next] serial: mpsc: use container_of to resolve mpsc_port_info from uart_port

Use container_of instead of casting first structure member. Signed-off-by: Fabian Frederick --- drivers/tty/serial/mpsc.c | 39 ++- 1 file changed, 26 insertions(+), 13 deletions(-) diff --git a/drivers/tty/serial/mpsc.c b/drivers/tty/serial/mpsc.c index

[PATCH 03/13 linux-next] serial: use container_of to resolve uart_ip22zilog_port from uart_port

Use container_of instead of casting first structure member. Signed-off-by: Fabian Frederick --- drivers/tty/serial/ip22zilog.c | 18 -- 1 file changed, 12 insertions(+), 6 deletions(-) diff --git a/drivers/tty/serial/ip22zilog.c b/drivers/tty/serial/ip22zilog.c index 99b7b86

[PATCH 02/13 linux-next] serial: sa1100: use container_of to resolve sa1100_port from uart_port

Use container_of instead of casting first structure member. Signed-off-by: Fabian Frederick --- drivers/tty/serial/sa1100.c | 45 ++--- 1 file changed, 30 insertions(+), 15 deletions(-) diff --git a/drivers/tty/serial/sa1100.c b/drivers/tty/serial/sa1100

[PATCH 01/13 linux-next] serial: use container_of to resolve uart_sio_port from uart_port

Use container_of instead of casting first structure member. Signed-off-by: Fabian Frederick --- drivers/tty/serial/m32r_sio.c | 42 -- 1 file changed, 28 insertions(+), 14 deletions(-) diff --git a/drivers/tty/serial/m32r_sio.c b/drivers/tty/serial

[PATCH 00/13 linux-next] drivers/tty: use container_of where possible

Small patchset using container_of instead of casting on first structure member address. All patches (cross)compiled but untested. Fabian Frederick (13): serial: use container_of to resolve uart_sio_port from uart_port serial: sa1100: use container_of to resolve sa1100_port from uart_port

[RFC 1/1] net: fix rcu access on phonet_routes

:17: error: incompatible types in comparison expression (different address spaces) Signed-off-by: Fabian Frederick --- net/phonet/pn_dev.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/net/phonet/pn_dev.c b/net/phonet/pn_dev.c index 56a6146..5c9c0b2f1 100644 --- a/net

[PATCH 1/1] GFS2: use _RET_IP_ instead of (unsigned long)__builtin_return_address(0)

use macro definition Signed-off-by: Fabian Frederick --- fs/gfs2/glock.c | 4 ++-- fs/gfs2/glops.c | 2 +- fs/gfs2/trans.c | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/fs/gfs2/glock.c b/fs/gfs2/glock.c index 7f513b1..8f0c19d 100644 --- a/fs/gfs2/glock.c +++ b/fs/gfs2

[PATCH 1/1 net-next] cipso: add __init to cipso_v4_cache_init

cipso_v4_cache_init is only called by __init cipso_v4_init Signed-off-by: Fabian Frederick --- net/ipv4/cipso_ipv4.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/ipv4/cipso_ipv4.c b/net/ipv4/cipso_ipv4.c index 05b708b..4715f25 100644 --- a/net/ipv4/cipso_ipv4.c +++ b

[PATCH 1/1 net-next] inet: frags: add __init to ip4_frags_ctl_register

ip4_frags_ctl_register is only called by __init ipfrag_init Signed-off-by: Fabian Frederick --- net/ipv4/ip_fragment.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/net/ipv4/ip_fragment.c b/net/ipv4/ip_fragment.c index 15f0e2b..2811cc1 100644 --- a/net/ipv4

[PATCH 1/1 net-next] tcp: add __init to tcp_init_mem

tcp_init_mem is only called by __init tcp_init. Signed-off-by: Fabian Frederick --- net/ipv4/tcp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/ipv4/tcp.c b/net/ipv4/tcp.c index cf5e508..5c17034 100644 --- a/net/ipv4/tcp.c +++ b/net/ipv4/tcp.c @@ -3172,7 +3172,7

[PATCH 1/1 net-next] ieee802154: fix __init functions

Commit 3243acd37fd9 ("ieee802154: add __init to lowpan_frags_sysctl_register") added __init to lowpan_frags_ns_sysctl_register instead of lowpan_frags_sysctl_register Suggested-by: Alexander Aring Signed-off-by: Fabian Frederick --- net/ieee802154/reassembly.c | 4 ++-- 1 file

[PATCH 1/1 linux-next] net/dccp/ccid.c: add __init to ccid_activate

ccid_activate is only called by __init ccid_initialize_builtins in same module. Signed-off-by: Fabian Frederick --- net/dccp/ccid.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/dccp/ccid.c b/net/dccp/ccid.c index 5975572..8349897 100644 --- a/net/dccp/ccid.c +++ b/net

[PATCH 1/1 linux-next] net/dccp/proto.c: add __init to dccp_mib_init

dccp_mib_init is only called by __init dccp_init in same module. Signed-off-by: Fabian Frederick --- net/dccp/proto.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/dccp/proto.c b/net/dccp/proto.c index e421edd..6d090c4 100644 --- a/net/dccp/proto.c +++ b/net/dccp

Re: [PATCH 1/1 linux-next] ieee802154: add __init to lowpan_frags_sysctl_register

> On 01 October 2014 at 02:25 Alexander Aring wrote: > > > Hi, > > On Tue, Sep 30, 2014 at 10:34:08PM +0200, Fabian Frederick wrote: > > lowpan_frags_sysctl_register is only called by __init lowpan_net_frag_init > > (part of the lowpan module). > >

[PATCH 1/1 linux-next] ieee802154: add __init to lowpan_frags_sysctl_register

lowpan_frags_sysctl_register is only called by __init lowpan_net_frag_init (part of the lowpan module). Signed-off-by: Fabian Frederick --- This is untested. net/ieee802154/reassembly.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/net/ieee802154/reassembly.c b/net

<    3   4   5   6   7   8   9   10   11   12   >