[PATCH 3/9 net-next] af_key: replace if/BUG by BUG_ON

2015-03-30 Thread Fabian Frederick
Signed-off-by: Fabian Frederick --- net/key/af_key.c | 25 - 1 file changed, 8 insertions(+), 17 deletions(-) diff --git a/net/key/af_key.c b/net/key/af_key.c index 9255fd9..c77dcc9 100644 --- a/net/key/af_key.c +++ b/net/key/af_key.c @@ -889,8 +889,7 @@ static struct

[PATCH 9/9 net-next] netfilter: replace if/BUG by BUG_ON

2015-03-30 Thread Fabian Frederick
Signed-off-by: Fabian Frederick --- net/netfilter/nfnetlink_log.c | 3 +-- net/netfilter/xt_u32.c| 8 +++- 2 files changed, 4 insertions(+), 7 deletions(-) diff --git a/net/netfilter/nfnetlink_log.c b/net/netfilter/nfnetlink_log.c index 957b83a..409f5fb 100644 --- a/net/netfilter

[PATCH 7/9 net-next] sunrpc: replace if/BUG by BUG_ON

2015-03-30 Thread Fabian Frederick
Signed-off-by: Fabian Frederick --- net/sunrpc/auth_gss/svcauth_gss.c | 9 +++-- net/sunrpc/svc_xprt.c | 3 +-- 2 files changed, 4 insertions(+), 8 deletions(-) diff --git a/net/sunrpc/auth_gss/svcauth_gss.c b/net/sunrpc/auth_gss/svcauth_gss.c index 1095be9..09f8a1c6 100644

[PATCH 8/9 net-next] rxrpc: replace if/BUG by BUG_ON

2015-03-30 Thread Fabian Frederick
Signed-off-by: Fabian Frederick --- net/rxrpc/ar-accept.c | 6 ++ net/rxrpc/ar-ack.c | 6 ++ net/rxrpc/ar-call.c | 3 +-- net/rxrpc/ar-input.c| 6 ++ net/rxrpc/ar-internal.h | 3 +-- net/rxrpc/ar-key.c | 3 +-- net/rxrpc/ar-recvmsg.c | 16

[PATCH 6/9 net-next] af_packet: replace if/BUG by BUG_ON

2015-03-30 Thread Fabian Frederick
Signed-off-by: Fabian Frederick --- net/packet/af_packet.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/net/packet/af_packet.c b/net/packet/af_packet.c index 5102c3c..fb94c39 100644 --- a/net/packet/af_packet.c +++ b/net/packet/af_packet.c @@ -547,8 +547,7 @@ static void

[PATCH 5/9 net-next] xfrm: replace if/BUG by BUG_ON

2015-03-30 Thread Fabian Frederick
Signed-off-by: Fabian Frederick --- net/xfrm/xfrm_policy.c | 5 ++--- net/xfrm/xfrm_user.c | 27 +-- 2 files changed, 11 insertions(+), 21 deletions(-) diff --git a/net/xfrm/xfrm_policy.c b/net/xfrm/xfrm_policy.c index 638af06..15de248 100644 --- a/net/xfrm

[PATCH 2/9 net-next] net: core: replace if/BUG by BUG_ON

2015-03-30 Thread Fabian Frederick
Signed-off-by: Fabian Frederick --- net/core/rtnetlink.c | 3 +-- net/core/skbuff.c| 15 ++- 2 files changed, 7 insertions(+), 11 deletions(-) diff --git a/net/core/rtnetlink.c b/net/core/rtnetlink.c index b96ac21..ed496d3 100644 --- a/net/core/rtnetlink.c +++ b/net/core

[PATCH 4/9 net-next] ipv6: replace if/BUG by BUG_ON

2015-03-30 Thread Fabian Frederick
Signed-off-by: Fabian Frederick --- net/ipv6/addrconf.c | 3 +-- net/ipv6/esp6.c | 3 +-- net/ipv6/netfilter/nf_conntrack_reasm.c | 3 +-- 3 files changed, 3 insertions(+), 6 deletions(-) diff --git a/net/ipv6/addrconf.c b/net/ipv6/addrconf.c index

[PATCH 1/1 linux-next] fs/coda/psdev.c: return is not a function

2015-03-30 Thread Fabian Frederick
remove parentheses in returns. Signed-off-by: Fabian Frederick --- fs/coda/psdev.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/fs/coda/psdev.c b/fs/coda/psdev.c index 8226291..8914ec8 100644 --- a/fs/coda/psdev.c +++ b/fs/coda/psdev.c @@ -193,7 +193,7 @@ static

[PATCH 3/3 linux-next] befs: replace typedef befs_inode_info by structure

2015-03-30 Thread Fabian Frederick
See Documentation/CodingStyle Cc: Andrew Morton Signed-off-by: Fabian Frederick --- fs/befs/befs.h | 7 +++ fs/befs/linuxvfs.c | 7 --- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/fs/befs/befs.h b/fs/befs/befs.h index 11461ab..1fead8d 100644 --- a/fs/befs/befs.h

