No need to store netlbl_unlabel_genl_init result and test it before returning.
Signed-off-by: Fabian Frederick f...@skynet.be
---
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
a...@linux-foundation.org
Signed-off-by: Fabian Frederick f...@skynet.be
---
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
Testing err before returning value is not needed.
Signed-off-by: Fabian Frederick f...@skynet.be
---
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
> 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
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
> 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
>
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
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
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
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
Use current logging functions and add module name prefix.
Signed-off-by: Fabian Frederick f...@skynet.be
---
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
static values are automatically initialized to NULL
Signed-off-by: Fabian Frederick f...@skynet.be
---
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
s/state/blocked
Signed-off-by: Fabian Frederick f...@skynet.be
---
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
no secid argument in netlbl_cfg_unlbl_static_del
Signed-off-by: Fabian Frederick f...@skynet.be
---
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
On 07 October 2014 at 22:33 Guenter Roeck li...@roeck-us.net wrote:
On Tue, Oct 07, 2014 at 04:18:32PM -0400, David Miller wrote:
From: Fabian Frederick f...@skynet.be
Date: Tue, 7 Oct 2014 22:16:36 +0200
static values are automatically initialized to NULL
Signed-off
static values are automatically initialized to 0
Signed-off-by: Fabian Frederick f...@skynet.be
---
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
On 07 October 2014 at 22:54 Hannes Frederic Sowa han...@stressinduktion.org
wrote:
On Di, 2014-10-07 at 22:49 +0200, Fabian Frederick wrote:
On 07 October 2014 at 22:33 Guenter Roeck li...@roeck-us.net wrote:
On Tue, Oct 07, 2014 at 04:18:32PM -0400, David Miller wrote
> 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).
>
> Not the w
> 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
: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
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
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
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
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
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
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
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
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
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 deprecated IRQF_DISABLED
See include/linux/interrupt.h:
Signed-off-by: Fabian Frederick f...@skynet.be
---
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
See include/linux/interrupt.h:
This flag is a NOOP and scheduled to be removed
Signed-off-by: Fabian Frederick f...@skynet.be
---
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 531ae59
See include/linux/interrupt.h:
This flag is a NOOP and scheduled to be removed
Signed-off-by: Fabian Frederick f...@skynet.be
---
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
index
See include/linux/interrupt.h:
This flag is a NOOP and scheduled to be removed
Signed-off-by: Fabian Frederick f...@skynet.be
---
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
See include/linux/interrupt.h:
This flag is a NOOP and scheduled to be removed
Signed-off-by: Fabian Frederick f...@skynet.be
---
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
See include/linux/interrupt.h:
This flag is a NOOP and scheduled to be removed
Signed-off-by: Fabian Frederick f...@skynet.be
---
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
See include/linux/interrupt.h:
This flag is a NOOP and scheduled to be removed
Signed-off-by: Fabian Frederick f...@skynet.be
---
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
expression
(different address spaces)
net/phonet/pn_dev.c:391:17: error: incompatible types in comparison expression
(different address spaces)
Suggested-by: Eric Dumazet eduma...@google.com
Signed-off-by: Fabian Frederick f...@skynet.be
---
V2: use rcu_access_pointer instead of rcu_dereference out
On 06 October 2014 at 19:40 Borislav Petkov b...@alien8.de 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 flag is a NOOP and scheduled to be removed
Note
On 06 October 2014 at 20:46 Borislav Petkov b...@alien8.de 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).
Not the whole patchset - I
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
Use container_of instead of casting first structure member.
Signed-off-by: Fabian Frederick f...@skynet.be
---
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
Use container_of instead of casting first structure member.
Signed-off-by: Fabian Frederick f...@skynet.be
---
drivers/tty/serial/sa1100.c | 45 ++---
1 file changed, 30 insertions(+), 15 deletions(-)
diff --git a/drivers/tty/serial/sa1100.c b/drivers/tty
Use container_of instead of casting first structure member.
Signed-off-by: Fabian Frederick f...@skynet.be
---
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
Use container_of instead of casting first structure member.
Signed-off-by: Fabian Frederick f...@skynet.be
---
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
Use container_of instead of casting first structure member.
Signed-off-by: Fabian Frederick f...@skynet.be
---
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
Use container_of instead of casting first structure member.
Signed-off-by: Fabian Frederick f...@skynet.be
---
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
Use container_of instead of casting first structure member.
Signed-off-by: Fabian Frederick f...@skynet.be
---
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
Use container_of instead of casting first structure member.
Signed-off-by: Fabian Frederick f...@skynet.be
---
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
Use container_of instead of casting first structure member.
Signed-off-by: Fabian Frederick f...@skynet.be
---
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
Use container_of instead of casting first structure member.
Signed-off-by: Fabian Frederick f...@skynet.be
---
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
Use container_of instead of casting first structure member.
Signed-off-by: Fabian Frederick f...@skynet.be
---
drivers/tty/serial/pnx8xxx_uart.c | 48 ++-
1 file changed, 32 insertions(+), 16 deletions(-)
diff --git a/drivers/tty/serial/pnx8xxx_uart.c
b
Use container_of instead of casting first structure member.
Signed-off-by: Fabian Frederick f...@skynet.be
---
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
Use container_of instead of casting first structure member.
Signed-off-by: Fabian Frederick f...@skynet.be
---
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
: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
expression
(different address spaces)
net/phonet/pn_dev.c:391:17: error: incompatible types in comparison expression
(different address spaces)
Signed-off-by: Fabian Frederick f...@skynet.be
---
net/phonet/pn_dev.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/net/phonet
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
use macro definition
Signed-off-by: Fabian Frederick f...@skynet.be
---
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
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
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
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
tcp_init_mem is only called by __init tcp_init.
Signed-off-by: Fabian Frederick f...@skynet.be
---
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
ip4_frags_ctl_register is only called by __init ipfrag_init
Signed-off-by: Fabian Frederick f...@skynet.be
---
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
cipso_v4_cache_init is only called by __init cipso_v4_init
Signed-off-by: Fabian Frederick f...@skynet.be
---
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
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
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
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
> 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).
> >
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
irlan_open is only called by __init irlan_init in same module.
Signed-off-by: Fabian Frederick
---
net/irda/irlan/irlan_common.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/net/irda/irlan/irlan_common.c b/net/irda/irlan/irlan_common.c
index 1bc49ed..5a2d0a6 100644
ceph_release_page_vector was defined twice in libceph.h
Signed-off-by: Fabian Frederick
---
include/linux/ceph/libceph.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/include/linux/ceph/libceph.h b/include/linux/ceph/libceph.h
index 279b0af..07bc359 100644
--- a/include/linux/ceph
ceph_release_page_vector was defined twice in libceph.h
Signed-off-by: Fabian Frederick f...@skynet.be
---
include/linux/ceph/libceph.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/include/linux/ceph/libceph.h b/include/linux/ceph/libceph.h
index 279b0af..07bc359 100644
--- a/include/linux
irlan_open is only called by __init irlan_init in same module.
Signed-off-by: Fabian Frederick f...@skynet.be
---
net/irda/irlan/irlan_common.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/net/irda/irlan/irlan_common.c b/net/irda/irlan/irlan_common.c
index 1bc49ed
lowpan_frags_sysctl_register is only called by __init lowpan_net_frag_init
(part of the lowpan module).
Signed-off-by: Fabian Frederick f...@skynet.be
---
This is untested.
net/ieee802154/reassembly.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/net/ieee802154
On 01 October 2014 at 02:25 Alexander Aring alex.ar...@gmail.com 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).
Signed-off-by: Fabian Frederick f
dccp_mib_init is only called by __init dccp_init in same module.
Signed-off-by: Fabian Frederick f...@skynet.be
---
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
ccid_activate is only called by __init ccid_initialize_builtins in same module.
Signed-off-by: Fabian Frederick f...@skynet.be
---
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
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 alex.ar...@gmail.com
Signed-off-by: Fabian Frederick f...@skynet.be
---
net/ieee802154/reassembly.c
key is set in affs_fill_super but never used.
Cc: Andrew Morton
Signed-off-by: Fabian Frederick
---
fs/affs/super.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/fs/affs/super.c b/fs/affs/super.c
index 51f1a95..a67dc72 100644
--- a/fs/affs/super.c
+++ b/fs/affs/super.c
head is set to AFFS_HEAD(bh) but never used.
Cc: Andrew Morton
Signed-off-by: Fabian Frederick
---
fs/affs/inode.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/fs/affs/inode.c b/fs/affs/inode.c
index bec2d1a..930aac3 100644
--- a/fs/affs/inode.c
+++ b/fs/affs/inode.c
@@ -20,7 +20,6
ucg is defined and set in ufs_bitmap_search but never used.
Cc: Evgeniy Dushistov
Cc: Andrew Morton
Signed-off-by: Fabian Frederick
---
fs/ufs/balloc.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/fs/ufs/balloc.c b/fs/ufs/balloc.c
index 7bc2080..2c10360 100644
--- a/fs/ufs/balloc.c
This patch defines maximum block number to 2^31.
It also converts bitmap_size and array_size to
unsigned int in omfs_get_imap
Suggested-By: Linus Torvalds
Suggested-By: Bob Copeland
Cc: Linus Torvalds
Cc: Bob Copeland
Cc: Andrew Morton
Signed-off-by: Fabian Frederick
---
This is untested
sys_tz is already declared extern struct in include/linux/time.h
Cc: Andrew Morton
Cc: OGAWA Hirofumi
Signed-off-by: Fabian Frederick
---
fs/fat/misc.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/fs/fat/misc.c b/fs/fat/misc.c
index 628e22a..d8da2d2 100644
--- a/fs/fat/misc.c
+++ b/fs
4 functions declared variables twice resulting in shadow warnings.
This patch renames internal variables and adds blank line after declarations.
Cc: Andrew Morton
Signed-off-by: Fabian Frederick
---
fs/affs/file.c | 39 ---
1 file changed, 24 insertions
> On 29 September 2014 at 07:40 Sudip Mukherjee
> wrote:
>
>
> On Sun, Sep 28, 2014 at 08:44:47PM +0200, Fabian Frederick wrote:
> > Adding parentheses around expression to avoid:
> > drivers/acpi/sbs.c:444:28: warning: dubious: !x & y
> >
On 29 September 2014 at 07:40 Sudip Mukherjee sudipm.mukher...@gmail.com
wrote:
On Sun, Sep 28, 2014 at 08:44:47PM +0200, Fabian Frederick wrote:
Adding parentheses around expression to avoid:
drivers/acpi/sbs.c:444:28: warning: dubious: !x y
Signed-off-by: Fabian Frederick f
4 functions declared variables twice resulting in shadow warnings.
This patch renames internal variables and adds blank line after declarations.
Cc: Andrew Morton a...@linux-foundation.org
Signed-off-by: Fabian Frederick f...@skynet.be
---
fs/affs/file.c | 39
1501 - 1600 of 3217 matches
Mail list logo