[PATCH] net: hns: use put_device() if device_register fail

2018-03-09 Thread Arvind Yadav
if device_register() returned an error! Always use put_device() to give up the reference initialized. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/net/ethernet/hisilicon/hns/hnae.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/net/et

[PATCH v3] net: iucv: Free memory obtained by kzalloc

2018-03-01 Thread Arvind Yadav
Free memory by calling put_device(), if afiucv_iucv_init is not successful. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- changes in v2: Calling put_device() before kfree(). changes in v3: No need to call kfree(). So removed kfree(). net/iucv/af_

[PATCH v2] net: iucv: Free memory obtained by kzalloc

2018-02-28 Thread Arvind Yadav
Free memory, if afiucv_iucv_init is not successful. So calling put_device() before kfree(). This will decrement the last reference. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- changes in v2: Calling put_device() before kfree(). net/iucv/af_iucv.c | 5 -

Re: [PATCH] net: iucv: Free memory obtained by kzalloc

2018-02-28 Thread Arvind Yadav
On Wednesday 28 February 2018 05:26 PM, Cornelia Huck wrote: On Wed, 28 Feb 2018 17:14:55 +0530 Arvind Yadav <arvind.yadav...@gmail.com> wrote: On Wednesday 28 February 2018 04:00 PM, Cornelia Huck wrote: On Wed, 28 Feb 2018 15:24:16 +0530 Arvind Yadav <arvind.yadav...@gmail.c

Re: [PATCH] net: iucv: Free memory obtained by kzalloc

2018-02-28 Thread Arvind Yadav
On Wednesday 28 February 2018 04:00 PM, Cornelia Huck wrote: On Wed, 28 Feb 2018 15:24:16 +0530 Arvind Yadav <arvind.yadav...@gmail.com> wrote: Free memory, if afiucv_iucv_init is not successful and removing a IUCV driver. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com&g

[PATCH] net: iucv: Free memory obtained by kzalloc

2018-02-28 Thread Arvind Yadav
Free memory, if afiucv_iucv_init is not successful and removing a IUCV driver. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- net/iucv/af_iucv.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/net/iucv/af_iucv.c b/net/iucv/af_iucv.c index 1e8cc7b..e

[PATCH] isdn: avm: Handle return value of skb_dequeue()

2017-12-16 Thread Arvind Yadav
skb_dequeue() will return NULL for an empty list or a pointer to the head element. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/isdn/hardware/avm/b1dma.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/isdn/hardware/avm/b1dma.c b/drivers/isdn/hardwa

[PATCH] NFC: Handle return value of skb_dequeue()

2017-12-16 Thread Arvind Yadav
skb_dequeue() will return NULL for an empty list or a pointer to the head element. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- net/nfc/rawsock.c | 4 1 file changed, 4 insertions(+) diff --git a/net/nfc/rawsock.c b/net/nfc/rawsock.c index e2188de..c6de1ac 100644 ---

Re: [PATCH 0/7 v2] net: Fix platform_get_irq's error checking

2017-12-05 Thread Arvind Yadav
Hi Sergei, On Tuesday 05 December 2017 03:27 PM, Sergei Shtylyov wrote: On 12/5/2017 12:54 PM, Sergei Shtylyov wrote: The platform_get_irq() function returns negative number if an error occurs, Zero if No irq is found and positive number if irq gets successful. No, returning 0 is not

Re: [PATCH 4/7 v2] net: ethernet: i825xx: Fix platform_get_irq's error checking