[PATCH 2/3 linux-next] befs: replace typedef befs_sb_info by structure

2015-03-30 Thread Fabian Frederick
See Documenation/CodingStyle Cc: Andrew Morton Signed-off-by: Fabian Frederick --- fs/befs/befs.h | 9 - fs/befs/datastream.c | 4 ++-- fs/befs/io.c | 2 +- fs/befs/linuxvfs.c | 5 ++--- fs/befs/super.c | 4 ++-- 5 files changed, 11 insertions(+), 13 deletions

[PATCH 1/3 linux-next] befs: replace typedef befs_mount_options by structure

2015-03-30 Thread Fabian Frederick
See Documentation/CodingStyle Cc: Andrew Morton Signed-off-by: Fabian Frederick --- fs/befs/befs.h | 6 +++--- fs/befs/linuxvfs.c | 4 ++-- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/fs/befs/befs.h b/fs/befs/befs.h index 3a7813a..d4751af 100644 --- a/fs/befs/befs.h

[RFC 1/1 linux-next] checkpatch: don't ask for asm/file.h to linux/file.h unconditionally

2015-03-30 Thread Fabian Frederick
Currently checkpatch warns when asm/file.h is included and linux/file.h exists. That conversion can be made when linux/file.h includes asm/file.h which is not always the case.(See signal.h) Signed-off-by: Fabian Frederick --- scripts/checkpatch.pl | 7 +-- 1 file changed, 5 insertions(+), 2

[RFC 1/1 linux-next] Btrfs: avoid using NULL compressed_pages in insert_inline_extent()

2015-03-30 Thread Fabian Frederick
insert_inline_extent() checked for compressed_pages to be NULL then it accessed it under compress_type != BTRFS_COMPRESS_NONE. This patch adds BUG() when compress_size != 0, compress_type != BTRFS_COMPRESS_NONE and compresses_pages == 0. Signed-off-by: Fabian Frederick --- fs/btrfs/inode.c | 8

[PATCH 1/1 linux-next] Btrfs: use BTRFS_COMPRESS_NONE instead of 0

2015-03-30 Thread Fabian Frederick
cow_file_range_inline() was called with 0 instead of actual definition. Signed-off-by: Fabian Frederick --- fs/btrfs/inode.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/fs/btrfs/inode.c b/fs/btrfs/inode.c index 686331f..f626519 100644 --- a/fs/btrfs/inode.c +++ b

[PATCH 4/5 linux-next] fotg210-hcd: remove unneccessary message level.

2015-03-30 Thread Fabian Frederick
KERN_WARNING is implicitely declared in pr_warn() Signed-off-by: Fabian Frederick --- drivers/usb/host/fotg210-hcd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/host/fotg210-hcd.c b/drivers/usb/host/fotg210-hcd.c index 475b21f..98b0d05 100644 --- a/drivers

[PATCH 5/5 linux-next] tpm_ibmvtpm: remove unneccessary message level.

2015-03-30 Thread Fabian Frederick
KERN_ERR is implicitely declared in pr_err() Signed-off-by: Fabian Frederick --- drivers/char/tpm/tpm_of.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/char/tpm/tpm_of.c b/drivers/char/tpm/tpm_of.c index c002d1b..eebe625 100644 --- a/drivers/char/tpm/tpm_of.c

[PATCH 3/5 linux-next] IB/mlx4: remove unneccessary message level.

2015-03-30 Thread Fabian Frederick
KERN_WARNING is implicitely declared in pr_warn() Signed-off-by: Fabian Frederick --- drivers/infiniband/hw/mlx4/main.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/infiniband/hw/mlx4/main.c b/drivers/infiniband/hw/mlx4/main.c index b972c0b..1298fe8 100644

[PATCH 2/5 linux-next] iw_cxgb4: remove unneccessary message level.

2015-03-30 Thread Fabian Frederick
KERN_ERR is implicitely declared in pr_err() Signed-off-by: Fabian Frederick --- drivers/infiniband/hw/cxgb4/device.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/infiniband/hw/cxgb4/device.c b/drivers/infiniband/hw/cxgb4/device.c index 8fb295e..59546b6

[PATCH 1/5 linux-next] IB/mlx5: remove unneccessary message level.

2015-03-30 Thread Fabian Frederick
KERN_ERR is implicitely declared in pr_err() Signed-off-by: Fabian Frederick --- drivers/infiniband/hw/mlx5/qp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/infiniband/hw/mlx5/qp.c b/drivers/infiniband/hw/mlx5/qp.c index be0cd35..e5b4253 100644 --- a/drivers

[RFC 1/1 linux-next] Btrfs: avoid using NULL compressed_pages in insert_inline_extent()

2015-03-30 Thread Fabian Frederick
insert_inline_extent() checked for compressed_pages to be NULL then it accessed it under compress_type != BTRFS_COMPRESS_NONE. This patch adds BUG() when compress_size != 0, compress_type != BTRFS_COMPRESS_NONE and compresses_pages == 0. Signed-off-by: Fabian Frederick f...@skynet.be --- fs

[RFC 1/1 linux-next] checkpatch: don't ask for asm/file.h to linux/file.h unconditionally

2015-03-30 Thread Fabian Frederick
Currently checkpatch warns when asm/file.h is included and linux/file.h exists. That conversion can be made when linux/file.h includes asm/file.h which is not always the case.(See signal.h) Signed-off-by: Fabian Frederick f...@skynet.be --- scripts/checkpatch.pl | 7 +-- 1 file changed, 5

[PATCH 1/5 linux-next] IB/mlx5: remove unneccessary message level.

2015-03-30 Thread Fabian Frederick
KERN_ERR is implicitely declared in pr_err() Signed-off-by: Fabian Frederick f...@skynet.be --- drivers/infiniband/hw/mlx5/qp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/infiniband/hw/mlx5/qp.c b/drivers/infiniband/hw/mlx5/qp.c index be0cd35..e5b4253 100644

[PATCH 1/1 linux-next] Btrfs: use BTRFS_COMPRESS_NONE instead of 0

2015-03-30 Thread Fabian Frederick
cow_file_range_inline() was called with 0 instead of actual definition. Signed-off-by: Fabian Frederick f...@skynet.be --- fs/btrfs/inode.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/fs/btrfs/inode.c b/fs/btrfs/inode.c index 686331f..f626519 100644 --- a/fs/btrfs

[PATCH 2/5 linux-next] iw_cxgb4: remove unneccessary message level.

2015-03-30 Thread Fabian Frederick
KERN_ERR is implicitely declared in pr_err() Signed-off-by: Fabian Frederick f...@skynet.be --- drivers/infiniband/hw/cxgb4/device.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/infiniband/hw/cxgb4/device.c b/drivers/infiniband/hw/cxgb4/device.c index 8fb295e

[PATCH 3/5 linux-next] IB/mlx4: remove unneccessary message level.

2015-03-30 Thread Fabian Frederick
KERN_WARNING is implicitely declared in pr_warn() Signed-off-by: Fabian Frederick f...@skynet.be --- drivers/infiniband/hw/mlx4/main.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/infiniband/hw/mlx4/main.c b/drivers/infiniband/hw/mlx4/main.c index b972c0b

[PATCH 4/5 linux-next] fotg210-hcd: remove unneccessary message level.

2015-03-30 Thread Fabian Frederick
KERN_WARNING is implicitely declared in pr_warn() Signed-off-by: Fabian Frederick f...@skynet.be --- drivers/usb/host/fotg210-hcd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/host/fotg210-hcd.c b/drivers/usb/host/fotg210-hcd.c index 475b21f..98b0d05 100644

[PATCH 5/5 linux-next] tpm_ibmvtpm: remove unneccessary message level.

2015-03-30 Thread Fabian Frederick
KERN_ERR is implicitely declared in pr_err() Signed-off-by: Fabian Frederick f...@skynet.be --- drivers/char/tpm/tpm_of.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/char/tpm/tpm_of.c b/drivers/char/tpm/tpm_of.c index c002d1b..eebe625 100644 --- a/drivers/char/tpm

[PATCH 2/3 linux-next] befs: replace typedef befs_sb_info by structure

2015-03-30 Thread Fabian Frederick
See Documenation/CodingStyle Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- fs/befs/befs.h | 9 - fs/befs/datastream.c | 4 ++-- fs/befs/io.c | 2 +- fs/befs/linuxvfs.c | 5 ++--- fs/befs/super.c | 4 ++-- 5 files

[PATCH 1/3 linux-next] befs: replace typedef befs_mount_options by structure

2015-03-30 Thread Fabian Frederick
See Documentation/CodingStyle Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- fs/befs/befs.h | 6 +++--- fs/befs/linuxvfs.c | 4 ++-- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/fs/befs/befs.h b/fs/befs/befs.h index 3a7813a

[PATCH 3/9 net-next] af_key: replace if/BUG by BUG_ON

2015-03-30 Thread Fabian Frederick
Signed-off-by: Fabian Frederick f...@skynet.be --- net/key/af_key.c | 25 - 1 file changed, 8 insertions(+), 17 deletions(-) diff --git a/net/key/af_key.c b/net/key/af_key.c index 9255fd9..c77dcc9 100644 --- a/net/key/af_key.c +++ b/net/key/af_key.c @@ -889,8 +889,7

[PATCH 3/3 linux-next] befs: replace typedef befs_inode_info by structure