2017-12-04 Thread Arvind Yadav
Hi David, On Monday 04 December 2017 11:55 PM, David Miller wrote: From: Arvind Yadav <arvind.yadav...@gmail.com> Date: Mon, 4 Dec 2017 23:18:20 +0530 @@ -120,9 +120,10 @@ static int sni_82596_probe(struct platform_device *dev) netdevice->dev_addr[5] = readb(eth_ad

[PATCH 3/7 v2] can: xilinx: Fix platform_get_irq's error checking

2017-12-04 Thread Arvind Yadav
platform_get_irq() can fail here and we must check its return value. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- changes in v2: commit message was not correct. drivers/net/can/xilinx_can.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/n

[PATCH 0/7 v2] net: Fix platform_get_irq's error checking

2017-12-04 Thread Arvind Yadav
The platform_get_irq() function returns negative number if an error occurs, Zero if No irq is found and positive number if irq gets successful. platform_get_irq() error checking for only zero is not correct. Removed Other 3 patch which is not related to this series. Arvind Yadav (7): [PATCH 1

[PATCH 2/7 v2] net: ezchip: nps_enet: Fix platform_get_irq's error checking

2017-12-04 Thread Arvind Yadav
The platform_get_irq() function returns negative number if an error occurs, Zero if No irq is found and positive number if irq gets successful. platform_get_irq() error checking only for zero is not correct. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- changes

[PATCH 4/7 v2] net: ethernet: i825xx: Fix platform_get_irq's error checking

2017-12-04 Thread Arvind Yadav
The platform_get_irq() function returns negative number if an error occurs, Zero if No irq is found and positive number if irq gets successful. platform_get_irq() error checking only for zero is not correct. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- changes

[PATCH 1/7 v2] net: bcmgenet: Fix platform_get_irq's error checking

2017-12-04 Thread Arvind Yadav
The platform_get_irq() function returns negative number if an error occurs, Zero if No irq is found and positive number if irq gets successful. platform_get_irq() error checking only for zero is not correct. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- changes

[PATCH 6/7 v2] net: ethernet: smsc: Fix platform_get_irq's error checking

2017-12-04 Thread Arvind Yadav
platform_get_irq() can fail here and we must check its return value. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- changes in v2: commit message was not correct. drivers/net/ethernet/smsc/smc911x.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drive

[PATCH 5/7 v2] net: ethernet: natsemi: Fix platform_get_irq's error checking

2017-12-04 Thread Arvind Yadav
platform_get_irq() can fail here and we must check its return value. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- changes in v2: commit message was not correct. drivers/net/ethernet/natsemi/jazzsonic.c | 5 + 1 file changed, 5 insertions(+) diff --git a/d

[PATCH 7/7 v2] net: fjes: Fix platform_get_irq's error checking

2017-12-04 Thread Arvind Yadav
platform_get_irq() can fail here and we must check its return value. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- changes in v2: commit message was not correct. drivers/net/fjes/fjes_main.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/ne

[PATCH 00/10] Handle return value of platform_get_*

2017-12-02 Thread Arvind Yadav
() Arvind Yadav (10): [PATCH 01/10] net: bcmgenet: Fix platform_get_irq's error checking [PATCH 02/10] net: bcmgenet: free netdev on of_match_node() error [PATCH 03/10] net: ezchip: nps_enet: Fix platform_get_irq's error checking [PATCH 04/10] can: xilinx: Handle return value

[PATCH 01/10] net: bcmgenet: Fix platform_get_irq's error checking

2017-12-02 Thread Arvind Yadav
The platform_get_irq() function returns negative if an error occurs. zero or positive number on success. platform_get_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/net/ethernet/broadcom/genet/bcmgenet.c | 2 +- 1 file chan

[PATCH 02/10] net: bcmgenet: free netdev on of_match_node() error

2017-12-02 Thread Arvind Yadav
The change is to call free_netdev(), If of_match_node() will fail. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/net/ethernet/broadcom/genet/bcmgenet.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/net/ethernet/broadcom/genet/bcmg

[PATCH 03/10] net: ezchip: nps_enet: Fix platform_get_irq's error checking

2017-12-02 Thread Arvind Yadav
The platform_get_irq() function returns negative if an error occurs. zero or positive number on success. platform_get_irq() error checking for zero is not correct. And remove unnecessary check for free_netdev(). Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/net/et

[PATCH 06/10] net: ethernet: natsemi: Handle return value of platform_get_irq

2017-12-02 Thread Arvind Yadav
platform_get_irq() can fail here and we must check its return value. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/net/ethernet/natsemi/jazzsonic.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/net/ethernet/natsemi/jazzsonic.c b/drivers/net/et

[PATCH 05/10] net: ethernet: i825xx: Fix platform_get_irq's error checking

2017-12-02 Thread Arvind Yadav
The platform_get_irq() function returns negative if an error occurs. zero or positive number on success. platform_get_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/net/ethernet/i825xx/sni_82596.c | 3 ++- 1 file chan

[PATCH 04/10] can: xilinx: Handle return value of platform_get_irq

2017-12-02 Thread Arvind Yadav
platform_get_irq() can fail here and we must check its return value. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/net/can/xilinx_can.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/net/can/xilinx_can.c b/drivers/net/can/xilinx_can.c index 89aec07..e

[PATCH 08/10] net: fjes: Handle return value of platform_get_irq and platform_get_resource

2017-12-02 Thread Arvind Yadav
platform_get_irq() and platform_get_resource() can fail here and we must check its return value. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/net/fjes/fjes_main.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/drivers/net/fjes/fjes_main.c b/drive

[PATCH 10/10] net: ethernet: cpmac: Handle return value of platform_get_irq_byname

2017-12-02 Thread Arvind Yadav
platform_get_irq_byname() can fail here and we must check its return value Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/net/ethernet/ti/cpmac.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/net/ethernet/ti/cpmac.c b/drivers/net/ethernet/ti/cpmac.c

[PATCH 07/10] net: ethernet: smsc: Handle return value of platform_get_irq

2017-12-02 Thread Arvind Yadav
platform_get_irq() can fail here and we must check its return value. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/net/ethernet/smsc/smc911x.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/net/ethernet/smsc/smc911x.c b/drivers/net/ethernet/smsc/smc

[PATCH 09/10] net: ethernet: korina: Handle return value of platform_get_irq_byname

2017-12-02 Thread Arvind Yadav
platform_get_irq_byname() can fail here and we must check its return value. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/net/ethernet/korina.c | 9 + 1 file changed, 9 insertions(+) diff --git a/drivers/net/ethernet/korina.c b/drivers/net/ethernet/korina.c

[PATCH 0/3] Handle return value of platform_get_irq

2017-11-30 Thread Arvind Yadav
The platform_get_irq() function returns negative if an error occurs. zero or positive number on success. platform_get_irq() error checking for zero is not correct. Remove unnecessary 'err' initialization for irda driver. Arvind Yadav (3): [PATCH 1/3] iio: trigger: Fix platform_get_irq's error

[PATCH 1/3] iio: trigger: Fix platform_get_irq's error checking

2017-11-30 Thread Arvind Yadav
The platform_get_irq() function returns negative if an error occurs. zero or positive number on success. platform_get_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/staging/iio/trigger/iio-trig-bfin-timer.c | 4 ++--

[PATCH 3/3] staging: irda: Remove unnecessary 'err' initialization.

2017-11-30 Thread Arvind Yadav
Here, variable 'err' is already initialised. So no need to reinitialize. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/staging/irda/drivers/pxaficp_ir.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/irda/drivers/pxaficp_ir.c b/drivers/stagin

[PATCH 2/3] staging: irda: Handle return value of platform_get_irq

2017-11-30 Thread Arvind Yadav
platform_get_irq() can fail here and we must check its return value. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/staging/irda/drivers/pxaficp_ir.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/drivers/staging/irda/drivers/pxaficp_ir.c b/drivers/s

[PATCH 08/12] isdn: hisax: Fix pnp_irq's error checking for setup_isurf

2017-11-15 Thread Arvind Yadav
The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/isdn/hisax/isurf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/hisax/isurf.c b/d

[PATCH 07/12] isdn: hisax: Handle return value of pnp_irq and pnp_port_start

2017-11-15 Thread Arvind Yadav
pnp_irq() and pnp_port_start() can fail here and we must check its return value. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/isdn/hisax/hisax_fcpcipnp.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/isdn/hisax/hisax_fcpcipnp.c b/drivers/isdn

[PATCH 12/12] isdn: hisax: Fix pnp_irq's error checking for setup_teles3

2017-11-15 Thread Arvind Yadav
The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/isdn/hisax/teles3.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/hisax/tele

[PATCH 05/12] isdn: hisax: Fix pnp_irq's error checking for setup_hfcsx

2017-11-15 Thread Arvind Yadav
The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/isdn/hisax/hfc_sx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/hisax/hfc_

[PATCH 09/12] isdn: hisax: Fix pnp_irq's error checking for setup_ix1micro

2017-11-15 Thread Arvind Yadav
The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/isdn/hisax/ix1_micro.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/hisax/ix1_mic

[PATCH 04/12] isdn: hisax: Fix pnp_irq's error checking for setup_elsa_isapnp

2017-11-15 Thread Arvind Yadav
The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/isdn/hisax/elsa.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/hisax/elsa.c b/d

[PATCH 11/12] isdn: hisax: Fix pnp_irq's error checking for setup_sedlbauer_isapnp

2017-11-15 Thread Arvind Yadav
The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/isdn/hisax/sedlbauer.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/hisax/sedlbau

[PATCH 10/12] isdn: hisax: Fix pnp_irq's error checking for setup_niccy

2017-11-15 Thread Arvind Yadav
The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/isdn/hisax/niccy.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/hisax/niccy.c b/d

[PATCH 06/12] isdn: hisax: Fix pnp_irq's error checking for setup_hfcs

2017-11-15 Thread Arvind Yadav
The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/isdn/hisax/hfcscard.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/hisax/hfcsca

[PATCH 01/12] isdn: hisax: Fix pnp_irq's error checking for setup_asuscom

2017-11-15 Thread Arvind Yadav
The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/isdn/hisax/asuscom.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/hisax/asusc

[PATCH 03/12] isdn: hisax: Fix pnp_irq's error checking for setup_diva_isapnp

2017-11-15 Thread Arvind Yadav
The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/isdn/hisax/diva.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/hisax/diva.c b/d

[PATCH 00/12] isdn: hisax: Fix pnp_irq's error checking

2017-11-15 Thread Arvind Yadav
The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Arvind Yadav (12): [PATCH 01/12] isdn: hisax: Fix pnp_irq's error checking for setup_asuscom [PATCH 02/12] isdn: hisax: Fix pnp_irq's error checking for avm_pnp_setup [PATCH 03/12] isdn

[PATCH 02/12] isdn: hisax: Fix pnp_irq's error checking for avm_pnp_setup

2017-11-15 Thread Arvind Yadav
The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/isdn/hisax/avm_pci.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/hisax/avm_p

[PATCH] atm: horizon: Fix irq release error

2017-11-14 Thread Arvind Yadav
atm_dev_register() can fail here and passed parameters to free irq which is not initialised. Initialization of 'dev->irq' happened after the 'goto out_free_irq'. So using 'irq' insted of 'dev->irq' in free_irq(). Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/a

[PATCH] atm: iphase: Fix space before '[' error.

2017-10-30 Thread Arvind Yadav
Fix checkpatch.pl error: ERROR: space prohibited before open square bracket '['. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/atm/iphase.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/atm/iphase.c b/drivers/atm/iphase.c index fc

[RFT] lan78xx: FIX use-after-free in lan78xx_write_reg

2017-09-27 Thread Arvind Yadav
-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/net/usb/lan78xx.c | 14 +- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/drivers/net/usb/lan78xx.c b/drivers/net/usb/lan78xx.c index b99a7fb..e653982 100644 --- a/drivers/net/usb/lan78xx.c +++ b/drivers/n

[PATCH] netfilter: nf_tables: Release memory obtained by kasprintf

2017-09-20 Thread Arvind Yadav
Free memory region, if nf_tables_set_alloc_name is not successful. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- net/netfilter/nf_tables_api.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/net/netfilter/nf_tables_api.c b/net/netfilter/nf_tables_api.c

[PATCH] RDS: constify rhashtable_params

2017-08-30 Thread Arvind Yadav
rhashtable_params are not supposed to change at runtime. All Functions rhashtable_* working with const rhashtable_params provided by . So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- net/rds/bind.c | 2 +- 1 file changed, 1 insertion

[PATCH] netfilter: nat: constify rhashtable_params

2017-08-30 Thread Arvind Yadav
rhashtable_params are not supposed to change at runtime. All Functions rhashtable_* working with const rhashtable_params provided by . So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- net/netfilter/nf_nat_core.c | 2 +- 1 file chan

[PATCH] net: stmmac: constify clk_div_table

2017-08-27 Thread Arvind Yadav
clk_div_table are not supposed to change at runtime. meson8b_dwmac structure is working with const clk_div_table. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/net/ethernet/stmicro/stmmac/dwmac-meson8b.c | 2 +- 1 file chan

[PATCH] netfilter: ipv6: nf_defrag: constify nf_hook_ops

2017-08-26 Thread Arvind Yadav
nf_hook_ops are not supposed to change at runtime. nf_register_net_hooks and nf_unregister_net_hooks are working with const nf_hook_ops. So mark the non-const nf_hook_ops structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- net/ipv6/netfilter/nf_defrag_ipv6_hooks

[PATCH] netfilter: ipv4: nf_defrag: constify nf_hook_ops

2017-08-26 Thread Arvind Yadav
nf_hook_ops are not supposed to change at runtime. nf_register_net_hooks and nf_unregister_net_hooks are working with const nf_hook_ops. So mark the non-const nf_hook_ops structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- net/ipv4/netfilter/nf_defrag_ipv4.c | 2

[PATCH 3/3 v3] net: tipc: constify genl_ops

2017-08-23 Thread Arvind Yadav
genl_ops are not supposed to change at runtime. All functions working with genl_ops provided by work with const genl_ops. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- Changes in v2: Patch Subject was wrong. changes

Re: [PATCH 3/3 v2] net: netlink: constify genl_ops

2017-08-23 Thread Arvind Yadav
Hi Kalle, On Wednesday 23 August 2017 03:50 PM, Kalle Valo wrote: Arvind Yadav <arvind.yadav...@gmail.com> writes: genl_ops are not supposed to change at runtime. All functions working with genl_ops provided by work with const genl_ops. So mark the non-const structs as const. Sign

[PATCH 0/3] constify dsa_switch_ops

2017-08-23 Thread Arvind Yadav
dsa_switch_ops are not supposed to change at runtime. All functions working with dsa_switch_ops provided by work with const dsa_switch_ops. So mark the non-const structs as const. Arvind Yadav (3): [PATCH 1/3] net: dsa: loop: constify dsa_switch_ops [PATCH 2/3] net: dsa: lan9303: constify

[PATCH 2/3] net: dsa: lan9303: constify dsa_switch_ops

2017-08-23 Thread Arvind Yadav
dsa_switch_ops are not supposed to change at runtime. All functions working with dsa_switch_ops provided by work with const dsa_switch_ops. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/net/dsa/lan9303-core.c | 2 +- 1 file c

[PATCH 1/3] net: dsa: loop: constify dsa_switch_ops

2017-08-23 Thread Arvind Yadav
dsa_switch_ops are not supposed to change at runtime. All functions working with dsa_switch_ops provided by work with const dsa_switch_ops. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/net/dsa/dsa_loop.c | 2 +- 1 file chan

[PATCH 3/3] net: dsa: mt7530: constify dsa_switch_ops

2017-08-23 Thread Arvind Yadav
dsa_switch_ops are not supposed to change at runtime. All functions working with dsa_switch_ops provided by work with const dsa_switch_ops. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/net/dsa/mt7530.c | 2 +- 1 file chan

[PATCH 3/3 v2] net: netlink: constify genl_ops

2017-08-23 Thread Arvind Yadav
genl_ops are not supposed to change at runtime. All functions working with genl_ops provided by work with const genl_ops. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- Changes in v2: Patch Subject was wrong. ne

Re: [PATCH 3/3] net: rsi: netlink: constify genl_ops

2017-08-23 Thread Arvind Yadav
Hi, On Wednesday 23 August 2017 03:32 PM, Kalle Valo wrote: Arvind Yadav <arvind.yadav...@gmail.com> writes: genl_ops are not supposed to change at runtime. All functions working with genl_ops provided by work with const genl_ops. So mark the non-const structs as const. Sign

[PATCH 1/3] net: rtlwifi: constify rate_control_ops

2017-08-23 Thread Arvind Yadav
rate_control_ops are not supposed to change at runtime. All functions working with rate_control_ops provided by work with const rate_control_ops. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/net/wireless/realtek/rtlwifi/rc

[PATCH 0/3] constify *_ops

2017-08-23 Thread Arvind Yadav
*_ops are not supposed to change at runtime and none of the structures is modified. Arvind Yadav (3): [PATCH 1/3] net: rtlwifi: constify rate_control_ops [PATCH 2/3] net: rsi: mac80211: constify ieee80211_ops [PATCH 3/3] net: rsi: netlink: constify genl_ops drivers/net/wireless/realtek

[PATCH 3/3] net: rsi: netlink: constify genl_ops

2017-08-23 Thread Arvind Yadav
genl_ops are not supposed to change at runtime. All functions working with genl_ops provided by work with const genl_ops. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- net/tipc/netlink_compat.c | 2 +- 1 file changed, 1 insertion

[PATCH 2/3] net: rsi: mac80211: constify ieee80211_ops

2017-08-23 Thread Arvind Yadav
ieee80211_ops are not supposed to change at runtime. All functions working with ieee80211_ops provided by work with const ieee80211_ops. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/net/wireless/rsi/rsi_91x_mac80211.c | 2 +-

[PATCH] net: amd: constify zorro_device_id

2017-08-22 Thread Arvind Yadav
zorro_device_id are not supposed to change at runtime. All functions working with zorro_device_id provided by work with const zorro_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/net/ethernet/amd/a2065.c | 2 +- d

[PATCH 5/5] net: defxx: constify eisa_device_id

2017-08-19 Thread Arvind Yadav
eisa_device_id are not supposed to change at runtime. All functions working with eisa_device_id provided by work with const eisa_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/net/fddi/defxx.c | 2 +- 1 file chan

[PATCH 4/5] net: hp100: constify eisa_device_id

2017-08-19 Thread Arvind Yadav
eisa_device_id are not supposed to change at runtime. All functions working with eisa_device_id provided by work with const eisa_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/net/ethernet/hp/hp100.c | 2 +- 1 file c

[PATCH 3/5] net: de4x5: constify eisa_device_id

2017-08-19 Thread Arvind Yadav
eisa_device_id are not supposed to change at runtime. All functions working with eisa_device_id provided by work with const eisa_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/net/ethernet/dec/tulip/de4x5.c | 2 +-

[PATCH 0/5] constify net eisa_device_id

2017-08-19 Thread Arvind Yadav
eisa_device_id are not supposed to change at runtime. All functions working with eisa_device_id provided by work with const eisa_device_id. So mark the non-const structs as const. Arvind Yadav (5): [PATCH 1/5] net: 3c509: constify eisa_device_id [PATCH 2/5] net: 3c59x: constify

[PATCH 1/5] net: 3c509: constify eisa_device_id

2017-08-19 Thread Arvind Yadav
eisa_device_id are not supposed to change at runtime. All functions working with eisa_device_id provided by work with const eisa_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/net/ethernet/3com/3c509.c | 2 +-

[PATCH 2/5] net: 3c59x: constify eisa_device_id

2017-08-19 Thread Arvind Yadav
eisa_device_id are not supposed to change at runtime. All functions working with eisa_device_id provided by work with const eisa_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/net/ethernet/3com/3c59x.c | 2 +-

[PATCH] net: ibm: ibmveth: constify vio_device_id

2017-08-17 Thread Arvind Yadav
vio_device_id are not supposed to change at runtime. All functions working with vio_device_id provided by work with const vio_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/net/ethernet/ibm/ibmveth.c | 2 +- 1 file c

[PATCH] net: ibm: ibmvnic: constify vio_device_id

2017-08-17 Thread Arvind Yadav
vio_device_id are not supposed to change at runtime. All functions working with vio_device_id provided by work with const vio_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/net/ethernet/ibm/ibmvnic.c | 2 +- 1 file c

[PATCH] net: 3c509: constify pnp_device_id

2017-08-15 Thread Arvind Yadav
pnp_device_id are not supposed to change at runtime. All functions working with pnp_device_id provided by work with const pnp_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/net/ethernet/3com/3c509.c | 2 +- 1 file c

[PATCH 4/4] net: ti: cpsw:: constify platform_device_id

2017-08-13 Thread Arvind Yadav
platform_device_id are not supposed to change at runtime. All functions working with platform_device_id provided by work with const platform_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/net/ethernet/ti/cpsw.c | 2

[PATCH 0/4] constify net platform_device_id

2017-08-13 Thread Arvind Yadav
platform_device_id are not supposed to change at runtime. All functions working with platform_device_id provided by work with const platform_device_id. So mark the non-const structs as const. Arvind Yadav (4): [PATCH 1/4] can: constify platform_device_id [PATCH 2/4] net: dpaa_eth: constify

[PATCH 2/4] net: dpaa_eth: constify platform_device_id

2017-08-13 Thread Arvind Yadav
platform_device_id are not supposed to change at runtime. All functions working with platform_device_id provided by work with const platform_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/net/ethernet/freescal

[PATCH 3/4] net: sh_eth: constify platform_device_id

2017-08-13 Thread Arvind Yadav
platform_device_id are not supposed to change at runtime. All functions working with platform_device_id provided by work with const platform_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/net/ethernet/renesas/sh_eth

[PATCH 1/4] can: constify platform_device_id

2017-08-13 Thread Arvind Yadav
platform_device_id are not supposed to change at runtime. All functions working with platform_device_id provided by work with const platform_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/net/can/c_can/c_can_platform

[PATCH v3 06/20] brcm80211: constify usb_device_id

2017-08-10 Thread Arvind Yadav
usb_device_id are not supposed to change at runtime. All functions working with usb_device_id provided by work with const usb_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- changes in v2: Re-submitting wi

[PATCH v2 15/20] rt73usb: constify usb_device_id

2017-08-09 Thread Arvind Yadav
usb_device_id are not supposed to change at runtime. All functions working with usb_device_id provided by work with const usb_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- changes in v2: Re-submitting wi

[PATCH v2 17/20] rtl8xxxu: constify usb_device_id

2017-08-09 Thread Arvind Yadav
usb_device_id are not supposed to change at runtime. All functions working with usb_device_id provided by work with const usb_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- changes in v2: Re-submitting wi

[PATCH v2 20/20] zd1211rw: constify usb_device_id

2017-08-09 Thread Arvind Yadav
usb_device_id are not supposed to change at runtime. All functions working with usb_device_id provided by work with const usb_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- changes in v2: Re-submitting wi

[PATCH v2 19/20] zd1201: constify usb_device_id

2017-08-09 Thread Arvind Yadav
usb_device_id are not supposed to change at runtime. All functions working with usb_device_id provided by work with const usb_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- changes in v2: Re-submitting wi

[PATCH v2 18/20] rtl8192cu: constify usb_device_id

2017-08-09 Thread Arvind Yadav
usb_device_id are not supposed to change at runtime. All functions working with usb_device_id provided by work with const usb_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- changes in v2: Re-submitting wi

[PATCH v2 16/20] rtl8187: constify usb_device_id

2017-08-09 Thread Arvind Yadav
usb_device_id are not supposed to change at runtime. All functions working with usb_device_id provided by work with const usb_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- changes in v2: Re-submitting wi

[PATCH v2 13/20] rt2500usb: constify usb_device_id

2017-08-09 Thread Arvind Yadav
usb_device_id are not supposed to change at runtime. All functions working with usb_device_id provided by work with const usb_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- changes in v2: Re-submitting wi

[PATCH v2 14/20] rt2800usb: constify usb_device_id

2017-08-09 Thread Arvind Yadav
usb_device_id are not supposed to change at runtime. All functions working with usb_device_id provided by work with const usb_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- changes in v2: Re-submitting wi

[PATCH v2 10/20] libertas_tf: constify usb_device_id

2017-08-09 Thread Arvind Yadav
usb_device_id are not supposed to change at runtime. All functions working with usb_device_id provided by work with const usb_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- changes in v2: Re-submitting wi

[PATCH v2 12/20] mt7601u: constify usb_device_id

2017-08-09 Thread Arvind Yadav
usb_device_id are not supposed to change at runtime. All functions working with usb_device_id provided by work with const usb_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- changes in v2: Re-submitting wi

[PATCH v2 11/20] mwifiex: constify usb_device_id

2017-08-09 Thread Arvind Yadav
usb_device_id are not supposed to change at runtime. All functions working with usb_device_id provided by work with const usb_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- changes in v2: Re-submitting wi

[PATCH v2 09/20] libertas: constify usb_device_id

2017-08-09 Thread Arvind Yadav
usb_device_id are not supposed to change at runtime. All functions working with usb_device_id provided by work with const usb_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- changes in v2: Re-submitting wi

[PATCH v2 08/20] p54: constify usb_device_id

2017-08-09 Thread Arvind Yadav
usb_device_id are not supposed to change at runtime. All functions working with usb_device_id provided by work with const usb_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- changes in v2: Re-submitting wi

[PATCH v2 00/20] constify wireless usb_device_id

2017-08-09 Thread Arvind Yadav
usb_device_id are not supposed to change at runtime. All functions working with usb_device_id provided by work with const usb_device_id. So mark the non-const structs as const. Arvind Yadav (20): [PATCH v2 01/20] ar5523: constify usb_device_id [PATCH v2 02/20] ath6kl: constify usb_device_id

[PATCH v2 07/20] orinoco: constify usb_device_id

2017-08-09 Thread Arvind Yadav
usb_device_id are not supposed to change at runtime. All functions working with usb_device_id provided by work with const usb_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- changes in v2: Re-submitting wi

[PATCH v2 02/20] ath6kl: constify usb_device_id

2017-08-09 Thread Arvind Yadav
usb_device_id are not supposed to change at runtime. All functions working with usb_device_id provided by work with const usb_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- changes in v2: Re-submitting wi

[PATCH v2 06/20] brcm80211: constify usb_device_id

2017-08-09 Thread Arvind Yadav
usb_device_id are not supposed to change at runtime. All functions working with usb_device_id provided by work with const usb_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- changes in v2: Re-submitting wi

  1   2   3   >