2015-03-30 Thread Fabian Frederick
See Documentation/CodingStyle Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- fs/befs/befs.h | 7 +++ fs/befs/linuxvfs.c | 7 --- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/fs/befs/befs.h b/fs/befs/befs.h index

[PATCH 1/9 net-next] ipv4: replace if/BUG by BUG_ON

2015-03-30 Thread Fabian Frederick
Signed-off-by: Fabian Frederick f...@skynet.be --- net/ipv4/devinet.c | 3 +-- net/ipv4/esp4.c | 3 +-- net/ipv4/icmp.c | 3 +-- net/ipv4/ip_output.c | 4 ++-- net/ipv4/ping.c | 3 +-- net/ipv4/tcp_input.c | 8 net/ipv4/tcp_minisocks.c | 4

[PATCH 6/9 net-next] af_packet: replace if/BUG by BUG_ON

2015-03-30 Thread Fabian Frederick
Signed-off-by: Fabian Frederick f...@skynet.be --- net/packet/af_packet.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/net/packet/af_packet.c b/net/packet/af_packet.c index 5102c3c..fb94c39 100644 --- a/net/packet/af_packet.c +++ b/net/packet/af_packet.c @@ -547,8 +547,7

[PATCH 5/9 net-next] xfrm: replace if/BUG by BUG_ON

2015-03-30 Thread Fabian Frederick
Signed-off-by: Fabian Frederick f...@skynet.be --- net/xfrm/xfrm_policy.c | 5 ++--- net/xfrm/xfrm_user.c | 27 +-- 2 files changed, 11 insertions(+), 21 deletions(-) diff --git a/net/xfrm/xfrm_policy.c b/net/xfrm/xfrm_policy.c index 638af06..15de248 100644 --- a/net

[PATCH 2/9 net-next] net: core: replace if/BUG by BUG_ON

2015-03-30 Thread Fabian Frederick
Signed-off-by: Fabian Frederick f...@skynet.be --- net/core/rtnetlink.c | 3 +-- net/core/skbuff.c| 15 ++- 2 files changed, 7 insertions(+), 11 deletions(-) diff --git a/net/core/rtnetlink.c b/net/core/rtnetlink.c index b96ac21..ed496d3 100644 --- a/net/core/rtnetlink.c +++ b

[PATCH 4/9 net-next] ipv6: replace if/BUG by BUG_ON

2015-03-30 Thread Fabian Frederick
Signed-off-by: Fabian Frederick f...@skynet.be --- net/ipv6/addrconf.c | 3 +-- net/ipv6/esp6.c | 3 +-- net/ipv6/netfilter/nf_conntrack_reasm.c | 3 +-- 3 files changed, 3 insertions(+), 6 deletions(-) diff --git a/net/ipv6/addrconf.c b/net/ipv6

[PATCH 1/1 linux-next] fs/coda/psdev.c: return is not a function

2015-03-30 Thread Fabian Frederick
remove parentheses in returns. Signed-off-by: Fabian Frederick f...@skynet.be --- fs/coda/psdev.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/fs/coda/psdev.c b/fs/coda/psdev.c index 8226291..8914ec8 100644 --- a/fs/coda/psdev.c +++ b/fs/coda/psdev.c @@ -193,7 +193,7

[PATCH 7/9 net-next] sunrpc: replace if/BUG by BUG_ON

2015-03-30 Thread Fabian Frederick
Signed-off-by: Fabian Frederick f...@skynet.be --- net/sunrpc/auth_gss/svcauth_gss.c | 9 +++-- net/sunrpc/svc_xprt.c | 3 +-- 2 files changed, 4 insertions(+), 8 deletions(-) diff --git a/net/sunrpc/auth_gss/svcauth_gss.c b/net/sunrpc/auth_gss/svcauth_gss.c index 1095be9

[PATCH 8/9 net-next] rxrpc: replace if/BUG by BUG_ON

2015-03-30 Thread Fabian Frederick
Signed-off-by: Fabian Frederick f...@skynet.be --- net/rxrpc/ar-accept.c | 6 ++ net/rxrpc/ar-ack.c | 6 ++ net/rxrpc/ar-call.c | 3 +-- net/rxrpc/ar-input.c| 6 ++ net/rxrpc/ar-internal.h | 3 +-- net/rxrpc/ar-key.c | 3 +-- net/rxrpc/ar-recvmsg.c | 16

[PATCH 9/9 net-next] netfilter: replace if/BUG by BUG_ON

2015-03-30 Thread Fabian Frederick
Signed-off-by: Fabian Frederick f...@skynet.be --- net/netfilter/nfnetlink_log.c | 3 +-- net/netfilter/xt_u32.c| 8 +++- 2 files changed, 4 insertions(+), 7 deletions(-) diff --git a/net/netfilter/nfnetlink_log.c b/net/netfilter/nfnetlink_log.c index 957b83a..409f5fb 100644

Re: [PATCH 32/35 linux-next] clk: constify of_device_id array

2015-03-22 Thread Fabian Frederick
> On 18 March 2015 at 15:15 Michael Turquette wrote: > > > Quoting Fabian Frederick (2015-03-16 12:59:06) > > of_device_id is always used as const. > > (See driver.of_match_table and open firmware functions) > > > > Signed-off-by: Fabian Frederick >

Re: [PATCH 1/1 linux-next] fanotify: fix a lock in fanotify_should_send_event()

2015-03-22 Thread Fabian Frederick
> On 21 March 2015 at 02:15 Lino Sanfilippo wrote: > > > On 21.03.2015 02:01, Lino Sanfilippo wrote: > > >> Should that be (marks_mask & FS_ISDIR & marks_ignored_mask)? > >> > > > > No, the current logic should be correct, since we want events for > > directories if we have FS_ISDIR set in the

Re: jbd2: fix incorrect unlock on j_list_lock

2015-03-22 Thread Fabian Frederick
That solution has already been submitted: http://marc.info/?l=linux-kernel=141106500709590=2 and refused here: http://marc.info/?l=linux-kernel=141107310613330=2 Regards, Fabian -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to

Re: jbd2: fix incorrect unlock on j_list_lock

2015-03-22 Thread Fabian Frederick
That solution has already been submitted: http://marc.info/?l=linux-kernelm=141106500709590w=2 and refused here: http://marc.info/?l=linux-kernelm=141107310613330w=2 Regards, Fabian -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to

Re: [PATCH 1/1 linux-next] fanotify: fix a lock in fanotify_should_send_event()

2015-03-22 Thread Fabian Frederick
On 21 March 2015 at 02:15 Lino Sanfilippo linosanfili...@gmx.de wrote: On 21.03.2015 02:01, Lino Sanfilippo wrote: Should that be (marks_mask FS_ISDIR marks_ignored_mask)? No, the current logic should be correct, since we want events for directories if we have FS_ISDIR set in

Re: [PATCH 32/35 linux-next] clk: constify of_device_id array

2015-03-22 Thread Fabian Frederick
On 18 March 2015 at 15:15 Michael Turquette mturque...@linaro.org wrote: Quoting Fabian Frederick (2015-03-16 12:59:06) of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick f...@skynet.be Acked-by: Michael

Re: [PATCH 33/35 linux-next] mmc: constify of_device_id array

2015-03-21 Thread Fabian Frederick
> On 20 March 2015 at 09:56 Ulf Hansson wrote: > > > On 17 March 2015 at 10:49, Ulf Hansson wrote: > > On 16 March 2015 at 20:59, Fabian Frederick wrote: > >> of_device_id is always used as const. > >> (See driver.of_match_table and open firmware functi

Re: [PATCH 1/1] fs/affs/file.c: unlock/release page on error

2015-03-21 Thread Fabian Frederick
> On 21 March 2015 at 18:31 Taesoo Kim wrote: > > > When affs_bread_ino() fails, correctly unlock the page and > release the page cache with proper error value. All write_end() > should unlock/release the page that was locked by write_beg(). > > Signed-off-by: Taesoo Kim > --- > 

Re: [PATCH 1/1] fs/affs/file.c: unlock/release page on error

2015-03-21 Thread Fabian Frederick
On 21 March 2015 at 18:31 Taesoo Kim tsgat...@gmail.com wrote: When affs_bread_ino() fails, correctly unlock the page and release the page cache with proper error value. All write_end() should unlock/release the page that was locked by write_beg(). Signed-off-by: Taesoo Kim

Re: [PATCH 33/35 linux-next] mmc: constify of_device_id array

2015-03-21 Thread Fabian Frederick
On 20 March 2015 at 09:56 Ulf Hansson ulf.hans...@linaro.org wrote: On 17 March 2015 at 10:49, Ulf Hansson ulf.hans...@linaro.org wrote: On 16 March 2015 at 20:59, Fabian Frederick f...@skynet.be wrote: of_device_id is always used as const. (See driver.of_match_table and open firmware

Re: [PATCH 1/1 linux-next] fanotify: fix a lock in fanotify_should_send_event()

2015-03-20 Thread Fabian Frederick
> On 20 March 2015 at 22:09 Andrew Morton wrote: > > > On Fri, 20 Mar 2015 21:56:08 +0100 Fabian Frederick wrote: > > > ltp/fanotify02 was locked since commit 66ba93c0d7fe > > ("fanotify: don't set FAN_ONDIR implicitly on a marks ignored mask") > >

Re: [PATCH 1/1] fs/affs/file.c: unlock/release page on error

2015-03-20 Thread Fabian Frederick
> On 20 March 2015 at 04:19 Taesoo Kim wrote: > > > When affs_bread_ino() fails, correctly unlock the page and > release the page cache with proper error value. All write_end() > should unlock/release the page that was locked by write_beg(). > > Signed-off-by: Taesoo Kim > --- > 

[PATCH 1/1 linux-next] fanotify: fix a lock in fanotify_should_send_event()

2015-03-20 Thread Fabian Frederick
ltp/fanotify02 was locked since commit 66ba93c0d7fe ("fanotify: don't set FAN_ONDIR implicitly on a marks ignored mask") Signed-off-by: Fabian Frederick --- fs/notify/fanotify/fanotify.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/fs/notify/fanotify/fanot

[PATCH 1/1 linux-next] fanotify: fix a lock in fanotify_should_send_event()

2015-03-20 Thread Fabian Frederick
ltp/fanotify02 was locked since commit 66ba93c0d7fe (fanotify: don't set FAN_ONDIR implicitly on a marks ignored mask) Signed-off-by: Fabian Frederick f...@skynet.be --- fs/notify/fanotify/fanotify.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/fs/notify/fanotify

Re: [PATCH 1/1] fs/affs/file.c: unlock/release page on error

2015-03-20 Thread Fabian Frederick
On 20 March 2015 at 04:19 Taesoo Kim tsgat...@gmail.com wrote: When affs_bread_ino() fails, correctly unlock the page and release the page cache with proper error value. All write_end() should unlock/release the page that was locked by write_beg(). Signed-off-by: Taesoo Kim

Re: [PATCH 1/1 linux-next] fanotify: fix a lock in fanotify_should_send_event()

2015-03-20 Thread Fabian Frederick
On 20 March 2015 at 22:09 Andrew Morton a...@linux-foundation.org wrote: On Fri, 20 Mar 2015 21:56:08 +0100 Fabian Frederick f...@skynet.be wrote: ltp/fanotify02 was locked since commit 66ba93c0d7fe (fanotify: don't set FAN_ONDIR implicitly on a marks ignored mask) What does ltp

Re: [PATCH 35/35 linux-next] pinctrl: constify of_device_id array

2015-03-19 Thread Fabian Frederick
> On 19 March 2015 at 02:58 Hongzhou Yang wrote: > > > On Mon, 2015-03-16 at 20:59 +0100, Fabian Frederick wrote: > > of_device_id is always used as const. > > (See driver.of_match_table and open firmware functions) > > > > Signed-off-by: Fabian Frederi

Re: [PATCH 35/35 linux-next] pinctrl: constify of_device_id array

2015-03-19 Thread Fabian Frederick
On 19 March 2015 at 02:58 Hongzhou Yang hongzhou.y...@mediatek.com wrote: On Mon, 2015-03-16 at 20:59 +0100, Fabian Frederick wrote: of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick f...@skynet.be

[PATCH 1/7 linux-next] ALSA: aoa: constify of_device_id array

2015-03-18 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick --- sound/aoa/soundbus/i2sbus/core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/aoa/soundbus/i2sbus/core.c b/sound/aoa/soundbus/i2sbus

[PATCH 7/7 linux-next] ASoC: rsnd: constify of_device_id array

2015-03-18 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick --- sound/soc/sh/rcar/core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/soc/sh/rcar/core.c b/sound/soc/sh/rcar/core.c index be03474

[PATCH 6/7 linux-next] ASoC: fsi: constify of_device_id array

2015-03-18 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick --- sound/soc/sh/fsi.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sound/soc/sh/fsi.c b/sound/soc/sh/fsi.c index bb20550..b593e28 100644

[PATCH 4/7 linux-next] ASoC: rt5631: constify of_device_id array

2015-03-18 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick --- sound/soc/codecs/rt5631.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/soc/codecs/rt5631.c b/sound/soc/codecs/rt5631.c index c618527

[PATCH 3/7 linux-next] ASoC: kirkwood: constify of_device_id array

2015-03-18 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick --- sound/soc/kirkwood/kirkwood-i2s.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/soc/kirkwood/kirkwood-i2s.c b/sound/soc/kirkwood

[PATCH 2/7 linux-next] ASoC: fsl: constify of_device_id array

2015-03-18 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick --- sound/soc/fsl/mpc5200_psc_ac97.c| 2 +- sound/soc/fsl/mpc5200_psc_i2s.c | 2 +- sound/soc/fsl/pcm030-audio-fabric.c | 2 +- 3 files changed, 3 insertions

[PATCH 5/7 linux-next] ASoC: ak4554: constify of_device_id array

2015-03-18 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick --- sound/soc/codecs/ak4554.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/soc/codecs/ak4554.c b/sound/soc/codecs/ak4554.c index 16ce9f9

[PATCH 0/7 linux-next] sound: constify of_device_id array

2015-03-18 Thread Fabian Frederick
This small patchset adds const to of_device_id arrays in sound branch. Fabian Frederick (7): ALSA: aoa: constify of_device_id array ASoC: fsl: constify of_device_id array ASoC: kirkwood: constify of_device_id array ASoC: rt5631: constify of_device_id array ASoC: ak4554: constify

[PATCH 0/7 linux-next] sound: constify of_device_id array

2015-03-18 Thread Fabian Frederick
This small patchset adds const to of_device_id arrays in sound branch. Fabian Frederick (7): ALSA: aoa: constify of_device_id array ASoC: fsl: constify of_device_id array ASoC: kirkwood: constify of_device_id array ASoC: rt5631: constify of_device_id array ASoC: ak4554: constify

[PATCH 5/7 linux-next] ASoC: ak4554: constify of_device_id array

2015-03-18 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick f...@skynet.be --- sound/soc/codecs/ak4554.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/soc/codecs/ak4554.c b/sound/soc/codecs/ak4554.c

[PATCH 2/7 linux-next] ASoC: fsl: constify of_device_id array

2015-03-18 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick f...@skynet.be --- sound/soc/fsl/mpc5200_psc_ac97.c| 2 +- sound/soc/fsl/mpc5200_psc_i2s.c | 2 +- sound/soc/fsl/pcm030-audio-fabric.c | 2 +- 3 files changed, 3

[PATCH 4/7 linux-next] ASoC: rt5631: constify of_device_id array

2015-03-18 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick f...@skynet.be --- sound/soc/codecs/rt5631.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/soc/codecs/rt5631.c b/sound/soc/codecs/rt5631.c

[PATCH 6/7 linux-next] ASoC: fsi: constify of_device_id array

2015-03-18 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick f...@skynet.be --- sound/soc/sh/fsi.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sound/soc/sh/fsi.c b/sound/soc/sh/fsi.c index bb20550

[PATCH 3/7 linux-next] ASoC: kirkwood: constify of_device_id array

2015-03-18 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick f...@skynet.be --- sound/soc/kirkwood/kirkwood-i2s.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/soc/kirkwood/kirkwood-i2s.c b/sound/soc

[PATCH 7/7 linux-next] ASoC: rsnd: constify of_device_id array

2015-03-18 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick f...@skynet.be --- sound/soc/sh/rcar/core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/soc/sh/rcar/core.c b/sound/soc/sh/rcar/core.c index

[PATCH 1/7 linux-next] ALSA: aoa: constify of_device_id array

2015-03-18 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick f...@skynet.be --- sound/aoa/soundbus/i2sbus/core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/aoa/soundbus/i2sbus/core.c b/sound/aoa

Re: [PATCH 01/35 linux-next] ata: constify of_device_id array

2015-03-17 Thread Fabian Frederick
> On 16 March 2015 at 23:16 Richard Weinberger > wrote: > > > Fabian, > > On Mon, Mar 16, 2015 at 8:17 PM, Fabian Frederick wrote: > > of_device_id is always used as const. > > (See driver.of_match_table and open firmware functions) > > Do you know the

[PATCH 11/16 linux-next] net: phy: constify of_device_id array

2015-03-17 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick --- drivers/net/phy/mdio-bcm-unimac.c | 2 +- drivers/net/phy/mdio-gpio.c| 2 +- drivers/net/phy/mdio-mux-gpio.c| 2 +- drivers/net/phy/mdio-mux-mmioreg.c

[PATCH 10/16 linux-next] orinoco: constify of_device_id array

2015-03-17 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick --- drivers/net/wireless/orinoco/airport.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/wireless/orinoco/airport.c b/drivers/net

[PATCH 02/16 linux-next] Altera TSE: constify of_device_id array

2015-03-17 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick --- drivers/net/ethernet/altera/altera_tse_main.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/ethernet/altera/altera_tse_main.c

[PATCH 00/16 linux-next] drivers/net: constify of_device_id array

2015-03-17 Thread Fabian Frederick
This small patchset adds const to of_device_id arrays in drivers/net branch. Fabian Frederick (16): net: netcp: constify of_device_id array Altera TSE: constify of_device_id array net/fsl: constify of_device_id array net: ethoc: constify of_device_id array drivers: net: xgene: constify

[PATCH 12/16 linux-next] can: constify of_device_id array

2015-03-17 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick --- drivers/net/can/cc770/cc770_platform.c | 2 +- drivers/net/can/grcan.c| 2 +- drivers/net/can/mscan/mpc5xxx_can.c| 2 +- drivers/net

[PATCH 13/16 linux-next] IBM-EMAC: constify of_device_id array

2015-03-17 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick --- drivers/net/ethernet/ibm/emac/core.c | 2 +- drivers/net/ethernet/ibm/emac/mal.c | 2 +- drivers/net/ethernet/ibm/emac/rgmii.c | 2 +- drivers/net/ethernet/ibm

[PATCH 04/16 linux-next] net: ethoc: constify of_device_id array

2015-03-17 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick --- drivers/net/ethernet/ethoc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/ethoc.c b/drivers/net/ethernet/ethoc.c index

[PATCH 05/16 linux-next] drivers: net: xgene: constify of_device_id array

2015-03-17 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick --- drivers/net/ethernet/apm/xgene/xgene_enet_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/apm/xgene/xgene_enet_main.c

[PATCH 03/16 linux-next] net/fsl: constify of_device_id array

2015-03-17 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick --- drivers/net/ethernet/freescale/fec_mpc52xx.c | 2 +- drivers/net/ethernet/freescale/fec_mpc52xx_phy.c | 2 +- drivers/net/ethernet/freescale/fs_enet

[PATCH 15/16 linux-next] net: via-rhine: constify of_device_id array

2015-03-17 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick --- drivers/net/ethernet/via/via-rhine.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/via/via-rhine.c b/drivers/net/ethernet

[PATCH 16/16 linux-next] via-velocity: constify of_device_id array

2015-03-17 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick --- drivers/net/ethernet/via/via-velocity.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/via/via-velocity.c b/drivers/net

[PATCH 09/16 linux-next] net: xilinx: constify of_device_id array

2015-03-17 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick --- drivers/net/ethernet/xilinx/ll_temac_main.c | 2 +- drivers/net/ethernet/xilinx/xilinx_axienet_main.c | 2 +- drivers/net/ethernet/xilinx/xilinx_emaclite.c

[PATCH 14/16 linux-next] ehea: constify of_device_id array

2015-03-17 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick --- drivers/net/ethernet/ibm/ehea/ehea_main.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/ethernet/ibm/ehea/ehea_main.c b

[PATCH 07/16 linux-next] netdev: octeon_mgmt: constify of_device_id array

2015-03-17 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick --- drivers/net/ethernet/octeon/octeon_mgmt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/octeon/octeon_mgmt.c b/drivers

[PATCH 01/16 linux-next] net: netcp: constify of_device_id array

2015-03-17 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick --- drivers/net/ethernet/ti/netcp_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/ti/netcp_core.c b/drivers/net/ethernet

[PATCH 08/16 linux-next] net: greth: constify of_device_id array

2015-03-17 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick --- drivers/net/ethernet/aeroflex/greth.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/aeroflex/greth.c b/drivers/net

[PATCH 06/16 linux-next] net: ethernet: apple: constify of_device_id array

2015-03-17 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick --- drivers/net/ethernet/apple/bmac.c | 2 +- drivers/net/ethernet/apple/mace.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net

Re: [PATCH 02/35 linux-next] regulator: constify of_device_id array

2015-03-17 Thread Fabian Frederick
> On 17 March 2015 at 18:19 Mark Brown wrote: > > > On Tue, Mar 17, 2015 at 06:10:12PM +0100, Fabian Frederick wrote: > > > Thanks Mark, I used a --cc-cmd script by Joe Perches with > > git send-email which uses --nom for cover-letter. > > > This limits rec

Re: [PATCH 02/35 linux-next] regulator: constify of_device_id array

2015-03-17 Thread Fabian Frederick
> On 17 March 2015 at 13:11 Mark Brown wrote: > > > On Mon, Mar 16, 2015 at 08:17:08PM +0100, Fabian Frederick wrote: > > of_device_id is always used as const. > > (See driver.of_match_table and open firmware functions) > > Applied, thanks.  Please remember to alwa

[PATCH 09/16 linux-next] net: xilinx: constify of_device_id array

2015-03-17 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick f...@skynet.be --- drivers/net/ethernet/xilinx/ll_temac_main.c | 2 +- drivers/net/ethernet/xilinx/xilinx_axienet_main.c | 2 +- drivers/net/ethernet/xilinx

[PATCH 15/16 linux-next] net: via-rhine: constify of_device_id array

2015-03-17 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick f...@skynet.be --- drivers/net/ethernet/via/via-rhine.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/via/via-rhine.c b

[PATCH 16/16 linux-next] via-velocity: constify of_device_id array

2015-03-17 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick f...@skynet.be --- drivers/net/ethernet/via/via-velocity.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/via/via-velocity.c b

Re: [PATCH 02/35 linux-next] regulator: constify of_device_id array

2015-03-17 Thread Fabian Frederick
On 17 March 2015 at 13:11 Mark Brown broo...@kernel.org wrote: On Mon, Mar 16, 2015 at 08:17:08PM +0100, Fabian Frederick wrote: of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Applied, thanks.  Please remember to always include people

[PATCH 13/16 linux-next] IBM-EMAC: constify of_device_id array

2015-03-17 Thread Fabian Frederick
of_device_id is always used as const. (See driver.of_match_table and open firmware functions) Signed-off-by: Fabian Frederick f...@skynet.be --- drivers/net/ethernet/ibm/emac/core.c | 2 +- drivers/net/ethernet/ibm/emac/mal.c | 2 +- drivers/net/ethernet/ibm/emac/rgmii.c | 2 +- drivers/net

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